Skip to content

feat: expose local_socket_addrs via HcP2p trait for WebRTC ICE#1

Closed
HexaField wants to merge 1 commit into0.6.0-coasysfrom
feat/expose-iroh-endpoint
Closed

feat: expose local_socket_addrs via HcP2p trait for WebRTC ICE#1
HexaField wants to merge 1 commit into0.6.0-coasysfrom
feat/expose-iroh-endpoint

Conversation

@HexaField
Copy link

Summary

Surfaces local_socket_addrs() from the kitsune2 transport layer through the HcP2p trait, enabling in-process consumers (e.g. AD4M executor) to retrieve transport-discovered socket addresses for WebRTC ICE candidate extraction.

Changes

  • HcP2p trait: Added local_socket_addrs() with default returning empty vec
  • HolochainP2pActor: Implements by calling kitsune.transport().local_socket_addrs()
  • Workspace Cargo.toml: Enabled kitsune2 path overrides for local development with transport changes

Usage

let addrs = conductor.holochain_p2p().local_socket_addrs().await?;
// Returns Vec<SocketAddr> — local + reflexive addresses discovered by Iroh

Dependencies

Requires holochain/kitsune2#476 for the local_socket_addrs() addition to the Transport trait.

See: coasys/ad4m#719

@HexaField HexaField marked this pull request as ready for review March 11, 2026 08:40
@HexaField HexaField force-pushed the feat/expose-iroh-endpoint branch from 8217ae0 to f5734ce Compare March 11, 2026 09:30
Adds local_socket_addrs() to HcP2p trait with default empty vec.
HolochainP2pActor implements it by calling through kitsune.transport().local_socket_addrs().
Enables kitsune2 path overrides for local development with transport changes.

Consumers (e.g. AD4M executor) can call conductor.holochain_p2p().local_socket_addrs().await

Part of: coasys/ad4m#719
@HexaField HexaField force-pushed the feat/expose-iroh-endpoint branch from f5734ce to 74c518f Compare March 11, 2026 09:47
@HexaField HexaField marked this pull request as draft March 11, 2026 11:18
@HexaField
Copy link
Author

Closing to restart with spec-driven approach.

@HexaField HexaField closed this Mar 11, 2026
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