fix-bug

name

fix-bug

description

Diagnose an integration/feature that "should work but didn't" — hooks not firing, status not updating, callbacks not arriving, jobs not running, webhooks silent. Lead with the concrete runtime contract (endpoints, env vars, file paths, log locations) and the single fastest diagnostic before listing hypotheses. Run when the user says "this didn't trigger", "X didn't update", "the hook never fired", "should have happened but didn't", "/fix-bug", or reports a missing side effect from an action they performed. Accepts `mode=fast|balanced|production` to control depth (default: balanced); also accepts `include=` / `skip=` overrides. Skip for crashes with a stack trace, type errors, or test failures — those have their own evidence already.

User-question protocol: Whenever this skill needs the user to pick between options, confirm an action, or answer a multiple-choice prompt, you MUST call the AskUserQuestion tool to render a proper interactive picker. Do NOT print numbered options as plain text and wait for the user to type a number — that produces a degraded UX. Free-form questions (open-ended typing) may be asked in prose, but any time you would write "1) … 2) … 3) …", use AskUserQuestion instead.

fix-bug

Diagnose silent integration failures — code that runs without error but the expected side effect never happens. The bug is almost always in the seams: a hook that wasn't loaded, an env var that wasn't injected, a fail-soft || true that swallowed the real error, a shell-quoting mistake, a stale process.

The user has already done the hard part — they noticed the absence. Don't make them ask follow-up questions to extract information you could have given upfront.

---

When NOT to run

The description covers when to activate. The non-obvious filter:

Do NOT run when the failure already has a concrete error message, stack trace, or failing test — those have evidence; this skill is for the no evidence case where the side effect silently failed to occur.

---

Modes

This skill accepts a mode= argument. Default — when no mode= is specified — is balanced: the full 7-step workflow below.

Mode	Behavior
fast	Skip the full "Response shape for first message" structure. Trace, identify, patch. For known-cause bugs where the user has already named the suspected line, or trivial single-file fixes.
balanced (default)	Full 7-step workflow: trace → runtime contract → silent-failure grep → single diagnostic → ranked hypotheses → read evidence literally → propose fix.
production	balanced + after Step 7 (propose fix), invoke agentsystem-core:add-regression-test to pin the bug with a test that fails on the pre-fix code and passes on the post-fix code.

include= / skip= overrides. Add or remove specific steps — mode=fast include=regression-test runs the lean trace + adds a regression test; mode=production skip=regression-test runs the full diagnostic without pinning.

Mode safety override. If mode=fast is requested for a bug touching auth, payments, secrets, schema integrity, or external webhooks, surface the conflict via AskUserQuestion and confirm before honoring. Silent integration failures in those areas often have second-order causes the runtime contract surfaces — mode=fast skips that surfacing.

Phase-gated NEVER scope. When mode=fast is in effect, "NEVER list hypotheses without first stating the runtime contract" is suspended for that run — fast mode explicitly opts out of the contract-first structure. The remaining NEVERs (no overconfident assertion, no parallel-hypothesis test asks, no theorizing past pasted evidence, no ignoring silent-failure patterns, no assuming the running process has latest code) stay in force in every mode.

---

Workflow

Step 1 — Trace the integration end-to-end with file:line refs

Before forming hypotheses, map the actual runtime path:

1. Where the side effect is triggered (the user's action → handler).
2. Where the side effect is dispatched (the outbound call: HTTP, IPC, queue, file write, env injection).
3. Where the side effect is received (server handler, listener, hook target).
4. Where the side effect is observed (DB row, UI status, log line).

Use Explore/grep liberally; cite each link with file:line. If any link is missing — that's the bug, stop.

Library boundary as a seam. If a third-party library mediates any step (terminal emulator, parser, framework router, RPC client, ORM), treat the library as part of the seam. Name the exact entry point you call (e.g. term.attachCustomKeyEventHandler) and the exact behavior you assume (e.g. returning false stops further processing). Assumed library behavior is a hypothesis until verified against the library's source — not its README, not a code comment, not prior memory (see NEVER #7 below).

State store as a seam. If the integration boundary is a stack-specific state layer (a React reducer/context, Redux/Zustand store, Vue reactivity, SwiftUI @State, etc.), it is its own seam — with its own update mechanics that can drop or delay messages (lazy updaters, batching, stale closures, double-invocation in dev modes). Don't treat the store as a verified link just because it "ran". When such a layer is in the trace, check whether a stack-specific debug skill exists (e.g. agentsystem-core:debug-react) and route to it for that layer's known silent-failure patterns. The generic skill covers I/O seams; stack skills cover state-machine seams.

Step 2 — Surface the runtime contract UPFRONT

In your first response to the user, include all of these without being asked:

• Exact endpoint(s) hit — full URL template with substitutions, e.g. POST http://127.0.0.1:<port>/api/hooks/claude?taskId=<id>.
• Auth shape — header name, where the token comes from (Bearer ${apiToken} from settings DB at …).
• Required env vars — name, where injected (MC_TASK_ID, MC_API_URL, MC_API_TOKEN injected at pty-manager.ts:106-108).
• On-disk artifacts — config files written, where (<cwd>/.claude/settings.local.json).
• Log/observation locations — where output goes, or "no logs — errors are swallowed at file:line".
• The exact event name / payload shape the system expects (hook_event_name: "UserPromptSubmit").

Rule of thumb: if the user could plausibly ask "what endpoint?" / "what env var?" / "where is it written?" — you failed Step 2. Put it in the first message.

Step 3 — Identify silent failure points

Grep the dispatch path for swallowing patterns and call them out explicitly:

rg -n '\|\| true|catch\s*\{\s*\}|catch\s*\(\s*\w*\s*\)\s*\{\s*\}|\.catch\(\s*\(\s*\)?\s*=>\s*(undefined|null|void)' <dispatch-files>
rg -n '>/dev/null 2>&1|2>/dev/null' <dispatch-files>

For each hit, note: "errors here are silently dropped — the only way to know if it ran is to instrument it." This is almost always where the bug is hiding.

Step 4 — Ask for the ONE piece of runtime evidence that disambiguates

Don't present 3 hypotheses and ask the user to test all of them. Pick the single observation that splits the hypothesis space in half, and ask for it.

Before picking, ask yourself: "Which single observation, if I had it right now, would let me discard the most hypotheses?" That's the diagnostic to request — not the easiest, not the most familiar, the most disambiguating.

Integration type	Symptom keywords	Fastest diagnostic
HTTP / webhook	"endpoint silent", "callback never came", "POST didn't arrive"	Tail the receiver log while user repros — does the request arrive at all?
Claude Code hook	"hook didn't fire", "/hooks"	Run /hooks in the Claude session — it prints execution errors verbatim
Env-var injection	"empty value", "command not found in subprocess"	In the spawned shell: echo "$VAR1|$VAR2|$VAR3" — empty fields = injection failed
Config file write	"settings missing", "config not picked up"	Does <path> exist? cat it. Diff against expected shape.
Queue / job worker	"job never ran", "task stuck queued"	Is the worker process alive? Tail its log. Inspect the queue depth/dead-letter.
Cron / scheduled	"didn't run at "	Check the scheduler's log for a fire entry; verify the schedule expression and timezone.
File watcher	"save didn't trigger rebuild"	lsof or platform-specific watcher list — is the path actually being watched?
MCP / tool call	"tool returned nothing", "agent didn't call it"	Check the MCP server's stderr; verify the tool is exposed in the tools list.
Library boundary	"lib's API didn't behave as documented", payload-swap fixes don't stick	Open the installed JS source (node_modules/<pkg>/lib/... or dist/), grep the function name, read the branch your code actually hits — .d.ts files often misrepresent runtime behavior.

Pick the cheapest diagnostic that the user can run in <30 seconds.

If the diagnostic comes back ambiguous (e.g., "the file exists but I can't tell if the curl ran"): instrument the swallowed-error site identified in Step 3. Replace the >/dev/null 2>&1 / empty catch {} with a redirect to a log file (or console.error), have the user repro once, then read the log. Treat this as the next single diagnostic — do not branch into multiple parallel checks.

If the user can't run shell commands (locked-down env, dashboard-only access, hosted CI without exec): swap the shell diagnostic for a paste-or-screenshot ask — the relevant dashboard panel, log viewer entry, network tab response, or queue UI row. Name the exact field/column you need ("the Last delivery column for webhook X", not "send a screenshot of the page") so the single round trip still disambiguates.

Step 5 — Rank hypotheses by likelihood, mark confidence honestly

When listing possible causes, order by likelihood given the evidence so far, and flag uncertainty explicitly:

• ✅ "Confirmed by <evidence>" — when you've verified.
• 🟡 "Likely — <reason>" — when the code path supports it.
• ⚪ "Possible — depends on <thing I don't know>" — speculation.

If the user pushes back ("are you sure?"), do not double down. Re-read the relevant docs/code and revise.

Step 6 — When evidence arrives, read it literally

If the user pastes runtime output (a /hooks dialog, log line, error message), the bug is usually in the output verbatim, not in your hypothesis list. Read the literal text — shell-quoting bugs, missing env vars, 401 responses, and typos all show up as plain text in real evidence.

Example from a real session: a Claude Code /hooks dialog showed sh -c if [ -z … then exit 0; fi … with a syntax error. The bug was the ["sh","-c",script].join(" ") produced sh -c <unquoted-script> — Claude Code wraps command in /bin/sh -c itself, so the inner sh -c left the script unquoted. The error was visible in the first line of output; no hypothesis-testing needed.

Step 7 — Propose the fix

Once root-caused, edit the file and add one sentence the standard fix narration doesn't already cover: why the existing code looked plausible — what made the bug ship past review. Do not refactor surrounding code unless it's part of the fix.

---

Response shape for first message

When invoked, structure the first response as:

**Trace:** <action> → <dispatch site at file:line> → <transport: HTTP/IPC/file> → <receiver at file:line> → <observed state at file:line>

**Runtime contract:**
- Endpoint: <full URL template>
- Auth: <header + source>
- Env vars required: <list with injection site>
- On-disk: <files written, paths>
- Logs/observation: <where, or "swallowed at file:line">
- Expected event/payload: <shape>

**Silent failure points found:** <list of fail-soft sites with file:line>

**Most likely causes (ranked):**
🟡 <hypothesis 1> — would manifest as <observable>
⚪ <hypothesis 2> — depends on <unknown>

**Single fastest diagnostic:** <one command or observation>, run it and paste the output.

Adapt headings to the situation, but every section above should appear or be explicitly noted as N/A.

---

NEVER

• NEVER list hypotheses without first stating the runtime contract. Instead: Endpoint, env vars, file paths, log location go in message #1, before any "could be X, could be Y". Why: The user asking "what endpoint does it hit?" is a sign you front-loaded speculation over facts. Facts narrow the search; speculation expands it.

• NEVER assert hook/integration behavior with confidence unless you've verified it in this session. Instead: Mark with 🟡/⚪. If you said something with confidence and the user pushes back, re-verify; don't double down. Why: Tooling behavior (Claude Code hook approval, framework lifecycle, env propagation) varies by version. Overconfident claims send the user down dead-end branches and erode trust.

• NEVER ask the user to test 3 hypotheses in parallel. Instead: Pick the single observation that disambiguates fastest, ask for that, then iterate. Why: Each parallel test costs the user a context switch. One sharp diagnostic finishes the loop in one round trip.

• NEVER theorize past runtime evidence the user has already pasted. Instead: Read the literal output first. Shell errors, 4xx codes, empty env vars, missing files are usually verbatim in the paste. Why: The actual error beats any hypothesis. Skipping it wastes a round and signals you didn't read carefully.

• NEVER ignore silent-failure patterns (|| true, empty catch {}, >/dev/null 2>&1). Instead: Grep for them on the dispatch path and call them out as the prime suspect for "ran but didn't work". Why: These are the #1 cause of "code looks fine but the side effect never lands". They are also the easiest to fix temporarily for diagnosis (redirect to a log file).

• NEVER assume the running process has the latest code. Instead: For Electron main / long-lived workers / daemons, ask whether the process was restarted since the relevant code landed. HMR only reloads renderers. Why: Stale main-process code is invisible — the file on disk is correct, the running behavior isn't. Easy to chase for an hour.

• NEVER trust a code comment, doc snippet, or prior memory about a third-party library's behavior at the bug's exact boundary. Instead: Open the library's installed source (node_modules/<pkg>/..., vendored path, or git source) and read the function you're calling. Confirm what it does on the branch your code takes — especially early returns, what DOM/events it does or doesn't suppress, and what side effects it still emits when your handler short-circuits. Library behavior at edge cases (modifier keys, error paths, lifecycle short-circuits) regularly diverges from comments and docs — e.g. a comment promised "ESC+CR is what wires up", but the real bug was xterm's not calling on early return, so a phantom re-emitted ; 30 lines of installed source would have caught it in round one.

---

Design note

This skill is intentionally single-file. Diagnostic decisions need full context in one read — splitting into references/ would force a routing step exactly when the agent should be tracing code paths.

---

Trigger question before responding

Before sending the first message, ask: "If the user reads only this message and nothing else, do they know the exact endpoint, env vars, file paths, log locations, and the one command to run next?" If not, the message is theory-heavy — add the contract.

---

Post-step: /code-simplify

Once the bug is fixed and verified, run agentsystem-core:code-simplify on the diff to clean up any duplication, magic numbers, or quick-fix shortcuts the patch introduced — without changing behavior.