@epoch-agent/plugin-lsp 3.1.2 → 4.0.0

package/README.md

@@ -174,10 +174,11 @@ PR-3，**刻意砍掉的**（不是做到一半放弃）—— 砍的判据和
 ## 开发
 
 ```bash
-pnpm --filter @epoch-agent/plugin-lsp test
+npx vitest run --project=plugin-lsp
 ```
 
-七个用例文件。两处值得先看：
+**八个**用例文件（2026-11-03 核：上一版写的是「七个」，漏了 `orphan-reap`）。
+两处值得先看：
 
 - [fixtures/fake-server.mjs](__tests__/fixtures/fake-server.mjs) 是一个**手写分帧**
   的假 LSP server（故意不用 `vscode-jsonrpc`，那样才是一份独立的参照实现）。

package/dist/index.js

@@ -34,7 +34,7 @@ var MAX_OPEN_DOCUMENTS = 64;
 var POLL_STEP_MS = 25;
 var LspServerDownError = class extends Error {
   constructor(detail) {
-    super(`language server \u5DF2\u9000\u51FA\uFF1A${detail}`);
+    super(`language server exited: ${detail}`);
     this.detail = detail;
     this.name = "LspServerDownError";
   }
@@ -137,8 +137,8 @@ var LspClient = class _LspClient {
     this.connection.onRequest(RegistrationRequest.type, () => void 0);
     this.connection.onRequest(UnregistrationRequest.type, () => void 0);
     this.connection.onRequest(WorkDoneProgressCreateRequest.type, () => void 0);
-    this.connection.onError(() => this.markDead("JSON-RPC \u8FDE\u63A5\u9519\u8BEF"));
-    this.connection.onClose(() => this.markDead("JSON-RPC \u8FDE\u63A5\u5173\u95ED"));
+    this.connection.onError(() => this.markDead("JSON-RPC connection error"));
+    this.connection.onClose(() => this.markDead("JSON-RPC connection closed"));
   }
   appendStderr(chunk) {
     this.stderr = (this.stderr + chunk).slice(-STDERR_KEEP);
@@ -146,7 +146,7 @@ var LspClient = class _LspClient {
   markDead(detail, onCrash) {
     if (this.dead) return;
     const tail = this.stderr.trim();
-    this.dead = tail ? `${detail}\uFF1Bstderr: ${tail.slice(-500)}` : detail;
+    this.dead = tail ? `${detail}; stderr: ${tail.slice(-500)}` : detail;
     if (!this.disposing) onCrash?.(this.dead);
   }
   async initialize(projectRoot) {
@@ -258,7 +258,7 @@ var LspClient = class _LspClient {
       this.connection.dispose();
     } catch {
     }
-    this.dead ??= "\u5DF2\u4E3B\u52A8\u5173\u95ED";
+    this.dead ??= "disposed";
     await killTrackedProcess(this.pidValue);
     if (tree.length > 0) await killStampedPids(tree, { escalate: true });
   }
@@ -322,7 +322,7 @@ var TYPESCRIPT = {
   pkg: "typescript-language-server",
   bin: "typescript-language-server",
   args: ["--stdio"],
-  install: "npm i -D typescript-language-server typescript\uFF08\u6216\u5168\u5C40 npm i -g\uFF09",
+  install: "npm i -D typescript-language-server typescript (or install globally with npm i -g)",
   initTimeoutMs: 3e4,
   initializationOptions: (projectRoot) => {
     const path = tsserverPath(projectRoot);
@@ -394,7 +394,7 @@ async function classify(paths, workDir, outcome) {
     const absPath = isAbsolute(raw) ? resolve2(raw) : resolve2(root, raw);
     const rel = relative(root, absPath);
     if (rel !== "" && (rel.startsWith("..") || isAbsolute(rel))) {
-      outcome.rejected.push({ path: raw, reason: `\u4E0D\u5728\u5DE5\u4F5C\u533A ${root} \u5185` });
+      outcome.rejected.push({ path: raw, reason: `outside the workspace ${root}` });
       continue;
     }
     const key = uriKey(`file://${absPath.split(sep).join("/")}`);
@@ -404,11 +404,11 @@ async function classify(paths, workDir, outcome) {
     try {
       const info = await stat(absPath);
       if (!info.isFile()) {
-        outcome.rejected.push({ path: display, reason: "\u4E0D\u662F\u6587\u4EF6" });
+        outcome.rejected.push({ path: display, reason: "not a file" });
         continue;
       }
     } catch {
-      outcome.rejected.push({ path: display, reason: "\u6587\u4EF6\u4E0D\u5B58\u5728" });
+      outcome.rejected.push({ path: display, reason: "file does not exist" });
       continue;
     }
     const match = serverForExtension(extname(absPath));
@@ -452,7 +452,7 @@ async function runGroup(spec, files, input, outcome) {
     } catch (err) {
       outcome.rejected.push({
         path: file.display,
-        reason: `\u8BFB\u4E0D\u51FA\u6765\uFF08${err instanceof Error ? err.message : String(err)}\uFF09`
+        reason: `cannot read it (${err instanceof Error ? err.message : String(err)})`
       });
       continue;
     }
@@ -464,7 +464,7 @@ async function runGroup(spec, files, input, outcome) {
   try {
     batch = await acquired.client.diagnose(documents, input.wait ?? DEFAULT_WAIT);
   } catch (err) {
-    const reason = err instanceof LspServerDownError ? err.message : `\u8BCA\u65AD\u5931\u8D25\uFF08${String(err)}\uFF09`;
+    const reason = err instanceof LspServerDownError ? err.message : `diagnostics failed (${String(err)})`;
     outcome.skipped.push(skip(spec, reason, files));
     return;
   }
@@ -477,7 +477,11 @@ async function runGroup(spec, files, input, outcome) {
   if (batch.pending.length > 0) {
     const stuck = batch.pending.map((key) => byKey.get(key)).filter((f) => !!f);
     outcome.skipped.push(
-      skip(spec, `server \u672A\u5728 ${(input.wait ?? DEFAULT_WAIT).timeoutMs}ms \u5185\u7ED9\u51FA\u7ED3\u679C`, stuck)
+      skip(
+        spec,
+        `the server gave no result within ${(input.wait ?? DEFAULT_WAIT).timeoutMs}ms`,
+        stuck
+      )
     );
   }
 }
@@ -491,12 +495,12 @@ async function diagnosePaths(input) {
   };
   const paths = input.paths.slice(0, MAX_FILES_PER_CALL);
   for (const extra of input.paths.slice(MAX_FILES_PER_CALL)) {
-    outcome.rejected.push({ path: extra, reason: `\u4E00\u6B21\u6700\u591A ${MAX_FILES_PER_CALL} \u4E2A\u6587\u4EF6` });
+    outcome.rejected.push({ path: extra, reason: `at most ${MAX_FILES_PER_CALL} files per call` });
   }
   const candidates = await classify(paths, resolve2(input.workDir), outcome);
   for (const group of groupBySpec(candidates).values()) {
     if (input.signal?.aborted) {
-      outcome.skipped.push(skip(group.spec, "\u5DF2\u53D6\u6D88", group.files));
+      outcome.skipped.push(skip(group.spec, "cancelled", group.files));
       continue;
     }
     await runGroup(group.spec, group.files, input, outcome);
@@ -533,7 +537,7 @@ function renderCounts(items) {
 }
 function renderFiles(files) {
   if (files.length <= MAX_FILE_NAMES) return files.join(", ");
-  return `${files.slice(0, MAX_FILE_NAMES).join(", ")} \u7B49 ${files.length} \u4E2A`;
+  return `${files.slice(0, MAX_FILE_NAMES).join(", ")} \u2026 (${files.length} total)`;
 }
 function pickItems(items) {
   if (items.length <= MAX_ITEMS) return { shown: items, dropped: 0 };
@@ -551,29 +555,29 @@ function formatOutcome(outcome) {
   const { checked, diagnostics, skipped, unsupported, rejected } = outcome;
   if (diagnostics.length === 0) {
     lines.push(
-      checked.length > 0 ? `\u672A\u53D1\u73B0\u95EE\u9898 \xB7 \u5DF2\u68C0\u67E5 ${checked.length} \u4E2A\u6587\u4EF6\uFF1A${renderFiles(checked)}` : "\u672A\u68C0\u67E5\u4EFB\u4F55\u6587\u4EF6"
+      checked.length > 0 ? `No problems found \xB7 checked ${checked.length} file(s): ${renderFiles(checked)}` : "No files were checked"
     );
   } else {
     const { shown, dropped } = pickItems(diagnostics);
     lines.push(
-      `${diagnostics.length} \u6761\u8BCA\u65AD\uFF08${renderCounts(diagnostics)}\uFF09 \xB7 \u5DF2\u68C0\u67E5 ${checked.length} \u4E2A\u6587\u4EF6`,
+      `${diagnostics.length} diagnostic(s) (${renderCounts(diagnostics)}) \xB7 checked ${checked.length} file(s)`,
       "",
       ...shown.map(renderItem)
     );
     if (dropped > 0) {
-      lines.push("", `\uFF08\u8FD8\u6709 ${dropped} \u6761\u672A\u5217\u51FA\uFF0C\u5DF2\u4F18\u5148\u4FDD\u7559 error\uFF09`);
+      lines.push("", `(${dropped} more not listed; errors were kept first)`);
     }
   }
   for (const skip2 of skipped) {
-    lines.push("", `\u8DF3\u8FC7 ${skip2.label}\uFF1A${skip2.reason}`);
-    if (skip2.install) lines.push(`  \u5B89\u88C5\uFF1A${skip2.install}`);
-    if (skip2.files.length > 0) lines.push(`  \u672A\u68C0\u67E5\uFF1A${renderFiles(skip2.files)}`);
+    lines.push("", `Skipped ${skip2.label}: ${skip2.reason}`);
+    if (skip2.install) lines.push(`  install: ${skip2.install}`);
+    if (skip2.files.length > 0) lines.push(`  not checked: ${renderFiles(skip2.files)}`);
   }
   if (unsupported.length > 0) {
-    lines.push("", `\u65E0\u5BF9\u5E94 language server\uFF0C\u672A\u68C0\u67E5\uFF1A${renderFiles(unsupported)}`);
+    lines.push("", `No matching language server; not checked: ${renderFiles(unsupported)}`);
   }
   for (const bad of rejected) {
-    lines.push("", `\u65E0\u6CD5\u68C0\u67E5 ${bad.path}\uFF1A${bad.reason}`);
+    lines.push("", `Cannot check ${bad.path}: ${bad.reason}`);
   }
   return lines.join("\n");
 }
@@ -664,7 +668,7 @@ function resolveServer(spec, projectRoot, options = {}) {
   if (launch) return { available: true, launch };
   return {
     available: false,
-    reason: `\u6CA1\u627E\u5230 ${spec.bin}\uFF08\u9879\u76EE node_modules \u548C PATH \u91CC\u90FD\u6CA1\u6709\uFF09`,
+    reason: `could not find ${spec.bin} (not in the project's node_modules, nor on PATH)`,
     install: spec.install
   };
 }
@@ -702,13 +706,13 @@ var LspServerPool = class {
         return { ok: true, client: existing.client };
       }
       this.entries.delete(key);
-      this.noteFailure(key, "language server \u8FDB\u7A0B\u5DF2\u9000\u51FA");
+      this.noteFailure(key, "language server process has exited");
     }
     const failed = this.failures.get(key);
     if (failed && failed.count >= MAX_FAILURES) {
       return {
         ok: false,
-        reason: `${failed.detail}\uFF08\u5DF2\u8FDE\u7EED\u5931\u8D25 ${failed.count} \u6B21\uFF0C\u672C\u6B21\u4F1A\u8BDD\u4E0D\u518D\u91CD\u542F\uFF09`
+        reason: `${failed.detail} (failed ${failed.count} times in a row; not restarting in this session)`
       };
     }
     const inflight = this.starting.get(key);
@@ -732,15 +736,18 @@ var LspServerPool = class {
       projectRoot,
       onCrash: (detail) => {
         this.entries.delete(key);
-        this.noteFailure(key, `language server \u5D29\u4E86\uFF08${detail}\uFF09`);
+        this.noteFailure(key, `language server crashed (${detail})`);
       }
     });
     if (!client) {
-      this.noteFailure(key, `language server \u8D77\u4E0D\u6765\u6216\u521D\u59CB\u5316\u8D85\u65F6\uFF08>${spec.initTimeoutMs}ms\uFF09`);
+      this.noteFailure(
+        key,
+        `language server failed to start or initialize timed out (>${spec.initTimeoutMs}ms)`
+      );
       const now = this.failures.get(key);
       return {
         ok: false,
-        reason: now && now.count >= MAX_FAILURES ? `${now.detail}\uFF08\u672C\u6B21\u4F1A\u8BDD\u4E0D\u518D\u91CD\u542F\uFF09` : now?.detail ?? "\u542F\u52A8\u5931\u8D25"
+        reason: now && now.count >= MAX_FAILURES ? `${now.detail} (not restarting in this session)` : now?.detail ?? "failed to start"
       };
     }
     this.entries.set(key, { client, lastUsedAt: this.now() });
@@ -814,18 +821,18 @@ function fail(code, message, suggestion, started) {
 var lspPlugin = {
   name: "@epoch-agent/plugin-lsp",
   version: "0.0.0",
-  description: "LSP \u8BCA\u65AD\u63D2\u4EF6 \u2014 \u7528\u9879\u76EE\u81EA\u5DF1\u7684 language server \u53D6\u7C7B\u578B / \u8BED\u6CD5\u9519\u8BEF",
+  description: "Language-server diagnostics \u2014 type and syntax errors from the project's own language server",
   tools: [
     {
       name: "lsp_diagnostics",
-      description: `\u7528\u9879\u76EE\u81EA\u5DF1\u7684 language server \u68C0\u67E5\u6587\u4EF6\uFF0C\u8FD4\u56DE\u7C7B\u578B\u9519\u8BEF\u3001\u8BED\u6CD5\u9519\u8BEF\u548C\u672A\u4F7F\u7528\u53D8\u91CF\u7B49\u8BCA\u65AD\u3002\u53C2\u6570: paths(\u5FC5\u586B, \u6587\u4EF6\u8DEF\u5F84\u6570\u7EC4, \u76F8\u5BF9\u5DE5\u4F5C\u76EE\u5F55)\u3002\u5F53\u524D\u652F\u6301 ${supportedExtensions().join(" ")}\u3002**\u6539\u5B8C\u4EE3\u7801\u4E4B\u540E\u3001\u5BA3\u5E03\u5B8C\u6210\u4E4B\u524D\u8C03\u4E00\u6B21**\uFF0C\u628A\u672C\u8F6E\u65B0\u5EFA\u6216\u4FEE\u6539\u8FC7\u7684\u6E90\u6587\u4EF6\u90FD\u5217\u8FDB\u53BB \u2014\u2014 \u5B83\u6BD4\u8BA9 shell \u8DD1\u4E00\u904D tsc \u5FEB\u5F97\u591A\uFF0C\u56E0\u4E3A language server \u662F\u5E38\u9A7B\u7684\u3002\u6CE8\u610F\u5B83\u53EA\u505A\u5355\u6587\u4EF6\u7EA7\u8BCA\u65AD\uFF0C\u4E0D\u7B49\u4E8E\u4E00\u6B21\u5B8C\u6574\u6784\u5EFA\uFF1A\u6784\u5EFA\u811A\u672C\u3001\u6253\u5305\u914D\u7F6E\u3001\u8DE8\u5305\u7C7B\u578B\u89E3\u6790\u51FA\u7684\u95EE\u9898\u5B83\u770B\u4E0D\u5230\u3002\u6CA1\u88C5\u5BF9\u5E94 language server \u65F6\u4F1A\u5982\u5B9E\u62A5\u300C\u8DF3\u8FC7\u300D\u5E76\u7ED9\u51FA\u5B89\u88C5\u547D\u4EE4\uFF0C\u4E0D\u4F1A\u8BA9\u8FD9\u6B21\u8C03\u7528\u5931\u8D25\u3002`,
+      description: `Check files with the project's own language server and return type errors, syntax errors and unused-variable warnings. Supported extensions: ${supportedExtensions().join(" ")}. Call it once after changing code and before saying the work is finished, listing the source files you created or modified. It is much faster than running tsc through the shell because the language server is already resident. It only does single-file diagnostics, which is not the same as a full build: problems from build scripts, bundler config or cross-package type resolution are invisible to it. When no language server is installed it reports "skipped" and prints the install command instead of failing.`,
       parameters: {
         type: "object",
         properties: {
           paths: {
             type: "array",
             items: { type: "string" },
-            description: '\u8981\u68C0\u67E5\u7684\u6587\u4EF6\u8DEF\u5F84\uFF08\u76F8\u5BF9\u5DE5\u4F5C\u76EE\u5F55\u6216\u7EDD\u5BF9\u8DEF\u5F84\uFF09\uFF0C\u4F8B\u5982 ["src/a.ts", "src/b.tsx"]'
+            description: 'File paths to check (relative to the working directory, or absolute), e.g. ["src/a.ts", "src/b.tsx"].'
           }
         },
         required: ["paths"]
@@ -839,8 +846,8 @@ var lspPlugin = {
         if (!paths) {
           return fail(
             ERR_BAD_PATHS,
-            "paths \u5FC5\u987B\u662F\u975E\u7A7A\u7684\u6587\u4EF6\u8DEF\u5F84\u6570\u7EC4",
-            '\u628A\u672C\u8F6E\u6539\u52A8\u8FC7\u7684\u6E90\u6587\u4EF6\u5217\u8FDB\u53BB\uFF0C\u4F8B\u5982 {"paths": ["src/a.ts"]}',
+            "paths must be a non-empty array of file paths",
+            'List the source files you changed in this turn, e.g. {"paths": ["src/a.ts"]}',
             started
           );
         }
@@ -855,8 +862,8 @@ var lspPlugin = {
         } catch (err) {
           return fail(
             ERR_UNEXPECTED,
-            `\u8BCA\u65AD\u5931\u8D25: ${err instanceof Error ? err.message : String(err)}`,
-            "\u53EF\u4EE5\u5148\u7EE7\u7EED\uFF0C\u6216\u6539\u7528 terminal \u8DD1\u9879\u76EE\u81EA\u5DF1\u7684\u7C7B\u578B\u68C0\u67E5\u547D\u4EE4",
+            `diagnostics failed: ${err instanceof Error ? err.message : String(err)}`,
+            "You can keep going, or run the project's own type-check command through terminal instead",
             started
           );
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@epoch-agent/plugin-lsp",
-  "version": "3.1.2",
+  "version": "4.0.0",
   "private": false,
   "description": "LSP 诊断插件 — 探测项目自带的 language server，按文件取语义诊断",
   "repository": {
@@ -24,8 +24,8 @@
   "dependencies": {
     "vscode-jsonrpc": "^9.0.1",
     "vscode-languageserver-protocol": "^3.17.5",
-    "@epoch-agent/infra": "3.1.2",
-    "@epoch-agent/protocol": "3.1.2"
+    "@epoch-agent/infra": "4.0.0",
+    "@epoch-agent/protocol": "4.0.0"
   },
   "devDependencies": {
     "@types/node": "^26.1.2",