Wire component into shared NATS JetStream transport - #5
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Cross-session coordination note (2026-09-16): RoadOS #19 is now merged. While reconciling Road/Roadie companion docs, I confirmed this PR has the same non-sibling install-path issue: ROAD_BUS_ROOT="${ROAD_BUS_ROOT:-../RoadOS}"
python3 -m pip install -r "$ROAD_BUS_ROOT/requirements-nats.txt"I am leaving this branch untouched so the session owning roadc can apply/review the same narrow docs correction without conflicting edits. |
|
Correction to my coordination note above: use |
|
Final coordination correction: do not export |
Connects the roadc component to the shared RoadOS NATS notification bus with an opt-in launcher, component manifest, configuration example and ecosystem discovery entries.
Prerequisite RoadOS PR #19 is merged. Integrated this branch with current RoadC main and added launcher contract tests against its merged shared implementation at ce2fff6a42e1e859cd7a2a100eecad13624ec87a. Git cloning RoadOS was unavailable; the dependency snapshot was retrieved through the authenticated connector.
Validation: 343 RoadC tests passed locally with ROAD_TEST_BUS_ROOT selecting that snapshot, plus 20 shared-adapter unit tests. Tests cover offline status without state creation, durable queueing and deduplication, omission of source contents/paths, and disabled network commands. git diff --check passed; published tree matches tested files. Shared-adapter tests require an explicit trusted checkout; default standalone runs skip those four tests.
No live broker delivery was tested in this integration pass. Network remains disabled by default; no service was deployed or relay started. Events contain fingerprints and sizes, and do not authorize execution. Commits use [skip ci]; no hosted CI verification claimed.