chore(bmm): retire bmad-investigate skill and its documentation#2509
Conversation
Removes the bmad-investigate skill, its case-file template, the EN/FR
forensic-investigation explainer docs, and every integration point:
Amelia's `IN` menu entry, the plugin marketplace manifest, module-help,
the workflow-map tables and HTML diagrams, and the agent-trigger tables.
EN/FR explanation sidebar orders are renumbered to close the gap left by
the removed doc.
Rationale: the skill's only non-redundant value was a durable, hand-off
case-file artifact — a workflow that is unproven in practice. Everything
else (evidence grading, hypothesis discipline, causal tracing) is
behavior a capable model already performs without a dedicated skill. The
skill also carried method-internal metaphor ("stronghold", "evidence
perimeter") that leaked into user-facing chat. Upstream signals agreed:
issue #2452 flagged the vocabulary leak and a maintainer noted the skill
should not auto-fire. Added in #2364.
Supersedes the in-flight cleanup in #2478 and resolves #2452 by removal.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
🤖 Augment PR SummarySummary: This PR fully retires the Changes:
Technical Notes: The PR is largely subtractive; it aims to prevent the skill’s internal “forensics” metaphor from leaking into user-facing chat by removing the skill entirely. 🤖 Was this summary useful? React with 👍 or 👎 |
📝 WalkthroughWalkthroughThe PR removes the Changesbmad-investigate removal and doc cleanup
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
The retirement commit removed the skill source and docs but did not add the removals.txt entry that triggers cleanup of the skill directory on existing installs during update. Add it, plus an Unreleased CHANGELOG note. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
What
Retires the
bmad-investigateskill in full. Removes the skill (SKILL.md,customize.toml,case-file-template.md), both EN/FRforensic-investigationexplainer docs, and every integration point:INmenu entry (bmad-agent-dev/customize.toml).claude-plugin/marketplace.json)module-help.csvagents.md,named-agents.md, EN + FR)EN/FR explanation sidebar
ordervalues are renumbered to close the gap left by the removed doc.npm run quality→ 0 errors.Why
I tested it. Four real "why" questions about a C++ codebase, each answered twice by identical agents — one using bmad-investigate, one just investigating normally.
Every pair landed on the same answer. The skill never got it more right. On one question the plain agent actually found more. The skill cost about 1.5× as much and took twice as long every time. One time out of a hundred where I need an investigation file, I can just ask the LLM to create it.
In my experience, this gets invoked unexpectedly in response to random questions, burns some tokens and produces output that speaks in birds language. In the few acts that I knew of its existence, I never actually asked for it or needed it per se.
Github issues pointing the same way:
Supersedes / resolves
Testing
npm ci && npm run quality→ Errors: 0 (17 pre-existingcs/vi-vn/zh-cnsidebar-order warnings, unrelated).🤖 Generated with Claude Code