ldy26098
/
BMAD-METHOD
mirror of https://github.com/bmad-code-org/BMAD-METHOD.git
1
0
Fork 0
Code Packages Projects Releases Wiki Activity

fix: add missing HELP_STEP placeholder replacement 

The activation-steps.txt template includes a {HELP_STEP} placeholder,
but activation-builder.js never calculated or replaced it. This caused
the literal string "{HELP_STEP}" to appear in compiled agent files.

Added helpStep calculation between menuStep and haltStep, and adjusted
subsequent step numbers accordingly.

Fixes #1441
This commit is contained in:
Michael Pursifull 2026-01-28 00:35:20 -06:00
parent 79394dda45
commit fcf96c4945
No known key found for this signature in database
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
tools/cli/lib/activation-builder.js
View File

@ -121,9 +121,10 @@ class ActivationBuilder {
// Calculate final step numbers // Calculate final step numbers
const menuStep = currentStepNum; const menuStep = currentStepNum;
const haltStep = currentStepNum + 1; const helpStep = currentStepNum + 1;
const inputStep = currentStepNum + 2; const haltStep = currentStepNum + 2;
const executeStep = currentStepNum + 3; const inputStep = currentStepNum + 3;
const executeStep = currentStepNum + 4;
// Replace placeholders // Replace placeholders
const processed = stepsTemplate const processed = stepsTemplate
@ -131,6 +132,7 @@ class ActivationBuilder {
.replace('{{module}}', metadata.module || 'core') // Fixed to use {{module}} .replace('{{module}}', metadata.module || 'core') // Fixed to use {{module}}
.replace('{AGENT_SPECIFIC_STEPS}', agentStepsXml) .replace('{AGENT_SPECIFIC_STEPS}', agentStepsXml)
.replace('{MENU_STEP}', menuStep.toString()) .replace('{MENU_STEP}', menuStep.toString())
.replace('{HELP_STEP}', helpStep.toString())
.replace('{HALT_STEP}', haltStep.toString()) .replace('{HALT_STEP}', haltStep.toString())
.replace('{INPUT_STEP}', inputStep.toString()) .replace('{INPUT_STEP}', inputStep.toString())
.replace('{EXECUTE_STEP}', executeStep.toString()); .replace('{EXECUTE_STEP}', executeStep.toString());