@sweny-ai/core 0.1.145 → 0.1.146

package/dist/harness/codex.d.ts

@@ -20,7 +20,7 @@
  *
  * Every sweny opinion in the Codex column of the enforcement matrix is
  * enforced natively here, kept by sweny (watchdog, fencing, env scoping,
- * output validation), reported in `degraded`, or refused under strict policy
+ * output parsing), reported in `degraded`, or refused under strict policy
  * by `policyGate`. Nothing is dropped silently.
  */
 import type { Logger, McpServerConfig, ToolContext } from "../types.js";

package/dist/harness/codex.js

@@ -20,7 +20,7 @@
  *
  * Every sweny opinion in the Codex column of the enforcement matrix is
  * enforced natively here, kept by sweny (watchdog, fencing, env scoping,
- * output validation), reported in `degraded`, or refused under strict policy
+ * output parsing), reported in `degraded`, or refused under strict policy
  * by `policyGate`. Nothing is dropped silently.
  */
 import { spawn, execFile } from "node:child_process";
@@ -201,15 +201,21 @@ export function isStrictCompatibleSchema(schema) {
 // ─── Process registry (crash cleanup) ────────────────────────────
 const liveChildren = new Set();
 let exitHookInstalled = false;
-/**
- * Stop Codex. It stays in sweny's process group (not detached), so a Ctrl-C
- * or a CI cancel reaches it too; its MCP servers exit when their stdin closes.
- */
+/** Stop the owned process tree, even after its leader has exited. */
 function stopChild(child, signal) {
-    if (child.pid === undefined || child.exitCode !== null || child.signalCode !== null)
+    if (child.pid === undefined)
         return;
     try {
-        child.kill(signal);
+        if (process.platform !== "win32") {
+            // exec() gives each run its own process group. Descendants can retain
+            // stdout/stderr after the leader exits, so do not gate this on exitCode.
+            process.kill(-child.pid, signal);
+        }
+        else {
+            // Windows has no POSIX process groups. Bound the tree-kill utility too.
+            execFile("taskkill", ["/pid", String(child.pid), "/T", "/F"], { timeout: 1000 }, () => { });
+            child.kill(signal);
+        }
     }
     catch {
         // already gone
@@ -220,10 +226,23 @@ function trackChild(child) {
     if (!exitHookInstalled) {
         exitHookInstalled = true;
         // A crashing sweny must not leave Codex (and its MCP servers) running.
-        process.on("exit", () => {
+        const cleanup = () => {
             for (const c of liveChildren)
                 stopChild(c, "SIGKILL");
-        });
+        };
+        process.on("exit", cleanup);
+        // Detached POSIX groups no longer receive our terminal's signals. Clean
+        // them up before host once-handlers remove themselves during dispatch.
+        for (const signal of ["SIGINT", "SIGTERM"]) {
+            const onSignal = () => {
+                cleanup();
+                if (process.listenerCount(signal) === 1) {
+                    process.removeListener(signal, onSignal);
+                    process.kill(process.pid, signal);
+                }
+            };
+            process.prependListener(signal, onSignal);
+        }
     }
 }
 function mapUsage(u) {
@@ -470,6 +489,7 @@ export class CodexHarness {
                 cwd: opts.spawn.cwd,
                 env: opts.spawn.env,
                 stdio: ["pipe", "pipe", "pipe"],
+                detached: process.platform !== "win32",
             });
         }
         catch (err) {
@@ -478,7 +498,9 @@ export class CodexHarness {
             return out;
         }
         trackChild(child);
+        let finish;
         const closed = new Promise((resolve) => {
+            finish = resolve;
             child.once("close", (code, signal) => {
                 out.exitCode = code;
                 out.exitSignal = signal;
@@ -495,9 +517,24 @@ export class CodexHarness {
                 return;
             stopped = true;
             stopChild(child, "SIGTERM");
-            killTimer = setTimeout(() => stopChild(child, "SIGKILL"), this.killGraceMs);
+            killTimer = setTimeout(() => {
+                stopChild(child, "SIGKILL");
+                // An escaped descendant or failed OS kill must not retain this await
+                // through inherited pipes after cancellation or leader exit.
+                child.stdin?.destroy();
+                child.stdout?.destroy();
+                child.stderr?.destroy();
+                child.unref();
+                finish();
+            }, this.killGraceMs);
             killTimer.unref?.();
         };
+        child.once("exit", (code, signal) => {
+            out.exitCode = code;
+            out.exitSignal = signal;
+            // The leader is done, but its descendants may still own our pipes.
+            stop();
+        });
         if (abort) {
             const onAbort = () => {
                 out.abortReason = abort.reason();
@@ -649,6 +686,7 @@ export class CodexHarness {
             onLine(buf);
         if (killTimer)
             clearTimeout(killTimer);
+        stopChild(child, "SIGKILL");
         abort?.clear();
         liveChildren.delete(child);
         return out;
@@ -677,11 +715,11 @@ export class CodexHarness {
             harness: this.info(),
             degraded: [...degraded, ...extra],
         });
-        const refused = (why) => {
+        const refused = (why, extra = []) => {
             const msg = `codex refused this node: ${why}`;
             this.logger.error(msg);
             // `refused` keeps fail_soft from softening a policy refusal (executor.ts).
-            return tag({ status: "failed", data: { error: msg, refused: true }, toolCalls: [] });
+            return tag({ status: "failed", data: { error: msg, refused: true }, toolCalls: [] }, extra);
         };
         if (gate.refuse)
             return refused(gate.refuse);
@@ -799,7 +837,7 @@ export class CodexHarness {
                 return refused(out.refuse);
             // OpenAI strict structured outputs reject some JSON schemas. When Codex
             // fails on the schema before doing anything, run once more without the
-            // native schema: sweny still asks for it in the prompt and checks it.
+            // native schema in relaxed mode. Prompt-only output is not schema-validated.
             if (schemaArgs.length > 0 &&
                 !out.completed &&
                 !out.abortReason &&
@@ -808,7 +846,10 @@ export class CodexHarness {
                 const why = isStrictCompatibleSchema(req.outputSchema)
                     ? "codex rejected the output schema"
                     : "the output schema is not OpenAI strict-compatible";
-                extraDegraded.push(`structured_output: ${why}; sweny parsed and checked the output instead`);
+                extraDegraded.push(`structured_output: ${why}; native schema validation unavailable`);
+                if (policy.strict) {
+                    return refused("strict policy requires structured_output validation; refusing prompt-only fallback", extraDegraded);
+                }
                 this.logger.warn(`  codex: ${why}; retrying without --output-schema`);
                 const again = await execOnce(false);
                 if ("refuse" in again)

package/package.json

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