Skip to content

Feat/p2p gossipsub - #177

Draft
aimendj wants to merge 5 commits into
unstablefrom
feat/p2p-gossipsub
Draft

Feat/p2p gossipsub#177
aimendj wants to merge 5 commits into
unstablefrom
feat/p2p-gossipsub

Conversation

@aimendj

@aimendj aimendj commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Implement GossipSub validation and broadcast handling for Cryptarchia blocks and Mantle transactions, migrate to the unified nim-bincode API, add P2P integration and AutoNAT tests, optimize hot networking accessors, and clean up internal APIs and spec references.

@aimendj aimendj self-assigned this Aug 27, 2026
# Conflicts:
#	logos_chain/node.nim
@github-actions

Copy link
Copy Markdown

Unit Test Results

       5 files  ±  0     665 suites  ±0   1m 19s ⏱️ +14s
   598 tests +  2     589 ✔️ +  6    9 💤  -   4  0 ±0 
3 150 runs  +10  3 105 ✔️ +30  45 💤  - 20  0 ±0 

Results for commit 4067b4a. ± Comparison against base commit a90edc8.

This pull request removes 17 and adds 19 tests. Note that renamed tests count towards both.
GossipSub: subscribes and publishes /logos-blockchain-testnet/cryptarchia/1.0.0 (testnet)
GossipSub: subscribes and publishes /logos-blockchain-testnet/mempool/1.0.0 (testnet)
GossipSub: subscribes and publishes /logos-blockchain/cryptarchia/1.0.0 (mainnet)
GossipSub: subscribes and publishes /logos-blockchain/mempool/1.0.0 (mainnet)
Kademlia: DHT protocol registered as /logos-blockchain-testnet/kad/1.0.0 (testnet)
Kademlia: DHT protocol registered as /logos-blockchain/kad/1.0.0 (mainnet)
deserializeSignedMantleTxAt reads one prefixed tx inside a longer buffer
serializeBlockToSeq / deserializeBlock roundtrip (default signature, empty txs)
serializeBlockToSeq / deserializeBlock roundtrip (genesis block)
serializeBlockToSeq / deserializeBlock roundtrip (non-default signature, one tx)
…
Bootstrap peer unreachable: node logs warning and proceeds standalone
Bootstrap via trusted peer node
Discovery finds new peers through mesh
GossipSub: subscribes, broadcasts, and applies blocks
GossipSub: subscribes, broadcasts, and ingests transactions
a transaction with exactly MantleMaxOps operations is accepted
decodeAt reads one prefixed tx inside a longer buffer
encode / decode DownloadBlocksRequest roundtrip
encode / decode DownloadBlocksResponse (Failure reasons)
encode / decode DownloadBlocksResponse roundtrip (NoMore)
…
This pull request removes 6 skipped tests and adds 2 skipped tests. Note that renamed tests count towards both.
GossipSub: subscribes and publishes /logos-blockchain-testnet/cryptarchia/1.0.0 (testnet)
GossipSub: subscribes and publishes /logos-blockchain-testnet/mempool/1.0.0 (testnet)
GossipSub: subscribes and publishes /logos-blockchain/cryptarchia/1.0.0 (mainnet)
GossipSub: subscribes and publishes /logos-blockchain/mempool/1.0.0 (mainnet)
Kademlia: DHT protocol registered as /logos-blockchain-testnet/kad/1.0.0 (testnet)
Kademlia: DHT protocol registered as /logos-blockchain/kad/1.0.0 (mainnet)
Bootstrap peer unreachable: node logs warning and proceeds standalone
Discovery finds new peers through mesh

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