Skip to content

fix(nightly-v2): drop the orphaned ACP settle pin - #94

Closed
saphid wants to merge 1 commit into
mainfrom
fix/nightly-v2-drop-orphaned-pin
Closed

saphid wants to merge 1 commit into
mainfrom
fix/nightly-v2-drop-orphaned-pin

Conversation

@saphid

@saphid saphid commented Sep 29, 2026 •

Copy link
Copy Markdown
Owner

Every Fork Nightly Orchestrator v2 run since 23 Sep has failed in resolve with 422 No commit found for SHA: b0ad1457…. That manifest entry pinned a commit from the PR branch for pingdotgg#13201, and the branch was rewritten after the pin, so the commit no longer exists on GitHub.

The fix merged upstream as 6134998b ("test(server): stop ACP soft-steer tests racing the prompt settle (pingdotgg#13201)"), which is already an ancestor of t3code/codex-turn-mapping (currently cb64162d). This PR drops the dead entry instead of re-pinning it.

After this merges, v2 gets past resolve, but its stack still conflicts with the current upstream branch. 75be8c11 (custom updater on V2 profiles) conflicts in apps/desktop/src/updates/updatesTestHarness.ts, and several later pins need refreshing too. The companion PR that skips identical failed inputs keeps that from re-emailing every five minutes.

Verification: parseManifest accepts the file. A local resolve against GitHub succeeds (Will build v0.0.42-nightly-v2.20260929… from cb64162dee). A local apply then stops at the 75be8c11 conflict described above. Independent review on Codex gpt-6-sol (high), via T3 delegated task, found no actionable findings and independently confirmed the 422 and the ancestry.

Work by Claude Opus 5.5 (1M context) in Claude Code, running in T3 Code.

🤖 Generated with Claude Code


Update: to stop the failure emails immediately, both Fork Nightly and Fork Nightly Orchestrator v2 are now disabled on saphid/t3code (gh workflow disable). After merging #93 (and #94), re-enable them with:

gh workflow enable 347086910 -R saphid/t3code   # Fork Nightly
gh workflow enable 352233980 -R saphid/t3code   # Fork Nightly Orchestrator v2

The v2 manifest pinned b0ad1457, which no longer exists on saphid/t3code
after its PR branch was rewritten, so every v2 run failed in resolve with
a 422. The fix merged upstream as pingdotgg#13201 (6134998) and is
already in t3code/codex-turn-mapping, so the pin is redundant.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@saphid

saphid commented Sep 30, 2026

Copy link
Copy Markdown
Owner Author

Superseded by #100, which refreshes the whole v2 stack and no longer includes the dead pin.

@saphid saphid closed this Sep 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant