Source: ce-code-review run 20260830-212446-4b8b0529 (testing + adversarial testing_gaps).
What
The launcher's core fix (catch-all spawn-failure collapse), the direnv fallback, exit-code pass-through, and stdin payload plumbing have no committed test: wire.rs is text-grep only, and the plan's verification smokes were session-scoped. A regression to throw passes CI.
Fix options
- wire.rs: assert
} catch { present and throw error absent (done: mechanism pair, not comment token).
- Stronger: commit a behavioral smoke in CI (stub comment-checker/direnv on a temp PATH, LD_FOR_BUILD set, pipe a flagged Write payload, assert exit 2 + report on stderr). Needs deno on the CI runner for the plugin surface.
Carried by PR #92: #92
Source: ce-code-review run 20260830-212446-4b8b0529 (testing + adversarial testing_gaps).
What
The launcher's core fix (catch-all spawn-failure collapse), the direnv fallback, exit-code pass-through, and stdin payload plumbing have no committed test: wire.rs is text-grep only, and the plan's verification smokes were session-scoped. A regression to
throwpasses CI.Fix options
} catch {present andthrow errorabsent (done: mechanism pair, not comment token).Carried by PR #92: #92