@kimbho/kimbho-cli 0.1.23 → 0.1.26

package/dist/index.cjs

@@ -12718,7 +12718,7 @@ function createCompletionRuntimeCommand(program2) {
 // package.json
 var package_default = {
   name: "@kimbho/kimbho-cli",
-  version: "0.1.23",
+  version: "0.1.26",
   description: "Kimbho CLI is a terminal-native coding agent for planning, execution, and verification.",
   type: "module",
   engines: {
@@ -24540,7 +24540,8 @@ async function runShellCommand(toolId, command, cwd, timeoutMs, signal) {
     command
   ], cwd, timeoutMs, signal);
   const success2 = !result.timedOut && result.code === 0;
-  const summary = result.timedOut ? `Command timed out after ${timeoutMs}ms.` : success2 ? `Command completed successfully.` : `Command exited with code ${result.code ?? "unknown"}.`;
+  const interactiveSetupSummary = !success2 ? detectInteractiveShellPrompt(command, result.stdout, result.stderr) : null;
+  const summary = result.timedOut ? `Command timed out after ${timeoutMs}ms.` : interactiveSetupSummary ? interactiveSetupSummary : success2 ? `Command completed successfully.` : `Command exited with code ${result.code ?? "unknown"}.`;
   return ToolResultSchema.parse({
     toolId,
     success: success2,
@@ -24549,6 +24550,18 @@ async function runShellCommand(toolId, command, cwd, timeoutMs, signal) {
     stderr: truncateOutput(result.stderr)
   });
 }
+function detectInteractiveShellPrompt(command, stdout, stderr) {
+  const combined = `${stdout}
+${stderr}`;
+  const normalizedCommand = command.trim().toLowerCase();
+  if (/how would you like to configure eslint\?/i.test(combined) || normalizedCommand.includes("next lint") && /eslint/i.test(combined) && /(strict \(recommended\)|base|cancel)/i.test(combined)) {
+    return "Interactive ESLint setup required before next lint can run.";
+  }
+  if (/\?\s+[^\n]+\n(?:\s*❯|\s*>|\s*\*)/i.test(combined) || /(would you like to|choose one of the following|select an option|enter a value)/i.test(combined)) {
+    return "Command requires interactive input before it can continue.";
+  }
+  return null;
+}
 async function executeFileRead(input, context) {
   if (context.signal?.aborted) {
     const error2 = new Error("Tool execution aborted.");
@@ -33108,6 +33121,24 @@ function isVerificationAction(action) {
   const command = typeof action.input.command === "string" ? action.input.command : "";
   return command.length > 0 && isVerificationCommand(command);
 }
+function extractShellCommand(action) {
+  if (action.type !== "tool" || action.tool !== "shell.exec") {
+    return "";
+  }
+  return typeof action.input.command === "string" ? action.input.command.trim() : "";
+}
+function isInteractiveVerificationSetupFailure(action, result) {
+  if (!isVerificationAction(action) || result.success) {
+    return false;
+  }
+  const command = extractShellCommand(action).toLowerCase();
+  const combined = [
+    result.summary,
+    result.stdout ?? "",
+    result.stderr ?? ""
+  ].join("\n").toLowerCase();
+  return combined.includes("interactive eslint setup required") || combined.includes("command requires interactive input before it can continue") || command.includes("lint") && combined.includes("how would you like to configure eslint");
+}
 function buildSystemPrompt(agent, task, request, allowedTools, plan, extraInstructions) {
   const toolShape = allowedTools.join("|");
   const dependencyTasks = plan ? flattenPlanTasks(plan).filter((candidate) => task.dependsOn.includes(candidate.id)) : [];
@@ -33150,6 +33181,7 @@ function buildSystemPrompt(agent, task, request, allowedTools, plan, extraInstru
     `- Keep paths relative to the workspace.`,
     `- In an existing repo, inspect and change the likely source files before running build or test verification. Do not front-load verification unless you are confirming an already-completed change or diagnosing a failure.`,
     `- After changing code, run verification with tests.run or shell.exec when appropriate.`,
+    `- If a verification command asks for interactive setup or operator input, do not rerun it unchanged. Choose a different non-interactive verifier, or configure that verifier only if the task explicitly requires it.`,
     `- Do not claim success unless the task acceptance criteria are satisfied.`,
     `- If the task is underspecified, make a pragmatic implementation choice and continue.`,
     ...task.subagentInstructions ? [
@@ -33820,6 +33852,7 @@ ${truncateForModel(customAgentMemory)}`);
     const applyToolResult = async (parsedAction, result, step, transcriptEntry) => {
       const mutatingAction = isMutatingAction(parsedAction);
       const verificationAction = isVerificationAction(parsedAction);
+      const interactiveVerificationSetupFailure = verificationAction && !result.success && isInteractiveVerificationSetupFailure(parsedAction, result);
       if (mutatingAction && result.success) {
         changedWorkspace = true;
         verifiedAfterLatestChange = false;
@@ -33834,6 +33867,10 @@ ${truncateForModel(customAgentMemory)}`);
           repairRequiredBeforeVerification = false;
           repairAppliedSinceFailure = false;
           lastVerificationFailure = null;
+        } else if (interactiveVerificationSetupFailure) {
+          repairRequiredBeforeVerification = false;
+          repairAppliedSinceFailure = false;
+          lastVerificationFailure = result;
         } else {
           verificationFailures += 1;
           repairRequiredBeforeVerification = true;
@@ -33853,9 +33890,25 @@ ${truncateForModel(customAgentMemory)}`);
       if (mutatingAction && result.success) {
         followUp.push("Code or workspace state changed. Inspect the diff if needed and run verification before finishing.");
       }
-      if (verificationAction && !result.success) {
+      if (verificationAction && interactiveVerificationSetupFailure) {
+        const failureSummary = `${result.toolId}: ${result.summary}`;
+        transcriptEntry.runtimeNote = [
+          `Verification could not run non-interactively. Latest failure: ${failureSummary}`,
+          "Executor should choose a different non-interactive verifier or configure the verifier explicitly."
+        ].join(" ");
+        await emitProgress({
+          type: "task-note",
+          sessionId,
+          taskId: task.id,
+          agentRole: task.agentRole,
+          step,
+          message: `Verification needs setup via ${failureSummary}. Choosing a different non-interactive verifier or configuration path.`
+        });
+        followUp.push(`The attempted verification was not a code failure. It requires interactive setup or input (${failureSummary}). Do not rerun the same verifier unchanged. Choose a different non-interactive verification command, or configure the verifier explicitly if the task requires it.`);
+      } else if (verificationAction && !result.success) {
+        const failureSummary = `${result.toolId}: ${result.summary}`;
         transcriptEntry.runtimeNote = [
-          `Verification failed (${verificationFailures}/${maxRepairAttempts} repair attempts used).`,
+          `Verification failed (${verificationFailures}/${maxRepairAttempts} repair attempts used). Latest failure: ${failureSummary}`,
           "Executor requires a repair action before the next verification run."
         ].join(" ");
         await emitProgress({
@@ -33864,9 +33917,9 @@ ${truncateForModel(customAgentMemory)}`);
           taskId: task.id,
           agentRole: task.agentRole,
           step,
-          message: `Verification failed. Repair attempts used: ${verificationFailures}/${maxRepairAttempts}.`
+          message: `Verification failed via ${failureSummary}. Repair attempts used: ${verificationFailures}/${maxRepairAttempts}.`
         });
-        followUp.push(`Verification failed. Inspect the failure output, repair the issue, and run verification again before finishing. Repair attempts used: ${verificationFailures}/${maxRepairAttempts}.`);
+        followUp.push(`Verification failed via ${failureSummary}. Inspect the failure output, repair the issue, and run verification again before finishing. Repair attempts used: ${verificationFailures}/${maxRepairAttempts}.`);
         if (verificationFailures >= maxRepairAttempts) {
           transcriptEntry.runtimeNote = [
             transcriptEntry.runtimeNote,
@@ -43930,6 +43983,14 @@ var SPINNER_FRAMES = [
   "|",
   "/"
 ];
+var ELLIPSIS_FRAMES = [
+  ".  ",
+  ".. ",
+  "...",
+  " ..",
+  "  .",
+  "   "
+];
 var IDLE_STATUS_LABELS = [
   "thinking",
   "musing",
@@ -44023,6 +44084,11 @@ function shouldAutoStartApprovedRuns(runtime) {
   const approvalMode = resolveRuntimeOverrideApprovalMode(runtime);
   return approvalMode === "auto" || approvalMode === "dontAsk" || approvalMode === "acceptEdits" || approvalMode === "bypassPermissions";
 }
+function shouldAutoStartProposal(runtime, proposal) {
+  return Boolean(
+    proposal && shouldAutoStartApprovedRuns(runtime) && !planRequiresOperatorReview(proposal.plan)
+  );
+}
 function resolveShellMaxAutoTasks(runtime) {
   return shouldAutoStartApprovedRuns(runtime) ? UNBOUNDED_MAX_AUTO_TASKS : DEFAULT_MAX_AUTO_TASKS;
 }
@@ -44170,7 +44236,7 @@ function createExecutionTelemetry() {
 }
 var ShellActivityIndicator = class {
   interval = null;
-  frameIndex = 0;
+  animationTick = 0;
   label;
   activeLine = false;
   constructor(label) {
@@ -44183,14 +44249,14 @@ var ShellActivityIndicator = class {
     this.activeLine = true;
     this.render();
     this.interval = setInterval(() => {
-      this.frameIndex = (this.frameIndex + 1) % SPINNER_FRAMES.length;
+      this.animationTick += 1;
       this.render();
-    }, 120);
+    }, 140);
     this.interval.unref?.();
   }
   update(label) {
     if (label !== this.label) {
-      this.frameIndex = 0;
+      this.animationTick = 0;
     }
     this.label = label;
     if (this.interval) {
@@ -44221,9 +44287,10 @@ var ShellActivityIndicator = class {
     if (!import_node_process26.default.stdout.isTTY) {
       return;
     }
-    const frame = color(AMBER, SPINNER_FRAMES[this.frameIndex]);
-    const status = renderShimmeringLabel(this.label, this.frameIndex);
-    const raw = `${frame} ${status}${color(DIM, "...")}`;
+    const frame = color(AMBER, SPINNER_FRAMES[this.animationTick % SPINNER_FRAMES.length]);
+    const status = renderShimmeringLabel(this.label, this.animationTick);
+    const dots = color(DIM, ELLIPSIS_FRAMES[this.animationTick % ELLIPSIS_FRAMES.length]);
+    const raw = `${frame} ${status}${dots}`;
     this.clear();
     (0, import_node_readline2.cursorTo)(import_node_process26.default.stdout, 0);
     import_node_process26.default.stdout.write(raw);
@@ -44532,6 +44599,27 @@ function hashString(value) {
 function pickStatusLabel(message) {
   const lower = message.toLowerCase();
   const choose = (labels) => labels[hashString(message) % labels.length];
+  if (lower.includes("verification needs setup") || lower.includes("different non-interactive verifier")) {
+    return choose([
+      "adapting",
+      "rerouting",
+      "recovering"
+    ]);
+  }
+  if (lower.includes("verification failed") || lower.includes("repair attempts used")) {
+    return choose([
+      "repairing",
+      "debugging",
+      "recovering"
+    ]);
+  }
+  if (lower.includes("repair budget exhausted") || lower.includes("debugger escalation")) {
+    return choose([
+      "escalating",
+      "stopping",
+      "blocking"
+    ]);
+  }
   if (lower.includes("synthesizing") || lower.includes("architecture brief")) {
     return choose([
       "planning",
@@ -44581,6 +44669,13 @@ function pickStatusLabel(message) {
       "untangling"
     ]);
   }
+  if (lower.includes("verify") || lower.includes("verification")) {
+    return choose([
+      "verifying",
+      "checking",
+      "confirming"
+    ]);
+  }
   return choose([
     "thinking",
     "musing",
@@ -44591,6 +44686,21 @@ function pickStatusLabel(message) {
 function pickIdleStatusLabel(seed) {
   return IDLE_STATUS_LABELS[hashString(seed) % IDLE_STATUS_LABELS.length];
 }
+function statusLabelForToolStart(event) {
+  if (event.toolId === "shell.exec" || event.toolId === "tests.run") {
+    const command = typeof event.input?.command === "string" ? event.input.command.trim().toLowerCase() : "";
+    if (command.includes("build") || command.includes("lint") || command.includes("test")) {
+      return "verifying";
+    }
+    if (command.includes("dev") || command.includes("start")) {
+      return "serving";
+    }
+    if (command.length > 0) {
+      return pickStatusLabel(command);
+    }
+  }
+  return pickStatusLabel(event.reason ?? event.toolId);
+}
 function statusLabelForEvent(event) {
   switch (event.type) {
     case "task-note":
@@ -44598,7 +44708,7 @@ function statusLabelForEvent(event) {
     case "task-started":
       return pickStatusLabel(event.task.title);
     case "tool-started":
-      return pickStatusLabel(event.reason ?? event.toolId);
+      return statusLabelForToolStart(event);
     case "model-usage":
       return "thinking";
     case "approval-requested":
@@ -44708,14 +44818,67 @@ function renderShellPlanSummary(plan) {
   }
   return lines;
 }
+function inferRecommendedPlanAnswer(question, plan) {
+  const lower = question.toLowerCase();
+  const goal = plan.goal.toLowerCase();
+  if (lower.includes("brand voice") || lower.includes("visual direction")) {
+    if (goal.includes("blog") || goal.includes("post")) {
+      return "Use a premium editorial direction: clean typography, strong whitespace, restrained motion, and a polished reading-first layout.";
+    }
+    return "Use a restrained premium product direction: crisp typography, strong whitespace, subtle motion, and a clear call to action.";
+  }
+  if (lower.includes("which sections")) {
+    if (goal.includes("blog") || goal.includes("post")) {
+      return "Start with navigation, hero, featured posts, topic/category grid, newsletter CTA, and a compact footer.";
+    }
+    return "Start with navigation, hero, key benefits, proof/examples, primary CTA, and footer.";
+  }
+  if (lower.includes("primary call to action")) {
+    if (goal.includes("blog") || goal.includes("post")) {
+      return "Drive readers to featured posts first, with newsletter signup as the secondary CTA.";
+    }
+    return "Drive the main product action with one clear primary CTA and keep secondary actions minimal.";
+  }
+  if (lower.includes("which stack") || lower.includes("deployment target")) {
+    return "Stay with the current repo stack and existing deployment shape unless the user asks for a migration.";
+  }
+  if (lower.includes("authentication model")) {
+    return "Assume no authentication in the first version unless the request explicitly needs gated user flows.";
+  }
+  if (lower.includes("external services") || lower.includes("apis")) {
+    return "Assume no new external integrations beyond what already exists in the repository.";
+  }
+  return "Proceed with the most pragmatic default that preserves the current repo and keeps the first version focused.";
+}
+function extractPlanReviewPrompts(plan) {
+  return plan.openQuestions.slice(0, 3).map((question) => ({
+    question,
+    recommendation: inferRecommendedPlanAnswer(question, plan)
+  }));
+}
+function planRequiresOperatorReview(plan) {
+  return extractPlanReviewPrompts(plan).length > 0;
+}
 function renderPendingRunProposal(proposal, options = {}) {
+  const reviewPrompts = extractPlanReviewPrompts(proposal.plan);
+  const requiresReview = reviewPrompts.length > 0;
   const lines = [
     `${color(BOLD, proposal.source === "queued" ? "Queued Plan Ready" : "Ready To Run")}`,
-    ...renderShellPlanSummary(proposal.plan),
-    "",
-    color(DIM, `saved plan: ${proposal.planPath}`),
-    options.autoStart ? color(DIM, "next: auto-starting in the current permission mode") : "next: /approve to start, /run revise <feedback> to adjust, /run cancel to drop it"
+    ...renderShellPlanSummary(proposal.plan)
   ];
+  if (requiresReview) {
+    lines.push("");
+    lines.push(color(BOLD, "Recommended Defaults"));
+    for (const prompt of reviewPrompts) {
+      lines.push(`\u2022 ${prompt.question}`);
+      lines.push(color(DIM, `  recommended: ${prompt.recommendation}`));
+    }
+  }
+  lines.push("");
+  lines.push(color(DIM, `saved plan: ${proposal.planPath}`));
+  lines.push(
+    requiresReview ? "next: /run approve to accept the recommended defaults, /run revise <feedback> to adjust, /run cancel to drop it" : options.autoStart ? color(DIM, "next: auto-starting in the current permission mode") : "next: /approve to start, /run revise <feedback> to adjust, /run cancel to drop it"
+  );
   if (proposal.source === "queued") {
     lines.splice(1, 0, color(DIM, "The next queued request has been planned and is waiting for your approval."));
   }
@@ -44813,6 +44976,24 @@ function renderConciseProgress(board, startedAt) {
     ] : []
   ];
 }
+function summarizeToolStart(event) {
+  if (event.toolId === "shell.exec" || event.toolId === "tests.run") {
+    const command = typeof event.input?.command === "string" ? event.input.command.trim() : "";
+    if (command) {
+      return `Running ${shortenMiddle(command, 90)}`;
+    }
+  }
+  if (event.toolId === "http.fetch" && typeof event.input?.url === "string") {
+    return `Checking ${shortenMiddle(event.input.url, 90)}`;
+  }
+  if (event.toolId === "process.start" && typeof event.input?.command === "string") {
+    return `Starting ${shortenMiddle(event.input.command, 90)}`;
+  }
+  if (event.toolId === "browser.open" && typeof event.input?.url === "string") {
+    return `Opening ${shortenMiddle(event.input.url, 90)}`;
+  }
+  return null;
+}
 function renderLiveExecutionEvent(event, board, startedAt) {
   switch (event.type) {
     case "task-started":
@@ -44828,6 +45009,13 @@ function renderLiveExecutionEvent(event, board, startedAt) {
         )} ${simplifyExecutionSummary(event.summary) || event.taskId}`,
         ...renderConciseProgress(board, startedAt)
       ];
+    case "tool-started": {
+      const summary = summarizeToolStart(event);
+      return summary ? [
+        `${color(DIM, "working")} ${summary}`,
+        ...renderConciseProgress(board, startedAt)
+      ] : [];
+    }
     case "task-note":
       return isUserVisibleTaskNote(event.message) ? [
         `${color(DIM, "note")} ${simplifyTaskNote(event.message)}`
@@ -45006,7 +45194,7 @@ async function drainQueuedWork(runtime) {
             source: "queued"
           });
           runtime.currentCwd = nextCwd;
-          if (shouldAutoStartApprovedRuns(runtime)) {
+          if (shouldAutoStartProposal(runtime, runtime.pendingRunProposal)) {
             console.log(color(DIM, "Auto-starting queued request in the current permission mode."));
             startPendingRunExecution(runtime);
           }
@@ -45179,7 +45367,7 @@ async function prepareRunProposal(cwd, goal, runtime, options = {}) {
   console.log(renderPendingRunProposal(
     runtime.pendingRunProposal,
     {
-      autoStart: shouldAutoStartApprovedRuns(runtime)
+      autoStart: shouldAutoStartProposal(runtime, runtime.pendingRunProposal)
     }
   ).join("\n"));
   return request.cwd;
@@ -46589,7 +46777,7 @@ async function handleShellCommand(cwd, input, state, runtime, execute) {
         const nextCwd = await prepareRunProposal(cwd, trimmed, runtime, {
           source: "direct"
         });
-        if (shouldAutoStartApprovedRuns(runtime)) {
+        if (shouldAutoStartProposal(runtime, runtime.pendingRunProposal)) {
           console.log(color(DIM, "Auto-starting in the current permission mode."));
           startPendingRunExecution(runtime);
         }
@@ -46719,7 +46907,7 @@ async function handleShellCommand(cwd, input, state, runtime, execute) {
       console.log(renderPendingRunProposal(
         runtime.pendingRunProposal,
         {
-          autoStart: shouldAutoStartApprovedRuns(runtime)
+          autoStart: shouldAutoStartProposal(runtime, runtime.pendingRunProposal)
         }
       ).join("\n"));
       return cwd;
@@ -46756,7 +46944,7 @@ async function handleShellCommand(cwd, input, state, runtime, execute) {
     const nextCwd = await prepareRunProposal(cwd, goal, runtime, {
       source: "direct"
     });
-    if (shouldAutoStartApprovedRuns(runtime)) {
+    if (shouldAutoStartProposal(runtime, runtime.pendingRunProposal)) {
       console.log(color(DIM, "Auto-starting in the current permission mode."));
       startPendingRunExecution(runtime);
     }