@trim21/personal-pi-extensions 0.0.403 → 0.0.407

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.407",
   "type": "module",
   "description": "Custom pi coding-agent extensions: bwrap sandbox, workspace guard, opencode edit, and more",
   "keywords": [

package/src/aft/tools.ts

@@ -8,7 +8,7 @@
 import { readFileSync } from "node:fs";
 import { stat } from "node:fs/promises";
 import { homedir } from "node:os";
-import { basename, isAbsolute, join, resolve } from "node:path";
+import { isAbsolute, join, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
 
 import {
@@ -23,7 +23,7 @@ import {
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { Type } from "typebox";
 
-import { formatDisplayPath } from "../lib/path.js";
+import { formatDisplayPath, formatSubtitlePath } from "../lib/path.js";
 import { type ToolPendant } from "../lib/pendant.js";
 import { callAftTool, SEMANTIC_INDEX_WAIT_TIMEOUT_MS } from "./bridge.js";
 
@@ -51,16 +51,6 @@ export function resolvePathArg(cwd: string, input: string): string {
   return isAbsolute(input) ? input : resolve(cwd, input);
 }
 
-/** subtitle 中路径的最大显示长度，超过时退化为仅文件名。 */
-export const MAX_SUBTITLE_PATH_LENGTH = 20;
-
-/** subtitle 用的显示路径：优先 `./…` / `~/…` / 绝对路径，结果过长时只显示文件名。 */
-export function formatSubtitlePath(cwd: string, filePath: string): string {
-  const display = formatDisplayPath(cwd, filePath);
-  if (display.length <= MAX_SUBTITLE_PATH_LENGTH) return display;
-  return basename(filePath);
-}
-
 export interface AftToolContext {
   cwd: string;
   pool: AftTransportPool;

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/claude-code/files.ts

@@ -15,6 +15,7 @@ import {
 import { Type } from "typebox";
 
 import { type LspService, registerLsp } from "../lib/lsp/lsp.js";
+import { formatSubtitlePath } from "../lib/path.js";
 import { guardWriteAccess } from "../lib/write-guard.js";
 import {
   type ClaudeCodeState,
@@ -295,7 +296,13 @@ export function registerFileTools(
         const snapshot = snapshotOf(image, false);
         const key = await readStateKey(filePath);
         state.reads.set(key, snapshot);
-        return { content, details: { reads: { [key]: snapshot } } };
+        return {
+          content,
+          details: {
+            reads: { [key]: snapshot },
+            pendant: { subtitle: formatSubtitlePath(ctx.cwd, filePath) },
+          },
+        };
       }
 
       const offset = params.offset ?? 1;
@@ -313,7 +320,10 @@ export function registerFileTools(
         previous.limit === limit &&
         snapshotsEqual(previous, snapshotOf(buffer))
       ) {
-        return { content: [{ type: "text", text: FILE_UNCHANGED_STUB }], details: {} };
+        return {
+          content: [{ type: "text", text: FILE_UNCHANGED_STUB }],
+          details: { pendant: { subtitle: formatSubtitlePath(ctx.cwd, filePath) } },
+        };
       }
       if (isBinary(buffer.subarray(0, SAMPLE_BYTES)))
         throw new Error(`Cannot read binary file: ${filePath}`);
@@ -340,7 +350,10 @@ export function registerFileTools(
       });
       return {
         content: [{ type: "text", text: formatted.text }],
-        details: { reads: { [key]: snapshot } },
+        details: {
+          reads: { [key]: snapshot },
+          pendant: { subtitle: formatSubtitlePath(ctx.cwd, filePath) },
+        },
       };
     },
   });
@@ -381,8 +394,8 @@ export function registerFileTools(
           replaceAll: params.replace_all,
         },
       });
-      const [message, details, diagnosticText] = await withFileMutationQueue<
-        [string, FileToolDetails, string]
+      const [message, details, diagnosticText, errorCount] = await withFileMutationQueue<
+        [string, FileToolDetails, string, number]
       >(filePath, async () => {
         const oldString = params.old_string;
         const newString = params.new_string;
@@ -414,10 +427,14 @@ export function registerFileTools(
           state.reads.set(key, snapshot);
           const diff = generateDiffString("", convertLeadingTabsToSpaces(newString));
           throwIfAborted(signal);
-          const diagnosticText = await service.lspDiagnosticsForFile(filePath, ctx.cwd, {
-            notify: (message, level) => ctx.ui.notify(message, level),
-            signal,
-          });
+          const { text: diagnosticText, errorCount } = await service.lspDiagnosticsForFile(
+            filePath,
+            ctx.cwd,
+            {
+              notify: (message, level) => ctx.ui.notify(message, level),
+              signal,
+            },
+          );
           return [
             `The file ${filePath} has been updated successfully.`,
             {
@@ -427,6 +444,7 @@ export function registerFileTools(
               reads: { [key]: snapshot },
             },
             diagnosticText,
+            errorCount,
           ];
         }
         const replaceAll = params.replace_all ?? false;
@@ -511,9 +529,13 @@ export function registerFileTools(
           ? `The file ${filePath} has been updated. All occurrences were successfully replaced.`
           : `The file ${filePath} has been updated successfully.`;
         throwIfAborted(signal);
-        const diagnosticText = await service.lspDiagnosticsForFile(filePath, ctx.cwd, {
-          notify: (message, level) => ctx.ui.notify(message, level),
-        });
+        const { text: diagnosticText, errorCount } = await service.lspDiagnosticsForFile(
+          filePath,
+          ctx.cwd,
+          {
+            notify: (message, level) => ctx.ui.notify(message, level),
+          },
+        );
         return [
           text,
           {
@@ -527,13 +549,20 @@ export function registerFileTools(
             reads: { [key]: snapshot },
           },
           diagnosticText,
+          errorCount,
         ];
       });
 
       const text = diagnosticText
         ? `${message}\n\nLSP errors detected in this file, please fix:\n${diagnosticText}`
         : message;
-      return { content: [{ type: "text" as const, text }], details };
+      return {
+        content: [{ type: "text" as const, text }],
+        details: {
+          ...details,
+          pendant: { subtitle: formatSubtitlePath(ctx.cwd, filePath, errorCount) },
+        },
+      };
     },
   });
 
@@ -564,8 +593,8 @@ export function registerFileTools(
         absolutePath: filePath,
         change: { oldText: "", newText: params.content },
       });
-      const [message, details, diagnosticText] = await withFileMutationQueue<
-        [string, FileToolDetails, string]
+      const [message, details, diagnosticText, errorCount] = await withFileMutationQueue<
+        [string, FileToolDetails, string, number]
       >(filePath, async () => {
         let original: string | undefined;
         let key: string | undefined;
@@ -595,9 +624,13 @@ export function registerFileTools(
             ? `File created successfully at: ${filePath}`
             : `The file ${filePath} has been updated successfully.`;
         throwIfAborted(signal);
-        const diagnosticText = await service.lspDiagnosticsForFile(filePath, ctx.cwd, {
-          notify: (message, level) => ctx.ui.notify(message, level),
-        });
+        const { text: diagnosticText, errorCount } = await service.lspDiagnosticsForFile(
+          filePath,
+          ctx.cwd,
+          {
+            notify: (message, level) => ctx.ui.notify(message, level),
+          },
+        );
         return [
           text,
           {
@@ -607,13 +640,20 @@ export function registerFileTools(
             reads: { [resolvedKey]: snapshot },
           },
           diagnosticText,
+          errorCount,
         ];
       });
 
       const text = diagnosticText
         ? `${message}\n\nLSP errors detected in this file, please fix:\n${diagnosticText}`
         : message;
-      return { content: [{ type: "text" as const, text }], details };
+      return {
+        content: [{ type: "text" as const, text }],
+        details: {
+          ...details,
+          pendant: { subtitle: formatSubtitlePath(ctx.cwd, filePath, errorCount) },
+        },
+      };
     },
   });
 }

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),
@@ -215,7 +208,11 @@ export interface LspService {
     options?: LspRequestOptions,
   ): Promise<void>;
   diagnostics(): Promise<Record<string, Diagnostic[]>>;
-  lspDiagnosticsForFile(file: string, cwd: string, options?: LspRequestOptions): Promise<string>;
+  lspDiagnosticsForFile(
+    file: string,
+    cwd: string,
+    options?: LspRequestOptions,
+  ): Promise<{ text: string; errorCount: number }>;
   shutdownAll(): Promise<void>;
   /** 注入 status 渲染回调；传入 undefined 表示不再渲染。 */
   attachStatus(render: StatusRenderer | undefined): void;
@@ -411,19 +408,21 @@ export function createLspService(
   }
 
   /**
-   * edit/write 用：等待文档诊断并返回该文件的 ERROR 报告（空串表示无错误）。
-   * 内部所有 LSP 失败都会被吞掉，不干扰写操作本身。
+   * edit/write 用：等待文档诊断并返回该文件的 ERROR 报告（text 空串表示无错误）
+   * 与 ERROR 数量。内部所有 LSP 失败都会被吞掉，不干扰写操作本身。
    */
   async function lspDiagnosticsForFile(
     file: string,
     cwd: string,
     options?: LspRequestOptions,
-  ): Promise<string> {
-    if (options?.signal?.aborted) return "";
+  ): Promise<{ text: string; errorCount: number }> {
+    if (options?.signal?.aborted) return { text: "", errorCount: 0 };
     await touchFile(file, cwd, "document", options);
     const all = await diagnostics();
     const normalized = normalize(file);
-    return report(normalized, all[normalized] ?? []);
+    const issues = all[normalized] ?? [];
+    const errorCount = issues.filter((item) => item.severity === 1).length;
+    return { text: report(normalized, issues), errorCount };
   }
 
   /** 终止全部服务器进程（session_shutdown 时调用）。 */

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),
-  );
-}

package/src/lib/path.ts

@@ -4,7 +4,7 @@
 
 import { stat } from "node:fs/promises";
 import { homedir } from "node:os";
-import { isAbsolute, join, relative, resolve } from "node:path";
+import { basename, isAbsolute, join, relative, resolve } from "node:path";
 
 /**
  * Expand a leading `~` to the user's home directory.
@@ -59,3 +59,14 @@ export function formatDisplayPath(cwd: string, filePath: string): string {
   }
   return filePath;
 }
+
+/** subtitle 中路径的最大显示长度，超过时退化为仅文件名。 */
+export const MAX_SUBTITLE_PATH_LENGTH = 20;
+
+/** subtitle 用的显示路径：优先 `./…` / `~/…` / 绝对路径，结果过长时只显示文件名。 */
+export function formatSubtitlePath(cwd: string, filePath: string, errorCount?: number): string {
+  const display = formatDisplayPath(cwd, filePath);
+  const base = display.length <= MAX_SUBTITLE_PATH_LENGTH ? display : basename(filePath);
+  if (errorCount === undefined || errorCount === 0) return base;
+  return `${base} (ⓧ ${errorCount})`;
+}

package/src/opencode/files.ts

@@ -30,6 +30,7 @@ import {
 import { Type } from "typebox";
 
 import { type LspService, registerLsp } from "../lib/lsp/lsp.js";
+import { formatSubtitlePath } from "../lib/path.js";
 import { guardWriteAccess } from "../lib/write-guard.js";
 import { applyEdit, normalizeToLF, stripBom } from "./edit-engine.js";
 
@@ -356,7 +357,7 @@ function registerReadTool(pi: ExtensionAPI, service: LspService): void {
 
         return {
           content: [{ type: "text", text: output }],
-          details: undefined,
+          details: { pendant: { subtitle: formatSubtitlePath(ctx.cwd, absolutePath) } },
         };
       }
 
@@ -384,7 +385,10 @@ function registerReadTool(pi: ExtensionAPI, service: LspService): void {
           { type: "text", text: "Image read successfully" },
           { type: "image", data: base64, mimeType },
         ];
-        return { content, details: undefined };
+        return {
+          content,
+          details: { pendant: { subtitle: formatSubtitlePath(ctx.cwd, absolutePath) } },
+        };
       }
 
       if (isBinaryExtension(absolutePath) || isBinaryFileBySample(await readSample(absolutePath))) {
@@ -431,7 +435,13 @@ function registerReadTool(pi: ExtensionAPI, service: LspService): void {
         // 后台 warm-up 失败不影响读取
       });
 
-      return { content, details };
+      return {
+        content,
+        details: {
+          ...details,
+          pendant: { subtitle: formatSubtitlePath(ctx.cwd, absolutePath) },
+        },
+      };
     },
   });
 }
@@ -482,7 +492,7 @@ function registerEditTool(pi: ExtensionAPI, service: LspService): void {
         change: { oldText: oldString, newText: newString, replaceAll },
       });
 
-      const [message, details, diagnosticText] = await withFileMutationQueue(
+      const [message, details, diagnosticText, errorCount] = await withFileMutationQueue(
         absolutePath,
         async () => {
           signal?.throwIfAborted();
@@ -509,13 +519,16 @@ function registerEditTool(pi: ExtensionAPI, service: LspService): void {
             await mkdir(dirname(absolutePath), { recursive: true });
             signal?.throwIfAborted();
             await writeFile(absolutePath, newString, "utf8");
-            const diagnosticText = await service.lspDiagnosticsForFile(absolutePath, ctx.cwd, {
-              signal,
-            });
+            const { text: diagnosticText, errorCount } = await service.lspDiagnosticsForFile(
+              absolutePath,
+              ctx.cwd,
+              { signal },
+            );
             return [
               "Edit applied successfully.",
               { diff: "", patch: "", firstChangedLine: 0 },
               diagnosticText,
+              errorCount,
             ] as const;
           }
 
@@ -547,13 +560,16 @@ function registerEditTool(pi: ExtensionAPI, service: LspService): void {
           const diffNew = normalizeToLF(applied.contentNew);
           const diffResult = generateDiffString(diffOld, diffNew);
           const patch = generateUnifiedPatch(filePath, diffOld, diffNew);
-          const diagnosticText = await service.lspDiagnosticsForFile(absolutePath, ctx.cwd, {
-            signal,
-          });
+          const { text: diagnosticText, errorCount } = await service.lspDiagnosticsForFile(
+            absolutePath,
+            ctx.cwd,
+            { signal },
+          );
           return [
             "Edit applied successfully.",
             { diff: diffResult.diff, patch, firstChangedLine: diffResult.firstChangedLine },
             diagnosticText,
+            errorCount,
           ] as const;
         },
       );
@@ -561,7 +577,13 @@ function registerEditTool(pi: ExtensionAPI, service: LspService): void {
       const text = diagnosticText
         ? `${message}\n\nLSP errors detected in this file, please fix:\n${diagnosticText}`
         : message;
-      return { content: [{ type: "text" as const, text }], details };
+      return {
+        content: [{ type: "text" as const, text }],
+        details: {
+          ...details,
+          pendant: { subtitle: formatSubtitlePath(ctx.cwd, absolutePath, errorCount) },
+        },
+      };
     },
   });
 }
@@ -614,7 +636,7 @@ function registerWriteTool(pi: ExtensionAPI, service: LspService): void {
       });
       const dir = dirname(absolutePath);
 
-      const [message, details, diagnosticText] = await withFileMutationQueue(
+      const [message, details, diagnosticText, errorCount] = await withFileMutationQueue(
         absolutePath,
         async () => {
           signal?.throwIfAborted();
@@ -639,18 +661,26 @@ function registerWriteTool(pi: ExtensionAPI, service: LspService): void {
           await mkdir(dir, { recursive: true });
           signal?.throwIfAborted();
           await writeFile(absolutePath, desiredBom + nextText, "utf8");
-          const diagnosticText = await service.lspDiagnosticsForFile(absolutePath, ctx.cwd, {
-            signal,
-          });
+          const { text: diagnosticText, errorCount } = await service.lspDiagnosticsForFile(
+            absolutePath,
+            ctx.cwd,
+            { signal },
+          );
 
-          return ["Wrote file successfully.", undefined, diagnosticText] as const;
+          return ["Wrote file successfully.", {}, diagnosticText, errorCount] as const;
         },
       );
 
       const text = diagnosticText
         ? `${message}\n\nLSP errors detected in this file, please fix:\n${diagnosticText}`
         : message;
-      return { content: [{ type: "text" as const, text }], details };
+      return {
+        content: [{ type: "text" as const, text }],
+        details: {
+          ...details,
+          pendant: { subtitle: formatSubtitlePath(ctx.cwd, absolutePath, errorCount) },
+        },
+      };
     },
   });
 }