@whittlelabs/sifter 0.35.0 → 0.36.0

package/bin.js

@@ -9961,6 +9961,8 @@ var require_progress = __commonJS({
       seq = 0;
       pending = [];
       textBuf = "";
+      /** The last two characters of reasoning emitted since the last tool; null when none. */
+      reasoningTail = null;
       timer = null;
       closed = false;
       flushMs;
@@ -9970,20 +9972,39 @@ var require_progress = __commonJS({
         this.flushMs = options.flushMs ?? 400;
         this.maxChars = options.maxChars ?? 1200;
       }
+      /** A fragment of reasoning, continuing whatever came before it. */
       reasoning(text) {
         if (this.closed || !text)
           return;
         this.textBuf += text;
+        this.reasoningTail = ((this.reasoningTail ?? "") + text).slice(-2);
         if (this.textBuf.length >= this.maxChars) {
           this.flush();
         } else {
           this.arm();
         }
       }
+      /**
+       * A whole block of reasoning (a thinking summary, a message between tool
+       * calls), trimmed and set off from the reasoning before it by one blank
+       * line. A consumer flows consecutive reasoning frames into one passage, so
+       * two blocks reported as fragments run together mid-sentence, and a
+       * block's own trailing blank lines (thinking summaries end in them) would
+       * show as empty lines before the next act.
+       */
+      reasoningBlock(text) {
+        const block = text.trim();
+        if (this.closed || !block)
+          return;
+        const tail = this.reasoningTail;
+        const gap = tail === null || tail.endsWith("\n\n") ? "" : tail.endsWith("\n") ? "\n" : "\n\n";
+        this.reasoning(gap + block);
+      }
       tool(name, target) {
         if (this.closed || !name)
           return;
         this.drainText();
+        this.reasoningTail = null;
         this.push({ kind: "tool", tool: target ? { name, target } : { name } });
         this.flush();
       }
@@ -10611,6 +10632,87 @@ var require_workspace_telemetry = __commonJS({
   }
 });
 
+// ../../packages/shuttle/dist/executors/cli-version.js
+var require_cli_version = __commonJS({
+  "../../packages/shuttle/dist/executors/cli-version.js"(exports2) {
+    "use strict";
+    Object.defineProperty(exports2, "__esModule", { value: true });
+    exports2.cliVersion = cliVersion;
+    var child_process_1 = require("child_process");
+    var VERSION_TIMEOUT_MS = 5e3;
+    function cliVersion(bin, env) {
+      return new Promise((resolve) => {
+        try {
+          (0, child_process_1.execFile)(bin, ["--version"], { timeout: VERSION_TIMEOUT_MS, ...env ? { env } : {} }, (err, stdout) => {
+            resolve(err ? null : String(stdout).match(/\d+\.\d+\.\d+/)?.[0] ?? null);
+          });
+        } catch {
+          resolve(null);
+        }
+      });
+    }
+  }
+});
+
+// ../../packages/shuttle/dist/version-check.js
+var require_version_check = __commonJS({
+  "../../packages/shuttle/dist/version-check.js"(exports2) {
+    "use strict";
+    Object.defineProperty(exports2, "__esModule", { value: true });
+    exports2.UpgradeRequiredError = exports2.LATEST_VERSION_HEADER = exports2.MIN_VERSION_HEADER = void 0;
+    exports2.compareSemver = compareSemver;
+    exports2.checkResponseMinVersion = checkResponseMinVersion;
+    exports2.readResponseLatestVersion = readResponseLatestVersion;
+    exports2.MIN_VERSION_HEADER = "x-sifter-min-version";
+    exports2.LATEST_VERSION_HEADER = "x-sifter-latest-version";
+    var UpgradeRequiredError2 = class extends Error {
+      currentVersion;
+      minVersion;
+      packageName;
+      constructor(opts) {
+        super(`The backend requires ${opts.packageName} ${opts.minVersion} or newer. You're running ${opts.currentVersion}. Upgrade: npm install -g ${opts.packageName}@latest`);
+        this.name = "UpgradeRequiredError";
+        this.currentVersion = opts.currentVersion;
+        this.minVersion = opts.minVersion;
+        this.packageName = opts.packageName;
+      }
+    };
+    exports2.UpgradeRequiredError = UpgradeRequiredError2;
+    function compareSemver(a, b) {
+      const parse = (v) => {
+        const core = v.replace(/^v/, "").split(/[-+]/, 1)[0];
+        const parts = core.split(".").map((p) => Number.parseInt(p, 10));
+        return [parts[0] ?? 0, parts[1] ?? 0, parts[2] ?? 0];
+      };
+      const [aMaj, aMin, aPat] = parse(a);
+      const [bMaj, bMin, bPat] = parse(b);
+      if (aMaj !== bMaj)
+        return aMaj < bMaj ? -1 : 1;
+      if (aMin !== bMin)
+        return aMin < bMin ? -1 : 1;
+      if (aPat !== bPat)
+        return aPat < bPat ? -1 : 1;
+      return 0;
+    }
+    function checkResponseMinVersion(opts) {
+      const minVersion = opts.response.headers.get(exports2.MIN_VERSION_HEADER);
+      if (!minVersion)
+        return;
+      if (compareSemver(opts.currentVersion, minVersion) >= 0)
+        return;
+      throw new UpgradeRequiredError2({
+        currentVersion: opts.currentVersion,
+        minVersion,
+        packageName: opts.packageName
+      });
+    }
+    function readResponseLatestVersion(response) {
+      const latest = response.headers.get(exports2.LATEST_VERSION_HEADER);
+      return latest && latest.trim().length > 0 ? latest.trim() : null;
+    }
+  }
+});
+
 // ../../packages/shuttle/dist/executors/claude-code.js
 var require_claude_code = __commonJS({
   "../../packages/shuttle/dist/executors/claude-code.js"(exports2) {
@@ -10634,9 +10736,12 @@ var require_claude_code = __commonJS({
     var json_answer_1 = require_json_answer();
     var models_1 = require_models();
     var workspace_telemetry_1 = require_workspace_telemetry();
+    var cli_version_1 = require_cli_version();
     var apply_1 = require_apply();
+    var version_check_1 = require_version_check();
     var DEFAULT_TIMEOUT = 6e5;
     var REVIEW_TOOLS = ["Read", "Grep", "Glob"];
+    var THINKING_DISPLAY_FLOOR = "2.1.94";
     function reviewRunArgs(settingSources) {
       return [
         "--setting-sources",
@@ -10891,13 +10996,17 @@ var require_claude_code = __commonJS({
           args.push("--model", model);
         if (effort)
           args.push("--effort", effort);
+        const version = await (0, cli_version_1.cliVersion)("claude", childEnv);
+        if (version !== null && (0, version_check_1.compareSemver)(version, THINKING_DISPLAY_FLOOR) >= 0) {
+          args.push("--thinking-display", "summarized");
+        }
         if (this.config.maxTurns !== void 0)
           args.push("--max-turns", String(this.config.maxTurns));
         if (allowedTools && allowedTools.length > 0) {
           args.push("--allowedTools", allowedTools.join(","));
         }
         const tap = reporter ? new progress_1.StreamJsonTap({
-          onReasoning: (t) => reporter.reasoning(t),
+          onReasoning: (t) => reporter.reasoningBlock(t),
           onTool: (name, target) => reporter.tool(name, target),
           onResult: () => reporter.flush()
         }) : null;
@@ -11395,6 +11504,11 @@ var require_codex = __commonJS({
         'approval_policy="never"',
         "-c",
         'web_search="disabled"',
+        // Reasoning summaries are the live view's narration. With the user config
+        // ignored, codex 0.162.1 emits no reasoning items at all; `detailed`
+        // brings them back without touching the answer.
+        "-c",
+        'model_reasoning_summary="detailed"',
         ...DISABLED_FEATURES.flatMap((f) => ["--disable", f])
       ];
     }
@@ -11649,12 +11763,17 @@ var require_codex = __commonJS({
         if (event.type === "item.started" && kind === "command_execution" && typeof item?.command === "string") {
           reporter.tool("shell", item.command);
         } else if (event.type === "item.completed" && kind === "reasoning" && typeof item?.text === "string") {
-          reporter.reasoning(item.text);
+          reporter.reasoningBlock(plainHeadings(item.text));
+        } else if (event.type === "item.completed" && kind === "agent_message" && typeof item?.text === "string" && (0, json_answer_1.extractJsonObject)(item.text) === void 0) {
+          reporter.reasoningBlock(item.text);
         } else if (event.type === "turn.completed") {
           reporter.flush();
         }
       });
     }
+    function plainHeadings(text) {
+      return text.replace(/^\*\*([^*\n]+)\*\*[ \t]*$/gm, "$1");
+    }
     function itemKind(item) {
       const kind = typeof item.type === "string" ? item.type : typeof item.item_type === "string" ? item.item_type : null;
       return kind === "assistant_message" ? "agent_message" : kind;
@@ -12182,7 +12301,7 @@ ${JSON.stringify(spec.outputSchema)}
           const target = [args.path, args.pattern].find((v) => typeof v === "string");
           reporter.tool(data.toolName, target);
         } else if (event.type === "assistant.reasoning" && typeof data.content === "string") {
-          reporter.reasoning(data.content);
+          reporter.reasoningBlock(data.content);
         } else if (event.type === "assistant.turn_end") {
           reporter.flush();
         }
@@ -13730,65 +13849,6 @@ var require_custom_script = __commonJS({
   }
 });
 
-// ../../packages/shuttle/dist/version-check.js
-var require_version_check = __commonJS({
-  "../../packages/shuttle/dist/version-check.js"(exports2) {
-    "use strict";
-    Object.defineProperty(exports2, "__esModule", { value: true });
-    exports2.UpgradeRequiredError = exports2.LATEST_VERSION_HEADER = exports2.MIN_VERSION_HEADER = void 0;
-    exports2.compareSemver = compareSemver;
-    exports2.checkResponseMinVersion = checkResponseMinVersion;
-    exports2.readResponseLatestVersion = readResponseLatestVersion;
-    exports2.MIN_VERSION_HEADER = "x-sifter-min-version";
-    exports2.LATEST_VERSION_HEADER = "x-sifter-latest-version";
-    var UpgradeRequiredError2 = class extends Error {
-      currentVersion;
-      minVersion;
-      packageName;
-      constructor(opts) {
-        super(`The backend requires ${opts.packageName} ${opts.minVersion} or newer. You're running ${opts.currentVersion}. Upgrade: npm install -g ${opts.packageName}@latest`);
-        this.name = "UpgradeRequiredError";
-        this.currentVersion = opts.currentVersion;
-        this.minVersion = opts.minVersion;
-        this.packageName = opts.packageName;
-      }
-    };
-    exports2.UpgradeRequiredError = UpgradeRequiredError2;
-    function compareSemver(a, b) {
-      const parse = (v) => {
-        const core = v.replace(/^v/, "").split(/[-+]/, 1)[0];
-        const parts = core.split(".").map((p) => Number.parseInt(p, 10));
-        return [parts[0] ?? 0, parts[1] ?? 0, parts[2] ?? 0];
-      };
-      const [aMaj, aMin, aPat] = parse(a);
-      const [bMaj, bMin, bPat] = parse(b);
-      if (aMaj !== bMaj)
-        return aMaj < bMaj ? -1 : 1;
-      if (aMin !== bMin)
-        return aMin < bMin ? -1 : 1;
-      if (aPat !== bPat)
-        return aPat < bPat ? -1 : 1;
-      return 0;
-    }
-    function checkResponseMinVersion(opts) {
-      const minVersion = opts.response.headers.get(exports2.MIN_VERSION_HEADER);
-      if (!minVersion)
-        return;
-      if (compareSemver(opts.currentVersion, minVersion) >= 0)
-        return;
-      throw new UpgradeRequiredError2({
-        currentVersion: opts.currentVersion,
-        minVersion,
-        packageName: opts.packageName
-      });
-    }
-    function readResponseLatestVersion(response) {
-      const latest = response.headers.get(exports2.LATEST_VERSION_HEADER);
-      return latest && latest.trim().length > 0 ? latest.trim() : null;
-    }
-  }
-});
-
 // ../../packages/shuttle/dist/branding/environment.js
 var require_environment = __commonJS({
   "../../packages/shuttle/dist/branding/environment.js"(exports2) {
@@ -29132,7 +29192,7 @@ var import_path = require("path");
 var import_promises = require("fs/promises");
 var import_yaml = __toESM(require_dist3());
 var import_shuttle = __toESM(require_dist5());
-var buildVersion = true ? "0.35.0" : pkg.version;
+var buildVersion = true ? "0.36.0" : pkg.version;
 var AGENT_EXECUTORS = ["claude-code", "codex", "copilot"];
 var sifterBrand = {
   product: {