@codyswann/lisa 2.309.2 → 2.309.3
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/core/upstream-evidence-manifest.js +4 -4
- package/package.json +1 -1
- package/plugins/lisa/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa/.codex-plugin/skills/lisa-bug-triage/SKILL.md +2 -2
- package/plugins/lisa/.codex-plugin/skills/lisa-reproduce-bug/SKILL.md +40 -69
- package/plugins/lisa/.codex-plugin/skills/lisa-reproduce-bug/agents/openai.yaml +2 -2
- package/plugins/lisa/.codex-plugin/skills/lisa-root-cause-analysis/SKILL.md +88 -108
- package/plugins/lisa/.codex-plugin/skills/lisa-root-cause-analysis/agents/openai.yaml +2 -2
- package/plugins/lisa/agents/debug-specialist.md +13 -97
- package/plugins/lisa/skills/lisa-bug-triage/SKILL.md +2 -2
- package/plugins/lisa/skills/lisa-reproduce-bug/SKILL.md +40 -69
- package/plugins/lisa/skills/lisa-reproduce-bug/agents/openai.yaml +2 -2
- package/plugins/lisa/skills/lisa-root-cause-analysis/SKILL.md +88 -108
- package/plugins/lisa/skills/lisa-root-cause-analysis/agents/openai.yaml +2 -2
- package/plugins/lisa-agy/agents/debug-specialist.md +13 -97
- package/plugins/lisa-agy/plugin.json +1 -1
- package/plugins/lisa-agy/skills/lisa-bug-triage/SKILL.md +2 -2
- package/plugins/lisa-agy/skills/lisa-reproduce-bug/SKILL.md +40 -69
- package/plugins/lisa-agy/skills/lisa-root-cause-analysis/SKILL.md +88 -108
- package/plugins/lisa-cdk/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-cdk/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-cdk-agy/plugin.json +1 -1
- package/plugins/lisa-cdk-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-cdk-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-copilot/agents/debug-specialist.agent.md +13 -97
- package/plugins/lisa-copilot/skills/lisa-bug-triage/SKILL.md +2 -2
- package/plugins/lisa-copilot/skills/lisa-reproduce-bug/SKILL.md +40 -69
- package/plugins/lisa-copilot/skills/lisa-root-cause-analysis/SKILL.md +88 -108
- package/plugins/lisa-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-cursor/agents/debug-specialist.md +13 -97
- package/plugins/lisa-cursor/skills/lisa-bug-triage/SKILL.md +2 -2
- package/plugins/lisa-cursor/skills/lisa-reproduce-bug/SKILL.md +40 -69
- package/plugins/lisa-cursor/skills/lisa-root-cause-analysis/SKILL.md +88 -108
- package/plugins/lisa-expo/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-expo/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-expo-agy/plugin.json +1 -1
- package/plugins/lisa-expo-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-expo-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric-agy/plugin.json +1 -1
- package/plugins/lisa-harper-fabric-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs-agy/plugin.json +1 -1
- package/plugins/lisa-nestjs-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw-agy/plugin.json +1 -1
- package/plugins/lisa-openclaw-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser-agy/plugin.json +1 -1
- package/plugins/lisa-phaser-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-rails/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-rails/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-rails-agy/plugin.json +1 -1
- package/plugins/lisa-rails-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-rails-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript-agy/plugin.json +1 -1
- package/plugins/lisa-typescript-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki-agy/plugin.json +1 -1
- package/plugins/lisa-wiki-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/src/base/agents/debug-specialist.md +13 -97
- package/plugins/src/base/skills/lisa-bug-triage/SKILL.md +2 -2
- package/plugins/src/base/skills/lisa-reproduce-bug/SKILL.md +40 -69
- package/plugins/src/base/skills/lisa-root-cause-analysis/SKILL.md +88 -108
|
@@ -1,96 +1,67 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: lisa-reproduce-bug
|
|
3
|
-
description: "How to
|
|
3
|
+
description: "How to reproduce a bug reliably and on the real path — choosing a reproduction method from the symptom, distinguishing prerequisites from behaviour-substituting scaffolding, and reporting the observed failure rate instead of rounding it."
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
# Reproduce Bug
|
|
7
7
|
|
|
8
|
-
|
|
8
|
+
A bug that cannot be reproduced cannot be verified as fixed. Root cause analysis does not begin until a reliable reproduction exists.
|
|
9
9
|
|
|
10
|
-
##
|
|
10
|
+
## The reproduction must exercise the real path
|
|
11
11
|
|
|
12
|
-
|
|
12
|
+
Left alone, an agent asked to demonstrate a defect will build the environment in which the defect appears — a stub, a fake, a fresh harness — and present that as the reproduction. It looks like proof and it is not: a fix that satisfies it may never touch the path the user is on.
|
|
13
13
|
|
|
14
|
-
|
|
15
|
-
- Capture the complete error output, stack trace, or unexpected behavior
|
|
16
|
-
- Record the exact command used so it can be repeated
|
|
14
|
+
Two kinds of setup get confused here, and only one is a problem:
|
|
17
15
|
|
|
18
|
-
|
|
16
|
+
- **Prerequisites** — state the real path genuinely needs: a seeded record, a logged-in user, a feature flag, a fixture recreating production-equivalent data. These are part of the reproduction. Removing them changes the precondition rather than testing anything, so do not remove them.
|
|
17
|
+
- **Replacement scaffolding** — anything standing in for behaviour the real path would perform: a mocked service, a stubbed function, a fake clock, a bespoke harness that bypasses the normal entry point. This is what makes a reproduction suspect.
|
|
19
18
|
|
|
20
|
-
|
|
21
|
-
- Note the timestamp and environment details (OS, runtime version, dependency versions)
|
|
22
|
-
- Screenshot or log any visual/UI issues
|
|
23
|
-
- Record the actual behavior vs. the expected behavior
|
|
19
|
+
So the reproduction states, explicitly:
|
|
24
20
|
|
|
25
|
-
|
|
21
|
+
- **The entry point the user actually hits** — the route, endpoint, command, or interaction.
|
|
22
|
+
- **Prerequisites**, listed as setup.
|
|
23
|
+
- **Replacement scaffolding**, each item with the real behaviour it substitutes.
|
|
24
|
+
- **Whether the failure survives with prerequisites in place and replacement scaffolding removed.** If it does not, this is a lead rather than a reproduction. Say so and keep going.
|
|
26
25
|
|
|
27
|
-
If the
|
|
26
|
+
If the real path is unreachable — no credentials, no environment, no data — that is a blocked reproduction. Report the missing access instead of substituting scaffolding and calling the bug reproduced.
|
|
28
27
|
|
|
29
|
-
|
|
30
|
-
- Check runtime versions (Node.js, Python, Java, etc.)
|
|
31
|
-
- Compare dependency versions (`package-lock.json`, `poetry.lock`, etc.)
|
|
32
|
-
- Check data differences (database state, seed data, user roles)
|
|
33
|
-
- Verify network conditions (DNS, proxies, firewalls, VPN)
|
|
34
|
-
- Check for platform-specific behavior (OS, architecture, container vs. host)
|
|
28
|
+
## Choose the method from the symptom
|
|
35
29
|
|
|
36
|
-
|
|
30
|
+
| Symptom | Reach for |
|
|
31
|
+
| --- | --- |
|
|
32
|
+
| Wrong value, bad output, thrown error | Failing test at the narrowest layer that still crosses the real path |
|
|
33
|
+
| Broken interface or journey | Browser or device driver against a running build (Playwright, Maestro) |
|
|
34
|
+
| Service or API behaviour | Direct request to the running service — client script or `curl` |
|
|
35
|
+
| Depends on particular data | Seeded fixture recreating that state, recorded as a prerequisite |
|
|
36
|
+
| Intermittent or timing-shaped | Loop the trigger; capture timestamps around async boundaries |
|
|
37
|
+
| Works locally, fails deployed | Do not chase it locally — reproduce against the environment that fails |
|
|
37
38
|
|
|
38
|
-
|
|
39
|
+
**A failing test is the preferred form** wherever it can cross the real path: it runs in CI, it becomes the regression guard, and `codify-verification` expects it. A script is the fallback. Manual steps are the last resort and must carry their prerequisites.
|
|
39
40
|
|
|
40
|
-
|
|
41
|
-
- Write a test that exercises the exact code path and asserts the expected behavior
|
|
42
|
-
- The test should fail with the same symptom as the reported bug
|
|
43
|
-
- A failing test is the most reliable reproduction because it runs in CI and prevents regression
|
|
41
|
+
## Report the failure rate, do not round it
|
|
44
42
|
|
|
45
|
-
|
|
46
|
-
- Write a standalone script that triggers the issue
|
|
47
|
-
- Minimize dependencies -- remove anything not needed to reproduce
|
|
48
|
-
- Include setup steps (data seeding, config) in the script itself
|
|
49
|
-
- The script should be runnable by anyone with access to the repo
|
|
43
|
+
Run the reproduction enough times to state a rate. A reproduction that fails half the time cannot prove a fix — one passing run afterwards means nothing at that rate. If the rate is too low to distinguish a fix from luck, say what would raise it: more iterations, a forced schedule, a narrowed trigger, a seeded clock.
|
|
50
44
|
|
|
51
|
-
|
|
52
|
-
- Document exact click-by-click or command-by-command steps
|
|
53
|
-
- Include prerequisite state (logged-in user, specific data, feature flags)
|
|
54
|
-
- Note any timing-sensitive aspects (race conditions, timeouts)
|
|
45
|
+
## When it will not reproduce
|
|
55
46
|
|
|
56
|
-
|
|
47
|
+
The difference is nearly always one of: runtime version, configuration or feature flags, data state, credentials and permissions, network posture, or platform. Diff the two environments along those axes rather than guessing between them, and report which axes you compared and what you found. That comparison is the finding when the bug stays hidden.
|
|
57
48
|
|
|
58
|
-
|
|
59
|
-
- For intermittent bugs, run enough iterations to establish the failure rate
|
|
60
|
-
- If intermittent, note any patterns (timing, load, specific data)
|
|
61
|
-
|
|
62
|
-
## Output Format
|
|
49
|
+
## Output
|
|
63
50
|
|
|
64
51
|
```text
|
|
65
52
|
## Reproduction
|
|
66
53
|
|
|
67
|
-
|
|
68
|
-
|
|
69
|
-
|
|
70
|
-
|
|
71
|
-
|
|
72
|
-
|
|
73
|
-
|
|
74
|
-
|
|
75
|
-
|
|
76
|
-
|
|
77
|
-
|
|
78
|
-
- OS: [platform]
|
|
79
|
-
- Dependencies: [relevant versions]
|
|
80
|
-
|
|
81
|
-
### Reproduction Type
|
|
82
|
-
[ ] Failing test: [path to test file]
|
|
83
|
-
[ ] Script: [path to script]
|
|
84
|
-
[ ] Manual steps: [documented above]
|
|
85
|
-
|
|
86
|
-
### Reliability
|
|
87
|
-
[Always / Intermittent (N/M runs) / Conditional (only when X)]
|
|
54
|
+
**Entry point:** the route, command, or action the user hits
|
|
55
|
+
**Command or steps:** exactly what to run
|
|
56
|
+
**Actual:** what happens · **Expected:** what should happen
|
|
57
|
+
**Prerequisites:** seeded data, auth state, flags the real path needs — or "none"
|
|
58
|
+
**Replacement scaffolding:** each mock, stub, fake, or bespoke harness and the
|
|
59
|
+
real behaviour it substitutes — or "none"
|
|
60
|
+
**Survives without replacement scaffolding:** yes / no — if no, this is a lead,
|
|
61
|
+
not a reproduction
|
|
62
|
+
**Observed failure rate:** n failures in m runs
|
|
63
|
+
**Form:** failing test `path` | script `path` | manual steps above
|
|
64
|
+
**Environment:** runtime, platform, relevant dependency versions
|
|
88
65
|
```
|
|
89
66
|
|
|
90
|
-
|
|
91
|
-
|
|
92
|
-
- Never skip reproduction. If you cannot reproduce, report what you tried and what you observed.
|
|
93
|
-
- A failing test is always the preferred reproduction method.
|
|
94
|
-
- Capture complete error output -- do not truncate or summarize.
|
|
95
|
-
- If the bug is environment-specific, document exactly which environment triggers it.
|
|
96
|
-
- Do not begin root cause analysis until you have a reliable reproduction.
|
|
67
|
+
Capture output whole — a truncated stack trace loses the line that mattered. But evidence carries whatever the system was holding, so **redact secrets, tokens, credentials, and personal data before a reproduction is handed on or attached to a work item**, and keep any unredacted capture only where the data class it contains is already permitted to live.
|
|
@@ -1,155 +1,135 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: lisa-root-cause-analysis
|
|
3
|
-
description: "
|
|
3
|
+
description: "Prove what causes a defect: hypotheses written down before evidence is gathered, positive confirmation by execution rather than by surviving a disproof, a symptom-keyed technique menu including git bisect, and a declared stopping point that escalates an unresolved verdict instead of drifting."
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
# Root Cause Analysis
|
|
7
7
|
|
|
8
|
-
|
|
8
|
+
Produce a proof, not an explanation. Every link in the chain rests on something observed — a log line, a stack frame, an exit code, a bisect verdict.
|
|
9
9
|
|
|
10
|
-
|
|
10
|
+
## Confirm by executing, not by reasoning
|
|
11
11
|
|
|
12
|
-
|
|
12
|
+
The characteristic failure of this work is a fluent wrong answer: a plausible story assembled from reading code and delivered with confidence. Reading is how a hypothesis is formed. Running something is how it is confirmed. **Do not report a cause you have not executed against.**
|
|
13
13
|
|
|
14
|
-
|
|
14
|
+
## Surviving a disproof is not proof
|
|
15
15
|
|
|
16
|
-
|
|
17
|
-
- Run tests with verbose logging enabled to capture execution flow
|
|
18
|
-
- Check framework-specific log locations (e.g., `.next/`, `dist/`, build output)
|
|
16
|
+
A candidate that no observation has killed is **still standing**, not confirmed. Elimination narrows the field; it does not establish a cause. Closing requires a **positive confirmation**: an execution whose output is what the cause predicts and would not be what it produces if the cause were something else.
|
|
19
17
|
|
|
20
|
-
|
|
18
|
+
That leaves three honest verdicts, and the output has to be able to say each:
|
|
21
19
|
|
|
22
|
-
-
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
- Look for AWS CLI wrappers, CloudWatch log group configurations
|
|
26
|
-
- Check for `.env` files referencing log groups or log streams
|
|
27
|
-
- Use discovered tools first before falling back to raw CLI commands
|
|
28
|
-
- When using AWS CLI directly:
|
|
29
|
-
```bash
|
|
30
|
-
# Discover available log groups
|
|
31
|
-
aws logs describe-log-groups --query 'logGroups[].logGroupName' --output text
|
|
20
|
+
- **confirmed** — a positive confirmation was executed and is recorded.
|
|
21
|
+
- **inconclusive** — one candidate is standing, nothing killed it, nothing confirmed it. Say so; do not promote it.
|
|
22
|
+
- **unresolved / blocked** — the investigation stopped before reaching a cause. See the stopping rule.
|
|
32
23
|
|
|
33
|
-
|
|
34
|
-
aws logs filter-log-events \
|
|
35
|
-
--log-group-name "/aws/lambda/function-name" \
|
|
36
|
-
--start-time $(date -d '30 minutes ago' +%s000) \
|
|
37
|
-
--filter-pattern "ERROR" \
|
|
38
|
-
--query 'events[].message' --output text
|
|
24
|
+
Only *confirmed* justifies a fix. An inconclusive verdict can still be useful — it narrows the next attempt — but it must be labelled.
|
|
39
25
|
|
|
40
|
-
|
|
41
|
-
aws logs tail "/aws/lambda/function-name" --follow --since 10m
|
|
42
|
-
```
|
|
26
|
+
## Write the hypotheses down first
|
|
43
27
|
|
|
44
|
-
|
|
28
|
+
Before gathering evidence, list the candidate causes — two or three is normal — and beside each, the observation that would **kill** it. Then go looking for the killing observations rather than for support.
|
|
45
29
|
|
|
46
|
-
|
|
47
|
-
- Read every function in the chain -- do not skip intermediate code
|
|
48
|
-
- Identify the exact line where behavior diverges from expectation
|
|
49
|
-
- Map the data flow: what value was expected vs. what value was actually present
|
|
30
|
+
A candidate you cannot state a disproof for is not a hypothesis; it is a hunch, and it will survive any amount of evidence. Keep the list updated as you work, and ship the eliminated candidates in the output: they are how a reader knows you looked.
|
|
50
31
|
|
|
51
|
-
##
|
|
32
|
+
## Pick the technique from the symptom
|
|
52
33
|
|
|
53
|
-
|
|
34
|
+
| What you know | Reach for |
|
|
35
|
+
| --- | --- |
|
|
36
|
+
| **It used to work**, and a good commit can be named | `git bisect` — preconditions below |
|
|
37
|
+
| A stack trace or error location | Work backward from the throw; read the frames that carry the value, skip the plumbing |
|
|
38
|
+
| Only that the result is wrong, no location | Instrument first. Reading code to localize an unlocalized defect is the slowest move available |
|
|
39
|
+
| Intermittent | Loop it. Log timestamps and identity either side of async boundaries; look for overlap, staleness, out-of-order completion |
|
|
40
|
+
| Works locally, fails deployed | Diff the environments — version, config, data, permissions, network — before touching code |
|
|
41
|
+
| Wrong shape, missing field, unexpected null | Log the actual value at each transformation, not the type you expect |
|
|
42
|
+
| Possibly a dependency | Pin the exact installed version; read its changelog and issues before blaming local code |
|
|
54
43
|
|
|
55
|
-
###
|
|
44
|
+
### git bisect
|
|
56
45
|
|
|
57
|
-
-
|
|
58
|
-
- **Include context** -- log the actual values, not just "reached here"
|
|
59
|
-
- **Use structured format** -- make logs easy to find and parse
|
|
46
|
+
The highest-leverage move available for a regression, and consistently underused: it answers *which change* in log₂(n) steps instead of log₂(n) hours of reading. It needs three things and wastes time without them.
|
|
60
47
|
|
|
61
|
-
|
|
62
|
-
|
|
63
|
-
|
|
64
|
-
console.log("data:", data);
|
|
48
|
+
1. **A known-good commit** — from the last release, the last green run, or the reporter's "it worked on…".
|
|
49
|
+
2. **A deterministic, non-interactive check** that exits non-zero on the defect. The failing test from `reproduce-bug` usually is one.
|
|
50
|
+
3. **A runnable checkout at every step.** Where a fresh checkout needs a dependency install or build before tests run, fold that into the bisect command. Otherwise every step fails for the wrong reason and the verdict is noise.
|
|
65
51
|
|
|
66
|
-
|
|
67
|
-
|
|
68
|
-
|
|
69
|
-
status: order.status,
|
|
70
|
-
itemCount: order.items.length,
|
|
71
|
-
timestamp: new Date().toISOString(),
|
|
72
|
-
});
|
|
52
|
+
```bash
|
|
53
|
+
git bisect start <bad> <good>
|
|
54
|
+
git bisect run <cmd> # <cmd> must install/build if the checkout needs it
|
|
73
55
|
```
|
|
74
56
|
|
|
75
|
-
|
|
57
|
+
Read the blamed commit before believing it. Bisect names the change that *surfaced* the defect, which is not always the change that introduced it.
|
|
76
58
|
|
|
77
|
-
|
|
78
|
-
|-----------|---------|
|
|
79
|
-
| Function entry | Confirm the function is called and with what arguments |
|
|
80
|
-
| Before conditional branches | Verify which branch is taken and why |
|
|
81
|
-
| Before/after async operations | Detect timing issues, race conditions, failed awaits |
|
|
82
|
-
| Before/after data transformations | Catch where data becomes corrupted or unexpected |
|
|
83
|
-
| Error handlers and catch blocks | Ensure errors are not silently swallowed |
|
|
59
|
+
## Find the logs the project already has
|
|
84
60
|
|
|
85
|
-
|
|
61
|
+
Look for existing tooling before reaching for a raw CLI: `package.json` scripts, `scripts/*log*`, `scripts/*tail*`, AWS CLI wrappers, log-group names in `.env`. Project tooling already encodes the credentials, regions, and group names you would otherwise guess at.
|
|
86
62
|
|
|
87
|
-
|
|
63
|
+
Where no wrapper exists:
|
|
88
64
|
|
|
89
|
-
|
|
65
|
+
```bash
|
|
66
|
+
aws logs describe-log-groups --query 'logGroups[].logGroupName' --output text
|
|
67
|
+
aws logs tail "/aws/lambda/<name>" --follow --since 30m
|
|
68
|
+
```
|
|
90
69
|
|
|
91
|
-
|
|
70
|
+
If remote logs are unreachable, name the log group and the time window needed rather than proceeding without them.
|
|
92
71
|
|
|
93
|
-
|
|
94
|
-
2. **The proximate cause** -- the line of code that directly produces the symptom
|
|
95
|
-
3. **The root cause** -- the underlying reason the proximate cause occurs
|
|
96
|
-
4. **The proof** -- log output, test result, or reproduction steps that confirm each link
|
|
72
|
+
## Instrument surgically
|
|
97
73
|
|
|
98
|
-
|
|
74
|
+
Add the fewest statements that decide between live hypotheses, and make each carry values rather than announce arrival. Guard the access — instrumentation that throws while reading its own subject tells you nothing about the defect.
|
|
99
75
|
|
|
100
|
-
```
|
|
101
|
-
|
|
102
|
-
|
|
103
|
-
v
|
|
104
|
-
Proximate cause: [file:line] -- [the line that directly produces the error]
|
|
105
|
-
|
|
|
106
|
-
v
|
|
107
|
-
Root cause: [file:line] -- [the underlying reason]
|
|
108
|
-
|
|
|
109
|
-
v
|
|
110
|
-
Proof: [log output / test result / reproduction that confirms the chain]
|
|
111
|
-
```
|
|
76
|
+
```typescript
|
|
77
|
+
// Useless: proves a line ran.
|
|
78
|
+
console.log("here", data);
|
|
112
79
|
|
|
113
|
-
|
|
80
|
+
// Useful: decides a hypothesis, and survives the shape it is investigating.
|
|
81
|
+
console.log("[DEBUG:issue-123] processOrder entry", {
|
|
82
|
+
orderId: order?.id,
|
|
83
|
+
status: order?.status,
|
|
84
|
+
itemCount: order?.items?.length ?? null,
|
|
85
|
+
});
|
|
86
|
+
```
|
|
114
87
|
|
|
115
|
-
|
|
88
|
+
Highest-yield placements: function entry (called at all, with what), either side of a branch (which way, on what value), either side of an `await` (timing, staleness), around transformations (where the shape changes), and inside `catch` blocks (what is being swallowed).
|
|
116
89
|
|
|
117
|
-
|
|
118
|
-
- Statements explicitly requested by the user
|
|
90
|
+
The `[DEBUG:<issue>]` prefix exists so cleanup is mechanical rather than remembered. Once the verdict is recorded, remove every one — keeping only logging that belongs in the product permanently — and verify across every source root the project has, not just one:
|
|
119
91
|
|
|
120
|
-
Verify cleanup:
|
|
121
92
|
```bash
|
|
122
|
-
|
|
123
|
-
grep -rn "\[DEBUG:" src/ --include="*.ts" --include="*.tsx" --include="*.js"
|
|
93
|
+
git grep -n "\[DEBUG:"
|
|
124
94
|
```
|
|
125
95
|
|
|
126
|
-
##
|
|
96
|
+
## Stop before you drift
|
|
97
|
+
|
|
98
|
+
Declare a budget before starting: a number of instrumentation rounds, or a wall-clock box. Two signatures mean stop now rather than push on.
|
|
99
|
+
|
|
100
|
+
- **Two consecutive hypotheses falsified with no new information gained.** Widening the search against the same evidence is not progress.
|
|
101
|
+
- **The budget is spent.**
|
|
102
|
+
|
|
103
|
+
Stopping is a legitimate outcome and not a silent one. Record the verdict as **unresolved / blocked** and escalate a decision-ready report: the symptom, the hypotheses tried and how each was killed, the evidence collected, and the single thing that would unblock the work — an access grant, a log group, a reproduction on the real path. A blocked investigation reported clearly is worth more than a confident guess, and costs the next agent far less.
|
|
104
|
+
|
|
105
|
+
## Output
|
|
106
|
+
|
|
107
|
+
`Cause` and `Fix` are required only for a **confirmed** verdict. For *inconclusive* or *unresolved*, record what is known and name the unblocker instead — an unresolved investigation must be representable without inventing a cause to fill the field.
|
|
127
108
|
|
|
128
109
|
```text
|
|
129
110
|
## Root Cause Analysis
|
|
130
111
|
|
|
131
|
-
|
|
132
|
-
| Step | Location | Evidence | Conclusion |
|
|
133
|
-
|------|----------|----------|------------|
|
|
134
|
-
| 1 | file:line | Log output or observed value | What this proves |
|
|
135
|
-
| 2 | file:line | Log output or observed value | What this proves |
|
|
112
|
+
**Verdict:** confirmed | inconclusive | unresolved
|
|
136
113
|
|
|
137
|
-
###
|
|
138
|
-
|
|
139
|
-
|
|
140
|
-
|
|
114
|
+
### Hypotheses
|
|
115
|
+
| Candidate | Would be killed by | Status |
|
|
116
|
+
|---|---|---|
|
|
117
|
+
| ... | the observation that would disprove it | eliminated / standing / confirmed |
|
|
141
118
|
|
|
142
|
-
###
|
|
143
|
-
|
|
144
|
-
|
|
119
|
+
### Evidence trail
|
|
120
|
+
| Step | Location | Observed | Proves |
|
|
121
|
+
|---|---|---|---|
|
|
122
|
+
| 1 | file:line | log output, value, exit code | what this establishes |
|
|
145
123
|
|
|
146
|
-
|
|
124
|
+
### Cause — confirmed verdicts only
|
|
125
|
+
**Proximate:** file:line — the line that directly produces the symptom.
|
|
126
|
+
**Root:** file:line — why that line behaves that way.
|
|
127
|
+
**Confirmation:** the command run and its output, and why that output would differ
|
|
128
|
+
if the cause were something else. Not an argument.
|
|
147
129
|
|
|
148
|
-
|
|
149
|
-
|
|
150
|
-
|
|
151
|
-
|
|
152
|
-
|
|
153
|
-
|
|
154
|
-
- If the root cause is in a third-party dependency, identify the exact version and known issue
|
|
155
|
-
- Always verify the fix resolves the issue -- do not mark investigation complete without proof
|
|
130
|
+
### Fix — confirmed verdicts only
|
|
131
|
+
What changes and why, with file:line references. Anything that must not change.
|
|
132
|
+
|
|
133
|
+
### Unblocker — inconclusive or unresolved verdicts
|
|
134
|
+
The single thing that would let the next attempt proceed, and who can grant it.
|
|
135
|
+
```
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: debug-specialist
|
|
3
|
-
description: Debug specialist agent.
|
|
3
|
+
description: Debug specialist agent. Proves what causes a defect — reproduction on the real path, hypotheses confirmed by execution, evidence chains, and log investigation both local and remote (CloudWatch, Sentry, project tooling). Escalates an unresolved verdict with a decision-ready packet rather than guessing when a cause will not yield.
|
|
4
4
|
skills:
|
|
5
5
|
- reproduce-bug
|
|
6
6
|
- root-cause-analysis
|
|
@@ -8,107 +8,23 @@ skills:
|
|
|
8
8
|
|
|
9
9
|
# Debug Specialist Agent
|
|
10
10
|
|
|
11
|
-
You
|
|
11
|
+
You prove causes. A conclusion you have not executed against is a hypothesis, however well it reads.
|
|
12
12
|
|
|
13
|
-
|
|
13
|
+
Both procedures live in your skills — `reproduce-bug` for establishing the failure, `root-cause-analysis` for proving its cause, including the verdict vocabulary, the stopping rule, and both output contracts. Follow them; nothing here restates them, so there is one place to change them.
|
|
14
14
|
|
|
15
|
-
|
|
15
|
+
## What you route
|
|
16
16
|
|
|
17
|
-
|
|
17
|
+
- **Which skill the work is in.** No investigation begins before `reproduce-bug` yields a reproduction or a blocked verdict. When it yields neither, that is your finding to report, not a step to work around.
|
|
18
|
+
- **Which technique the symptom calls for.** `root-cause-analysis` carries the menu; choosing badly costs more than any other decision in the session, and a regression with a nameable good commit goes to `git bisect` before anyone reads code.
|
|
19
|
+
- **When the session ends.** You own the budget and the escalation, and an unresolved verdict handed over clearly is a valid end — not a failure to be dressed up as a finding.
|
|
18
20
|
|
|
19
|
-
|
|
21
|
+
## What you hand to bug-fixer
|
|
20
22
|
|
|
21
|
-
|
|
22
|
-
- Statements explicitly requested by the user
|
|
23
|
+
You do not implement the fix. Pass on, in the forms the two skills define:
|
|
23
24
|
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
# Search for any remaining debug markers
|
|
27
|
-
grep -rn "\[DEBUG:" src/ --include="*.ts" --include="*.tsx" --include="*.js"
|
|
28
|
-
```
|
|
25
|
+
- The reproduction — its entry point, its form (failing test, script, or manual steps), and its observed failure rate. **Do not require it to be a failing test**: `reproduce-bug` permits a script or manual steps where the real path allows nothing better, and `bug-fixer` codifies a regression test from whichever form arrived.
|
|
26
|
+
- The verdict, and for a confirmed one, proximate and root cause with `file:line` plus the confirming execution. For an inconclusive or unresolved verdict, the unblocker instead — never a cause invented to fill the field.
|
|
29
27
|
|
|
30
|
-
##
|
|
28
|
+
## How you are judged
|
|
31
29
|
|
|
32
|
-
|
|
33
|
-
|
|
34
|
-
```
|
|
35
|
-
## Debug Investigation
|
|
36
|
-
|
|
37
|
-
### Symptom
|
|
38
|
-
What was observed -- exact error message, stack trace, or behavior description.
|
|
39
|
-
|
|
40
|
-
### Reproduction
|
|
41
|
-
The exact command or sequence that triggers the issue.
|
|
42
|
-
|
|
43
|
-
### Evidence Trail
|
|
44
|
-
| Step | Location | Evidence | Conclusion |
|
|
45
|
-
|------|----------|----------|------------|
|
|
46
|
-
| 1 | file:line | Log output or observed value | What this proves |
|
|
47
|
-
| 2 | file:line | Log output or observed value | What this proves |
|
|
48
|
-
| ... | ... | ... | ... |
|
|
49
|
-
|
|
50
|
-
### Root Cause
|
|
51
|
-
**Proximate cause:** The line that directly produces the error.
|
|
52
|
-
**Root cause:** The underlying reason this line behaves incorrectly.
|
|
53
|
-
**Proof:** The specific evidence that confirms this beyond doubt.
|
|
54
|
-
|
|
55
|
-
### Fix
|
|
56
|
-
What needs to change and why. Include file:line references.
|
|
57
|
-
|
|
58
|
-
### Verification
|
|
59
|
-
Command to run that proves the fix resolves the issue.
|
|
60
|
-
Expected output after the fix.
|
|
61
|
-
```
|
|
62
|
-
|
|
63
|
-
## Common Investigation Patterns
|
|
64
|
-
|
|
65
|
-
### Silent Error Swallowing
|
|
66
|
-
```typescript
|
|
67
|
-
// Symptom: Function returns undefined, no error visible
|
|
68
|
-
// Investigation: Check for empty catch blocks
|
|
69
|
-
try {
|
|
70
|
-
return await riskyOperation();
|
|
71
|
-
} catch {
|
|
72
|
-
// Bug: Error swallowed silently -- caller gets undefined
|
|
73
|
-
}
|
|
74
|
-
```
|
|
75
|
-
|
|
76
|
-
### Race Condition
|
|
77
|
-
```typescript
|
|
78
|
-
// Symptom: Intermittent failures, works "sometimes"
|
|
79
|
-
// Investigation: Log timestamps around async operations
|
|
80
|
-
console.log("[DEBUG] before await:", Date.now());
|
|
81
|
-
const result = await asyncOp();
|
|
82
|
-
console.log("[DEBUG] after await:", Date.now(), result);
|
|
83
|
-
// Look for: overlapping timestamps, stale values, out-of-order execution
|
|
84
|
-
```
|
|
85
|
-
|
|
86
|
-
### Wrong Data Shape
|
|
87
|
-
```typescript
|
|
88
|
-
// Symptom: TypeError: Cannot read property 'x' of undefined
|
|
89
|
-
// Investigation: Log the actual object at each transformation step
|
|
90
|
-
console.log("[DEBUG] raw response:", JSON.stringify(response, null, 2));
|
|
91
|
-
console.log("[DEBUG] after transform:", JSON.stringify(transformed, null, 2));
|
|
92
|
-
// Look for: missing fields, null where object expected, array where single item expected
|
|
93
|
-
```
|
|
94
|
-
|
|
95
|
-
### Environment Mismatch
|
|
96
|
-
```bash
|
|
97
|
-
# Symptom: Works locally, fails in staging/production
|
|
98
|
-
# Investigation: Compare environment configurations
|
|
99
|
-
diff <(env | sort) <(ssh staging 'env | sort')
|
|
100
|
-
# Check: Node.js version, env vars, dependency versions, config files
|
|
101
|
-
```
|
|
102
|
-
|
|
103
|
-
## Rules
|
|
104
|
-
|
|
105
|
-
- Never guess at root cause -- prove it with evidence
|
|
106
|
-
- Always reproduce the issue before investigating
|
|
107
|
-
- Read the actual code in the execution path -- do not rely on function names or comments to infer behavior
|
|
108
|
-
- When adding debug logs, use a consistent prefix (e.g., `[DEBUG:issue-name]`) so they are easy to find and clean up
|
|
109
|
-
- Remove all temporary debug log statements after investigation is complete
|
|
110
|
-
- If remote log access is unavailable, report what logs would be needed and from where
|
|
111
|
-
- Prefer project-specific tooling and scripts over raw CLI commands for log access
|
|
112
|
-
- If the root cause is in a third-party dependency, identify the exact version and known issue
|
|
113
|
-
- When multiple hypotheses exist, design a log placement strategy that eliminates all but one
|
|
114
|
-
- Always verify the fix resolves the issue -- do not mark investigation complete without proof
|
|
30
|
+
Not by whether you find a cause; some defects do not yield in one session. By whether every claim rests on something observed, and whether a reader can tell without asking which parts you confirmed, which are merely standing, and which you never reached.
|
|
@@ -10,8 +10,8 @@ Follow this 8-step triage process before implementing any bug fix. Do not skip t
|
|
|
10
10
|
## Triage Steps
|
|
11
11
|
|
|
12
12
|
1. Verify you have all information needed to reproduce the bug (authentication requirements, environment information, etc.). Do not make assumptions. If anything is missing, stop and ask before proceeding.
|
|
13
|
-
2. Reproduce the bug. If you cannot reproduce it, stop and report what you tried and what you observed.
|
|
14
|
-
3. Once reproduced,
|
|
13
|
+
2. Reproduce the bug on the path the user actually hits. Prerequisites the real path needs — seeded data, auth state, flags — are part of the reproduction; scaffolding that substitutes real behaviour is not, and a failure that survives only with it in place is a lead rather than a reproduction. Say which you have. Without a real-path reproduction you may not claim a root cause at all. If you cannot reproduce it, stop and report what you tried and what you observed.
|
|
14
|
+
3. Once reproduced, name a candidate cause and the observation that would disprove it, and go get that observation. Surviving the disproof is not proof — it leaves the candidate standing, not confirmed — so before implementing, execute something whose output the candidate predicts and a different cause would not produce. If you cannot get that confirmation, record the verdict as inconclusive and say so rather than proceeding as if certain; add logging, trace the path, or bisect until you can.
|
|
15
15
|
4. Verify you have access to the tools, environments, and permissions needed to deploy and verify this fix (e.g. CI/CD pipelines, deployment targets, logging/monitoring systems, API access, database access). If any are missing or inaccessible, stop and raise them before starting implementation.
|
|
16
16
|
5. Define the tests you will write to confirm the fix and prevent a regression.
|
|
17
17
|
6. Define the documentation you will create or update to explain this bug so another developer understands the "how" and "what" behind it.
|