@formigio/fazemos-cli 0.10.0 → 0.10.2

package/dist/index.js

@@ -4298,7 +4298,7 @@ step
     .action(async (instanceId, stepId, opts) => {
     try {
         const data = await api('POST', `/api/pipeline-instances/${instanceId}/steps/${stepId}/revise`, {
-            reason: opts.reason,
+            feedback: opts.reason,
         });
         console.log(chalk.green(`Revision requested: ${data.step?.step_name || stepId}`));
         console.log(`  Status: ${data.step?.status}`);
@@ -4368,12 +4368,16 @@ step
     .argument('<stepId>', 'Step instance ID')
     .requiredOption('-f, --feedback <text>', 'What the step got wrong and what to fix (required)')
     .option('--no-cascade', 'Skip resetting downstream steps')
+    .option('--cascade-through-skipped', 'Include skipped steps in the reset cascade. By default, cascade stops when it reaches a skipped step. ' +
+    'Use this flag when the skipped step was pre-skipped incorrectly and should be treated as blocked pending new output.')
     .action(async (instanceId, stepId, opts) => {
     try {
-        const data = await api('POST', `/api/pipeline-instances/${instanceId}/steps/${stepId}/reset`, {
+        const body = {
             feedback: opts.feedback,
             cascade: opts.cascade,
-        });
+            cascade_through_skipped: opts.cascadeThroughSkipped === true,
+        };
+        const data = await api('POST', `/api/pipeline-instances/${instanceId}/steps/${stepId}/reset`, body);
         const stepName = data.step?.step_name || stepId;
         const status = data.step?.status || 'unknown';
         console.log(chalk.green(`Step reset: "${stepName}" (${status})`));
@@ -4387,18 +4391,45 @@ step
         }
         else if (!cascadeResult || cascadeResult.blocked_count === 0) {
             console.log(`  Cascade: no downstream steps to block`);
+            // I13: inform operator when flag was used but no skipped steps were found
+            if (opts.cascadeThroughSkipped) {
+                console.log(`  (No skipped steps in cascade path)`);
+            }
         }
         else {
             console.log(`  Cascade: ${cascadeResult.blocked_count} downstream step${cascadeResult.blocked_count === 1 ? '' : 's'} blocked`);
             if (cascadeResult.blocked_steps?.length) {
                 for (const bs of cascadeResult.blocked_steps) {
-                    console.log(`    - ${bs.step_name} (was: ${bs.previous_status})`);
+                    // I13: annotate cascade-blocked skipped steps
+                    const skippedNote = bs.previous_status === 'skipped' ? '  ← cascade-through-skipped' : '';
+                    console.log(`    - ${bs.step_name} (was: ${bs.previous_status})${skippedNote}`);
+                }
+            }
+            // I13: inform operator when flag was used but no skipped steps were encountered
+            if (opts.cascadeThroughSkipped) {
+                const hasSkippedSteps = cascadeResult.blocked_steps?.some((bs) => bs.previous_status === 'skipped');
+                if (!hasSkippedSteps) {
+                    console.log(`  (No skipped steps in cascade path)`);
                 }
             }
         }
     }
     catch (err) {
-        console.error(chalk.red(err.message));
+        if (err.code === 'DOWNSTREAM_RUNNING') {
+            console.error(chalk.red(err.message));
+            console.error('');
+            console.error(`Cancel the running step first, then retry:`);
+            // I13: echo --cascade-through-skipped in retry suggestion when flag was used
+            const cascadeFlag = opts.cascadeThroughSkipped ? ' --cascade-through-skipped' : '';
+            console.error(`  fazemos pl step reset ${instanceId} ${stepId} --feedback "..."${cascadeFlag}`);
+        }
+        else if (err.code === 'INVALID_STATE') {
+            console.error(chalk.red(`Error: ${err.message}`));
+            console.error(`Reset only works on completed steps. Use 'retry' for failed steps.`);
+        }
+        else {
+            console.error(chalk.red(err.message));
+        }
         process.exit(1);
     }
 });