@trim21/personal-pi-extensions 0.0.403 → 0.0.406

package/README.md

@@ -94,7 +94,7 @@ bash 工具（opencode 风格 `bash`、Claude Code 风格 `Bash`）注册了 `da
 }
 ```
 
-`dangerouslyDisableSandbox: true` 的审批流程：先按 `approvalRules` 匹配（含嵌套 `$(...)` 内的命令，规则后写优先），命中 allow/deny 直接放行/拒绝，未命中才弹确认框。
+`dangerouslyDisableSandbox: true` 的审批流程：先按 `approvalRules` 匹配（含嵌套 `$(...)` 内的命令，规则后写优先），命中 allow/deny 直接放行/拒绝，未命中才弹确认框。含文件输出重定向（`>` / `>>` / `&>` 等）的命令即使命令规则全匹配也不会自动放行，避免 `echo *` 把 `echo '' > file` 带过；管道（`echo | tail`）和 fd 复制（`2>&1`）不受影响。
 
 ### 使用
 

package/package.json

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

package/src/bwrap/approval-rules.ts

@@ -3,7 +3,8 @@
  * 用 tree-sitter 解析命令（含嵌套 `$(...)`），按 BashArity 生成命令模式
  * （`git checkout main` → `git checkout *`），再用通配匹配对 allow/deny
  * 规则求值。接入 bwrap 的 `dangerouslyDisableSandbox` 审批：命中规则自动
- * 放行/拒绝，未命中才弹审批对话框。
+ * 放行/拒绝，未命中才弹审批对话框。文件输出重定向（`>` / `>>` / `&>`
+ * 等）不会因命令规则自动放行，避免 `echo *` 把 `echo '' > file` 带过。
  *
  * 参考实现：
  * - opencode packages/opencode/src/permission/arity.ts（BashArity 表）
@@ -27,6 +28,8 @@ export interface ParsedBash {
   commands: BashCommand[];
   /** 语法错误时的提示（解析失败不抛错，退化为无法匹配）。 */
   error?: string;
+  /** 含写入文件的重定向（`>` / `>>` / `&>` / `<>` 等）；fd 复制与纯输入除外。 */
+  hasFileOutputRedirect: boolean;
 }
 
 export type ApprovalAction = "allow" | "deny";
@@ -108,6 +111,27 @@ function extractParts(node: Node): { name: string; args: string[] } {
   return { name: name.join(" "), args };
 }
 
+/** 会打开/截断/追加文件的重定向算子；`>&` / `<&` 是 fd 复制，不算。 */
+const FILE_OUTPUT_REDIRECT_OPS = new Set([">", ">>", ">|", "&>", "&>>"]);
+
+function fileRedirectWritesToFile(node: Node): boolean {
+  for (let i = 0; i < node.childCount; i++) {
+    const child = node.child(i);
+    if (!child) continue;
+    if (FILE_OUTPUT_REDIRECT_OPS.has(child.type)) return true;
+    // `<>` 被解析成 `<` + 含 `>` 的 ERROR
+    if (child.type === "ERROR" && child.text.includes(">")) return true;
+  }
+  return false;
+}
+
+function treeHasFileOutputRedirect(root: Node): boolean {
+  for (const node of root.descendantsOfType("file_redirect")) {
+    if (fileRedirectWritesToFile(node)) return true;
+  }
+  return false;
+}
+
 function collectCommand(node: Node, all: Node[]): BashCommand | undefined {
   const { name, args } = extractParts(node);
   if (!name) return undefined;
@@ -139,10 +163,11 @@ export async function parseBashCommands(command: string): Promise<ParsedBash> {
       const parsed = collectCommand(node, all);
       if (parsed) commands.push(parsed);
     }
-    return { commands };
+    return { commands, hasFileOutputRedirect: treeHasFileOutputRedirect(tree.rootNode) };
   } catch (error) {
     return {
       commands: [],
+      hasFileOutputRedirect: false,
       error: error instanceof Error ? error.message : String(error),
     };
   }
@@ -197,20 +222,25 @@ export function matchRule(input: string, pattern: string): boolean {
  * 命令（含所有嵌套命令）的权限模式列表（命令替换里的命令也展开）。
  * 供规则求值与"allow forever"写规则复用。
  */
-export async function commandPatternsFor(command: string): Promise<string[]> {
-  const parsed = await parseBashCommands(command);
+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 parsed.commands) visit(cmd);
+  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（有命令未命中规则，交给人审），避免未允许的命令被同链放行带过。
  * 规则内后写优先（findLast，对齐 opencode PermissionV2）。
@@ -219,7 +249,8 @@ export async function evaluateBashApproval(
   command: string,
   rules: readonly ApprovalRule[],
 ): Promise<ApprovalAction | undefined> {
-  const patterns = await commandPatternsFor(command);
+  const parsed = await parseBashCommands(command);
+  const patterns = patternsFromCommands(parsed.commands);
   if (patterns.length === 0) return;
   let allowed = 0;
   for (const pattern of patterns) {
@@ -227,5 +258,6 @@ export async function evaluateBashApproval(
     if (rule?.action === "deny") return "deny";
     if (rule?.action === "allow") allowed++;
   }
+  if (parsed.hasFileOutputRedirect) return;
   return allowed === patterns.length ? "allow" : undefined;
 }

package/src/lib/lsp/lsp.ts

@@ -6,6 +6,7 @@
  * - 配置来源：全局 `~/.pi/agent/lsp.json` + 本地 `<cwd>/.pi/lsp.json`
  *   （本地覆盖全局）：`servers` 按 id 合并（同名 id 整体覆盖、新增 id，全局
  *   其余服务器保留），`enabled`/`disabled` 白名单与各超时参数继续生效；
+ *   没有内置默认服务器，所有服务器均须在配置里定义；
  *   配置在每个工具的调用 cwd 下惰性读取；enabled/disabled 引用不存在的
  *   服务器 id 是配置错误：全局配置在扩展加载（createLspService）时抛错，
  *   本地配置在 session 开始预加载时通知，工具调用时校验抛错兜底；
@@ -26,13 +27,7 @@ import { Value } from "typebox/value";
 import { type LspServerAdapter } from "./adapter.js";
 import { create, type CreateInput, type Diagnostic, type Info as LspClient } from "./client.js";
 import { report } from "./diagnostic.js";
-import {
-  createAdapters,
-  defaultServers,
-  mergeServerConfigs,
-  mergeServerRecords,
-  serverConfigSchema,
-} from "./server-config.js";
+import { createAdapters, mergeServerRecords, serverConfigSchema } from "./server-config.js";
 
 /** 超时值：number（毫秒，>=1）或字符串（"500"、"5s"、"1m"），Parse 后由 toMs 统一换算。 */
 const timeoutValue = Type.Union([Type.Number({ minimum: 1 }), Type.String()]);
@@ -41,7 +36,7 @@ const timeoutValue = Type.Union([Type.Number({ minimum: 1 }), Type.String()]);
 const lspConfigSchema = Type.Object({
   /** 配置文件版本（当前 1）；未知版本会被 typebox 严格校验拒绝并回退空配置。 */
   version: Type.Optional(Type.Number()),
-  /** 配置驱动的语言服务器定义（id → 配置）；按 id 与内置默认服务器合并（覆盖/enabled:false 禁用）。 */
+  /** 配置驱动的语言服务器定义（id → 配置）；无内置默认，全部在此定义。 */
   servers: Type.Optional(Type.Record(Type.String(), serverConfigSchema)),
   /** 只启用列出的服务器 id（缺省 = 全部启用）。 */
   enabled: Type.Optional(Type.Array(Type.String())),
@@ -124,9 +119,7 @@ function readConfigFileSync(filePath: string): LspConfig {
  */
 function validateConfig(config: LspConfig, adapters?: LspServerAdapter[]): void {
   const available = new Set(
-    adapters
-      ? adapters.map((adapter) => adapter.id)
-      : Object.keys(mergeServerConfigs(defaultServers, config.servers)),
+    adapters ? adapters.map((adapter) => adapter.id) : Object.keys(config.servers ?? {}),
   );
   const unknown = [...(config.enabled ?? []), ...(config.disabled ?? [])].filter(
     (id) => !available.has(id),

package/src/lib/lsp/server-config.ts

@@ -48,69 +48,6 @@ export const serverConfigSchema = Type.Object({
 
 export type ServerConfig = Static<typeof serverConfigSchema>;
 
-/** 内置默认服务器（id → 配置，可被用户配置按 id 覆盖，全量字段均可配置化表达）。 */
-export const defaultServers: Record<string, ServerConfig> = {
-  typescript: {
-    include: ["**/*.{ts,tsx,js,jsx,mjs,cjs,mts,cts}"],
-    rootMarkers: ["package-lock.json", "pnpm-lock.yaml", "yarn.lock", "bun.lock", "bun.lockb"],
-    bin: "typescript-language-server",
-    args: ["--stdio"],
-    cwd: "{root}",
-    languageIdByExtension: {
-      ".ts": "typescript",
-      ".tsx": "typescriptreact",
-      ".js": "javascript",
-      ".jsx": "javascriptreact",
-      ".mjs": "javascript",
-      ".cjs": "javascript",
-      ".mts": "typescript",
-      ".cts": "typescript",
-    },
-  },
-  pyright: {
-    include: ["**/*.py", "**/*.pyi"],
-    rootMarkers: [
-      "pyproject.toml",
-      "setup.py",
-      "setup.cfg",
-      "requirements.txt",
-      "Pipfile",
-      "pyrightconfig.json",
-    ],
-    bin: "pyright-langserver",
-    args: ["--stdio"],
-    cwd: "{root}",
-    languageIdByExtension: { ".py": "python", ".pyi": "python" },
-  },
-  ruff: {
-    include: ["**/*.py", "**/*.pyi"],
-    rootMarkers: ["pyproject.toml", "ruff.toml", ".ruff.toml"],
-    bin: "ruff",
-    args: ["server"],
-    cwd: "{root}",
-    languageIdByExtension: { ".py": "python", ".pyi": "python" },
-  },
-  clangd: {
-    include: ["**/*.{c,h,cpp,hpp,cc,cxx,c++,hh,hxx,h++}"],
-    rootMarkers: ["compile_commands.json", "compile_flags.txt", ".clangd"],
-    bin: "clangd",
-    args: ["--background-index", "--clang-tidy"],
-    cwd: "{root}",
-    languageIdByExtension: {
-      ".c": "c",
-      ".h": "c",
-      ".cpp": "cpp",
-      ".hpp": "cpp",
-      ".cc": "cpp",
-      ".cxx": "cpp",
-      ".c++": "cpp",
-      ".hh": "cpp",
-      ".hxx": "cpp",
-      ".h++": "cpp",
-    },
-  },
-};
-
 /** 按 id 合并 servers record：同名 id 整体覆盖（不做逐字段 merge），其余保留；返回 undefined 表示没有任何 servers 定义。 */
 export function mergeServerRecords(
   ...records: (Readonly<Record<string, ServerConfig>> | undefined)[]
@@ -123,21 +60,11 @@ export function mergeServerRecords(
   return Object.keys(merged).length > 0 ? merged : undefined;
 }
 
-/** 用户 servers（id → 配置）与默认配置合并：同名 id 整体覆盖、新增 id、enabled:false 移除。 */
-export function mergeServerConfigs(
-  defaults: Readonly<Record<string, ServerConfig>>,
-  user: Readonly<Record<string, ServerConfig>> | undefined,
-): Record<string, ServerConfig> {
-  if (!user) return { ...defaults };
-  const merged = { ...defaults };
-  for (const [id, server] of Object.entries(user)) {
-    if (server.enabled === false) {
-      delete merged[id];
-      continue;
-    }
-    merged[id] = server;
-  }
-  return merged;
+/** 组装启用的服务器列表：全部来自用户配置 servers（id → 配置），无内置默认。 */
+export function createAdapters(
+  userServers?: Readonly<Record<string, ServerConfig>>,
+): LspServerAdapter[] {
+  return Object.entries(userServers ?? {}).map(([id, config]) => new ConfigAdapter(id, config));
 }
 
 /** {root} / {cwd} 模板替换（bin / cwd 字段均支持）。 */
@@ -220,12 +147,3 @@ export class ConfigAdapter implements LspServerAdapter {
     };
   }
 }
-
-/** 组装启用的服务器列表：默认配置 + 用户 servers（id → 配置）合并。 */
-export function createAdapters(
-  userServers?: Readonly<Record<string, ServerConfig>>,
-): LspServerAdapter[] {
-  return Object.entries(mergeServerConfigs(defaultServers, userServers)).map(
-    ([id, config]) => new ConfigAdapter(id, config),
-  );
-}