pi-claude-code-ui 1.0.85 → 1.0.87

package/CHANGELOG.md

@@ -3,6 +3,18 @@
 > [!IMPORTANT]
 > **1.0.69 — package rename (permanent).** Canonical npm name is now [`pi-claude-code-ui`](https://www.npmjs.com/package/pi-claude-code-ui). `pi-claude-style-tools` is legacy and will not receive further releases. Install with `pi install npm:pi-claude-code-ui` or `npm i pi-claude-code-ui`.
 
+## 1.0.87 — 2026-10-19
+
+### Fixed
+
+- **Expanding image reads no longer crashes Pi** — clear the image-source reuse cache together with the hidden image components when a `read` result is collapsed. Re-expanding standalone or grouped image reads now creates valid image components instead of reusing a missing component and throwing `TypeError: Cannot read properties of undefined (reading 'render')`.
+
+## 1.0.86 — 2026-10-08
+
+### Fixed
+
+- **Display-math scanning no longer splits fenced blocks on `$$` and `\[ … \]`** (Perry Trolard, [#33](https://github.com/FammasMaz/pi-cc-tools/issues/33)) - paragraph rebuilding now respects the same Markdown code regions as inline math formatting, so `$$` (shell PID) and `\[ … \]` (grep/ripgrep char-class regexes) inside ```` ``` ```` fences and inline code stay literal instead of cutting the block into a math segment.
+
 ## 1.0.83 — 2026-09-07
 
 ### Fixed

package/extensions/index.ts

@@ -2001,6 +2001,32 @@ function patchTerminalWriteTagScrubber(): void {
 	proto[TERMINAL_SCRUB_PATCH_FLAG] = true;
 }
 
+// Code spans and fenced blocks are literal Markdown source. Math preprocessing
+// must never touch delimiters inside them: `$$` is the shell PID and `\[ … \]
+// appears in grep/ripgrep class regexes. Inline formatting splits on this
+// expression before transforming; display-math block scanning below reuses it
+// so a math candidate opening inside code is skipped in favor of real prose.
+const MARKDOWN_CODE_REGION_RE = /(`{3,}[\s\S]*?`{3,}|`[^`\n]*`)/g;
+
+interface CodeRegion {
+	start: number;
+	end: number;
+}
+
+function findMarkdownCodeRegions(text: string): CodeRegion[] {
+	const regions: CodeRegion[] = [];
+	MARKDOWN_CODE_REGION_RE.lastIndex = 0;
+	let match: RegExpExecArray | null;
+	while ((match = MARKDOWN_CODE_REGION_RE.exec(text))) {
+		regions.push({ start: match.index, end: match.index + match[0].length });
+	}
+	return regions;
+}
+
+function codeRegionContaining(regions: CodeRegion[], index: number): CodeRegion | undefined {
+	return regions.find((region) => index >= region.start && index < region.end);
+}
+
 function replaceInlineMath(text: string): string {
 	if (!hasInlineMathMarkers(text)) return text;
 	const transform = (segment: string): string => {
@@ -2012,10 +2038,7 @@ function replaceInlineMath(text: string): string {
 			return `${prefix}${codeSpan(formatMathForDisplay(body, false))}`;
 		});
 	};
-	// Markdown code spans and fenced blocks are literal source. Apply math
-	// formatting only to prose segments so shell variables such as `$base` and
-	// `$scope` remain byte-for-byte unchanged in displayed/copied commands.
-	const parts = text.split(/(`{3,}[\s\S]*?`{3,}|`[^`\n]*`)/g);
+	const parts = text.split(MARKDOWN_CODE_REGION_RE);
 	return parts.map((part, index) => index % 2 === 1 ? part : transform(part)).join("");
 }
 
@@ -2132,14 +2155,31 @@ function buildParagraphSegments(
 		return segments;
 	}
 	const scanLoose = shouldScanLooseBracketMath(text);
+	// Code regions are computed once for the whole paragraph: a display-math
+	// candidate must be rejected before its fence gets cut into separate
+	// Markdown segments (issue #33, sibling of the inline fix in 2ccd158).
+	const codeRegions = findMarkdownCodeRegions(text);
+	// `cursor` marks the start of prose not yet emitted; `scanFrom` only advances
+	// past rejected code regions so skipped candidates never erase prose.
 	let cursor = 0;
-	while (cursor < text.length) {
-		const next = findNextDisplayMathBlock(text, cursor, scanLoose);
+	let scanFrom = 0;
+	while (scanFrom < text.length) {
+		let next = findNextDisplayMathBlock(text, scanFrom, scanLoose);
+		// Math delimiters found inside a fenced block or code span are literal
+		// source, not math. Resume the scan after that code region so a later
+		// genuine `$$ … $$` in prose is still honored.
+		while (next) {
+			const region = codeRegionContaining(codeRegions, next.index);
+			if (!region) break;
+			scanFrom = region.end;
+			next = findNextDisplayMathBlock(text, scanFrom, scanLoose);
+		}
 		if (!next) break;
 		appendMarkdownSegment(segments, text.slice(cursor, next.index), theme, options);
 		const raw = text.slice(next.contentStart, next.contentEnd).trim();
 		if (raw) segments.push({ kind: "math", raw });
 		cursor = next.endIndex;
+		scanFrom = next.endIndex;
 	}
 	appendMarkdownSegment(segments, text.slice(cursor), theme, options);
 	return segments;
@@ -2976,6 +3016,8 @@ function removeImageChildren(component: any): void {
 		try { component.removeChild?.(child); } catch { /* noop */ }
 	}
 	component.imageComponents = [];
+	// Pi reuses image components by matching imageSources; clear both caches.
+	component.imageSources = [];
 	component.imageSpacers = [];
 }
 
@@ -7876,3 +7918,11 @@ export default function (pi: ExtensionAPI) {
 		bumpToolBranchVisualEpoch();
 	});
 }
+
+// Test-only escape hatch: this package is a default-export extension and the
+// export surface Pi consumes must stay minimal. Regression tests for math
+// preprocessing reach these internals through this namespace instead of
+// widening the public export (see scripts/math-preprocess.test.ts).
+export const __mathPreprocessInternals = {
+	buildParagraphSegments,
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-claude-code-ui",
-  "version": "1.0.85",
+  "version": "1.0.87",
   "description": "Claude Code-style grouped tool rows for pi with Ctrl+O/Ctrl+Shift+O previews, math rendering, and consistent built-in, MCP, and custom tool rendering",
   "keywords": [
     "pi-package",
@@ -37,7 +37,7 @@
     "CHANGELOG.md"
   ],
   "scripts": {
-    "test": "node --experimental-strip-types --test scripts/*.test.ts",
+    "test": "node --experimental-strip-types --test scripts/*.test.ts scripts/*.test.mts",
     "test:fullscreen": "node scripts/fullscreen-mouse.test.mjs",
     "typecheck": "tsc --noEmit --target ES2022 --module NodeNext --moduleResolution NodeNext --skipLibCheck extensions/index.ts extensions/spinner.ts",
     "pack:dry-run": "npm pack --dry-run",