@sweny-ai/core 0.1.140 → 0.1.142

package/dist/cli/main.js

@@ -34,7 +34,7 @@ import { createVerboseToolObserver } from "./verbose-observer.js";
 import { createRunLogger, renderReceiptLine, summarizeRun, writeStepSummary, WORKFLOW_RUN_DESCRIPTION, WORKFLOW_RUN_OPTIONS, } from "./run-output.js";
 import { writeRunComment } from "./comment-output.js";
 import { registerTriageCommand, registerImplementCommand, parseCliInputs, validateInputs, validateWarnings, parsePositiveInt, } from "./config.js";
-import { c, formatBanner, getStepDetails, formatStepLine, formatDagResultHuman, formatDagResultMarkdown, formatResultJson, formatValidationErrors, formatCrashError, formatCheckResults, } from "./output.js";
+import { c, formatBanner, getStepDetails, formatStepLine, formatDagResultHuman, formatDagResultMarkdown, writeResultJson, formatValidationErrors, formatCrashError, formatCheckResults, } from "./output.js";
 import { checkProviderConnectivity, detectClaudeCodeLogin, discoverWorkflowSkillIds, validateCheckInputs, } from "./check.js";
 import { registerSetupCommand } from "./setup.js";
 import { registerPublishCommand } from "./publish.js";
@@ -392,7 +392,7 @@ triageCmd.action(async (options) => {
         const durationMs = Date.now() - runStart;
         // Output
         if (config.json) {
-            console.log(formatResultJson(results));
+            await writeResultJson(results);
         }
         else {
             console.log(formatDagResultHuman(results, durationMs, config));
@@ -477,7 +477,7 @@ triageCmd.action(async (options) => {
     }
 });
 // ── sweny implement ───────────────────────────────────────────────────
-const implementCmd = registerImplementCommand(program);
+const implementCmd = registerImplementCommand(program).option("--json", "Output result as JSON", false);
 implementCmd.action(async (issueId, options) => {
     const fileConfig = loadConfigFile();
     // Build a minimal CliConfig for the implement command by merging CLI opts with env/file
@@ -604,17 +604,22 @@ implementCmd.action(async (issueId, options) => {
         catch {
             // silent
         }
+        if (config.json) {
+            await writeResultJson(results);
+        }
         if (hasFailed) {
             console.error(chalk.red(`\n  Implement workflow failed\n`));
             process.exit(1);
         }
-        const prResult = results.get("create_pr");
-        const prUrl = prResult?.data?.prUrl;
-        if (prUrl) {
-            console.log(chalk.green(`\n  PR created: ${prUrl}\n`));
-        }
-        else {
-            console.log(chalk.green(`\n  Implement workflow completed\n`));
+        if (!config.json) {
+            const prResult = results.get("create_pr");
+            const prUrl = prResult?.data?.prUrl;
+            if (prUrl) {
+                console.log(chalk.green(`\n  PR created: ${prUrl}\n`));
+            }
+            else {
+                console.log(chalk.green(`\n  Implement workflow completed\n`));
+            }
         }
         // Legacy /api/report back-compat — see triage path.
         try {
@@ -956,7 +961,7 @@ export async function workflowRunAction(file, options) {
             });
         }
         if (isJson) {
-            process.stdout.write(JSON.stringify(Object.fromEntries(results), null, 2) + "\n");
+            await writeResultJson(results);
             process.exit(wfHasFailed ? 1 : 0);
             return;
         }

package/dist/cli/output.d.ts

@@ -24,6 +24,8 @@ export declare function extractCredentialHint(err: unknown): string | null;
 export declare function formatCrashError(error: unknown): string;
 export declare function formatCheckResults(results: CheckResult[]): string;
 export declare function formatResultJson(results: Map<string, NodeResult>): string;
+/** Drain terminal JSON before process.exit(), including when stdout is a pipe. */
+export declare function writeResultJson(results: Map<string, NodeResult>): Promise<void>;
 /**
  * Format triage results as GitHub-flavored markdown for `$GITHUB_STEP_SUMMARY`.
  *

package/dist/cli/output.js

@@ -405,6 +405,17 @@ export function formatCheckResults(results) {
 export function formatResultJson(results) {
     return JSON.stringify(Object.fromEntries(results), null, 2);
 }
+/** Drain terminal JSON before process.exit(), including when stdout is a pipe. */
+export function writeResultJson(results) {
+    return new Promise((resolve, reject) => {
+        process.stdout.write(formatResultJson(results) + "\n", (error) => {
+            if (error)
+                reject(error);
+            else
+                resolve();
+        });
+    });
+}
 export function formatDagResultMarkdown(results, durationMs, config, options = {}) {
     const duration = formatDuration(durationMs);
     const lines = [];

package/dist/executor.js

@@ -171,8 +171,8 @@ export async function execute(workflow, input, options) {
         // Build context: input + all prior node results.
         // Each prior node entry is the node's `data` augmented with `evals`
         // (a Record<name, EvalResult> for downstream lookup like
-        // `priorNode.evals.tests_run_clean.pass`). When `data` already has an
-        // `evals` key, the data field wins (back-compat with existing workflows).
+        // `priorNode.evals.tests_run_clean.pass`). This namespace is reserved
+        // for runtime verdicts; an agent-provided `data.evals` never overrides it.
         const context = {
             input,
             ...Object.fromEntries([...results.entries()].map(([k, v]) => [k, buildPriorNodeContext(v)])),
@@ -488,23 +488,15 @@ function warnOnJudgeBudget(workflow, logger) {
  *
  * Spec contract (https://spec.sweny.ai/nodes/#evalresult-type): downstream
  * nodes can read `priorNode.evals.<name>.pass` via the natural lookup path.
- * To honor this without breaking back-compat with workflows that read
- * `priorNode.<dataField>` directly, we spread `data` first and then attach
- * an `evals` namespace keyed by evaluator name. If the agent's structured
- * output happens to include a literal `evals` field, that field wins (and
- * the lookup degrades to "evals not available" for downstream readers).
+ * Preserve ordinary data fields, but reserve `evals` for runtime verdicts.
+ * Agent output must not forge a pass for downstream `requires` gates,
+ * including when no evaluator ran (the trusted namespace is then empty).
  */
 function buildPriorNodeContext(result) {
     const data = (result.data ?? {});
     const evals = result.evals ?? [];
-    if (evals.length === 0)
-        return data;
-    const evalsByName = {};
-    for (const e of evals) {
-        evalsByName[e.name] = e;
-    }
-    // Spread evals first so a data-side `evals` key shadows it (back-compat).
-    return { evals: evalsByName, ...data };
+    const evalsByName = Object.fromEntries(evals.map((e) => [e.name, e]));
+    return { ...data, evals: evalsByName };
 }
 /**
  * Build the prior-node entry shown to the LLM route evaluator.
@@ -564,14 +556,9 @@ function buildRouteEvalEntry(result, sourceNode) {
         dataView = fullData;
     }
     const evals = result.evals ?? [];
-    if (evals.length === 0)
-        return { view: dataView, missing };
-    const evalsByName = {};
-    for (const e of evals) {
-        evalsByName[e.name] = e;
-    }
-    // Spread evals first so a data-side `evals` key shadows it (back-compat).
-    return { view: { evals: evalsByName, ...dataView }, missing };
+    const evalsByName = Object.fromEntries(evals.map((e) => [e.name, e]));
+    // Schema projection never grants agent data authority over runtime verdicts.
+    return { view: { ...dataView, evals: evalsByName }, missing };
 }
 /**
  * Return the declared output property names for a node, or null when the

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sweny-ai/core",
-  "version": "0.1.140",
+  "version": "0.1.142",
   "type": "module",
   "bin": {
     "sweny": "./dist/cli/main.js"