skills(running-tend): nightly defers repo-wide CI breakage to tend-ci-fix#3320
Open
worktrunk-bot wants to merge 1 commit into
Open
skills(running-tend): nightly defers repo-wide CI breakage to tend-ci-fix#3320worktrunk-bot wants to merge 1 commit into
worktrunk-bot wants to merge 1 commit into
Conversation
…d-ci-fix Run 28357689944 timed out at the 3600s session cap after shipping #3318 (merged) then chaining a second cache-apt-pkgs-action fix PR, stranding the branch fix/ci-cache-apt-pkgs-pin-sha (pushed, no PR). Add a nightly-reference rule: ship the sweep PR, monitor only its CI, and leave unrelated repo-wide CI breakage to the dedicated tend-ci-fix workflow. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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.
What
Adds a "Session Budget — Don't Chain a Second CI-Fix PR" rule to the nightly reference (
.claude/skills/running-tend/references/nightly-cleaner.md).Why
The nightly run 28357689944 timed out at the hard 3600s session cap (supervisor:
status=timeout elapsed=3601s claude_exit=143), which is why it shows asfailurewith$0.00/ 0 tokens in the token report — the SIGTERM pre-empted the result event.Reconstructing the session log, the run did productive work and then over-reached:
fix(statusline): treat COLUMNS=0 as no width) — merged by the maintainer at 08:58. ✅code-coveragefailing on an apt-install step, correctly traced it to an upstreamawalsh128/cache-apt-pkgs-actiontag regression ("Packages argument is empty"), and decided to open a second PR to SHA-pin the action across four.github/sites.git add-ing that second change. The branchfix/ci-cache-apt-pkgs-pin-shawas pushed but no PR was ever opened — the diagnosis and the fix are stranded on an orphan branch.A repo-wide
code-coveragebreak is squarelytend-ci-fix's job: that workflow triggers on anyci-workflowfailureonmain, and a failedcode-coveragejob produces afailureconclusion (non-required is a merge-protection concept, not a workflow-conclusion one). The nightly should have noted the breakage and finished, leaving the fix to the dedicated workflow.Gate assessment
running-in-ci"End the turn only when work is shipped" guidance covers the backgrounding variant but not the second-deliverable / hard-timeout variant — this fills that specific gap without duplication.Note for the maintainer (out of this PR's scope)
The breakage the nightly diagnosed may still be live:
ci.yamlandnightly.yamlonmainnow pincache-apt-pkgs-action@v1.6.2(via merged dependabot #3311) whiletest-setupstill pins@v1.6.1. The ready-made fix sits unmerged onfix/ci-cache-apt-pkgs-pin-sha.tend-ci-fixshould pick this up on the nextfailure-concludedmainCI run (recent ones werecancelledby rapid pushes, so it kept skipping). Flagging in case you want to open that PR or rerun a cleanmainbuild to givetend-ci-fixa trigger.🤖 Generated by the tend review-runs workflow (run).