| name | debug-everything |
| description | Use when the user wants a comprehensive multi-issue health pass — fix all bugs, resolve broken tests, clear CI failures, fix lint and type errors, restore green builds — not for a single targeted bug fix. Diagnoses and resolves every outstanding open bug ticket plus all validation failures (test, lint, type, format, infrastructure) using orchestrated sub-agents with TDD discipline, in tier order. Trigger phrases include 'debug everything', 'fix all bugs', 'fix the failing tests', 'CI is failing', 'build is broken', 'clean up the codebase', 'restore green', 'project health pass'. |
| user-invocable | true |
| allowed-tools | Read, Write, Edit, Glob, Grep, Bash |
Requires Agent tool. If running as a sub-agent (Agent tool unavailable), STOP and return: "ERROR: /dso:debug-everything requires Agent tool; invoke from orchestrator."
Debug Everything: Full Project Health Restoration
You are a Senior Software Engineer at Google brought in to restore a project to full health. The project has accumulated bugs, test failures, lint errors, type errors, CI failures, and possibly infrastructure issues. In addition to validation failures, you must resolve ALL open ticket issues of type bug. Your mandate is simple: find every problem and fix it, using disciplined engineering practices.
Mindset
- You own everything. No "out of scope." Investigate and resolve all pre-existing failures using the fix-bug skill — "pre-existing" is a reason to fix, not skip.
- Diagnose before treating. Run all diagnostics first. Understand the full landscape of failures before fixing anything.
- TDD is selective, not reflexive. Behavioral bugs get a failing test BEFORE the fix. Mechanical fixes (imports, type annotations, config) rely on existing test coverage. See TDD Enforcement.
- Never skip tests. Disabling, skipping, or deleting tests is never acceptable.
- Fix in dependency order. Format > lint > type errors > unit tests > E2E > integration > infrastructure.
- One logical fix at a time. One fix, verify, commit. No batching unrelated fixes.
- Guard the orchestrator's context. The orchestrator is a coordinator, not a worker. Delegate ALL verbose operations (validation, diagnostics, auto-fixes, triage) to sub-agents. The orchestrator should only see compact summaries. Use
model: "haiku" for validation-only sub-agents to minimize cost.
Usage
/dso:debug-everything # Full diagnostic + fix cycle
/dso:debug-everything --dry-run # Diagnose only — create issues, no fixes
/dso:debug-everything --aws # Include proactive AWS infrastructure scan in Phase B
ci-pr mode (when dso.workflow=ci-pr): /dso:debug-everything runs in ci-pr mode. Both Bug-Fix Mode and Diagnostic Mode commit fix batches to per-chunk sub-branches (bug-batch/<session-id>/tier-<N>-batch-<K> for Diagnostic Mode tiers 2–7; bug-batch/<session-id>/bug-fix-batch-<K> for Bug-Fix Mode tier-7-only), each opened as a PR against SESSION_BRANCH (not main) and merged to the session branch after review. The session's aggregate progress is tracked in a Debug: draft PR. Phase B Step 1 creates a .debug-active marker (schema v1) on the repo root; Phase K removes it. The draft PR is also created in Phase B Step 1.
KNOWN GAP (tracked under upcoming remediation epic; post-mortem bug 576b-a6c7-3de3-4eef): in ci-pr mode, sub-branch PRs currently do NOT trigger internal LLM review. .github/workflows/per-branch-review.yml (which previously provided per-sub-PR review) was deleted by story 20d7-09d6-b831-4c3a. ci.yml's llm-review job is gated to base_ref == 'main' and does not fire on sub-branch PRs. /dso:review is HARD-GATED to no-op under dso.workflow=ci-pr. External advisory reviewers (CodeRabbit, Gitar) may run but are not required checks. Internal review coverage currently fires only at the eventual session→main PR, on the cumulative diff. The remediation epic restores per-sub-PR internal review with a no-duplicate-checks invariant.
Note on AWS CLI: The --aws flag controls only the proactive infrastructure scan in Phase B. When debugging Tier 6 infrastructure issues, AWS CLI is always available regardless of this flag.
If AWS auth is not configured, infrastructure checks are skipped gracefully.
Orchestration Flow
Step 0 (always first): GitHub Actions Pre-Scan — scans configured GHA workflows and creates bug tickets for untracked CI failures. Runs unconditionally before the open-bug-count pre-check so newly discovered failures are visible to mode selection. Skipped when debug.gha_scan_enabled=false or debug.gha_workflows is absent/empty.
Two entry modes that share the same bug-resolution loop (prompts/dispatch-fix-batch.md):
- Bug-Fix Mode — when open bug tickets exist, skip diagnostics/triage and chunk the open-bug list into tier-7 sub-branches via the shared loop, then enter Validation Mode (inner loop, bounded by
debug.max_fix_validate_cycles).
- Diagnostic Mode — when no open bugs exist, run Phase B diagnostic scan, Phase C triage, then fix in tier order (Phases E-I), funneling each tier's chunks through the same shared loop.
Both modes converge at Phase J (Full Validation). The outer loop (Phase B-J, max 5 cycles) and inner validation loop (Bug-Fix Mode, max debug.max_fix_validate_cycles) are independent and must not nest multiplicatively. The only structural difference between the two modes is that Diagnostic Mode does more to discover new bugs before entering the resolution loop; the loop itself is shared.
Bug-Fix Mode note: /dso:fix-bug runs as a two-phase parallel pipeline (prompts/dispatch-fix-batch.md): Phase 1 dispatches investigation-only sub-agents in parallel (each investigates inline and writes findings to ticket scratch under fix-bug:investigation); Phase 2 dispatches fix-application sub-agents in parallel (each loads scratch and runs Phases D–I). This split exists because sub-agents launched via the Agent tool cannot dispatch their own sub-agents — investigation must be inline within the sub-agent. The orchestrator's context sees only compact summaries; bulk investigation findings live in scratch. Conflict-aware grouping in Phase 2 prevents overlapping-file fixes from racing.
Migration Check
Idempotently apply plugin-shipped ticket migrations (marker-gated; no-op once migrated, never blocks the skill):
PLUGIN_SCRIPTS="${CLAUDE_PLUGIN_ROOT}/scripts"
bash "$PLUGIN_SCRIPTS/ticket-migrate-brainstorm-tags.sh" 2>/dev/null || true
bash "$PLUGIN_SCRIPTS/migrate-design-notes-to-design-md.sh" 2>/dev/null || true
Pre-Dispatch: Push Session Branch (worktree isolation fix)
Before dispatching any sub-agents, detect whether the orchestrator is running in a session worktree:
IS_SESSION_WORKTREE=$([ -f "$(git rev-parse --show-toplevel)/.git" ] && echo "true" || echo "false")
If IS_SESSION_WORKTREE=true, push session branch before first sub-agent dispatch:
git push -u origin HEAD
SESSION_HEAD=$(git rev-parse HEAD)
SESSION_BRANCH=$(git rev-parse --abbrev-ref HEAD)
Inject SESSION_BRANCH and SESSION_HEAD into each sub-agent's prompt (see worktree-dispatch.md).
Only push ONCE per invocation (not before every sub-agent). If IS_SESSION_WORKTREE=false (orchestrator on main), skip the push.
Phase A: GitHub Actions Pre-Scan (/dso:debug-everything)
Scan configured GitHub Actions workflows for CI failures and create bug tickets for any untracked failures. This step runs before the open-bug-count pre-check so newly discovered failures are visible to mode selection.
Stale .debug-active Cleanup: Before running the GHA pre-scan, clean up any stale .debug-active marker. The orchestrator executes the following bash block directly via the Bash tool — it is not pseudocode and is not extracted to a shell script; the LLM orchestrator runs it inline at Phase A entry:
_debug_marker="$(git rev-parse --show-toplevel)/.debug-active"
if [[ -f "$_debug_marker" ]]; then
_schema_ver=$(grep '^schema_version=' "$_debug_marker" 2>/dev/null | sed 's/schema_version=//' || true)
if [[ -z "$_schema_ver" || ! "$_schema_ver" =~ ^[0-9]+$ || "$_schema_ver" -lt 1 ]]; then
printf 'ERROR: .debug-active marker is from a pre-upgrade session. Remove it manually and re-run.\n' >&2
exit 1
fi
_marker_ts=$(grep '^debug-session-id=' "$_debug_marker" 2>/dev/null | cut -d= -f2 | cut -d- -f1,2 || true)
_ttl_hours=$(bash "$PLUGIN_SCRIPTS/read-config.sh" debug.session_ttl_hours 2>/dev/null || echo 24)
_ttl_hours=${_ttl_hours:-24}
_ttl_secs=$(( _ttl_hours * 3600 ))
_now=$(date +%s 2>/dev/null || echo 0)
_date_part=$(echo "$_marker_ts" | cut -d- -f1)
_time_part=$(echo "$_marker_ts" | cut -d- -f2)
_ts_iso="${_date_part:0:4}-${_date_part:4:2}-${_date_part:6:2} ${_time_part:0:2}:${_time_part:2:2}:${_time_part:4:2}"
_ts_epoch=$(date -d "$_ts_iso" +%s 2>/dev/null || date -j -f '%Y%m%d-%H%M%S' "$_marker_ts" +%s 2>/dev/null || echo 0)
if [[ "$_ts_epoch" -eq 0 ]]; then
printf 'Phase A: .debug-active marker timestamp unreadable — skipping stale check\n'
elif [[ $(( _now - _ts_epoch )) -ge $_ttl_secs ]]; then
rm -f "$_debug_marker"
printf 'Phase A: removed stale .debug-active marker (age > %d hours)\n' "$_ttl_hours"
fi
fi
Execute prompts/gha-dispatch.md with EPIC_COMMENT_LABEL="GHA scan complete". New tickets (tagged gha:<workflow-file-name>) are picked up by the open-bug-count check in Phase B Step 2 and processed in Bug-Fix Mode.
Phase B: Full Diagnostic Scan + Clustering (/dso:debug-everything)
Run ALL diagnostic checks and cluster related failures. The orchestrator runs only Step 1 (session lock). Everything else is delegated.
Step 1: Initialize & Acquire Session Lock (/dso:debug-everything)
The Bug-Fix Mode entry gate is Step 2 (below). When OPEN_BUG_COUNT > 0, skip this Step 1 setup entirely and route to Bug-Fix Mode.
When OPEN_BUG_COUNT == 0, execute prompts/session-init.md to bind:
REPO_ROOT, PLUGIN_ROOT, PLUGIN_SCRIPTSDISPATCH_ISOLATION (worktree isolation flag, applied to all sub-agent dispatches in Phases C/F/G/H/I/J/L/Validation Mode)MAX_FIX_VALIDATE_CYCLES (validation-loop bound, capped to [0, 10]; when 0, skip validation loop entirely and proceed directly to Phase J after Bug-Fix Mode)LOCK_ID (acquired via agent-batch-lifecycle.sh lock-acquire; persisted to $(get_artifacts_dir)/debug-lock-id for compaction recovery; released in Phase K)INTERACTIVE_SESSION (governs Non-Interactive Deferral Protocol behavior at each gate)
That prompt also runs the Resume Check (parse CHECKPOINT N/6 lines on in-progress issues; fast-close, re-dispatch, or revert per checkpoint progress).
Mode Detection & Draft PR (ci-pr mode only): After session-init.md binding, detect dso.workflow and initialize the debug session. The orchestrator executes this bash block directly via the Bash tool at Phase B Step 1:
DEBUG_MODE=$(.claude/scripts/dso read-config.sh dso.workflow 2>/dev/null || echo 'local')
DEBUG_MODE="${DEBUG_MODE:-local}"
_debug_marker="$(git rev-parse --show-toplevel)/.debug-active"
if [[ "$DEBUG_MODE" == 'ci-pr' ]]; then
SESSION_BRANCH=$(bash "$(git rev-parse --show-toplevel)/.claude/scripts/dso" resolve-session-branch.sh)
_rsb_exit=$?
if [[ $_rsb_exit -ne 0 ]]; then
printf 'ERROR: Phase B Step 1 SESSION_BRANCH resolution failed (exit %d) — cannot initialize ci-pr mode session\n' "$_rsb_exit" >&2
exit 1
fi
_session_id="$(date -u +%Y%m%d-%H%M%S)-$(set +o pipefail; LC_ALL=C tr -dc 'a-z0-9' </dev/urandom | head -c6)"
DRAFT_PR_URL=$(DRAFT_PR_TITLE_PREFIX=Debug: SESSION_BRANCH="$SESSION_BRANCH" \
PRIMARY_TICKET_ID="${EPIC_ID:-debug}" EPIC_TITLE='Debug Session' \
bash "$(git rev-parse --show-toplevel)/.claude/scripts/dso" create-sprint-draft-pr.sh 2>&1)
if [[ -z "$DRAFT_PR_URL" ]]; then
printf 'ERROR: Phase B Step 1 draft PR creation failed — cannot initialize ci-pr mode session\n' >&2
exit 1
fi
printf 'schema_version=1\ndebug-session-id=%s\n' "$_session_id" > "$_debug_marker"
printf 'dso.workflow=ci-pr — debug session running in ci-pr mode. Draft PR: %s\n' "$DRAFT_PR_URL"
else
printf 'dso.workflow=local — debug session running in local mode\n'
fi
Schema reference: ${CLAUDE_PLUGIN_ROOT}/skills/debug-everything/docs/debug-active-marker-schema.md
Step 2: BUG-FIX MODE GATE — Skip Diagnostics If Open Bugs Exist (/dso:debug-everything)
Check for open and in_progress bug tickets before launching the diagnostic scan. This is the Bug-Fix Mode entry gate:
_open_bugs=$((.claude/scripts/dso ticket list --type=bug --status=open 2>/dev/null | grep -o '"ticket_id"' || true) | wc -l)
_inprog_bugs=$((.claude/scripts/dso ticket list --type=bug --status=in_progress 2>/dev/null | grep -o '"ticket_id"' || true) | wc -l)
OPEN_BUG_COUNT=$((_open_bugs + _inprog_bugs))
- If
OPEN_BUG_COUNT > 0: Enter Bug-Fix Mode. Skip Phase B diagnostic scan (Steps 3, 4, 5, 6, 7) and Phase C triage entirely. Proceed to the Bug-Fix Mode section below. MUST NOT dispatch AskUserQuestion in this branch (guard 4369-a264 — instruction-locality co-locator for the fuller HARD-GATE at the Bug-Fix Mode header). The user invoking /dso:debug-everything IS the scope authorization regardless of OPEN_BUG_COUNT magnitude (1, 54, or 540). No "should I do P1 only / triage first / split this up?" prompt is valid here.
- If
OPEN_BUG_COUNT == 0: Continue to Step 3 (normal diagnostic flow).
Step 3: Context Budget Check (/dso:debug-everything)
Before launching diagnostics, estimate context load:
.claude/scripts/dso estimate-context-load.sh debug-everything 2>/dev/null | tail -5
If the static context load is >10,000 tokens, trim MEMORY.md before continuing to avoid premature compaction (per CLAUDE.md). Removing stale/redundant entries from MEMORY.md is sufficient — aim to bring the static load under 10,000 tokens before proceeding.
Step 4: Run Validation Gate (/dso:debug-everything)
Run validate.sh --ci to populate the validation state file. This serves two purposes:
- Seeds the gate state file so sub-agents aren't blocked by the validation gate hook
- Produces per-category pass/fail results that the diagnostic sub-agent can use to skip passing categories
.claude/scripts/dso validate.sh --ci 2>&1 || true
Bash timeout: Use timeout: 600000 (10 minutes — the TaskOutput hard cap). The smart CI wait in validate.sh can poll for up to 15 minutes, but the TaskOutput tool caps at 600000ms; use || true and check the state file for CI results if the call times out.
Note: The || true ensures we continue regardless of outcome — /dso:debug-everything is the skill that fixes validation failures, so it must not stop here.
Parse the state file to extract passing and failing categories:
source "${CLAUDE_PLUGIN_ROOT}/hooks/lib/deps.sh"
VALIDATION_STATE_FILE="$(get_artifacts_dir)/status"
The state file contains:
- Line 1:
passed or failed
failed_checks=<comma-separated list> (only present when failed)
Known check names: format, ruff, mypy, tests, migrate, ci, e2e. (docker only appears in an early-exit path, not in the normal failed_checks accumulator.)
Build the category lists:
- If line 1 is
passed: ALL categories passed. Set validatePassedAll = true.
- If line 1 is
failed: read failed_checks. Every known check NOT in that list passed. Store both lists:
validateFailedChecks: the comma-separated failed checks (e.g., ci,e2e)validatePassedChecks: the remaining checks that passed (e.g., format,ruff,mypy,tests,migrate)
Pass these to the diagnostic sub-agent in Step 7.
Step 5: Pre-Flight Infrastructure Check (/dso:debug-everything)
Before launching diagnostics, verify that Docker Desktop and the database are running. The diagnostic sub-agent runs E2E tests which require both.
$PLUGIN_SCRIPTS/agent-batch-lifecycle.sh preflight --start-db
The script outputs structured key-value pairs:
DOCKER_STATUS: running | not_runningDB_STATUS: running | started | stopped | failed_to_start | skipped
Exit 0 means all checks pass. Exit 1 means at least one check failed.
Action on failure:
DOCKER_STATUS: not_running → Release session lock (lock-release <lock-id> "Docker Desktop not running"), report to user: "Docker Desktop is not running. Please start it and re-run /dso:debug-everything." STOP.DB_STATUS: failed_to_start → Release session lock, report to user: "Database failed to start. Check Docker Desktop and run make db-start manually." STOP.- Both passing → proceed to Step 7.
Step 6: Check for Sprint Validation State (/dso:debug-everything)
Before launching the full diagnostic scan, check for a validation state file
written by /dso:sprint that indicates which categories already passed:
- Look for
/tmp/sprint-validation-*.json files. If multiple exist, use the
most recent one (highest generatedAt timestamp).
- If a fresh file exists (generatedAt < 1 hour old):
- Read
postBatchResults to identify categories that passed in the sprint's
post-batch validation.
- Read
ciFailure for the CI failure URL and failed job names.
- Read
batchInfo for changed files and task IDs — use these to focus the
diagnostic scan on likely failure sources.
- Set an internal flag:
sprintContext = true and store the passing
categories. Pass this context to the diagnostic sub-agent in Step 7.
- Log:
"Loaded sprint validation state from <file> — categories passing in sprint post-batch: <list>. Will focus diagnostics on failing categories."
- If no file exists or all files are stale (>1 hour): proceed with full
diagnostics. Set
sprintContext = false.
Step 7: Launch Diagnostic & Clustering Sub-Agent (/dso:debug-everything)
Launch a single sub-agent that runs all diagnostics, collects verbose output, clusters related failures, and returns a structured failure inventory.
Sub-agent prompt: Read $PLUGIN_ROOT/skills/debug-everything/prompts/diagnostic-and-cluster.md and use its contents as the sub-agent prompt.
If validatePassedAll = true: ALL validation categories passed. Append to the sub-agent prompt:
### Validation Gate Results (all passed)
validate.sh --ci reported ALL categories passing. Skip Step 1 (summary diagnostics)
and Step 7 verbose checks for format, ruff, mypy, unit tests, and e2e. Go directly
to Step 3 (tickets & git state) and Step 4 (clustering of any ticket bugs only).
Report 0 for all validation categories in the failure inventory.
If validatePassedAll = false (some checks failed): Append to the sub-agent prompt:
### Validation Gate Results (partial pass)
validate.sh --ci already ran these checks. Use these results to skip redundant work:
Passing categories (SKIP verbose diagnostics for these): {validatePassedChecks}
Failing categories (RUN verbose diagnostics for these): {validateFailedChecks}
In Step 1, you MAY skip running validate.sh --full --ci entirely since we already
have the summary. In Step 7, only run verbose error collection for the FAILING
categories. Report passing categories as count=0 in the failure inventory.
If sprintContext = true: ALSO append to the sub-agent prompt:
### Sprint Context (skip passing categories)
The following validation categories passed in a recent sprint post-batch check
and are unlikely to have regressed. You MAY skip running diagnostics for these
categories to save time, but MUST still run them if the CI failure log suggests
they might be involved:
Passing categories: {list from postBatchResults where value is "pass"}
CI failure URL: {ciFailure.url}
Failed CI jobs: {ciFailure.failedJobs}
Changed files in failing batch: {batchInfo.changedFiles}
Focus your diagnostic scan on the failing categories and the changed files.
If a "passing" category shows up in the CI failure log, run it anyway.
If --aws flag is set, append to the prompt:
### AWS Infrastructure Scan
Also run these commands and include results in the inventory:
if aws sts get-caller-identity &>/dev/null; then
aws elasticbeanstalk describe-environment-health --environment-name $EB_STAGING_ENV --attribute-names All 2>&1
aws logs tail /aws/elasticbeanstalk/$EB_STAGING_ENV --since 1h --filter-pattern "ERROR" 2>&1
else
echo "AWS auth not configured — skipping infrastructure checks"
fi
Subagent: subagent_type="general-purpose", model="opus" # Complex investigation: must correlate failures across validation categories, cluster related errors, and distinguish root causes from symptoms in complex output. (error-debugging:error-detective is NOT a valid subagent_type — the Agent tool only accepts built-in types. Use general-purpose with the prompt from the named agent file.)
The sub-agent returns: the path to the diagnostic file + a ≤15-line summary (category counts + top-3 clusters + open bug count). The full report is saved to $(get_artifacts_dir)/debug-diag.md on disk; do NOT receive the full report inline. Store the DIAGNOSTIC_FILE path for Phase C.
Flow control: If any inventory row has count > 0 OR open bugs exist, proceed to Phase C. Only skip to Phase K if ALL validation categories pass AND zero open bugs.
Bug-Fix Mode (/dso:debug-everything) — sub-branch routing: ci-pr mode commits fix-bug output to current sub-branch; local mode commits to session branch
Entry condition: Open bug tickets detected in Step 2 (OPEN_BUG_COUNT > 0).
Rationale: When open bug tickets already exist, the diagnostic scan (Phase B) and triage sub-agent (Phase C) are unnecessary — they exist to discover new issues. Bug-Fix Mode skips both and applies /dso:fix-bug directly to each known ticket. All bugs are in scope — including pre-existing ones. "Pre-existing" means the bug existed before this session; it does not mean the bug should be skipped or deferred. Every open bug ticket must be investigated and resolved via /dso:fix-bug.
**Bug-Fix Mode entry — two prohibited drift patterns.** The orchestrator MUST observe both before doing anything else in this mode:
-
DO NOT ask the user to confirm or narrow scope (guard dd88-afb6 + 5554-9125). When OPEN_BUG_COUNT is large (e.g., N=53), the instinct to ask "P1 only, P1+P2, all 53, or triage duplicates first?" is exactly the failure mode this gate prevents. Scope is fixed by the skill's contract: every open bug, in priority order, until graceful shutdown is authorized by a literal compaction-event banner. The user invoking /dso:debug-everything IS the scope authorization. No AskUserQuestion between Phase A and Bug-Fix Mode Step 2 is valid. Begin processing immediately.
-
Resolve every ticket through the two-batch fix-bug pipeline (guard jira-dig-1662). For each chunk, the orchestrator's job is to launch a Phase 1 investigation-only batch, await its compact summaries, then launch a separate Phase 2 fix batch — reading $PLUGIN_ROOT/skills/fix-bug/SKILL.md inline and following it as a script. The orchestrator coordinates the two batches; fix-bug's sub-agents do the investigation in Phase 1 and the fix in Phase 2. Structural invariant: the orchestrator MUST NOT produce code, patches, or fixes itself — that work belongs to the Phase 2 fix-batch sub-agents.
Companion guards: f9b5-213b (Phase K premature shutdown drift). All three drift patterns share root cause: orchestrator pattern-matches toward "make progress fast" and skips the contract.
What is skipped in Bug-Fix Mode
- Diagnostic scan skipped (Phase B Steps 3, 4, 5, 6, 7): No
validate.sh --ci, no preflight checks, no diagnostic sub-agent, no clustering.
- Triage skipped (Phase C): No triage sub-agent dispatch, no new epic creation, no issue clustering.
<COMPACTION_RESUME>
If resuming after an auto-compact event in Bug-Fix Mode: re-establish OPEN_BUG_COUNT (canonical bash, see step 1 below — never filter ticket list --type=bug output via Python t.get('type') == 'bug' because --type=bug strips the type field), re-read $PLUGIN_ROOT/skills/fix-bug/SKILL.md inline, and let fix-bug resume itself from its own most recent CHECKPOINT comment. Fix-bug owns its per-ticket compaction-resume protocol — do not duplicate that logic here. After the in-progress ticket completes, do NOT stop — re-query remaining open and in_progress bugs and continue processing them in priority order. Compaction that triggered this resume is historical state, not a Phase K shutdown trigger.
Sub-branch recovery (ci-pr mode): After compaction detection, before any commit or fix-bug dispatch, read the most-recent DEBUG_BRANCH_TRACKING: comment to recover the active sub-branch:
_tracking=$(.claude/scripts/dso ticket comment-list <epic-id> 2>/dev/null | grep '^DEBUG_BRANCH_TRACKING: ' | tail -1)
Parser uses exact-prefix match: 'DEBUG_BRANCH_TRACKING: ' (note the trailing space) — NOT substring match; does NOT match WORKTREE_TRACKING: or any other sprint anchor. This exact-prefix isolation is a regression test requirement: the parser must not accidentally match sprint WORKTREE_TRACKING: comments.
- If found: parse
sub_branch=<name> from _tracking and checkout that branch before any work.
- If not found: proceed on session branch (no active sub-branch to recover).
</COMPACTION_RESUME>
Bug-Fix Mode Execution
**No pre-committed scope reduction (guard d260-f307).** Before iterating the loop, the orchestrator MUST NOT pre-decide to truncate the batch based on context-budget reasoning. The following phrasings are PROHIBITED — observed in past sessions and codified here so they cannot be reframed:
- "I'll commit per fix and stop if context tightens." (no — there is no internal context-fill measurement; the rationale is fabricated)
- "Top N actionable then stop, reporting status." (no — N is determined by the bug list, not by a vibe-based budget)
- "Pace through; report at compaction." (no — pacing is the harness's job, not yours; just execute)
- "Be strategic about which to fix." (no — strategy is fixed: priority order, every ticket)
- "This will exhaust context quickly, so let me batch." (no — see f9b5-213b at Phase H Step 13)
Begin the loop. Process each ticket via /dso:fix-bug per the steps below. Continue until either (a) every ticket has been processed, or (b) a literal context-compaction-event banner from Claude Code arrives — at which point Phase K → Phase L → Phase M is mandatory. No third option exists. (Companion: f9b5-213b extended below; same axis of drift as 5554-9125 / jira-dig-1662 / 4369-a264 — orchestrator pattern-matches toward "manage cost / shorten work" rather than executing the contract.)
-
List all open and in_progress bug tickets:
{ .claude/scripts/dso ticket list --type=bug --status=open; .claude/scripts/dso ticket list --type=bug --status=in_progress; } 2>/dev/null
Collect all returned ticket IDs (deduplicate by ticket_id in case a ticket appears in both queries). Order by priority (P0 first, then P1, P2, P3, P4).
-
Apply Phase G's chunking + dispatch loop to the open-bug list (shared with Diagnostic Mode):
Bug-Fix Mode and Diagnostic Mode share the bug-resolution loop. Both paths chunk the bug list into sub-branches and delegate each bug to /dso:fix-bug via prompts/dispatch-fix-batch.md. The only difference between the modes is that Diagnostic Mode's Phases B–C–D–F populate the list (discovery, triage, safeguard analysis, auto-fix of tiers 0–1) before this step; Bug-Fix Mode enters with the list already populated as pre-existing open bug tickets (tier 7).
Chunking — apply the algorithm from Phase G § Sub-Branch Chunking (ci-pr mode):
- Group the open-bug list into sub-branches of at most 5 bugs each, preserving priority order (P0 → P1 → P2 → P3 → P4 within tier 7).
- File-conflict split: bugs in the same chunk that modify the same file are split into separate sub-branches.
- Large-fix exception: a single bug whose estimated diff exceeds 500 lines gets its own sub-branch with a
-large suffix.
- Branch naming:
bug-batch/<debug-session-id>/bug-fix-batch-<K> (analogous to Diagnostic Mode's tier-<N>-batch-<K>; Bug-Fix Mode uses bug-fix-batch-K because there is only one tier in scope here — tier 7).
- Branch base: create off SESSION_HEAD (
git checkout -b <name> while on the session branch at SESSION_HEAD).
- Serialized integration: the orchestrator MUST serialize sub-agent integration into the shared sub-branch ref — no concurrent push; dispatch one sub-agent at a time for integration, or use a lock mechanism (mirrors Phase G § Serialized integration rule).
- Record a tracking comment per sub-branch:
DEBUG_BRANCH_TRACKING: sub_branch=<name> tier=7 batch=<K> timestamp=<UTC>.
In local mode (DEBUG_MODE=direct or absent): no sub-branch chunking; commit each fix-bug result directly to the session branch as before.
Per-chunk dispatch — for each chunk, execute prompts/dispatch-fix-batch.md (the shared loop):
- dispatch-fix-batch.md runs the chunk as TWO sequential sub-agent batches: first a Phase 1 investigation-only batch (one sub-agent per bug,
MODE: investigation-only, writes findings to fix-bug:investigation scratch); then, after that batch returns, a separate Phase 2 fix batch (one sub-agent per fixable bug, consumes the scratch findings and applies RED test + fix + commit). Launch the investigation batch, await its compact summaries, then launch the fix batch.
- The prompt also handles pre-batch checks (
agent-batch-lifecycle.sh pre-check), the MAX_AGENTS protocol, task claim, blackboard write, and file-ownership context. Each batch sub-agent enforces fix-bug's own HARD-GATE (intent search, complexity scoring, RED test, Phase D approval); debug-everything does NOT re-implement those gates or pre-write fixes.
PROHIBITED (jira-dig-1662) — the orchestrator MUST NOT investigate bugs, read source files related to bugs, write code, or produce fixes itself. These are fix-bug's responsibilities, not the orchestrator's.
Per-chunk PR + integration (ci-pr mode):
- After all fix-bug calls in the chunk return, push the sub-branch and open a PR against
SESSION_BRANCH (NOT main): gh pr create --base "$SESSION_BRANCH" --head <sub-branch> --title "bug-fix: batch-<K>" --body "Bug-Fix Mode chunk <K>". Use the Phase G assert-batch-branch.sh invariant pre-flight before gh pr create.
- Await CI; classify outcome per Phase G's MERGED / ESCALATED / ERROR rules (lines 842–847). ESCALATED writes a
SUBBRANCH_ESCALATED: ticket comment and updates BLOCKED_SUBBRANCHES: PR annotation, but does NOT halt the loop — continue to the next chunk.
- After each chunk merge, run session-leakage detection:
STORY_BRANCH_PREFIX=bug-batch/ bash "$PLUGIN_SCRIPTS/detect-session-leakage.sh" (non-fatal).
-
Error handling: If /dso:fix-bug fails for a ticket (unrecoverable error, repeated failure, or explicit escalation), write a CHECKPOINT note and continue to the next ticket:
.claude/scripts/dso ticket comment <id> "CHECKPOINT: Bug-Fix Mode — fix-bug failed: <error>. Resume from: re-attempt fix."
Do NOT abort Bug-Fix Mode when a single ticket fails — process all remaining tickets.
After each fix-bug invocation (whether it succeeded or failed), run session-leakage detection (non-fatal):
STORY_BRANCH_PREFIX=bug-batch/ bash "$PLUGIN_SCRIPTS/detect-session-leakage.sh" 2>&1 || true
Closed-bug-trailer race protection: When a sub-agent commit references a ticket closed during parallel batch execution, the commit's DSO-Bug: trailer references a closed ticket. Route to --force-route-to-pending state: log SHA + closed-ticket-id and flag for post-session manual attribution rather than stalling integration. This is the closed-bug-trailer race condition; handle it gracefully without aborting the loop.
-
After all bug tickets have been attempted, run the Between-Batch GHA Refresh before proceeding to Validation Mode.
Between-Batch GHA Refresh (Bug-Fix Mode)
Run unconditionally after all bug tickets in the current batch have been attempted and before Validation Mode entry — regardless of MAX_FIX_VALIDATE_CYCLES (including 0).
Execute prompts/gha-dispatch.md with EPIC_COMMENT_LABEL="GHA between-batch scan". After the scan completes, re-query open + in_progress bug tickets:
{ .claude/scripts/dso ticket list --type=bug --status=open; .claude/scripts/dso ticket list --type=bug --status=in_progress; } 2>/dev/null
Use the refreshed list for the Validation Mode entry decision.
- Proceed to Validation Mode.
Validation Mode (/dso:debug-everything)
Entry condition: Entered after Bug-Fix Mode completes one full pass over all open bug tickets.
Purpose: Detect failures newly exposed by bug fixes (regressions or previously hidden issues), create tickets for them, and loop back to Bug-Fix Mode — up to MAX_FIX_VALIDATE_CYCLES iterations.
Scope: This is an INNER loop within the Bug-Fix Mode → Phase J flow. It is bounded by debug.max_fix_validate_cycles (configured at session start). The outer Phase B→J loop is separate and bounded by Phase J's 5-cycle safety limit. These loops are independent and must NOT be conflated.
Step 1: Check Iteration Count
Initialize on first entry: VALIDATION_ITERATION=1
On each re-entry (looping from Bug-Fix Mode): VALIDATION_ITERATION=$((VALIDATION_ITERATION + 1))
Persist iteration count as an epic ticket comment for resume continuity:
.claude/scripts/dso ticket comment <epic-id> "VALIDATION_LOOP_ITERATION: ${VALIDATION_ITERATION}/${MAX_FIX_VALIDATE_CYCLES}"
On resume (Step 1 resume check), parse existing VALIDATION_LOOP_ITERATION: comments to restore VALIDATION_ITERATION and MAX_FIX_VALIDATE_CYCLES.
Edge-case normalization (apply once, before Step 1 iteration logic):
- Value
<= 0 (zero or negative): set MAX_FIX_VALIDATE_CYCLES=0; skip validation loop entirely and proceed directly to Phase J.
- Value
> 10: set MAX_FIX_VALIDATE_CYCLES=10 (capping at 10); log "WARNING: max_fix_validate_cycles exceeds cap of 10 — capped to 10".
- Non-numeric value: default to 3; log
"WARNING: max_fix_validate_cycles not numeric — defaulting to 3".
If MAX_FIX_VALIDATE_CYCLES <= 0: Skip validation loop entirely. Proceed directly to Phase J.
Step 2: Run Diagnostic Scan After Bug-Fix
Reuse the same diagnostic sub-agent pattern as Phase B. Dispatch a diagnostic sub-agent to scan for newly exposed failures:
Sub-agent prompt: Read $PLUGIN_ROOT/skills/debug-everything/prompts/tier-transition-validation.md and use its contents as the sub-agent prompt.
Subagent: Resolve via discover-agents.sh routing category test_fix_unit, model="haiku"
On scan failure (sub-agent error, timeout, or corrupt output): Log warning: "WARNING: Validation Mode diagnostic scan failed (iteration ${VALIDATION_ITERATION}/${MAX_FIX_VALIDATE_CYCLES}) — treating as failures remain". Decrement remaining iterations: MAX_FIX_VALIDATE_CYCLES=$((MAX_FIX_VALIDATE_CYCLES - 1)). Do NOT create tickets from partial or corrupt scan results. Proceed to Step 4.
Step 3: Create Tickets for Newly Discovered Failures
For each new failure discovered in the diagnostic scan, create a bug ticket — but deduplicate first.
Deduplication — before creating any new ticket, check for an existing open or in_progress bug ticket covering the same failure:
{ .claude/scripts/dso ticket list --type=bug --status=open; .claude/scripts/dso ticket list --type=bug --status=in_progress; } 2>/dev/null
Compare each discovered failure against:
- Tickets already created by Phase I regression detection
- Tickets from previous validation iterations (check
VALIDATION_LOOP_ITERATION: comments to identify those iterations)
- Tickets from original Phase C triage
If an open bug ticket already exists for a failure (by title similarity or matching error message), skip ticket creation — use the existing ticket. Only ONE ticket per unique failure across all iterations.
For genuinely new failures (no matching open ticket exists), create a ticket. Follow skills/create-bug/SKILL.md for title and description format:
CHANNEL=$(DSO_FILING_CONTEXT=debug-everything .claude/scripts/dso infer-detected-by.sh 2>/dev/null || echo "other")
BUG_CREATE_OUT=$(.claude/scripts/dso ticket create bug "[Component]: [Condition] -> [Observed Result]" -d "## Incident Overview ..." --tags "detected_by:$CHANNEL" 2>/tmp/ticket_create_stderr.tmp)
BUG_CREATE_ERR=$(cat /tmp/ticket_create_stderr.tmp); rm -f /tmp/ticket_create_stderr.tmp
NEW_TICKET_ID=$(echo "$BUG_CREATE_OUT" | tail -1)
if echo "$BUG_CREATE_ERR" | grep -q "does not match required pattern"; then
.claude/scripts/dso ticket edit "$NEW_TICKET_ID" --title="[Component]: [Condition] -> [Observed Result]"
fi
The title format MUST follow [Component]: [Condition] -> [Observed Result]. Do not proceed with a non-conforming title.
Collect newly created ticket IDs as NEW_BUG_TICKETS.
Step 4: Decide — Loop or Stop and Report
If VALIDATION_ITERATION >= MAX_FIX_VALIDATE_CYCLES:
Max fix-validate cycles reached. Do NOT loop back. Stop and report:
Max validation iterations (MAX_FIX_VALIDATE_CYCLES) reached — remaining issues reported as open tickets.
Open bugs remaining: <list ticket IDs and titles>
Log: "Max validation iterations (${MAX_FIX_VALIDATE_CYCLES}) reached — remaining issues reported as open tickets". Proceed to Phase J.
If new bugs were found AND VALIDATION_ITERATION < MAX_FIX_VALIDATE_CYCLES:
New failures discovered. Loop back to Bug-Fix Mode with the newly created tickets:
- Update
OPEN_BUG_COUNT to include NEW_BUG_TICKETS
- Return to Bug-Fix Mode (Step 2: process all open bug tickets by priority)
If no new bugs were found:
No new failures. The fix cycle is clean. Proceed to Phase J.
Phase C: Triage & Issue Creation (/dso:debug-everything)
Delegate ALL triage work to a sub-agent. The orchestrator passes the diagnostic report and receives back issue IDs.
Launch Triage Sub-Agent
Sub-agent prompt: Read $PLUGIN_ROOT/skills/debug-everything/prompts/triage-and-create.md and use its contents as the sub-agent prompt. Pass the diagnostic file path as context — do NOT append the full report inline:
DIAGNOSTIC_FILE: $(get_artifacts_dir)/debug-diag.md
If all validation categories passed but open ticket bugs exist, also append: All validation categories passed — only open ticket bugs need triage. Skip cluster cross-referencing (no validation failures to cluster). Assign all bugs to Tier 7.
If resuming an existing tracker, append: Existing epic ID: <epic-id>. Do NOT create a new epic. Link new issues to this epic with .claude/scripts/dso ticket link <issue-id> <epic-id> relates_to.
Subagent: subagent_type="general-purpose", model="sonnet" # Tier 2: must cross-reference failure clusters with existing issues and make severity/priority judgments for new issue creation
Orchestrator Actions After Sub-Agent Returns
- Parse the triage report: extract issue IDs, epic ID,
HAS_STAGING_ISSUES flag
- If
HAS_STAGING_ISSUES=true: record staging symptoms for Phase L verification
- Report triage summary to user:
- Total distinct failures discovered
- New issues created (with IDs and titles)
- Pre-existing issues found (with IDs)
- Epic ID
- Recommended fix order by tier
If --dry-run: Stop here. Output the full triage report and exit.
Phase D: Safeguard Bug Analysis (/dso:debug-everything)
After triage, identify which issues touch safeguarded files and route them through user-approval before fixing.
Step 1: Detect Safeguarded Issues (/dso:debug-everything)
Safeguarded file patterns (from CLAUDE.md rule:no-safeguard-edits):
${CLAUDE_PLUGIN_ROOT}/skills/**, ${CLAUDE_PLUGIN_ROOT}/hooks/**, ${CLAUDE_PLUGIN_ROOT}/docs/workflows/**.claude/settings.json, .claude/docs/**scripts/**, CLAUDE.md
For each issue from the Phase C triage report, check if the issue description, title, or root cause references files matching these patterns. Build two lists:
SAFEGUARD_BUGS: issues that require editing safeguarded filesNORMAL_BUGS: all other issues (proceed directly to Phase E)
If SAFEGUARD_BUGS is empty, skip to Phase E.
Step 2: Launch Analysis Sub-Agent (/dso:debug-everything)
Sub-agent prompt: Read $PLUGIN_ROOT/skills/debug-everything/prompts/safeguard-analysis.md and use its contents as the sub-agent prompt. Pass the SAFEGUARD_BUGS list (IDs and titles) and WORKTREE name as context.
Subagent: subagent_type="general-purpose", model="opus"
Complex investigation: must read safeguarded files, understand bug context,
and propose precise line-level fixes — requires deep code comprehension and judgment.
(error-debugging:error-detective is NOT a valid subagent_type — use general-purpose
with the named agent file loaded verbatim as the prompt.)
The sub-agent returns: path to proposals file + summary (count + per-bug one-liner).
Step 3: Present Proposals to User (/dso:debug-everything)
Non-interactive: apply Non-Interactive Deferral Protocol (see Phase B Step 1) using gate_name=safeguard_approval. Auto-defer ALL safeguard bugs; skip to Phase E with SAFEGUARD_BUGS removed from the fix queue (they remain open). Add a ticket comment to each deferred bug: .claude/scripts/dso ticket comment <id> "INTERACTIVITY_DEFERRED: gate=safeguard_approval — requires user approval of safeguard file edits; deferred in non-interactive session."
Interactive mode: Read the proposals file from disk. Present each proposal to the user:
SAFEGUARD BUG PROPOSALS (require approval per CLAUDE.md `rule:no-safeguard-edits`)
================================================================
[1] <bug-id>: <title>
File: <path> (lines <N>-<M>)
Risk: <level>
Change: <description>
Preview:
- <removed line>
+ <added line>
[2] ...
Wait for user response. The user may approve all, approve specific bugs, or defer.
Step 4: Route Approved Bugs (/dso:debug-everything)
- Approved bugs: Add to
NORMAL_BUGS list with the proposal as fix guidance.
.claude/scripts/dso ticket comment <id> "SAFEGUARD APPROVED: user approved editing <file>. Proposed fix: <description>"
- Deferred bugs: Leave open with note:
.claude/scripts/dso ticket comment <id> "SAFEGUARD DEFERRED: requires editing <file>, deferred by user."
Remove from the fix queue.
Proceed to Phase E with the combined list of normal + approved bugs.
Phase E: Fix Planning (/dso:debug-everything)
Fix Tiers (Dependency Order)
Fixes MUST be applied in this order. Each tier may resolve failures in later tiers.
Tier 0: Format (auto-fix with make format)
Tier 1: Lint — ruff auto-fix, then manual fixes for remaining
Tier 2: Type errors (mypy) — often root causes of runtime failures
Tier 3: Unit test failures — TDD required
Tier 4: E2E test failures — may require DB, browser
Tier 5: Integration test failures — may require external services
Tier 6: Infrastructure issues — AWS CLI always available for this tier
Tier 7: Open ticket bugs — pre-existing tracked bugs not covered by tiers 0-6
Tier 6 AWS CLI access: Sub-agents working on Tier 6 issues should use AWS CLI freely. If AWS auth is not configured, report and recommend aws sso login.
Tier 7: Ticket bugs that map to tiers 0-6 go in that tier instead. Only bugs that don't fit earlier tiers remain in Tier 7.
Tier 7 sub-categories (batch in this order within the tier):
- Code bugs (wrong behavior, missing persistence, logic errors): Use TDD flow. Debugger agent (sonnet).
- Infrastructure bugs (AWS config, Docker, deployment): Attempt with AWS CLI. If auth unavailable or manual steps required, update the bug with specific next steps and skip gracefully. Never silently skip.
- Investigation bugs (unknown root cause, needs analysis): Error-detective agent. Output is either a fix OR updated findings with concrete fix plan added to the bug description.
Priority ordering within Tier 7: P1 bugs first, then P2, then P3. Attempt all priorities. Stop only on session limits (context, time), not priority boundaries.
Consult prior session data: If {auto-memory-dir}/debug-sessions.md exists, read it. Use prior session outcomes to prefer agent types that succeeded for similar failure categories and flag recurring patterns.
Load bug classification rules: Read $PLUGIN_ROOT/skills/debug-everything/prompts/bug-accountability-guide.md now. You will need these rules in Phase L Step 2; loading here avoids loading them when context is tighter.
Research complex fixes: For any Tier 3+ fix where the failure involves unfamiliar library behavior, a non-obvious tradeoff, or an external system whose current behavior you cannot derive from the codebase — spawn a research sub-agent before choosing a fix strategy. Pass findings as a Research Context section in the fix sub-agent prompt. See ${CLAUDE_PLUGIN_ROOT}/docs/RESEARCH-PATTERN.md for trigger criteria, guardrails, and sub-agent prompt template.
After completing each tier, re-run the relevant diagnostics for subsequent tiers. Update the failure inventory. Close issues that resolved themselves.
Batch Planning Within Tiers
Within each tier, group independent fixes into batches sized by the MAX_AGENTS value from the pre-batch check:
-
Fixes that unblock other fixes (dependency order first)
-
Fixes affecting the most files/tests (largest blast radius)
-
Independent fixes that can be parallelized
-
File overlap assessment via static analysis + NxN conflict matrix:
For each candidate issue in the batch:
a. Extract seed file paths from the issue description/triage report.
For fully-qualified Python names (e.g., src.services.pipeline.process_document),
strip the trailing function/class/method components and convert only the module portion
to a file path (e.g., src/services/pipeline.py). Verify the path exists before using it.
b. Run python3 "$PLUGIN_SCRIPTS/analyze-file-impact.py" --root $REPO_ROOT/app <seed-files> to get # shim-exempt: internal orchestration script
files_likely_modified and files_likely_read for each candidate (timeout: 30s)
c. Graceful degradation: If analyze-file-impact.py is missing, errors, or times out,
fall back to text-based file extraction from issue descriptions (existing behavior).
Debug sessions must not break if static analysis is unavailable.
Batch conflict detection: read prompts/batch-conflict-matrix.md. Write-write conflicts defer the lower-priority issue; read-read overlap is allowed; conflicts are logged to stderr in CONFLICT_MATRIX: format.
Phase F: Auto-Fix Sub-Agent (Tiers 0-1) — one sub-branch per tier (ci-pr mode) (/dso:debug-everything)
Sub-Branch Creation (ci-pr mode only — when DEBUG_MODE=pr)
Phase F creates exactly one sub-branch per tier that produces changes. Tiers: Tier 0 (format/make format) and Tier 1 (lint/ruff --fix).
Resolve SESSION_BRANCH (ci-pr mode, fail-fast on failure):
SESSION_BRANCH=$(bash "$(git rev-parse --show-toplevel)/.claude/scripts/dso" resolve-session-branch.sh 2>&1)
if [[ $? -ne 0 ]]; then
printf '%s\n' "$SESSION_BRANCH" >&2
printf 'ERROR: Phase F SESSION_BRANCH resolution failed — cannot create sub-branch PR\n' >&2
exit 1
fi
- Branch naming:
bug-batch/<debug-session-id>/tier-0 and bug-batch/<debug-session-id>/tier-1
- Use direct
git checkout -b bug-batch/<debug-session-id>/tier-<N> (create-story-branch.sh appends // and cannot produce tier branch names)
- When a tier has 0 bugs/changes: no sub-branch is created AND no annotation is added to the aggregate draft PR for that tier (zero-bug tier skip)
- Apply all tier changes on the sub-branch, push to origin, then open a PR against
SESSION_BRANCH (not main): gh pr create --base "$SESSION_BRANCH" --head <sub-branch> --title "auto-fix: tier-<N>" --body "Auto-fix tier <N> changes". Sub-branch PR internal review is currently degraded under dso.workflow=ci-pr (see KNOWN GAP near top of this file); the cumulative review fires at the eventual session→main PR.
**Sub-branch invariant pre-flight (Flow C analog of bug da45-7d92-6c86-42bc)**: BEFORE invoking `gh pr create` for the tier sub-branch, run `assert-batch-branch.sh` from the session branch directory to confirm the bug-batch sub-branch has been created locally AND pushed to origin. Without the push, no PR can be opened and LLM Sub-Branch Review will be silently bypassed — the exact failure pattern that left every prior debug-everything bug-batch PR unreviewed.
.claude/scripts/dso assert-batch-branch.sh "bug-batch/${DEBUG_SESSION_ID}/tier-${N}" || {
echo "ABORT: bug-batch sub-branch invariant violated. Push the sub-branch to origin before opening the PR." >&2
exit 1
}
In local mode (DEBUG_MODE=direct) this is a silent no-op (exit 0). In ci-pr mode the orchestrator MUST NOT open a sub-branch PR if the branch was not created+pushed. Do NOT skip, soften, or work around this gate.
In local mode (DEBUG_MODE=direct or absent): no sub-branch created; commit directly to session branch.
Per-Sub-Branch CI Review (ci-pr mode only — Phase F)
KNOWN GAP (tracked under upcoming remediation epic; post-mortem bug 576b-a6c7-3de3-4eef): Per-sub-branch internal LLM review is currently NOT firing. .github/workflows/per-branch-review.yml was deleted by story 20d7. ci.yml's llm-review job is gated to base_ref == 'main'. /dso:review is HARD-GATED to no-op under dso.workflow=ci-pr. External advisory reviewers (CodeRabbit, Gitar) may run but are not required checks. Cumulative review fires only at the eventual session→main PR. Remediation epic restores per-sub-branch internal review.
Until remediation epic ships, sub-branches are merged into the session branch after CI passes (Hook Tests, Script Tests, lint) but without an enforced per-sub-branch internal LLM review gate:
- Return discriminated merge outcome for this sub-branch:
MERGED: CI review passed; merge sub-branch into session branch.ESCALATED: CI review failed or PR blocked; do NOT merge; write SUBBRANCH_ESCALATED: <sub-branch> reason=<...> ticket comment FIRST (ticket comment is the authoritative source of truth — COMPACTION_RESUME reconciles PR annotation from ticket comments on resume), then update aggregate draft PR BLOCKED_SUBBRANCHES: annotation SECOND; continue to next sub-branch (ESCALATED does not halt the tier loop — loop continues to next sub-branch after ESCALATED).
ESCALATED does NOT halt the tier loop — Phase F continues to the next sub-branch. Per-sub-branch outcomes (MERGED, ESCALATED) determine merge eligibility independently for each sub-branch.
After each batch merge in Phase F, run session-leakage detection (non-fatal):
STORY_BRANCH_PREFIX=bug-batch/ bash "$PLUGIN_SCRIPTS/detect-session-leakage.sh" 2>&1 || true
Launch Auto-Fix Sub-Agent
Sub-agent prompt: Read $PLUGIN_ROOT/skills/debug-everything/prompts/auto-fix.md and use its contents as the sub-agent prompt.
Subagent: Resolve via discover-agents.sh routing category code_simplify (see agent-routing.conf), model="sonnet"
Orchestrator Actions After Sub-Agent Returns
-
Verify the sub-agent's report
-
Close any issues resolved by auto-fix: .claude/scripts/dso ticket transition <id> open closed --reason="Fixed: resolved by auto-fix (format/lint)"
-
Update the failure inventory with remaining errors
-
CONTEXT ANCHOR: After the commit workflow completes, continue immediately at Step 5 below (Phase F). Do NOT stop or wait for user input after committing.
Read and execute ${CLAUDE_PLUGIN_ROOT}/docs/workflows/COMMIT-WORKFLOW.md inline. Do NOT use the /dso:commit Skill tool — nested skill invocations do not return control to the orchestrator.
-
Remaining ruff violations that couldn't be auto-fixed become sub-agent tasks in Phase G
Phase G: Sub-Agent Fix Batches — DEBUG_BRANCH_TRACKING written per sub-branch (/dso:debug-everything)
Shared with Bug-Fix Mode: the chunking algorithm and per-sub-branch CI loop below are the canonical bug-resolution loop. Bug-Fix Mode Execution step 2 invokes the same logic with tier 7 as its sole input (sub-branch suffix bug-fix-batch-K instead of tier-N-batch-K). Edits to the algorithm here are load-bearing for both modes.
Sub-Branch Chunking (ci-pr mode only — when DEBUG_MODE=pr)
Resolve SESSION_BRANCH (ci-pr mode, fail-fast on failure):
SESSION_BRANCH=$(bash "$(git rev-parse --show-toplevel)/.claude/scripts/dso" resolve-session-branch.sh 2>&1)
if [[ $? -ne 0 ]]; then
printf '%s\n' "$SESSION_BRANCH" >&2
printf 'ERROR: Phase G SESSION_BRANCH resolution failed — cannot create sub-branch PRs\n' >&2
exit 1
fi
In ci-pr mode, Phase G chunks Tier 2–7 bugs into sub-branches of at most 5 bugs each, preserving tier ordering. Branch naming: bug-batch/<debug-session-id>/tier-<N>-batch-<K> (e.g., tier-3-batch-1, tier-3-batch-2).
Chunking algorithm: tier-first ordering, then chunk each tier's bugs into groups of ≤5 bugs per sub-branch.
File-conflict split: when two bugs in the same tier-batch modify the same file, split them into separate sub-branches (still ≤5 per branch); splitting is iterative until no file conflicts remain within a batch. This file-conflict split produces additional sub-branches as needed.
Large-fix exception: a single bug whose estimated diff exceeds 500 lines (>500 lines) gets its own sub-branch with a -large suffix (e.g., tier-3-large). PRECEDENCE: large-fix exception wins over file-conflict split — a >500-line bug is routed to -large and removed from any file-conflict consideration. The large-fix takes precedence before conflict resolution applies.
Serialized integration: the orchestrator MUST serialize sub-agent integration into the shared sub-branch ref — no concurrent push; dispatch one sub-agent at a time for integration, or use a lock mechanism. This prevents race conditions on shared refs.
After creating each sub-branch, record a tracking comment using the actual batch index K:
.claude/scripts/dso ticket comment <epic-id> "DEBUG_BRANCH_TRACKING: sub_branch=<name> tier=<N> batch=<K> timestamp=<UTC>"
In local mode: commit directly to session branch (sub-branch chunking does not apply).
Per-Sub-Branch CI Review (ci-pr mode only — Phase G)
After each sub-branch is created and changes committed in ci-pr mode, push the sub-branch to origin, open a PR against SESSION_BRANCH, and wait for CI review before merging:
**Sub-branch invariant pre-flight (Flow C analog of bug da45-7d92-6c86-42bc)**: BEFORE invoking `gh pr create`, run `assert-batch-branch.sh` from the session branch directory with the per-batch sub-branch name. The gate confirms the bug-batch sub-branch was created locally AND pushed to origin — the two preconditions for opening a PR. Without this pre-flight, an orchestrator that skipped `git push` would attempt to open a PR for a branch that does not exist on GitHub.
.claude/scripts/dso assert-batch-branch.sh "bug-batch/${DEBUG_SESSION_ID}/tier-${N}-batch-${K}" || {
echo "ABORT: bug-batch sub-branch invariant violated. Push the sub-branch to origin before opening the PR." >&2
exit 1
}
In local mode (DEBUG_MODE=direct) this is a silent no-op (exit 0). In ci-pr mode the orchestrator MUST NOT open a sub-branch PR if the branch was not created+pushed. Do NOT skip, soften, or work around this gate — bypassing reproduces the Flow C silent-skip pattern.
- Open the sub-branch PR against
SESSION_BRANCH (not main): gh pr create --base "$SESSION_BRANCH" --head <sub-branch> --title "auto-fix: tier-<N>-batch-<K>" --body "Auto-fix tier <N> batch <K> changes". Internal LLM review is currently degraded under dso.workflow=ci-pr (see KNOWN GAP earlier in this file).
- Wait for CI test jobs (Hook Tests, Script Tests, ShellCheck, Lint) to complete before merging the sub-branch into the session branch.
- Return discriminated merge outcome for this sub-branch:
MERGED: CI review passed; merge sub-branch into session branch.ESCALATED: CI review failed or PR blocked; do NOT merge; write SUBBRANCH_ESCALATED: <sub-branch> reason=<...> ticket comment FIRST (ticket comment is the authoritative source of truth — COMPACTION_RESUME reconciles PR annotation from ticket comments on resume), then update aggregate draft PR BLOCKED_SUBBRANCHES: annotation SECOND; continue to next sub-branch (ESCALATED does not halt the tier loop — loop continues to next sub-branch after ESCALATED).ERROR: CI workflow failed to produce a result; same as ESCALATED handling but reason=ci-workflow-error; write SUBBRANCH_ESCALATED: <sub-branch> reason=ci-workflow-error ticket comment, then update BLOCKED_SUBBRANCHES: annotation; continue to next sub-branch.
ESCALATED does NOT halt the tier loop — Phase G continues to the next sub-branch. Per-sub-branch review outcomes (MERGED, ESCALATED, ERROR) determine merge eligibility independently for each sub-branch.
After each batch merge in Phase G, run session-leakage detection (non-fatal):
STORY_BRANCH_PREFIX=bug-batch/ bash "$PLUGIN_SCRIPTS/detect-session-leakage.sh" 2>&1 || true
For remaining failures (Tiers 2–7), launch sub-agent batches by executing prompts/dispatch-fix-batch.md. That prompt covers:
- Pre-batch checks (
agent-batch-lifecycle.sh pre-check [--db]) and the MAX_AGENTS protocol (unlimited / N / 0).
- Task claim (
ticket transition <id> in_progress) and SAFEGUARD APPROVED: known-solution detection.
- Blackboard write + per-agent
file_ownership_context build.
- Task tool dispatch with
/dso:fix-bug delegation, isolation: "worktree" when DISPATCH_ISOLATION=true, and triage-classification-as-pre-loaded-context.
Resolve subagent_type via the table in prompts/agent-routing-table.md.
Complete assembled Task prompt (individual bug — orchestrator MUST include all three sections):
/dso:fix-bug <bug-id>
### Triage Classification Context (pre-loaded — do not re-score)
Bug ID: <bug-id>
Triage tier: <tier-number>
Severity (from triage priority): <P0=critical/2pts | P1=high/2pts | P2=medium/1pt | P3=low/0pts>
Environment: <CI failure | staging | local — from triage report>
### File Ownership Context
{file_ownership_context}
The file_ownership_context value follows the format: You own: file1.py, file2.py. Other agents own: <task-id-X> owns file3.py. (empty string when the blackboard is unavailable).
Phase H: Post-Batch Checkpoint (/dso:debug-everything)
After ALL sub-agents in a batch return:
Step 1: Dispatch Failure Recovery (/dso:debug-everything)
Before verifying results, check whether any sub-agent Task call returned an infrastructure-level dispatch failure (no STATUS: line, no FILES_MODIFIED: line, error message references agent type or internal errors — as opposed to task-level failures where the agent ran but produced incorrect work).
For each dispatch failure:
- Retry with
subagent_type="general-purpose", same model and prompt. Log: "Dispatch failure for task <id> with subagent_type=<original-type> — retrying with general-purpose."
- If retry fails: escalate model (sonnet → opus) and retry once more with
subagent_type="general-purpose".
- If all retries fail: mark task as failed.
Dispatch failure retries are sequential (error recovery, not planned work) and do not count toward batch size limits.
Step 2: Worktree Integration (/dso:debug-everything)
When DISPATCH_ISOLATION=true, sub-agents in Phase G ran in isolated worktree branches — their changes are NOT on the session branch in the orchestrator's CWD. Before any subsequent step runs git diff (Step 4 file-overlap, Step 5 critic review, Step 10 semantic conflict check) or git commit (Step 11), each sub-agent's worktree changes MUST be integrated onto the session branch.
When DISPATCH_ISOLATION=true: For each sub-agent that returned successfully (including any that succeeded on retry from Step 1), follow skills/shared/prompts/single-agent-integrate.md to integrate its worktree changes back into the session branch. This mirrors the Bug-Fix Mode per-result integration pattern (see Bug-Fix Mode Execution step 2 above) so downstream git diff / commit operations in Phase H observe the combined batch changes.
When DISPATCH_ISOLATION=false: Skip this step — sub-agents wrote directly to the session branch and the changes are already visible via git diff in Step 4 and beyond.
Step 3: Verify Results (/dso:debug-everything)
For each sub-agent (including any that succeeded on retry), check the Task result:
- Did it report success?
- Were the expected files modified? (spot-check with Glob)
- Did it follow TDD? (check for new test files)
Step 4: File Overlap Check (Safety Net) (/dso:debug-everything)
Sub-agents may modify files beyond what their task description predicts. Run overlap detection on collected modified files (from Task results or git diff --name-only):
$PLUGIN_SCRIPTS/agent-batch-lifecycle.sh file-overlap \
--agent=<task-id-1>:<file1>,<file2> \
--agent=<task-id-2>:<file3>,<file4>
Zero conflicts → proceed to Step 10. One or more CONFLICT: lines → execute prompts/file-overlap-resolution.md (capture secondary agent diffs, revert, re-run secondaries one-at-a-time in priority order with conflict-resolution context). The same prompt covers the oscillation guard fired by repeat CONCERN outcomes.
Non-interactive: when INTERACTIVE_SESSION=false and a file-overlap conflict requires user escalation (e.g., unresolvable write-write conflict after re-run), defer the escalation — add an INTERACTIVITY_DEFERRED: gate=file_overlap ticket comment on the conflicting tasks and proceed to Step 10 with the lower-priority task reverted.
Step 5: Critic Review (Complex Fixes Only) (/dso:debug-everything)
Launch a critic sub-agent before committing when ANY trigger applies: model="opus" was used (complex multi-file bug), Tier 5–6, ≥3 files modified, or TDD was required (behavioral code change — not imports, annotations, or config).
Capture the diff first:
git diff --stat
git diff
Sub-agent prompt: read $PLUGIN_ROOT/skills/debug-everything/prompts/critic-review.md; replace {full_diff captured by orchestrator via \git diff`}with the actual diff. Subagent:general-purpose, model="sonnet"`.
PASS → Step 11.CONCERN → if valid: revert (git checkout -- <files>), reopen with concern noted for next fix attempt. If false positive: Step 11. On 2nd CONCERN for the same issue, the oscillation guard in file-overlap-resolution.md fires.
Non-interactive: when INTERACTIVE_SESSION=false and the oscillation guard fires (requiring user to choose between two diverging fix approaches), defer escalation to user — add an INTERACTIVITY_DEFERRED: gate=oscillation_guard ticket comment, revert the most recent attempt, and leave the ticket open for the next session.
Step 6: Validate via Sub-Agent (/dso:debug-everything)
Do NOT run validation directly in the orchestrator. Launch a validation sub-agent:
Sub-agent prompt: Read $PLUGIN_ROOT/skills/debug-everything/prompts/post-batch-validation.md and use its contents as the sub-agent prompt. Replace {list of files modified by batch} with the actual file list from this batch.
Subagent: Resolve via discover-agents.sh routing category test_fix_unit (see agent-routing.conf), model="haiku" # Tier 1: runs validate-phase.sh post-batch and relays output verbatim — pure command execution with one bounded LIKELY_CAUSE inference from provided file list
Step 7: Handle Failures (/dso:debug-everything)
| Sub-agent outcome | Action |
|---|
| Success + tests pass | .claude/scripts/dso ticket transition <id> open closed --reason="Fixed: <summary>" |
| Partial success | .claude/scripts/dso ticket comment <id> "Partial: <details>." |
| Failure | .claude/scripts/dso ticket transition <id> open then .claude/scripts/dso ticket comment <id> "Failed: <error>." |
| Regression | Revert changes (git checkout -- <files>), reopen, note regression |
FIX_RESULT: unresolved | Leave ticket OPEN. Add comment: .claude/scripts/dso ticket comment <id> "Investigated: <investigation_summary> — could not fix. <reason>". Surface in session summary under ESCALATED BUGS. |
Bug close constraint: Only close a bug when there is an actual code change that fixes it. NEVER close a bug with reason Escalated to user: — closing removes the bug from ticket list visibility, which is the opposite of escalation. When no code fix is possible:
- Add investigation findings as a ticket comment:
.claude/scripts/dso ticket comment <id> "Investigated: <findings> — no code fix possible."
- Leave the ticket OPEN (do NOT transition to closed)
- Surface the ticket in the session summary under the
ESCALATED BUGS section so the user sees it
Before closing with Fixed:, invoke the bug classifier (3-step process):
- Dispatch the
bug-classifier-haiku sub-agent with the ticket ID to classify the bug.
- Extract the slug from the agent result (
CLASSIFIER_OUTPUT=<slug>).
- Apply the tag:
CLASSIFIER_OUTPUT=<slug> .claude/scripts/dso classify-bug-at-closure.sh <ticket-id> "Fixed:"
Valid close example (after code fix):
.claude/scripts/dso ticket transition <id> in_progress closed --reason="Fixed: added comment_penalty to quality_helpers.py"
Invalid (prohibited — do NOT do this):
.claude/scripts/dso ticket transition <id> in_progress closed --reason="Escalated to user: code path is correct, no fix possible"
Step 8: COMPLEX Escalation Handling (/dso:debug-everything)
Scan each fix-bug sub-agent result for COMPLEX_ESCALATION: true. If absent in all results, proceed to Step 9.
If present in any result, execute prompts/complex-escalation-handler.md. That prompt parses the escalation report fields, applies the Non-Interactive Deferral Protocol when INTERACTIVE_SESSION=false, and otherwise re-dispatches /dso:fix-bug at orchestrator level with ### COMPLEX_ESCALATION Context pre-loaded so fix-bug skips to its own Phase D Step 1 (Fix Approval). All complex-escalated bugs are tracked in the COMPLEX_BUGS list for the session summary.
Step 9: Decision Log (/dso:debug-everything)
Record the batch decisions and outcomes on the epic for observability:
.claude/scripts/dso ticket comment <epic-id> "BATCH {N} | Tier {T}
Issues: {id1} ({status}), {id2} ({status}), ...
Agent types: {type} ({id1}), {type} ({id2}), ...
Model tier: {model}
Critic review: {PASS|CONCERN|skipped}
Outcome: {N} fixed, {M} failed, {K} reverted
Remaining in tier: {count}"
Step 10: Semantic Conflict Check (/dso:debug-everything)
Before committing, run the semantic conflict check on the combined diff:
git diff | python3 "$PLUGIN_SCRIPTS/semantic-conflict-check.py"
Parse the JSON output:
"clean": true — proceed with commit (Step 11)."clean": false — log conflicts, present to orchestrator for review. If any conflict has "severity": "high", revert the conflicting files and re-dispatch the responsible sub-agent. If all conflicts are medium/low, note them in ticket and proceed.
"error" field present — log warning, proceed with commit (graceful degradation). Semantic conflict check failure is non-fatal.
Step 11: Commit & Sync (/dso:debug-everything)
CONTEXT ANCHOR: After the commit workflow completes, continue immediately at Step 12 (Discovery Collection) below. Do NOT stop or wait for user input after committing.
Read and execute ${CLAUDE_PLUGIN_ROOT}/docs/workflows/COMMIT-WORKFLOW.md inline. Do NOT use the /dso:commit Skill tool — nested skill invocations do not return control to the orchestrator, causing the debug-everything workflow to stall waiting for user input.
Blackboard cleanup: After the commit, run write-blackboard.sh --clean to remove the blackboard file:
.claude/scripts/dso write-blackboard.sh --clean
If blackboard cleanup fails, log a warning and continue — cleanup failure is non-fatal and must not block the next batch or graceful shutdown.
Step 12: Discovery Collection (/dso:debug-everything)
After the commit completes and before launching the next batch, collect discoveries from sub-agents:
DISCOVERIES=$(.claude/scripts/dso collect-discoveries.sh --format=prompt)
If discoveries exist (non-empty and not just "None."), inject the ## PRIOR_BATCH_DISCOVERIES section into the next batch's sub-agent prompts by appending it to the fix-task prompt context.
If collect-discoveries.sh fails, log a warning and proceed without discovery propagation (graceful degradation).
Step 13: Continuation Decision (/dso:debug-everything)
Default is CONTINUE, not shutdown. Only shut down on a concrete, verifiable signal — never on a "felt sense" of context fullness (a54a-95fc, f9b5-213b).
- If you received a literal context-compaction event banner from Claude Code during this session → Phase K (graceful shutdown). CRITICAL: On compaction, LOCK_ID may be lost from context. Recover it from the artifact file before Phase K:
LOCK_ID=$(cat "$(get_artifacts_dir)/debug-lock-id" 2>/dev/null). Phase K MUST release the lock and write epic summary notes — these are the two obligations that prior sessions lost after compaction.
- If more failures remain in this tier → Phase G (next batch)
- If tier is clear → Phase I (re-diagnose)
Do NOT shut down based on an internal estimate of session context usage. There is no way to self-measure context fill. If no compaction event occurred, keep fixing bugs.
Anti-pattern guard (f9b5-213b) — premature shutdown is bypass, not graceful shutdown. The following are all PROHIBITED shutdown rationales:
- "Context is at %, I should stop now." — there is no internal context-fill measurement; any percentage cited is fabricated.
- "Sub-agent dispatches are expensive, so few-bug-shutdown is responsible." — sub-agent budget is not a shutdown gate.
- "Many bugs remain (N=53) and I cannot fix them all this session, so I will report progress and stop." — partial completion is the normal state of multi-session work; report it via Phase K checkpoint AFTER Phase L, not before.
- "I committed the fixes; the user can merge themselves." — Phase L is REQUIRED (see Phase L section: "The
/dso:debug-everything command is NOT complete until changes are merged to main and CI passes"). Offloading Phase L to the user is incomplete execution, not graceful shutdown.
- "I'll commit per fix and stop if context tightens." (extension d260-f307) — same fabricated context-fill rationale as the first bullet, reframed as pacing.
- "Top N actionable then stop, reporting status." (extension d260-f307) — N is fixed by the ticket list, not a vibe-based budget.
- "Pace through; stop at the next compaction." (extension d260-f307) — pacing is the harness's responsibility, not yours.
Only a literal compaction-event banner authorizes shutdown. When that banner arrives, the path is mandatory: Phase K → Phase L → Phase M, no off-ramps.
Phase I: Re-Diagnose & Next Tier (/dso:debug-everything)
After completing a tier, re-validate to check for transitive resolutions.
Step 1: Launch Re-Diagnosis Sub-Agent (/dso:debug-everything)
Same pattern as Phase H Step 6, but run the full diagnostic set:
Sub-agent prompt: Read $PLUGIN_ROOT/skills/debug-everything/prompts/tier-transition-validation.md and use its contents as the sub-agent prompt.
Subagent: Resolve via discover-agents.sh routing category test_fix_unit (see agent-routing.conf), model="haiku" # Tier 1: runs validate-phase.sh tier-transition and relays structured output verbatim — no interpretation required, pass/fail reporting only
Step 2: Update Failure Inventory (/dso:debug-everything)
Compare sub-agent report against the inventory:
- Resolved without direct fix (transitive resolution) → close their issues
- New failures not in original inventory (regressions) → treat as P0
- Remaining failures → proceed to next tier
Step 3: Continue or Finish (/dso:debug-everything)
- If failures remain in higher tiers → return to Phase E
- If all tiers are clear → proceed to Phase J (Full Validation)
Phase J: Full Validation (/dso:debug-everything)
When all known issues across all tiers are addressed, delegate validation to a sub-agent.
CI is checked post-merge, not here. Phase J validates local code health only (format, lint, tests). CI runs on main, not the worktree branch — checking CI here would show the pre-fix state and produce a false failure. CI status is verified in Phase L after merging to main.
Launch Validation Sub-Agent
Sub-agent prompt: Read $PLUGIN_ROOT/skills/debug-everything/prompts/full-validation.md and use its contents as the sub-agent prompt.
Subagent: Resolve via discover-agents.sh routing category test_fix_unit (see agent-routing.conf), model="haiku" # Tier 1: runs validate-phase.sh full --skip-ci and relays structured output verbatim — pure command execution, ALL_PASS/SOME_FAIL is explicit in script output
Interpret Result
ALL_PASS + zero open bugs → Phase K (Completion)SOME_FAIL or open bugs remain → Return to Phase C (re-triage via sub-agent)
This is a remediation pass. Apply the same discipline: triage new failures, create issues, fix in tier order.
Safety bound: Maximum 5 full diagnostic cycles (Phase B→J loops). If the project is not healthy after 5 cycles, proceed to graceful shutdown and report to the user.
Phase K: Issue Closure & Graceful Shutdown (/dso:debug-everything)
Entry gate (f9b5-213b): Phase K is reachable from exactly two paths:
- Phase J reports
ALL_PASS AND zero open bugs (success path).
- A literal context-compaction event banner was received (graceful-shutdown path).
No other entry to Phase K is valid. In particular: "context usage feels high", "many bugs remain", "few commits made", "sub-agents are expensive", and "user can merge themselves" are all PROHIBITED entry rationales — they are the exact failure mode this gate prevents. If you find yourself reasoning toward Phase K without one of the two valid signals, STOP and return to Phase G/I/J.
On Success (All Checks Pass + Zero Open Bugs)
-
Clean up discoveries and release the session lock:
$PLUGIN_SCRIPTS/agent-batch-lifecycle.sh cleanup-discoveries
$PLUGIN_SCRIPTS/agent-batch-lifecycle.sh lock-release <lock-id> "All diagnostics passing, all bugs resolved"
.claude/scripts/dso ticket comment <epic-id> "Health restored."
.claude/scripts/dso ticket transition <epic-id> open closed
Discovery cleanup failure is non-fatal; log a warning and continue with lock release.
Run the final session-leakage sweep before removing the .debug-active marker:
STORY_BRANCH_PREFIX=bug-batch/ bash "$PLUGIN_SCRIPTS/detect-session-leakage.sh" 2>&1 || true
rm -f "$(git rev-parse --show-toplevel)/.debug-active" 2>/dev/null || true
-
Proceed to Phase L (Merge to Main & Verify).
On Graceful Shutdown
- Clean up discoveries and release the session lock:
$PLUGIN_SCRIPTS/agent-batch-lifecycle.sh cleanup-discoveries
$PLUGIN_SCRIPTS/agent-batch-lifecycle.sh lock-release <lock-id> "Graceful shutdown — work remains"
Discovery cleanup failure is non-fatal; log a warning and continue with lock release.
rm -f "$(git rev-parse --show-toplevel)/.debug-active" 2>/dev/null || true
- Do NOT launch new sub-agents.
- Stage modifications via
git status --short (do NOT run a full test/lint pass — make test-unit-only exceeds the tool timeout ceiling per CLAUDE.md rule:no-broad-tests-bash; the post-batch validation sub-agent in Phase H has already validated this batch).
- Commit checkpoint:
git add <modified files>
git commit -m "checkpoint: project health restoration — partial
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>"
- Update ALL in-progress issues:
.claude/scripts/dso ticket comment <id> "Session shutdown. Progress: <summary>. Next: <what remains>."
- Update the epic with remaining work summary:
.claude/scripts/dso ticket show <epic-id>
.claude/scripts/dso ticket comment <epic-id> "Graceful shutdown. Resolved: <N resolved>/<M total> issues. Remaining open: <list IDs and titles>. Next session should resume with .claude/scripts/dso ticket list."
The epic stays open so the next /dso:debug-everything session can resume it (see Epic Lifecycle).
- Commit partial work and proceed to Phase L (Merge to Main & Verify). After Phase L completes successfully, check context usage:
