@zhushanwen/pi-permission 1.3.4 → 1.4.0

package/src/classifier/classifier.ts

@@ -8,7 +8,7 @@
  * 关键设计：
  *   - 依赖注入 ClassifierDeps（callLLM 注入，便于测试 mock，不直接 import）
  *   - C1a 收口：LLM 调用走 llm-shared callLLM（内部完成凭证 getApiKeyAndHeaders +
- *     completeSimple + stopReason 归一化：error/aborted → {ok:false, recoverable:true,
+ *     completeSimple + stopReason 归一化：error/aborted → {ok:false,
  *     stopReason 透传}），不再走 production.ts 的 streamSimple
  *   - G3 修正（由 callLLM 承接）：completeSimple 的 EventStream.result() 只 resolve 不 reject，
  *     error/aborted 也 resolve（带 stopReason）。callLLM 已把 error/aborted 归一为 ok:false +
@@ -90,7 +90,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), recoverable: true },
+				result: { ok: false as const, error: error instanceof Error ? error.message : String(error) },
 			}),
 		),
 	];

package/src/classifier/model-resolver.ts

@@ -64,14 +64,11 @@ export interface ResolvedModelEntry {
  * 「无可选模型」提示）。
  *
  * @param ctx 含 modelRegistry 的上下文（model 列表 + auth 判定）
- * @param onWarning 保留参数位（契约兼容；E2 后无文件读取，无警告源）
  * @returns Map<providerName, ResolvedModelEntry[]>
  */
 export function listAvailableModels(
 	ctx: ListAvailableModelsCtx,
-	onWarning?: (msg: string) => void,
 ): Map<string, ResolvedModelEntry[]> {
-	void onWarning; // 无警告源（不再读文件），保留参数位兼容调用方
 	const entries: ResolvedModelEntry[] = [];
 	for (const m of ctx.modelRegistry.getAll()) {
 		if (!ctx.modelRegistry.hasConfiguredAuth(m)) continue;

package/src/index.ts

@@ -18,6 +18,7 @@ import {
 	type ExtensionContext,
 	getAgentDir,
 } from "@earendil-works/pi-coding-agent";
+import { oncePerProcess } from "@zhushanwen/pi-ext-guards";
 import { getLogger } from "@zhushanwen/pi-extension-logger";
 import { migrateLegacyConfig } from "@zhushanwen/pi-llm-shared";
 
@@ -44,8 +45,9 @@ import { paletteFromTheme, type PermissionPalette } from "./statusline-palette.j
 // Added in v1.0.0. Remove after v2.0.0 (one major past).
 // session_start 首次触发时把旧路径 permission-config.json 迁到 config/permission-ext-config.json。
 // 幂等、best-effort（migrateLegacyConfig 见 @zhushanwen/pi-llm-shared）。
-// 模块级 once flag 防同进程重复触发；agentDir 由 getAgentDir() 推导（尊重 PI_CODING_AGENT_DIR）。
-let configMigrationChecked = false;
+// 写 agentDir 全局配置文件属跨 session 副作用——oncePerProcess 守卫防 factory 二调/handler
+// 累积双跑（u-audit-fix 收编原内联 once flag；key 带包名前缀避撞，见 pi-ext-guards JSDoc）。
+// agentDir 由 getAgentDir() 推导（尊重 PI_CODING_AGENT_DIR）。
 
 /** 默认配置 warning 回调（loadAndWatchConfig 共用，透传 logger.warn）。 */
 const defaultConfigWarn = (msg: string): void => logger.warn(msg);
@@ -86,10 +88,8 @@ const _UI_OPTIONS_PARAM_INDEX = 2;
  */
 export default function permissionExtension(pi: ExtensionAPI): void {
 	// W7：注入 listAvailableModels 真实实现（model-picker.ts 默认返回空 Map）。
-	// E2 签名：(ctx, onWarning?) → ctx.modelRegistry.getAll() + hasConfiguredAuth 过滤；warning 透传到 console.warn。
-	setDefaultListAvailableModels((ctx, onWarning) =>
-		listAvailableModels(ctx, onWarning ?? defaultConfigWarn),
-	);
+	// E2 签名：(ctx) → ctx.modelRegistry.getAll() + hasConfiguredAuth 过滤。
+	setDefaultListAvailableModels((ctx) => listAvailableModels(ctx));
 
 	// ──────────────────────── 配置读取（读时刷新，回归 llm-shared 框架） ────────────────────────
 	// 不持有跨调用缓存：每次需要配置直接 loadAndWatchConfig()，llm-shared 内部 mtime+size 去重，
@@ -100,10 +100,9 @@ export default function permissionExtension(pi: ExtensionAPI): void {
 	// ──────────────────────── session_start：迁移旧路径配置 ────────────────────────
 	pi.on("session_start", (_event: unknown, ctx: ExtensionContext) => {
 		// [MIGRATION] Added in v1.0.0. Remove after v2.0.0.
-		if (!configMigrationChecked) {
-			configMigrationChecked = true;
+		oncePerProcess("permission:migrate-legacy-config", () => {
 			migrateLegacyConfig(getAgentDir(), "permission-config.json", "config/permission-ext-config.json");
-		}
+		});
 		// 无需手动刷新：去闭包后每次 loadAndWatchConfig 读时刷新；迁移改写文件后下次读取自动生效。
 		// 注册 footer line renderer（consumer 端握手，pi-statusline 是 canonical owner）。
 		// renderer 无状态：render 时用 statusline 传入的 theme + loadAndWatchConfig 读最新 config。
@@ -195,7 +194,7 @@ export default function permissionExtension(pi: ExtensionAPI): void {
 					},
 					config,
 					{
-						listModels: (pickerCtx) => listAvailableModels(pickerCtx, (m) => logger.warn(m)),
+						listModels: (pickerCtx) => listAvailableModels(pickerCtx),
 						save: (newConfig) => {
 							const r = saveConfig(newConfig);
 							if (r.success) requestFooterRender();

package/src/model-picker.ts

@@ -23,6 +23,10 @@ import { type Component, Container, type SelectItem, SelectList, type SelectList
 import type { Api, Model } from "@earendil-works/pi-ai";
 
 import type { ResolvedModelEntry } from "./classifier/model-resolver.js";
+import { DEFAULT_SELECT_THEME } from "./select-theme.js";
+
+// 保持既有 public 名（model-picker.test.ts MPT7 直接消费此路径）。
+export { DEFAULT_SELECT_THEME };
 
 // ──────────────────────── 类型 ────────────────────────
 
@@ -31,22 +35,6 @@ export type SelectionResult =
 	| { kind: "auto" }
 	| { kind: "specific"; provider: string; modelId: string };
 
-// ──────────────────────── DEFAULT_SELECT_THEME（G2 修正） ────────────────────────
-
-/**
- * SelectList 默认主题。
- *
- * G2/WR2 修正：selectedPrefix 实现为 `(t) => '▶ ' + t`（非 identity），
- * 选中行有视觉区分（▶ 前缀）。
- */
-export const DEFAULT_SELECT_THEME: SelectListTheme = {
-	selectedPrefix: (t: string): string => "\u25B6 " + t,
-	selectedText: (t: string): string => t,
-	description: (t: string): string => t,
-	scrollInfo: (t: string): string => t,
-	noMatch: (t: string): string => t,
-};
-
 /** provider stage 的 'auto' 项 value（用特殊标记区分真实 provider 名）。 */
 const AUTO_VALUE = "__auto__";
 
@@ -189,7 +177,7 @@ export class ProviderModelSelectorComponent extends Container {
 	/** provider stage items：'Auto' + 各 provider（含 model 数量）。 */
 	private buildProviderItems(): SelectItem[] {
 		const items: SelectItem[] = [
-			{ value: AUTO_VALUE, label: "Auto", description: "auto-select cheapest available model" },
+			{ value: AUTO_VALUE, label: "Auto", description: "select first available model" },
 		];
 		for (const provider of this.providers) {
 			const models = this.modelsByProvider.get(provider) ?? [];
@@ -297,7 +285,6 @@ export interface ModelPickerContext {
 /** listAvailableModels 注入签名（E2 改走 ctx.modelRegistry，签名带 ctx；便于测试 mock）。 */
 export type ListAvailableModelsFn = (
 	ctx: ModelPickerContext,
-	onWarning?: (msg: string) => void,
 ) => Map<string, ResolvedModelEntry[]>;
 
 /**

package/src/pipeline.ts

@@ -203,95 +203,140 @@ export function applyAutoApproveOverrides(
 
 // ──────────────────────── runLayer3WithRacing（G3 abort 时序） ────────────────────────
 
+/** 外层 abort 传播到内层 controller（session abort 时级联取消 AI + 用户审批）。 */
+function linkOuterSignal(controller: AbortController, outerSignal: AbortSignal | undefined): void {
+	if (outerSignal) {
+		if (outerSignal.aborted) controller.abort();
+		else outerSignal.addEventListener("abort", () => controller.abort(), { once: true });
+	}
+}
+
 /**
- * 层 3：AI Classifier + 用户审批竞速（移植 pi-permission-system classifier-racing.ts）。
- *
- * 竞速语义：
- *  - 启动 AI 分类（aiPromise）+ 用户审批（userPromise）。
- *  - 用户先返回 → controller.abort() 取消 AI（已无用的 AI 计算尽早终止）。
- *  - AI 先返回：
- *      - AI outcome='allow' → resolveUser({approved:false}) 关闭用户对话框，返回 allow。
- *      - AI outcome='deny'  → resolveUser({approved:false}) 关闭用户对话框，返回 deny。
- *      - AI outcome='ask'   → 不 resolveUser，等用户最终决策（AI 不确定时转人工）。
- *
- * G3 修正：requestUserApproval（TUI 分支）在调 ctx.ui.custom 前先检查 signal.aborted，
- * 避免 AI 先于 UI factory 执行时 controller.abort() 触发但 comp 尚未创建导致 cancel 落空。
- * 此处再叠一层：AI 赢且 outcome 是 allow/deny 时，先 abort 再 resolveUser，
- * 保证 UI factory 看到 aborted 状态短路。
- *
- * @param deps 注入依赖（classifier + requestUserApproval）
- * @param ctx 工具调用上下文
- * @param config classifier 配置
- * @param outerSignal 外层 signal（session abort 时传播）
- * @param trigger 审批触发原因（m2：AST 检测到的具体危险结构等；默认通用文案）
- * @returns PermissionDecision（source='ai' 或 'user'）
+ * 启动 AI 分类 promise。
+ * M4：aiPromise 挂 .catch 转 fallback（确保永不 reject）。
+ * classifier 内部有 try/catch，但 resolveModel/buildModel/buildContext 在 try 外，
+ * 用户先返回 abort 后 classifier 晚 reject → race 已 settle → unhandledRejection。
  */
-export async function runLayer3WithRacing(
+function startAiClassification(
 	deps: CheckPermissionDeps,
 	ctx: ToolInvocationContext,
 	config: ClassifierConfig,
-	outerSignal: AbortSignal | undefined,
-	trigger: string = "awaiting approval (auto mode: AI classifier racing with user prompt)",
+	signal: AbortSignal,
+): Promise<ClassifierResult> {
+	return deps.classifier.classifyRisk(ctx, config, signal).catch((err: unknown) => {
+		const msg = err instanceof Error ? err.message : String(err);
+		return {
+			outcome: "ask" as const,
+			risk_level: "medium" as const,
+			reasoning: `classifier error: ${msg}`,
+			confidence: 0,
+		};
+	});
+}
+
+/** AI 判定 → PermissionDecision 信封（source='ai'，透传 overridden 的 reasoning/风险/置信度）。 */
+function aiDecision(action: PermissionAction, overridden: ClassifierResult): PermissionDecision {
+	return {
+		action,
+		reason: overridden.reasoning,
+		source: "ai",
+		riskLevel: overridden.risk_level,
+		confidence: overridden.confidence,
+	};
+}
+
+/**
+ * M1：headless 模式（json/print）下纯等 AI，不启动 user promise。
+ * 否则 requestHeadless 立即 deny 会抢占 race → AI 永远没机会赢 → auto 退化为 strict。
+ * AI 返回 allow/deny → 按结果返回；AI 返回 ask/超时/fail → fail-closed deny。
+ */
+async function settleHeadlessAi(
+	aiPromise: Promise<ClassifierResult>,
+	controller: AbortController,
+	config: ClassifierConfig,
 ): Promise<PermissionDecision> {
-	const controller = new AbortController();
-	// 外层 abort 传播到内层
-	if (outerSignal) {
-		if (outerSignal.aborted) controller.abort();
-		else outerSignal.addEventListener("abort", () => controller.abort(), { once: true });
+	const aiResult = await aiPromise;
+	const overridden = applyAutoApproveOverrides(aiResult, config);
+	if (overridden.outcome === "allow") {
+		controller.abort();
+		return aiDecision("allow", overridden);
+	}
+	if (overridden.outcome === "deny") {
+		controller.abort();
+		return aiDecision("deny", overridden);
 	}
+	// AI ask → headless 无 UI 可问 → fail-closed deny
+	controller.abort();
+	return {
+		action: "deny",
+		reason: "headless mode: AI inconclusive (ask), fail-closed deny",
+		source: "ai",
+		riskLevel: overridden.risk_level,
+		confidence: overridden.confidence,
+	};
+}
 
-	// M4：aiPromise 挂 .catch 转 fallback（确保永不 reject）。
-	// classifier 内部有 try/catch，但 resolveModel/buildModel/buildContext 在 try 外，
-	// 用户先返回 abort 后 classifier 晚 reject → race 已 settle → unhandledRejection。
-	const aiPromise = deps.classifier
-		.classifyRisk(ctx, config, controller.signal)
-		.catch((err: unknown) => {
-			const msg = err instanceof Error ? err.message : String(err);
-			return {
-				outcome: "ask" as const,
-				risk_level: "medium" as const,
-				reasoning: `classifier error: ${msg}`,
-				confidence: 0,
-			};
-		});
-
-	// M1：headless 模式（json/print）下纯等 AI，不启动 user promise。
-	// 否则 requestHeadless 立即 deny 会抢占 race → AI 永远没机会赢 → auto 退化为 strict。
-	// AI 返回 allow/deny → 按结果返回；AI 返回 ask/超时/fail → fail-closed deny。
-	if (deps.isHeadless()) {
-		const aiResult = await aiPromise;
-		const overridden = applyAutoApproveOverrides(aiResult, config);
-		if (overridden.outcome === "allow") {
-			controller.abort();
-			return {
-				action: "allow",
-				reason: overridden.reasoning,
-				source: "ai",
-				riskLevel: overridden.risk_level,
-				confidence: overridden.confidence,
-			};
-		}
-		if (overridden.outcome === "deny") {
-			controller.abort();
-			return {
-				action: "deny",
-				reason: overridden.reasoning,
-				source: "ai",
-				riskLevel: overridden.risk_level,
-				confidence: overridden.confidence,
-			};
-		}
-		// AI ask → headless 无 UI 可问 → fail-closed deny
+/**
+ * AI 赢（非 ask）分支：先 abort 再 resolveUser 关闭用户对话框，返回 AI 决策
+ * （G3：先 abort 再 resolveUser，保证 UI factory 看到 aborted 状态短路）。
+ * 返回 null = AI ask → 转人工（不 resolveUser，等用户最终决策）。
+ */
+function settleAiWin(
+	overridden: ClassifierResult,
+	controller: AbortController,
+	resolveUser: (d: UserDecision) => void,
+): PermissionDecision | null {
+	if (overridden.outcome === "allow") {
+		// AI 放行 → 关闭用户对话框（若已弹出），返回 allow
 		controller.abort();
-		return {
-			action: "deny",
-			reason: "headless mode: AI inconclusive (ask), fail-closed deny",
-			source: "ai",
-			riskLevel: overridden.risk_level,
-			confidence: overridden.confidence,
-		};
+		resolveUser({ approved: false, reason: "AI classifier allowed; dialog dismissed" });
+		return aiDecision("allow", overridden);
+	}
+	if (overridden.outcome === "deny") {
+		controller.abort();
+		resolveUser({ approved: false, reason: "AI classifier denied; dialog dismissed" });
+		return aiDecision("deny", overridden);
 	}
+	return null;
+}
 
+/**
+ * AI ask → 转 human（等用户最终决策，不关闭对话框）。
+ * M3：超时兜底防止永久挂起。若 AI 返回 ask 后用户恰好与 AI 在同一 tick 操作，
+ * comp.cancel() 因 _resolved 守卫不调 done → requestUserApproval promise 永不
+ * resolve → realUserPromise pending → 永久挂起（G5 串行化放大为全链卡死）。
+ * 5 分钟超时后 fail-closed 拒绝（不静默放行）。
+ */
+async function awaitUserAfterAiAsk(
+	realUserPromise: Promise<UserDecision>,
+): Promise<PermissionDecision> {
+	const APPROVAL_TIMEOUT_MS = 300_000;
+	const userFinal = await Promise.race<UserDecision>([
+		realUserPromise,
+		new Promise<UserDecision>((resolve) =>
+			setTimeout(
+				() => resolve({ approved: false, reason: "approval dialog timeout (fail-closed)" }),
+				APPROVAL_TIMEOUT_MS,
+			),
+		),
+	]);
+	const action: PermissionAction = userFinal.approved ? "allow" : "deny";
+	return {
+		action,
+		reason: userFinal.reason ?? (userFinal.approved ? "approved by user (after AI ask)" : "denied by user (after AI ask)"),
+		source: "user",
+	};
+}
+
+/** 竞速主体（TUI 形态）：AI vs 用户并行，race 后按先 settle 的一侧出决策。 */
+async function raceAiWithUser(
+	deps: CheckPermissionDeps,
+	ctx: ToolInvocationContext,
+	config: ClassifierConfig,
+	aiPromise: Promise<ClassifierResult>,
+	controller: AbortController,
+	trigger: string,
+): Promise<PermissionDecision> {
 	// 用户审批 promise：可被 resolveUser 外部 resolve（AI 赢时关闭对话框）
 	let resolveUser: (d: UserDecision) => void = () => {
 		/* 占位，下方立即覆写 */
@@ -339,50 +384,60 @@ export async function runLayer3WithRacing(
 
 	// AI 赢
 	const overridden = applyAutoApproveOverrides(aiSettled.result, config);
-	if (overridden.outcome === "allow") {
-		// AI 放行 → 关闭用户对话框（若已弹出），返回 allow
-		controller.abort();
-		resolveUser({ approved: false, reason: "AI classifier allowed; dialog dismissed" });
-		return {
-			action: "allow",
-			reason: overridden.reasoning,
-			source: "ai",
-			riskLevel: overridden.risk_level,
-			confidence: overridden.confidence,
-		};
+	const decided = settleAiWin(overridden, controller, resolveUser);
+	if (decided !== null) return decided;
+	return await awaitUserAfterAiAsk(realUserPromise);
+}
+
+/**
+ * 层 3：AI Classifier + 用户审批竞速（移植 pi-permission-system classifier-racing.ts）。
+ *
+ * 竞速语义：
+ *  - 启动 AI 分类（aiPromise）+ 用户审批（userPromise）。
+ *  - 用户先返回 → controller.abort() 取消 AI（已无用的 AI 计算尽早终止）。
+ *  - AI 先返回：
+ *      - AI outcome='allow' → resolveUser({approved:false}) 关闭用户对话框，返回 allow。
+ *      - AI outcome='deny'  → resolveUser({approved:false}) 关闭用户对话框，返回 deny。
+ *      - AI outcome='ask'   → 不 resolveUser，等用户最终决策（AI 不确定时转人工）。
+ *
+ * G3 修正：requestUserApproval（TUI 分支）在调 ctx.ui.custom 前先检查 signal.aborted，
+ * 避免 AI 先于 UI factory 执行时 controller.abort() 触发但 comp 尚未创建导致 cancel 落空。
+ * 此处再叠一层：AI 赢且 outcome 是 allow/deny 时，先 abort 再 resolveUser，
+ * 保证 UI factory 看到 aborted 状态短路。
+ *
+ * classifier.enabled=false：AI 层被用户显式关闭 → 跳过 classifier 与 racing，直接人工审批
+ * （复用 askUser 路径）。headless 下 requestUserApproval 走 requestHeadless 立即 fail-closed
+ * deny，与 AI 判定 ask 时的短路 deny 语义一致。
+ *
+ * @param deps 注入依赖（classifier + requestUserApproval）
+ * @param ctx 工具调用上下文
+ * @param config classifier 配置
+ * @param outerSignal 外层 signal（session abort 时传播）
+ * @param trigger 审批触发原因（m2：AST 检测到的具体危险结构等；默认通用文案）
+ * @returns PermissionDecision（source='ai' 或 'user'）
+ */
+export async function runLayer3WithRacing(
+	deps: CheckPermissionDeps,
+	ctx: ToolInvocationContext,
+	config: ClassifierConfig,
+	outerSignal: AbortSignal | undefined,
+	trigger: string = "awaiting approval (auto mode: AI classifier racing with user prompt)",
+): Promise<PermissionDecision> {
+	// classifier.enabled=false → AI 层关闭，直接人工审批（不启动 AI promise，不进入 racing）
+	if (config.enabled === false) {
+		return await askUser(deps, ctx, "AI classifier disabled (classifier.enabled=false)", outerSignal);
 	}
-	if (overridden.outcome === "deny") {
-		controller.abort();
-		resolveUser({ approved: false, reason: "AI classifier denied; dialog dismissed" });
-		return {
-			action: "deny",
-			reason: overridden.reasoning,
-			source: "ai",
-			riskLevel: overridden.risk_level,
-			confidence: overridden.confidence,
-		};
+	const controller = new AbortController();
+	// 外层 abort 传播到内层
+	linkOuterSignal(controller, outerSignal);
+
+	const aiPromise = startAiClassification(deps, ctx, config, controller.signal);
+
+	// M1：headless 模式下纯等 AI（不启动 user promise）
+	if (deps.isHeadless()) {
+		return await settleHeadlessAi(aiPromise, controller, config);
 	}
-	// AI ask → 转 human（等用户最终决策，不关闭对话框）
-	// M3：超时兜底防止永久挂起。若 AI 返回 ask 后用户恰好与 AI 在同一 tick 操作，
-	// comp.cancel() 因 _resolved 守卫不调 done → requestUserApproval promise 永不
-	// resolve → realUserPromise pending → 永久挂起（G5 串行化放大为全链卡死）。
-	// 5 分钟超时后 fail-closed 拒绝（不静默放行）。
-	const APPROVAL_TIMEOUT_MS = 300_000;
-	const userFinal = await Promise.race<UserDecision>([
-		realUserPromise,
-		new Promise<UserDecision>((resolve) =>
-			setTimeout(
-				() => resolve({ approved: false, reason: "approval dialog timeout (fail-closed)" }),
-				APPROVAL_TIMEOUT_MS,
-			),
-		),
-	]);
-	const action: PermissionAction = userFinal.approved ? "allow" : "deny";
-	return {
-		action,
-		reason: userFinal.reason ?? (userFinal.approved ? "approved by user (after AI ask)" : "denied by user (after AI ask)"),
-		source: "user",
-	};
+	return await raceAiWithUser(deps, ctx, config, aiPromise, controller, trigger);
 }
 
 // ──────────────────────── checkPermission（纯函数主入口） ────────────────────────
@@ -392,7 +447,8 @@ export async function runLayer3WithRacing(
  *
  * 四档模式分支：
  *  - yolo：完全放行（return allow，source='mode'）。不跑任何层。
- *  - auto：AST → 规则（allow 通过 / deny 拒绝 / ask → 层 3 Racing AI+用户）。
+ *  - auto：AST → 规则（allow 通过 / deny 拒绝 / ask → 层 3 Racing AI+用户；
+ *    classifier.enabled=false 时 ask → 直接人工审批，无 AI）。
  *  - approve：AST → 规则（allow 通过 / deny+ask → 人工审批，无 AI）。
  *  - strict：全部人工审批（不跑 AST/规则/AI）。
  *

package/src/production.ts

@@ -2,8 +2,8 @@
  * 生产装配层（W5 集成层）。
  *
  * P3 收口：classifier 的 model 解析改走 llm-shared resolveModel（三源合并含 OAuth/env/auth.json），
- * 废弃自读 models.json。'auto' 保留作 scoped 别名（向后兼容），scoped 空时 fallback
- * available（CL-scoped-fallback）。
+ * 废弃自读 models.json。'auto' 由 permission 本地处理：直接取 ctx.modelRegistry.getAvailable()[0]
+ * （不经过 llm-shared 的非精确 selector），与下方 createProductionClassifier JSDoc 一致。
  *
  * C1a 收口：classifier 的 LLM 调用改走 llm-shared callLLM（凭证 getApiKeyAndHeaders +
  * completeSimple + stopReason 归一化都在 callLLM 内部完成），删除 getApiProvider +

package/src/rule-editor-component.ts

@@ -8,7 +8,7 @@
  * G8：build 不生成 id。
  * G9：tool 只列 bash。
  * G13：sessionIdCounter 立即赋值真实 id。
- * WR8：focusIndex 模 5（Custom form 5 项 Tab 循环）。
+ * WR8：focusIndex 以 customChildren.length 为模（New 模式 5 项、Edit 模式 6 项 Tab 循环）。
  */
 
 import { type Component, Container, Input, isFocusable, type SelectItem, SelectList, type SelectListTheme, truncateToWidth } from "@earendil-works/pi-tui";
@@ -23,16 +23,7 @@ import {
 	type TemplateSelections,
 } from "./rule-templates.js";
 import type { PermissionAction, Rule } from "./types.js";
-
-// ──────────────────────── DEFAULT_SELECT_THEME ────────────────────────
-
-const THEME: SelectListTheme = {
-	selectedPrefix: (t: string): string => "\u25B6 " + t,
-	selectedText: (t: string): string => t,
-	description: (t: string): string => t,
-	scrollInfo: (t: string): string => t,
-	noMatch: (t: string): string => t,
-};
+import { DEFAULT_SELECT_THEME } from "./select-theme.js";
 
 const MAX_VISIBLE = 12;
 
@@ -111,7 +102,7 @@ export class RuleEditorComponent extends Container {
 	/** deny-family 子阶段：'cmd' = 选命令，'subcmd' = 选子命令。 */
 	private denySubStage: "cmd" | "subcmd" = "cmd";
 
-	/** Custom form：5 项焦点索引（WR8：focusIndex 模 5）。 */
+	/** Custom form 焦点索引（WR8：Tab 循环以 customChildren.length 为模）。 */
 	private customFocusIndex = 0;
 	private customChildren: Component[] = [];
 
@@ -124,7 +115,7 @@ export class RuleEditorComponent extends Container {
 		initialRules: readonly Rule[],
 		sessionIdCounter: () => string,
 		done: (ops: RuleOp[]) => void,
-		theme: SelectListTheme = THEME,
+		theme: SelectListTheme = DEFAULT_SELECT_THEME,
 	) {
 		super();
 		this.initialRules = initialRules;

package/src/rule-editor.ts

@@ -244,7 +244,7 @@ async function rpcSelectCommand(
 	return preset?.cmd ?? cmdChoice;
 }
 
-/** RPC edit flow：选择 action（edit fields / delete）。 */
+/** RPC edit flow：仅支持删除已有规则（Delete / Cancel，不支持改字段）。 */
 async function rpcEditFlow(
 	select: (title: string, options: string[]) => Promise<string | undefined>,
 	_input: (title: string, placeholder?: string) => Promise<string | undefined>,

package/src/rule-templates.ts

@@ -1,5 +1,5 @@
 /**
- * W8: Rule templates + preset commands + applyOps + classifyRuleTemplate.
+ * W8: Rule templates + preset commands + applyOps.
  *
  * G4：pattern 用 wildcard（`npm *`），不用正则（`\bnpm\b`）。
  * W3 matcher 对 user 规则走 `wildcardToRegExp(pattern)`，末尾 ` *` 匹配无参 + 有参。
@@ -85,20 +85,6 @@ export const PRESET_COMMANDS: readonly PresetCommand[] = [
 	{ cmd: "date", label: "date (system date)", category: "System & Info" },
 ] as const;
 
-/** PRESET_COMMANDS 按 category 分组（UI 展示用）。 */
-export function presetCommandsByCategory(): Map<string, PresetCommand[]> {
-	const map = new Map<string, PresetCommand[]>();
-	for (const cmd of PRESET_COMMANDS) {
-		const list = map.get(cmd.category);
-		if (list) {
-			list.push(cmd);
-		} else {
-			map.set(cmd.category, [cmd]);
-		}
-	}
-	return map;
-}
-
 // ──────────────────────── RuleTemplate ────────────────────────
 
 /** 模板 selections 值类型（各字段按模板分支含义不同）。 */
@@ -234,43 +220,6 @@ export const ALL_TEMPLATES: readonly RuleTemplate[] = [
 	customTemplate,
 ] as const;
 
-// ──────────────────────── classifyRuleTemplate ────────────────────────
-
-/**
- * FAMILY_RE：`<cmd> *` 形式（allow-family / deny-family any / ask-before）。
- * SUBCMD_RE：`<cmd> <subcmd> *` 形式（deny-family specific / allow-subcmd）。
- */
-const FAMILY_RE = /^([^ *?]+) \*$/;
-const SUBCMD_RE = /^([^ *?]+) ([^ *?]+) \*$/;
-
-/**
- * 根据 rule 的 pattern + action 推断其模板 id。
- * 无法推断时返回 undefined（如 Custom 规则）。
- */
-export function classifyRuleTemplate(rule: Rule): string | undefined {
-	const { pattern, action } = rule;
-
-	const subMatch = SUBCMD_RE.exec(pattern);
-	if (subMatch !== null) {
-		// `<cmd> <subcmd> *`
-		if (action === "deny") return "deny-family";
-		if (action === "allow") return "allow-subcmd";
-		return undefined;
-	}
-
-	const famMatch = FAMILY_RE.exec(pattern);
-	if (famMatch !== null) {
-		// `<cmd> *`
-		if (action === "allow") return "allow-family";
-		if (action === "deny") return "deny-family";
-		if (action === "ask") return "ask-before";
-		return undefined;
-	}
-
-	// 不匹配任何已知 pattern 格式 → custom
-	return "custom";
-}
-
 // ──────────────────────── applyOps ────────────────────────
 
 /** RuleOp discriminated union（G13：add 时 rule.id 已是真实 user-N）。 */

package/src/rules/matcher.ts

@@ -13,7 +13,7 @@
  *  用 `new RegExp(pattern, 'i')` 编译；其余（builtin-safe/user）是 OpenCode
  *  wildcard，用 `wildcardToRegExp(pattern)` 编译。resolvePattern 按 source 分发。
  *
- *  ── G3: 24+5 白名单用函数（C3 修正：不再短路）──
+ *  ── G3: 50+9 白名单用函数（C3 修正：不再短路）──
  *  matchRulesForArgv 先遍历 rules（last-match-wins）；若无 deny/allow 命中（ask），
  *  再查 isKnownSafeCommand(argv) 兜底返回 allow。白名单不再短路：用户 deny 规则
  *  可覆盖白名单。白名单不进 Rule[] 数组（Codex flag 子检查无法用单条 wildcard 表达）。

package/src/select-theme.ts

@@ -0,0 +1,15 @@
+/**
+ * SelectList 默认主题（model-picker 与 rule-editor 共享单一常量，避免字面量双份漂移）。
+ *
+ * G2/WR2 修正：selectedPrefix 实现为 `(t) => '▶ ' + t`（非 identity），
+ * 选中行有视觉区分（▶ 前缀）。
+ */
+import type { SelectListTheme } from "@earendil-works/pi-tui";
+
+export const DEFAULT_SELECT_THEME: SelectListTheme = {
+	selectedPrefix: (t: string): string => "\u25B6 " + t,
+	selectedText: (t: string): string => t,
+	description: (t: string): string => t,
+	scrollInfo: (t: string): string => t,
+	noMatch: (t: string): string => t,
+};

package/src/statusline-palette.ts

@@ -24,7 +24,8 @@ export interface PermissionPalette {
  * theme.fg(token, text) 是 Pi 的语义着色 API（token 如 "dim"/"text"/"success"）。
  * 本地包装为 PermissionPalette 接口，便于 footer renderer 与测试共用纯函数。
  *
- * @param theme Pi Theme 对象（ctx.ui.theme，需含 fg 函数）
+ * @param theme Pi Theme 对象（需含 fg 函数）。不由 ctx.ui.theme 提供——由 pi-statusline
+ *     每次 render 时经 render(ctx, theme) 第二参传入（见 index.ts footer renderer 装配）。
  */
 export function paletteFromTheme(theme: { fg(token: string, text: string): string }): PermissionPalette {
 	return {