Skip to content

fix: complete sharding of market research customer analysis steps#1486

Merged
bmadcode merged 2 commits intobmad-code-org:mainfrom
alexeyv:fix/1399-orphan-market-research-steps
Feb 1, 2026
Merged

fix: complete sharding of market research customer analysis steps#1486
bmadcode merged 2 commits intobmad-code-org:mainfrom
alexeyv:fix/1399-orphan-market-research-steps

Conversation

@alexeyv
Copy link
Copy Markdown
Collaborator

@alexeyv alexeyv commented Jan 31, 2026

Connect orphaned detailed customer analysis chain (steps 02-behavior, 03-pain-points, 04-decisions) by updating step-01-init to load step-02-customer-behavior instead of the condensed step-02-customer-insights. Remove the now-superseded monolithic file.

Fixes #1399

Connect orphaned detailed customer analysis chain (steps 02-behavior,
03-pain-points, 04-decisions) by updating step-01-init to load
step-02-customer-behavior instead of the condensed
step-02-customer-insights. Remove the now-superseded monolithic file.

Fixes bmad-code-org#1399
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 31, 2026

📝 Walkthrough

Walkthrough

The changes complete workflow sharding by redirecting the market research entry point from the condensed customer-insights analysis to the detailed customer-behavior analysis and removing the now-obsolete customer-insights file. This shifts the workflow from a 4-step to 6-step process with more comprehensive customer analysis coverage.

Changes

Cohort / File(s) Summary
Workflow Entry Point Redirect
src/bmm/workflows/1-analysis/research/market-steps/step-01-init.md
Updated next-step target from step-02-customer-insights.md to step-02-customer-behavior.md, enabling access to the detailed 6-step analysis chain.
Obsolete Analysis Step Removal
src/bmm/workflows/1-analysis/research/market-steps/step-02-customer-insights.md
Deleted the condensed 200-line customer insights workflow file that is superseded by the multi-step customer-behavior/pain-points/decisions chain.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: completing the sharding of market research workflow steps by reconnecting the orphaned customer analysis chain.
Description check ✅ Passed The description accurately describes the changeset: updating step-01-init to load step-02-customer-behavior instead of step-02-customer-insights, removing the superseded file, and fixing issue #1399.
Linked Issues check ✅ Passed The PR fully implements Option 1 from issue #1399: updating step-01-init to load step-02-customer-behavior and removing the unused step-02-customer-insights file, completing the workflow sharding.
Out of Scope Changes check ✅ Passed All changes are directly related to the issue objectives: one file updated to load the correct next step, and the superseded file removed as intended to complete the sharding.
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.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
src/bmm/workflows/1-analysis/research/market-steps/step-01-init.md (1)

178-180: ⚠️ Potential issue | 🟠 Major

Fix contradictory next-step instruction (still points to removed file).

Line 180 conflicts with the updated next-step load and will send agents to a deleted step. Align it with the new flow.

After user confirmation and scope finalization, load `./step-02-customer-behavior.md` to begin detailed market research with customer behavior analysis.

@bmadcode bmadcode merged commit ef7abb7 into bmad-code-org:main Feb 1, 2026
5 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
dickymoore pushed a commit to dickymoore/BMAD-METHOD that referenced this pull request Feb 6, 2026
…ad-code-org#1486)

Connect orphaned detailed customer analysis chain (steps 02-behavior,
03-pain-points, 04-decisions) by updating step-01-init to load
step-02-customer-behavior instead of the condensed
step-02-customer-insights. Remove the now-superseded monolithic file.

Fixes bmad-code-org#1399

Co-authored-by: Brian <bmadcode@gmail.com>
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.

Orphan Step Files in Market Research Workflow

2 participants