@pi-archimedes/core 2.7.3 → 2.9.0

package/src/startup/version.ts

@@ -1,26 +0,0 @@
-const NPM_REGISTRY_URL = "https://registry.npmjs.org/@earendil-works/pi-coding-agent/latest";
-const FETCH_TIMEOUT_MS = 4000;
-
-export async function fetchLatestVersion(): Promise<string | undefined> {
-  try {
-    const controller = new AbortController();
-    const timeout = setTimeout(() => controller.abort(), FETCH_TIMEOUT_MS);
-    const res = await fetch(NPM_REGISTRY_URL, { signal: controller.signal });
-    clearTimeout(timeout);
-    if (!res.ok) return undefined;
-    const data = (await res.json()) as { version?: string };
-    return data.version;
-  } catch {
-    return undefined;
-  }
-}
-
-export function compareVersions(a: string, b: string): number {
-  const pa = a.replace(/^v/, "").split(".").map(Number);
-  const pb = b.replace(/^v/, "").split(".").map(Number);
-  for (let i = 0; i < 3; i++) {
-    if ((pa[i] ?? 0) > (pb[i] ?? 0)) return 1;
-    if ((pa[i] ?? 0) < (pb[i] ?? 0)) return -1;
-  }
-  return 0;
-}

package/src/thinking/patch.test.ts

@@ -1,389 +0,0 @@
-import { describe, it, expect, vi, afterEach } from "vitest";
-
-// Symbols used by patch.ts to mark patched prototypes
-const PATCHED_KEY = Symbol.for("archimedes:thinkingPatched");
-const PATCH_VERSION_KEY = Symbol.for("archimedes:thinkingPatchVersion");
-
-// Dynamic import after mocking the pi-coding-agent module
-async function importPatch() {
-	vi.resetModules();
-	const mod = await import("./patch.js");
-	return mod.patchThinkingRenderer;
-}
-
-describe("patchThinkingRenderer", () => {
-	afterEach(() => {
-		vi.resetModules();
-	});
-
-	it("returns early when AssistantMessageComponent is undefined", async () => {
-		vi.doMock("@earendil-works/pi-coding-agent", () => ({
-			AssistantMessageComponent: undefined,
-			VERSION: "1.0.0",
-			highlightCode: vi.fn(),
-		}));
-
-		const patch = await importPatch();
-		// Should not throw
-		patch(() => ({} as any));
-	});
-
-	it("returns early when prototype is null", async () => {
-		const MockClass = function AssistantMessageComponent() {};
-		MockClass.prototype = null;
-
-		vi.doMock("@earendil-works/pi-coding-agent", () => ({
-			AssistantMessageComponent: MockClass,
-			VERSION: "1.0.0",
-			highlightCode: vi.fn(),
-		}));
-
-		const patch = await importPatch();
-		patch(() => ({} as any));
-		// Prototype must still be null — no patching occurred
-		expect(MockClass.prototype).toBeNull();
-	});
-
-	it("returns early when updateContent is not a function", async () => {
-		const MockClass = function AssistantMessageComponent() {};
-		MockClass.prototype.updateContent = "not a function";
-
-		vi.doMock("@earendil-works/pi-coding-agent", () => ({
-			AssistantMessageComponent: MockClass,
-			VERSION: "1.0.0",
-			highlightCode: vi.fn(),
-		}));
-
-		const patch = await importPatch();
-		patch(() => ({} as any));
-		// PATCHED_KEY must NOT be set — early return before patching
-		expect(MockClass.prototype[PATCHED_KEY]).toBeUndefined();
-	});
-
-	it("returns early when class name does not match", async () => {
-		const WrongName = function () {};
-		WrongName.prototype.updateContent = function () {};
-
-		vi.doMock("@earendil-works/pi-coding-agent", () => ({
-			AssistantMessageComponent: WrongName,
-			VERSION: "1.0.0",
-			highlightCode: vi.fn(),
-		}));
-
-		const patch = await importPatch();
-		patch(() => ({} as any));
-		// PATCHED_KEY must NOT be set — name mismatch causes early return
-		expect(WrongName.prototype[PATCHED_KEY]).toBeUndefined();
-	});
-
-	it("returns early when source lacks thinking check", async () => {
-		const MockClass = function AssistantMessageComponent() {};
-		MockClass.prototype.updateContent = function updateContent() {
-			// Does NOT contain: content.type === "thinking"
-			this.doSomething();
-		};
-
-		vi.doMock("@earendil-works/pi-coding-agent", () => ({
-			AssistantMessageComponent: MockClass,
-			VERSION: "1.0.0",
-			highlightCode: vi.fn(),
-		}));
-
-		const patch = await importPatch();
-		patch(() => ({} as any));
-		// PATCHED_KEY must NOT be set — signature mismatch causes early return
-		expect(MockClass.prototype[PATCHED_KEY]).toBeUndefined();
-	});
-
-	it("returns early when source lacks markdownTheme reference", async () => {
-		const MockClass = function AssistantMessageComponent() {};
-		MockClass.prototype.updateContent = function updateContent() {
-			// Has thinking check but no markdownTheme
-			if (this.content.type === "thinking") {
-				this.render();
-			}
-		};
-
-		vi.doMock("@earendil-works/pi-coding-agent", () => ({
-			AssistantMessageComponent: MockClass,
-			VERSION: "1.0.0",
-			highlightCode: vi.fn(),
-		}));
-
-		const patch = await importPatch();
-		patch(() => ({} as any));
-		// PATCHED_KEY must NOT be set — signature mismatch causes early return
-		expect(MockClass.prototype[PATCHED_KEY]).toBeUndefined();
-	});
-
-	it("patches successfully when signature matches", async () => {
-		const MockClass = function AssistantMessageComponent() {};
-		MockClass.prototype.updateContent = function updateContent() {
-			if (this.content.type === "thinking") {
-				this.markdownTheme.codeBlockIndent = "";
-			}
-		};
-
-		vi.doMock("@earendil-works/pi-coding-agent", () => ({
-			AssistantMessageComponent: MockClass,
-			VERSION: "1.0.0",
-			highlightCode: vi.fn(),
-		}));
-
-		const patch = await importPatch();
-		patch(() => ({} as any));
-
-		// The prototype should be marked as patched
-		expect(MockClass.prototype[PATCHED_KEY]).toBe(true);
-		expect(MockClass.prototype[PATCH_VERSION_KEY]).toBe("1.0.0");
-	});
-
-	it("marks prototype with correct version", async () => {
-		const MockClass = function AssistantMessageComponent() {};
-		MockClass.prototype.updateContent = function updateContent() {
-			if (this.content.type === "thinking") {
-				this.markdownTheme.codeBlockIndent = "";
-			}
-		};
-
-		vi.doMock("@earendil-works/pi-coding-agent", () => ({
-			AssistantMessageComponent: MockClass,
-			VERSION: "2.5.0",
-			highlightCode: vi.fn(),
-		}));
-
-		const patch = await importPatch();
-		patch(() => ({} as any));
-
-		expect(MockClass.prototype[PATCH_VERSION_KEY]).toBe("2.5.0");
-	});
-
-	it("re-patches when version changes", async () => {
-		// First patch with version 1.0.0
-		const MockClassV1 = function AssistantMessageComponent() {};
-		MockClassV1.prototype.updateContent = function updateContent() {
-			if (this.content.type === "thinking") {
-				this.markdownTheme.codeBlockIndent = "";
-			}
-		};
-
-		vi.doMock("@earendil-works/pi-coding-agent", () => ({
-			AssistantMessageComponent: MockClassV1,
-			VERSION: "1.0.0",
-			highlightCode: vi.fn(),
-		}));
-
-		const patchV1 = await importPatch();
-		patchV1(() => ({} as any));
-		expect(MockClassV1.prototype[PATCH_VERSION_KEY]).toBe("1.0.0");
-
-		// Now simulate a version change with a fresh class
-		const MockClassV2 = function AssistantMessageComponent() {};
-		MockClassV2.prototype.updateContent = function updateContent() {
-			if (this.content.type === "thinking") {
-				this.markdownTheme.codeBlockIndent = "";
-			}
-		};
-
-		vi.doMock("@earendil-works/pi-coding-agent", () => ({
-			AssistantMessageComponent: MockClassV2,
-			VERSION: "2.0.0",
-			highlightCode: vi.fn(),
-		}));
-
-		const patchV2 = await importPatch();
-		patchV2(() => ({} as any));
-		expect(MockClassV2.prototype[PATCH_VERSION_KEY]).toBe("2.0.0");
-	});
-
-	it("accepts a minified thinking-check variant (no whitespace around ===)", async () => {
-		const MockClass = function AssistantMessageComponent() {};
-		MockClass.prototype.updateContent = function updateContent() {
-			// Minified dist-chunk shape: no whitespace around ===, single quotes
-			const content = { type: "thinking" };
-			if (content.type==="thinking") {
-				this.markdownTheme.codeBlockIndent="";
-			}
-		};
-
-		vi.doMock("@earendil-works/pi-coding-agent", () => ({
-			AssistantMessageComponent: MockClass,
-			VERSION: "1.0.0",
-			highlightCode: vi.fn(),
-		}));
-
-		const patch = await importPatch();
-		patch(() => ({} as any));
-
-		// The minification-safe regex probe must accept the minified variant
-		expect(MockClass.prototype[PATCHED_KEY]).toBe(true);
-	});
-
-	it("rejects a negated thinking check", async () => {
-		const MockClass = function AssistantMessageComponent() {};
-		MockClass.prototype.updateContent = function updateContent() {
-			// pi 0.84.3's own minified chunk contains
-			// `thinkingContent.type!=="thinking"` (inner batch-loop break). A source whose
-			// ONLY thinking-relations are negations must NOT pass the probe.
-			const content = { type: "thinking" };
-			if (content.type!=="thinking") {
-				this.markdownTheme.codeBlockIndent="";
-			}
-		};
-
-		vi.doMock("@earendil-works/pi-coding-agent", () => ({
-			AssistantMessageComponent: MockClass,
-			VERSION: "1.0.0",
-			highlightCode: vi.fn(),
-		}));
-
-		const patch = await importPatch();
-		patch(() => ({} as any));
-
-		// PATCHED_KEY must NOT be set — the probe must not match `!==`
-		expect(MockClass.prototype[PATCHED_KEY]).toBeUndefined();
-	});
-
-	it("re-patches on same version to update getTheme closure", async () => {
-		const MockClass = function AssistantMessageComponent() {};
-		MockClass.prototype.updateContent = function updateContent() {
-			if (this.content.type === "thinking") {
-				this.markdownTheme.codeBlockIndent = "";
-			}
-		};
-
-		vi.doMock("@earendil-works/pi-coding-agent", () => ({
-			AssistantMessageComponent: MockClass,
-			VERSION: "1.0.0",
-			highlightCode: vi.fn(),
-		}));
-
-		const patch = await importPatch();
-
-		// First patch
-		patch(() => ({} as any));
-		expect(MockClass.prototype[PATCHED_KEY]).toBe(true);
-		const first = MockClass.prototype.updateContent;
-
-		// Second patch (same version) — must produce a new function
-		patch(() => ({} as any));
-		expect(MockClass.prototype[PATCHED_KEY]).toBe(true);
-		expect(MockClass.prototype[PATCH_VERSION_KEY]).toBe("1.0.0");
-		// The patched function must be a new closure (not the same reference)
-		expect(MockClass.prototype.updateContent).not.toBe(first);
-	});
-
-	// ── Configurable thinking label (issue #36) ────────────────────────────
-
-	// Patches with a valid AssistantMessageComponent + mocked pi-tui so the
-	// Markdown content rendered for a thinking block can be inspected.
-	async function patchAndRender(
-		config?: { labelText?: string; labelColor?: string },
-		thinkingText = "Let me consider this carefully.",
-	) {
-		const MockClass = function AssistantMessageComponent() {};
-		MockClass.prototype.updateContent = function updateContent() {
-			if (this.content.type === "thinking") {
-				this.markdownTheme.codeBlockIndent = "";
-			}
-		};
-
-		vi.doMock("@earendil-works/pi-coding-agent", () => ({
-			AssistantMessageComponent: MockClass,
-			VERSION: "1.0.0",
-			highlightCode: vi.fn(),
-		}));
-
-		const capturedMarkdown: any[] = [];
-		class MockMarkdown {
-			content: string;
-			constructor(content: string, ..._rest: any[]) {
-				this.content = content;
-				capturedMarkdown.push(this);
-			}
-		}
-		class MockSpacer {}
-		class MockText {}
-		vi.doMock("@earendil-works/pi-tui", () => ({
-			Markdown: MockMarkdown,
-			Spacer: MockSpacer,
-			Text: MockText,
-		}));
-
-		vi.resetModules();
-		const mod = await import("./patch.js");
-		mod.patchThinkingRenderer(
-			() => ({ getFgAnsi: () => "", fg: (_t: string, text: string) => text }) as any,
-			config,
-		);
-
-		const instance = {
-			contentContainer: { clear: vi.fn(), addChild: vi.fn() },
-			isStreaming: false,
-			markdownTheme: { codeBlockIndent: "" },
-			markdownTransformers: [],
-			hideThinkingBlock: false,
-			outputPad: 1,
-		};
-		const message = {
-			content: [{ type: "thinking", thinking: thinkingText }],
-			stopReason: undefined,
-		};
-		MockClass.prototype.updateContent.call(instance, message, false);
-		return capturedMarkdown;
-	}
-
-	it("uses configured labelText/labelColor for the thinking label", async () => {
-		const captured = await patchAndRender({
-			labelText: "Yapping...",
-			labelColor: "255,215,0",
-		});
-
-		expect(captured).toHaveLength(1);
-		expect(
-			captured[0]!.content.startsWith("\x1b[1m\x1b[38;2;255;215;0mYapping...\x1b[39m\x1b[22m\n\n"),
-		).toBe(true);
-	});
-
-	it("defaults to the original Thinking... label when no config is given", async () => {
-		const captured = await patchAndRender();
-
-		expect(captured).toHaveLength(1);
-		// Byte-identical to the previous hardcoded THINKING_LABEL
-		expect(
-			captured[0]!.content.startsWith("\x1b[1m\x1b[38;2;255;215;0mThinking...\x1b[39m\x1b[22m\n\n"),
-		).toBe(true);
-	});
-
-	it("trims whitespace from configured labelText and labelColor", async () => {
-		const captured = await patchAndRender({
-			labelText: "  Yapping...  ",
-			labelColor: " 255, 215, 0 ",
-		});
-
-		expect(
-			captured[0]!.content.startsWith("\x1b[1m\x1b[38;2;255;215;0mYapping...\x1b[39m\x1b[22m\n\n"),
-		).toBe(true);
-	});
-
-	it("falls back to 255,215,0 when labelColor is not a valid RGB triple", async () => {
-		const captured = await patchAndRender({
-			labelText: "Hmm",
-			labelColor: "not-a-color",
-		});
-
-		expect(
-			captured[0]!.content.startsWith("\x1b[1m\x1b[38;2;255;215;0mHmm\x1b[39m\x1b[22m\n\n"),
-		).toBe(true);
-	});
-
-	it("does not double-prepend the label when content already starts with it", async () => {
-		const label = "\x1b[1m\x1b[38;2;255;215;0mYapping...\x1b[39m\x1b[22m";
-		const captured = await patchAndRender(
-			{ labelText: "Yapping...", labelColor: "255,215,0" },
-			`${label}\n\nAlready labelled body.`,
-		);
-
-		expect(captured[0]!.content).toBe(`${label}\n\nAlready labelled body.`);
-	});
-});

package/src/thinking/patch.ts

@@ -1,288 +0,0 @@
-import type { Theme } from "@earendil-works/pi-coding-agent";
-import { AssistantMessageComponent, VERSION } from "@earendil-works/pi-coding-agent";
-import { Markdown, type MarkdownOptions, type MarkdownTheme, Spacer, Text } from "@earendil-works/pi-tui";
-import { buildMutedMarkdownTheme } from "./theme.js";
-
-// Track which pi version we patched against to detect incompatibility
-const PATCHED_KEY = Symbol.for("archimedes:thinkingPatched");
-const PATCH_VERSION_KEY = Symbol.for("archimedes:thinkingPatchVersion");
-
-/**
- * Patches `AssistantMessageComponent.prototype.updateContent` so thinking
- * blocks render with a muted `MarkdownTheme`. Called on every session_start
- * to capture a fresh `getTheme` closure (required for /resume).
- *
- * Re-patches when pi version changes to catch breaking upstream changes.
- *
- * @param config Optional labelText/labelColor overrides for the thinking
- *   block header. When omitted (or empty/invalid), the original defaults
- *   ("Thinking..." / "255,215,0") are used, producing byte-identical output.
- */
-export function patchThinkingRenderer(
-  getTheme: () => Theme,
-  config?: { labelText?: string; labelColor?: string },
-): void {
-  if (!AssistantMessageComponent) return;
-
-  const proto = AssistantMessageComponent.prototype;
-  if (
-    !proto ||
-    typeof proto.updateContent !== "function" ||
-    AssistantMessageComponent.name !== "AssistantMessageComponent"
-  ) {
-    return;
-  }
-
-  const src = proto.updateContent.toString();
-  // NOTE: pi ships the interactive TUI in a minified bundle chunk at runtime, so
-  // the source we see via .toString() can be `content.type==="thinking"` (no
-  // spaces) even where dist is readable. The probe below must therefore be
-  // minification-safe: a whitespace-tolerant regex rather than an exact
-  // substring. A bare `space === "thinking"` (or any other field) still does
-  // not match, as required.
-  const hasThinkingCheck = /content\.type\s*===\s*["']thinking["']/.test(src);
-  const hasMarkdownTheme = src.includes("this.markdownTheme");
-  if (!hasThinkingCheck || !hasMarkdownTheme) {
-    console.warn(
-      `[archimedes] Skipping thinking renderer patch — signature mismatch
-  hasThinkingCheck: ${hasThinkingCheck}, hasMarkdownTheme: ${hasMarkdownTheme}
-  This likely means pi's AssistantMessageComponent changed. The muted theme
-  for thinking blocks will not be applied.`,
-    );
-    return;
-  }
-
-  // Check if already patched against the current pi version
-  const currentVersion = VERSION ?? "unknown";
-  const patchVersion = (proto as any)[PATCH_VERSION_KEY] as string | undefined;
-  if ((proto as any)[PATCHED_KEY] && patchVersion === currentVersion) {
-    // Already patched for this version — just update the closure by re-patching
-    // (needed for /resume to get fresh getTheme)
-  } else {
-    // First patch or version changed — warn if version mismatch
-    if ((proto as any)[PATCHED_KEY] && patchVersion && patchVersion !== currentVersion) {
-      console.warn(`[archimedes] Re-patching thinking renderer: pi version changed ${patchVersion} → ${currentVersion}`);
-    }
-  }
-
-  // Re-patched every session_start — /resume needs a fresh getTheme closure.
-  //
-  // Shape: 0.84.3 pi native updateContent:
-  //   updateContent(message, isStreaming = this.isStreaming) {
-  //     this.lastMessage = message;
-  //     this.isStreaming = isStreaming;
-  //     this.contentContainer.clear();
-  //     ...
-  //     // batches consecutive "thinking" parts into thinkingBlocks
-  //     //   (skipping empties), i-- after the inner loop,
-  //     //   renders as ONE Markdown section of thinkingBlocks.join("\n\n")
-  //     // or ONE static Text label when hidden.
-  //     // stop-reason: const hasToolCalls = content.some(...);
-  //     //   this.hasToolCalls = hasToolCalls; (render() uses for OSC-133 zones)
-  //     //   stopReason === "length" → Spacer + "truncated" Text
-  //     //   else if (!hasToolCalls) { aborted / error branches }
-  (proto as any).updateContent = function (this: any, message: any, isStreaming?: boolean): void {
-    this.lastMessage = message;
-    if (isStreaming !== undefined) this.isStreaming = isStreaming;
-
-    this.markdownTheme.codeBlockIndent = "";
-    this.contentContainer.clear();
-
-    // Build the thinking-block header from the closure config once per render.
-    // Defaults preserve the original byte-identical output ("Thinking..." in
-    // bold truecolor 255,215,0).
-    const buildThinkingLabel = (): string => {
-      const label = config?.labelText?.trim() ? config.labelText.trim() : "Thinking...";
-      const color = config?.labelColor?.trim() ? config.labelColor.trim() : "255,215,0";
-      const parts = color.split(",").map((p) => p.trim());
-      // Valid iff exactly three 0..255 components (an "R,G,B" triple).
-      const valid =
-        parts.length === 3 &&
-        parts.every((p) => /^\d{1,3}$/.test(p) && Number(p) >= 0 && Number(p) <= 255);
-      const [r, g, b] = valid ? parts : ["255", "215", "0"];
-      return `\x1b[1m\x1b[38;2;${r};${g};${b}m${label}\x1b[39m\x1b[22m`;
-    };
-
-    const hasVisibleContent = message.content.some(
-      (c: any) =>
-        (c.type === "text" && c.text.trim()) ||
-        (c.type === "thinking" && c.thinking.trim()),
-    );
-
-    if (hasVisibleContent) {
-      this.contentContainer.addChild(new Spacer(1));
-    }
-
-    // Lazy muted theme: built once per updateContent call.
-    let mutedTheme: ReturnType<typeof buildMutedMarkdownTheme> | undefined;
-    let theme: Theme | undefined;
-    let themeFailed = false;
-
-    const ensureTheme = (): Theme | undefined => {
-      if (themeFailed) return undefined;
-      if (!theme) {
-        try {
-          theme = getTheme();
-        } catch {
-          themeFailed = true;
-          return undefined;
-        }
-      }
-      return theme;
-    };
-
-    const ensureMuted = (): MarkdownTheme | undefined => {
-      if (!mutedTheme) {
-        const t = ensureTheme();
-        if (!t) return undefined;
-        mutedTheme = buildMutedMarkdownTheme(t);
-      }
-      return mutedTheme;
-    };
-
-    // Pi's native updateContent passes a `transform` (createMarkdownTransform)
-    // to Markdown so the markdown-transformer pipeline runs — that is what
-    // renders Mermaid blocks to ASCII and applies any extension transformers.
-    // We must preserve it here, otherwise those transformers are silently
-    // dropped when this patch replaces updateContent.
-    //
-    // NOTE: `createMarkdownTransform` is not exported from pi-coding-agent, so
-    // we inline an equivalent pipeline over `this.markdownTransformers`.
-    //
-    // The `transform` option was added to @earendil-works/pi-tui in 0.84.1.
-    // At runtime older pi-tui ignores the field, 0.84.1+ honors it.
-    type MarkdownOptionsWithTransform = MarkdownOptions & {
-      transform?: (markdown: string, availableWidth: number) => string;
-    };
-    const transformFor =
-      (messageType: "assistant" | "assistant-thinking") =>
-      (markdown: string, availableWidth: number): string => {
-        let out = markdown;
-        for (const transformer of (this as any).markdownTransformers ?? []) {
-          try {
-            const transformed = transformer(out, {
-              messageType,
-              isStreaming: this.isStreaming,
-              availableWidth,
-            });
-            if (typeof transformed === "string") out = transformed;
-          } catch {
-            // Keep the current markdown and continue with the next transformer.
-          }
-        }
-        return out;
-      };
-
-    // Render content in order.
-    for (let i = 0; i < message.content.length; i++) {
-      const content = message.content[i];
-      if (content.type === "text" && content.text.trim()) {
-        this.contentContainer.addChild(
-          new Markdown(
-            content.text.trim(),
-            this.outputPad ?? 1,
-            0,
-            this.markdownTheme,
-            undefined,
-            { transform: transformFor("assistant") } as MarkdownOptionsWithTransform,
-          ),
-        );
-      } else if (content.type === "thinking") {
-        // Batch a consecutive run of thinking parts into one section
-        // (mirrors 0.84.3 pi native behaviour: thinkBlocks, i-- on the
-        // inner loop, early continue on zero-length runs).
-        const thinkBlocks: string[] = [];
-        for (; i < message.content.length; i++) {
-          const thinkingPart = message.content[i];
-          if (thinkingPart.type !== "thinking") break;
-          const trimmed = thinkingPart.thinking.trim();
-          if (trimmed) thinkBlocks.push(trimmed);
-        }
-        i--;
-        if (thinkBlocks.length === 0) continue;
-
-        const hasVisibleContentAfter = message.content
-          .slice(i + 1)
-          .some(
-            (c: any) =>
-              (c.type === "text" && c.text.trim()) ||
-              (c.type === "thinking" && c.thinking.trim()),
-          );
-
-        if (this.hideThinkingBlock) {
-          // One static label for the whole run when hidden.
-          const t = ensureTheme();
-          if (!t) continue;
-          this.contentContainer.addChild(
-            new Text(t.italic(t.fg("thinkingText", this.hiddenThinkingLabel)), this.outputPad ?? 1, 0),
-          );
-          if (hasVisibleContentAfter) {
-            this.contentContainer.addChild(new Spacer(1));
-          }
-        } else {
-          let thinkingContent = thinkBlocks.join("\n\n");
-          const label = buildThinkingLabel();
-          if (!thinkingContent.startsWith(label)) {
-            thinkingContent = `${label}\n\n${thinkingContent}`;
-          }
-          const t = ensureTheme();
-          if (!t) continue;
-          const muted = ensureMuted();
-          this.contentContainer.addChild(
-            new Markdown(
-              thinkingContent,
-              this.outputPad ?? 1,
-              0,
-              muted ?? this.markdownTheme,
-              {
-                color: (text: string) => t.fg("thinkingText", text),
-                italic: true,
-              },
-              { transform: transformFor("assistant-thinking") } as MarkdownOptionsWithTransform,
-            ),
-          );
-          if (hasVisibleContentAfter) {
-            this.contentContainer.addChild(new Spacer(1));
-          }
-        }
-      }
-    }
-
-    // Stop-reason handling — 0.84.3 pi shape. `hasToolCalls` is required by
-    // the component's render() for OSC-133 prompt zones, so it must be set.
-    const hasToolCalls = message.content.some((c: any) => c.type === "toolCall");
-    this.hasToolCalls = hasToolCalls;
-
-    if (message.stopReason === "length") {
-      this.contentContainer.addChild(new Spacer(1));
-      const t = ensureTheme();
-      if (t)
-        this.contentContainer.addChild(
-          new Text(t.fg("error", "Response was truncated before completion."), this.outputPad ?? 1, 0),
-        );
-    } else if (!hasToolCalls) {
-      if (message.stopReason === "aborted") {
-        const abortMessage =
-          message.errorMessage && message.errorMessage !== "Request was aborted"
-            ? message.errorMessage
-            : "Operation aborted";
-        this.contentContainer.addChild(new Spacer(1));
-        const t = ensureTheme();
-        if (t) this.contentContainer.addChild(new Text(t.fg("error", abortMessage), this.outputPad ?? 1, 0));
-      } else if (message.stopReason === "error") {
-        const errorMsg = message.errorMessage || "Unknown error";
-        this.contentContainer.addChild(new Spacer(1));
-        const t = ensureTheme();
-        if (t) {
-          this.contentContainer.addChild(
-            new Text(t.fg("error", `Error: ${errorMsg}`), this.outputPad ?? 1, 0),
-          );
-        }
-      }
-    }
-  };
-
-  // Mark as patched with version for incompatibility detection
-  (proto as any)[PATCHED_KEY] = true;
-  (proto as any)[PATCH_VERSION_KEY] = currentVersion;
-}