feat: cwd hard gate, test hermeticity, opus parity withdrawal (0.0.47) - #46
Merged
okisdev merged 4 commits intoAug 1, 2026
Merged
Conversation
…he sandbox root in footers
…us 5 orchestrator parity
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
summary
--cwd <toplevel>to target the repository root or--cwd <subdirectory>to confirm the narrow root deliberately. the 0.0.41 post-run warning surface and the codexexplicitCwdstamping leave as dead code, and grok, which had no detection at all, gains the same gatesandbox: <cwd>betweenjob:and the delivery lines, documented in the shared companion contract), and the orchestration rules require an explicit--cwd "<absolute workspace root>"on every dispatch that starts an engine run, current workspace included, so a drifted shell fails loudly at dispatch instead of silently sandboxing to a subtreeprocessInspectionAvailable()probe in the companion harness lets 128 job-creating tests skip with a reason when the environment denies process inspection (COMPANION_TEST_NO_PROCESS_INSPECTION=1forces the path for verification), andgitIsolationbecomes a function that injectscommit.gpgsign=falsewith a sandbox-local home and empty template dir so host signing config cannot abort fixturestest plan
already verified
npm test-> 1074 tests, 1073 pass, 0 fail, 1 pre-existing skipCOMPANION_TEST_NO_PROCESS_INSPECTION=1-> 43 pass, 128 skipped with reason, 0 failGIT_CONFIG_GLOBAL(gpgsign on, nonexistent ssh signer) -> identical to the clean run--cwdof toplevel or subdirectory accepted (companion suite cases)reviewer should verify
tasksubcommand without--cwdand confirm it fails withfailure: input, names both--cwdremedies, and creates no job recordsandbox: <cwd>betweenjob:anddelivery:notes
sandbox:footer line is additive, but any external parser pinned to the exact footer block shape should learn it; both contract docs show the new block/fusion:setupto sync the rules copy