bmad-method 6.9.1-next.1 → 6.9.1-next.11

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.
Files changed (23) hide show
  1. package/.claude-plugin/marketplace.json +0 -1
  2. package/package.json +3 -2
  3. package/removals.txt +3 -0
  4. package/src/bmm-skills/4-implementation/bmad-agent-dev/customize.toml +0 -5
  5. package/src/bmm-skills/4-implementation/bmad-code-review/SKILL.md +0 -1
  6. package/src/bmm-skills/4-implementation/bmad-code-review/steps/step-02-review.md +2 -2
  7. package/src/bmm-skills/4-implementation/bmad-code-review/steps/step-03-triage.md +12 -5
  8. package/src/bmm-skills/4-implementation/bmad-code-review/steps/step-04-present.md +1 -1
  9. package/src/bmm-skills/4-implementation/bmad-dev-auto/SKILL.md +0 -1
  10. package/src/bmm-skills/4-implementation/bmad-dev-auto/spec-template.md +8 -0
  11. package/src/bmm-skills/4-implementation/bmad-dev-auto/step-01-clarify-and-route.md +1 -1
  12. package/src/bmm-skills/4-implementation/bmad-dev-auto/step-04-review.md +36 -7
  13. package/src/bmm-skills/4-implementation/bmad-quick-dev/SKILL.md +0 -1
  14. package/src/bmm-skills/4-implementation/bmad-quick-dev/step-04-review.md +9 -4
  15. package/src/bmm-skills/module-help.csv +0 -1
  16. package/src/core-skills/bmad-forge-idea/SKILL.md +49 -21
  17. package/src/core-skills/bmad-review-adversarial-general/SKILL.md +1 -1
  18. package/src/core-skills/bmad-review-edge-case-hunter/SKILL.md +1 -1
  19. package/tools/installer/modules/custom-module-manager.js +15 -9
  20. package/tools/validate-skills.js +6 -1
  21. package/src/bmm-skills/4-implementation/bmad-investigate/SKILL.md +0 -196
  22. package/src/bmm-skills/4-implementation/bmad-investigate/customize.toml +0 -62
  23. package/src/bmm-skills/4-implementation/bmad-investigate/references/case-file-template.md +0 -127
@@ -63,7 +63,6 @@
63
63
  "./src/bmm-skills/3-solutioning/bmad-create-epics-and-stories",
64
64
  "./src/bmm-skills/3-solutioning/bmad-generate-project-context",
65
65
  "./src/bmm-skills/4-implementation/bmad-agent-dev",
66
- "./src/bmm-skills/4-implementation/bmad-investigate",
67
66
  "./src/bmm-skills/4-implementation/bmad-dev-story",
68
67
  "./src/bmm-skills/4-implementation/bmad-quick-dev",
69
68
  "./src/bmm-skills/4-implementation/bmad-checkpoint-preview",
package/package.json CHANGED
@@ -1,7 +1,7 @@
1
1
  {
2
2
  "$schema": "https://json.schemastore.org/package.json",
3
3
  "name": "bmad-method",
4
- "version": "6.9.1-next.1",
4
+ "version": "6.9.1-next.11",
5
5
  "description": "Breakthrough Method of Agile AI-driven Development",
6
6
  "keywords": [
7
7
  "agile",
@@ -42,10 +42,11 @@
42
42
  "prepare": "command -v husky >/dev/null 2>&1 && husky || exit 0",
43
43
  "quality": "npm run format:check && npm run lint && npm run lint:md && npm run docs:build && npm run test:install && npm run test:urls && npm run validate:refs && npm run validate:skills && npm run docs:validate-sidebar",
44
44
  "rebundle": "node tools/installer/bundlers/bundle-web.js rebundle",
45
- "test": "npm run test:refs && npm run test:install && npm run test:urls && npm run test:channels && npm run lint && npm run lint:md && npm run format:check",
45
+ "test": "npm run test:refs && npm run test:install && npm run test:urls && npm run test:channels && npm run test:skills && npm run lint && npm run lint:md && npm run format:check",
46
46
  "test:channels": "node test/test-installer-channels.js",
47
47
  "test:install": "node test/test-installation-components.js",
48
48
  "test:refs": "node test/test-file-refs-csv.js",
49
+ "test:skills": "node test/test-validate-skills.js",
49
50
  "test:urls": "node test/test-parse-source-urls.js",
50
51
  "validate:refs": "node tools/validate-file-refs.js --strict",
51
52
  "validate:skills": "node tools/validate-skills.js --strict"
package/removals.txt CHANGED
@@ -60,3 +60,6 @@ bmad-distillator
60
60
  # bmad-create-ux-design: renamed to bmad-ux (spine-based skill with separate
61
61
  # DESIGN.md and EXPERIENCE.md outputs).
62
62
  bmad-create-ux-design
63
+ # bmad-investigate: retired. Plain investigation reaches the same conclusions at
64
+ # lower cost.
65
+ bmad-investigate
@@ -88,8 +88,3 @@ skill = "bmad-create-story"
88
88
  code = "ER"
89
89
  description = "Party mode review of all work completed across an epic"
90
90
  skill = "bmad-retrospective"
91
-
92
- [[agent.menu]]
93
- code = "IN"
94
- description = "Forensic case investigation with evidence-graded findings, calibrated to the input"
95
- skill = "bmad-investigate"
@@ -50,7 +50,6 @@ Load config from `{project-root}/_bmad/bmm/config.yaml` and resolve:
50
50
  - `date` as system-generated current datetime
51
51
  - `sprint_status` = `{implementation_artifacts}/sprint-status.yaml`
52
52
  - `project_context` = `**/project-context.md` (load if exists)
53
- - CLAUDE.md / memory files (load if exist)
54
53
  - YOU MUST ALWAYS SPEAK OUTPUT in your Agent communication style with the config `{communication_language}`
55
54
 
56
55
  ### Step 5: Greet the User
@@ -18,9 +18,9 @@ failed_layers: '' # set at runtime: comma-separated list of layers that failed o
18
18
 
19
19
  2. Launch parallel subagents without conversation context. If subagents are not available, generate prompt files in `{implementation_artifacts}` — one per reviewer role below — and HALT. Ask the user to run each in a separate session (ideally a different LLM) and paste back the findings. When findings are pasted, resume from this point and proceed to step 3.
20
20
 
21
- - **Blind Hunter** — receives inline `{diff_output}` only. No spec, no context docs, no project access. Invoke via the `bmad-review-adversarial-general` skill.
21
+ - **Blind Hunter** — prompt: "Use the `bmad-review-adversarial-general` skill on `{diff_output}`."
22
22
 
23
- - **Edge Case Hunter** — receives `{diff_output}` and read access to the project. Invoke via the `bmad-review-edge-case-hunter` skill.
23
+ - **Edge Case Hunter** — prompt: "Use the `bmad-review-edge-case-hunter` skill on `{diff_output}`."
24
24
 
25
25
  - **Acceptance Auditor** (only if `{review_mode}` = `"full"`) — receives `{diff_output}`, the content of the file at `{spec_file}`, and any loaded context docs. Its prompt:
26
26
  > You are an Acceptance Auditor. Review this diff against the spec and context docs. Check for: violations of acceptance criteria, deviations from spec intent, missing implementation of specified behavior, contradictions between spec constraints and actual code. Output findings as a Markdown list. Each finding: one-line title, which AC/constraint it violates, and evidence from the diff.
@@ -6,7 +6,6 @@
6
6
  ## RULES
7
7
 
8
8
  - YOU MUST ALWAYS SPEAK OUTPUT in your Agent communication style with the config `{communication_language}`
9
- - Be precise. When uncertain between categories, prefer the more conservative classification.
10
9
 
11
10
  ## INSTRUCTIONS
12
11
 
@@ -29,7 +28,15 @@
29
28
  - Append any unique detail, reasoning, or location references from the other finding(s) into the surviving `detail` field.
30
29
  - Set `source` to the merged sources (e.g., `blind+edge`).
31
30
 
32
- 3. **Classify** each finding into exactly one bucket:
31
+ 3. **Read the code before rating.** Before assigning severity, open the source at each finding's location and read enough surrounding code to judge reachability -- call sites, guards, and validation that live outside the diff hunk. Do not rate from the diff hunk alone. Severity reflects the real consequence at a real call site, not the worst theoretical reading.
32
+
33
+ 4. **Assign severity** to each finding by consequence for the artifact's main consumer (software user, document reader, etc).
34
+ Disregard any severity assigned by a reviewing subagent. Review subagents operate under by-design information asymmetry and do not have enough context to set final severity for this workflow.
35
+ - `low` -- none or cosmetic
36
+ - `medium` -- tolerable
37
+ - `high` -- intolerable
38
+
39
+ 5. **Route** each finding into exactly one triage bucket:
33
40
  - **decision_needed** -- There is an ambiguous choice that requires human input. The code cannot be correctly patched without knowing the user's intent. Only possible if `{review_mode}` = `"full"`.
34
41
  - **patch** -- Code issue that is fixable without human input. The correct fix is unambiguous.
35
42
  - **defer** -- Pre-existing issue not caused by the current change. Real but not actionable now.
@@ -37,11 +44,11 @@
37
44
 
38
45
  If `{review_mode}` = `"no-spec"` and a finding would otherwise be `decision_needed`, reclassify it as `patch` (if the fix is unambiguous) or `defer` (if not).
39
46
 
40
- 4. **Drop** all `dismiss` findings. Record the dismiss count for the summary.
47
+ 6. **Drop** all `dismiss` findings. Record the dismiss count for the summary.
41
48
 
42
- 5. If `{failed_layers}` is non-empty, report which layers failed before announcing results. If zero findings remain after dropping dismissed AND `{failed_layers}` is non-empty, warn the user that the review may be incomplete rather than announcing a clean review.
49
+ 7. If `{failed_layers}` is non-empty, report which layers failed before announcing results. If zero findings remain after dropping dismissed AND `{failed_layers}` is non-empty, warn the user that the review may be incomplete rather than announcing a clean review.
43
50
 
44
- 6. If zero findings remain after triage (all rejected or none raised): state "✅ Clean review — all layers passed." (Step 3 already warned if any review layers failed via `{failed_layers}`.)
51
+ 8. If zero findings remain after triage (all rejected or none raised): state "✅ Clean review — all layers passed." (Step 3 already warned if any review layers failed via `{failed_layers}`.)
45
52
 
46
53
 
47
54
  ## NEXT
@@ -86,7 +86,7 @@ Skip this section if `{spec_file}` is not set.
86
86
 
87
87
  #### Determine new status based on review outcome
88
88
 
89
- - If all `decision-needed` and `patch` findings were resolved (fixed or dismissed) AND no unresolved HIGH/MEDIUM issues remain: set `{new_status}` = `done`. Update the story file Status section to `done`.
89
+ - If all `decision-needed` and `patch` findings were resolved (fixed or dismissed) AND no unresolved `high`/`medium` findings remain: set `{new_status}` = `done`. Update the story file Status section to `done`.
90
90
  - If `patch` findings were left as action items, or unresolved issues remain: set `{new_status}` = `in-progress`. Update the story file Status section to `in-progress`.
91
91
 
92
92
  Save the story file.
@@ -81,7 +81,6 @@ Load config from `{project-root}/_bmad/bmm/config.yaml` and resolve:
81
81
  - `communication_language`, `document_output_language`, `user_skill_level`
82
82
  - `date` as system-generated current datetime
83
83
  - `project_context` = `**/project-context.md` (load if exists)
84
- - CLAUDE.md / memory files (load if exist)
85
84
  - YOU MUST ALWAYS SPEAK OUTPUT in your Agent communication style with the config `{communication_language}`
86
85
  - Language MUST be tailored to `{user_skill_level}`
87
86
  - Generate all documents in `{document_output_language}`
@@ -4,6 +4,7 @@ type: 'feature' # feature | bugfix | refactor | chore
4
4
  created: '{date}'
5
5
  status: 'draft' # draft | ready-for-dev | in-progress | in-review | done | blocked
6
6
  review_loop_iteration: 0 # incremented by step-04 before each review loopback
7
+ followup_review_recommended: false # set by step-04 on status: done from the final review pass significance judgment
7
8
  context: [] # optional: `{project-root}/`-prefixed paths to project-wide standards/docs the implementation agent should load. Keep short — only what isn't already distilled into the spec body.
8
9
  warnings: [] # optional: machine-readable warnings for orchestration, e.g. oversized, multiple-goals
9
10
  ---
@@ -71,6 +72,13 @@ warnings: [] # optional: machine-readable warnings for orchestration, e.g. overs
71
72
  the amendment avoids, and any KEEP instructions (what worked well and must survive re-derivation).
72
73
  Empty until the first bad_spec loopback. -->
73
74
 
75
+ ## Review Triage Log
76
+
77
+ <!-- Append-only. Populated by step-04 on EVERY review pass, including loopbacks and blocked exits.
78
+ Each entry records triage decision counts for intent_gap, bad_spec, patch, defer, and reject,
79
+ with per-category severity breakdowns using low/medium/high, plus the findings addressed in
80
+ that pass. Empty until the first review pass. -->
81
+
74
82
  ## Design Notes
75
83
 
76
84
  <!-- If the approach is straightforward, DELETE THIS ENTIRE SECTION. Do not write "N/A" or "None". -->
@@ -20,7 +20,7 @@ If the invocation prompt explicitly points to an existing spec file with recogni
20
20
  - `ready-for-dev` or `in-progress` → `./step-03-implement.md`
21
21
  - `in-review` → `./step-04-review.md`
22
22
  - `blocked` → HALT with status `blocked` and blocking condition `blocked spec supplied`.
23
- - `done` → ingest as context and proceed to INSTRUCTIONS do not resume.
23
+ - `done` → set `review_loop_iteration` to `0` in the frontmatter, then **EARLY EXIT** to `./step-04-review.md` for a fresh review pass. (A `done` spec is a completed run, so this starts a follow-up review, not a resumption.)
24
24
 
25
25
  Otherwise, treat the invocation prompt as starting intent. This may be a story ID, ticket ID, file path, short description, or longer free-form intent. Do not infer workflow state from non-spec files.
26
26
  If the invocation prompt does not contain enough intent to identify what to implement, HALT with status `blocked` and blocking condition `unclear intent`.
@@ -25,22 +25,44 @@ Do NOT `git add` anything — this is read-only inspection.
25
25
 
26
26
  Launch two subagents without prior session context.
27
27
 
28
- - **Blind hunter** — receives inline `{diff_output}` only. No spec, no context docs, no project access. Invoke via the `bmad-review-adversarial-general` skill.
29
- - **Edge case hunter** — receives `{diff_output}` and read access to the project. Invoke via the `bmad-review-edge-case-hunter` skill.
28
+ - **Blind hunter** — prompt: "Use the `bmad-review-adversarial-general` skill on `{diff_output}`."
29
+ - **Edge case hunter** — prompt: "Use the `bmad-review-edge-case-hunter` skill on `{diff_output}`."
30
30
 
31
31
  ### Classify
32
32
 
33
33
  1. Deduplicate all review findings.
34
- 2. Classify each finding. The first three categories are **this story's problem** caused or exposed by the current change. The last two are **not this story's problem**.
34
+ 2. Assign severity to each finding by consequence for the artifact's main consumer (software user, document reader, etc).
35
+ Disregard any severity assigned by a reviewing subagent. Review subagents operate under by-design information asymmetry and do not have enough context to set final severity for this workflow.
36
+ - `low`: none or cosmetic
37
+ - `medium`: tolerable
38
+ - `high`: intolerable
39
+ 3. Route each finding into exactly one triage category. The first three categories are **this story's problem** — caused or exposed by the current change. The last two are **not this story's problem**.
35
40
  - **intent_gap** — caused by the change; cannot be resolved from the spec because the captured intent is incomplete. Do not infer intent unless there is exactly one possible reading.
36
41
  - **bad_spec** — caused by the change, including direct deviations from spec. The spec should have been clear enough to prevent it. When in doubt between bad_spec and patch, prefer bad_spec — a spec-level fix is more likely to produce coherent code.
37
42
  - **patch** — caused by the change; trivially fixable without human input. Just part of the diff.
38
43
  - **defer** — pre-existing issue not caused by this story, surfaced incidentally by the review. Collect for later focused attention.
39
44
  - **reject** — noise. Drop silently. When unsure between defer and reject, prefer reject — only defer findings you are confident are real.
40
- 3. Process findings in cascading order. If intent_gap exists, lower findings are moot; follow the intent_gap branch below. If bad_spec exists, lower findings are moot since code will be re-derived. If neither exists, process patch and defer normally. Before each bad_spec loopback, read `{spec_file}` frontmatter `review_loop_iteration` (missing means `0`), increment it by 1, and write it back. If it exceeds 5, HALT with status `blocked` and blocking condition `review repair loop exceeded 5 iterations`.
41
- - **intent_gap** — Root cause is inside `<intent-contract>`. Revert code changes. HALT with status `blocked`, blocking condition `intent gap in intent contract`, and include the intent-gap findings.
42
- - **bad_spec** — Root cause is outside `<intent-contract>`. Do not modify content inside `<intent-contract>`. Before reverting code: extract KEEP instructions for positive preservation (what worked well and must survive re-derivation). Revert code changes. Read the `## Spec Change Log` in `{spec_file}` and strictly respect all logged constraints when amending the sections outside `<intent-contract>` that contain the root cause. Append a new change-log entry recording: the triggering finding, what was amended, the known-bad state avoided, and the KEEP instructions. Read fully and follow `./step-03-implement.md` to re-derive the code, then this step will run again.
43
- - **patch** — Auto-fix. These are the only findings that survive loopbacks.
45
+ 4. Append a new entry to the `## Review Triage Log` section in `{spec_file}`, in this format:
46
+ ```markdown
47
+ ### {date} Review pass
48
+ - intent_gap: count
49
+ - bad_spec: count
50
+ - patch: count
51
+ - defer: count
52
+ - reject: count
53
+ - addressed_findings:
54
+ - `[high|medium|low]` `[patch|bad_spec]` <finding summary and action taken in this pass>
55
+ ```
56
+ Where `count` is either just `0`, or total with breakdown by severity `N: (high Nhigh, medium Nmedium, low Nlow)`.
57
+ If no patch was fixed and no bad_spec repair loopback was triggered in this pass, write:
58
+ ```markdown
59
+ - addressed_findings:
60
+ - none
61
+ ```
62
+ 5. Process findings in cascading order. If intent_gap exists, lower findings are moot; follow the intent_gap branch below. If bad_spec exists, lower findings are moot since code will be re-derived. If neither exists, process patch and defer normally. Before each bad_spec loopback, read `{spec_file}` frontmatter `review_loop_iteration` (missing means `0`), increment it by 1, and write it back. If it exceeds 5, append the triage-log entry for this pass with `addressed_findings: none`, then HALT with status `blocked` and blocking condition `review repair loop exceeded 5 iterations (non-convergence)`.
63
+ - **intent_gap** — Root cause is inside `<intent-contract>`. Revert code changes. Append the triage-log entry for this pass with `addressed_findings: none`, then HALT with status `blocked`, blocking condition `intent gap in intent contract`, and include the intent-gap findings.
64
+ - **bad_spec** — Root cause is outside `<intent-contract>`. Do not modify content inside `<intent-contract>`. Before reverting code: extract KEEP instructions for positive preservation (what worked well and must survive re-derivation). Revert code changes. Read the `## Spec Change Log` in `{spec_file}` and strictly respect all logged constraints when amending the sections outside `<intent-contract>` that contain the root cause. Append a new change-log entry recording: the triggering finding, what was amended, the known-bad state avoided, and the KEEP instructions. Append the triage-log entry for this pass, listing every bad_spec finding that triggered the spec amendment and implementation loopback under `addressed_findings`. Read fully and follow `./step-03-implement.md` to re-derive the code, then this step will run again.
65
+ - **patch** — Auto-fix. These are the only findings that survive loopbacks. After auto-fixing, append the triage-log entry for this pass, listing every patch fixed in this pass under `addressed_findings`.
44
66
  - **defer** — Append one new entry to `{deferred_work_file}` using this format. Do not modify existing entries or look for duplicates.
45
67
  ```markdown
46
68
  - source_spec: `{spec_file}`
@@ -55,7 +77,14 @@ Prepare `Auto Run Result` details:
55
77
  - Summary of implemented change
56
78
  - Files changed with one-line descriptions
57
79
  - Review findings breakdown: patches applied, items deferred, items rejected
80
+ - Follow-up review recommendation: `true` when the final review pass made review-driven changes significant enough to benefit from an independent follow-up review; otherwise `false`. Use judgment, not a fixed numeric threshold. Base the judgment on the final pass's triage log and fixes, including patched-finding volume, consequence/severity, breadth, behavior/API/security/data impact, and implementation complexity. Many low-severity patched findings can be significant by volume. Do not recommend follow-up for only a few localized low-consequence fixes.
58
81
  - Verification performed, including command outcomes or manual inspection notes
59
82
  - Any residual risks
60
83
 
84
+ Set `{spec_file}` frontmatter `followup_review_recommended` from the judgment above.
85
+
86
+ If version control is available, commit. Do not push.
87
+
88
+ Capture `final_revision` (current HEAD after committing, or `NO_VCS` if version control is unavailable) into `{spec_file}` frontmatter.
89
+
61
90
  HALT with status `done`.
@@ -71,7 +71,6 @@ Load config from `{project-root}/_bmad/bmm/config.yaml` and resolve:
71
71
  - `date` as system-generated current datetime
72
72
  - `sprint_status` = `{implementation_artifacts}/sprint-status.yaml`
73
73
  - `project_context` = `**/project-context.md` (load if exists)
74
- - CLAUDE.md / memory files (load if exist)
75
74
  - YOU MUST ALWAYS SPEAK OUTPUT in your Agent communication style with the config `{communication_language}`
76
75
  - Language MUST be tailored to `{user_skill_level}`
77
76
  - Generate all documents in `{document_output_language}`
@@ -24,19 +24,24 @@ Do NOT `git add` anything — this is read-only inspection.
24
24
 
25
25
  Launch two subagents without conversation context. If no subagents are available, generate two review prompt files in `{implementation_artifacts}` — one per reviewer role below — and HALT. Ask the human to run each in a separate session (ideally a different LLM) and paste back the findings.
26
26
 
27
- - **Blind hunter** — receives inline `{diff_output}` only. No spec, no context docs, no project access. Invoke via the `bmad-review-adversarial-general` skill.
28
- - **Edge case hunter** — receives `{diff_output}` and read access to the project. Invoke via the `bmad-review-edge-case-hunter` skill.
27
+ - **Blind hunter** — prompt: "Use the `bmad-review-adversarial-general` skill on `{diff_output}`."
28
+ - **Edge case hunter** — prompt: "Use the `bmad-review-edge-case-hunter` skill on `{diff_output}`."
29
29
 
30
30
  ### Classify
31
31
 
32
32
  1. Deduplicate all review findings.
33
- 2. Classify each finding. The first three categories are **this story's problem** caused or exposed by the current change. The last two are **not this story's problem**.
33
+ 2. Assign severity to each finding by consequence for the artifact's main consumer (software user, document reader, etc).
34
+ Disregard any severity assigned by a reviewing subagent. Review subagents operate under by-design information asymmetry and do not have enough context to set final severity for this workflow.
35
+ - `low`: none or cosmetic
36
+ - `medium`: tolerable
37
+ - `high`: intolerable
38
+ 3. Route each finding into exactly one triage category. The first three categories are **this story's problem** — caused or exposed by the current change. The last two are **not this story's problem**.
34
39
  - **intent_gap** — caused by the change; cannot be resolved from the spec because the captured intent is incomplete. Do not infer intent unless there is exactly one possible reading.
35
40
  - **bad_spec** — caused by the change, including direct deviations from spec. The spec should have been clear enough to prevent it. When in doubt between bad_spec and patch, prefer bad_spec — a spec-level fix is more likely to produce coherent code.
36
41
  - **patch** — caused by the change; trivially fixable without human input. Just part of the diff.
37
42
  - **defer** — pre-existing issue not caused by this story, surfaced incidentally by the review. Collect for later focused attention.
38
43
  - **reject** — noise. Drop silently. When unsure between defer and reject, prefer reject — only defer findings you are confident are real.
39
- 3. Process findings in cascading order. If intent_gap or bad_spec findings exist, they trigger a loopback — lower findings are moot since code will be re-derived. If neither exists, process patch and defer normally. Before each loopback, read `{spec_file}` frontmatter `review_loop_iteration` (missing means `0`), increment it by 1, and write it back. If it exceeds 5, HALT and escalate to the human.
44
+ 4. Process findings in cascading order. If intent_gap or bad_spec findings exist, they trigger a loopback — lower findings are moot since code will be re-derived. If neither exists, process patch and defer normally. Before each loopback, read `{spec_file}` frontmatter `review_loop_iteration` (missing means `0`), increment it by 1, and write it back. If it exceeds 5, HALT and escalate to the human.
40
45
  - **intent_gap** — Root cause is inside `<frozen-after-approval>`. Revert code changes. Loop back to the human to resolve. Once resolved, read fully and follow `./step-02-plan.md` to re-run steps 2–4.
41
46
  - **bad_spec** — Root cause is outside `<frozen-after-approval>`. Before reverting code: extract KEEP instructions for positive preservation (what worked well and must survive re-derivation). Revert code changes. Read the `## Spec Change Log` in `{spec_file}` and strictly respect all logged constraints when amending the non-frozen sections that contain the root cause. Append a new change-log entry recording: the triggering finding, what was amended, the known-bad state avoided, and the KEEP instructions. Read fully and follow `./step-03-implement.md` to re-derive the code, then this step will run again.
42
47
  - **patch** — Auto-fix. These are the only findings that survive loopbacks.
@@ -29,4 +29,3 @@ BMad Method,bmad-code-review,Code Review,CR,Story cycle: If issues back to DS if
29
29
  BMad Method,bmad-checkpoint-preview,Checkpoint,CK,Guided walkthrough of a change from purpose and context into details. Use for human review of commits branches or PRs.,,,4-implementation,,,false,,
30
30
  BMad Method,bmad-qa-generate-e2e-tests,QA Automation Test,QA,Generate automated API and E2E tests for implemented code. NOT for code review or story validation — use CR for that.,,,4-implementation,bmad-dev-story,,false,implementation_artifacts,test suite
31
31
  BMad Method,bmad-retrospective,Retrospective,ER,Optional at epic end: Review completed work lessons learned and next epic or if major issues consider CC.,,,4-implementation,bmad-code-review,,false,implementation_artifacts,retrospective
32
- BMad Method,bmad-investigate,Investigate,IN,Forensic case investigation calibrated to the input. Evidence-graded analysis with hypothesis tracking. Produces a structured case file.,,4-implementation,,,false,implementation_artifacts,investigation report
@@ -7,13 +7,13 @@ description: Pressure-test an idea through persona-driven interrogation until it
7
7
 
8
8
  ## Overview
9
9
 
10
- Take a half-formed idea out of the user's head and pressure-test it now, in conversation, where changing your mind is free until what survives is something they can act on with earned conviction, or it dies cheaply. The enemy is the hole you cannot see in your own idea: every unexamined assumption and unresolved branch is a crack that otherwise surfaces later, in the build or the launch, when it costs far more to fix.
10
+ Take a half-formed idea and pressure-test it in conversation, while changing your mind is still cheap, until it becomes something the user can act on with conviction or reject. The main risk is what the user has not examined yet: unchecked assumptions and unresolved decisions usually become more expensive problems later.
11
11
 
12
- The product is the quality of the user's thinking, not an artifact. Hardening an idea, proving or disproving it, or just being an unsparing thinking partner are each a complete outcome. A distilled `forged-idea.md` and a handoff downstream are one optional exit, never the destination — so never herd the user toward "shall we build it?"
12
+ The main goal is better thinking, not producing an artifact. Strengthening an idea, rejecting it, or thinking it through more clearly are all complete outcomes. Writing `forged-idea.md` to hand off to another workflow is optional. Do not steer the conversation toward "shall we build it?"
13
13
 
14
- This is domain-agnostic — the idea may be software, a business model, a creative concept, a research hypothesis, a life decision, or a frivolous thought experiment. When it's a product or feature — net-new or a change inside an existing project — the forge stands in as an alternative analysis-and-definition tool, and what survives distills into `forged-idea.md` for downstream planning.
14
+ This skill can be used on many kinds of ideas. When the idea is about a product or feature, what survives may be written to `forged-idea.md` for later planning.
15
15
 
16
- Act as an exacting interrogator who would rather find the crack than spare the feelings. This is interactive and socratic by nature; there is no headless mode.
16
+ Lead by questioning, not lecturing. Ask one question at a time, press on weak points, and do not let vague claims pass without examination.
17
17
 
18
18
  ## Conventions
19
19
 
@@ -31,21 +31,47 @@ Act as an exacting interrogator who would rather find the crack than spare the f
31
31
 
32
32
  ## Open the session
33
33
 
34
- Open cold. Acknowledging the idea is not endorsing it — do not praise it before it has survived anything, on this turn or any turn. The pull to validate the idea up front to build rapport is the exact reflex this skill exists to refuse.
34
+ Start by scrutinizing the idea, not endorsing it.
35
35
 
36
- Determine the goal before pressing (if a persona is already active with an idea on the table, confirm it in a line rather than asking). Otherwise ask in one message: what is the idea, and what do you want — harden it, prove or kill it, or just think it through? The goal steers the push: proving goes for the load-bearing claim first; hardening drives each branch to a resolved answer. Note whether the idea is net-new or a change inside an existing project.
36
+ ### Discover intent
37
+ Identify:
38
+ - the subject idea,
39
+ - the user's goal for the session,
40
+ - whether the idea is new or a change to an existing project
37
41
 
38
- Tell the user the gear they can call anytime: **"adversarial on this"** (attacked to destruction — you attack, they defend; "switch roles," "you defend now, they attack"). The room is always in play once the topic is set (see The personas) — they can name any persona or call a whole party by name to steer who's at the table.
42
+ If any of these are already clear from the prompt that invoked this skill or previous context, ask the user to confirm and continue.
43
+
44
+ Otherwise ask for what's missing, in order:
45
+ - what is the idea?
46
+ - do you want to clarify and understand it, test whether it holds up, or make it better?
47
+ - is it a new idea or a change to an existing project? If the latter, what project is it, and where can I find its files or other relevant materials?
48
+
49
+ ### Steering the conversation
50
+
51
+ Tell the user they can say **"attack this"**, **"defend this"**, or **"switch roles"** at any time to change how the current idea is argued. In attack mode, do not agree with the idea; look for contradictions, weak assumptions, and failure cases. In defend mode, argue for the strongest version of the idea. Tell the user they can also name a persona or party at any time to change who participates in the session.
52
+
53
+ ### Set up the session
39
54
 
40
55
  Derive a kebab-case `{slug}` for the idea and bind the session workspace `{workspace} = {workflow.forge_output_path}/{workflow.run_folder_pattern}` (the pattern fills with `{slug}`). Create the memlog once the goal is known:
41
56
  `uv run {project-root}/_bmad/scripts/memlog.py init --workspace {workspace} --field idea="<idea>" --field goal="<goal>"`
57
+
42
58
  Tell the user the path; state is on disk now, so the session survives interruption. If init fails, don't abort — run the forge in-conversation and tell the user state won't persist this session.
43
59
 
44
60
  ## The forge
45
61
 
46
- Work one question at a time, in dependency order. Put your own recommended answer on the table each time — a position to push against gets further than an open prompt. Find discoverable answers yourself rather than asking. Treat the user's own words as suspect too: when a term is fuzzy or carries two meanings — a business 'user' versus 'buyer' versus 'payer', not just a code noun — name the ambiguity and force a precise choice before the branch resolves, because a branch built on an overloaded word resolves falsely. When the idea lands inside an existing project, that project's material is your ground truth, and a label is not a move: find the relevant material yourself, check the claim against it, and when it contradicts, make the contradiction the next question. When a branch resolves, give the user a beat before moving on — the crack they were holding back surfaces in that opening.
62
+ Let the session goal set the first move: for clarifying, pin down terms, boundaries, and assumptions; for testing, go after the central claim first; for making it better, drive each unresolved branch to a concrete decision.
63
+
64
+ Work one question at a time, in dependency order.
47
65
 
48
- **Never default-agree.** Reflexive agreement lowers the pressure and the user thinks shallower for it. Attack the weak point or build on the strong one whichever drives deeper thinking and praise only what genuinely earns it. The objective is the best idea, not a comfortable user.
66
+ Include your current best answer or hypothesis when it helps the user respond. A concrete proposal is easier to accept, reject, or revise than an open-ended prompt. Find discoverable answers yourself instead of asking.
67
+
68
+ Do not assume the user's terms are precise. When a term is fuzzy or overloaded, name the ambiguity and ask for a precise choice before continuing. For example, do not let `user`, `buyer`, and `payer` collapse into one entity unless the idea actually requires that.
69
+
70
+ For ideas about an existing project, treat the project's files and materials as the source of truth. Do not accept a label or summary as proof. Find the relevant material yourself and check the user's claim against it. If the material contradicts the user's claim, stop and resolve that before continuing.
71
+
72
+ When a branch resolves, pause before moving on. Give the user a chance to raise any remaining concern.
73
+
74
+ Do not use agreement or praise to make the interaction smoother; they lower pressure and lead to shallower thinking. Agreement is allowed only when it helps the user think better. Praise is noise. Continued engagement and ego-stroking are not objectives. In attack mode, never agree with the idea until the user ends the mode. For each answer, either challenge the weak point or build on the strong point, whichever helps the user think better.
49
75
 
50
76
  Capture as you go — each decision, assumption, crack, kill, and locked idea, one bullet in the user's meaning:
51
77
  `uv run {project-root}/_bmad/scripts/memlog.py append --workspace {workspace} --type <decision|assumption|crack|kill|direction|lock|note> --text "<gist>"`
@@ -53,27 +79,29 @@ A `lock` is an idea the user hardens — settled, not to be reopened; locks are
53
79
 
54
80
  ## The personas
55
81
 
56
- The forge is voiced, not generic and once the topic is set it always runs with the room, because a branch worked by two sharp characters goes deeper and lands harder than a faceless assistant ever could. A persona loaded at activation leads throughout and holds character.
82
+ If a BMad persona was already active when the forge started, keep that persona as the lead voice.
57
83
 
58
- Resolve the pool once, as soon as the goal is known:
84
+ Resolve the available persona pool once, as soon as the goal is known:
59
85
  `uv run {skill-root}/scripts/resolve_personas.py --project-root {project-root} --skill {skill-root}`
60
- It returns the installed BMad roster (`agents`), any custom personas the user authored (`members`), and their saved party groups (`parties` each with an optional `scene` to play, open-cast rooms flagged) everything `bmad-party-mode` knows, without invoking it.
86
+ The script returns installed BMad agents (`agents`), user-defined personas (`members`), and saved parties (`parties`). Parties may include a `scene`; some are open-cast. This gives you the same roster information as `bmad-party-mode` without invoking it.
87
+
88
+ Each turn uses two voices:
89
+ - **One available persona** — choose an installed agent or user-defined persona whose expertise fits the current branch. Vary this voice every few turns; do not let one voice dominate. If the user names a specific persona, use it. If the user calls a saved party, use the whole party and its scene. If the user asks to go one-on-one, use only the requested persona. If no pool is available, generate this voice yourself.
90
+ - **One generated persona** — create a fresh outside voice, such as a competitor, buyer, finance reviewer, domain expert, or critic. Give it a name and enough characterization to keep its viewpoint distinct.
61
91
 
62
- From then on, every turn brings two voices to the branch witnesses you cross-examine, not a panel that debates:
63
- - **One from the user's pool** — an installed agent or custom persona they'll recognize, whose expertise fits the branch in play. Vary who shows up every few turns to keep the pressure high and the angles fresh; don't let the same voice dominate. If the user calls a specific name, bring them in. If the pool resolves empty (a core-only install with no roster), generate both voices on the fly so every branch still arrives with two.
64
- - **One you generate on the fly** — a fresh persona the topic conjures (a hostile competitor, a skeptical CFO, a domain specialist, a historical persona or expert), named and characterized so it's unmistakably itself.
92
+ Use these voices in character to pressure-test the current branch: find sharper objections, missing assumptions, and stronger defenses. Cross-examine them for what matters, then synthesize their input into your next question. Do not let the session turn into a panel debate or persona performance.
65
93
 
66
- They hammer the branch in character; you synthesize their hits into your next question and drive it to a resolved answer. The user steers anytime — name a specific person, call a whole saved party for its scene, or go one-on-one. Voice them yourself by default; spawn separate agents (as `bmad-party-mode` does) only when a branch needs genuinely independent minds a verdict that shouldn't be colored by one voice speaking for all.
94
+ Voice the personas yourself by default. Spawn separate agents only when a branch needs independent reasoning that should not be influenced by one shared voice.
67
95
 
68
96
  ## Exits
69
97
 
70
- The session ends however the thinking lands, and every landing is a real outcome:
98
+ The session can end in three valid states:
71
99
 
72
- - **Hardened** — the idea survived. Distill the memlog into `{workspace}/forged-idea.md`: super succinct the locked items and what was killed and why, in the user's meaning. Not a prose retelling, not a template, not the conversation replayed — the load-bearing residue, nothing else. If it reads like a document, it's too long. Note it can feed `bmad-spec`, `bmad-prd`, or `bmad-prfaq`.
73
- - **Killed** — the idea did not survive. Say so plainly and record why. Finding this cheaply is a win, not a failure.
74
- - **Clearer** — the user simply thinks straighter now. The memlog stands on its own; no `forged-idea.md` needed (the report below still renders).
100
+ - **Hardened** — the idea is stronger and specific enough to use. Distill the memlog into `{workspace}/forged-idea.md`. Keep it extremely short: only the decisions, rejected options, and reasons that matter downstream, in the user's meaning. Do not write a prose summary, template, or conversation recap. If it reads like a document, it is too long. Note that it can feed `bmad-spec`, `bmad-prd`, or `bmad-prfaq`.
101
+ - **Killed** — the idea does not hold up. Say so plainly and record why. Finding that out early is a valid outcome.
102
+ - **Clearer** — the user understands the idea better, but there is no hardened idea to hand off. Leave the memlog as the record; no `forged-idea.md` is needed.
75
103
 
76
- However it lands, render the verdict as a self-contained HTML report the user can open — `{workspace}/forge-report.html`, written every time, no asking. Strike it with a bespoke wax-seal/stamp matched to the outcome: **HARDENED** for a survivor, an **Idea Death Certificate** stamped **KILLED** (with the cause of death) for one that didn't, or a fitting bespoke seal for wherever else it landed (e.g. **CLARIFIED**). Lay out the load-bearing residue — the locked items, what was killed and why, the cracks that held in the user's meaning, and credit the room: the personas and parties that pressure-tested it, by name, icon, and voice. One nicely-styled page (inline CSS, an inline-SVG seal, light flourish only where it lifts the piece) a genuine keepsake, not a templated dump. Tell the user the path.
104
+ Always render `{workspace}/forge-report.html` as a self-contained HTML file the user can open, with inline CSS and an inline-SVG seal or stamp. Summarize the outcome, the locked decisions, what was rejected and why, and the weak points that survived scrutiny, in the user's meaning. Credit the personas and parties that pressure-tested the idea by name, icon, and voice. Render a prominent wax-seal-style or stamped outcome mark, matched to the result: `HARDENED`, an `Idea Death Certificate` stamped `KILLED` with the cause of death, or `CLARIFIED`. Tell the user the path.
77
105
 
78
106
  Flip the status at the end: `uv run {project-root}/_bmad/scripts/memlog.py set --workspace {workspace} --key status --value complete`.
79
107
  If `{workflow.on_complete}` is non-empty, run all instructions in order.
@@ -28,7 +28,7 @@ Review with extreme skepticism — assume problems exist. Find at least ten issu
28
28
 
29
29
  ### Step 3: Present Findings
30
30
 
31
- Output findings as a Markdown list (descriptions only).
31
+ Output findings as a Markdown list: descriptions only, no severity, priority, or ranking.
32
32
 
33
33
 
34
34
  ## HALT CONDITIONS
@@ -16,7 +16,7 @@ Ignore the rest of the codebase unless the provided content explicitly reference
16
16
 
17
17
  **MANDATORY: Execute steps in the Execution section IN EXACT ORDER. DO NOT skip steps or change the sequence. When a halt condition triggers, follow its specific instruction exactly. Each action within a step is a REQUIRED action to complete that step.**
18
18
 
19
- **Your method is exhaustive path enumeration — mechanically walk every branch, not hunt by intuition. Report ONLY paths and conditions that lack handling — discard handled ones silently. Do NOT editorialize or add filler findings only.**
19
+ **Your method is exhaustive path enumeration — mechanically walk every branch, not hunt by intuition. Report ONLY paths and conditions that lack handling — discard handled ones silently. Do NOT editorialize or add filler. Do not assign severity labels, rankings, or priority levels.**
20
20
 
21
21
 
22
22
  ## EXECUTION
@@ -11,6 +11,18 @@ function quoteCustomRef(ref) {
11
11
  return `"${ref}"`;
12
12
  }
13
13
 
14
+ function isLocalSourcePath(input) {
15
+ return (
16
+ input.startsWith('/') ||
17
+ input.startsWith('./') ||
18
+ input.startsWith('../') ||
19
+ input.startsWith('.\\') ||
20
+ input.startsWith('..\\') ||
21
+ input.startsWith('~') ||
22
+ path.win32.isAbsolute(input)
23
+ );
24
+ }
25
+
14
26
  /**
15
27
  * Manages custom modules installed from user-provided sources.
16
28
  * Supports any Git host (GitHub, GitLab, Bitbucket, self-hosted) and local file paths.
@@ -83,13 +95,7 @@ class CustomModuleManager {
83
95
  // Avoid consuming the @ in `git@host:owner/repo` — `before` wouldn't end with a path separator
84
96
  // in that case. Require that the @ comes after the host/path, not inside the auth segment.
85
97
  // Rule: the @ is a version suffix only if `before` looks like a complete URL or local path.
86
- const beforeLooksLikeRepo =
87
- before.startsWith('/') ||
88
- before.startsWith('./') ||
89
- before.startsWith('../') ||
90
- before.startsWith('~') ||
91
- /^https?:\/\//i.test(before) ||
92
- /^git@[^:]+:.+/.test(before);
98
+ const beforeLooksLikeRepo = isLocalSourcePath(before) || /^https?:\/\//i.test(before) || /^git@[^:]+:.+/.test(before);
93
99
  if (beforeLooksLikeRepo) {
94
100
  versionSuffix = candidate;
95
101
  trimmed = before;
@@ -97,8 +103,8 @@ class CustomModuleManager {
97
103
  }
98
104
  }
99
105
 
100
- // Local path detection: starts with /, ./, ../, or ~
101
- if (trimmed.startsWith('/') || trimmed.startsWith('./') || trimmed.startsWith('../') || trimmed.startsWith('~')) {
106
+ // Local path detection: POSIX, Windows, relative, or home-relative.
107
+ if (isLocalSourcePath(trimmed)) {
102
108
  if (versionSuffix) {
103
109
  return {
104
110
  type: 'local',
@@ -312,6 +312,11 @@ function validateSkill(skillDir) {
312
312
  const name = skillFm && skillFm.name;
313
313
  const description = skillFm && skillFm.description;
314
314
 
315
+ // Deprecated skills are thin compatibility shims that forward to a replacement.
316
+ // They intentionally omit a "Use when" trigger so users are steered to the new
317
+ // skill instead, so exempt them from the SKILL-06 trigger-phrase requirement.
318
+ const isDeprecated = typeof description === 'string' && /^\s*deprecated\b/i.test(description);
319
+
315
320
  // --- SKILL-04: name format ---
316
321
  if (name && !NAME_REGEX.test(name)) {
317
322
  findings.push({
@@ -349,7 +354,7 @@ function validateSkill(skillDir) {
349
354
  });
350
355
  }
351
356
 
352
- if (!/use\s+when\b/i.test(description) && !/use\s+if\b/i.test(description)) {
357
+ if (!isDeprecated && !/use\s+when\b/i.test(description) && !/use\s+if\b/i.test(description)) {
353
358
  findings.push({
354
359
  rule: 'SKILL-06',
355
360
  title: 'description Quality',
@@ -1,196 +0,0 @@
1
- ---
2
- name: bmad-investigate
3
- description: Forensic case investigation with evidence-graded findings, calibrated to the input. Use when the user asks to investigate a bug, trace what caused an incident, walk through unfamiliar code, or build a mental model of a code area before working on it.
4
- ---
5
-
6
- # Investigate
7
-
8
- ## Overview
9
-
10
- Reconstruct what's happening, or what an unfamiliar area does, from the available evidence. Produce a structured case
11
- file another engineer can pick up cold. Calibrate continuously between defect-chasing (symptom-driven) and
12
- area-exploration (no symptom); the same discipline applies on both ends.
13
-
14
- **Args:** A ticket ID, log file path, diagnostic archive, error message, code area name, problem description, or a path
15
- to an existing case file. The last form resumes a prior investigation; everything else opens a new case.
16
-
17
- **Output:** `{implementation_artifacts}/{workflow.case_file_subdir}/{workflow.case_file_filename}`. Reference inputs
18
- are recorded; raw content is not read into the parent context until an outcome calls for it.
19
-
20
- `{slug}` is the ticket ID when one is provided, otherwise a short descriptive name agreed with the user, sanitized to
21
- lowercase alphanumeric with hyphens. On collision with an existing case file at the resolved path, ask whether to
22
- rename to `slug-YYYY-MM-DD.md` or resume the existing file (resuming routes to Outcome 0).
23
-
24
- After every outcome, present what was learned and pause for the user before continuing.
25
-
26
- ## Principles
27
-
28
- - **Evidence grading.**
29
- - **Confirmed.** Directly observed; cite `path:line`, log timestamp, or commit hash.
30
- - **Deduced.** Logically follows from Confirmed evidence; show the chain.
31
- - **Hypothesized.** Plausible but unconfirmed; state what would confirm or refute it.
32
- - **Stronghold first.** Anchor in one Confirmed piece of evidence and expand outward. Never start from a theory and
33
- hunt for support. When evidence is sparse, switch to evidence-light mode (Outcome 1 branch).
34
- - **Challenge the premise.** The user's description is a hypothesis, not a fact. Verify independently; if evidence
35
- contradicts, say so.
36
- - **Follow the evidence, not the narrative.** When evidence contradicts the working theory, update the theory — never
37
- the other way around. Resist confirmation bias even when the user is convinced.
38
- - **Hypotheses are never deleted.** Update Status (Open / Confirmed / Refuted) and add a Resolution. Wrong turns are
39
- part of the deliverable.
40
- - **Missing evidence is itself a finding.** Document the gap, what it would resolve, and how to obtain it.
41
- - **Write it down early.** Initialize the case file as soon as the slug is agreed; it is the persistent state across
42
- interruptions.
43
- - **Path:line citations** use CWD-relative format, no leading `/`, so they're clickable in IDE-embedded terminals.
44
- - **Delegation discipline.** When a step requires reading 5+ files or any file >10K tokens, delegate to a subagent
45
- that returns structured JSON only. Cite `path:line` from the result; don't re-read in the parent.
46
- - **Issue independent operations in parallel** (multi-grep, multi-read, parallel inventories) — one message, multiple
47
- tool calls.
48
- - **Communication.** Evidence-first language ("the evidence shows", "unconfirmed, requires X to verify"). No hedging,
49
- no narrative.
50
-
51
- ## On Activation
52
-
53
- ### Step 1: Resolve the workflow block
54
-
55
- Run: `python3 {project-root}/_bmad/scripts/resolve_customization.py --skill {skill-root} --key workflow`
56
-
57
- If the script fails, stop and surface the error.
58
-
59
- ### Step 2: Execute prepend steps
60
-
61
- Run each entry in `{workflow.activation_steps_prepend}` in order.
62
-
63
- ### Step 3: Load persistent facts
64
-
65
- Treat each entry in `{workflow.persistent_facts}` as foundational context. `file:` prefixes are paths or globs under
66
- `{project-root}` (load contents); other entries are facts verbatim.
67
-
68
- ### Step 4: Load config
69
-
70
- Load `{project-root}/_bmad/bmm/config.yaml` and resolve `{user_name}`, `{communication_language}`,
71
- `{document_output_language}`, `{implementation_artifacts}`, `{project_knowledge}`. If `{implementation_artifacts}` is
72
- unresolved, fall back to `./investigations/` and surface the fallback before initializing.
73
-
74
- ### Step 5: Greet
75
-
76
- Greet `{user_name}` in `{communication_language}`.
77
-
78
- ### Step 6: Execute append steps
79
-
80
- Run each entry in `{workflow.activation_steps_append}` in order.
81
-
82
- Activation is complete. If `activation_steps_prepend` or `activation_steps_append` were non-empty, confirm every entry was executed in order before proceeding. Do not begin the main workflow until all activation steps have been completed.
83
-
84
- ### Step 7: Acknowledge and route
85
-
86
- Acknowledge the input as a reference (record paths and IDs; don't read raw content). Path to an existing case file →
87
- Outcome 0. Otherwise → Outcome 1.
88
-
89
- ## Procedure
90
-
91
- ### Outcome 0: Existing case is loaded and surfaced
92
-
93
- Read the case file. Surface, in order: open hypotheses (Status = Open) with their confirm/refute criteria; open
94
- backlog (Status ≠ Done); missing-evidence rows; last Conclusion with confidence. Ask which thread to pull. New
95
- evidence opens a new `## Follow-up: {YYYY-MM-DD}` block (append `#2`, `#3` on same-day reentry). Pause for user with the recap above; wait for direction.
96
-
97
- ### Outcome 1: Scope and stronghold are established
98
-
99
- Acknowledge each input shape — record location, scope, time window only; bulk reads happen in Outcome 2.
100
-
101
- - **Issue tracker ticket.** Fetch full details via available MCP tools.
102
- - **Diagnostic archive.** Record path, file count, time window.
103
- - **Log file or stack trace.** Record path and time window; only the stack frame already in the user's message is in
104
- scope here.
105
- - **Free-text description.** Capture verbatim; treat as hypothesis.
106
- - **Code area name** (no symptom). Record entry point.
107
- - **Recent commit area.** Record commit range.
108
-
109
- If the user arrived with a hypothesis, register it as Hypothesis #1. Find the stronghold *independently*; the user's
110
- hypothesis is one of the things the stronghold validates or refutes.
111
-
112
- Find a stronghold: a Confirmed piece of evidence (error message, function name, HTTP route, config parameter, test
113
- case). Anchor here.
114
-
115
- **Initialize `{case_file}` before branching.** The path is
116
- `{implementation_artifacts}/{workflow.case_file_subdir}/{workflow.case_file_filename}` with `{slug}` substituted (slug
117
- and collision rules in Overview). Create the file from `{workflow.case_file_template}` and fill Hand-off Brief
118
- (rough), Case Info, Problem Statement, initial Evidence Inventory.
119
-
120
- **Evidence-light branch.** When no Confirmed evidence is reachable: mark the case evidence-light in the Hand-off
121
- Brief; populate the Investigation Backlog with prioritized data-collection items; record "to make progress, I need one
122
- of: …"; pause for the user to provide evidence or authorize Outcome 2 to scan more broadly.
123
-
124
- Otherwise present scope, stronghold, file path, proposed approach. Pause for user with the recap above; wait for direction.
125
-
126
- ### Outcome 2: Evidence perimeter is mapped
127
-
128
- Survey the scene: inventory available evidence in parallel across these independent categories: diagnostic archives;
129
- issue tracker; version control; test results; static analysis; source code. For any category exceeding ~10K tokens,
130
- delegate to a subagent that returns a JSON manifest (paths, sizes, time windows, key fragments cited as `path:line`).
131
-
132
- Classify each Available, Partial, or Missing — Missing is itself a finding. Update Evidence Inventory and Investigation
133
- Backlog. Pause for user with the recap above; wait for direction.
134
-
135
- ### Outcome 3: Cause is reasoned about with discipline
136
-
137
- - **Trace causality.** Symptom-driven: trace backward from the symptom to producing conditions and the state that
138
- emerged. Exploration: trace backward from outputs (returns, side effects, messages sent) to producing conditions.
139
- Same technique, different anchor.
140
- - **Reconstruct the timeline** by cross-referencing logs, system events, version control, user observations.
141
- - **Form and test hypotheses.** State, identify confirming/refuting evidence, search, grade
142
- (Confirmed / Refuted / Open). Update Status. Never delete.
143
- - **Refutation pass.** Each time a hypothesis transitions toward Confirmed, actively look for refuting evidence first.
144
- Record the attempt in Resolution.
145
- - **Verify the user's premise.** If evidence contradicts, say so explicitly.
146
- - **Add discovered paths to the backlog.** Stay focused on the current thread.
147
-
148
- Update Confirmed Findings, Deduced Conclusions, Hypothesized Paths, Backlog, Timeline. Highlight contradictions to the
149
- original premise. Pause for user with the recap above; wait for direction.
150
-
151
- ### Outcome 4: Source has been traced where it matters
152
-
153
- Issue these first-pass scans as parallel tool calls in one message: grep for exact error strings; glob the affected
154
- directory for parallel implementations; `git log` for recent changes.
155
-
156
- Then sequentially: read the surrounding code; follow the caller chain; watch for language and process boundary
157
- crossings (compiled→scripts, IPC, host→device, configuration flow).
158
-
159
- Lean by case type:
160
-
161
- - **Exploration:** I/O mapping (triggers, outputs, dependencies); frequent-terms scan; control-flow filtering
162
- (branches, loops, error handling, state-machine transitions).
163
- - **Symptom-driven:** depth assessment — is the root cause reachable from local context, or is a broader area model
164
- required? Surface escalations; never silently expand scope. Trivial-fix assessment — off-by-one, missing null check,
165
- swapped argument → one-line code suggestion or draft diff in the report; non-trivial → stop at the root cause area.
166
-
167
- Investigation stops at the diagnosis; implementation is out of scope. Update Source Code Trace (Error origin, Trigger,
168
- Condition, Related files; area model when broader). Pause for user with the recap above; wait for direction.
169
-
170
- ### Outcome 5: Report is finalized and the hand-off is clean
171
-
172
- Update `{case_file}`:
173
-
174
- - **Hand-off Brief** rewritten to final form (3 sentences, 15-second read).
175
- - **Final Conclusion** with confidence: **High** (Confirmed root cause, deterministic repro), **Medium** (Deduced;
176
- minor uncertainty), **Low** (Hypothesized; clear data gap).
177
- - **Fix direction** when applicable (categorize by mechanism if multiple combine).
178
- - **Diagnostic steps** if uncertainty remains.
179
- - **Reproduction Plan** when applicable, or a verification plan for exploration cases.
180
- - **Status:** Active / Concluded / Blocked on evidence.
181
-
182
- Present the conclusion, then a concrete next-steps menu: trivial fix → `bmad-quick-dev`; scope/plan adjustment →
183
- `bmad-correct-course`; tracked story → `bmad-create-story`; fresh review → `bmad-code-review`. Recommend the
184
- highest-value action. Mitigations and workarounds are generated only on explicit request — investigation stops at the
185
- diagnosis. Execute `{workflow.on_complete}` if non-empty. Pause for user with the recap above; wait for direction.
186
-
187
- ## Follow-up Iterations
188
-
189
- Continue work by appending to `{case_file}` under a new `## Follow-up: {YYYY-MM-DD}` block (`#2`, `#3` on same-day
190
- reentry). The investigation is complete when:
191
-
192
- - Root cause is Confirmed.
193
- - Root cause is Hypothesized with a clear data gap.
194
- - The mental model is sufficient for the user's stated goal (exploration cases).
195
- - The backlog contains only items requiring unavailable evidence.
196
- - The user explicitly concludes.
@@ -1,62 +0,0 @@
1
- # DO NOT EDIT -- overwritten on every update.
2
- #
3
- # Workflow customization surface for bmad-investigate. Mirrors the
4
- # agent customization shape under the [workflow] namespace.
5
-
6
- [workflow]
7
-
8
- # --- Configurable below. Overrides merge per BMad structural rules: ---
9
- # scalars: override wins • arrays (persistent_facts, activation_steps_*): append
10
- # arrays-of-tables with `code`/`id`: replace matching items, append new ones.
11
-
12
- # Steps to run before the standard activation (config load, greet).
13
- # Overrides append. Use for pre-flight loads, compliance checks, etc.
14
-
15
- activation_steps_prepend = []
16
-
17
- # Steps to run after greet but before the workflow begins.
18
- # Overrides append. Use for context-heavy setup that should happen
19
- # once the user has been acknowledged.
20
-
21
- activation_steps_append = []
22
-
23
- # Persistent facts the workflow keeps in mind for the whole run.
24
- # Use for citation conventions (path:line vs path#L42), grading-scale
25
- # overrides (ITIL severity 1-5 instead of High/Medium/Low), tone
26
- # directives (engineering vs exec-facing), or compliance constraints
27
- # the case file must respect.
28
- # Distinct from the runtime memory sidecar — these are static context
29
- # loaded on activation. Overrides append.
30
- #
31
- # Each entry is either:
32
- # - a literal sentence, e.g. "Use ITIL severity 1-5 instead of High/Medium/Low for confidence."
33
- # - a file reference prefixed with `file:`, e.g. "file:{project-root}/docs/standards.md"
34
- # (glob patterns are supported; the file's contents are loaded and treated as facts).
35
-
36
- persistent_facts = [
37
- "file:{project-root}/**/project-context.md",
38
- ]
39
-
40
- # Scalar: path to the case-file template, resolved from the skill root.
41
- # Override to point at an org-shaped template (compliance sections,
42
- # SLA fields, post-mortem hooks, ITIL fields).
43
-
44
- case_file_template = "references/case-file-template.md"
45
-
46
- # Scalar: subdirectory under {implementation_artifacts} where case files land.
47
- # Override for org taxonomies (forensics/, cases/, incidents/, bug-bash/).
48
-
49
- case_file_subdir = "investigations"
50
-
51
- # Scalar: filename pattern for new case files. {slug} expands to the
52
- # ticket ID or a short user-agreed name.
53
-
54
- case_file_filename = "{slug}-investigation.md"
55
-
56
- # Scalar: executed when the workflow finalizes the case file at Outcome 5,
57
- # after the conclusion is presented. Override wins. Use for post-case
58
- # automation: post the case to Slack/Teams, push fields back to ticketing,
59
- # link the case to a sprint, trigger a follow-up retro.
60
- # Leave empty for no custom post-completion behavior.
61
-
62
- on_complete = ""
@@ -1,127 +0,0 @@
1
- # Investigation: {title}
2
-
3
- ## Hand-off Brief
4
-
5
- 1. **What happened.** {one-sentence problem statement, evidence-graded}
6
- 2. **Where the case stands.** {status, last finding, what would unblock progress}
7
- 3. **What's needed next.** {single recommended action with rationale}
8
-
9
- ## Case Info
10
-
11
- | Field | Value |
12
- | ---------------- | -------------------------------------------------------------------------- |
13
- | Ticket | {ticket-id or "N/A"} |
14
- | Date opened | {date} |
15
- | Status | Active |
16
- | System | {OS, version, relevant environment details} |
17
- | Evidence sources | {diagnostic archive, logs, crash dump, code, version control, etc.} |
18
-
19
- ## Problem Statement
20
-
21
- {User-reported description; the initial claim. May be refined or contradicted by evidence.}
22
-
23
- ## Evidence Inventory
24
-
25
- | Source | Status | Notes |
26
- | -------- | ------------------------------- | --------- |
27
- | {source} | {Available / Partial / Missing} | {details} |
28
-
29
- ## Investigation Backlog
30
-
31
- | # | Path to Explore | Priority | Status | Notes |
32
- | - | --------------- | --------------------- | ------------------------------------- | --------- |
33
- | 1 | {description} | {High / Medium / Low} | {Open / In Progress / Done / Blocked} | {context} |
34
-
35
- ## Timeline of Events
36
-
37
- | Time | Event | Source | Confidence |
38
- | ----------- | ------------------- | --------------------- | --------------------- |
39
- | {timestamp} | {event description} | {log file, commit, …} | {Confirmed / Deduced} |
40
-
41
- ## Confirmed Findings
42
-
43
- ### Finding 1: {title}
44
-
45
- **Evidence:** {citation — `path:line`, log timestamp, or commit hash}
46
-
47
- **Detail:** {description}
48
-
49
- ## Deduced Conclusions
50
-
51
- ### Deduction 1: {title}
52
-
53
- **Based on:** {which Confirmed Findings}
54
-
55
- **Reasoning:** {logical chain}
56
-
57
- **Conclusion:** {what follows}
58
-
59
- ## Hypothesized Paths
60
-
61
- ### Hypothesis 1: {title}
62
-
63
- **Status:** {Open / Confirmed / Refuted}
64
-
65
- **Theory:** {description}
66
-
67
- **Supporting indicators:** {what makes this plausible}
68
-
69
- **Would confirm:** {specific evidence that would prove this}
70
-
71
- **Would refute:** {specific evidence that would disprove this}
72
-
73
- **Resolution:** {when Status changes from Open, what evidence settled it}
74
-
75
- ## Missing Evidence
76
-
77
- | Gap | Impact | How to Obtain |
78
- | ---------------- | ------------------------------------ | --------------- |
79
- | {what's missing} | {what it would confirm or eliminate} | {how to get it} |
80
-
81
- ## Source Code Trace
82
-
83
- | Element | Detail |
84
- | ------------- | ------------------------------------------- |
85
- | Error origin | {file:line, function name} |
86
- | Trigger | {what causes this code to execute} |
87
- | Condition | {what state produces the observed behavior} |
88
- | Related files | {other files in the same code path} |
89
-
90
- ## Conclusion
91
-
92
- **Confidence:** {High / Medium / Low}
93
-
94
- {Summary stating what is Confirmed vs. what remains Hypothesized. If a root cause is identified, state it; otherwise
95
- name the most promising hypothesized paths and what would resolve the remaining uncertainty.}
96
-
97
- ## Recommended Next Steps
98
-
99
- ### Fix direction
100
-
101
- {What needs to change and why. Categorize by mechanism when multiple issues combine.}
102
-
103
- ### Diagnostic
104
-
105
- {Steps to confirm the root cause: additional logging, targeted tests, data to collect.}
106
-
107
- ## Reproduction Plan
108
-
109
- {Setup, trigger, expected results. Scale from isolated proof to full system reproduction.}
110
-
111
- ## Side Findings
112
-
113
- Tangential observations surfaced during the investigation, evidence-graded, with citation when applicable.
114
-
115
- - {observation}
116
-
117
- ## Follow-up: {date}
118
-
119
- ### New Evidence
120
-
121
- ### Additional Findings
122
-
123
- ### Updated Hypotheses
124
-
125
- ### Backlog Changes
126
-
127
- ### Updated Conclusion