Skip to content

Pass the harness Go cache through Codex's shell_environment_policy (#401) - #404

Merged
DanielNetzer merged 1 commit into
mainfrom
forgecell/issue-401-17b2b99990c24d461244af3c
Nov 7, 2026
Merged

DanielNetzer merged 1 commit into
mainfrom
forgecell/issue-401-17b2b99990c24d461244af3c

Conversation

@DanielNetzer

Copy link
Copy Markdown
Owner

Implements issue #401.

Molecule: mol-401-13d8cf86cfd1fcee8cd50533
Formula: forgecell-github-issue
Base: 5a4e0c89deaa46dc185ed9156b8f05b11f5d44e3
Delivery: 44e6de3d01103b1ca104a60c310d63ed5b6bf9dc

Awaiting exact-commit CI and human review. Harness-reported checks are not independent CI evidence. No merge, deployment or issue closure is authorized.

@DanielNetzer

Copy link
Copy Markdown
Owner Author

Protected regression: two tests failed by design and were accepted with --accept-protected-failures:

Manual check before the change: under codex sandbox (Codex 0.167.0), -c shell_environment_policy.set.GOCACHE=… overrode an inherited GOCACHE, and the Lab cache directory became writable through writable_roots. The real proof is the Codex retest on forgecell-trial after release.

Protocol 9: saved bindings need a reviewed rebind after upgrading. All checks passed.

@DanielNetzer
DanielNetzer marked this pull request as ready for review November 7, 2026 22:34
@DanielNetzer
DanielNetzer merged commit 3e71c02 into main Nov 7, 2026
2 checks passed
@DanielNetzer
DanielNetzer deleted the forgecell/issue-401-17b2b99990c24d461244af3c branch November 7, 2026 22:52
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