docs(handoff): RETRACT the CI-flake ranked step β that failure is already root-caused in scripts/ci-repro/README.md - #1229
Merged
Conversation
β¦out PR #1166, the subsystem ind Claude-Session-Id: 2b50cc65-f7bf-4f03-8f8e-19bd8fabe25c
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.
Corrects the
resume-state-worktree-resolutionhandoff doc, which landed in #1166 with a ranked next-step that would send the next session to do work that is already done.The defect
Rank 2 read "File the CI-flake issue β four distinct tests, four PRs, one session, each needing a fresh push."
That failure is already root-caused, on
main.scripts/ci-repro/README.mdnames the mechanism βserver.py:_replace_bytesfsyncs the file and then the parent directory inside the request, before the response is written, anddevrc-ciis pinned to one node, so stacked runs contend on one disk. It states outright that it hits PRs whose diff cannot reach it, docs-only included. Thedevrc/tests.mdindex entry carries the same finding dated 2026-09-01, measured across two docs-only PRs: 4 reds, 3 different tests,scripts/teststargets 464β530 s, each passing 3/3 locally in ~5 s with a--collect-onlypositive control proving the failing test was actually selected.So the ranked step is retracted, and the Open-investigations block that fed it is superseded in place rather than deleted β the prior reading stays visible, which is the point of an append-only section.
Why it happened, recorded as a gotcha
I measured that same flake four times across four PRs β attribution against the PR's own file list, a pristine-
maincontrol, a same-window sibling PR β and concluded it was unexplained. The answer was in the subsystem index the whole time, with a deeper diagnosis than any of those four measurements produced./resumestep 4 exists for exactly this and costs one command.Second gotcha, found by a completion audit rather than by any gate: the index write is per-effort, and a multi-effort session will skip the second one.
/handoffstep 4 ran for effort 1 and not for effort 2. Nothing noticed β the doc, the PRs and the claims were all clean. If a session lands more than one PR, runsubsystem_touch --pr <n>once per effort.Also updated
6421df3c(closes fix(resume-state): an explicit handoff path in a LINKED WORKTREE falls back and reconciles the wrong initiative β #1159 does not close itΒ #1164), test(handoff_doc): close the audit ladder's filed scaffolding gaps (#1093, #1115)Β #1146 βedbc596f(closes test(handoff_doc): two scaffolding gaps from #1046's audit ladder β a misdirecting failure message and a degenerate-value fixtureΒ #1093, test(handoff_doc): four scaffolding gaps from #1046 round 8 β a dead mutation anchor, an unpinned predicate, and unpinned exit valuesΒ #1115), docs(handoff): stale-base-guard close-out β #1146 merged, the ladder closed on the attribution gate, two follow-ups filedΒ #1166 βe2a9f781. All verified by content onorigin/main, never by ancestry.readlink -fshowsscripts/resume-state.shresolves to itself (live on a plaingit pull, 0 references innix/) whileclaude/skills/resume/SKILL.mdis a store symlink needing ahome-manager switch. Pull without switching and the narrowed resolver is live while the deployed prose still describes the old rule.3d0b77e5) and raisedMAX_BYTESto 27,000, so the headroom is 236 B, not the 7 B an earlier note claimed.Docs only β one file, no code, no tests, no behaviour.
π€ Generated with Claude Code
https://claude.ai/code/session_01Emy2y7tGJjRee3GV2cNuaJ