Skip to content

[SEAM] W5.7 identify/ping: the mock connections of #45 #55

Open

Description

@leolara

Seam connections for merged PR #45 (W5.7 identify and ping). PR #45 merged at f4e35cb with two own mocks carried, per the organiser ruling of 7 November: an approved PR with a mock of a merged dependency merges, and the swaps land through this issue. The identity and multiaddr rows of the old table are done: the identity swap landed inside #45 on 6 November, and the multiaddr swap landed with the earlier base merge.

Own mock Real dependency The connections to make Window
Protocols.Mock.Core #15 W5.2 (merged, PR #64) delete the mock, import Libp2p.Core, re-type the handlers over the real StreamHandler and StreamContext; the identifyId, identifyPushId, and pingId constants re-point to the real ProtocolId 19:22 UTC 2026-11-07
Protocols.Mock.Host #19 W5.6 (merged, PR #61) delete the mock, take the peerstore and the served stream from the real host seam, and serve identify and push through the swarm registry 00:32 UTC 2026-11-08

Definition of done

Ground rules

  • The identify/ping author (PR feat(protocols): W5.7, identify and ping #45) owes this work; comment claim to take it if they do not.
  • Work lands as a follow-up pull request against hackathon26 from the author's fork. This issue closes when the tip is clean of the mocks.
  • Prose register everywhere: CLAUDE.md, "Writing Style".

Activity

  1. IvanAnishchuk commented on Nov 10, 2026

    @IvanAnishchuk
    Contributor

    #86 moves the host onto the merged libp2p/negotiate and deletes Negotiate.Legacy. No existing signature changes. The host adds four names:

    • Libp2p.Host.upgrade runs the upgrade, with the secured checkpoint after the handshake and before the muxer stage.
    • Libp2p.Host.negotiateStream runs multistream-select on a muxed Stream, and hands the bytes after the selection to the handler.
    • Upgraded and Admission now live in Libp2p.Host. No code outside the host named them before.

    Once #86 merges, Tests/Host.lean runs real hosts over the sim (unit 1b). A swap of this unit's host mock can build its test hosts the same way.

  2. IvanAnishchuk commented on Nov 10, 2026

    @IvanAnishchuk
    Contributor

    #87 deletes Protocols/Mock/Muxer.lean and Protocols/Mock/HostStream.lean, and edits the package ARCHITECTURE and PLAN of libp2p/protocols. No public signature changes. Mock/Core.lean and Mock/Host.lean stay for this issue.

    Tests/HostPair.lean (new in #87) builds two real hosts over the sim with Noise and yamux: hostPair answers the client, the server, the proved server peer, and the server's failures. The identify and ping suites can run on the same pair after their swap.

  3. IvanAnishchuk commented on Nov 10, 2026

    @IvanAnishchuk
    Contributor

    claim. 8 hours have passed, or twelve or however much, was too long in any case and I have another PR depending on this 🤷

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions