Repository navigation
Fix LM5 fixture termination and qualify network evidence - #480
Merged
Merged
Conversation
The cf7-closeout worktree was pruned after #443 merged, so the receipt's launch commands pointed at nothing. The saved homes are restored into the cf7-review-kit worktree from the task-evidence copy, and both saved Task Notes reopen read-only on a build of 0a8c513. Records the two things a reviewer will see: the notes predate #450, and the homes name the old repository path. No human row changes. Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
BunsDev
added a commit
that referenced
this pull request
Nov 4, 2026
* Correct merged PR dates to October 31 UTC (#470) * docs: point the CF-7 review kit at its restored location (#280) (#473) The cf7-closeout worktree was pruned after #443 merged, so the receipt's launch commands pointed at nothing. The saved homes are restored into the cf7-review-kit worktree from the task-evidence copy, and both saved Task Notes reopen read-only on a build of 0a8c513. Records the two things a reviewer will see: the notes predate #450, and the homes name the old repository path. No human row changes. Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
This branch was successfully deployed
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.
Fix two review findings in #475. The AC3 fixture now races server arrival against the first turn result and propagates test cancellation, so an early failure reaches cleanup instead of waiting indefinitely. Scope the compaction and failure-network evidence to requests observed by the recording loopback server; traffic to other destinations was not monitored.
Validation: build and the full unit suite passed with 2,281 passes and 35 skips. An already-aborted first-turn negative control hung before the repair until an external 12-second deadline; afterward it failed promptly with the interrupted result and exited normally. The normal installed-Codex AC3 wire fixture passed in about 2 seconds. Independent review found no blockers; its minor test-title correction is included. No UI code changed and no additional E2E suite was run.
This integration retains main’s CF-7 checkpoint restoration and corrected landing dates. It targets lm5-fixtures; #475’s separate Val merge gate, product admission and native/offline acceptance remain open.