xab 8.0.0 → 10.0.0

package/dist/index.js

@@ -448,7 +448,28 @@ async function runStreamedWithProgress(thread, prompt, onProgress, turnOpts) {
           const cmd = item.command ?? "";
           const status = item.status;
           if (status === "in_progress") {
-            onProgress("exec", `$ ${cmd}`);
+            const inner = cmd.replace(/^\/bin\/(?:ba)?sh\s+-\w+\s+['"](.*)['"]$/s, "$1") || cmd;
+            const readMatch = inner.match(/\b(?:cat|head|tail|less|bat|nl)\s+(?:-\w+\s+)*['"]?([^\s'"|\]]+)/);
+            const sedMatch = inner.match(/\bsed\s+-n\s+['"]?\d+.*?['"]?\s+['"]?([^\s'"]+)/);
+            const rgMatch = inner.match(/\brg\s+(?:-[^\s]+\s+)*['"]?(.+?)['"]?\s+(\S+)/);
+            const gitShowMatch = inner.match(/\bgit\s+show\s+(\S+)/);
+            const gitDiffMatch = inner.match(/\bgit\s+diff\b/);
+            const gitLogMatch = inner.match(/\bgit\s+log\b/);
+            if (readMatch) {
+              onProgress("read", readMatch[1]);
+            } else if (sedMatch) {
+              onProgress("read", sedMatch[1]);
+            } else if (rgMatch) {
+              onProgress("grep", `"${rgMatch[1]}" in ${rgMatch[2]}`);
+            } else if (gitShowMatch) {
+              onProgress("read", `git show ${gitShowMatch[1]}`);
+            } else if (gitDiffMatch) {
+              onProgress("exec", `$ ${inner.slice(0, 120)}`);
+            } else if (gitLogMatch) {
+              onProgress("exec", `$ ${inner.slice(0, 120)}`);
+            } else {
+              onProgress("exec", `$ ${inner.slice(0, 120)}`);
+            }
           } else if (status === "completed") {
             const output = item.aggregated_output ?? "";
             if (output) {
@@ -468,17 +489,45 @@ async function runStreamedWithProgress(thread, prompt, onProgress, turnOpts) {
           }
           break;
         }
+        case "mcp_tool_call": {
+          const tool = item.tool ?? "";
+          const args = item.arguments ?? {};
+          const status = item.status;
+          if (status === "in_progress") {
+            if (tool.toLowerCase().includes("read")) {
+              onProgress("read", (args.file_path ?? args.path ?? tool).slice(0, 120));
+            } else if (tool.toLowerCase().includes("grep") || tool.toLowerCase().includes("search")) {
+              onProgress("grep", `"${(args.pattern ?? "").slice(0, 60)}" ${args.path ?? ""}`);
+            } else if (tool.toLowerCase().includes("glob") || tool.toLowerCase().includes("find")) {
+              onProgress("glob", (args.pattern ?? args.path ?? tool).slice(0, 120));
+            } else {
+              onProgress("tool", `${tool} ${JSON.stringify(args).slice(0, 80)}`);
+            }
+          }
+          break;
+        }
         case "reasoning": {
           const text = item.text ?? "";
-          if (text && event.type === "item.completed") {
-            onProgress("think", text.split(`
-`)[0].slice(0, 120));
+          if (text) {
+            const lines = text.split(`
+`).filter(Boolean);
+            for (const line of lines.slice(0, 3)) {
+              onProgress("think", line.slice(0, 150));
+            }
           }
           break;
         }
         case "agent_message": {
+          const text = item.text ?? "";
           if (event.type === "item.completed") {
-            finalResponse = item.text ?? "";
+            finalResponse = text;
+          }
+          if (event.type === "item.updated" && text) {
+            const lastLine = text.split(`
+`).filter(Boolean).pop();
+            if (lastLine && lastLine.length > 10) {
+              onProgress("think", lastLine.slice(0, 150));
+            }
           }
           break;
         }
@@ -774,26 +823,41 @@ async function reviewAppliedDiff(worktreePath, packet, onProgress) {
     normal: "Be thorough but reasonable. Reject clear issues, accept minor style differences.",
     lenient: "Focus on correctness and safety. Accept reasonable adaptations even if imperfect."
   };
-  const prompt = `You are reviewing a curated merge. A commit from "${packet.sourceBranch}" was applied to a branch based on "${packet.targetBranch}".
+  const prompt = `You are reviewing a curated merge commit. Codex adapted a source commit from "${packet.sourceBranch}" and applied it to this worktree (based on "${packet.targetBranch}").
 
-## Codex analysis of the source commit
-Summary: ${packet.analysis.summary}
-Decision: alreadyInTarget=${packet.analysis.alreadyInTarget}
-Strategy: ${packet.analysis.applicationStrategy}
-Affected components: ${packet.analysis.affectedComponents.join(", ")}
+Your job: review the NEW commit Codex created. Verify it is correct, clean, and faithful to the source commit's intent while respecting the target's architecture.
 
-## Source commit
+## Source commit (what was being backmerged)
 Hash: ${packet.commitHash}
 Message: ${packet.commitMessage}
+Branch: ${packet.sourceBranch}
+
+### Original source diff:
+\`\`\`diff
+${packet.sourcePatch.slice(0, 15000)}
+\`\`\`
+
+## Codex's analysis
+Summary: ${packet.analysis.summary}
+Already in target: ${packet.analysis.alreadyInTarget}
+Strategy used: ${packet.analysis.applicationStrategy}
+Components: ${packet.analysis.affectedComponents.join(", ")}
 
-## Applied diff (what was actually committed):
+## Codex's applied commit (what you are reviewing)
+${packet.newCommitHash ? `Commit: ${packet.newCommitHash} (in this worktree)
+Run \`git show ${packet.newCommitHash.slice(0, 8)}\` to inspect it.` : `Run \`git show HEAD\` to inspect it.`}
+
+### Applied diff:
 \`\`\`diff
 ${packet.appliedDiff.slice(0, 30000)}
 \`\`\`
 
-## Diff stat:
+### Diff stat:
 ${packet.appliedDiffStat}
 
+## Key question: does the applied commit correctly adapt the source commit's intent?
+Compare the source diff with the applied diff. The applied commit should capture the same behavior/fix but adapted for the target branch's codebase structure.
+
 ${packet.repoContext ? `## Repository context
 ${packet.repoContext}
 ` : ""}
@@ -904,15 +968,22 @@ Your objections will be sent back to the apply agent for fixing, so be specific
       }
     }
     if (message.type === "result") {
-      if ("result" in message) {
-        resultText = message.result;
+      const msg = message;
+      if (msg.structured_output) {
+        resultText = typeof msg.structured_output === "string" ? msg.structured_output : JSON.stringify(msg.structured_output);
+      } else if (msg.result) {
+        resultText = msg.result;
       }
       break;
     }
   }
   if (!resultText) {
+    if (onProgress)
+      onProgress("review", "WARNING: no structured_output or result in review response");
     return { approved: false, issues: ["Review produced no output"], summary: "Review failed", confidence: "low" };
   }
+  if (onProgress)
+    onProgress("review", `got ${resultText.length} chars of review output`);
   try {
     return JSON.parse(resultText);
   } catch {
@@ -1712,6 +1783,7 @@ async function processOneCommit(o) {
         packet = {
           commitHash: commit.hash,
           commitMessage: commit.message,
+          newCommitHash: validation.newCommitHash ?? undefined,
           sourceBranch: o.sourceRef,
           targetBranch: o.targetRef,
           analysis,
@@ -1975,8 +2047,46 @@ async function runBatch(opts) {
     onProgress(phase, msg) {
       if (jsonl)
         emitJsonl({ event: "progress", phase, msg });
-      const icon = phase === "apply" ? chalk.green("\u25B8") : chalk.blue("\u25B8");
-      log(`  ${ts()} ${icon} ${chalk.dim(msg)}`);
+      const subMatch = msg.match(/^\[(\w+)\]\s*(.*)/);
+      const sub = subMatch ? subMatch[1] : phase;
+      const text = subMatch ? subMatch[2] : msg;
+      let icon;
+      switch (sub) {
+        case "read":
+          icon = chalk.cyan("\uD83D\uDCD6");
+          break;
+        case "grep":
+          icon = chalk.cyan("\uD83D\uDD0D");
+          break;
+        case "glob":
+          icon = chalk.cyan("\uD83D\uDCC2");
+          break;
+        case "exec":
+          icon = chalk.yellow("\u26A1");
+          break;
+        case "file":
+          icon = chalk.green("\u270F\uFE0F");
+          break;
+        case "think":
+          icon = chalk.blue("\uD83D\uDCAD");
+          break;
+        case "tool":
+          icon = chalk.dim("\uD83D\uDD27");
+          break;
+        case "analyze":
+          icon = chalk.blue("\u25C6");
+          break;
+        case "apply":
+          icon = chalk.green("\u25B8");
+          break;
+        case "review":
+          icon = chalk.magenta("\u25CF");
+          break;
+        default:
+          icon = chalk.dim("\xB7");
+          break;
+      }
+      log(`  ${ts()} ${icon} ${chalk.dim(text)}`);
     },
     onLog(msg, color) {
       if (jsonl)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "xab",
-  "version": "8.0.0",
+  "version": "10.0.0",
   "description": "AI-powered curated branch reconciliation engine",
   "type": "module",
   "bin": {