@trim21/personal-pi-extensions 0.0.422 → 0.0.423

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@trim21/personal-pi-extensions",
-  "version": "0.0.422",
+  "version": "0.0.423",
   "type": "module",
   "description": "Custom pi coding-agent extensions: bwrap sandbox, workspace guard, opencode edit, and more",
   "keywords": [

package/src/bwrap/approval-rules.ts

@@ -1,13 +1,13 @@
 /**
- * approval-rules —— bash 命令审核组件（对齐 opencode 的权限方法）：
- * 用 tree-sitter 解析命令（含嵌套 `$(...)`），按 BashArity 生成命令模式
- * （`git checkout main` → `git checkout *`），再用通配匹配对 allow/deny
- * 规则求值。接入 bwrap 的 `dangerouslyDisableSandbox` 审批：命中规则自动
+ * approval-rules —— bash 命令匹配引擎（对齐 opencode 的权限方法）：
+ * 用 tree-sitter 解析命令（含嵌套 `$(...)`），对每条命令的原文做通配
+ * 匹配，对 allow/deny 规则求值。规则匹配不经过任何命令归一——BashArity
+ * 归一模式只用于 "allow forever" 的建议规则（见 approval-suggest.ts）。
+ * 接入 bwrap 的 `dangerouslyDisableSandbox` 审批：命中规则自动
  * 放行/拒绝，未命中才弹审批对话框。文件输出重定向（`>` / `>>` / `&>`
  * 等）不会因命令规则自动放行，避免 `echo *` 把 `echo '' > file` 带过。
  *
  * 参考实现：
- * - opencode packages/opencode/src/permission/arity.ts（BashArity 表）
  * - opencode packages/core/src/util/wildcard.ts（通配匹配）
  * - opencode packages/opencode/src/tool/shell.ts（tree-sitter 命令提取）
  */
@@ -173,34 +173,6 @@ export async function parseBashCommands(command: string): Promise<ParsedBash> {
   }
 }
 
-// ── BashArity：命令前缀 → token 数（参考 opencode arity.ts）─────────────────
-
-/**
- * 命令前缀 → 定义该命令的 token 数。`git checkout main` → `git` 的 arity 2，
- * 权限模式取前 2 个 token + `*`（`git checkout *`），避免具体参数进规则。
- * 表来自 opencode packages/opencode/src/permission/arity.ts（Apache-2.0），
- * 数据存放在 arity.json（所有 key 带引号）。
- */
-const ARITY = JSON.parse(readFileSync(new URL("arity.json", import.meta.url), "utf8")) as Record<
-  string,
-  number
->;
-
-/**
- * 生成命令的权限模式：BashArity 前缀 + `*`。
- * `git checkout main` → `git checkout *`；未收录的命令 → 命令名 + `*`。
- */
-export function commandPattern(command: BashCommand): string {
-  const tokens = [command.name, ...command.args];
-  for (let len = tokens.length; len > 0; len--) {
-    const prefix = tokens.slice(0, len).join(" ");
-    const arity = ARITY[prefix];
-    if (arity !== undefined) return [...tokens.slice(0, arity), "*"].join(" ");
-  }
-  if (tokens.length === 0) return "*";
-  return [tokens[0], "*"].join(" ");
-}
-
 // ── 通配匹配（参考 opencode wildcard.ts）────────────────────────────────────
 
 /**
@@ -218,31 +190,15 @@ export function matchRule(input: string, pattern: string): boolean {
 
 // ── 规则求值 ────────────────────────────────────────────────────────────────
 
-/**
- * 命令（含所有嵌套命令）的权限模式列表（命令替换里的命令也展开）。
- * 供规则求值与"allow forever"写规则复用。
- */
-function patternsFromCommands(commands: BashCommand[]): string[] {
-  const flat: string[] = [];
-  const visit = (cmd: BashCommand) => {
-    flat.push(commandPattern(cmd));
-    for (const nested of cmd.nested) visit(nested);
-  };
-  for (const cmd of commands) visit(cmd);
-  return flat;
-}
-
-export async function commandPatternsFor(command: string): Promise<string[]> {
-  const parsed = await parseBashCommands(command);
-  return patternsFromCommands(parsed.commands);
-}
-
 /**
  * 对命令（含所有嵌套命令）求值：
  * - deny 优先：任一命令命中 deny 规则即整体拒绝
  * - 文件输出重定向不自动放行：即使命令规则全匹配，也返回 undefined 交人审
  * - allow 需全量：所有命令都命中 allow 规则才整体放行，否则返回
  *   undefined（有命令未命中规则，交给人审），避免未允许的命令被同链放行带过。
+ * - 匹配输入是命令原文（tree-sitter command 节点 text，含嵌套逐条展开），
+ *   对齐 opencode shell.ts 的 patterns；通配规则按字面写，`--` 与普通
+ *   token 无区别。
  * 规则内后写优先（findLast，对齐 opencode PermissionV2）。
  */
 export async function evaluateBashApproval(
@@ -250,14 +206,19 @@ export async function evaluateBashApproval(
   rules: readonly ApprovalRule[],
 ): Promise<ApprovalAction | undefined> {
   const parsed = await parseBashCommands(command);
-  const patterns = patternsFromCommands(parsed.commands);
-  if (patterns.length === 0) return;
+  const raws: string[] = [];
+  const visit = (cmd: BashCommand) => {
+    raws.push(cmd.raw);
+    for (const nested of cmd.nested) visit(nested);
+  };
+  for (const cmd of parsed.commands) visit(cmd);
+  if (raws.length === 0) return;
   let allowed = 0;
-  for (const pattern of patterns) {
-    const rule = rules.findLast((r) => matchRule(pattern, r.pattern));
+  for (const raw of raws) {
+    const rule = rules.findLast((r) => matchRule(raw, r.pattern));
     if (rule?.action === "deny") return "deny";
     if (rule?.action === "allow") allowed++;
   }
   if (parsed.hasFileOutputRedirect) return;
-  return allowed === patterns.length ? "allow" : undefined;
+  return allowed === raws.length ? "allow" : undefined;
 }

package/src/bwrap/approval-suggest.ts

@@ -0,0 +1,53 @@
+/**
+ * approval-suggest —— "allow forever" 的建议规则生成：
+ * 用 BashArity 表把命令归一成模式（`git checkout main` → `git checkout *`），
+ * 供用户选择"永久允许"时写入配置。归一模式不参与规则判定——匹配引擎
+ * （approval-rules）按命令原文做通配匹配。
+ *
+ * 参考实现：opencode packages/opencode/src/permission/arity.ts（Apache-2.0）
+ */
+import { readFileSync } from "node:fs";
+
+import { type BashCommand, parseBashCommands } from "./approval-rules.js";
+
+/**
+ * 命令前缀 → 定义该命令的 token 数。`git checkout main` → `git` 的 arity 2，
+ * 权限模式取前 2 个 token + `*`（`git checkout *`），避免具体参数进规则。
+ * 表来自 opencode packages/opencode/src/permission/arity.ts（Apache-2.0），
+ * 数据存放在 arity.json（所有 key 带引号）。
+ */
+const ARITY = JSON.parse(readFileSync(new URL("arity.json", import.meta.url), "utf8")) as Record<
+  string,
+  number
+>;
+
+/**
+ * 生成命令的 BashArity 建议模式：BashArity 前缀 + `*`。
+ * `git checkout main` → `git checkout *`；未收录的命令 → 命令名 + `*`。
+ */
+export function commandPattern(command: BashCommand): string {
+  const tokens = [command.name, ...command.args];
+  for (let len = tokens.length; len > 0; len--) {
+    const prefix = tokens.slice(0, len).join(" ");
+    const arity = ARITY[prefix];
+    if (arity !== undefined) return [...tokens.slice(0, arity), "*"].join(" ");
+  }
+  if (tokens.length === 0) return "*";
+  return [tokens[0], "*"].join(" ");
+}
+
+/** 命令（含所有嵌套命令）的 BashArity 建议模式列表（命令替换里的命令也展开）。 */
+function patternsFromCommands(commands: BashCommand[]): string[] {
+  const flat: string[] = [];
+  const visit = (cmd: BashCommand) => {
+    flat.push(commandPattern(cmd));
+    for (const nested of cmd.nested) visit(nested);
+  };
+  for (const cmd of commands) visit(cmd);
+  return flat;
+}
+
+export async function commandPatternsFor(command: string): Promise<string[]> {
+  const parsed = await parseBashCommands(command);
+  return patternsFromCommands(parsed.commands);
+}

package/src/bwrap/runtime.ts

@@ -23,12 +23,8 @@ import {
   selectCheckboxActions,
   selectWithOptionalInput,
 } from "../lib/ui.js";
-import {
-  type ApprovalRule,
-  commandPatternsFor,
-  evaluateBashApproval,
-  matchRule,
-} from "./approval-rules.js";
+import { type ApprovalRule, evaluateBashApproval, matchRule } from "./approval-rules.js";
+import { commandPatternsFor } from "./approval-suggest.js";
 import {
   type BwrapMode,
   createBwrapBashOperations,