@selesai/code 0.13.12 → 0.13.14

package/dist/extensions/pi-zentui/test/extension-compliance.test.ts

@@ -12,6 +12,7 @@ import { tmpdir } from "node:os";
 import { dirname, join } from "node:path";
 import type { Theme } from "@earendil-works/pi-coding-agent";
 import {
+	AssistantMessageComponent,
 	initTheme,
 	ModelSelectorComponent,
 	SettingsSelectorComponent,
@@ -113,6 +114,8 @@ const settingsCommandDefaults: SettingsCommandDeps = {
 	setAccentRail() {},
 	setMinimalist() {},
 	setUserMessagesComponent() {},
+	thinkingStepsCapability: { available: false },
+	setThinkingStepsComponent: () => ({ applied: true }),
 	setWorkingLineComponent: () => ({ applied: true }),
 	setSelectorBordersComponent() {},
 	setFooterComponent() {},
@@ -180,6 +183,7 @@ function canonicalizeTestConfig(config: PolishedTuiConfig): PolishedTuiConfig {
 					: messages.colorSource,
 				styles: { ...messages.styles },
 			},
+			thinkingSteps: { ...config.components.thinkingSteps },
 			workingLine: {
 				...config.components.workingLine,
 				messages: {
@@ -559,7 +563,7 @@ function loadExtension(options: { thinkingLevel?: string; commands?: Map<string,
 		})}\n`,
 	);
 	const handlers = new Map<string, Handler[]>();
-	zentui({
+	const api = {
 		on(eventName: string, handler: Handler) {
 			handlers.set(eventName, [...(handlers.get(eventName) ?? []), handler]);
 		},
@@ -569,7 +573,8 @@ function loadExtension(options: { thinkingLevel?: string; commands?: Map<string,
 		getThinkingLevel() {
 			return options.thinkingLevel ?? "off";
 		},
-	} as never);
+	} as Record<string, unknown>;
+	zentui(api as never);
 	return handlers;
 }
 
@@ -757,6 +762,48 @@ describe("fixed-editor retirement contract", () => {
 });
 
 describe("Pi docs compliance", () => {
+	it("does not register a public Thinking Markdown transformer or command", async () => {
+		writeFileSync(
+			join(isolatedAgentDir.path, "zentui.json"),
+			JSON.stringify({ components: { thinkingSteps: { enabled: false, mode: "tree" } } }),
+		);
+		const commands = new Map<string, unknown>();
+		const handlers = loadExtension({ commands });
+		const indexSource = readFileSync(
+			join(import.meta.dirname, "../extensions/zentui/index.ts"),
+			"utf8",
+		);
+		expect(indexSource).not.toContain("registerMarkdown" + "Transformer");
+		const ctx = makeContext();
+		await emit(handlers, "session_start", ctx);
+		await emit(handlers, "session_shutdown", ctx);
+		expect(commands.has("thinking-steps")).toBe(false);
+	});
+
+	it("reloads external Thinking configuration synchronously for each session startup", async () => {
+		const configPath = join(isolatedAgentDir.path, "zentui.json");
+		writeFileSync(
+			configPath,
+			JSON.stringify({ components: { thinkingSteps: { enabled: false, mode: "tree" } } }),
+		);
+		const nativeUpdate = AssistantMessageComponent.prototype.updateContent;
+		const handlers = loadExtension();
+		const ctx = makeContext();
+
+		await emit(handlers, "session_start", ctx);
+		expect(AssistantMessageComponent.prototype.updateContent).toBe(nativeUpdate);
+		await emit(handlers, "session_shutdown", ctx);
+
+		writeFileSync(
+			configPath,
+			JSON.stringify({ components: { thinkingSteps: { enabled: true, mode: "rail" } } }),
+		);
+		await emit(handlers, "session_start", ctx);
+		expect(AssistantMessageComponent.prototype.updateContent).not.toBe(nativeUpdate);
+		await emit(handlers, "session_shutdown", ctx);
+		expect(AssistantMessageComponent.prototype.updateContent).toBe(nativeUpdate);
+	});
+
 	it("derives lazy data requirements from only the active wide and compact candidates", () => {
 		const starship = defaultConfig.components.footer.styles.starship;
 		const customWide = {
@@ -5733,7 +5780,7 @@ describe("Pi docs compliance", () => {
 		const themeLines = await renderSettings(defaultConfig);
 		expect(themeLines[0]).toContain("[borderMuted]────");
 		expect(themeLines.join("\n")).toContain("Appearance");
-		expect(themeLines.join("\n")).toContain("(1/8)");
+		expect(themeLines.join("\n")).toContain("(1/9)");
 		expect(themeLines.join("\n")).toContain("Tab/Shift+Tab to switch sections");
 		expect(themeLines.at(-1)).toContain("[borderMuted]────");
 		expect(themeLines.every((line) => visibleWidth(stripTestTags(line)) <= settingsWidth)).toBe(
@@ -5851,7 +5898,7 @@ describe("Pi docs compliance", () => {
 					const component = factory({ requestRender() {} }, makeTheme(), {}, () => {}) as {
 						handleInput?: (data: string) => void;
 					};
-					for (let index = 0; index < 4; index += 1) component.handleInput?.("\t");
+					for (let index = 0; index < 5; index += 1) component.handleInput?.("\t");
 					for (let index = 0; index < 3; index += 1) component.handleInput?.("\x1b[B");
 					component.handleInput?.(" ");
 					component.handleInput?.("\x1b[B");
@@ -5923,7 +5970,7 @@ describe("Pi docs compliance", () => {
 						{},
 						() => {},
 					) as { handleInput?: (data: string) => void };
-					for (let index = 0; index < 4; index += 1) component.handleInput?.("\t");
+					for (let index = 0; index < 5; index += 1) component.handleInput?.("\t");
 					for (let index = 0; index < 6; index += 1) component.handleInput?.("\x1b[B");
 					component.handleInput?.(" ");
 					component.handleInput?.(" ");
@@ -5941,7 +5988,7 @@ describe("Pi docs compliance", () => {
 			"Separator: pipe",
 		]);
 		expect(dependencyRenderRequests).toBe(4);
-		expect(tuiRenderRequests).toBe(8);
+		expect(tuiRenderRequests).toBe(9);
 	});
 
 	it("cycles branch length presets and returns custom JSON values to full", async () => {
@@ -5992,7 +6039,7 @@ describe("Pi docs compliance", () => {
 						const component = factory({ requestRender() {} }, makeTheme(), {}, () => {}) as {
 							handleInput?: (data: string) => void;
 						};
-						for (let index = 0; index < 6; index += 1) component.handleInput?.("\t");
+						for (let index = 0; index < 7; index += 1) component.handleInput?.("\t");
 						component.handleInput?.("\x1b[B");
 						for (let index = 0; index < presses; index += 1) component.handleInput?.(" ");
 					},
@@ -6149,6 +6196,7 @@ describe("Pi docs compliance", () => {
 			| "Appearance"
 			| "Editor"
 			| "User messages"
+			| "Thinking"
 			| "Working line"
 			| "Footer"
 			| "Segments"
@@ -6159,6 +6207,7 @@ describe("Pi docs compliance", () => {
 			"Appearance",
 			"Editor",
 			"User messages",
+			"Thinking",
 			"Working line",
 			"Footer",
 			"Segments",
@@ -6417,7 +6466,7 @@ describe("Pi docs compliance", () => {
 
 		expect(placements).toEqual([{ key: "alpha", placement: "off" }]);
 		expect(dependencyRenderRequests).toBe(1);
-		expect(tuiRenderRequests).toBe(8);
+		expect(tuiRenderRequests).toBe(9);
 	});
 
 	it("does not show inactive saved placements in the extension segments tab", async () => {
@@ -7172,7 +7221,7 @@ describe("three-state Footer lifecycle", () => {
 			const settings = factory({ requestRender() {} }, makeTheme(), {}, () => {}) as {
 				handleInput(data: string): void;
 			};
-			for (let index = 0; index < 4; index++) settings.handleInput("\t");
+			for (let index = 0; index < 5; index++) settings.handleInput("\t");
 			settings.handleInput(" ");
 		};
 		const ctx = makeContext({ ui: harness.ui });

package/dist/extensions/pi-zentui/test/package-contents.mjs

@@ -0,0 +1,33 @@
+import assert from "node:assert/strict";
+import { execFileSync } from "node:child_process";
+import { readFileSync } from "node:fs";
+
+const npmCli = process.env.npm_execpath;
+const reportJson = execFileSync(
+	npmCli ? process.execPath : "npm",
+	[...(npmCli ? [npmCli] : []), "pack", "--dry-run", "--json"],
+	{ encoding: "utf8" },
+);
+const report = JSON.parse(reportJson);
+const packages = Array.isArray(report) ? report : Object.values(report);
+assert.equal(packages.length, 1, "npm pack must report exactly one package");
+
+const files = packages[0]?.files;
+assert.ok(Array.isArray(files), "npm pack must report its package file list");
+for (const required of [
+	"extensions/zentui/thinking-steps.ts",
+	"extensions/zentui/thinking-experimental.ts",
+	"extensions/zentui/thinking-status.ts",
+]) {
+	assert.ok(
+		files.some(({ path }) => path === required),
+		`npm pack must include ${required}`,
+	);
+}
+
+const renderer = readFileSync("extensions/zentui/thinking-experimental.ts", "utf8");
+assert.match(renderer, /Copyright \(c\) 2026 Zach Yuen/);
+assert.match(renderer, /Copyright \(c\) 2026 Marc Mironescu \/ FluxGear/);
+assert.equal((renderer.match(/Permission is hereby granted, free of charge/g) ?? []).length, 2);
+
+console.log(`Package and license assertions passed (${files.length} files total)`);

package/dist/extensions/pi-zentui/test/prototype-patch-registry.test.ts

@@ -0,0 +1,131 @@
+import { describe, expect, it, vi } from "vitest";
+import {
+	installPrototypePatch,
+	isPrototypePatchCurrent,
+	ZENTUI_PROTOTYPE_PATCH_REGISTRY,
+} from "../extensions/zentui/prototype-patch-registry";
+
+describe("prototype patch registry updateContent ownership", () => {
+	it("forwards every argument, reuses its wrapper, and restores an inherited predecessor", () => {
+		const predecessor = vi.fn(function (this: { value: number }, ...args: unknown[]) {
+			return [this.value, ...args];
+		});
+		const parent = { updateContent: predecessor };
+		const target = Object.create(parent) as object & {
+			updateContent: (...args: unknown[]) => unknown;
+		};
+		const behavior = vi.fn(({ predecessor: previous, receiver, args }) =>
+			Reflect.apply(previous, receiver, args),
+		);
+		const cleanupFirst = installPrototypePatch(
+			target,
+			"updateContent",
+			"thinking-experimental-update-content",
+			behavior,
+		);
+		const wrapper = target.updateContent;
+		expect(target.updateContent.call({ value: 7 }, "message", true, "future-argument")).toEqual([
+			7,
+			"message",
+			true,
+			"future-argument",
+		]);
+		expect(behavior.mock.calls[0]?.[0].args).toEqual(["message", true, "future-argument"]);
+		const cleanupSecond = installPrototypePatch(
+			target,
+			"updateContent",
+			"thinking-experimental-update-content",
+			behavior,
+		);
+		expect(target.updateContent).toBe(wrapper);
+		expect(cleanupFirst.token).not.toBe(cleanupSecond.token);
+		expect(
+			isPrototypePatchCurrent(
+				target,
+				"updateContent",
+				"thinking-experimental-update-content",
+				cleanupFirst.token,
+			),
+		).toBe(false);
+		expect(
+			isPrototypePatchCurrent(
+				target,
+				"updateContent",
+				"thinking-experimental-update-content",
+				cleanupSecond.token,
+			),
+		).toBe(true);
+		cleanupFirst();
+		expect(target.updateContent).toBe(wrapper);
+		expect(target.updateContent.call({ value: 8 }, "newest")).toEqual([8, "newest"]);
+		cleanupSecond();
+		expect(Object.hasOwn(target, "updateContent")).toBe(false);
+		expect(target.updateContent).toBe(predecessor);
+		expect(
+			(target as unknown as Record<PropertyKey, unknown>)[ZENTUI_PROTOTYPE_PATCH_REGISTRY],
+		).toBeUndefined();
+	});
+
+	it("preserves an exact own descriptor and never restores over a foreign successor", () => {
+		const target = {} as { updateContent: (...args: unknown[]) => unknown };
+		const predecessor = function predecessor() {};
+		const descriptor = {
+			value: predecessor,
+			writable: false,
+			enumerable: false,
+			configurable: true,
+		};
+		Object.defineProperty(target, "updateContent", descriptor);
+		const cleanup = installPrototypePatch(
+			target,
+			"updateContent",
+			"thinking-experimental-update-content",
+			({ predecessor: previous, receiver, args }) => Reflect.apply(previous, receiver, args),
+		);
+		expect(Object.getOwnPropertyDescriptor(target, "updateContent")).toMatchObject({
+			writable: false,
+			enumerable: false,
+			configurable: true,
+		});
+		const foreign = function foreign() {};
+		Object.defineProperty(target, "updateContent", { ...descriptor, value: foreign });
+		expect(
+			isPrototypePatchCurrent(target, "updateContent", "thinking-experimental-update-content"),
+		).toBe(false);
+		cleanup();
+		expect(target.updateContent).toBe(foreign);
+	});
+
+	it("rejects missing, noncallable, and nonconfigurable targets without leaving registry state", () => {
+		for (const target of [{}, { updateContent: 1 }]) {
+			expect(() =>
+				installPrototypePatch(
+					target,
+					"updateContent",
+					"thinking-experimental-update-content",
+					() => undefined,
+				),
+			).toThrow(/predecessor is not a function/);
+			expect(
+				(target as Record<PropertyKey, unknown>)[ZENTUI_PROTOTYPE_PATCH_REGISTRY],
+			).toBeUndefined();
+		}
+		const fixed = {} as { updateContent: () => void };
+		Object.defineProperty(fixed, "updateContent", {
+			value: () => {},
+			writable: false,
+			configurable: false,
+		});
+		expect(() =>
+			installPrototypePatch(
+				fixed,
+				"updateContent",
+				"thinking-experimental-update-content",
+				() => undefined,
+			),
+		).toThrow();
+		expect(
+			(fixed as unknown as Record<PropertyKey, unknown>)[ZENTUI_PROTOTYPE_PATCH_REGISTRY],
+		).toBeUndefined();
+	});
+});

package/dist/extensions/pi-zentui/test/responsive-dependencies.test.ts

@@ -693,7 +693,7 @@ describe("responsive footer dependency reconciliation", () => {
 			const component = factory({ requestRender() {} }, makeTheme(), {}, () => {}) as {
 				handleInput?: (data: string) => void;
 			};
-			for (let index = 0; index < 4; index++) component.handleInput?.("\t");
+			for (let index = 0; index < 5; index++) component.handleInput?.("\t");
 			for (let index = 0; index < 3; index++) component.handleInput?.("\x1b[B");
 			component.handleInput?.(" ");
 		});
@@ -737,7 +737,7 @@ describe("responsive footer dependency reconciliation", () => {
 				render(width: number): string[];
 				handleInput(data: string): void;
 			};
-			for (let index = 0; index < 6; index++) component.handleInput("\t");
+			for (let index = 0; index < 7; index++) component.handleInput("\t");
 			const label = target === "showTag" ? "Show exact-match tag" : "Ignore submodules";
 			for (let attempts = 0; attempts < 12; attempts++) {
 				if (component.render(140).some((line) => line.includes(`> ${label}`))) break;
@@ -768,7 +768,7 @@ describe("responsive footer dependency reconciliation", () => {
 			const component = factory({ requestRender() {} }, makeTheme(), {}, () => {}) as {
 				handleInput?: (data: string) => void;
 			};
-			for (let index = 0; index < 5; index++) component.handleInput?.("\t");
+			for (let index = 0; index < 6; index++) component.handleInput?.("\t");
 			for (let index = 0; index < 11; index++) component.handleInput?.("\x1b[B");
 			component.handleInput?.(" ");
 		});
@@ -788,7 +788,7 @@ describe("responsive footer dependency reconciliation", () => {
 			const component = factory({ requestRender() {} }, makeTheme(), {}, () => {}) as {
 				handleInput?: (data: string) => void;
 			};
-			for (let index = 0; index < 4; index++) component.handleInput?.("\t");
+			for (let index = 0; index < 5; index++) component.handleInput?.("\t");
 			for (let index = 0; index < 4; index++) component.handleInput?.("\x1b[B");
 			component.handleInput?.(" ");
 		});