Conversation
|
There was a problem hiding this comment.
Pull request overview
This PR introduces an optional allowlist mechanism for peer connections in chain configurations, allowing specific chains to wire connections only to designated peers instead of the full mesh network.
Changes:
- Added
allowed_peersfield to chain configuration interface and templates - Enhanced
filterConnectionsfunction with type safety and peer allowlist filtering - Added
beforeEachhook to test suite to ensure proper stage initialization
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| packages/stg-evm-v2/devtools/config/utils.ts | Added allowed_peers field to Chain interface, implemented peer filtering logic (isPeerAllowed, isAllowedPeerConnection), and enhanced filterConnections with type safety and peer validation |
| packages/stg-evm-v2/devtools/config/testnet/chainsConfig/0-template-chain.yml | Added allowed_peers field to testnet chain template with documentation |
| packages/stg-evm-v2/devtools/config/mainnet/01/chainsConfig/0-template-chain.yml | Added allowed_peers field to mainnet chain template with documentation |
| packages/stg-evm-v2/test/devtools/utils.test.ts | Added beforeEach hook to set stage for filterConnections test suite |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
ravinagill15
left a comment
There was a problem hiding this comment.
Let's add a changeset too
@ravinagill15, I will merge #508 into this branch and will update the change set, adding the modifications on this PR |
Uh oh!
There was an error while loading. Please reload this page.