refactor(skills): convert create-architecture workflow to native skill

[alexeyv]

Summary

• mechanically convert src/bmm/workflows/3-solutioning/create-architecture into native skill packaging at src/bmm/workflows/3-solutioning/bmad-create-architecture
• add SKILL.md and switch manifest to type: skill
• update workflow/step path references required by directory rename + skill packaging
• update dependent references:
  • src/bmm/module-help.csv -> skill:bmad-create-architecture
  • src/bmm/agents/architect.agent.yaml exec path -> renamed workflow directory

Mechanical Constraints

• no behavioral redesign/hardening/optimization
• workflow logic preserved; only required path/name adjustments applied

Validation

• installer run:
  • node tools/cli/bmad-cli.js install --directory /Users/alex/src/bmad --modules bmm --tools claude-code --yes
• full repo checks:
  • npm test
• manifest/install verification:
  • bmad-create-architecture appears in _bmad/_config/skill-manifest.csv
  • no bmad-create-architecture entry in _bmad/_config/workflow-manifest.csv
• equivalence protocol (mechanical pass):
  • normalized old-vs-new parity for workflow.md and all step files
  • exact parity for template + data files

[augmentcode]

🤖 Augment PR Summary

Summary: Migrates the legacy create-architecture workflow into a native skill directory (bmad-create-architecture) for skill-based installers/IDEs.

Changes:

• Renames the workflow folder and updates the workflow frontmatter/name to bmad-create-architecture
• Adds native-skill packaging files (SKILL.md, bmad-skill-manifest.yaml with type: skill)
• Updates internal references (agent menu exec path + step/workflow path references) to match the new directory layout

Technical Notes: Workflow now uses relative paths (e.g., installed_path = ., ./steps/…) consistent with skill-directory execution.

_{🤖 Was this summary useful? React with 👍 or 👎}

[augmentcode]

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

[coderabbitai]

📝 Walkthrough

Walkthrough

This PR relocates and refactors the create-architecture workflow from create-architecture/ to bmad-create-architecture/, updating the agent menu exec path and normalizing all step file references from absolute project-root paths to relative paths.

Changes

Cohort / File(s)	Summary
Agent Configuration src/bmm/agents/architect.agent.yaml	Updated exec path from create-architecture/workflow.md to bmad-create-architecture/workflow.md to point to the new workflow location.
New Workflow Structure src/bmm/workflows/3-solutioning/bmad-create-architecture/workflow.md, SKILL.md, bmad-skill-manifest.yaml	Introduced new workflow definition with renamed workflow from "create-architecture" to "bmad-create-architecture", installed_path changed to relative ".", and added skill documentation and manifest files.
Workflow Steps src/bmm/workflows/3-solutioning/bmad-create-architecture/steps/step-0{1,2,3,4,5,6,7}-*.md	Normalized all step file path references from absolute {project-root}/_bmad/bmm/workflows/... patterns to relative ./steps/ paths across all 7 step files.
Old Workflow Cleanup src/bmm/workflows/3-solutioning/create-architecture/bmad-skill-manifest.yaml	Removed workflow metadata entries (canonicalId, type, description) from the original manifest file.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• refactor: convert workflow.yaml to workflow.md (steps 1-3) #1842: Converts workflows to markdown-based format and updates agent.menu exec paths to point to new workflow.md locations.
• fix: standardize all relative step file refs to {project-root}/_bmad/ paths #1722: Modifies step file path references in the create-architecture workflow area; directly related though in opposite direction (absolute vs. relative paths).
• chore(correct-course): convert workflow.yaml to unified workflow.md #1858: Updates agent workflow references to point from older workflow files to unified workflow.md paths via agent YAML exec modifications.

Suggested reviewers

• bmadcode

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The pull request title accurately summarizes the main change: converting a workflow into a native skill, which aligns with the substantial refactoring shown across multiple files and directories.
Description check	✅ Passed	The pull request description is comprehensive and directly related to the changeset, detailing the mechanical conversion, path updates, dependent reference changes, and validation performed.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

📝 Coding Plan

• Generate coding plan for human review comments

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

🧹 Nitpick comments (1)

src/bmm/workflows/3-solutioning/bmad-create-architecture/steps/step-01-init.md (1)

149-152: Minor: "decision making" should be hyphenated.

Static analysis flagged "decision making" should be "decision-making" when used as a compound modifier. This is a minor grammar nit.

📝 Proposed fix

-After user selects [C] to continue, only after ensuring all the template output has been created, then load `./steps/step-02-context.md` to analyze the project context and begin architectural decision making.
+After user selects [C] to continue, only after ensuring all the template output has been created, then load `./steps/step-02-context.md` to analyze the project context and begin architectural decision-making.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In
`@src/bmm/workflows/3-solutioning/bmad-create-architecture/steps/step-01-init.md`
around lines 149 - 152, Replace the unhyphenated compound "decision making" in
the NEXT STEP paragraph with the correct hyphenated form "decision-making" so
the compound modifier is grammatically correct; locate the text in
step-01-init.md (the "NEXT STEP:" block that references loading
./steps/step-02-context.md) and update the phrase accordingly.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In
`@src/bmm/workflows/3-solutioning/bmad-create-architecture/steps/step-01-init.md`:
- Around line 149-152: Replace the unhyphenated compound "decision making" in
the NEXT STEP paragraph with the correct hyphenated form "decision-making" so
the compound modifier is grammatically correct; locate the text in
step-01-init.md (the "NEXT STEP:" block that references loading
./steps/step-02-context.md) and update the phrase accordingly.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 0ce1d17e-a52f-40fc-b73d-a67982731d7f

📥 Commits

Reviewing files that changed from the base of the PR and between 7b4875b and 429bc04.

⛔ Files ignored due to path filters (3)

• src/bmm/module-help.csv is excluded by !**/*.csv
• src/bmm/workflows/3-solutioning/bmad-create-architecture/data/domain-complexity.csv is excluded by !**/*.csv
• src/bmm/workflows/3-solutioning/bmad-create-architecture/data/project-types.csv is excluded by !**/*.csv

📒 Files selected for processing (15)

• src/bmm/agents/architect.agent.yaml
• src/bmm/workflows/3-solutioning/bmad-create-architecture/SKILL.md
• src/bmm/workflows/3-solutioning/bmad-create-architecture/architecture-decision-template.md
• src/bmm/workflows/3-solutioning/bmad-create-architecture/bmad-skill-manifest.yaml
• src/bmm/workflows/3-solutioning/bmad-create-architecture/steps/step-01-init.md
• src/bmm/workflows/3-solutioning/bmad-create-architecture/steps/step-01b-continue.md
• src/bmm/workflows/3-solutioning/bmad-create-architecture/steps/step-02-context.md
• src/bmm/workflows/3-solutioning/bmad-create-architecture/steps/step-03-starter.md
• src/bmm/workflows/3-solutioning/bmad-create-architecture/steps/step-04-decisions.md
• src/bmm/workflows/3-solutioning/bmad-create-architecture/steps/step-05-patterns.md
• src/bmm/workflows/3-solutioning/bmad-create-architecture/steps/step-06-structure.md
• src/bmm/workflows/3-solutioning/bmad-create-architecture/steps/step-07-validation.md
• src/bmm/workflows/3-solutioning/bmad-create-architecture/steps/step-08-complete.md
• src/bmm/workflows/3-solutioning/bmad-create-architecture/workflow.md
• src/bmm/workflows/3-solutioning/create-architecture/bmad-skill-manifest.yaml

💤 Files with no reviewable changes (1)

• src/bmm/workflows/3-solutioning/create-architecture/bmad-skill-manifest.yaml