Skip to content

Comments

fix: correct relative path to prd-purpose.md in step-11-polish#1497

Merged
bmadcode merged 1 commit intobmad-code-org:mainfrom
arcaven:fix/step-11-polish-relative-path
Feb 1, 2026
Merged

fix: correct relative path to prd-purpose.md in step-11-polish#1497
bmadcode merged 1 commit intobmad-code-org:mainfrom
arcaven:fix/step-11-polish-relative-path

Conversation

@arcaven
Copy link
Contributor

@arcaven arcaven commented Jan 31, 2026

Summary

The purposeFile reference in step-11-polish.md uses ./data/prd-purpose.md but the data/ directory is a sibling of the steps-c/ directory, not inside it. The correct relative path from steps-c/ is ../data/prd-purpose.md.

File: src/bmm/workflows/2-plan-workflows/create-prd/steps-c/step-11-polish.md (line 8)

- purposeFile: './data/prd-purpose.md'
+ purposeFile: '../data/prd-purpose.md'

Directory structure:

create-prd/
├── data/
│   └── prd-purpose.md    ← target file
├── steps-c/
│   └── step-11-polish.md ← referencing file (./data/ is wrong, ../data/ is correct)
└── ...

Fixes #1495

🔍 This broken reference was detected by the cross-file reference validator proposed in #1494. With that validator in CI, this class of bug would be caught automatically at PR time.

Test plan

  • npm test passes (all linting, schema validation, formatting checks)
  • Verified ../data/prd-purpose.md resolves to the correct file

The purposeFile reference uses ./data/prd-purpose.md but data/ is a
sibling of steps-c/, not a child. Corrected to ../data/prd-purpose.md.

Fixes bmad-code-org#1495
@coderabbitai
Copy link

coderabbitai bot commented Jan 31, 2026

📝 Walkthrough

Walkthrough

A relative path reference in a workflow markdown file is corrected from ./data/prd-purpose.md to ../data/prd-purpose.md to point to the correct sibling directory instead of a non-existent child directory.

Changes

Cohort / File(s) Summary
Relative Path Fix
src/bmm/workflows/2-plan-workflows/create-prd/steps-c/step-11-polish.md
Corrected relative path for purposeFile reference from ./data/prd-purpose.md to ../data/prd-purpose.md to properly point to the data directory at the correct hierarchy level.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related issues

Suggested reviewers

  • alexeyv
  • pbean
  • muratkeremozcan
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: fixing an incorrect relative path in step-11-polish.md.
Description check ✅ Passed The description is directly related to the changeset, explaining the path correction, directory structure, and verification steps.
Linked Issues check ✅ Passed The pull request directly addresses issue #1495 by implementing the exact fix specified: changing './data/prd-purpose.md' to '../data/prd-purpose.md'.
Out of Scope Changes check ✅ Passed The change is narrowly scoped to correcting the single relative path reference specified in issue #1495 with no extraneous modifications.
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

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

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

@bmadcode bmadcode merged commit 0a7329f into bmad-code-org:main Feb 1, 2026
7 checks passed
bmadcode pushed a commit that referenced this pull request Feb 1, 2026
### 🎁 Features
* Add generate-project-context workflow
* Shard market research customer analysis

### 🐛 Bug Fixes
* Fix npm install peer dependency issues (PR #1476)
* Fix leaked source paths in PRD validation report (#1481)
* Fix orphaned market research customer analysis (#1486)
* Fix duplicate 2-letter brainstorming code (BS → BSP)
* Fix tech writer sidecar functionality (#1487)
* Fix relative paths in workflow steps (#1497, #1498)
* Fix party-mode workflow file extension (#1499)
* Fix generated slash commands (#1501)
* Fix agent scan and help CSV files
* Fix HELP_STEP placeholder replacement (#1437)

### 📚 Documentation
* Add exact slash commands to Getting Started guide (#1505)
* Remove .claude/commands from version control (#1506)

### 🔧 Maintenance
* Update Starlight to 0.37.5
* Add GitHub issue templates
@arcaven arcaven deleted the fix/step-11-polish-relative-path branch February 5, 2026 22:05
dickymoore pushed a commit to dickymoore/BMAD-METHOD that referenced this pull request Feb 6, 2026
…code-org#1497)

The purposeFile reference uses ./data/prd-purpose.md but data/ is a
sibling of steps-c/, not a child. Corrected to ../data/prd-purpose.md.

Fixes bmad-code-org#1495
dickymoore pushed a commit to dickymoore/BMAD-METHOD that referenced this pull request Feb 6, 2026
### 🎁 Features
* Add generate-project-context workflow
* Shard market research customer analysis

### 🐛 Bug Fixes
* Fix npm install peer dependency issues (PR bmad-code-org#1476)
* Fix leaked source paths in PRD validation report (bmad-code-org#1481)
* Fix orphaned market research customer analysis (bmad-code-org#1486)
* Fix duplicate 2-letter brainstorming code (BS → BSP)
* Fix tech writer sidecar functionality (bmad-code-org#1487)
* Fix relative paths in workflow steps (bmad-code-org#1497, bmad-code-org#1498)
* Fix party-mode workflow file extension (bmad-code-org#1499)
* Fix generated slash commands (bmad-code-org#1501)
* Fix agent scan and help CSV files
* Fix HELP_STEP placeholder replacement (bmad-code-org#1437)

### 📚 Documentation
* Add exact slash commands to Getting Started guide (bmad-code-org#1505)
* Remove .claude/commands from version control (bmad-code-org#1506)

### 🔧 Maintenance
* Update Starlight to 0.37.5
* Add GitHub issue templates
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: broken relative path in create-prd step-11-polish.md

2 participants