From 6d21c049a681fc9d3c24756c1faffcb4e52b97bc Mon Sep 17 00:00:00 2001 From: tend-agent Date: Mon, 8 Jun 2026 16:26:27 +0000 Subject: [PATCH] =?UTF-8?q?skills(triage):=20drop=20"run=20full=20suite"?= =?UTF-8?q?=20=E2=80=94=20defer=20to=20PR=20CI=20per=20"ship=20before=20en?= =?UTF-8?q?d=5Fturn"?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Triage Step 6.3's "Run the full test suite and lints" conflicts with running-in-ci's "End the turn only when work is shipped" (#661): a long suite kicked off before push pushes the agent into mid-wait end_turn and the deliverable never ships. Fold the targeted-test check into Step 2 and renumber. Closes a 2-occurrence pattern (May tend-mention #5741, June tend-triage on PRQL/prql#5988) where the agent finished real work, called ScheduleWakeup expecting resumption, and exited without commit/push. Co-Authored-By: Claude --- plugins/tend-ci-runner/skills/triage/SKILL.md | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/plugins/tend-ci-runner/skills/triage/SKILL.md b/plugins/tend-ci-runner/skills/triage/SKILL.md index bdbc9e00..c0c9df7a 100644 --- a/plugins/tend-ci-runner/skills/triage/SKILL.md +++ b/plugins/tend-ci-runner/skills/triage/SKILL.md @@ -105,9 +105,8 @@ Step 3's duplicate check catches identical fixes. It misses the *same root cause ### If fixing 1. Fix the root cause (not just the symptom) -2. Confirm the test now passes -3. Run the full test suite and lints (use project test commands from CLAUDE.md) -4. Create branch, commit, push, and create PR: +2. Confirm the reproduction test now passes — that targeted pass plus a clean compile is enough local confidence to ship. Leave the comprehensive suite to PR CI per `/tend-ci-runner:running-in-ci`'s "End the turn only when work is shipped"; backgrounding a long suite before push pushes the agent into mid-wait `end_turn` and the deliverable never ships. +3. Create branch, commit, push, and create PR: ```bash git checkout -b fix/issue-$ARGUMENTS git add -A @@ -129,7 +128,7 @@ Step 3's duplicate check catches identical fixes. It misses the *same root cause --- Closes # — automated triage" ``` -5. Monitor CI using the approach from /tend-ci-runner:running-in-ci. +4. Monitor CI using the approach from /tend-ci-runner:running-in-ci. ### If reproduction test works but fix is not confident