pikiloom 0.4.26 → 0.4.28

package/dist/agent/utils.js

@@ -1,7 +1,3 @@
-/**
- * Pure utility functions shared across the agent layer.
- * No filesystem or session state side effects.
- */
 import fs from 'node:fs';
 import { writeScopedLog } from '../core/logging.js';
 export const Q = (a) => {
@@ -70,7 +66,6 @@ export function normalizeStreamPreviewPlan(value) {
         steps,
     };
 }
-/** Parse a TodoWrite tool input into a StreamPreviewPlan. */
 export function parseTodoWriteAsPlan(input) {
     if (!input || typeof input !== 'object')
         return null;
@@ -95,11 +90,6 @@ export function parseTodoWriteAsPlan(input) {
     return { explanation: null, steps };
 }
 export function normalizeActivityLine(text) { return text.replace(/\s+/g, ' ').trim(); }
-// The activity feed is only ever rendered as a tail — downstream previews trim
-// it to ~900 chars (trimActivityForPreview) and the final reply to ~1600 — yet
-// every tool event rebuilds `s.activity = recentActivity.join('\n')`. A 500-line
-// cap made that rejoin effectively O(n²) over a tool-heavy turn for history no
-// view ever shows; 80 lines comfortably covers the largest consumer.
 export function pushRecentActivity(lines, line, maxLines = 80) {
     const cleaned = normalizeActivityLine(line);
     if (!cleaned)
@@ -118,10 +108,6 @@ export function firstNonEmptyLine(text) {
     }
     return '';
 }
-// MCP tool results carry structured content blocks (e.g. screenshot returns
-// `[{type:'image',...}, {type:'text', text:'Saved as ...'}]`). Coerce that to
-// plain text by keeping only the `type:'text'` blocks; otherwise `String([{…}])`
-// silently becomes the literal "[object Object]" in activity summaries.
 export function coerceToolResultText(value) {
     if (typeof value === 'string')
         return value;
@@ -171,24 +157,6 @@ export function joinErrorMessages(errors) {
         return '';
     return errors.map(error => normalizeErrorMessage(error)).filter(Boolean).join('; ').trim();
 }
-/**
- * Detect Claude Code's synthetic "API Error: …" assistant message. When the
- * upstream Anthropic API returns a transient error (529 Overloaded, 5xx, gateway
- * timeouts, …), the Claude CLI swallows it and replaces the assistant turn with
- * a single `text` block whose body is literally `API Error: <reason>`. The
- * turn's stop_reason still claims `end_turn`, so the driver can't distinguish
- * it from a normal short reply without inspecting the text.
- *
- * Heuristics — keep them tight so real prose mentioning "API Error" doesn't
- * trip the detector:
- *  - exact prefix "API Error: "
- *  - total length ≤ 200 chars (the synthetic line is always short)
- *  - no newlines (legit prose containing "API Error" virtually always wraps)
- *
- * Returns the trimmed reason (e.g. "Overloaded", "Internal server error") when
- * matched, otherwise null. Callers decide whether the reason is retryable —
- * `looksRetryable` answers that.
- */
 export function detectClaudeApiError(text) {
     if (!text)
         return null;
@@ -198,36 +166,12 @@ export function detectClaudeApiError(text) {
     const m = trimmed.match(/^API Error:\s*(.+)$/i);
     return m ? m[1].trim() : null;
 }
-/**
- * Retryable Claude Code API errors — transient upstream conditions that
- * usually clear within seconds. Non-retryable conditions (auth, quota,
- * context length) fall through and surface to the user immediately.
- */
 export function isRetryableClaudeApiError(reason) {
     const r = reason.toLowerCase();
     if (/rate limit|rate limited|quota|usage limit|session limit/i.test(r))
         return false;
     return /overloaded|overload|timeout|timed out|500|502|503|504|529|temporar|gateway|connection|network|internal (server )?error/i.test(r);
 }
-/**
- * Detect Claude Code's "selected model is unavailable" notice — emitted when
- * the requested `--model` id is disabled / not provisioned for the account (a
- * 404 model_not_found). Its delivery differs by mode:
- *   - `-p`/stream-json: a `<synthetic>` assistant event carrying
- *     `error:"model_not_found"` plus the banner text, and a `result` event with
- *     `is_error` — both reach the parser.
- *   - TUI: the banner is *only* painted to the PTY screen. It is never written
- *     to the transcript JSONL and fires no Stop hook (verified 2026-06-13), so
- *     the screen scrape is the sole signal and the turn would otherwise hang
- *     until the stall watchdog (3–10 min).
- *
- * Matching is whitespace-insensitive on purpose: the TUI renders the banner
- * character-by-character with cursor positioning, so after ANSI stripping the
- * words lose their spaces and wrap arbitrarily ("issuewiththeselectedmodel").
- * Collapsing whitespace on both sides makes the match survive that rendering.
- * Callers compose the user-facing message via `claudeModelErrorMessage` with
- * the concrete model id they hold.
- */
 export function detectClaudeModelError(text) {
     if (!text)
         return false;
@@ -235,7 +179,6 @@ export function detectClaudeModelError(text) {
     return collapsed.includes('issuewiththeselectedmodel')
         || collapsed.includes('maynotexistoryoumaynothaveaccess');
 }
-/** User-facing message for an unavailable / no-access model (see {@link detectClaudeModelError}). */
 export function claudeModelErrorMessage(model) {
     const id = (model || '').trim();
     return `The selected model${id ? ` (${id})` : ''} is unavailable — it may not exist, or this account doesn't have access to it. Switch to a different model in pikiloom settings.`;
@@ -267,7 +210,6 @@ export function computeContext(s) {
         : null;
     return { contextUsedTokens: used, contextPercent: pct };
 }
-/** Max structured tool calls carried per preview emit (most recent win). */
 const PREVIEW_TOOL_CALLS_MAX = 40;
 export function buildStreamPreviewMeta(s) {
     const ctx = computeContext(s);
@@ -276,7 +218,6 @@ export function buildStreamPreviewMeta(s) {
         cachedInputTokens: s.cachedInputTokens,
         contextUsedTokens: ctx.contextUsedTokens, contextPercent: ctx.contextPercent,
     };
-    // Turn-cumulative output: finished calls' total + the in-flight call.
     const turnOutput = (s.turnOutputTokensBase ?? 0) + (s.outputTokens ?? 0);
     if (turnOutput > 0)
         meta.turnOutputTokens = turnOutput;
@@ -306,12 +247,6 @@ export function buildStreamPreviewMeta(s) {
     }
     return meta;
 }
-/**
- * Bounded, human-readable input detail for a live tool-call row. Bash shows
- * the raw command (the summary already carries the description); everything
- * else gets compact JSON. Returns null when there's nothing beyond the
- * summary worth expanding.
- */
 export function previewToolCallInput(name, input, max = 500) {
     if (input == null)
         return null;
@@ -329,11 +264,6 @@ export function previewToolCallInput(name, input, max = 500) {
         return null;
     }
 }
-/**
- * Bounded text preview of a tool result. Accepts the JSONL tool_result
- * `content` (string | block array) or a hook `tool_response` (string |
- * object). Extracts text blocks where present, falls back to compact JSON.
- */
 export function previewToolCallResult(content, max = 500) {
     if (content == null)
         return null;
@@ -348,7 +278,6 @@ export function previewToolCallResult(content, max = 500) {
         return text ? clipText(text, max) : null;
     }
     if (typeof content === 'object') {
-        // Hook tool_response commonly nests the payload under `content` / `result`.
         if (content.content != null && content.content !== content) {
             const nested = previewToolCallResult(content.content, max);
             if (nested)
@@ -373,7 +302,6 @@ function clipText(text, max) {
         return text;
     return `${text.slice(0, Math.max(0, max - 1)).trimEnd()}…`;
 }
-// Claude tool use helpers (used by driver-claude.ts)
 export function summarizeClaudeToolUse(name, input) {
     const tool = String(name || '').trim() || 'Tool';
     const description = shortValue(input?.description, 120);
@@ -408,10 +336,6 @@ export function summarizeClaudeToolUse(name, input) {
         }
         case 'TodoWrite': return 'Update plan';
         case 'AskUserQuestion': {
-            // Claude's built-in clarify tool. The CLI in `-p` mode self-resolves it
-            // with an error and degrades to a plain-text question in the same turn —
-            // we just surface the question text in the activity panel so users see
-            // what was asked.
             const qs = Array.isArray(input?.questions) ? input.questions : [];
             const first = qs[0];
             const q = shortValue(first?.question || input?.question, 120);
@@ -428,7 +352,6 @@ export function summarizeClaudeToolUse(name, input) {
             return c ? `Run shell: ${c}` : 'Run shell command';
         }
         default: {
-            // MCP tools come through as `mcp__<server>__<tool>` — unwrap common pikiloom tools
             const mcpMatch = tool.match(/^mcp__[^_]+__(.+)$/);
             const bare = mcpMatch ? mcpMatch[1] : tool;
             if (bare === 'im_send_file') {
@@ -464,7 +387,6 @@ export function summarizeClaudeToolResult(tool, block, toolUseResult) {
         return `${summary} done`;
     return `${summary} -> ${shortValue(detail, 120)}`;
 }
-// Usage helpers (used by drivers)
 export function roundPercent(value) {
     const n = Number(value);
     if (!Number.isFinite(n))
@@ -580,7 +502,6 @@ export function stripInjectedPrompts(text) {
         if (idx >= 0)
             text = text.slice(0, idx).trim();
     }
-    // Strip Codex IDE context prefix ("# Context from my IDE setup: ...")
     if (text.startsWith('# Context from')) {
         const tag = '## My request for Codex:\n';
         const idx = text.indexOf(tag);
@@ -595,18 +516,7 @@ export const SESSION_PREVIEW_IGNORED_USER_PATTERNS = [
 ];
 export const SESSION_PREVIEW_IMAGE_PLACEHOLDER_RE = /\[Image:[^\]]+\]/gi;
 export const SESSION_PREVIEW_FILE_PLACEHOLDER_RE = /\[Attached file:[^\]]+\]/gi;
-/**
- * Claude TUI mode prepends `@/abs/path/file.ext` mentions to the prompt as
- * positional argv (see `src/agent/drivers/claude-tui.ts`) — that's how the TUI
- * ingests local image files. The mentions end up baked into the JSONL user
- * `content` string verbatim. This regex captures them so:
- *   - the dashboard's user bubble (via `getClaudeSessionMessages`) can lift
- *     them into structured `image` blocks for thumbnail rendering;
- *   - session-list previews don't surface a long absolute path.
- * Whitespace-free paths only — matches what `claude-tui.ts` emits.
- */
 export const CLAUDE_AT_MENTION_IMAGE_RE = /(^|\s)@(\/[^\s@\n]+\.(?:png|jpe?g|gif|webp|svg))(?=\s|$)/gi;
-/** Pull the absolute paths out of every image-mention in `text`. */
 export function extractClaudeAtMentionImagePaths(text) {
     if (!text)
         return [];
@@ -615,8 +525,6 @@ export function extractClaudeAtMentionImagePaths(text) {
         out.push(m[2]);
     return out;
 }
-/** Remove image @-mentions from `text` while preserving the leading boundary
- *  character (start-of-string or whitespace) so adjacent content stays joinable. */
 export function stripClaudeAtMentionImages(text) {
     if (!text)
         return text;
@@ -638,18 +546,6 @@ export function sanitizeSessionUserPreviewText(text) {
 export function isPendingSessionId(sessionId) {
     return typeof sessionId === 'string' && sessionId.startsWith('pending_');
 }
-/**
- * Update a stream-state's session id and notify the caller in one step.
- *
- * Drivers used to assign `s.sessionId = ev.session_id ?? s.sessionId` at every
- * place the CLI surfaced an id, then leave promotion until `finalizeStreamResult`
- * at end-of-stream. That meant an early abort (before the result line) or a
- * mid-stream rotation (Claude `--resume` rewriting the session id) was invisible
- * to the bot runtime — leaving the runtime stuck on a pending id, or letting a
- * later insertion land on a phantom session. Routing through this helper makes
- * every observed id change propagate immediately to `opts.onSessionId`, which
- * in bot.ts wires straight into `promoteSessionRuntime`.
- */
 export function emitSessionIdUpdate(s, rawId) {
     if (typeof rawId !== 'string')
         return;
@@ -660,22 +556,8 @@ export function emitSessionIdUpdate(s, rawId) {
     try {
         s._emitSessionId?.(trimmed);
     }
-    catch { /* listeners must not break the stream loop */ }
-}
-/**
- * Canonical session-list display title used by *every* surface (IM channels
- * + dashboard). The order is intentional:
- *
- *   1. `title` — set ONCE from the original user prompt that started the
- *      session. Stable; never overwritten by sub-agent or tool prompts.
- *   2. `lastQuestion` — most recent user message. Fallback only, because for
- *      Claude this can be a Task-tool sub-agent prompt and we don't want
- *      sub-agent text leaking into the title.
- *   3. `sessionId` — last-resort identifier.
- *
- * The dashboard frontend (`dashboard/src/utils.ts`) mirrors this order — keep
- * the two in sync.
- */
+    catch { }
+}
 export function sessionListDisplayTitle(session) {
     const title = sanitizeSessionUserPreviewText(String(session.title || ''));
     if (title)