Skip to content

main is red: StaleCheckpointFreshTurnTests fail after fe4ed3a5 landed on top of 1b45c641 - #741

Merged
AdamFrisby merged 2 commits into
mainfrom
codeybox/138337e9
Oct 13, 2026
Merged

AdamFrisby merged 2 commits into
mainfrom
codeybox/138337e9

Conversation

@AdamFrisby

Copy link
Copy Markdown
Owner

Automated via CodeyBox — work item 138337e94f8c4e7e82078b352ca483e3

Initiated by CodeyBox operator


Co-Authored-By: CodeyBox noreply@codeybox.invalid
🤖 Generated with CodeyBox

AdamFrisby and others added 2 commits October 12, 2026 05:39
The fork-point resume baseline (1b45c64) is an ancestor of the checkpoint by construction, so the stale check built on it could never fire: stale checkpoints resumed instead of being discarded, and a concurrent retained-sandbox lease surfaced as a generic checkpoint-changed failure instead of the explicit lease refusal. The branch tip alone cannot be the signal either, because a same-base branch advance must keep resuming while a base move must discard, and the two are indistinguishable after the pickup's upstream refresh rewrites the base ref.

Snapshot the base-branch tip before the pickup refresh and decide staleness as a pre/post-refresh difference: any base move means the checkpoint's base was superseded. Same-base advances leave the base untouched and resume via the fork-point baseline as before; a missing snapshot falls back to base ancestry. Adds a combination test (stale checkpoint plus an advanced tip) proving the discard still stages the fresh turn at the current tip.

CodeyBox-Prompt-Revision: 1

Co-Authored-By: CodeyBox <noreply@codeybox.invalid>
CodeyBox-WorkItem: 138337e94f8c4e7e82078b352ca483e3
CodeyBox-Agent: copilot/muse-spark-1.3-contributor
CodeyBox-Prompt-Revision: 1
Co-Authored-By: CodeyBox <noreply@codeybox.invalid>
@AdamFrisby
AdamFrisby merged commit b29c359 into main Oct 13, 2026
@AdamFrisby
AdamFrisby deleted the codeybox/138337e9 branch October 13, 2026 08:13
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