Diagnose
Analyze a bug from error logs or reproduction steps, fix it, and create a GH Issue + PR.From its SKILL.md
npx -y skills add pillip/claude-dev-kit --skill diagnoseAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 11 stars11 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
SKILL.md
9.2 KB, as published. Nobody here has run it
Kit Preamble — diagnose
Kit Script Root
Kit root: ${CLAUDE_PLUGIN_ROOT}
- Absolute path above → plugin install (substituted at load time; no project
scripts/dir): prefix every kit script command with it, e.g.bash <kit-root>/scripts/checkpoint.sh …. Absolute paths also work from worktrees. - Literal
${…}placeholder above → standalone layout: run commands as written.
Project Context Detection
Run these checks silently at the start. Use results to adapt behavior:
[ -f issues.md ]— if true, this project uses the sprint system. Respect issue numbering and STATUS.md.[ -f docs/sprint_state.md ]— if true and Status showsrunning, a sprint is active. Be aware of parallel work in worktrees.[ -f docs/prd_digest.md ]— if true, read it for quick project context before starting.
Kit Rules
- Verify
gh auth statusbefore any GitHub operation.
Checkpoint Verification Pattern
Every phase has a checkpoint. Run the verification command and check the exit code.
- Exit non-zero (blocking gate): STOP immediately, report failure, do NOT proceed.
- Exit 0 with an
ADVISORY:line (advisory gate): report the gap, self-correct, continue. Standard prefix:
bash scripts/checkpoint.sh
Append --skill <name> --phase <phase> --issue <ID> for the specific check.
checkpoint.sh resolves the main repo root internally, so the command stays
a single prefix-matchable form (safe to allowlist as Bash(bash scripts/checkpoint.sh *)).
Worktree Setup Pattern
Pipeline skills operate in git worktrees to isolate changes from main.
- Create + freeze:
WT="$(bash scripts/wt_setup.sh <branch>)"— creates the worktree viascripts/worktree.sh createand writes.claude-kit/freeze-dir.txtinside it in a single step. - Resolve main root:
bash scripts/worktree.sh root - Remove safely:
bash scripts/wt_cleanup.sh <branch>— cd's to main root inside a subshell, then removes the worktree (never leaves CWD dangling). All file operations happen inside$WT/. Shared files live on main only.
Registry Update Pattern
Shared files (issues.md, STATUS.md, CHANGELOG.md) are managed on main only.
Always use registry_edit.sh for concurrent-safe writes — it resolves the
main repo root internally and delegates to flock_edit.sh:
bash scripts/registry_edit.sh issues.md -- bash -c '<update command>'
Never commit these files to feature branches.
Checkpoint Rules — MANDATORY
Every phase in this skill that has a CHECKPOINT block must be verified. Run the verification command after completing each phase. Blocking gates exit non-zero on failure: STOP immediately, report, do NOT proceed. Advisory gates always exit 0 and print an ADVISORY: line on failure: report the gap, self-correct, then continue (ISSUE-031). Never skip running either tier.
Slug convention: After creating the worktree, store the branch slug (e.g., fix/bookmark-none-subscript) for use in checkpoint commands.
Argument Validation (run before anything else)
- If
$ARGUMENTSis empty or blank, ask the user: "Please provide an error description, stack trace, or file path to diagnose." - Do NOT proceed until the user provides diagnostic context.
Steps:
- Ensure
ghauthenticated (gh auth status). - Gather the bug context from $ARGUMENTS (error message, stack trace, file path, or reproduction steps).
- If a file path is provided, read it. If an error message is provided, use Grep to locate the source.
- Trace the execution path from the error backward to identify the root cause.
- Form 1–3 ranked hypotheses and verify each by reading relevant code.
5.5) Self-Review (MANDATORY):
- Re-read the root cause hypothesis and the proposed fix.
- Trace backward from the fix: does it address the root cause, or just a symptom?
- Actively search for evidence that contradicts the hypothesis.
- Check all callers/consumers of the modified code for unintended side effects.
- List 3+ edge cases and verify the fix handles them.
- Rate confidence (High/Medium/Low).
- Low → gather more info, do NOT proceed.
- Medium → present uncertainty to user with specific questions.
- High → proceed to step 6.
- Present the confirmed root cause and a minimal fix to the user.
- After user approval, apply the fix.
- Run
pytestto confirm no regressions. Suggest a regression test if none exists. - Create worktree + auto-freeze in one step:
WT="$(bash scripts/wt_setup.sh fix/<slug>)"wt_setup.shcreates the worktree and writes the freeze marker inside.claude-kit/freeze-dir.txtatomically. Apply the fix inside$WT/, run tests from$WT/.
CHECKPOINT — ADVISORY (report & continue) Run:
bash scripts/checkpoint.sh --skill diagnose --phase worktree --issue "$SLUG"Advisory: exits 0 even on failure, printing anADVISORY:line — report the gap, self-correct, then continue.
CHECKPOINT — MANDATORY — NEVER SKIP Run:
bash scripts/checkpoint.sh --skill diagnose --phase test --issue "$SLUG"If exit code ≠ 0: STOP immediately and report the failure. Do NOT proceed.
- Create GH Issue:
gh issue create --title "fix: <concise bug description>" --body "<body>"- Body must include: error summary, root cause analysis, fix description, and affected files.
- Commit + push (from
$WT/).
CHECKPOINT — ADVISORY (report & continue) Run:
bash scripts/checkpoint.sh --skill diagnose --phase push --issue "$SLUG"Advisory: exits 0 even on failure, printing anADVISORY:line — report the gap, self-correct, then continue.
- Create PR:
gh pr create --title "fix: <concise bug description>" --body "Closes #<issue_number>\n\n<details>"
- Report the PR URL to the user — continue with
/reviewand/ship.
Sprint Integration (optional)
If issues.md exists in the project root, register this work in the sprint ecosystem:
- Read
issues.mdto find the next available ISSUE-NNN number. - Append a new issue entry via the registry wrapper:
Issue fields:bash scripts/registry_edit.sh issues.md -- bash -c '<append issue entry>'- Title: same as GH Issue title
- Track: platform
- Priority: P2
- Status: done (PR already created)
- GH-Issue:
<number> - PR:
<pr_url> - Depends-On: none
- This allows team-lead to track standalone skill work in sprint_state.md.
If issues.md does not exist, skip this step silently.
Error Handling
- If
gh auth statusfails: stop and instruct the user to rungh auth login. - If the error cannot be located in the codebase: ask the user for more context (full stack trace, reproduction steps).
- If multiple root causes are plausible: present all hypotheses ranked by likelihood and ask the user to help narrow down.
- If tests fail after fix: do NOT push or create PR. Report failing tests and stop.
Rollback
- Use
bash scripts/wt_cleanup.sh <branch>for safe worktree removal — the wrapper cd's to main root and removes the worktree in a single subshell. - If failure occurs after worktree creation but before PR:
bash scripts/wt_cleanup.sh <branch>git push origin --delete <branch>(remote cleanup, if pushed)
- If failure occurs after PR creation:
gh pr close <pr_number>then clean up worktree and branch as above.
Shared Registry Files
IMPORTANT: Never commit issues.md, STATUS.md, or CHANGELOG.md to the feature branch.
These are registry files managed only on main. Always use bash scripts/registry_edit.sh <file> -- bash -c '<update command>' — the wrapper resolves the main repo root internally.
Guidelines
- Never guess-and-patch. Always confirm the root cause before proposing a fix.
- Keep fixes minimal — do not refactor surrounding code.
- Always write a regression test that fails before the fix and passes after.
- Present hypotheses ranked by likelihood — let the user help narrow down if needed.
- Document the chain of causation in the GH Issue body for future reference.
Execution Principles (absorbed from the diagnostician persona — ISSUE-034)
- Trace backward from the error to the root cause before proposing any fix — symptom suppression creates new bugs.
- One bug per PR; do not refactor surrounding code or fix unrelated bugs in the same change.
- Write a regression test that fails before the fix and passes after. Never "fix" by suppressing errors (bare
except, emptycatch). - If multiple causes are possible, present them ranked by likelihood and ask before applying a fix. Document the causation chain in the issue body.