@kortix/ui 0.14.4 → 0.14.6

package/dist/session/tool/tools/bash-tool.js

@@ -2,7 +2,7 @@
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { HighlightedCode } from '../../../markdown/code/index.js';
 import { TextShimmer } from '../../../primitives/text-shimmer.js';
-import { BasicTool, partInput, partMetadata, partOutput, partStatus, partStreamingInput, StructuredOutput, ToolRunningContext, useToolCardFrame, useToolCardPad, useToolIndent, useToolOpen, partRawOutput, } from '../shared/infrastructure.js';
+import { BasicTool, partInput, partMetadata, partOutput, partStatus, partStreamingInput, StructuredOutput, RecoveredCallsContext, ToolRunningContext, useToolCardFrame, useToolCardPad, useToolIndent, useToolOpen, partRawOutput, } from '../shared/infrastructure.js';
 import { GenericTool } from '../generic-tool.js';
 import { ToolRegistry } from '../shared/registry.js';
 import { cn } from '../../../lib/utils.js';
@@ -140,7 +140,7 @@ export function dedentCommand(raw) {
  * The exit-code strip keeps its shorter 8px vertical inset: it is metadata
  * about the card, not a third pane of content.
  */
-function CommandBlock({ command, output, richOutput, exitCode, settled, }) {
+function CommandBlock({ command, output, richOutput, exitCode, recovered = false, settled, }) {
     const tI18nComplete = useTranslations('hardcodedUi.i18nComplete');
     const hasOutput = Boolean(richOutput || output);
     const failed = typeof exitCode === 'number' && exitCode !== 0;
@@ -161,7 +161,7 @@ function CommandBlock({ command, output, richOutput, exitCode, settled, }) {
     _jsxs("div", { className: cn('relative overflow-hidden', frame), children: [_jsx("div", { "data-scrollable": true, className: cn('max-h-64 overflow-auto', frame && 'bg-muted/40'), children: _jsx("pre", { className: cn('text-foreground font-mono text-xs leading-relaxed wrap-break-word whitespace-pre-wrap [&_code]:border-none [&_code]:bg-transparent [&_code]:p-0 [&_code]:text-xs [&_code]:leading-relaxed [&_code]:whitespace-pre-wrap [&_pre]:whitespace-pre-wrap [&_span]:border-none [&_span]:outline-none', paneInset, 'pr-11'), children: _jsx(HighlightedCode, { code: command, language: "bash", children: command }) }) }), _jsx("div", { className: "absolute top-2 right-2", children: _jsx(CopyButton, { code: command, className: "text-muted-foreground hover:text-foreground" }) }), (hasOutput || settled) && (_jsx("div", { className: "border-border/60 border-t", children: richOutput ? (richOutput) : output ? (_jsxs("div", { className: "relative", children: [_jsx("div", { "data-scrollable": true, className: "max-h-80 overflow-auto", children: _jsx("div", { className: cn('text-muted-foreground font-mono text-xs leading-relaxed wrap-break-word whitespace-pre-wrap', paneInset, 'pr-11'), children: output }) }), _jsx("div", { className: "absolute top-2 right-2", children: _jsx(CopyButton, { code: output, className: "text-muted-foreground hover:text-foreground" }) })] })) : (_jsx("p", { className: cn('text-muted-foreground', paneInset, 'text-xs leading-relaxed'), children: tI18nComplete.raw('textf7e31759b202') })) })), failed && (_jsxs("div", { className: cn(
                 // `kortix-red`, matching the trigger's "Command failed": the strip
                 // is the card's own copy of the verdict, so the two agree in tone.
-                'border-border/60 text-kortix-red border-t py-2 font-mono text-xs tabular-nums', 
+                'border-border/60 border-t py-2 font-mono text-xs tabular-nums', recovered ? 'text-muted-foreground' : 'text-kortix-red', 
                 // Metadata about the card, not a third pane: it keeps the shorter
                 // 8px vertical inset, and only the horizontal one is the surface's
                 // to supply.
@@ -244,9 +244,13 @@ export function BashTool({ call, defaultOpen, forceOpen, locked }) {
     const exitCode = useMemo(() => {
         if (call.status !== 'complete')
             return undefined;
-        return shellExitCode(partRawOutput(call));
+        // OpenCode tags the code in the output; pi records it beside the output (KRTX-3021).
+        const recorded = partMetadata(call).exit;
+        return shellExitCode(partRawOutput(call)) ?? (typeof recorded === 'number' ? recorded : undefined);
     }, [call]);
     const failed = typeof exitCode === 'number' && exitCode !== 0;
+    // A failure the agent moved past keeps its words, not the red (KRTX-3021).
+    const recovered = useContext(RecoveredCallsContext).has(call.id);
     // `teams send "…"` / `slack send …` / `telegram send …` IS the reply the
     // person in the channel received. It renders as that message — badge, text,
     // attachment — with the shell call kept underneath for the record. A failed
@@ -268,12 +272,12 @@ export function BashTool({ call, defaultOpen, forceOpen, locked }) {
     if (!command && toolKind(call.name) !== 'bash') {
         return _jsx(GenericTool, { call: call, defaultOpen: defaultOpen, forceOpen: forceOpen, locked: locked });
     }
-    return (_jsxs(BasicTool, { icon: send ? (_jsx(ChannelBrandMark, { platform: send.platform, className: "size-4 shrink-0" })) : (_jsx(TerminalIcon, { className: "size-4 shrink-0" })), trigger: isStalePending ? (_jsx("div", { className: "flex min-w-0 flex-1 items-center gap-1.5", children: _jsx(TextShimmer, { duration: 1, spread: 2, children: tI18nComplete.raw('textb93900bded31') }) })) : commandPreview ? (_jsx(BashTrigger, { title: title, failed: failed, command: command, commandPreview: send ? (sendText ?? send.file ?? '') : commandPreview, extraLines: send ? 0 : extraLines, live: running && status !== 'completed' && status !== 'error' })) : null, 
+    return (_jsxs(BasicTool, { icon: send ? (_jsx(ChannelBrandMark, { platform: send.platform, className: "size-4 shrink-0" })) : (_jsx(TerminalIcon, { className: "size-4 shrink-0" })), trigger: isStalePending ? (_jsx("div", { className: "flex min-w-0 flex-1 items-center gap-1.5", children: _jsx(TextShimmer, { duration: 1, spread: 2, children: tI18nComplete.raw('textb93900bded31') }) })) : commandPreview ? (_jsx(BashTrigger, { title: title, failed: failed && !recovered, command: command, commandPreview: send ? (sendText ?? send.file ?? '') : commandPreview, extraLines: send ? 0 : extraLines, live: running && status !== 'completed' && status !== 'error' })) : null, 
         // The reply is the point of the row, so it opens showing it.
         defaultOpen: defaultOpen || Boolean(send), forceOpen: forceOpen, locked: locked, children: [send && (_jsx("div", { className: cn('mb-1.5', indent && 'mt-1.5', indent), children: _jsx(ChannelSendCard, { send: send }) })), command && (
             // `CommandBlock` is a bordered card like the shared three, so it takes
             // the same gate: the seam belongs to the inline row it hangs under, not
             // to the panel body that already supplies `px-3 py-3`.
-            _jsx("div", { className: cn(indent && 'mt-1.5', indent), children: _jsx(CommandBlock, { command: command, output: plainOutput, richOutput: richOutput, exitCode: exitCode, settled: status === 'completed' || status === 'error' }) }))] }));
+            _jsx("div", { className: cn(indent && 'mt-1.5', indent), children: _jsx(CommandBlock, { command: command, output: plainOutput, richOutput: richOutput, exitCode: exitCode, recovered: recovered, settled: status === 'completed' || status === 'error' }) }))] }));
 }
 ToolRegistry.register('bash', BashTool);

package/dist/session/tool/tools/web-search-tool.js

@@ -1,13 +1,19 @@
 'use client';
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { BasicTool, isErrorOutput, partInput, partOutput, partStatus, ToolOutputFallback, partRawOutput, } from '../shared/infrastructure.js';
+import { JsonFailureOutputCard } from '../shared/infrastructure-output.js';
 import { ToolRegistry } from '../shared/registry.js';
 import { ToolResultCard } from '../shared/result-card.js';
 import { WebSourceRow } from '../shared/web-source-row.js';
 import { MagnifyingGlassIcon as Search } from '@phosphor-icons/react';
 import { useMemo } from 'react';
 import { humanizeSearchQuery } from '@kortix/sdk';
-import { parseWebSearchOutput, } from '@kortix/sdk';
+import { parseJsonFailure, parseWebSearchOutput, } from '@kortix/sdk';
+/** A failed query's error as the shared failure card reads it: a nested JSON
+ *  message (a provider's "over plan quota") becomes its own line. */
+function queryFailure(error) {
+    return parseJsonFailure(JSON.stringify({ success: false, error })) ?? { errorSummary: error };
+}
 /**
  * Every source, flat, inside one bordered card.
  *
@@ -26,8 +32,24 @@ import { parseWebSearchOutput, } from '@kortix/sdk';
  * get a muted one-line query caption between segments — a caption, not a
  * control. No accordions, no "show more" button.
  */
+/** The one error every query of a batch failed with, or null. A provider
+ *  limit fails them all alike, and five copies of it read as five problems. */
+function sharedFailure(queryResults) {
+    const first = queryResults[0]?.error;
+    if (queryResults.length < 2 || !first)
+        return null;
+    return queryResults.every((qr) => qr.error === first && qr.sources.length === 0) ? first : null;
+}
+function QueryCaption({ query }) {
+    return (_jsxs("div", { className: "text-muted-foreground flex items-center gap-2 px-2 pt-2 pb-1 text-xs", children: [_jsx(Search, { className: "size-3 shrink-0" }), _jsx("span", { className: "truncate", children: humanizeSearchQuery(query) })] }));
+}
 function FlatSourceList({ queryResults }) {
-    return (_jsx(ToolResultCard, { children: queryResults.map((qr, qi) => (_jsxs("div", { children: [queryResults.length > 1 && qr.sources.length > 0 && (_jsxs("div", { className: "text-muted-foreground/70 flex items-center gap-2 px-2 pt-2 pb-1 text-xs", children: [_jsx(Search, { className: "size-3 shrink-0" }), _jsx("span", { className: "truncate", children: humanizeSearchQuery(qr.query) })] })), qr.sources.map((src) => (_jsx(WebSourceRow, { url: src.url, title: src.title || src.url }, src.url)))] }, qi))) }));
+    const shared = sharedFailure(queryResults);
+    if (shared) {
+        // The failure once, then the searches it stopped (KRTX-3022).
+        return (_jsxs(ToolResultCard, { children: [_jsx(JsonFailureOutputCard, { failure: queryFailure(shared) }), queryResults.map((qr, qi) => (_jsx(QueryCaption, { query: qr.query }, qi)))] }));
+    }
+    return (_jsx(ToolResultCard, { children: queryResults.map((qr, qi) => (_jsxs("div", { children: [queryResults.length > 1 && (qr.sources.length > 0 || qr.error) && _jsx(QueryCaption, { query: qr.query }), qr.error && _jsx(JsonFailureOutputCard, { failure: queryFailure(qr.error) }), qr.sources.map((src) => (_jsx(WebSourceRow, { url: src.url, title: src.title || src.url }, src.url)))] }, qi))) }));
 }
 export function WebSearchTool({ call, defaultOpen, forceOpen, locked }) {
     const input = partInput(call);
@@ -58,7 +80,7 @@ export function WebSearchTool({ call, defaultOpen, forceOpen, locked }) {
     const triggerBadge = status === 'completed' && !isError && totalSources > 0
         ? `${totalSources} ${totalSources === 1 ? 'result' : 'results'}`
         : undefined;
-    return (_jsx(BasicTool, { icon: _jsx(Search, { className: "size-3.5 shrink-0" }), trigger: _jsxs("div", { className: "flex min-w-0 flex-1 items-center gap-1.5", children: [_jsx("span", { className: "text-foreground min-w-0 truncate text-sm", children: triggerLabel }), triggerBadge && (_jsx("span", { className: "text-muted-foreground/70 ml-auto shrink-0 text-sm whitespace-nowrap", children: triggerBadge }))] }), defaultOpen: defaultOpen, forceOpen: forceOpen, locked: locked, children: _jsx(_Fragment, { children: isError ? (_jsx(ToolOutputFallback, { output: output, toolName: "web_search" })) : queryResults.length > 0 ? (_jsx(FlatSourceList, { queryResults: queryResults })) : output ? (_jsx(ToolOutputFallback, { output: output, isStreaming: status === 'running', toolName: "web_search" })) : null }) }));
+    return (_jsx(BasicTool, { icon: _jsx(Search, { className: "size-3.5 shrink-0" }), trigger: _jsxs("div", { className: "flex min-w-0 flex-1 items-center gap-1.5", children: [_jsx("span", { className: "text-foreground min-w-0 truncate text-sm", children: triggerLabel }), triggerBadge && (_jsx("span", { className: "text-muted-foreground ml-auto shrink-0 text-sm whitespace-nowrap", children: triggerBadge }))] }), defaultOpen: defaultOpen, forceOpen: forceOpen, locked: locked, children: _jsx(_Fragment, { children: isError ? (_jsx(ToolOutputFallback, { output: output, toolName: "web_search" })) : queryResults.length > 0 ? (_jsx(FlatSourceList, { queryResults: queryResults })) : output ? (_jsx(ToolOutputFallback, { output: output, isStreaming: status === 'running', toolName: "web_search" })) : null }) }));
 }
 ToolRegistry.register('websearch', WebSearchTool);
 ToolRegistry.register('web-search', WebSearchTool);

package/dist/session/turn/burst-step-body.js

@@ -5,6 +5,7 @@ import { ActivityGroupStep } from './activity-group-step.js';
 import { ActivityStep, StepLabelRow } from './activity-step.js';
 import { AnsweredQuestionStep, isAnsweredQuestionCall } from './answered-question-step.js';
 import { stepLabel } from '@kortix/sdk';
+import { CompactionMarker } from './compaction-card.js';
 import { ToolRegistry } from '../tool/shared/registry.js';
 /** A host's `ToolRegistry.register` row for this tool wins over the chip and question rows. */
 const hostRow = (call) => ToolRegistry.replaced(call.name, call.kind);
@@ -44,6 +45,9 @@ export function StepBody({ step, bareRow = false, running, sessionId, disableNav
         const calls = step.blocks;
         return isFileChipRun(calls) ? (_jsx(ActivityFileChipStep, { calls: calls, sessionId: sessionId, disableNavigation: disableNavigation })) : (_jsx(ActivityGroupStep, { step: { id: calls[0].call_id, family: step.family, label: step.label, parts: calls, status: step.status, durationMs: step.durationMs }, sessionId: sessionId, disableNavigation: disableNavigation }));
     }
+    if (step.block.type === 'compaction') {
+        return _jsx(CompactionMarker, { running: running && step.block.summary === null, summary: step.block.summary ?? undefined });
+    }
     // Neither tool nor reasoning: the label row only. The turn modules never
     // silently drop a block (`stepLabel` falls back to a generic 'Used' label).
     if (step.block.type !== 'tool_call') {

package/dist/session/turn/recovered-failures.d.ts

@@ -0,0 +1,8 @@
+import { type SessionLogBlock } from '@kortix/sdk';
+/**
+ * The failed commands of a turn that a later step worked past: a call after them
+ * completed and did not fail. A probe such as `ls missing-dir` exits non-zero by
+ * design, and the agent carries on; only a failure the turn ended on is still
+ * the news (KRTX-3021).
+ */
+export declare function recoveredShellFailures(blocks: readonly SessionLogBlock[]): Set<string>;

package/dist/session/turn/recovered-failures.js

@@ -0,0 +1,32 @@
+import { shellExitCode, toolCallMetadata, toolCallRawOutput, } from '@kortix/sdk';
+/** A shell command that ran and exited non-zero, by either harness's record of its code. */
+function failedCommand(block) {
+    if (block.name !== 'bash' || block.status !== 'complete')
+        return false;
+    const recorded = toolCallMetadata(block).exit;
+    const exit = shellExitCode(toolCallRawOutput(block)) ?? (typeof recorded === 'number' ? recorded : undefined);
+    return typeof exit === 'number' && exit !== 0;
+}
+/**
+ * The failed commands of a turn that a later step worked past: a call after them
+ * completed and did not fail. A probe such as `ls missing-dir` exits non-zero by
+ * design, and the agent carries on; only a failure the turn ended on is still
+ * the news (KRTX-3021).
+ */
+export function recoveredShellFailures(blocks) {
+    const recovered = new Set();
+    const pending = [];
+    for (const block of blocks) {
+        if (block.type !== 'tool_call')
+            continue;
+        if (failedCommand(block)) {
+            pending.push(block.id);
+        }
+        else if (block.status === 'complete') {
+            for (const id of pending)
+                recovered.add(id);
+            pending.length = 0;
+        }
+    }
+    return recovered;
+}

package/dist/session/turn/streaming-cadence.js

@@ -1,5 +1,5 @@
 'use client';
-import { useEffect, useRef, useState } from 'react';
+import { useEffect, useState } from 'react';
 /**
  * Streamed text reaches the client in network chunks of any size and at any
  * spacing: one token every 16 ms, or a paragraph after a 300 ms stall.
@@ -287,14 +287,11 @@ export function useStreamingCadence(rawValue, active, revealKey, clock = default
         const resume = kept && clock.now() - kept.at < RESUME_WINDOW_MS && kept.len < value.length;
         return { text: resume ? value.slice(0, kept.len) : value, streaming: active };
     });
-    const pacerRef = useRef(null);
-    if (pacerRef.current === null) {
-        pacerRef.current = createStreamPacer((text, streaming) => setState({ text, streaming }), state.text, clock, state.streaming);
-    }
+    const [pacer] = useState(() => createStreamPacer((text, streaming) => setState({ text, streaming }), state.text, clock, state.streaming));
     useEffect(() => {
-        pacerRef.current?.push(value, active);
-    }, [value, active]);
-    useEffect(() => () => pacerRef.current?.dispose(), []);
+        pacer.push(value, active);
+    }, [pacer, value, active]);
+    useEffect(() => () => pacer.dispose(), [pacer]);
     useEffect(() => {
         if (revealKey && state.streaming)
             rememberReveal(revealKey, state.text.length, clock.now());