chore: move document project to v6 shims - #2700
Conversation
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe marketplace now resolves ChangesDocument project skill migration
Estimated code review effort: 1 (Trivial) | ~5 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/bmm-skills/v6-shims/bmad-document-project/SKILL.md`:
- Around line 10-14: Update the translated established-project guidance in the
referenced Chinese, Vietnamese, and French pages to remove recommendations for
the deprecated bmad-document-project workflow and direct complex-project users
to invoke bmad-project-context with setup intent instead, preserving the
original request and supplied paths/documents forwarding behavior described by
the bmad-document-project skill.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: eb4149d6-f0d1-4582-bde1-42e3c8fa4668
📒 Files selected for processing (2)
.claude-plugin/marketplace.jsonsrc/bmm-skills/v6-shims/bmad-document-project/SKILL.md
There was a problem hiding this comment.
Caution
Inline review comments failed to post. This is likely due to GitHub's internal server error or limits when posting large numbers of comments. If you are seeing this consistently it is likely a permissions issue. Please check "Moderation" -> "Code review limits" under your organization settings.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/bmm-skills/v6-shims/bmad-document-project/SKILL.md`:
- Around line 10-14: Update the translated established-project guidance in the
referenced Chinese, Vietnamese, and French pages to remove recommendations for
the deprecated bmad-document-project workflow and direct complex-project users
to invoke bmad-project-context with setup intent instead, preserving the
original request and supplied paths/documents forwarding behavior described by
the bmad-document-project skill.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: eb4149d6-f0d1-4582-bde1-42e3c8fa4668
📒 Files selected for processing (2)
.claude-plugin/marketplace.jsonsrc/bmm-skills/v6-shims/bmad-document-project/SKILL.md
🛑 Comments failed to post (1)
src/bmm-skills/v6-shims/bmad-document-project/SKILL.md (1)
10-14: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/usr/bin/env bash set -euo pipefail if [ -d docs ]; then rg -n -i \ 'bmad-document-project|bmad-project-context|generate project docs|project documentation|src/bmm-skills/plan/bmad-document-project' \ docs || true else echo "docs/ does not exist; confirm that no documentation update is required." fiRepository: bmad-code-org/BMAD-METHOD
Length of output: 7647
🏁 Script executed:
#!/usr/bin/env bash set -euo pipefail echo "== Target skill ==" sed -n '1,80p' src/bmm-skills/v6-shims/bmad-document-project/SKILL.md echo echo "== Relevant docs: reference/agents ==" sed -n '1,55p' docs/reference/agents.md echo echo "== Relevant docs: how-to/established-projects ==" sed -n '30,65p' docs/how-to/established-projects.md echo echo "== Relevant docs: zh-cn how-to/established-projects ==" sed -n '55,63p' docs/zh-cn/how-to/established-projects.md echo echo "== Relevant docs: vi-vn how-to/established-projects ==" sed -n '50,62p' docs/vi-vn/how-to/established-projects.md echo echo "== Relevant docs: fr how-to/established-projects ==" sed -n '50,62p' docs/fr/how-to/established-projects.mdRepository: bmad-code-org/BMAD-METHOD
Length of output: 7043
Update translated project-context pages that still recommend
bmad-document-project.
docs/zh-cn/how-to/established-projects.md,docs/vi-vn/how-to/established-projects.md, anddocs/fr/how-to/established-projects.mdstill say complex projects can use the deprecated workflow. Point users tobmad-project-contextwith setup intent instead.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/bmm-skills/v6-shims/bmad-document-project/SKILL.md` around lines 10 - 14, Update the translated established-project guidance in the referenced Chinese, Vietnamese, and French pages to remove recommendations for the deprecated bmad-document-project workflow and direct complex-project users to invoke bmad-project-context with setup intent instead, preserving the original request and supplied paths/documents forwarding behavior described by the bmad-document-project skill.Source: Path instructions
1 participant
What
Moves the deprecated
bmad-document-projectskill fromplan/tov6-shims/and updates its marketplace registration.Why
The skill now exists only as a compatibility forwarder to
bmad-project-context, so classifying it as a v6 shim reflects its current role and keeps it out of the active planning skill set.How
.claude-plugin/marketplace.jsonat the new location.Testing
HUSKY=0 npm ci && npm run quality