jules-orchestrator-kit 0.37.0 → 0.38.0

package/README.md

@@ -122,7 +122,7 @@ Autonomous coding agents can write software at 100× human speed—but unconstra
 
 * **🚀 Zero-Test Bootstrapping (`agentctl bootstrap`):** Synthesizes deterministic syntax-check and smoke-test verification oracles for untested legacy repositories so agents always operate against a falsifiable feedback loop.
 
-* **📈 Proven Scale & Reliability:** Empirically tested with **547 unit tests across 80 suites passing in < 10.0s**. An adversarial red-team suite (`test/adversarial-claims.test.mjs`) continuously attempts to falsify the safety guarantees documented above — every probe in it currently holds, with no open gaps — including cross-platform probes for the case-insensitive filesystems on macOS and Windows — and a documentation-sync gate (`scripts/doc-sync-check.mjs`) blocks any release whose docs have drifted from the code.
+* **📈 Proven Scale & Reliability:** Empirically tested with **555 unit tests across 81 suites passing in < 10.0s**. An adversarial red-team suite (`test/adversarial-claims.test.mjs`) continuously attempts to falsify the safety guarantees documented above — every probe in it currently holds, with no open gaps — including cross-platform probes for the case-insensitive filesystems on macOS and Windows — and a documentation-sync gate (`scripts/doc-sync-check.mjs`) blocks any release whose docs have drifted from the code.
 
 <br/>
 

package/bin/agentctl.mjs

@@ -1278,6 +1278,10 @@ async function main() {
 }
 
 main().catch((err) => {
+  if (err.name === "WizardCancelledError" || err.code === 130 || (err instanceof Error && err.message?.includes("cancelled by user"))) {
+    console.log(`\n🛑 ${err.message || "Operation cancelled by user."}`);
+    process.exit(130);
+  }
   console.error(`[FATAL ERROR] ${err.message}`);
   const exitCode = typeof err.code === "number" ? err.code : 1;
   process.exit(exitCode);

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "jules-orchestrator-kit",
-  "version": "0.37.0",
-  "description": "Orchestration kit for running Google Jules autonomous agents.",
+  "version": "0.38.0",
+  "description": "Zero-dependency safety gatekeeper, test oracle generator, and multi-agent coordination protocol for Google Jules (jules) autonomous agents.",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/FullThrottle83/jules-orchestrator-kit.git"
@@ -36,7 +36,7 @@
   ],
   "scripts": {
     "init": "node bin/init.js",
-    "test": "node --test test/*.test.mjs",
+    "test": "node scripts/run-tests.mjs",
     "jules:dispatch": "node scripts/jules-dispatch.mjs",
     "jules:queue": "node bin/agentctl.mjs queue",
     "jules:create": "node scripts/jules-create.mjs",

package/scripts/doc-sync-check.mjs

@@ -186,6 +186,33 @@ export function checkDocSync(root = process.cwd(), opts = {}) {
         ? `v${unreleased.join(", v")} marked (Unreleased) but <= shipped v${version}`
         : "no released version left marked (Unreleased)"
     );
+
+    // An unchecked item under "Shipped Milestones" is a claim the release
+    // contradicts. v0.37.0 shipped base64 decoding while the deferral note
+    // under v0.32.6 still read "still does not decode" — two statements about
+    // the same feature, thirty lines apart, disagreeing. Both survived every
+    // other check here, because nothing compared a milestone to its own
+    // section heading. Deferrals belong under a target milestone, not a
+    // shipped one; moving the item is the fix, ticking it off is the other.
+    const shippedStart = roadmap.search(/^##\s.*Shipped Milestones/m);
+    if (shippedStart === -1) {
+      add("ROADMAP shipped items resolved", false, "no `## … Shipped Milestones` heading found");
+    } else {
+      const rest = roadmap.slice(shippedStart);
+      // Search from index 1 so the section's own heading cannot terminate it.
+      const nextHeading = rest.slice(1).search(/^##\s/m);
+      const section = nextHeading === -1 ? rest : rest.slice(0, nextHeading + 1);
+      const openItems = [...section.matchAll(/^-\s\[ \]\s+(.+)$/gm)].map((m) =>
+        m[1].replace(/\*\*/g, "").split("—")[0].trim().slice(0, 60)
+      );
+      add(
+        "ROADMAP shipped items resolved",
+        openItems.length === 0,
+        openItems.length
+          ? `unchecked under Shipped Milestones: ${openItems.join("; ")}`
+          : "every item under Shipped Milestones is resolved"
+      );
+    }
   }
 
   // 3. CHANGELOG.md — a released version needs a matching entry (release.mjs

package/scripts/run-tests.mjs

@@ -0,0 +1,19 @@
+#!/usr/bin/env node
+import { readdirSync } from "node:fs";
+import { join } from "node:path";
+import { spawnSync } from "node:child_process";
+
+const root = process.cwd();
+const testDir = join(root, "test");
+const testFiles = readdirSync(testDir)
+  .filter((f) => f.endsWith(".test.mjs"))
+  .sort()
+  .map((f) => join("test", f));
+
+const res = spawnSync(process.execPath, ["--test", ...testFiles], {
+  cwd: root,
+  stdio: "inherit",
+  env: process.env,
+});
+
+process.exit(res.status ?? 1);

package/src/git.mjs

@@ -1,4 +1,4 @@
-import { execFileSync } from "node:child_process";
+import { execFileSync, execSync } from "node:child_process";
 import { readFileSync, existsSync } from "node:fs";
 import { join } from "node:path";
 import { normalizePath } from "./config.mjs";
@@ -49,20 +49,26 @@ export function runCmd(command, opts = {}) {
   }
 
   try {
-    const isWin = process.platform === "win32";
-    const execBin = useShell ? (isWin ? (process.env.ComSpec || "cmd.exe") : "/bin/sh") : binary;
-    const execArgs = useShell ? (isWin ? ["/d", "/s", "/c", shellCmd] : ["-c", shellCmd]) : args;
-
-    const stdout = execFileSync(execBin, execArgs, {
-      cwd,
-      encoding: "utf-8",
-      shell: false,
-      stdio: ["ignore", "pipe", "pipe"],
-      env: opts.env || process.env,
-      timeout,
-      maxBuffer,
-    });
-    return { status: 0, stdout: stdout.trim(), stderr: "" };
+    const stdout = useShell
+      ? execSync(shellCmd, {
+          cwd,
+          encoding: "utf-8",
+          stdio: ["ignore", "pipe", "pipe"],
+          env: opts.env || process.env,
+          timeout,
+          maxBuffer,
+        })
+      : execFileSync(binary, args, {
+          cwd,
+          encoding: "utf-8",
+          shell: false,
+          stdio: ["ignore", "pipe", "pipe"],
+          env: opts.env || process.env,
+          timeout,
+          maxBuffer,
+        });
+
+    return { status: 0, stdout: String(stdout || "").trim(), stderr: "" };
   } catch (err) {
     const isTimeout = err.code === "ETIMEDOUT" || (err.signal === "SIGTERM" && err.killed);
     const isNobufs = err.code === "ENOBUFS" || (err.message && err.message.includes("maxBuffer"));

package/src/state.mjs

@@ -13,6 +13,7 @@ import {
 import { join, basename } from "node:path";
 import { createHash, randomUUID } from "node:crypto";
 import { hostname } from "node:os";
+import { execFileSync } from "node:child_process";
 import { resolveRoot } from "./config.mjs";
 import { redactSecrets } from "./security.mjs";
 
@@ -478,11 +479,17 @@ export function getProcessStartTime(pid) {
       return parseProcStat(stat);
     } catch (_) {}
   }
+  if (process.platform === "darwin") {
+    try {
+      const out = execFileSync("ps", ["-p", String(pid), "-o", "lstart="], { encoding: "utf-8", stdio: ["pipe", "pipe", "ignore"] });
+      return out.trim() || null;
+    } catch (_) {}
+  }
   return null;
 }
 
 /**
- * Checks if a PID is alive with PID-recycling start time validation on Linux.
+ * Checks if a PID is alive with PID-recycling start time validation on Linux and macOS.
  */
 export function isPidAlive(pid, expectedStartTime = null) {
   if (!pid || typeof pid !== "number") return false;
@@ -492,10 +499,10 @@ export function isPidAlive(pid, expectedStartTime = null) {
     if (err?.code !== "EPERM") return false;
   }
 
-  if (expectedStartTime !== null && expectedStartTime !== undefined && process.platform === "linux") {
+  if (expectedStartTime !== null && expectedStartTime !== undefined) {
     try {
       const actualStartTime = getProcessStartTime(pid);
-      if (!actualStartTime || String(actualStartTime) !== String(expectedStartTime)) {
+      if (actualStartTime && String(actualStartTime) !== String(expectedStartTime)) {
         return false;
       }
     } catch (_) {

package/src/tui.mjs

@@ -42,10 +42,11 @@ export function styleText(text, style) {
 }
 
 import { WizardCancelledError } from "./ux/terminal-session.mjs";
+import { createKeyDecoder } from "./ux/key-decoder.mjs";
 export { WizardCancelledError };
 
 /**
- * Read keypress in raw mode from TTY input stream.
+ * Read keypress in raw mode from TTY input stream without destroying stream on early return.
  * @param {import("node:stream").Readable} stream
  * @returns {AsyncGenerator<string, void, unknown>}
  */
@@ -54,20 +55,84 @@ async function* readKeypresses(stream = process.stdin) {
   if (stream.setRawMode) {
     stream.setRawMode(true);
   }
+
+  const decoder = createKeyDecoder();
+  const queue = [];
+  let waiting = null;
+  let error = null;
+
+  const onData = (chunk) => {
+    const events = decoder.push(chunk);
+    for (const ev of events) {
+      const str = ev.sequence;
+      if (waiting) {
+        const resolve = waiting;
+        waiting = null;
+        resolve({ value: str, done: false });
+      } else {
+        queue.push(str);
+      }
+    }
+  };
+
+  const onEnd = () => {
+    const flushed = decoder.flush();
+    for (const ev of flushed) {
+      queue.push(ev.sequence);
+    }
+    if (waiting) {
+      const resolve = waiting;
+      waiting = null;
+      if (queue.length > 0) {
+        resolve({ value: queue.shift(), done: false });
+      } else {
+        resolve({ done: true });
+      }
+    }
+  };
+
+  const onError = (err) => {
+    error = err;
+    if (waiting) {
+      const reject = waiting;
+      waiting = null;
+      reject(err);
+    }
+  };
+
+  stream.on("data", onData);
+  stream.on("end", onEnd);
+  stream.on("error", onError);
+
   stream.resume();
   stream.setEncoding("utf-8");
 
   try {
-    for await (const chunk of stream) {
-      const str = String(chunk);
-      if (str === "\u0003") {
+    while (true) {
+      if (error) throw error;
+      let nextVal;
+      if (queue.length > 0) {
+        nextVal = queue.shift();
+      } else {
+        const res = await new Promise((resolve) => {
+          waiting = resolve;
+        });
+        if (res.done) break;
+        nextVal = res.value;
+      }
+
+      if (nextVal === "\u0003") {
         // Ctrl+C SIGINT
         process.stdout.write(ANSI.showCursor + "\n");
         throw new WizardCancelledError("Wizard operation cancelled by user (SIGINT).");
       }
-      yield str;
+      yield nextVal;
     }
   } finally {
+    stream.off("data", onData);
+    stream.off("end", onEnd);
+    stream.off("error", onError);
+    stream.pause();
     if (stream.setRawMode) {
       stream.setRawMode(Boolean(wasRaw));
     }
@@ -135,11 +200,11 @@ export async function select(options, prompt, opts = {}) {
       }
 
       let moved = false;
-      if (key === "\u001b[A" || key === "k") {
+      if (key === "\u001b[A" || key === "\u001bOA" || key === "k") {
         // Up
         currentIdx = (currentIdx - 1 + options.length) % options.length;
         moved = true;
-      } else if (key === "\u001b[B" || key === "j") {
+      } else if (key === "\u001b[B" || key === "\u001bOB" || key === "j") {
         // Down
         currentIdx = (currentIdx + 1) % options.length;
         moved = true;
@@ -219,10 +284,10 @@ export async function multiSelect(options, prompt, opts = {}) {
       }
 
       let redraw = false;
-      if (key === "\u001b[A" || key === "k") {
+      if (key === "\u001b[A" || key === "\u001bOA" || key === "k") {
         currentIdx = (currentIdx - 1 + options.length) % options.length;
         redraw = true;
-      } else if (key === "\u001b[B" || key === "j") {
+      } else if (key === "\u001b[B" || key === "\u001bOB" || key === "j") {
         currentIdx = (currentIdx + 1) % options.length;
         redraw = true;
       } else if (key === " ") {
@@ -276,7 +341,20 @@ export async function input(prompt, opts = {}) {
     while (true) {
       const defaultHint = defaultValue ? ` ${ANSI.dim}(${defaultValue})${ANSI.reset}` : "";
       const query = `${ANSI.bold}${ANSI.cyan}?${ANSI.reset} ${ANSI.bold}${prompt}${defaultHint}: `;
-      const rawAns = await rl.question(query);
+      let rawAns;
+      try {
+        rawAns = await rl.question(query);
+      } catch (err) {
+        if (err.name === "AbortError" || err.code === "ABORT_ERR" || err.message?.includes("Ctrl+C")) {
+          stdout.write("\n");
+          throw new WizardCancelledError("Wizard operation cancelled by user (SIGINT).");
+        }
+        throw err;
+      }
+      if (rawAns.includes("\u0003") || rawAns.includes("\x03")) {
+        stdout.write("\n");
+        throw new WizardCancelledError("Wizard operation cancelled by user (SIGINT).");
+      }
       const ans = rawAns.trim() || defaultValue;
 
       const validRes = await validate(ans);

package/src/wizard-init.mjs

@@ -220,11 +220,20 @@ export function loadPresets(root = process.cwd()) {
 export async function runInitWizard(root = process.cwd(), options = {}) {
   const interactive = options.interactive !== false && isTTY(options.stdin || process.stdin);
 
-  if (!interactive && !options.allowDefaults && !options.tier) {
+  // Preserve existing config if present
+  let existingConfig = {};
+  const existingConfigPath = join(root, ".agent", "config.yml");
+  if (existsSync(existingConfigPath)) {
+    try {
+      existingConfig = parseYaml(readFileSync(existingConfigPath, "utf-8")) || {};
+    } catch (_) {}
+  }
+
+  if (!interactive && !options.allowDefaults && !options.tier && !existingConfig.tier) {
     throw new Error("Non-interactive init requires explicit options or allowDefaults: true");
   }
 
-  let selectedTier = options.tier || "pro";
+  let selectedTier = options.tier || existingConfig.tier || "pro";
   let testCmd = options.testCmd;
   let buildCmd = options.buildCmd;
   let selectedPresets = options.presets;
@@ -235,28 +244,35 @@ export async function runInitWizard(root = process.cwd(), options = {}) {
     await new Promise((resolve) => setTimeout(resolve, 300));
     sp.stop(`Detected Stack: ${oracle.stack}`);
 
+    const optionsList = tierOptions();
+    const defaultTierIdx = Math.max(0, optionsList.findIndex((t) => t.value === selectedTier));
+
     selectedTier = await select(
-      tierOptions(),
+      optionsList,
       "Which plan does your Jules account use? (limits are adjustable later)",
-      options
+      { ...options, defaultIdx: defaultTierIdx }
     );
 
     testCmd = await input("Verification Test Command", {
-      defaultValue: oracle.candidates.testCmd || "npm test",
+      defaultValue: testCmd || existingConfig.verify?.test || oracle.candidates.testCmd || "npm test",
       stdin: options.stdin,
       stdout: options.stdout,
     });
 
     buildCmd = await input("Verification Build Command", {
-      defaultValue: oracle.candidates.buildCmd || "",
+      defaultValue: buildCmd || existingConfig.verify?.build || oracle.candidates.buildCmd || "",
       stdin: options.stdin,
       stdout: options.stdout,
     });
 
+    const defaultPresetSet = new Set(
+      selectedPresets || existingConfig.presets || BUILTIN_PRESETS.map((p) => p.id)
+    );
+
     const presetOptions = BUILTIN_PRESETS.map((p) => ({
       label: p.title,
       value: p.id,
-      checked: true,
+      checked: defaultPresetSet.has(p.id),
       description: p.description,
     }));