Install BMAD Method (core + BMM) and add Codex skills#6
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 53d4401d00
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| else: | ||
| output = merged | ||
|
|
||
| sys.stdout.write(json.dumps(output, indent=2, ensure_ascii=False) + "\n") |
There was a problem hiding this comment.
Write resolve_config output as UTF-8
On Windows shells or any environment with a non-UTF-8 stdout (for example PYTHONIOENCODING=cp1252), resolving the agent roster now fails because the merged config contains emoji icons and this write emits raw Unicode via ensure_ascii=False. The companion customization resolver already reconfigures stdout for this exact scenario; without the same handling here, bmad-party-mode and bmad-advanced-elicitation cannot load agents in those environments.
Useful? React with 👍 / 👎.
| 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,, | ||
| 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 | ||
| 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 | ||
| BMad Method,bmad-document-project,Document Project,DP,Analyze an existing project to produce useful documentation.,,,anytime,,,false,project-knowledge,* |
There was a problem hiding this comment.
Use the actual project_knowledge variable name
Rows that use project-knowledge cannot be resolved by bmad-help, because the installed config keys are project_knowledge in both _bmad/config.toml and _bmad/bmm/config.yaml (neighboring catalog rows also use the underscore form). For this row the output location is only project-knowledge, so BMad Help will miss existing project documentation and completion artifacts for Document Project instead of checking the configured docs folder.
Useful? React with 👍 / 👎.
Motivation
Description
_bmad/and generated the Codex skill catalog under.agents/skills/, adding many workflow skill definitions, templates, and helper scripts (project context, PRD, epics/stories, architecture, brainstorming, code-review, investigate, document-project, market/domain research, party-mode, and more)._bmad/scripts/and an executable scannerbmad-customize/scripts/list_customizable_skills.pywith unit tests and related test helpers..gitignoreentries to exclude Python bytecode and caches and created a number of templates/assets for workflows (PRD templates, validation skeletons, research templates, deep-dive templates, etc.).npx bmad-method install --directory . --modules bmm --tools codex --yes, which produced_bmad/and.agents/skills/content and configuration manifests.Testing
python3 -m pytest _bmad/scripts/testswhich passed:1 passed.Codex Task