pi-ui-extend 1.0.14 → 1.0.16

package/dist/schemas/pi-tools-suite-schema.d.ts

@@ -12,6 +12,7 @@ export declare const PiToolsSuiteConfigSchema: Type.TObject<{
     enabledModules: Type.TOptional<Type.TArray<Type.TString>>;
     modules: Type.TOptional<Type.TRecord<"^.*$", Type.TBoolean>>;
     todoThinking: Type.TOptional<Type.TBoolean>;
+    todoThinkingOverrides: Type.TOptional<Type.TRecord<"^.*$", Type.TUnion<[Type.TLiteral<"off">, Type.TLiteral<"minimal">, Type.TLiteral<"low">, Type.TLiteral<"medium">, Type.TLiteral<"high">, Type.TLiteral<"xhigh">, Type.TLiteral<"max">, Type.TNull]>>>;
     lookupModel: Type.TOptional<Type.TUnion<[Type.TString, Type.TNull]>>;
     terminalBell: Type.TOptional<Type.TObject<{
         sound: Type.TOptional<Type.TBoolean>;

package/dist/schemas/pi-tools-suite-schema.js

@@ -221,6 +221,16 @@ export const PiToolsSuiteConfigSchema = Type.Object({
     enabledModules: Type.Optional(Type.Array(Type.String(), { description: "List of module names to explicitly enable, including modules that are disabled by default." })),
     modules: Type.Optional(Type.Record(Type.String(), Type.Boolean(), { description: "Per-module enable/disable map. credential-firewall is disabled by default and can be enabled here." })),
     todoThinking: Type.Optional(Type.Boolean({ description: "Enable per-todo thinking levels and automatic thinking switch/restore when tasks become in-progress/completed." })),
+    todoThinkingOverrides: Type.Optional(Type.Record(Type.String(), Type.Union([
+        Type.Literal("off"),
+        Type.Literal("minimal"),
+        Type.Literal("low"),
+        Type.Literal("medium"),
+        Type.Literal("high"),
+        Type.Literal("xhigh"),
+        Type.Literal("max"),
+        Type.Null(),
+    ]), { description: "Force per-todo thinking for matching provider/model or bare-model keys. Keys support * and ? wildcards; null removes an inherited override." })),
     lookupModel: Type.Optional(Type.Union([Type.String(), Type.Null()], { description: "Vision-capable provider/model used by GLM's lookup tool; unset or null disables lookup." })),
     terminalBell: Type.Optional(TerminalBellConfig),
     dcp: Type.Optional(DcpConfig),

package/external/pi-tools-suite/README.md

@@ -62,6 +62,18 @@ Saved prompt slash commands are stored under `promptCommands`. Use `/prompt-comm
 }
 ```
 
+Todo thinking can be enabled globally and forced to a fixed level for selected models. `todoThinkingOverrides` keys accept exact `provider/model` or bare-model names plus `*` and `?` wildcards. Full provider/model matches beat bare-model matches, exact matches beat wildcards, and the more specific wildcard wins. The override is applied at runtime to create/update and batch create/update mutations even when the model requests another level or omits `thinking`. Unsupported levels are normalized to the nearest level supported by the current model. Later config layers can remove an inherited entry with `null`.
+
+```jsonc
+{
+  "todoThinking": true,
+  "todoThinkingOverrides": {
+    "zai/glm-5.3": "max",
+    "cheap-provider/*": "high"
+  }
+}
+```
+
 DCP settings are stored only under `dcp` in the user shared config file `~/.config/pi/pi-tools-suite.jsonc`. Legacy standalone `dcp.jsonc`, `$PI_CONFIG_DIR`, and project-local `.pi/pi-tools-suite.jsonc` DCP settings are intentionally ignored by the ported headless DCP module.
 
 ```jsonc

package/external/pi-tools-suite/src/codex-reasoning-fix/index.ts

@@ -1,11 +1,13 @@
 /**
- * WORKAROUND for @earendil-works/pi-ai bug: the Codex / OpenAI Responses API
- * rejects HTTP 400 `Unknown parameter: 'input[N].content'` when non-message
- * items (reasoning, function_call_output) carry a spurious `content` field.
+ * WORKAROUNDS for Codex / OpenAI Responses payload compatibility:
+ * - non-message items (reasoning, function_call_output) must not carry the
+ *   spurious `content` field rejected with HTTP 400;
+ * - the Codex backend rejects the legacy `prompt_cache_retention` field for
+ *   all current models, while direct OpenAI GPT-5.6+ uses the newer prompt
+ *   cache options shape.
  *
- * Stray fields can come from replayed pi-ai items or from another extension
- * that modifies the provider payload. The sanitizer therefore must be the
- * LAST `before_provider_request` handler registered by pi-tools-suite.
+ * The sanitizer must be the LAST `before_provider_request` handler registered
+ * by pi-tools-suite so another payload hook cannot undo these guards.
  *
  * In pi-ai >= 0.80.6 the final `before_provider_request` payload feeds both the
  * WebSocket delta builder and the zstd-compressed SSE fallback body. Running the
@@ -27,6 +29,9 @@ type ProviderRequestContext = {
 	model?: unknown;
 };
 
+const OPENAI_CODEX_PROVIDER = "openai-codex";
+const OPENAI_PROVIDER = "openai";
+
 /**
  * Strip spurious `content` from any object that carries an `input` or
  * `messages` array. Shared core for payload sanitization.
@@ -75,12 +80,21 @@ export default function codexReasoningFix(pi: ExtensionAPI): void {
 	// src/index.ts deliberately registers this module last. A later payload
 	// modifier could otherwise reintroduce invalid content after sanitization,
 	// and transport encoding happens after this hook.
-	pi.on("before_provider_request", async (event: ProviderRequestEvent, _ctx: ProviderRequestContext) => {
-		const result = stripReasoningContentFromPayload(event.payload);
+	pi.on("before_provider_request", async (event: ProviderRequestEvent, ctx: ProviderRequestContext) => {
+		const result = sanitizeCodexProviderPayload(event.payload, ctx.model);
 		return result === event.payload ? undefined : result;
 	});
 }
 
+/**
+ * Apply all final Codex payload compatibility guards. Returns the original
+ * reference when no guard changes the payload.
+ */
+export function sanitizeCodexProviderPayload(payload: unknown, model: unknown): unknown {
+	const contentSanitized = stripReasoningContentFromPayload(payload);
+	return stripUnsupportedPromptCacheRetention(contentSanitized, model);
+}
+
 /**
  * Strip spurious `content` from non-message items in a full payload. Returns the same reference
  * when nothing changed; exported for unit testing.
@@ -90,6 +104,72 @@ export function stripReasoningContentFromPayload(payload: unknown): unknown {
 	return result ? result.obj : payload;
 }
 
+/**
+ * Remove legacy prompt-cache retention only where it is known to be rejected:
+ * every current Codex model, and direct OpenAI GPT-5.6 or newer. Match the
+ * selected model rather than a bare payload id so other providers keep the
+ * field untouched.
+ */
+export function stripUnsupportedPromptCacheRetention(payload: unknown, model: unknown): unknown {
+	if (!isRecord(payload) || !Object.prototype.hasOwnProperty.call(payload, "prompt_cache_retention")) {
+		return payload;
+	}
+	if (!rejectsLegacyPromptCacheRetention(model, payload.model)) return payload;
+
+	const { prompt_cache_retention: _drop, ...rest } = payload;
+	return rest;
+}
+
+function rejectsLegacyPromptCacheRetention(model: unknown, payloadModel: unknown): boolean {
+	const selected = modelIdentity(model);
+	if (selected.provider !== undefined) {
+		return isAffectedProviderModel(selected.provider, selected.id);
+	}
+	if (selected.id?.includes("/")) {
+		return isAffectedQualifiedModel(selected.id);
+	}
+
+	return typeof payloadModel === "string" && isAffectedQualifiedModel(payloadModel.trim().toLowerCase());
+}
+
+function isAffectedQualifiedModel(modelRef: string): boolean {
+	const slash = modelRef.indexOf("/");
+	if (slash <= 0 || slash === modelRef.length - 1) return false;
+	return isAffectedProviderModel(modelRef.slice(0, slash), modelRef.slice(slash + 1));
+}
+
+function isAffectedProviderModel(provider: string, modelId: string | undefined): boolean {
+	if (provider === OPENAI_CODEX_PROVIDER) return true;
+	return provider === OPENAI_PROVIDER && modelId !== undefined && isGpt56OrNewer(modelId);
+}
+
+function isGpt56OrNewer(modelId: string): boolean {
+	const match = /^gpt-(\d+)(?:\.(\d+))?(?:-|$)/u.exec(modelId);
+	if (!match) return false;
+	const major = Number(match[1]);
+	const minor = Number(match[2] ?? 0);
+	return major > 5 || (major === 5 && minor >= 6);
+}
+
+function modelIdentity(model: unknown): { provider?: string; id?: string } {
+	if (typeof model === "string") return { id: model.trim().toLowerCase() };
+	if (!isRecord(model)) return {};
+
+	const provider = firstString(model.provider, model.providerId, model.providerID);
+	const id = firstString(model.id, model.modelId, model.modelID, model.model);
+	return {
+		provider: provider?.toLowerCase(),
+		id: id?.toLowerCase(),
+	};
+}
+
+function firstString(...values: unknown[]): string | undefined {
+	for (const value of values) {
+		if (typeof value === "string" && value.trim()) return value.trim();
+	}
+	return undefined;
+}
+
 function isRecord(value: unknown): value is Record<string, unknown> {
 	return value !== null && typeof value === "object" && !Array.isArray(value);
 }

package/external/pi-tools-suite/src/config.ts

@@ -9,6 +9,7 @@ export interface PiToolsSuiteConfig {
 	enabled: boolean;
 	disabledModules: string[];
 	todoThinking: boolean;
+	todoThinkingOverrides: Record<string, TodoThinkingLevel>;
 	/** Vision-capable model used by the coding-discipline lookup tool; unset disables lookup. */
 	lookupModel?: string;
 	/**
@@ -25,6 +26,7 @@ type MutableConfig = {
 	enabled: boolean;
 	disabledModules: Set<string>;
 	todoThinking: boolean;
+	todoThinkingOverrides: Map<string, TodoThinkingLevel>;
 	lookupModel: string | undefined;
 	codingDisciplineStrictness: CodingDisciplineStrictness;
 };
@@ -32,6 +34,8 @@ type MutableConfig = {
 export const CODING_DISCIPLINE_STRICTNESS_VALUES = ["strict", "lenient"] as const;
 export type CodingDisciplineStrictness = (typeof CODING_DISCIPLINE_STRICTNESS_VALUES)[number];
 export const DEFAULT_CODING_DISCIPLINE_STRICTNESS: CodingDisciplineStrictness = "lenient";
+const TODO_THINKING_OVERRIDE_LEVELS = ["off", "minimal", "low", "medium", "high", "xhigh", "max"] as const;
+export type TodoThinkingLevel = (typeof TODO_THINKING_OVERRIDE_LEVELS)[number];
 
 type Env = Record<string, string | undefined>;
 
@@ -42,6 +46,7 @@ const DISABLED_LIST_KEYS = ["disabledModules", "disabledExtensions"];
 const ENABLED_LIST_KEYS = ["enabledModules", "enabledExtensions"];
 const MODULE_MAP_KEYS = ["modules", "extensions"];
 const DEFAULT_DISABLED_MODULES = new Set<string>(["credential-firewall"]);
+const DEFAULT_TODO_THINKING_OVERRIDES = new Map<string, TodoThinkingLevel>([["zai/glm-5.3", "max"]]);
 
 export function getPiToolsSuiteUserConfigPath(homeDir = homedir()): string {
 	return join(homeDir, ".config", "pi", "pi-tools-suite.jsonc");
@@ -79,6 +84,23 @@ function normalizeCodingDisciplineStrictness(raw: unknown): CodingDisciplineStri
 	return raw === "strict" ? "strict" : "lenient";
 }
 
+function isTodoThinkingLevel(raw: unknown): raw is TodoThinkingLevel {
+	return TODO_THINKING_OVERRIDE_LEVELS.includes(raw as TodoThinkingLevel);
+}
+
+function mergeTodoThinkingOverrides(config: MutableConfig, raw: unknown): void {
+	if (!isRecord(raw)) return;
+	for (const [rawPattern, value] of Object.entries(raw)) {
+		const pattern = rawPattern.trim().toLowerCase();
+		if (!pattern) continue;
+		if (value === null) {
+			config.todoThinkingOverrides.delete(pattern);
+			continue;
+		}
+		if (isTodoThinkingLevel(value)) config.todoThinkingOverrides.set(pattern, value);
+	}
+}
+
 function boolFromEnv(value: string | undefined): boolean | undefined {
 	if (value === undefined) return undefined;
 	const normalized = value.trim().toLowerCase();
@@ -137,6 +159,7 @@ function removeDisabled(config: MutableConfig, value: unknown, knownModules: Rea
 function mergeConfigLayer(config: MutableConfig, raw: Record<string, unknown>, knownModules: ReadonlySet<string>): MutableConfig {
 	if (typeof raw.enabled === "boolean") config.enabled = raw.enabled;
 	if (typeof raw.todoThinking === "boolean") config.todoThinking = raw.todoThinking;
+	mergeTodoThinkingOverrides(config, raw.todoThinkingOverrides);
 	if (Object.prototype.hasOwnProperty.call(raw, "lookupModel")) config.lookupModel = normalizeLookupModel(raw.lookupModel);
 	if (Object.prototype.hasOwnProperty.call(raw, "codingDisciplineStrictness")) {
 		config.codingDisciplineStrictness = normalizeCodingDisciplineStrictness(raw.codingDisciplineStrictness);
@@ -197,6 +220,7 @@ export function loadPiToolsSuiteConfig(moduleNames: readonly string[], options:
 		enabled: true,
 		disabledModules: new Set([...DEFAULT_DISABLED_MODULES].filter((name) => knownModules.has(name))),
 		todoThinking: false,
+		todoThinkingOverrides: new Map(DEFAULT_TODO_THINKING_OVERRIDES),
 		lookupModel: undefined,
 		codingDisciplineStrictness: DEFAULT_CODING_DISCIPLINE_STRICTNESS,
 	};
@@ -217,6 +241,7 @@ export function loadPiToolsSuiteConfig(moduleNames: readonly string[], options:
 		enabled: config.enabled,
 		disabledModules: [...config.disabledModules].sort(),
 		todoThinking: config.todoThinking,
+		todoThinkingOverrides: Object.fromEntries(config.todoThinkingOverrides),
 		...(config.lookupModel ? { lookupModel: config.lookupModel } : {}),
 		codingDisciplineStrictness: config.codingDisciplineStrictness,
 	};

package/external/pi-tools-suite/src/default-pi-tools-suite-config.ts

@@ -15,6 +15,12 @@ export const DEFAULT_PI_TOOLS_SUITE_CONFIG_JSONC = String.raw`{
   // When true, todo items may carry a per-task thinking level and the todo
   // module will switch/restore Pi's thinking level as in-progress tasks change.
   "todoThinking": true,
+  // Force every todo mutation made under matching models to the configured
+  // thinking level. Supports provider/model or bare-model keys with * / ? globs.
+  // Set an inherited key to null in a later config layer to remove it.
+  "todoThinkingOverrides": {
+    "zai/glm-5.3": "max"
+  },
   // Vision-capable model used by the coding-discipline lookup tool for blind-model
   // screenshot/image questions. Remove or set to null to disable lookup.
   "lookupModel": "openai-codex/gpt-5.4-mini",

package/external/pi-tools-suite/src/todo/index.ts

@@ -1,5 +1,5 @@
 import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
-import { loadPiToolsSuiteConfig } from "../config.js";
+import { loadPiToolsSuiteConfig, type TodoThinkingLevel as ConfigTodoThinkingLevel } from "../config.js";
 import { isAgentBusyRaceError } from "../context-usage.js";
 import { autoClearCompletedTodos } from "./state/auto-clear.js";
 import { loadPersistedPlan, syncPersistedPlan } from "./state/persistence.js";
@@ -31,6 +31,7 @@ function isStaleExtensionContextError(error: unknown): boolean {
 
 type ModelLike = {
 	provider?: string;
+	providerId?: string;
 	id?: string;
 	modelId?: string;
 	reasoning?: boolean;
@@ -38,6 +39,51 @@ type ModelLike = {
 	compat?: { thinkingFormat?: unknown };
 };
 
+function escapeRegExp(text: string): string {
+	return text.replace(/[|\\{}()[\]^$+?.]/g, "\\$&");
+}
+
+function modelPatternMatches(pattern: string, candidate: string): boolean {
+	let source = "^";
+	for (const char of pattern) {
+		if (char === "*") source += ".*";
+		else if (char === "?") source += ".";
+		else source += escapeRegExp(char);
+	}
+	return new RegExp(`${source}$`, "i").test(candidate);
+}
+
+function modelKeys(model: unknown): { bare?: string; full?: string } {
+	const candidate = model as ModelLike | undefined;
+	const provider = candidate?.provider ?? candidate?.providerId;
+	const rawId = candidate?.modelId ?? candidate?.id;
+	if (!rawId) return {};
+	if (rawId.includes("/")) {
+		const slash = rawId.indexOf("/");
+		return { bare: rawId.slice(slash + 1), full: rawId };
+	}
+	return { bare: rawId, ...(provider ? { full: `${provider}/${rawId}` } : {}) };
+}
+
+function resolveTodoThinkingOverride(
+	model: unknown,
+	overrides: Record<string, ConfigTodoThinkingLevel>,
+): TodoThinkingLevel | undefined {
+	const keys = modelKeys(model);
+	let best: { level: TodoThinkingLevel; score: number } | undefined;
+	for (const [rawPattern, level] of Object.entries(overrides)) {
+		const pattern = rawPattern.trim();
+		const isFull = pattern.includes("/");
+		const candidate = isFull ? keys.full : keys.bare;
+		if (!candidate || !modelPatternMatches(pattern, candidate)) continue;
+		const exact = !pattern.includes("*") && !pattern.includes("?");
+		const literalLength = pattern.replace(/[?*]/g, "").length;
+		const score = (isFull ? 30_000 : 10_000) + (exact ? 10_000 : 0) + literalLength;
+		if (!best || score >= best.score) best = { level, score };
+	}
+	return best?.level;
+}
+
 function isTodoThinkingLevel(value: unknown): value is TodoThinkingLevel {
 	return TODO_THINKING_LEVEL_VALUES.includes(value as TodoThinkingLevel);
 }
@@ -167,7 +213,9 @@ function emitPersistedPlanPrompt(pi: ExtensionAPI, ctx: ExtensionContext, prompt
 
 export default function (pi: ExtensionAPI) {
 	let currentModel: unknown;
-	const todoThinkingEnabled = loadPiToolsSuiteConfig(["todo"]).todoThinking;
+	const todoConfig = loadPiToolsSuiteConfig(["todo"]);
+	const todoThinkingEnabled = todoConfig.todoThinking;
+	const todoThinkingOverrides = todoConfig.todoThinkingOverrides;
 	const rememberedThinkingByTaskId = new Map<number, TodoThinkingLevel>();
 	let lastNudgedSignature: string | undefined;
 	let nudgeTimer: ReturnType<typeof setTimeout> | undefined;
@@ -237,7 +285,11 @@ export default function (pi: ExtensionAPI) {
 
 	function prepareTodoThinkingMutation(state: ReturnType<typeof getState>, params: TaskMutationParams): TaskMutationParams {
 		let nextParams = params;
-		if (params.thinking !== undefined) {
+		const configuredOverride = resolveTodoThinkingOverride(currentModel, todoThinkingOverrides);
+		if (configuredOverride !== undefined) {
+			const forced = normalizeTodoThinkingLevelForModel(currentModel, configuredOverride);
+			nextParams = { ...nextParams, thinking: forced };
+		} else if (params.thinking !== undefined) {
 			const normalized = normalizeTodoThinkingLevelForModel(currentModel, params.thinking);
 			if (normalized !== params.thinking) nextParams = { ...nextParams, thinking: normalized };
 		}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-ui-extend",
-  "version": "1.0.14",
+  "version": "1.0.16",
   "description": "Pix: a workspace-first terminal UI for Pi with tabs, readable tool activity, voice input, and bundled agent tools.",
   "private": false,
   "repository": {

package/schemas/pi-tools-suite.json

@@ -36,6 +36,47 @@
       "type": "boolean",
       "description": "Enable per-todo thinking levels and automatic thinking switch/restore when tasks become in-progress/completed."
     },
+    "todoThinkingOverrides": {
+      "type": "object",
+      "patternProperties": {
+        "^.*$": {
+          "anyOf": [
+            {
+              "type": "string",
+              "const": "off"
+            },
+            {
+              "type": "string",
+              "const": "minimal"
+            },
+            {
+              "type": "string",
+              "const": "low"
+            },
+            {
+              "type": "string",
+              "const": "medium"
+            },
+            {
+              "type": "string",
+              "const": "high"
+            },
+            {
+              "type": "string",
+              "const": "xhigh"
+            },
+            {
+              "type": "string",
+              "const": "max"
+            },
+            {
+              "type": "null"
+            }
+          ]
+        }
+      },
+      "description": "Force per-todo thinking for matching provider/model or bare-model keys. Keys support * and ? wildcards; null removes an inherited override."
+    },
     "lookupModel": {
       "anyOf": [
         {