@opengsd/gsd-pi 1.1.1-dev.616a1a1 → 1.1.1-dev.74e8dd1

package/packages/gsd-agent-modes/dist/modes/interactive/controllers/chat-controller.js

@@ -20,6 +20,9 @@ let orphanedSegments = [];
 // Invocation matching only reconciles IDs reported by different event streams.
 // Same-source identical invocations are separate concurrent tool calls.
 const toolRegistrationSources = new WeakMap();
+const PROVISIONAL_PRE_TOOL_ACTIONS = "check|inspect|look(?:\\s+into)?|read|open|search|grep|scan|run|verify|test|trace|review|investigate|reproduce|use|gather|find|pull|query|take a look|update|patch|edit|change|modify|write|create|add|remove|apply";
+const FIRST_PERSON_PROVISIONAL_PRE_TOOL_RE = new RegExp(`^(?:i(?:'ll| will)|i(?:'m| am) going to|let me|i need to)\\s+(?:${PROVISIONAL_PRE_TOOL_ACTIONS})\\b`, "i");
+const GERUND_PROVISIONAL_PRE_TOOL_RE = /^(?:checking|inspecting|reading|searching|running|verifying|testing|tracing|reviewing|investigating|scanning|updating|patching|editing|writing|creating|applying)\b/i;
 function findPendingToolByInvocation(pendingTools, toolName, args, source) {
     let fallback;
     for (const component of pendingTools.values()) {
@@ -244,6 +247,64 @@ function buildDesiredSegments(blocks, options = {}) {
     closeRun();
     return desired;
 }
+function isToolUseBlock(block) {
+    return block?.type === "toolCall" || block?.type === "serverToolUse";
+}
+function isMcpToolBlock(block) {
+    if (!isToolUseBlock(block))
+        return false;
+    const toolName = typeof block?.name === "string" ? block.name : "";
+    return typeof block?.mcpServer === "string" || toolName.startsWith("mcp__");
+}
+function hasPostToolText(blocks, firstToolIdx) {
+    if (firstToolIdx < 0)
+        return false;
+    return blocks.some((b, idx) => (idx > firstToolIdx
+        && b?.type === "text"
+        && typeof b?.text === "string"
+        && b.text.trim().length > 0));
+}
+function normalizeProvisionalText(text) {
+    return text
+        .trim()
+        .replace(/[’`]/g, "'")
+        .replace(/\s+/g, " ");
+}
+export function isProvisionalPreToolProse(text) {
+    const normalized = normalizeProvisionalText(text);
+    if (!normalized)
+        return false;
+    if (textInvitesUserReply(normalized))
+        return false;
+    if (/\?\s*$/.test(normalized))
+        return false;
+    return FIRST_PERSON_PROVISIONAL_PRE_TOOL_RE.test(normalized)
+        || GERUND_PROVISIONAL_PRE_TOOL_RE.test(normalized);
+}
+function getProvisionalPreToolPrunePlan(message) {
+    const blocks = message.content;
+    const firstToolIdx = blocks.findIndex(isToolUseBlock);
+    return {
+        firstToolIdx,
+        shouldPrune: message.provider === "claude-code"
+            && firstToolIdx >= 0
+            && blocks.some(isMcpToolBlock)
+            && hasPostToolText(blocks, firstToolIdx),
+    };
+}
+function buildDesiredSegmentsForMessage(message) {
+    const { shouldPrune, firstToolIdx } = getProvisionalPreToolPrunePlan(message);
+    return buildDesiredSegments(message.content, {
+        shouldSkipTextBlock: (block, index) => {
+            if (!shouldPrune || firstToolIdx < 0 || index >= firstToolIdx)
+                return false;
+            if (getVisibleTextLikeBlockType(block) !== "text")
+                return false;
+            const textValue = typeof block?.text === "string" ? block.text : "";
+            return isProvisionalPreToolProse(textValue);
+        },
+    });
+}
 function hasVisibleAssistantContent(message) {
     return message.content.some((c) => getVisibleTextLikeBlockType(c) !== undefined);
 }
@@ -576,44 +637,21 @@ export async function handleAgentEvent(host, event) {
                 // Build desired segment plan from content[].
                 {
                     const blocks = host.streamingMessage.content;
-                    const isClaudeCodeProvider = host.streamingMessage.provider === "claude-code";
-                    const hasMcpToolBlock = blocks.some((b) => {
-                        if (b?.type === "toolCall") {
-                            return typeof b?.mcpServer === "string" || String(b?.name ?? "").startsWith("mcp__");
-                        }
-                        if (b?.type === "serverToolUse") {
-                            return typeof b?.mcpServer === "string" || String(b?.name ?? "").startsWith("mcp__");
-                        }
-                        return false;
-                    });
-                    const firstToolIdx = blocks.findIndex((b) => b.type === "toolCall" || b.type === "serverToolUse");
-                    const hasPostToolText = firstToolIdx >= 0
-                        && blocks.some((b, idx) => (idx > firstToolIdx
-                            && b?.type === "text"
-                            && typeof b?.text === "string"
-                            && b.text.trim().length > 0));
                     // Only prune provisional pre-tool prose after post-tool prose exists,
                     // so MCP tool-only windows do not blank the assistant content.
-                    const shouldDropPreToolProse = isClaudeCodeProvider && hasMcpToolBlock && hasPostToolText;
-                    let desired = buildDesiredSegments(blocks, {
-                        shouldSkipTextBlock: (block, index) => {
-                            if (!shouldDropPreToolProse || firstToolIdx < 0 || index >= firstToolIdx)
-                                return false;
-                            if (getVisibleTextLikeBlockType(block) !== "text")
-                                return false;
-                            const textValue = typeof block?.text === "string" ? block.text : "";
-                            return !/\?\s*$/.test(textValue.trim());
-                        },
-                    });
+                    const { shouldPrune: shouldPruneProvisionalPreToolProse } = getProvisionalPreToolPrunePlan(host.streamingMessage);
+                    let desired = buildDesiredSegmentsForMessage(host.streamingMessage);
                     desired = filterRedundantDiscussTextRuns(desired, blocks);
                     // Claude Code MCP can emit provisional pre-tool prose that gets
                     // superseded by post-tool output. Prune stale text-run segments so
                     // the final assistant output remains below tool output.
-                    if (shouldDropPreToolProse && firstToolIdx >= 0) {
+                    if (shouldPruneProvisionalPreToolProse) {
                         if (orphanedSegments.length > 0) {
                             const remainingOrphans = [];
                             for (const orphan of orphanedSegments) {
-                                if (orphan.kind === "text-run" && orphan.contentType === "text") {
+                                if (orphan.kind === "text-run"
+                                    && orphan.contentType === "text"
+                                    && isProvisionalPreToolProse(orphan.cachedText ?? "")) {
                                     host.chatContainer.removeChild(orphan.component);
                                     if (host.streamingComponent === orphan.component) {
                                         host.streamingComponent = undefined;
@@ -799,14 +837,14 @@ export async function handleAgentEvent(host, event) {
                 const shouldRenderAssistant = hasVisibleAssistantContent(host.streamingMessage)
                     || ((host.streamingMessage.stopReason === "aborted" || host.streamingMessage.stopReason === "error")
                         && !hasAssistantToolBlocks(host.streamingMessage));
-                const suppressRedundantHandoff = shouldSuppressRedundantHandoffText(host.session.messages, extractAssistantText(host.streamingMessage), orphanedSegments, renderedSegments);
+                const suppressRedundantHandoff = shouldSuppressRedundantHandoffText(host.session.messages, extractAssistantText(host.streamingMessage), orphanedSegments, []);
                 // The final message_end payload can contain additional text/thinking
                 // blocks that never arrived via message_update (e.g. SDK result
                 // aggregation). Rebuild this in-flight turn from final content so
                 // ranges/components don't keep stale partial indices.
                 if (renderedSegments.length > 0) {
                     const finalBlocks = host.streamingMessage.content;
-                    const desired = filterRedundantDiscussTextRuns(buildDesiredSegments(finalBlocks), finalBlocks);
+                    const desired = filterRedundantDiscussTextRuns(buildDesiredSegmentsForMessage(host.streamingMessage), finalBlocks);
                     const toolComponentsById = new Map();
                     for (const [toolId, component] of host.pendingTools.entries()) {
                         toolComponentsById.set(toolId, component);
@@ -954,6 +992,9 @@ export async function handleAgentEvent(host, event) {
                 host.streamingComponent.setShowMetadata(true);
                 host.streamingComponent.updateContent(host.streamingMessage);
             }
+            for (const component of new Set(host.pendingTools.values())) {
+                component.markHistoricalNoResult();
+            }
             replaceCompactToolRowsWithPhaseSummary(host);
             host.streamingComponent = undefined;
             host.streamingMessage = undefined;