Skip to content

session-relay: land the 4 binaries — atomic flip to bin/relay, delete the Node payload (rust-port steps 7-8)#5

Merged
DocksDocks merged 4 commits into
mainfrom
explore/rust-port-and-okf
Jul 1, 2026
Merged

session-relay: land the 4 binaries — atomic flip to bin/relay, delete the Node payload (rust-port steps 7-8)#5
DocksDocks merged 4 commits into
mainfrom
explore/rust-port-and-okf

Conversation

@DocksDocks

Copy link
Copy Markdown
Owner

What's in here

Completes the session-relay Rust port (steps 7–8 of the plan) plus the ship of the knowledge-format plan:

  • 4 committed arch binaries from build-binaries.yml run 28552485456 (both darwin from the arm64 runner, both linux-musl from ubuntu; per-runner transit checksums verified before commit) + regenerated bin/SHA256SUMS + the POSIX-sh arch-dispatch launcher bin/relay — all five entries mode 100755, .gitattributes marks relay-* as binary (plain blobs, never LFS).
  • Atomic manifest flip per the plan's Interfaces table: Claude plugin.json MCP + hooks.json (exec form) on ${CLAUDE_PLUGIN_ROOT}/bin/relay; Codex bus.mcp.json on native ${PLUGIN_ROOT}; codex-hooks.json shell form.
  • Node payload deleted: mcp/bus.mjs, lib/store.mjs, lib/discover.mjs, hooks/session-start.mjs, skills/.../scripts/relay.mjs — every store toucher is now the flock-locked binary (all-or-nothing by construction).
  • Gate hardening: gateRust never overwrites the committed binary; host-rebuild byte-identity FAILS in CI (same image as the producer workflow) and warns locally (embedded paths + distro linker make cross-machine identity unachievable — recorded in the plan). shellHooks() now shellcheck-lints the launcher (SC1007 fixed: CDPATH='').
  • Plans: knowledge-format-lint-and-citations shipped (review passed, ship_commit 9e7a732); rust-port plan in_review with a partial completion review — all headless acceptance criteria pass; the two live-session legs (real Claude⇄Codex round-trip, Codex ${PLUGIN_ROOT} substitution on a real install) remain open before ship.

Verification

  • node scripts/ci.mjs green (2 plugins + repo-wide): fmt/clippy clean, committed checksums verify, shellcheck-linted launcher, 39-check selftest against the committed binaries.
  • cargo test 10/10 incl. the cross-process flock race + .lock dir→file migration.
  • Step-7/8 acceptance greps: all four manifests contain bin/relay, zero "command": "node", native-var-only Codex manifest, exec-bit index check (git ls-files -s → only SHA256SUMS non-755), five .mjsNo such file.

🤖 Generated with Claude Code

https://claude.ai/code/session_01UyDDEDsq6yGeJKzNAwW3ZQ

DocksDocks and others added 4 commits July 1, 2026 19:42
…letion re-review at 9e7a732)

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UyDDEDsq6yGeJKzNAwW3ZQ
…binaries + launcher + manifests (rust-port plan, step 7/8)

Binaries from build-binaries.yml run 28552485456 (transit checksums
verified); SHA256SUMS regenerated in bin/; launcher shellcheck-linted
(SC1007: CDPATH='') and dispatching via uname -sm; .gitattributes marks
relay-* binary (plain blobs, never LFS); all five bin/ entries except
SHA256SUMS committed 100755. Manifest flips per the plan's Interfaces
table: Claude plugin.json MCP + hooks.json exec form on
${CLAUDE_PLUGIN_ROOT}/bin/relay, Codex bus.mcp.json on NATIVE
${PLUGIN_ROOT}, codex-hooks.json shell form. gateRust hardened: never
overwrites the committed binary; rebuild byte-identity FAILS in CI
(same image as producer) and warns locally — cross-machine identity is
unachievable (embedded paths + distro linker), recorded in the plan.
shellHooks() now lints a rust capability's sh launcher.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UyDDEDsq6yGeJKzNAwW3ZQ
…me plugin (rust-port plan, step 8/8)

git rm mcp/bus.mjs lib/store.mjs lib/discover.mjs
hooks/session-start.mjs skills/.../scripts/relay.mjs. Every store
toucher is now the flock-locked bin/relay binary; full gate green
(shellcheck lints the launcher, checksums verify, 39-check selftest
against the committed binaries). Plan -> in_review.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UyDDEDsq6yGeJKzNAwW3ZQ
…criteria all pass; 2 live-verify legs open)

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UyDDEDsq6yGeJKzNAwW3ZQ
@DocksDocks DocksDocks merged commit 5b41d88 into main Jul 1, 2026
1 check passed
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