sharkbait 1.0.12 → 1.0.14

package/dist/cli.js

@@ -746,14 +746,6 @@ var REQUIRES_CONFIRMATION = [
     undoHint: "chown with original owner"
   }
 ];
-var SHELL_CHAINING_PATTERNS = [
-  /;/,
-  /\|\|/,
-  /&&/,
-  /\$\(/,
-  /`[^`]+`/,
-  /\|\s*\w/
-];
 var DANGEROUS_PREFIXES = ["sudo", "doas", "pkexec", "exec"];
 function classifyCommand(command) {
   const trimmed = command.trim();
@@ -780,17 +772,30 @@ function classifyCommand(command) {
       reversibility: "effort"
     };
   }
-  for (const pattern of SHELL_CHAINING_PATTERNS) {
-    if (pattern.test(trimmed)) {
-      return {
-        status: "requires_confirmation",
-        reason: "Command contains shell operators (;, &&, ||, |, $()) — each sub-command cannot be independently verified",
-        reversibility: "effort"
-      };
+  if (/\$\(/.test(trimmed) || /`[^`]+`/.test(trimmed)) {
+    return {
+      status: "requires_confirmation",
+      reason: "Command contains command substitution ($() or backticks)",
+      reversibility: "effort"
+    };
+  }
+  const hasChaining = /;|&&|\|\||(\|\s*\w)/.test(trimmed);
+  if (hasChaining) {
+    const subCommands = trimmed.split(/\s*(?:;|&&|\|\|)\s*/).flatMap((segment) => segment.split(/\s*\|\s*/)).map((s) => s.trim()).filter(Boolean);
+    for (const sub of subCommands) {
+      const subSafety = classifySimpleCommand(sub);
+      if (subSafety.status === "blocked")
+        return subSafety;
+      if (subSafety.status === "requires_confirmation")
+        return subSafety;
     }
+    return { status: "allowed" };
   }
-  const baseCommand = extractBaseCommand(trimmed);
-  if (isOnAllowlist(baseCommand, trimmed)) {
+  return classifySimpleCommand(trimmed);
+}
+function classifySimpleCommand(command) {
+  const baseCommand = extractBaseCommand(command);
+  if (isOnAllowlist(baseCommand, command)) {
     return { status: "allowed" };
   }
   return {
@@ -6423,7 +6428,7 @@ import React3 from "react";
 import { render } from "ink";
 
 // src/ui/app.tsx
-import React2, { useState as useState2, useRef } from "react";
+import React2, { useState as useState2, useEffect as useEffect2, useCallback, useRef } from "react";
 import { Box as Box12, Text as Text12, useInput, useApp } from "ink";
 
 // src/ui/message.tsx
@@ -6700,7 +6705,7 @@ function Spinner({
   useEffect(() => {
     const timer = setInterval(() => {
       setFrameIndex((prev) => (prev + 1) % frames.length);
-    }, 100);
+    }, 200);
     return () => clearInterval(timer);
   }, [frames.length]);
   return /* @__PURE__ */ jsxDEV3(Box3, {
@@ -8262,8 +8267,28 @@ function App({ contextFiles: initialContextFiles, enableBeads: initialBeadsEnabl
     return config.azure.deployment;
   });
   const abortControllerRef = useRef(null);
+  const pendingOutputRef = useRef("");
+  const outputTimerRef = useRef(null);
   const { exit } = useApp();
   const workingDir = currentDir;
+  const flushOutput = useCallback(() => {
+    if (pendingOutputRef.current) {
+      setCurrentOutput(pendingOutputRef.current);
+    }
+    outputTimerRef.current = null;
+  }, []);
+  const throttledSetOutput = useCallback((content) => {
+    pendingOutputRef.current = content;
+    if (!outputTimerRef.current) {
+      outputTimerRef.current = setTimeout(flushOutput, 50);
+    }
+  }, [flushOutput]);
+  useEffect2(() => {
+    return () => {
+      if (outputTimerRef.current)
+        clearTimeout(outputTimerRef.current);
+    };
+  }, []);
   const agent = React2.useMemo(() => new Agent({
     contextFiles,
     enableBeads: beadsEnabled
@@ -8427,7 +8452,7 @@ function App({ contextFiles: initialContextFiles, enableBeads: initialBeadsEnabl
         switch (event.type) {
           case "text":
             assistantContent += event.content;
-            setCurrentOutput(assistantContent);
+            throttledSetOutput(assistantContent);
             const chunkTokens = estimateTokens(event.content);
             setTokenCount((prev) => prev + chunkTokens);
             setSessionCost((prev) => prev + chunkTokens * 0.00003);
@@ -8497,18 +8522,18 @@ ${event.consolidated}`,
           }
           case "tool_result": {
             const duration = event.duration;
-            setActiveToolCalls((prev) => prev.map((tc) => tc.name === event.name && tc.status === "running" ? {
-              ...tc,
-              status: "success",
-              duration,
-              result: typeof event.result === "string" ? event.result.slice(0, 100) : JSON.stringify(event.result).slice(0, 100)
-            } : tc));
             setActiveToolCalls((prev) => {
-              const completed = prev.find((tc) => tc.name === event.name && tc.status === "success");
+              const updated = prev.map((tc) => tc.name === event.name && tc.status === "running" ? {
+                ...tc,
+                status: "success",
+                duration,
+                result: typeof event.result === "string" ? event.result.slice(0, 100) : JSON.stringify(event.result).slice(0, 100)
+              } : tc);
+              const completed = updated.find((tc) => tc.name === event.name && tc.status === "success");
               if (completed) {
                 completedToolCalls.push(completed);
               }
-              return prev.filter((tc) => !(tc.name === event.name && tc.status === "success"));
+              return updated.filter((tc) => !(tc.name === event.name && tc.status === "success"));
             });
             break;
           }
@@ -8526,6 +8551,11 @@ ${event.consolidated}`,
             setTokenCount(event.totalTokens);
             break;
           case "done":
+            if (outputTimerRef.current) {
+              clearTimeout(outputTimerRef.current);
+              outputTimerRef.current = null;
+            }
+            pendingOutputRef.current = "";
             if (assistantContent.trim()) {
               setMessages((prev) => [...prev, {
                 role: "assistant",
@@ -8636,7 +8666,8 @@ ${event.consolidated}`,
           }, undefined, false, undefined, this),
           currentOutput && /* @__PURE__ */ jsxDEV12(MessageView, {
             role: "assistant",
-            content: currentOutput
+            content: currentOutput,
+            enableHighlighting: false
           }, undefined, false, undefined, this)
         ]
       }, undefined, true, undefined, this),
@@ -8699,7 +8730,7 @@ ${event.consolidated}`,
 }
 
 // src/version.ts
-var VERSION = "1.0.12";
+var VERSION = "1.0.14";
 
 // src/agent/start-chat.ts
 async function startChat(options = {}) {
@@ -8714,7 +8745,7 @@ async function startChat(options = {}) {
     enableBeads: options.beads ?? true,
     version: VERSION,
     workingDir
-  }));
+  }), { patchConsole: true });
   await waitUntilExit();
 }
 // src/commands/init.ts
@@ -9876,7 +9907,7 @@ ${"━".repeat(60)}`);
 }
 
 // src/version.ts
-var VERSION2 = "1.0.12";
+var VERSION2 = "1.0.14";
 
 // src/ui/logo.tsx
 import { Box as Box14, Text as Text14 } from "ink";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sharkbait",
-  "version": "1.0.12",
+  "version": "1.0.14",
   "description": "AI-powered coding assistant for the command line. Uses OpenAI Responses API (not Chat). Autonomous agents, parallel code reviews, 36 tools.",
   "type": "module",
   "main": "./dist/cli.js",