Skip to content

feat(p2p-04): add client p2p module shell (#671)#14

Open
jlon wants to merge 1 commit into
p2p-03-p2p-config-surfacefrom
p2p-04-client-p2p-module-shell
Open

feat(p2p-04): add client p2p module shell (#671)#14
jlon wants to merge 1 commit into
p2p-03-p2p-config-surfacefrom
p2p-04-client-p2p-module-shell

Conversation

@jlon

@jlon jlon commented Mar 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • Create the client p2p namespace and base discovery/transfer types.
  • Keep the module as compile-time scaffolding only, with no read-path or service behavior.

Design

  • Limit this PR to DTOs and snapshots that later P2P layers can reuse.
  • Make ChunkId file/version aware so later cache and fetch work does not need another key rewrite.

Implementation

  • Export curvine_client::p2p from the crate root.
  • Add discovery snapshots plus transfer request/response and chunk identity types.

Verification

  • cargo check -p curvine-client
  • cargo test -p curvine-client --lib

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@jlon jlon force-pushed the p2p-03-p2p-config-surface branch from b493d15 to ab77386 Compare March 26, 2026 10:25
@jlon jlon force-pushed the p2p-04-client-p2p-module-shell branch from 032f2e6 to e389ef7 Compare March 26, 2026 10:26
@jlon jlon force-pushed the p2p-03-p2p-config-surface branch from ab77386 to 9e4f0ba Compare March 27, 2026 09:32
@jlon jlon force-pushed the p2p-04-client-p2p-module-shell branch from e389ef7 to e8e08e2 Compare March 27, 2026 09:32
@jlon jlon force-pushed the p2p-03-p2p-config-surface branch from 9e4f0ba to 8fa482c Compare March 30, 2026 00:54
@jlon jlon force-pushed the p2p-04-client-p2p-module-shell branch from e8e08e2 to 848bc0d Compare March 30, 2026 00:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant