Skip to content

chore: phase 4 protocol adapter boundary#26

Merged
Lythaeon merged 1 commit intomainfrom
chore/architecture-phase4-protocol-adapter-boundary
Feb 14, 2026
Merged

chore: phase 4 protocol adapter boundary#26
Lythaeon merged 1 commit intomainfrom
chore/architecture-phase4-protocol-adapter-boundary

Conversation

@Lythaeon
Copy link
Copy Markdown
Owner

Summary

  • Implement Phase 4 of the hexagonal migration by replacing protocol runtime switching with registry-driven transport adapters.
  • Move protocol selection flow to domain ProtocolKind across planning and local-run execution seams.
  • Keep behavior stable while preserving existing protocol runtime setup functions behind adapter boundaries.

Changes

  • Added TransportAdapter and updated ProtocolRegistry to register and dispatch adapters keyed by domain::run::ProtocolKind.
  • Refactored builtin protocol registrations to carry protocol metadata plus sender setup callbacks; removed central runtime match dispatch.
  • Threaded domain protocol selection through local run orchestration (LocalRunExecutionCommand, TrafficPort, runtime traffic adapter).
  • Added explicit CLI-to-domain conversion helpers for protocol/load mode and switched plan-time protocol validation to domain keys.
  • Updated protocol registry/runtime tests and local-run tests for the new transport dispatch signature.
  • Marked Phase 4 artifacts as implemented in docs/architecture/ard/ARCHITECTURE_RISKS_HEXAGONAL_PLAN.md.

Checks

  • cargo make format
  • cargo make clippy
  • cargo make test
  • cargo make architecture-check
  • cargo make test-wasm (not applicable)
  • cargo make audit (not applicable)
  • cargo make deny (not applicable)

@Lythaeon Lythaeon merged commit cfe03e2 into main Feb 14, 2026
1 check passed
@Lythaeon Lythaeon deleted the chore/architecture-phase4-protocol-adapter-boundary branch February 14, 2026 11:42
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