@formigio/fazemos-cli 0.10.1 → 0.10.4

package/dist/index.d.ts

@@ -1,2 +1,4 @@
 #!/usr/bin/env node
-export {};
+import { Command } from 'commander';
+declare const program: Command;
+export { program };

package/dist/index.js

@@ -4212,7 +4212,8 @@ step
     .option('--callback-token <token>', 'Callback token')
     .action(async (instanceId, stepId, opts) => {
     try {
-        const body = { text: opts.text };
+        // I19: API expects `append` not `text`. Flag stays --text (operator muscle memory).
+        const body = { append: opts.text };
         if (opts.callbackToken)
             body.callbackToken = opts.callbackToken;
         await api('PATCH', `/api/pipeline-instances/${instanceId}/steps/${stepId}/output`, body);
@@ -4368,12 +4369,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 +4392,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);
     }
 });
@@ -7201,5 +7233,12 @@ docs
         handleScopedError(err);
     }
 });
-program.parse();
+// Only auto-parse when invoked as the entry point (i.e. `node dist/index.js ...`
+// or `npx tsx src/index.ts ...`). When imported as a module — by tests using
+// `vi.mock` to intercept `api()` and asserting on `program.parseAsync(...)` —
+// skip auto-parse so the test controls invocation.
+const isMainModule = import.meta.url === `file://${process.argv[1]}`;
+if (isMainModule)
+    program.parse();
+export { program };
 //# sourceMappingURL=index.js.map