From fcf96c49457b99e15b1b0d12ebe66ed36a560a89 Mon Sep 17 00:00:00 2001
From: Michael Pursifull <mike@arcaven.com>
Date: Wed, 28 Jan 2026 00:35:20 -0600
Subject: [PATCH] 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
---
 tools/cli/lib/activation-builder.js | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/tools/cli/lib/activation-builder.js b/tools/cli/lib/activation-builder.js
index 9b91c2a9..81e11158 100644
--- a/tools/cli/lib/activation-builder.js
+++ b/tools/cli/lib/activation-builder.js
@@ -121,9 +121,10 @@ class ActivationBuilder {
 
     // Calculate final step numbers
     const menuStep = currentStepNum;
-    const haltStep = currentStepNum + 1;
-    const inputStep = currentStepNum + 2;
-    const executeStep = currentStepNum + 3;
+    const helpStep = currentStepNum + 1;
+    const haltStep = currentStepNum + 2;
+    const inputStep = currentStepNum + 3;
+    const executeStep = currentStepNum + 4;
 
     // Replace placeholders
     const processed = stepsTemplate
@@ -131,6 +132,7 @@ class ActivationBuilder {
       .replace('{{module}}', metadata.module || 'core') // Fixed to use {{module}}
       .replace('{AGENT_SPECIFIC_STEPS}', agentStepsXml)
       .replace('{MENU_STEP}', menuStep.toString())
+      .replace('{HELP_STEP}', helpStep.toString())
       .replace('{HALT_STEP}', haltStep.toString())
       .replace('{INPUT_STEP}', inputStep.toString())
       .replace('{EXECUTE_STEP}', executeStep.toString());