@zhushanwen/pi-permission 1.4.2 → 1.4.4

package/src/approval.ts

@@ -20,7 +20,7 @@
  * TUI 分支因 pi-tui Input 组件集成成本较高，暂保留简化 deny（TODO 后续迭代）。
  */
 
-import { type Component, matchesKey, type SelectItem, truncateToWidth } from "@earendil-works/pi-tui";
+import { type Component, matchesKey, truncateToWidth } from "@earendil-works/pi-tui";
 
 import type { ApprovalRequest } from "./pipeline.js";
 import type { ToolInvocationContext, UserDecision } from "./types.js";
@@ -198,20 +198,65 @@ function requestHeadless(req: ApprovalRequest, approvalCtx: ApprovalContext): Us
 	return { approved: false, reason: `headless mode (${approvalCtx.mode}): cannot prompt, auto-deny` };
 }
 
-// ──────────────────────── 辅助：标题格式化 ────────────────────────
+// ──────────────────────── 辅助：字段集内核 + 标题格式化 ────────────────────────
 
-/** 格式化审批标题（RPC select / TUI 顶部）。 */
+/**
+ * E6（M6/D5）：审批卡字段集（内核，唯一字段来源）。
+ *
+ * 「审批卡显示哪些内容」由 buildApprovalFieldSet 单点决定；两外壳只做排版——
+ * formatTitle（RPC）压紧凑行数组（AI 单行摘要 + reasoning 行），renderApprovalView
+ * （TUI）展开缩进块。排版是两外壳的形态差异，不属双写。
+ */
+interface ApprovalFieldSet {
+	title: string;
+	tool: string;
+	command: string | undefined;
+	reason: string;
+	ai?: {
+		risk: string;
+		outcome: string;
+		confidence: number;
+		reasoning: string;
+	};
+}
+
+/** 构造审批卡字段集（title/tool/command/reason + 可选 AI 四元组含 reasoning）。 */
+function buildApprovalFieldSet(req: ApprovalRequest): ApprovalFieldSet {
+	return {
+		title: "[pi-permission] Approval required",
+		tool: req.toolName,
+		// 空字符串与 undefined 同义（两外壳的 Command 行都只在非空时渲染）
+		command: req.command !== undefined && req.command.length > 0 ? req.command : undefined,
+		reason: req.reason,
+		...(req.preClassification
+			? {
+					ai: {
+						risk: req.preClassification.risk_level,
+						outcome: req.preClassification.outcome,
+						confidence: req.preClassification.confidence,
+						reasoning: req.preClassification.reasoning,
+					},
+				}
+			: {}),
+	};
+}
+
+/**
+ * 格式化审批标题（RPC select）。消费字段集，压成紧凑行数组（6 行）。
+ * M6 漂移修复：补 reasoning 行——此前 RPC 用户看不到 AI 理由（TUI 一直有）。
+ */
 function formatTitle(req: ApprovalRequest): string {
+	const fields = buildApprovalFieldSet(req);
 	const lines: string[] = [];
-	lines.push(`[pi-permission] Approval required`);
-	lines.push(`Tool: ${req.toolName}`);
-	if (req.command !== undefined && req.command.length > 0) {
-		lines.push(`Command: ${req.command}`);
+	lines.push(fields.title);
+	lines.push(`Tool: ${fields.tool}`);
+	if (fields.command !== undefined) {
+		lines.push(`Command: ${fields.command}`);
 	}
-	lines.push(`Reason: ${req.reason}`);
-	if (req.preClassification) {
-		const pc = req.preClassification;
-		lines.push(`AI: risk=${pc.risk_level} outcome=${pc.outcome} (conf=${pc.confidence})`);
+	lines.push(`Reason: ${fields.reason}`);
+	if (fields.ai) {
+		lines.push(`AI: risk=${fields.ai.risk} outcome=${fields.ai.outcome} (conf=${fields.ai.confidence})`);
+		lines.push(`  reasoning: ${fields.ai.reasoning}`);
 	}
 	return lines.join("\n");
 }
@@ -260,11 +305,6 @@ export class ApprovalComponent implements Component {
 		this.cachedLines = undefined;
 	}
 
-	private rerender(): void {
-		this.invalidate();
-		this.tui.requestRender();
-	}
-
 	render(width: number): string[] {
 		if (this.cachedWidth === width && this.cachedLines) {
 			return this.cachedLines;
@@ -316,25 +356,25 @@ export class ApprovalComponent implements Component {
 /** box 边框左右各占用 1 列（│ × 2） */
 const BORDER_OVERHEAD = 2;
 
-/** 渲染审批视图（纯函数，便于单测）。 */
+/** 渲染审批视图（纯函数，便于单测）。消费字段集（E6 内核），排版为 TUI 展开块。 */
 export function renderApprovalView(req: ApprovalRequest, width: number): string[] {
+	const fields = buildApprovalFieldSet(req);
 	const innerWidth = Math.max(0, width - BORDER_OVERHEAD); // 减去左右边框
 	const inner: string[] = [];
-	inner.push("[pi-permission] Approval required");
+	inner.push(fields.title);
 	inner.push("");
-	inner.push(`Tool: ${req.toolName}`);
-	if (req.command !== undefined && req.command.length > 0) {
-		inner.push(`Command: ${req.command}`);
+	inner.push(`Tool: ${fields.tool}`);
+	if (fields.command !== undefined) {
+		inner.push(`Command: ${fields.command}`);
 	}
-	inner.push(`Reason: ${req.reason}`);
-	if (req.preClassification) {
-		const pc = req.preClassification;
+	inner.push(`Reason: ${fields.reason}`);
+	if (fields.ai) {
 		inner.push("");
 		inner.push(`AI classification:`);
-		inner.push(`  risk: ${pc.risk_level}`);
-		inner.push(`  outcome: ${pc.outcome}`);
-		inner.push(`  confidence: ${pc.confidence}`);
-		inner.push(`  reasoning: ${pc.reasoning}`);
+		inner.push(`  risk: ${fields.ai.risk}`);
+		inner.push(`  outcome: ${fields.ai.outcome}`);
+		inner.push(`  confidence: ${fields.ai.confidence}`);
+		inner.push(`  reasoning: ${fields.ai.reasoning}`);
 	}
 	inner.push("");
 	inner.push("[Enter] Approve  [Esc] Deny");
@@ -349,6 +389,3 @@ export function renderApprovalView(req: ApprovalRequest, width: number): string[
 	lines.push(`└${"─".repeat(innerWidth)}┘`);
 	return lines;
 }
-
-// re-export SelectItem 仅为测试便利（mock ctx.ui.select 时构造选项）
-export type { SelectItem };

package/src/ast/analyzer.ts

@@ -19,6 +19,7 @@
  */
 
 import type { Node, Tree } from "web-tree-sitter";
+import { toErrorMessage } from "@zhushanwen/pi-ext-guards";
 import { getLogger } from "@zhushanwen/pi-extension-logger";
 
 import type { BashAnalysis } from "../types.js";
@@ -245,7 +246,7 @@ export async function analyzeBashStructure(command: string): Promise<BashAnalysi
 		};
 	} catch (err) {
 		// 任何意外异常 → fail-closed
-		const msg = err instanceof Error ? err.message : String(err);
+		const msg = toErrorMessage(err);
 		logger.warn("analyzeBashStructure exception", { error: msg });
 		return failClosed(msg || "EXCEPTION");
 	} finally {

package/src/ast/loader.ts

@@ -25,6 +25,7 @@ import { dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
 
 import { Language, Parser } from "web-tree-sitter";
+import { toErrorMessage } from "@zhushanwen/pi-ext-guards";
 import { getLogger } from "@zhushanwen/pi-extension-logger";
 
 const logger = getLogger("pi-permission/ast");
@@ -121,7 +122,7 @@ export function getBashParser(): Promise<Parser | null> {
 		} catch (err) {
 			// fail-closed：清空缓存让下次调用重试，warn 后返回 null。
 			parserPromise = null;
-			const msg = err instanceof Error ? err.message : String(err);
+			const msg = toErrorMessage(err);
 			logger.warn("getBashParser init failed", { error: msg });
 			return null;
 		}

package/src/classifier/classifier.ts

@@ -18,6 +18,7 @@
  */
 
 import type { Api, Message, Model } from "@earendil-works/pi-ai";
+import { toErrorMessage } from "@zhushanwen/pi-ext-guards";
 import type { CallLLMOptions, CallLLMResult } from "@zhushanwen/pi-llm-shared";
 
 import type {
@@ -90,7 +91,7 @@ function raceResultWithDeadline(
 			(result) => ({ kind: "ok" as const, result }),
 			(error) => ({
 				kind: "ok" as const,
-				result: { ok: false as const, error: error instanceof Error ? error.message : String(error) },
+				result: { ok: false as const, error: toErrorMessage(error) },
 			}),
 		),
 	];
@@ -229,7 +230,7 @@ export function createClassifier(deps: ClassifierDeps): {
 		try {
 			settled = await raceResultWithDeadline(callPromise, timeoutMs, signal);
 		} catch (error) {
-			const message = error instanceof Error ? error.message : String(error);
+			const message = toErrorMessage(error);
 			onLog?.(`[pi-permission] classifier: callLLM race threw: ${message}`);
 			return { ...CLASSIFY_FALLBACK_RESULT };
 		}

package/src/classifier/index.ts

@@ -1,22 +1,10 @@
 /**
- * AI Classifier barrel（层 3 入口）
+ * AI Classifier barrel（层 3 入口）——只暴露有真实消费的 2 符号（E10 收敛）。
  *
- * 对外暴露 createClassifier + ClassifierDeps + prompt/json-parser/model-resolver 辅助 API。
- * ClassifierResult / ClassifierConfig / ToolInvocationContext 从 ../types.js re-export，
- * 单一类型来源（types.ts）。
+ * 唯一生产消费者 production.ts：createClassifier（+ 装配所需的 ClassifierDeps 类型）。
+ * prompt / json-parser / model-resolver 的消费方与测试均走深路径 import；
+ * 类型一律直接从 ../types.js import，barrel 不做类型 re-export。
  */
 
 export type { ClassifierDeps } from "./classifier.js";
 export { createClassifier } from "./classifier.js";
-export { PARSE_FALLBACK_RESULT,parseClassifierResponse } from "./json-parser.js";
-export type { ListAvailableModelsCtx, ResolvedModelEntry } from "./model-resolver.js";
-export { listAvailableModels } from "./model-resolver.js";
-export { buildClassifierUserPrompt,CLASSIFIER_SYSTEM_PROMPT } from "./prompt.js";
-
-// 类型 re-export（单一来源：types.ts）
-export type {
-	ClassifierConfig,
-	ClassifierResult,
-	RiskLevel,
-	ToolInvocationContext,
-} from "../types.js";

package/src/classifier/model-resolver.ts

@@ -8,6 +8,7 @@
  * hasConfiguredAuth() 过滤——用户只经 `pi auth login` 配的内置/OAuth provider 同样可见，
  * 不再自读 models.json（loadModelsJson / flattenModels 已删除）。
  * E1：ResolvedModelEntry.apiKey 死字段已删（P3 收口后凭证走 modelRegistry，无消费者）。
+ * E5：name/baseUrl/cost 死字段已删（picker 只读 id/api）。
  */
 
 import type { Api, Model } from "@earendil-works/pi-ai";
@@ -27,27 +28,17 @@ export interface ListAvailableModelsCtx {
 
 // ──────────────────────── 条目类型 ────────────────────────
 
-/** 单个 model 的 cost 结构（与 pi-ai Model.cost 同形；缺失时填零默认） */
-interface ModelCost {
-	input: number;
-	output: number;
-	cacheRead: number;
-	cacheWrite: number;
-}
-
 /**
  * 解析后的「扁平 model 条目」：附带其 provider 名。
  *
  * listAvailableModels 的中间表示，用于 picker 分组展示。
- * 注：hasApiKey 语义已不存在（E2 后用 hasConfiguredAuth 过滤，能进列表即已配 auth）。
+ * 注：hasApiKey 语义已不存在（E2 后用 hasConfiguredAuth 过滤，能进列表即已配 auth）；
+ * E5：name/baseUrl/cost 死字段已删（picker 只读 id/api，生产零读者）。
  */
 export interface ResolvedModelEntry {
 	provider: string;
 	id: string;
-	name: string;
 	api: string;
-	baseUrl?: string;
-	cost: ModelCost;
 }
 
 // ──────────────────────── listAvailableModels（W7 model picker 用） ────────────────────────
@@ -75,11 +66,7 @@ export function listAvailableModels(
 		entries.push({
 			provider: m.provider,
 			id: m.id,
-			name: m.name ?? m.id,
 			api: m.api,
-			...(m.baseUrl ? { baseUrl: m.baseUrl } : {}),
-			// 缺失 cost 时填零默认（picker 展示稳定，不依赖真实成本）
-			cost: m.cost ?? { input: 0, output: 0, cacheRead: 0, cacheWrite: 0 },
 		});
 	}
 

package/src/commands.ts

@@ -13,9 +13,9 @@
  *   /permission status       显示当前配置详情
  */
 
-import type { ResolvedModelEntry } from "./classifier/model-resolver.js";
+import { listAvailableModels } from "./classifier/model-resolver.js";
 import { type ModelPickerContext,pickModelViaOverlay } from "./model-picker.js";
-import { type RuleEditorContext,type RuleEditorResult } from "./rule-editor.js";
+import { editRulesViaOverlay, type RuleEditorContext } from "./rule-editor.js";
 import { applyOps } from "./rule-templates.js";
 import {
 	isValidPermissionMode,
@@ -118,34 +118,26 @@ function modeHighlight(mode: PermissionMode): string {
 
 // ──────────────────────── /permission model（W7） ────────────────────────
 
-/** handlePermissionModelCommand 的依赖（DI 便于测试 mock）。 */
-export interface PermissionModelCommandDeps {
-	/** 列出可用模型（按 provider 分组）。E2 后走 ctx.modelRegistry（签名带 ctx）。 */
-	listModels: (ctx: ModelPickerContext) => Map<string, ResolvedModelEntry[]>;
-	/** 保存新配置，返回 {success, error?}。 */
-	save: (config: PermissionConfig) => { success: boolean; error?: string };
-}
-
 /**
  * /permission model 命令：overlay 选择 classifier model（provider/model 或 auto）。
  *
  * 流程：
- *  1. listModels：拿可用模型 Map。空 → 降级 notify + return。
+ *  1. listAvailableModels（ctx.modelRegistry 数据源）：拿可用模型 Map。空 → 降级 notify + return。
  *  2. pickModelViaOverlay：TUI/RPC/headless 三模式分发。
  *  3. 写回：save(newConfig)。失败 → notify error。
  *
- * 用依赖注入（deps 参数）便于测试 mock listModels/save。
- * ctx 用 ModelPickerContext 子集（mode + ui.notify/custom/select）。
+ * save 为普通函数参数（唯一真实变化轴：保存 + footer 重绘副作用）。
+ * ctx 用 ModelPickerContext 子集（mode + modelRegistry + ui.notify/custom/select）。
  */
 export async function handlePermissionModelCommand(
 	ctx: ModelPickerContext,
 	config: PermissionConfig,
-	deps: PermissionModelCommandDeps,
+	save: (config: PermissionConfig) => { success: boolean; error?: string },
 ): Promise<void> {
 	const current = config.classifier.model;
 
-	// 1. listModels（E2：ctx 传 modelRegistry 数据源）
-	const models = deps.listModels(ctx);
+	// 1. listAvailableModels（ctx.modelRegistry 数据源）
+	const models = listAvailableModels(ctx);
 	if (models.size === 0) {
 		ctx.ui.notify(
 			"[pi-permission] No available models. Run `pi auth login` or configure provider API keys, then retry.",
@@ -168,7 +160,7 @@ export async function handlePermissionModelCommand(
 		classifier: { ...config.classifier, model: selected },
 		userRules: config.userRules.map((r) => ({ ...r })),
 	};
-	const result = deps.save(newConfig);
+	const result = save(newConfig);
 	if (!result.success) {
 		ctx.ui.notify(
 			`[pi-permission] Failed to save: ${result.error ?? "unknown error"}`,
@@ -181,35 +173,24 @@ export async function handlePermissionModelCommand(
 
 // ──────────────────────── /permission rule（W8） ────────────────────────
 
-/** handlePermissionRuleCommand 的依赖（DI 便于测试 mock）。 */
-export interface PermissionRuleCommandDeps {
-	/** 保存新配置，返回 {success, error?}。 */
-	save: (config: PermissionConfig) => { success: boolean; error?: string };
-	/** 规则编辑 overlay（注入便于测试 mock）。 */
-	editRulesViaOverlay: (
-		ctx: RuleEditorContext,
-		initialRules: readonly import("./types.js").Rule[],
-		sessionIdCounter: () => string,
-		rpcDeps?: import("./rule-editor.js").RuleEditorRpcDeps,
-	) => Promise<RuleEditorResult>;
-}
-
 /**
  * /permission rule 命令：overlay CRUD 编辑 userRules。
  *
  * 流程：
- *  1. editRulesViaOverlay：TUI/RPC/headless 三模式分发。
+ *  1. editRulesViaOverlay（模块直调）：TUI/RPC/headless 三模式分发。
  *  2. ops 非空 → applyOps + saveConfig + notify。
  *  3. 空 → notify no changes。
+ *
+ * save 为普通函数参数（唯一真实变化轴：保存 + footer 重绘副作用，与同步路径 onSave 同款）。
  */
 export async function handlePermissionRuleCommand(
 	ctx: RuleEditorContext,
 	config: PermissionConfig,
 	sessionIdCounter: () => string,
-	deps: PermissionRuleCommandDeps,
+	save: (config: PermissionConfig) => { success: boolean; error?: string },
 ): Promise<void> {
 	// 1. editRulesViaOverlay
-	const ops = await deps.editRulesViaOverlay(ctx, config.userRules, sessionIdCounter);
+	const ops = await editRulesViaOverlay(ctx, config.userRules, sessionIdCounter);
 
 	if (ops === undefined || ops.length === 0) {
 		ctx.ui.notify("[pi-permission] No changes applied.", "info");
@@ -224,7 +205,7 @@ export async function handlePermissionRuleCommand(
 		classifier: { ...config.classifier },
 		userRules: newUserRules,
 	};
-	const result = deps.save(newConfig);
+	const result = save(newConfig);
 	if (!result.success) {
 		ctx.ui.notify(
 			`[pi-permission] Failed to save rules: ${result.error ?? "unknown error"}`,

package/src/config.ts

@@ -17,11 +17,12 @@
 import { existsSync, mkdirSync, writeFileSync } from "node:fs";
 import { dirname, join } from "node:path";
 
-import type { ModelThinkingLevel } from "@earendil-works/pi-ai";
 import { getAgentDir } from "@earendil-works/pi-coding-agent";
+import { isRecord, toErrorMessage } from "@zhushanwen/pi-ext-guards";
 import {
 	clearConfigCache,
 	getConfigPath as getLlmConfigPath,
+	isThinkingLevel,
 	loadConfig,
 	saveConfig as saveLlmConfig,
 } from "@zhushanwen/pi-llm-shared";
@@ -50,26 +51,8 @@ export function getConfigPath(): string {
 
 // ──────────────────────── 归一化 ────────────────────────
 
-function isPlainObject(value: unknown): value is Record<string, unknown> {
-	return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-
-const THINKING_LEVELS: ReadonlySet<string> = new Set([
-	"off",
-	"minimal",
-	"low",
-	"medium",
-	"high",
-	"xhigh",
-	"max",
-]);
-
-function isModelThinkingLevel(value: unknown): value is ModelThinkingLevel {
-	return typeof value === "string" && THINKING_LEVELS.has(value);
-}
-
 function normalizeClassifierConfig(raw: unknown): ClassifierConfig {
-	const record = isPlainObject(raw) ? raw : {};
+	const record = isRecord(raw) ? raw : {};
 	const timeout = Number(record.timeout);
 	// C3b：classifier.model 只接受 string（'auto' 或 'provider/model-id'）。对象形式（如
 	// `{ "type": "available" }`）不受支持，此前会被静默忽略回落默认——现在显式 warn 消除静默。
@@ -77,7 +60,7 @@ function normalizeClassifierConfig(raw: unknown): ClassifierConfig {
 		logger.warn("Ignoring invalid classifier.model (expected string 'auto' or 'provider/model-id'), using default auto");
 	}
 	// thinkingLevel 校验：合法值为 'off'|'minimal'|'low'|'medium'|'high'|'xhigh'|'max'
-	const thinkingLevel = isModelThinkingLevel(record.thinkingLevel)
+	const thinkingLevel = isThinkingLevel(record.thinkingLevel)
 		? record.thinkingLevel
 		: DEFAULT_CLASSIFIER_CONFIG.thinkingLevel;
 	return {
@@ -91,7 +74,7 @@ function normalizeClassifierConfig(raw: unknown): ClassifierConfig {
 }
 
 function normalizeRule(raw: unknown, fallbackId: string): Rule | null {
-	if (!isPlainObject(raw)) return null;
+	if (!isRecord(raw)) return null;
 	const tool = typeof raw.tool === "string" ? raw.tool : "*";
 	const pattern = typeof raw.pattern === "string" ? raw.pattern : "*";
 	const action = raw.action;
@@ -103,7 +86,7 @@ function normalizeRule(raw: unknown, fallbackId: string): Rule | null {
 }
 
 function normalizeConfig(raw: unknown): PermissionConfig {
-	const record = isPlainObject(raw) ? raw : {};
+	const record = isRecord(raw) ? raw : {};
 	const mode = isValidPermissionMode(record.mode) ? record.mode : DEFAULT_CONFIG.mode;
 	const enabled = record.enabled !== false;
 	const classifier = normalizeClassifierConfig(record.classifier);
@@ -133,7 +116,7 @@ function ensureConfigFile(configPath: string, onWarning?: (msg: string) => void)
 		mkdirSync(dirname(configPath), { recursive: true });
 		writeFileSync(configPath, createDefaultConfigContent(), { encoding: "utf-8", mode: 0o600 });
 	} catch (error) {
-		const message = error instanceof Error ? error.message : String(error);
+		const message = toErrorMessage(error);
 		onWarning?.(`[pi-permission] Failed to create default config at '${configPath}': ${message}`);
 	}
 }

package/src/index.ts

@@ -16,19 +16,17 @@ import {
 	type ExtensionAPI,
 	type ExtensionCommandContext,
 	type ExtensionContext,
+	type ExtensionUIContext,
 	getAgentDir,
 } from "@earendil-works/pi-coding-agent";
-import { oncePerProcess } from "@zhushanwen/pi-ext-guards";
+import { oncePerProcess, toErrorMessage } from "@zhushanwen/pi-ext-guards";
 import { getLogger } from "@zhushanwen/pi-extension-logger";
 import { migrateLegacyConfig } from "@zhushanwen/pi-llm-shared";
 
 const logger = getLogger("pi-permission");
 
-import { listAvailableModels } from "./classifier/model-resolver.js";
 import { handlePermissionCommand, handlePermissionModelCommand, handlePermissionRuleCommand } from "./commands.js";
 import { loadAndWatchConfig, saveConfig } from "./config.js";
-import { setDefaultListAvailableModels } from "./model-picker.js";
-import { editRulesViaOverlay } from "./rule-editor.js";
 import { makeNextIdCounter } from "./rule-templates.js";
 import { checkPermission, type CheckPermissionDeps } from "./pipeline.js";
 import { createPipelineDeps } from "./production.js";
@@ -80,6 +78,39 @@ interface ToolCallResult {
  */
 const _UI_OPTIONS_PARAM_INDEX = 2;
 
+// ──────────────────────── ctx.ui 适配（E8/M11 单源） ────────────────────────
+
+/**
+ * E8（M11）：pi ctx.ui → 本包局部 UI 上下文的统一适配（原三份闭包收敛为单源）。
+ *
+ * 三处消费：/permission rule（RuleEditorContext.ui）、/permission model
+ * （ModelPickerContext.ui）、tool_call 审批（ApprovalContext.ui）。三个目标接口的
+ * custom 泛型形态不同（前两者 factory 返回 unknown、后者返回 Component），统一靠
+ * `as Parameters<typeof ui.custom<T>>[0]` cast 吸收（T5）。
+ *
+ * select/input 的 opts 类型经 `Parameters<typeof ...>[typeof _UI_OPTIONS_PARAM_INDEX]`
+ * 索引提取（字面量索引会触发 no-magic-numbers，typeof 常量索引语义等价）。
+ * input 可选守卫展开：SDK 提供，mock/headless 可能缺失——rule-editor custom 模板
+ * 文本输入与 Reject-with-Reason（approval.ts collectRejectReason）都依赖此透传。
+ */
+function makeUiAdapter(ui: ExtensionUIContext) {
+	return {
+		notify: (msg: string, type?: "info" | "warning" | "error") => ui.notify(msg, type),
+		select: (title: string, options: string[], opts?: Parameters<typeof ui.select>[typeof _UI_OPTIONS_PARAM_INDEX]) =>
+			ui.select(title, options, opts),
+		custom: <T,>(
+			factory: (tui: unknown, theme: unknown, kb: unknown, done: (result: T) => void) => unknown,
+			options?: { overlay?: boolean },
+		): Promise<T> => ui.custom<T>(factory as Parameters<typeof ui.custom<T>>[0], options),
+		...(typeof ui.input === "function"
+			? {
+					input: (title: string, placeholder?: string, opts?: Parameters<typeof ui.input>[typeof _UI_OPTIONS_PARAM_INDEX]) =>
+						ui.input(title, placeholder, opts),
+				}
+			: {}),
+	};
+}
+
 // ──────────────────────── 扩展工厂 ────────────────────────
 
 /**
@@ -87,10 +118,6 @@ const _UI_OPTIONS_PARAM_INDEX = 2;
  * （llm-shared mtime 去重零成本，详见 config.ts「热重载契约」）。
  */
 export default function permissionExtension(pi: ExtensionAPI): void {
-	// W7：注入 listAvailableModels 真实实现（model-picker.ts 默认返回空 Map）。
-	// E2 签名：(ctx) → ctx.modelRegistry.getAll() + hasConfiguredAuth 过滤。
-	setDefaultListAvailableModels((ctx) => listAvailableModels(ctx));
-
 	// ──────────────────────── 配置读取（读时刷新，回归 llm-shared 框架） ────────────────────────
 	// 不持有跨调用缓存：每次需要配置直接 loadAndWatchConfig()，llm-shared 内部 mtime+size 去重，
 	// 文件未变时零额外 IO（只 statSync）。这是 llm-shared config「热重载契约」的正确用法——
@@ -140,32 +167,14 @@ export default function permissionExtension(pi: ExtensionAPI): void {
 				await handlePermissionRuleCommand(
 					{
 						mode: ctx.mode,
-						ui: {
-							notify: (msg: string, type?: "info" | "warning" | "error") => ctx.ui.notify(msg, type),
-							select: (title: string, options: string[], opts?: Parameters<typeof ctx.ui.select>[typeof _UI_OPTIONS_PARAM_INDEX]) =>
-								ctx.ui.select(title, options, opts),
-							custom: <T,>(
-								factory: (tui: unknown, theme: unknown, kb: unknown, done: (result: T) => void) => unknown,
-								options?: { overlay?: boolean },
-							) =>
-								ctx.ui.custom<T>(factory as Parameters<typeof ctx.ui.custom<T>>[0], options),
-							// 连接 ctx.ui.input（rule-editor custom 模板文本输入用）。
-							// approval.ts 已声明可选 input（SDK 提供，mock 可能缺失）。
-							...(typeof ctx.ui.input === "function"
-								? { input: (title: string, placeholder?: string, opts?: Parameters<typeof ctx.ui.input>[typeof _UI_OPTIONS_PARAM_INDEX]) => ctx.ui.input(title, placeholder, opts) }
-								: {}),
-						},
+						ui: makeUiAdapter(ctx.ui),
 					},
 					config,
 					makeNextIdCounter(config.userRules),
-					{
-						save: (newConfig) => {
-							const r = saveConfig(newConfig);
-							if (r.success) requestFooterRender();
-							return r;
-						},
-						editRulesViaOverlay: (ctx, initialRules, sessionIdCounter, rpcDeps) =>
-							editRulesViaOverlay(ctx, initialRules, sessionIdCounter, rpcDeps),
+					(newConfig) => {
+						const r = saveConfig(newConfig);
+						if (r.success) requestFooterRender();
+						return r;
 					},
 				);
 				return;
@@ -177,29 +186,14 @@ export default function permissionExtension(pi: ExtensionAPI): void {
 						mode: ctx.mode,
 						// E2：model picker 数据源（listAvailableModels 走 modelRegistry）
 						modelRegistry: ctx.modelRegistry,
-						ui: {
-							notify: (msg: string, type?: "info" | "warning" | "error") => ctx.ui.notify(msg, type),
-							select: (title: string, options: string[], opts?: Parameters<typeof ctx.ui.select>[typeof _UI_OPTIONS_PARAM_INDEX]) =>
-								ctx.ui.select(title, options, opts),
-							custom: <T,>(
-								factory: (tui: unknown, theme: unknown, kb: unknown, done: (result: T) => void) => unknown,
-								options?: { overlay?: boolean },
-							) =>
-								ctx.ui.custom<T>(factory as Parameters<typeof ctx.ui.custom<T>>[0], options),
-							// 连接 ctx.ui.input（与 rule handler 一致；model picker 当前不用，但保持 ctx 对称）。
-							...(typeof ctx.ui.input === "function"
-								? { input: (title: string, placeholder?: string, opts?: Parameters<typeof ctx.ui.input>[typeof _UI_OPTIONS_PARAM_INDEX]) => ctx.ui.input(title, placeholder, opts) }
-								: {}),
-						},
+						// input 透传保持 ctx 对称（model picker 当前不用，rule handler 用）
+						ui: makeUiAdapter(ctx.ui),
 					},
 					config,
-					{
-						listModels: (pickerCtx) => listAvailableModels(pickerCtx),
-						save: (newConfig) => {
-							const r = saveConfig(newConfig);
-							if (r.success) requestFooterRender();
-							return r;
-						},
+					(newConfig) => {
+						const r = saveConfig(newConfig);
+						if (r.success) requestFooterRender();
+						return r;
 					},
 				);
 				return;
@@ -320,20 +314,7 @@ async function processToolCall(
 	// 装配 deps（每次 tool_call 重新装配，捕获当前 ctx.mode/ui；classifier 走 ctx.modelRegistry）
 	const approvalCtx = {
 		mode: ctx.mode,
-		ui: {
-			notify: (msg: string, type?: "info" | "warning" | "error") => ctx.ui.notify(msg, type),
-			select: (title: string, options: string[], opts?: Parameters<typeof ctx.ui.select>[typeof _UI_OPTIONS_PARAM_INDEX]) => ctx.ui.select(title, options, opts),
-			custom: <T,>(
-				factory: (tui: unknown, theme: unknown, kb: unknown, done: (result: T) => void) => unknown,
-				options?: { overlay?: boolean },
-			) =>
-				ctx.ui.custom<T>(factory as Parameters<typeof ctx.ui.custom<T>>[0], options),
-			// W6 T9 G3：Reject-with-Reason。ctx.ui.input 存在则透传（采集真实拒绝理由）。
-			// approval.ts 的 collectRejectReason 会用 typeof 判断是否可用，不可用则 fallback。
-			...(typeof ctx.ui.input === "function"
-				? { input: (title: string, placeholder?: string, opts?: Parameters<typeof ctx.ui.input>[typeof _UI_OPTIONS_PARAM_INDEX]) => ctx.ui.input(title, placeholder, opts) }
-				: {}),
-		},
+		ui: makeUiAdapter(ctx.ui),
 	};
 	const deps: CheckPermissionDeps = createPipelineDeps(approvalCtx, ctx);
 
@@ -360,7 +341,7 @@ async function processToolCall(
 		};
 	} catch (error) {
 		// fail-closed：异常 → block + reason（绝不放行）
-		const msg = error instanceof Error ? error.message : String(error);
+		const msg = toErrorMessage(error);
 		logger.warn("tool_call handler exception", { toolName, error: msg });
 		return {
 			block: true,