@herbertgao/pi-extensions 2026.9.4 → 2026.9.5

package/node_modules/@herbertgao/pi-cc-extensions/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@herbertgao/pi-cc-extensions",
-	"version": "0.9.1",
+	"version": "0.9.2",
 	"license": "MIT",
 	"description": "A Pi productivity suite with Claude Code-style UI, context inspection, and agent/session references",
 	"type": "module",
@@ -69,8 +69,8 @@
 	},
 	"x-upstream": {
 		"package": "pi-cc-extensions",
-		"version": "0.8.70",
+		"version": "0.8.71",
 		"repository": "https://github.com/minuque/pi-cc-extensions",
-		"commit": "9f200064951ffb148ecec2e1cf0dab7a2940f823"
+		"commit": "e43e0041b59f5d7f03be9b9d103a5f9e954e4c11"
 	}
 }

package/node_modules/@herbertgao/sol-pi/src/sol-pi/extensions/online-context-compact/economics.ts

@@ -218,9 +218,8 @@ export function decideCompaction(input: {
 		carriedDebtTokens: input.carriedDebtTokens,
 		cacheDebtRepaymentTokens: input.cacheDebtRepaymentTokens,
 		compact,
-		reason: !compressible
-			? "non_positive_saving"
-			: windowProtection
+		reason: compressible
+			? windowProtection
 				? "window_protection"
 				: economic
 					? "economic"
@@ -232,6 +231,7 @@ export function decideCompaction(input: {
 								? "deferred_subsequent_margin"
 								: !firstCompaction && baseEconomic && !carriedDebtGateOpen
 									? "deferred_carried_debt"
-									: "deferred_economic",
+									: "deferred_economic"
+			: "non_positive_saving",
 	};
 }

package/node_modules/@herbertgao/sol-pi/src/sol-pi/extensions/online-context-compact/extension.ts

@@ -2,12 +2,13 @@
  * SPDX-FileCopyrightText: Copyright (c) 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
  * SPDX-License-Identifier: MIT
  */
-import type { AgentMessage, AgentToolResult } from "@earendil-works/pi-agent-core";
+import type { AgentMessage, AgentToolResult, ThinkingLevel } from "@earendil-works/pi-agent-core";
 import {
 	buildSessionContext,
 	estimateTokens,
 	findCutPoint,
 	sessionEntryToContextMessages,
+	type ExtensionAPI,
 	type ExtensionContext,
 	type ExtensionFactory,
 	type SessionEntry,
@@ -165,6 +166,37 @@ function validPositiveInteger(value: unknown): value is number {
 	return typeof value === "number" && Number.isSafeInteger(value) && value > 0;
 }
 
+type CompactionThinkingOverride = {
+	readonly previous: ThinkingLevel;
+	readonly applied: ThinkingLevel;
+};
+
+// Native summaries share their output cap with reasoning; reserve the cap for summary text.
+function lowerCompactionThinking(
+	pi: ExtensionAPI,
+	reasoning: boolean,
+): CompactionThinkingOverride | undefined {
+	if (!reasoning) return;
+	try {
+		const previous = pi.getThinkingLevel();
+		if (previous !== "high" && previous !== "xhigh" && previous !== "max") return;
+		pi.setThinkingLevel("medium");
+		const applied = pi.getThinkingLevel();
+		return applied === previous ? undefined : { previous, applied };
+	} catch {
+		return;
+	}
+}
+
+function restoreCompactionThinking(pi: ExtensionAPI, override: CompactionThinkingOverride | undefined): void {
+	if (!override) return;
+	try {
+		if (pi.getThinkingLevel() === override.applied) pi.setThinkingLevel(override.previous);
+	} catch {
+		// A compaction result must not be masked by a best-effort state restore.
+	}
+}
+
 function releaseParentContinuation(continuation: PendingContinuation | undefined): void {
 	if (continuation) setTimeout(continuation.resolve, 0);
 }
@@ -182,6 +214,7 @@ export function createOnlineContextCompactExtension(options: OnlineContextCompac
 		let activeDebt: CacheDebt | undefined;
 		let nextContinuation: PendingContinuation | undefined;
 		let compactionInFlight = false;
+		let compactionThinkingOverride: CompactionThinkingOverride | undefined;
 
 		const releaseContinuation = (): void => {
 			const continuation = nextContinuation;
@@ -349,6 +382,7 @@ export function createOnlineContextCompactExtension(options: OnlineContextCompac
 			let compactionError: Error | undefined;
 			try {
 				compactionInFlight = true;
+				compactionThinkingOverride = lowerCompactionThinking(pi, context.model?.reasoning === true);
 				await new Promise<void>((resolve) => {
 					let finished = false;
 					const finish = (): void => {
@@ -382,6 +416,8 @@ export function createOnlineContextCompactExtension(options: OnlineContextCompac
 						},
 					});
 				});
+				restoreCompactionThinking(pi, compactionThinkingOverride);
+				compactionThinkingOverride = undefined;
 				compactionInFlight = false;
 				if (
 					compactionError &&
@@ -422,6 +458,8 @@ export function createOnlineContextCompactExtension(options: OnlineContextCompac
 					await continuation.promise;
 				}
 			} finally {
+				restoreCompactionThinking(pi, compactionThinkingOverride);
+				compactionThinkingOverride = undefined;
 				compactionInFlight = false;
 				activeDebt = undefined;
 				releaseParentContinuation(parentContinuation);
@@ -445,6 +483,8 @@ export function createOnlineContextCompactExtension(options: OnlineContextCompac
 		});
 
 		pi.on("session_shutdown", () => {
+			restoreCompactionThinking(pi, compactionThinkingOverride);
+			compactionThinkingOverride = undefined;
 			releaseContinuation();
 			pendingBoundary = undefined;
 			selected = undefined;

package/node_modules/@juicesharp/rpiv-ask-user-question/ask-user-question.ts

@@ -18,6 +18,7 @@ import {
 import { type DialogUI, hasDialogUI, runRpcQuestionnaire } from "./rpc-fallback.js";
 import { displayLabel, t } from "./state/i18n-bridge.js";
 import { sentinelsToAppend } from "./state/row-intent.js";
+import { normalizeQuestionParams } from "./tool/normalize-params.js";
 import { buildQuestionnaireResponse, buildToolResult } from "./tool/response-envelope.js";
 import {
 	MAX_OPTIONS,
@@ -310,7 +311,10 @@ export function registerAskUserQuestionTool(pi: ExtensionAPI): void {
 		parameters: QuestionParamsSchema,
 
 		async execute(_toolCallId, params, _signal, _onUpdate, ctx) {
-			const typed = params as unknown as QuestionParams;
+			// Line-terminator normalization runs once here, ahead of validation, so
+			// every downstream consumer — validator, TUI, RPC walker, envelope, prompt
+			// event — sees the same clean text (#192).
+			const typed = normalizeQuestionParams(params as unknown as QuestionParams);
 			if (!ctx.hasUI) return rejectWithoutUi();
 
 			const validation = validateQuestionnaire(typed);

package/node_modules/@juicesharp/rpiv-ask-user-question/events.ts

@@ -38,7 +38,11 @@ export interface AskUserBlockedEventPayload {
 }
 
 export interface AskUserPromptQuestion {
-	/** The full question text, exactly as the agent authored it. */
+	/**
+	 * The full question text as the agent authored it, with line terminators
+	 * normalized at tool entry (`\r\n` → `\n`, lone `\r` removed — #192). The
+	 * same normalization applies to `header` and every option field below.
+	 */
 	question: string;
 	/** The short chip/tag shown next to the question. */
 	header: string;

package/node_modules/@juicesharp/rpiv-ask-user-question/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@juicesharp/rpiv-ask-user-question",
-	"version": "2.9.0",
+	"version": "2.10.1",
 	"description": "Pi extension. A structured questionnaire the model can put to you when it would otherwise guess, with typed options instead of free-form replies.",
 	"keywords": [
 		"pi-package",
@@ -56,6 +56,7 @@
 		"state/state-reducer.ts",
 		"state/state.ts",
 		"tool/format-answer.ts",
+		"tool/normalize-params.ts",
 		"tool/response-envelope.ts",
 		"tool/types.ts",
 		"tool/validate-questionnaire.ts",
@@ -94,7 +95,7 @@
 		}
 	},
 	"dependencies": {
-		"@juicesharp/rpiv-config": "^2.9.0",
+		"@juicesharp/rpiv-config": "^2.10.1",
 		"typebox": "^1.1.24"
 	}
 }

package/node_modules/@juicesharp/rpiv-ask-user-question/tool/normalize-params.ts

@@ -0,0 +1,56 @@
+import type { QuestionParams } from "./types.js";
+
+/**
+ * Normalize line terminators in one model-supplied text field (#192).
+ *
+ * Some models serialize a bare carriage return inside tool-call string
+ * arguments at token boundaries where the text was meant to be contiguous
+ * (`GEMBA\r_LOG\r_FILE` for `GEMBA_LOG_FILE`). A raw CR is a cursor-control
+ * byte, not text: pi-tui ≤0.80 writes it straight into the row, so the
+ * terminal returns to column 0 and later text overwrites the pointer and
+ * number; pi-tui ≥0.84 splits `wrapTextWithAnsi` on `\r`, so one option
+ * fragments into stacked rows. Both symptoms have the same fix at our
+ * boundary:
+ *
+ * - `\r\n` → `\n` keeps genuine multi-line content (preview markdown) intact.
+ * - a lone `\r` is deleted — never a space (phantom gaps inside words) and
+ *   never `\n` (reintroduces the vertical fragmentation). This matches
+ *   pi-coding-agent's own display normalization (`normalizeDisplayText`).
+ */
+export function normalizeLineTerminators(text: string): string {
+	return text.replace(/\r\n/g, "\n").replace(/\r/g, "");
+}
+
+/**
+ * Copy `obj`, normalizing the listed keys that hold strings. Keys absent from
+ * `obj` (e.g. an omitted `preview`) stay absent so `"preview" in option`
+ * checks and `hasPreview` derivations are unchanged.
+ */
+function normalizeStringFields<T extends object>(obj: T, keys: readonly (keyof T)[]): T {
+	const out = { ...obj };
+	for (const key of keys) {
+		const value = obj[key];
+		if (typeof value === "string") out[key] = normalizeLineTerminators(value) as T[typeof key];
+	}
+	return out;
+}
+
+/**
+ * Return a copy of the tool params with every user-facing string field
+ * (`question`, `header`, `options[].label`, `options[].description`,
+ * `options[].preview`) line-terminator-normalized. Runs once at tool entry,
+ * BEFORE `validateQuestionnaire`, so the reserved-label and duplicate-label
+ * guards compare the text the user will actually see (`"Other\r"` must not
+ * slip past `reserved_label`), and so the TUI, the RPC dialog walker, the
+ * envelope echo, and the `rpiv:ask-user:prompt` payload all carry the same
+ * clean text. Pure: the input object is never mutated.
+ */
+export function normalizeQuestionParams(params: QuestionParams): QuestionParams {
+	return {
+		...params,
+		questions: params.questions.map((q) => ({
+			...normalizeStringFields(q, ["question", "header"]),
+			options: q.options.map((o) => normalizeStringFields(o, ["label", "description", "preview"])),
+		})),
+	};
+}

package/node_modules/@juicesharp/rpiv-ask-user-question/view/components/option-list-view.ts

@@ -13,7 +13,7 @@ export interface OptionListViewConfig {
 }
 
 /**
- * Per-tick projection of OptionListView state. After Phase 11b, `inputBuffer`
+ * Per-tick projection of OptionListView state. `inputBuffer`
  * is part of the props bag — the session-owned `inlineInput` (a headless
  * `pi-tui` Editor instance) supplies its current `getText()` here per tick.
  * `OptionListView` is purely props-driven; the imperative buffer surface and

package/node_modules/@juicesharp/rpiv-ask-user-question/view/components/tab-bar.ts

@@ -6,8 +6,7 @@ import type { StatefulView } from "../stateful-view.js";
  * Per-tick projection of TabBar state. The selector
  * (`selectTabBarProps`) hoists every render-time derivation
  * (`allAnswered`, `answered`, `isActive`, `submitActive`) into props so
- * `render()` is pure styling. Replaces the prior `setConfig(TabBarConfig)`
- * snowflake and the inline `+ 1` magic at `props-adapter.ts:127`.
+ * `render()` is pure styling.
  */
 export interface TabBarProps {
 	/** One per author-defined question, in order. */

package/node_modules/@pi-plugins/fast-mode/README.md

@@ -1,12 +1,7 @@
 # `@pi-plugins/fast-mode`
 
-A [pi-agent](https://github.com/earendil-works/pi) extension that requests a
-provider's fast (priority) service tier for configured models — faster inference at a
-higher token cost.
-
-Fast mode is a per-model opt-in: you decide which `provider/model-id` pairs it
-applies to, and a single session toggle turns it on and off. Requests for any other
-model are left untouched.
+Request priority inference for selected models in
+[pi](https://github.com/earendil-works/pi) with a session toggle.
 
 ## Install
 
@@ -14,49 +9,45 @@ model are left untouched.
 pi install npm:@pi-plugins/fast-mode
 ```
 
-For a one-off run without adding it to settings:
+To try it for one session without changing your settings:
 
 ```bash
 pi -e npm:@pi-plugins/fast-mode
 ```
 
-For local development, load it straight from this directory:
-
-```bash
-pi -e ./plugins/fast-mode
-```
-
-## Supported providers
-
-| Provider       | Mechanism                          |
-| -------------- | ---------------------------------- |
-| `openai`       | `service_tier: "priority"` payload |
-| `openai-codex` | `service_tier: "priority"` payload |
-
 ## Usage
 
-```text
-/fast          # toggle fast mode for this session
-/fast on       # enable
-/fast off      # disable
-/fast status   # show the current state and why it is (in)active
-```
+Use `/fast` to control fast mode for the current session:
 
-Start a session with fast mode already enabled:
+| Command        | Action                                                                       |
+| -------------- | ---------------------------------------------------------------------------- |
+| `/fast`        | Toggle fast mode.                                                            |
+| `/fast on`     | Enable fast mode.                                                            |
+| `/fast off`    | Disable fast mode.                                                           |
+| `/fast status` | Show the current state and explain whether it applies to the selected model. |
+
+To start a session with fast mode enabled:
 
 ```bash
 pi --fast
 ```
 
-While fast mode is on and the current model is configured for it, a dim `[fast mode]`
-indicator is shown at the right of a status line right above the editor, shared with
-other pi-plugins extensions (e.g. `@pi-plugins/speed` renders its measurements at the
-left of the same line).
+Fast mode applies only to models listed in the configuration. Other models are left
+unchanged. A `[fast mode]` indicator appears above the editor while active.
+
+## Supported providers
+
+| Provider     | Pi provider    |
+| ------------ | -------------- |
+| OpenAI       | `openai`       |
+| OpenAI Codex | `openai-codex` |
+
+Priority availability depends on the provider and model.
 
 ## Configuration
 
-Optional config file at `<agent-dir>/extensions/fast-mode.json` (typically
-`~/.pi/agent/extensions/fast-mode.json`):
+Optionally create `<agent-dir>/extensions/fast-mode.json`, typically
+`~/.pi/agent/extensions/fast-mode.json`. The defaults are:
 
 ```json
 {
@@ -79,16 +70,12 @@ Optional config file at `<agent-dir>/extensions/fast-mode.json` (typically
 }
 ```
 
-- `enabled`: fast-mode state at session start (`/fast` overrides it for the session
-  without writing the toggle back to the file).
-- `models`: the `provider/model-id` keys fast mode applies to. The defaults above are
-  used when the file is absent. Models whose API has no fast-mode support are
-  ignored.
-- `showStatus`: show the `[fast mode]` indicator above the editor while active.
+| Setting      | Description                                                                                                 |
+| ------------ | ----------------------------------------------------------------------------------------------------------- |
+| `enabled`    | Enable fast mode at session start. `/fast` changes only the current session and does not write to the file. |
+| `models`     | The exact `provider/model-id` pairs fast mode applies to.                                                   |
+| `showStatus` | Show the status indicator while fast mode is active.                                                        |
 
 ## Notes
 
-- The priority service tier bills at a higher rate than the default tier — check your
-  provider's pricing before leaving it enabled.
-- If another extension already set a `service_tier` on the payload, this extension
-  leaves it alone.
+Priority inference can cost more. Check your provider's pricing before enabling it.