clawdie/colibri
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 33

fix(spawner): FreeBSD jail command=<binary> syntax (salvaged from Codex) #39

Merged
clawdie merged 1 commit from fix/jail-command-syntax into main 2026-06-13 21:35:55 +02:00
Conversation 0 Commits 1 Files changed 1 +5 -6
clawdie commented 2026-06-13 21:30:32 +02:00
Owner
Copy link

Salvages the FreeBSD jail(8) command-syntax fix from Codex's fix/jail-spawner-freebsd-command-format, rebased cleanly onto current main.

Why: the merged jail_wrap emitted the ephemeral jail command as two argv tokens ("command", binary), but jail(8) needs a single name=value parameter (command=<binary>). Without this the jail -c ephemeral path fails on FreeBSD. Fixes the builder + the two jail_tests expectations.

Why not just merge Codex's branch: it predates the doc fix and also re-ran a markdown formatter, which would re-corrupt docs/COLIBRI-JAILED-AGENT-SPAWN-DESIGN.md (mac_do → mac*do) and churn the README. This takes only the validated spawner.rs change.

Supersedes fix/jail-spawner-freebsd-command-format (safe to close).

daemon lib tests (58) green; clippy -D warnings + fmt clean.

Salvages the FreeBSD `jail(8)` command-syntax fix from Codex's `fix/jail-spawner-freebsd-command-format`, rebased cleanly onto current `main`. **Why:** the merged `jail_wrap` emitted the ephemeral jail command as two argv tokens (`"command"`, `binary`), but `jail(8)` needs a single name=value parameter (`command=<binary>`). Without this the `jail -c` ephemeral path fails on FreeBSD. Fixes the builder + the two `jail_tests` expectations. **Why not just merge Codex's branch:** it predates the doc fix and also re-ran a markdown formatter, which would re-corrupt `docs/COLIBRI-JAILED-AGENT-SPAWN-DESIGN.md` (`mac_do` → `mac*do`) and churn the README. This takes only the validated `spawner.rs` change. Supersedes `fix/jail-spawner-freebsd-command-format` (safe to close). daemon lib tests (58) green; clippy `-D warnings` + fmt clean.
clawdie added 1 commit 2026-06-13 21:30:34 +02:00
fix(spawner): use FreeBSD jail command=<binary> parameter syntax
Some checks failed
CI / rust (pull_request) Has been cancelled
Details
CI / markdown (pull_request) Has been cancelled
Details
c38f6e5a73 
Salvages Codex's FreeBSD-validated fix from
fix/jail-spawner-freebsd-command-format, taking ONLY the spawner.rs change onto
current main. (The original branch also re-ran a markdown formatter and would
have re-corrupted the jailed-spawn design doc + README, so those are dropped.)

My merged jail_wrap emitted the ephemeral jail command as two argv tokens
("command", binary), but jail(8) expects a single name=value parameter
(command=<binary>). Without this the `jail -c` ephemeral path fails on FreeBSD.
Fixes the ephemeral builder and the two jail_tests expectations.

Co-authored-by: Sam & Codex (fix/jail-spawner-freebsd-command-format)
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
clawdie merged commit c8f6c7d024 into main 2026-06-13 21:35:55 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
first-proof blocker

gates the first proven end-to-end

  
hardening

security/robustness follow-up, not a first-proof gate

No labels
first-proof blocker
hardening
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: clawdie/colibri#39
No description provided.
Delete branch "fix/jail-command-syntax"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?