docs(core): include deep-recon in core skill count references - #2626
Open
santhiprakash wants to merge 1 commit into
Open
docs(core): include deep-recon in core skill count references#2626santhiprakash wants to merge 1 commit into
santhiprakash wants to merge 1 commit into
Conversation
- Problem: docs/reference/core-tools.md and commands.md still describe the core module as 7/8 skills but omit bmad-deep-recon from the summary list, even though src/core-skills/bmad-deep-recon is a shipped core skill. - Fix: update both summaries to four thinking skills and include deep-recon. - Verification: re-read upstream/main, ran tools/validate-file-refs.js --strict (0 broken refs), preflight_ship.py (clear), node tools/validate-doc-links.js, node tools/validate-sidebar-order.js, node tools/build-docs.mjs, and node tools/validate-skills.js --strict.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe documentation now describes the core module as providing eight skills, including deep-recon, and updates the related built-in tools reference. ChangesCore skills documentation
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 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 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Sync the English core-tools and commands reference pages with the shipped core skill set so
bmad-deep-reconis counted and listed.Why
src/core-skills/bmad-deep-reconexists, butdocs/reference/core-tools.mdsays the module has "seven core skills" with three thinking skills, anddocs/reference/commands.mdsays "8 built-in tools" while only naming seven. Both summaries omitdeep-recon.How
core-tools.md: update the intro to "eight core skills" / "four thinking skills" and adddeep-reconto the thinking-skills list.commands.md: adddeep-reconto the parenthetical list of thinking skills.Testing
git show upstream/main:<file>confirmed the bug marker is still present.python3 ../scripts/preflight_ship.pypassed (no open duplicates, fix marker absent).node tools/validate-file-refs.js --strict→ 0 broken refs.node tools/validate-doc-links.js,node tools/validate-sidebar-order.js,node tools/build-docs.mjs, andnode tools/validate-skills.js --strictall passed.