keystone-cli 2.1.8 → 2.1.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "keystone-cli",
-  "version": "2.1.8",
+  "version": "2.1.10",
   "description": "A local-first, declarative, agentic workflow orchestrator built on Bun",
   "type": "module",
   "bin": {

package/src/runner/executors/dynamic-executor.ts

@@ -269,6 +269,12 @@ export async function executeDynamicStep(
 
     return buildFinalResult(state);
   } catch (error) {
+    // Re-throw suspension/waiting errors so the workflow can properly suspend
+    if (error instanceof Error) {
+      if (error.name === 'WorkflowSuspendedError' || error.name === 'WorkflowWaitingError') {
+        throw error;
+      }
+    }
     return await handleExecutionError(step, state, dbState, stateManager, options.saveState, error);
   }
 }

package/src/runner/workflow-state.ts

@@ -324,8 +324,17 @@ export class WorkflowState {
           try {
             output = JSON.parse(exec.output);
           } catch (e) {
-            /* ignore */
+            this.logger.warn(
+              `Failed to parse output for step "${stepId}": ${e instanceof Error ? e.message : String(e)}`
+            );
+            // If parsing fails, try using the raw output
+            output = exec.output;
           }
+        } else if (exec.status === 'success') {
+          // If step succeeded but has no output, log a warning
+          this.logger.warn(
+            `Step "${stepId}" completed with status "${exec.status}" but has no output. This may cause issues for dependent steps.`
+          );
         }
 
         this.stepContexts.set(stepId, {

package/src/templates/scaffolding/scaffold-feature.yaml

@@ -42,14 +42,14 @@ steps:
     needs: [generate]
     foreach: ${{ steps.generate.outputs.files.filter(f => f.path.endsWith('.yaml') && f.path.includes('workflows')) }}
     run: |
-      echo "${{ item.content }}" | npx tsx src/cli.ts validate --stdin --type workflow
+      echo "${{ item.content }}" | bun run src/cli.ts validate --stdin --type workflow
     allowFailure: true
     transform: |
       {
         path: item.path,
         content: item.content,
-        valid: result.exitCode === 0,
-        error: result.exitCode !== 0 ? result.stderr : null
+        valid: exitCode === 0,
+        error: exitCode !== 0 ? stderr : null
       }
 
   - id: repair_workflows
@@ -76,7 +76,7 @@ steps:
       3. A `steps` array with at least one step
       4. Each step must have `id` and `type` fields
 
-      Please return the corrected YAML content that conforms to the schema. Return ONLY the fixed YAML content, nothing else.
+      Return a JSON object with a single "content" field containing the corrected YAML as a string.
     outputSchema:
       type: object
       properties:
@@ -88,10 +88,15 @@ steps:
   - id: merge_files
     type: shell
     needs: [validate_workflows, repair_workflows]
+    env:
+      GENERATED: ${{ JSON.stringify(steps.generate.outputs.files) }}
+      VALIDATED: ${{ JSON.stringify(steps.validate_workflows.output) }}
+      REPAIRED: ${{ JSON.stringify(steps.repair_workflows.output) }}
     run: |
-      const generated = ${{ JSON.stringify(steps.generate.outputs.files) }};
-      const validated = ${{ JSON.stringify(steps.validate_workflows.output) }};
-      const repaired = ${{ JSON.stringify(steps.repair_workflows.output) }};
+      bun -e '
+      const generated = JSON.parse(process.env.GENERATED || "[]");
+      const validated = JSON.parse(process.env.VALIDATED || "[]");
+      const repaired = JSON.parse(process.env.REPAIRED || "[]");
       
       const repairedMap = new Map();
       if (Array.isArray(repaired)) {
@@ -110,7 +115,10 @@ steps:
         return file;
       });
       
-      return { files: final };
+      console.log(JSON.stringify({ files: final }));
+      '
+    transform: |
+      JSON.parse(stdout)
 
   - id: write_files
     type: file