fix(workflows): adjust loop steps count and target index calculation

Update bmad-review loop steps from 2 to 3 and fix target index calculation in post step directives

Author: moazbuilds

src/workflows/step/hooks.ts

@@ -217,10 +217,10 @@ export async function processPostStepDirectives(context: ProcessPostStepContext)
     if (postResult.newActiveLoop !== undefined) {
       ctx.setActiveLoop(postResult.newActiveLoop);
     }
-    // Return target index (add 1 because newIndex is 0-based target - 1)
+    // Return target index as-is (INPUT_RECEIVED will increment by 1)
     return {
       type: 'loop',
-      targetIndex: postResult.newIndex + 1,
+      targetIndex: postResult.newIndex,
       newActiveLoop: postResult.newActiveLoop ?? null,
     };
   }

templates/workflows/bmad.workflow.js

@@ -38,6 +38,6 @@ export default {
     separator("⟲ Story Loop ⟲"),
     resolveStep('bmad-stories', {interactive: false}),
     resolveStep('bmad-dev', {interactive: false}),
-    resolveModule('bmad-review', {interactive: false, loopSteps: 2 }),
+    resolveModule('bmad-review', {interactive: false, loopSteps: 3 }),
   ],
 };