@xzzpig/pi-starline 0.4.1 → 0.5.0

package/README.md

@@ -176,7 +176,7 @@ Settings live in `~/.pi/agent/starline.json`. The file is optional — anything
 {
 	"footerStyle": "pill",
 	"colors": { "gitBranch": "bold purple" },
-	"mouse": { "clickToExpandTools": false }
+	"mouse": { "clickToToggleExpandable": false }
 }
 ```
 

package/extensions/starline/config.ts

@@ -109,8 +109,8 @@ export type MouseConfig = {
 	wheelRouting: boolean;
 	/** Show a "copied to clipboard" notice for a copy Starline itself performs. */
 	copyNotice: boolean;
-	/** Clicking a tool box's `ctrl+o to expand` hint expands that one box. */
-	clickToExpandTools: boolean;
+	/** Click any collapsible box (a tool box, a custom message, an expandable text) to expand or collapse it. */
+	clickToToggleExpandable: boolean;
 	/** Copy transcript selections without the painted rails, rules and frames. */
 	transcriptCleanCopy: boolean;
 };
@@ -119,7 +119,10 @@ const FIXED_EDITOR_KEY_MAP: Record<string, keyof MouseConfig> = {
 	enabled: "enabled",
 	mouseScroll: "wheelRouting",
 	copyNotice: "copyNotice",
-	clickToExpandTools: "clickToExpandTools",
+	// The 0.2.x-era name for click-to-toggle; Pi 0.86.0 owns the click itself
+	// for tool boxes, but the fork keeps the general collapsible-box toggle,
+	// so the old value is carried to the renamed key instead of being dropped.
+	clickToExpandTools: "clickToToggleExpandable",
 };
 
 /**
@@ -418,7 +421,7 @@ export const defaultConfig: PolishedTuiConfig = {
 		enabled: true,
 		wheelRouting: true,
 		copyNotice: true,
-		clickToExpandTools: true,
+		clickToToggleExpandable: true,
 		transcriptCleanCopy: true,
 	},
 };
@@ -838,10 +841,14 @@ function normalizeMouseConfig(record: Record<string, unknown>): MouseConfig {
 				: defaultConfig.mouse.wheelRouting,
 		copyNotice:
 			typeof record.copyNotice === "boolean" ? record.copyNotice : defaultConfig.mouse.copyNotice,
-		clickToExpandTools:
-			typeof record.clickToExpandTools === "boolean"
-				? record.clickToExpandTools
-				: defaultConfig.mouse.clickToExpandTools,
+		clickToToggleExpandable:
+			// The new key wins outright; a config carrying the pre-rename
+			// mouse.clickToExpandTools value is honoured as a fallback.
+			typeof record.clickToToggleExpandable === "boolean"
+				? record.clickToToggleExpandable
+				: typeof record.clickToExpandTools === "boolean"
+					? record.clickToExpandTools
+					: defaultConfig.mouse.clickToToggleExpandable,
 		transcriptCleanCopy:
 			typeof record.transcriptCleanCopy === "boolean"
 				? record.transcriptCleanCopy
@@ -1306,8 +1313,8 @@ export function saveMousePatch(patch: Partial<MouseConfig>, path = configPath):
 			...(patch.enabled !== undefined ? { enabled: patch.enabled } : {}),
 			...(patch.wheelRouting !== undefined ? { wheelRouting: patch.wheelRouting } : {}),
 			...(patch.copyNotice !== undefined ? { copyNotice: patch.copyNotice } : {}),
-			...(patch.clickToExpandTools !== undefined
-				? { clickToExpandTools: patch.clickToExpandTools }
+			...(patch.clickToToggleExpandable !== undefined
+				? { clickToToggleExpandable: patch.clickToToggleExpandable }
 				: {}),
 			...(patch.transcriptCleanCopy !== undefined
 				? { transcriptCleanCopy: patch.transcriptCleanCopy }

package/extensions/starline/mouse/api-consumer.ts

@@ -22,7 +22,7 @@ import type { MouseEventsApi } from "@xzzpig/pi-mouse-events/api";
  * MUST match `MOUSE_EVENTS_API_KEY` in `@xzzpig/pi-mouse-events/api` — the
  * contract test above is what keeps that true.
  */
-const MOUSE_EVENTS_API_KEY = Symbol.for("pi-mouse-events.api.v1");
+const MOUSE_EVENTS_API_KEY = Symbol.for("pi-mouse-events.api.v2");
 
 export function getMouseEventsApi(): MouseEventsApi | undefined {
 	const candidate = (globalThis as Record<symbol, unknown>)[MOUSE_EVENTS_API_KEY] as
@@ -31,7 +31,7 @@ export function getMouseEventsApi(): MouseEventsApi | undefined {
 	// The symbol key pins the contract name, not its shape: reject an object
 	// from a different contract major so callers never dereference a
 	// mismatched surface (the key is only bumped together with the version).
-	if (!candidate || candidate.version !== 1) {
+	if (!candidate || candidate.version !== 2) {
 		return undefined;
 	}
 	return candidate;

package/extensions/starline/mouse/capabilities.ts

@@ -29,7 +29,7 @@ export type MouseCapability = MouseReceiverCapabilities | "copySlot";
 
 export type MouseFeature =
 	| "selectionHint"
-	| "clickToExpandTools"
+	| "clickToToggleExpandable"
 	| "editorWheelScroll"
 	| "editorClickToCaret"
 	| "editorBufferCopy"
@@ -77,7 +77,7 @@ const REQUIREMENTS: Record<MouseFeature, readonly MouseCapability[]> = {
 	// whatever tool box happens to sit behind it — and `requestRender`, because
 	// consuming the press means Pi never reaches its own repaint for that
 	// event.
-	clickToExpandTools: ["hasOverlay", "requestRender"],
+	clickToToggleExpandable: ["hasOverlay", "requestRender"],
 	// Same shape as click-to-expand: the notch arrives through the handler
 	// slot, `hasOverlay` keeps a notch aimed at a dialog from scrolling the
 	// draft hidden behind it, and consuming the notch means `requestRender` is
@@ -113,7 +113,7 @@ const REQUIREMENTS: Record<MouseFeature, readonly MouseCapability[]> = {
 	],
 };
 
-function isCallable(receiver: object, name: string): boolean {
+function isCallable<T extends object>(receiver: T, name: string): boolean {
 	try {
 		return typeof (receiver as Record<string, unknown>)[name] === "function";
 	} catch {
@@ -122,7 +122,7 @@ function isCallable(receiver: object, name: string): boolean {
 	}
 }
 
-export function probeCapabilities(receiver: object): ReadonlySet<MouseCapability> {
+export function probeCapabilities<T extends object>(receiver: T): ReadonlySet<MouseCapability> {
 	const found = new Set<MouseCapability>();
 	for (const capability of CAPABILITIES) {
 		if (capability === "copySlot") continue;

package/extensions/starline/mouse/index.ts

@@ -21,7 +21,7 @@
  *   located and how the live editor is reached; the scroll itself is
  *   `editor-scroll.ts`.
  *
- * `clickToExpandTools` → `addMouseHandler`
+ * `clickToToggleExpandable` → `addMouseHandler`
  * - A left-button press anywhere on an expandable component records the
  *   candidate; when the button is released on the same cell with no motion in
  *   between — a plain click — that one component is toggled and the release
@@ -89,8 +89,9 @@ import { activeEditor, pointerOverEditor } from "./editor-mouse";
 import { scrollEditorBy } from "./editor-scroll";
 import { editorVisualRowCount } from "./editor-text-cursor";
 import { type BoxLike, scrollContentLinesFor } from "./hit-test";
+import { keyTextFor } from "./key-text";
 import { externalEditorName, selectionHintText } from "./selection-state";
-import { type ExpandTarget, expandKeyText, expandTargetAt, keyTextFor } from "./tool-box";
+import { type ExpandTarget, expandKeyText, expandTargetAt } from "./tool-box";
 import { cleanTranscriptRows } from "./transcript-copy";
 
 /** Pi's interrupt chord — refused by the range-delete branch, never consumed. */
@@ -664,7 +665,7 @@ function expandClickTarget(
 	deps: InstallMouseDeps,
 ): ExpandTarget | undefined {
 	try {
-		if (!deps.getConfig().mouse.clickToExpandTools) return undefined;
+		if (!deps.getConfig().mouse.clickToToggleExpandable) return undefined;
 		// Pi resolves no scroll view while an overlay is up, so neither does this
 		// — a click on a dialog must not reach the transcript behind it.
 		if (receiver.hasOverlay?.()) return undefined;
@@ -806,7 +807,7 @@ export function installMouseFeaturesOn(
 		api.addMouseHandler(
 			({ event, tui }) => {
 				const receiver = bindReceiver(tui);
-				if (!receiver || !featureOn("clickToExpandTools")) {
+				if (!receiver || !featureOn("clickToToggleExpandable")) {
 					expandPress = undefined;
 					return undefined;
 				}

package/extensions/starline/mouse/key-text.ts

@@ -0,0 +1,64 @@
+/**
+ * The keys spelled the way Pi's on-screen hints spell them.
+ *
+ * The primary source is `pi-coding-agent`'s own exported `keyText` — literally
+ * the function `keyHint` calls to build the hint text the UI shows, so there
+ * is no second formatting rule to drift (it is `keyText` that turns `alt`
+ * into `option` on macOS and joins alternatives with "/"), and no second
+ * keybinding registry to disagree.
+ *
+ * That last point is not theoretical. `getKeybindings()` is a *singleton* per
+ * copy of pi-tui, and a copy is per `node_modules` tree: in this repo
+ * `pi-coding-agent` nests its own pi-tui, so the registry Starline's direct
+ * import reaches is not the registry the hints are rendered from. Production
+ * resolves both to Pi's own copy, but reading the value through the same
+ * package that renders it makes the question moot instead of assumed.
+ *
+ * pi-tui's registry is kept as a fallback for the case where `keyText` yields
+ * nothing. Both routes return "" when the binding is unbound, and callers
+ * read "" as "show no key" rather than as a wildcard.
+ */
+
+import { keyText } from "@earendil-works/pi-coding-agent";
+import { getKeybindings } from "@earendil-works/pi-tui";
+
+/**
+ * `keyText(keybinding)` with the registry fallback, shared by every feature
+ * that quotes a key on screen. See the module header for why the primary
+ * route is pi-coding-agent's own function rather than this repo's registry.
+ */
+export function keyTextFor(keybinding: string): string {
+	try {
+		const rendered = keyText(keybinding as never);
+		if (typeof rendered === "string" && rendered.length > 0) return rendered;
+	} catch {
+		// A Pi build that has moved this function is one where quoting the key
+		// simply shows nothing; it is never a reason to break a mouse feature.
+	}
+	return fallbackKeyText(keybinding);
+}
+
+/**
+ * `keyText` re-derived from pi-tui's registry: keys joined with "/", `alt`
+ * shown as `option` on macOS, matching `formatKeyText` in
+ * `keybinding-hints.js`.
+ */
+function fallbackKeyText(keybinding: string): string {
+	try {
+		const bound: unknown = getKeybindings().getKeys(keybinding as never);
+		if (!Array.isArray(bound) || bound.length === 0) return "";
+		const keys = bound.filter((key): key is string => typeof key === "string");
+		return keys
+			.map((key: string) =>
+				key
+					.split("+")
+					.map((part) =>
+						process.platform === "darwin" && part.toLowerCase() === "alt" ? "option" : part,
+					)
+					.join("+"),
+			)
+			.join("/");
+	} catch {
+		return "";
+	}
+}

package/extensions/starline/mouse/tool-box.ts

@@ -45,12 +45,11 @@
  * skill/branch/compaction summaries), so an expanded box used to have no
  * clickable way back at all without `pi-toolbox`'s collapse anchor.
  *
- * ## A component that handles the mouse keeps its clicks
- *
- * If any component on the clicked row's path implements `onMouse`, resolution
- * declines. The layout folds the whole transcript into one box, so
- * `pi-mouse-events` dispatch can never deliver an event to such a component —
- * bowing out here is the only way its `onMouse` can mean what it says.
+ * Ownership is decided by resolution alone: a click is taken over iff it
+ * resolves to an expandable component (one with `setExpanded`). A component
+ * that handles the mouse itself — Pi's own tool boxes wrapped in a core
+ * `MouseRegion` included — is simply not expandable by this predicate, so it
+ * falls through to Pi's native handling instead of being fought over.
  *
  * ## Why resolution goes through the component tree
  *
@@ -86,10 +85,10 @@
  * same click a row above or below behaves normally.
  */
 
-import { keyText } from "@earendil-works/pi-coding-agent";
-import { getKeybindings, stripTerminalSequences } from "@earendil-works/pi-tui";
+import { stripTerminalSequences } from "@earendil-works/pi-tui";
 import { createComponentTree, isComponentLike, isExpandableComponent } from "./component-tree";
 import { type BoxLike, boxesAt, scrollContentLinesFor, scrollContentOrigin } from "./hit-test";
+import { keyTextFor } from "./key-text";
 
 /** The slice of a Pi message component this module calls. */
 export type ExpandableComponent = { setExpanded(expanded: boolean): void };
@@ -165,15 +164,6 @@ export function expandedStateOf(component: object): boolean | undefined {
 	return undefined;
 }
 
-/** Whether the component declares its own mouse handling (`onMouse`). */
-function handlesMouse(component: unknown): boolean {
-	return (
-		typeof component === "object" &&
-		component !== null &&
-		typeof (component as { onMouse?: unknown }).onMouse === "function"
-	);
-}
-
 /**
  * The keys spelled the way the hint on screen spells them.
  *
@@ -200,47 +190,6 @@ export function expandKeyText(): string {
 	return keyTextFor(KEYBINDING);
 }
 
-/**
- * `keyText(keybinding)` with the registry fallback, shared by every feature
- * that quotes a key on screen. See `expandKeyText` for why the primary route
- * is pi-coding-agent's own function rather than this repo's registry.
- */
-export function keyTextFor(keybinding: string): string {
-	try {
-		const rendered = keyText(keybinding as never);
-		if (typeof rendered === "string" && rendered.length > 0) return rendered;
-	} catch {
-		// A Pi build that has moved this function is one where quoting the key
-		// simply shows nothing; it is never a reason to break a mouse feature.
-	}
-	return fallbackKeyText(keybinding);
-}
-
-/**
- * `keyText` re-derived from pi-tui's registry: keys joined with "/", `alt`
- * shown as `option` on macOS, matching `formatKeyText` in
- * `keybinding-hints.js`.
- */
-function fallbackKeyText(keybinding: string): string {
-	try {
-		const bound: unknown = getKeybindings().getKeys(keybinding as never);
-		if (!Array.isArray(bound) || bound.length === 0) return "";
-		const keys = bound.filter((key): key is string => typeof key === "string");
-		return keys
-			.map((key: string) =>
-				key
-					.split("+")
-					.map((part) =>
-						process.platform === "darwin" && part.toLowerCase() === "alt" ? "option" : part,
-					)
-					.join("+"),
-			)
-			.join("/");
-	} catch {
-		return "";
-	}
-}
-
 /** The innermost scroll view whose box contains the screen cell. */
 function scrollViewAt(root: BoxLike, x: number, y: number): unknown {
 	let found: unknown;
@@ -291,14 +240,6 @@ export function expandTargetAt(
 	const tree = createComponentTree(origin.component, origin.rect.width, lines);
 	const path = tree.pathAt(row);
 
-	// A component anywhere on the path that handles the mouse itself owns this
-	// click. The layout folds the transcript into one box, so `pi-mouse-events`
-	// dispatch can never deliver an event to it — the only way its `onMouse`
-	// can mean what it says is for this module to bow out.
-	for (const span of path) {
-		if (handlesMouse(span.component)) return undefined;
-	}
-
 	for (let index = path.length - 1; index >= 0; index--) {
 		const span = path[index];
 		if (!isExpandableComponent(span.component)) continue;

package/extensions/starline/settings-command.ts

@@ -469,11 +469,12 @@ function buildItems(
 				values: featureStateValues,
 			});
 			items.push({
-				id: "mouseClickToExpandTools",
-				label: "Click to expand tools",
+				id: "mouseClickToToggleExpandable",
+				label: "Click to toggle expandable",
 				description:
-					"Click a tool box's border or its expand hint to expand that one box. Needs wheel routing.",
-				currentValue: featureValue(config.mouse.clickToExpandTools),
+					"Click any collapsible component (tool boxes, bash boxes, skill/summary blocks, " +
+					"custom entries/messages) to expand or collapse just that one.",
+				currentValue: featureValue(config.mouse.clickToToggleExpandable),
 				values: featureStateValues,
 			});
 		}
@@ -831,11 +832,11 @@ export function registerStarlineSettingsCommand(pi: ExtensionAPI, deps: Settings
 									tui.requestRender();
 									return;
 								}
-								if (id === "mouseClickToExpandTools" && isFeatureState(newValue)) {
-									deps.setMouseConfig({ clickToExpandTools: newValue === "enabled" }, ctx);
+								if (id === "mouseClickToToggleExpandable" && isFeatureState(newValue)) {
+									deps.setMouseConfig({ clickToToggleExpandable: newValue === "enabled" }, ctx);
 									settingsList.updateValue(id, newValue);
 									deps.requestRender();
-									ctx.ui.notify(`Click to expand tools: ${newValue}`, "info");
+									ctx.ui.notify(`Click to toggle expandable: ${newValue}`, "info");
 									tui.requestRender();
 									return;
 								}

package/extensions/starline/user-message.ts

@@ -30,6 +30,7 @@ type UserMessageRenderCache = {
 	width?: number;
 	theme?: Theme;
 	configKey?: string;
+	transformers?: readonly unknown[];
 	renderedLines?: string[];
 };
 
@@ -69,6 +70,49 @@ function getCachedMarkdownText(instance: object): string | undefined {
 	return text;
 }
 
+type MarkdownTransformerFn = (
+	markdown: string,
+	context: {
+		messageType: "user" | "assistant" | "assistant-thinking";
+		isStreaming: boolean;
+		availableWidth: number;
+	},
+) => unknown;
+
+/**
+ * pi 官方管道:UserMessageComponent 把扩展注册的 markdownTransformers 通过
+ * Markdown 的 transform 选项跑一遍(createMarkdownTransform)。我们重写了 render,
+ * 必须自己把这条管道接上,否则任何 registerMarkdownTransformer 的扩展
+ * (如 pi-ide-context 的选区引用行)在 starline 样式下都会被旁路。
+ */
+function getMarkdownTransformersRef(instance: object): readonly unknown[] | undefined {
+	if (!isRecord(instance)) return undefined;
+	const t = instance.markdownTransformers;
+	return Array.isArray(t) ? t : undefined;
+}
+
+function applyMarkdownTransformers(
+	text: string,
+	availableWidth: number,
+	transformers: readonly unknown[],
+): string {
+	let out = text;
+	for (const t of transformers) {
+		if (typeof t !== "function") continue;
+		try {
+			const r = (t as MarkdownTransformerFn)(out, {
+				messageType: "user",
+				isStreaming: false,
+				availableWidth,
+			});
+			if (typeof r === "string") out = r;
+		} catch {
+			// 与 pi 的 applyMarkdownTransformers 一致:单个 transformer 抛错不影响其他
+		}
+	}
+	return out;
+}
+
 function getUserMessageConfigKey(config: PolishedTuiConfig): string {
 	return [
 		config.features.copyFriendly ? "copy" : "chrome",
@@ -161,12 +205,14 @@ function renderStarlineUserMessage(
 	if (width <= 0) return [""];
 
 	const configKey = getUserMessageConfigKey(config);
+	const transformersRef = getMarkdownTransformersRef(instance);
 	const cached = userMessageRenderCache.get(instance);
 	if (
 		cached?.hasMarkdownText &&
 		cached.width === width &&
 		cached.theme === theme &&
 		cached.configKey === configKey &&
+		cached.transformers === transformersRef &&
 		cached.renderedLines
 	) {
 		return cached.renderedLines;
@@ -174,17 +220,30 @@ function renderStarlineUserMessage(
 
 	const railWidth = visibleWidth(renderPromptBoxRail(theme, config));
 	const contentWidth = Math.max(1, width - railWidth);
-	const renderer = new Markdown(text, 0, 0, makeMarkdownTheme(theme), {
-		color: (content) =>
-			theme && config.colors.userMessageText
-				? renderStyleForSource(
-						theme,
-						config.colorSources.userMessages,
-						config.colors.userMessageText,
-						content,
-					)
-				: themeFg(theme, "userMessageText", content),
-	});
+	const renderer = new Markdown(
+		text,
+		0,
+		0,
+		makeMarkdownTheme(theme),
+		{
+			color: (content) =>
+				theme && config.colors.userMessageText
+					? renderStyleForSource(
+							theme,
+							config.colorSources.userMessages,
+							config.colors.userMessageText,
+							content,
+						)
+					: themeFg(theme, "userMessageText", content),
+		},
+		// 接上官方 transformer 管道;Markdown 每次 render 都会按当前宽度重跑
+		{
+			transform: transformersRef?.length
+				? (markdown, availableWidth) =>
+						applyMarkdownTransformers(markdown, availableWidth, transformersRef)
+				: undefined,
+		},
+	);
 	const body = renderer.render(contentWidth);
 	const contentLines = body.length > 0 ? body : [""];
 	const border = theme
@@ -212,6 +271,7 @@ function renderStarlineUserMessage(
 		width,
 		theme,
 		configKey,
+		transformers: transformersRef,
 		renderedLines: lines,
 	});
 	return lines;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xzzpig/pi-starline",
-  "version": "0.4.1",
+  "version": "0.5.0",
   "description": "A Starship-inspired statusline and Opencode-style TUI for Pi.",
   "type": "module",
   "license": "MIT",
@@ -23,10 +23,10 @@
     "extensions"
   ],
   "peerDependencies": {
-    "@earendil-works/pi-ai": ">=0.84.4",
-    "@earendil-works/pi-coding-agent": ">=0.84.4",
-    "@earendil-works/pi-tui": ">=0.84.4",
-    "@xzzpig/pi-mouse-events": ">=0.1.0 <2"
+    "@earendil-works/pi-ai": ">=0.86.0 <0.88",
+    "@earendil-works/pi-coding-agent": ">=0.86.0 <0.88",
+    "@earendil-works/pi-tui": ">=0.86.0 <0.88",
+    "@xzzpig/pi-mouse-events": ">=0.2.0 <3"
   },
   "pi": {
     "extensions": [
@@ -39,13 +39,13 @@
   },
   "devDependencies": {
     "@biomejs/biome": "^2.5.0",
-    "@earendil-works/pi-ai": "^0.84.4",
-    "@earendil-works/pi-coding-agent": "^0.84.4",
-    "@earendil-works/pi-tui": "^0.84.4",
+    "@earendil-works/pi-ai": "^0.87.1",
+    "@earendil-works/pi-coding-agent": "^0.87.1",
+    "@earendil-works/pi-tui": "^0.87.1",
     "@types/node": "^26.0.0",
     "typescript": "^7.0.2",
     "vitest": "^4.1.8",
-    "@xzzpig/pi-mouse-events": "0.1.2"
+    "@xzzpig/pi-mouse-events": "0.2.0"
   },
   "peerDependenciesMeta": {
     "@xzzpig/pi-mouse-events": {