Skip to content

Let a coding run blocked only by the sandbox go on to the Lab's checks (#402) - #405

Merged
DanielNetzer merged 1 commit into
mainfrom
forgecell/issue-402-f70fc1a1a13bf88da94b77c4
Nov 8, 2026
Merged

DanielNetzer merged 1 commit into
mainfrom
forgecell/issue-402-f70fc1a1a13bf88da94b77c4

Conversation

@DanielNetzer

Copy link
Copy Markdown
Owner

Implements issue #402.

Molecule: mol-402-35b8c31d2bd32cf43bf33731
Formula: forgecell-github-issue
Base: 3e71c02c78507281820dd277b0b1d7196622e858
Delivery: 6b7f66aa922192ab605526e1ff57c1b3b292786a

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

Notes:

  • Scope amendment (revision 2): lab/internal/molecule/verification_history.go was added. completedCoding also admits environment-blocked with captured changes, so --continuation checks-only and the delivery preview accept it. The other callers (RebaseFormula, retainedWorkCertain, next_actions) widen too; that is intended, because such an attempt is finished work.
  • Protected regression, accepted by design: TestAdapterInvocationChangesRequireProtocolBump (no pin for protocol 10) and TestAdapterCompatibility/{claude-code,codex,cursor}/molecule (they compare the coding instruction and outcome schema against the old contracts.json).
  • Flaky under memory pressure (LOW): one full-suite run failed TestRetainedWorkNoteComesBeforeTheOperatorNote (fixture ended blocked) and TestPendingChecksOnlyRejectsUnsafeAmendments/paused-tree (exec: WaitDelay expired before I/O complete) while the host had under 1 GiB free. Both pass when run alone on this tree, and the checks-only rerun passed. Worth a follow-up if they show up in CI.
  • Protocol 10: saved bindings need a reviewed rebind after upgrading.

All required checks passed.

@DanielNetzer
DanielNetzer marked this pull request as ready for review November 8, 2026 00:21
@DanielNetzer
DanielNetzer merged commit b15771e into main Nov 8, 2026
2 checks passed
@DanielNetzer
DanielNetzer deleted the forgecell/issue-402-f70fc1a1a13bf88da94b77c4 branch November 8, 2026 00:37
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