-
-
Notifications
You must be signed in to change notification settings - Fork 2
feat(p2p): 2026Q1 commitments #353
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
b90a319 to
c57cf96
Compare
c57cf96 to
ccdee37
Compare
fbarbu15
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, that's just some linting errors to fix.
See https://github.com/vacp2p/roadmap/actions/runs/20029982344/job/57436372524?pr=353
8efa1a2 to
03c6abc
Compare
03c6abc to
4b01125
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds 2026Q1 commitments for the p2p team, introducing seven new commitment documents and updating the roadmap to reflect the quarter shift from 2025Q4 to 2026Q1.
Key Changes
- Updates roadmap timeline from 2025Q4 to 2026Q1, reorganizing planned features
- Adds seven new 2026Q1 commitment documents covering WebTransport, connection management, GossipSub extensions, and other improvements
- Reorganizes roadmap sections to reflect current quarter planning
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
| content/p2p/preview.md | Updates roadmap from 2025Q4 to 2026Q1 timeframe and reorganizes feature priorities |
| content/p2p/index.md | Adds 2026Q1 section with links to seven new commitment documents |
| content/p2p/ift/2026q1-nimlibp2p-webtransport.md | Defines WebTransport support implementation tasks |
| content/p2p/ift/2026q1-nimlibp2p-universal-connectivity-app.md | Outlines universal connectivity app extensions |
| content/p2p/ift/2026q1-nimlibp2p-maintenance.md | Documents maintenance tasks and Kademlia interop tests |
| content/p2p/ift/2026q1-nimlibp2p-logos-core.md | Describes logos-core module development |
| content/p2p/ift/2026q1-nimlibp2p-gossipsub-extensions.md | Details GossipSub extensions and partial messages implementation |
| content/p2p/ift/2026q1-nimlibp2p-connection-management.md | Specifies connection management improvements |
| content/p2p/ift/2026q1-nimlibp2p-cbindings.md | Outlines c-bindings extensions including Mix functionality |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| ## Description | ||
|
|
||
| Extend the number of transports supported by nim-libp2p via the developent of Webtransport, |
Copilot
AI
Dec 16, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Corrected spelling of 'developent' to 'development'.
| Extend the number of transports supported by nim-libp2p via the developent of Webtransport, | |
| Extend the number of transports supported by nim-libp2p via the development of Webtransport, |
|
|
||
| ## Task List | ||
|
|
||
| ### Chat |
Copilot
AI
Dec 16, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The task heading 'Chat' doesn't match the task name 'kad-dht' in the fully qualified name below. The heading should be 'Kad-DHT' or similar to accurately describe the Kademlia DHT feature being implemented.
| ### Chat | |
| ### Kad-DHT |
| Continue exposing key nim-libp2p functionality via c-bindings to enables downstream projects like logoscore to embed libp2p functionality directly. | ||
|
|
||
| ## Description | ||
| Using nim-libp2p's c-bindings develop logo-core modules that would let libp2p features to be consumed from within Logos |
Copilot
AI
Dec 16, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inconsistent naming: 'logo-core' should be 'logos-core' to match the document title and fully qualified name.
| Using nim-libp2p's c-bindings develop logo-core modules that would let libp2p features to be consumed from within Logos | |
| Using nim-libp2p's c-bindings develop logos-core modules that would let libp2p features to be consumed from within Logos |
| * start-date: 2026/01/01 | ||
| * end-date: 2026/03/31 | ||
|
|
||
| #### description |
Copilot
AI
Dec 16, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Section headers should use title case for consistency. This should be 'Description' to match the capitalization used in other commitment documents.
| #### description | |
| #### Description |
| Using the c-bindings module from nim-libp2p, develop a logos module that would allow nim-libp2p | ||
| to be used in logos. | ||
|
|
||
| #### deliverables |
Copilot
AI
Dec 16, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Section headers should use title case for consistency. This should be 'Deliverables' to match the capitalization used in other commitment documents.
| #### deliverables | |
| #### Deliverables |
| - Test Extension implemented (optionally enabled via compile time flag) | ||
|
|
||
| ### Implement Gossipsub Partial Messages Extension | ||
| * fully qualified name: `vac:p2p:ift:2026q1-nimlibp2p-gossipsub-extensions:extensions` |
Copilot
AI
Dec 16, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The fully qualified name suffix 'extensions' is duplicated from the previous task. This should be 'partial-messages' to uniquely identify this task about the Partial Messages Extension.
| * fully qualified name: `vac:p2p:ift:2026q1-nimlibp2p-gossipsub-extensions:extensions` | |
| * fully qualified name: `vac:p2p:ift:2026q1-nimlibp2p-gossipsub-extensions:partial-messages` |
| Continue exposing key nim-libp2p functionality via c-bindings to enables downstream projects like logoscore to embed libp2p functionality directly. | ||
|
|
||
| ## Description | ||
| Continue the work started in `vac:p2p:ift:2026q1-nimlibp2p-cbindings` to continue exposing nim-libp2p functionality via a c static library, enabling external projects to integrate without depending on nim toolchains. Develop a logos core module that uses these bindings |
Copilot
AI
Dec 16, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The phrase 'continue' is used twice in this sentence making it redundant. Consider rephrasing to: 'Continue the work started in vac:p2p:ift:2026q1-nimlibp2p-cbindings by exposing nim-libp2p functionality...'.
| Continue the work started in `vac:p2p:ift:2026q1-nimlibp2p-cbindings` to continue exposing nim-libp2p functionality via a c static library, enabling external projects to integrate without depending on nim toolchains. Develop a logos core module that uses these bindings | |
| Continue the work started in `vac:p2p:ift:2026q1-nimlibp2p-cbindings` by exposing nim-libp2p functionality via a c static library, enabling external projects to integrate without depending on nim toolchains. Develop a logos core module that uses these bindings |
| * start-date: 2026/01/01 | ||
| * end-date: 2026/03/31 | ||
|
|
||
| #### description |
Copilot
AI
Dec 16, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Section headers should use title case for consistency. This should be 'Description' to match the capitalization used in other commitment documents.
| #### description | |
| #### Description |
| - Choose whether exit node is destination or not | ||
| This task depends on `vac:p2p:ift:2026q1-nimlibp2p-cbindings:core` being implemented (interaction with streams) | ||
|
|
||
| #### deliverables |
Copilot
AI
Dec 16, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Section headers should use title case for consistency. This should be 'Deliverables' to match the capitalization used in other commitment documents.
| #### deliverables | |
| #### Deliverables |
6c91474 to
376d209
Compare
No description provided.