Skip to content

Wire component into shared NATS JetStream transport - #5

Open
blackboxprogramming wants to merge 1 commit into
mainfrom
codex/nats-ecosystem-20260914
Open

blackboxprogramming wants to merge 1 commit into
mainfrom
codex/nats-ecosystem-20260914

Conversation

@blackboxprogramming

Copy link
Copy Markdown
Owner

This adds the roadc component to the shared RoadOS NATS notification bus.

Adds a component launcher and manifest, explicit environment settings, setup documentation, and ecosystem discovery entries.

Requires the shared implementation in https://github.com/blackboxprogramming/RoadOS/pull/19. Merge that first, or check out codex/nats-ecosystem-20260914 in the sibling RoadOS repository.

Verified locally: this repository's launcher published through a real NATS JetStream broker and the receiver persisted its notification. Offline status also passed. The coordinated RoadOS suite passed all 617 tests, including 24 NATS checks. Road/Roadies/shell/RoadC regression suites passed (RoadC has one existing skip); Roadie Micro's independent tests require unavailable torch.

Network access is disabled by default. The relay must be explicitly started against a selected file. Events carry content hashes and sizes, not file contents, and do not authorize execution. No live service deployment is included.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 14, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-14T05:39:02.581614Z 0f51ef3 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Copy link
Copy Markdown
Owner Author

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: NATS.md documents ../RoadOS/requirements-nats.txt even when ROAD_BUS_ROOT is set. Road and Roadie now use:

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.

Copy link
Copy Markdown
Owner Author

Correction to my coordination note above: use export ROAD_BUS_ROOT="${ROAD_BUS_ROOT:-../RoadOS}", not a non-exported shell assignment. Roadie’s follow-up review caught that the launcher reads os.environ, so export is required for later nats_bus.py processes. I am still leaving this branch untouched for the session owning roadc.

Copy link
Copy Markdown
Owner Author

Final coordination correction: do not export ROAD_BUS_ROOT=../RoadOS for the sibling case. Leave ROAD_BUS_ROOT unset so the launcher keeps its script-relative sibling default. For a non-sibling checkout only, use an explicit absolute path, e.g. export ROAD_BUS_ROOT="/absolute/path/to/RoadOS", then install from $ROAD_BUS_ROOT/requirements-nats.txt. This avoids current-working-directory-dependent failures.

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