@trim21/personal-pi-extensions 0.1.541 → 0.1.544

package/package.json

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

package/src/claude-code/files.ts

@@ -14,7 +14,7 @@ import {
 } from "@earendil-works/pi-coding-agent";
 import { Type } from "typebox";
 
-import { appendLspDiagnosticText } from "../lib/lsp/diagnostic.js";
+import { appendLspDiagnosticText, type DiagnosticReport } from "../lib/lsp/diagnostic.js";
 import { registerLspInspectTools } from "../lib/lsp/inspect-tool.js";
 import { createLspManager, type LspService, type LspServiceOptions } from "../lib/lsp/lsp.js";
 import { registerLspRenameTool } from "../lib/lsp/rename-tool.js";
@@ -333,24 +333,25 @@ export function registerFileTools(
       const snapshot = snapshotOf(buffer);
       state.reads.set(key, snapshot);
       // 与 Edit / Write 同一条驻留路径：didOpen 后等待该文件的诊断并报告
-      const {
-        text: diagnosticText,
-        errorCount,
-        warningCount,
-      } = await getService().lspDiagnosticsForFile(filePath, ctx.cwd, {
+      const diagnostics = await getService().lspDiagnosticsForFile(filePath, ctx.cwd, {
         notify: (message, level) => ctx.ui.notify(message, level),
       });
       return {
         content: [
           {
             type: "text",
-            text: appendLspDiagnosticText(formatted.text, diagnosticText, errorCount),
+            text: appendLspDiagnosticText(formatted.text, diagnostics.text),
           },
         ],
         details: {
           reads: { [key]: snapshot },
           pendant: {
-            subtitle: formatSubtitlePath(ctx.cwd, filePath, errorCount, warningCount),
+            subtitle: formatSubtitlePath(
+              ctx.cwd,
+              filePath,
+              diagnostics.errorCount,
+              diagnostics.warningCount,
+            ),
             title: "Read",
           } satisfies ToolPendant,
         },
@@ -394,178 +395,167 @@ export function registerFileTools(
           replaceAll: params.replace_all,
         },
       });
-      const [message, details, diagnosticText, errorCount, warningCount] =
-        await withFileMutationQueue<[string, FileToolDetails, string, number, number]>(
-          filePath,
-          async () => {
-            const oldString = params.old_string;
-            const newString = params.new_string;
-            if (oldString === newString) {
-              throw new Error(
-                "No changes to make: old_string and new_string are exactly the same.",
-              );
-            }
-            // 空 old_string：创建新文件或填充空文件（不需要先 Read，对齐 Claude Code）
-            if (oldString === "") {
-              let exists = true;
-              try {
-                const value = await stat(filePath);
-                if (value.isFile()) {
-                  const content = await readFile(filePath, "utf8");
-                  if (content.trim() !== "") {
-                    throw new Error("Cannot create new file - file already exists.");
-                  }
-                }
-              } catch (error) {
-                if (error instanceof Error && "code" in error && error.code === "ENOENT") {
-                  exists = false;
-                } else {
-                  throw error;
-                }
-              }
-              if (!exists) await mkdir(dirname(filePath), { recursive: true });
-              await writeFile(filePath, newString, "utf8");
-              const snapshot = snapshotOf(newString);
-              const key = await readStateKey(filePath);
-              state.reads.set(key, snapshot);
-              const diff = generateDiffString("", convertLeadingTabsToSpaces(newString));
-              signal?.throwIfAborted();
-              const {
-                text: diagnosticText,
-                errorCount,
-                warningCount,
-              } = await getService().lspDiagnosticsForFile(filePath, ctx.cwd, {
-                notify: (message, level) => ctx.ui.notify(message, level),
-                signal,
-              });
-              return [
-                `The file ${filePath} has been updated successfully.`,
-                {
-                  diff: diff.diff,
-                  patch: generateUnifiedPatch(filePath, "", convertLeadingTabsToSpaces(newString)),
-                  firstChangedLine: diff.firstChangedLine,
-                  reads: { [key]: snapshot },
-                },
-                diagnosticText,
-                errorCount,
-                warningCount,
-              ];
-            }
-            const replaceAll = params.replace_all ?? false;
-            // 防止 OOM 的大文件检查（对齐 Claude Code）
-            try {
-              const { size } = await stat(filePath);
-              if (size > MAX_EDIT_FILE_SIZE) {
-                throw new Error(
-                  `File is too large to edit (${formatFileSize(size)}). Maximum editable file size is ${formatFileSize(MAX_EDIT_FILE_SIZE)}.`,
-                );
-              }
-            } catch (error) {
-              if (!(error instanceof Error) || !("code" in error) || error.code !== "ENOENT") {
-                throw error;
+      const [message, details, diagnostics] = await withFileMutationQueue<
+        [string, FileToolDetails, DiagnosticReport]
+      >(filePath, async () => {
+        const oldString = params.old_string;
+        const newString = params.new_string;
+        if (oldString === newString) {
+          throw new Error("No changes to make: old_string and new_string are exactly the same.");
+        }
+        // 空 old_string：创建新文件或填充空文件（不需要先 Read，对齐 Claude Code）
+        if (oldString === "") {
+          let exists = true;
+          try {
+            const value = await stat(filePath);
+            if (value.isFile()) {
+              const content = await readFile(filePath, "utf8");
+              if (content.trim() !== "") {
+                throw new Error("Cannot create new file - file already exists.");
               }
             }
-            let content: Buffer;
-            try {
-              content = await readFile(filePath);
-            } catch (error) {
-              if (error instanceof Error && "code" in error && error.code === "ENOENT") {
-                const suggestion = await didYouMean(filePath, ctx.cwd);
-                throw new Error(
-                  `File does not exist. Note: your current working directory is ${ctx.cwd}.${suggestion ? ` Did you mean ${suggestion}?` : ""}`,
-                  { cause: error },
-                );
-              }
+          } catch (error) {
+            if (error instanceof Error && "code" in error && error.code === "ENOENT") {
+              exists = false;
+            } else {
               throw error;
             }
-            if (extname(filePath).toLowerCase() === ".ipynb") {
-              throw new Error(
-                "File is a Jupyter Notebook. Use the NotebookEditTool to edit this file.",
-              );
-            }
-            const key = await readStateKey(filePath);
-            requireCurrentRead(state, key, filePath, content);
-            await access(filePath, constants.R_OK | constants.W_OK);
-            signal?.throwIfAborted();
-            const original = content.toString("utf8");
-
-            // CRLF 规范化后匹配（old_string 不需要带 \r），写回时恢复原行尾
-            const crlfCount = (original.match(/\r\n/g) ?? []).length;
-            const lfCount = (original.match(/(?<!\r)\n/g) ?? []).length;
-            const lineEnding = crlfCount > lfCount ? "\r\n" : "\n";
-            const normalized = original.replaceAll("\r\n", "\n");
-            const actualOldString = findActualString(normalized, oldString) ?? oldString;
-            const matches = normalized.split(actualOldString).length - 1;
-            if (matches === 0) {
-              throw new Error(`String to replace not found in file.\nString: ${oldString}`);
-            }
-            if (!replaceAll && matches > 1) {
-              throw new Error(
-                `Found ${matches} matches of the string to replace, but replace_all is false. To replace all occurrences, set replace_all to true. To replace only one occurrence, please provide more context to uniquely identify the instance.\nString: ${oldString}`,
-              );
-            }
-            const actualNewString = preserveQuoteStyle(oldString, actualOldString, newString);
-            // 删除场景（new_string 为空）：old_string 不以换行结尾且文件里是
-            // "old_string\n" 时连换行一起删，避免留下空行（对齐 Claude Code
-            // applyEditToFile 的 stripTrailingNewline 语义）
-            let searchString = actualOldString;
-            if (
-              actualNewString === "" &&
-              !actualOldString.endsWith("\n") &&
-              normalized.includes(actualOldString + "\n")
-            ) {
-              searchString = actualOldString + "\n";
-            }
-            // split/join 与函数替换：replacement 含 $ 时不会触发 $& 等特殊语义
-            const updated = replaceAll
-              ? normalized.split(searchString).join(actualNewString)
-              : normalized.replace(searchString, () => actualNewString);
-            const restored = lineEnding === "\r\n" ? updated.replaceAll("\n", "\r\n") : updated;
-            await writeFile(filePath, restored, "utf8");
-            const snapshot = snapshotOf(restored);
-            state.reads.set(key, snapshot);
-            // patch/diff 仅供显示：前导 tab 转空格，避免 UI 渲染错位（对齐 Claude Code）
-            const diff = generateDiffString(
+          }
+          if (!exists) await mkdir(dirname(filePath), { recursive: true });
+          await writeFile(filePath, newString, "utf8");
+          const snapshot = snapshotOf(newString);
+          const key = await readStateKey(filePath);
+          state.reads.set(key, snapshot);
+          const diff = generateDiffString("", convertLeadingTabsToSpaces(newString));
+          signal?.throwIfAborted();
+          const diagnostics = await getService().lspDiagnosticsForFile(filePath, ctx.cwd, {
+            notify: (message, level) => ctx.ui.notify(message, level),
+            signal,
+          });
+          return [
+            `The file ${filePath} has been updated successfully.`,
+            {
+              diff: diff.diff,
+              patch: generateUnifiedPatch(filePath, "", convertLeadingTabsToSpaces(newString)),
+              firstChangedLine: diff.firstChangedLine,
+              reads: { [key]: snapshot },
+            },
+            diagnostics,
+          ];
+        }
+        const replaceAll = params.replace_all ?? false;
+        // 防止 OOM 的大文件检查（对齐 Claude Code）
+        try {
+          const { size } = await stat(filePath);
+          if (size > MAX_EDIT_FILE_SIZE) {
+            throw new Error(
+              `File is too large to edit (${formatFileSize(size)}). Maximum editable file size is ${formatFileSize(MAX_EDIT_FILE_SIZE)}.`,
+            );
+          }
+        } catch (error) {
+          if (!(error instanceof Error) || !("code" in error) || error.code !== "ENOENT") {
+            throw error;
+          }
+        }
+        let content: Buffer;
+        try {
+          content = await readFile(filePath);
+        } catch (error) {
+          if (error instanceof Error && "code" in error && error.code === "ENOENT") {
+            const suggestion = await didYouMean(filePath, ctx.cwd);
+            throw new Error(
+              `File does not exist. Note: your current working directory is ${ctx.cwd}.${suggestion ? ` Did you mean ${suggestion}?` : ""}`,
+              { cause: error },
+            );
+          }
+          throw error;
+        }
+        if (extname(filePath).toLowerCase() === ".ipynb") {
+          throw new Error(
+            "File is a Jupyter Notebook. Use the NotebookEditTool to edit this file.",
+          );
+        }
+        const key = await readStateKey(filePath);
+        requireCurrentRead(state, key, filePath, content);
+        await access(filePath, constants.R_OK | constants.W_OK);
+        signal?.throwIfAborted();
+        const original = content.toString("utf8");
+
+        // CRLF 规范化后匹配（old_string 不需要带 \r），写回时恢复原行尾
+        const crlfCount = (original.match(/\r\n/g) ?? []).length;
+        const lfCount = (original.match(/(?<!\r)\n/g) ?? []).length;
+        const lineEnding = crlfCount > lfCount ? "\r\n" : "\n";
+        const normalized = original.replaceAll("\r\n", "\n");
+        const actualOldString = findActualString(normalized, oldString) ?? oldString;
+        const matches = normalized.split(actualOldString).length - 1;
+        if (matches === 0) {
+          throw new Error(`String to replace not found in file.\nString: ${oldString}`);
+        }
+        if (!replaceAll && matches > 1) {
+          throw new Error(
+            `Found ${matches} matches of the string to replace, but replace_all is false. To replace all occurrences, set replace_all to true. To replace only one occurrence, please provide more context to uniquely identify the instance.\nString: ${oldString}`,
+          );
+        }
+        const actualNewString = preserveQuoteStyle(oldString, actualOldString, newString);
+        // 删除场景（new_string 为空）：old_string 不以换行结尾且文件里是
+        // "old_string\n" 时连换行一起删，避免留下空行（对齐 Claude Code
+        // applyEditToFile 的 stripTrailingNewline 语义）
+        let searchString = actualOldString;
+        if (
+          actualNewString === "" &&
+          !actualOldString.endsWith("\n") &&
+          normalized.includes(actualOldString + "\n")
+        ) {
+          searchString = actualOldString + "\n";
+        }
+        // split/join 与函数替换：replacement 含 $ 时不会触发 $& 等特殊语义
+        const updated = replaceAll
+          ? normalized.split(searchString).join(actualNewString)
+          : normalized.replace(searchString, () => actualNewString);
+        const restored = lineEnding === "\r\n" ? updated.replaceAll("\n", "\r\n") : updated;
+        await writeFile(filePath, restored, "utf8");
+        const snapshot = snapshotOf(restored);
+        state.reads.set(key, snapshot);
+        // patch/diff 仅供显示：前导 tab 转空格，避免 UI 渲染错位（对齐 Claude Code）
+        const diff = generateDiffString(
+          convertLeadingTabsToSpaces(original),
+          convertLeadingTabsToSpaces(restored),
+        );
+        const text = replaceAll
+          ? `The file ${filePath} has been updated. All occurrences were successfully replaced.`
+          : `The file ${filePath} has been updated successfully.`;
+        signal?.throwIfAborted();
+        const diagnostics = await getService().lspDiagnosticsForFile(filePath, ctx.cwd, {
+          notify: (message, level) => ctx.ui.notify(message, level),
+        });
+        return [
+          text,
+          {
+            diff: diff.diff,
+            patch: generateUnifiedPatch(
+              filePath,
               convertLeadingTabsToSpaces(original),
               convertLeadingTabsToSpaces(restored),
-            );
-            const text = replaceAll
-              ? `The file ${filePath} has been updated. All occurrences were successfully replaced.`
-              : `The file ${filePath} has been updated successfully.`;
-            signal?.throwIfAborted();
-            const {
-              text: diagnosticText,
-              errorCount,
-              warningCount,
-            } = await getService().lspDiagnosticsForFile(filePath, ctx.cwd, {
-              notify: (message, level) => ctx.ui.notify(message, level),
-            });
-            return [
-              text,
-              {
-                diff: diff.diff,
-                patch: generateUnifiedPatch(
-                  filePath,
-                  convertLeadingTabsToSpaces(original),
-                  convertLeadingTabsToSpaces(restored),
-                ),
-                firstChangedLine: diff.firstChangedLine,
-                reads: { [key]: snapshot },
-              },
-              diagnosticText,
-              errorCount,
-              warningCount,
-            ];
+            ),
+            firstChangedLine: diff.firstChangedLine,
+            reads: { [key]: snapshot },
           },
-        );
+          diagnostics,
+        ];
+      });
 
-      const text = appendLspDiagnosticText(message, diagnosticText, errorCount);
+      const text = appendLspDiagnosticText(message, diagnostics.text);
       return {
         content: [{ type: "text" as const, text }],
         details: {
           ...details,
           pendant: {
-            subtitle: formatSubtitlePath(ctx.cwd, filePath, errorCount, warningCount),
+            subtitle: formatSubtitlePath(
+              ctx.cwd,
+              filePath,
+              diagnostics.errorCount,
+              diagnostics.warningCount,
+            ),
             title: "Edit",
           } satisfies ToolPendant,
         },
@@ -600,66 +590,65 @@ export function registerFileTools(
         absolutePath: filePath,
         change: { oldText: "", newText: params.content },
       });
-      const [message, details, diagnosticText, errorCount, warningCount] =
-        await withFileMutationQueue<[string, FileToolDetails, string, number, number]>(
-          filePath,
-          async () => {
-            let original: string | undefined;
-            let key: string | undefined;
-            try {
-              const value = await stat(filePath);
-              if (value.isFile()) {
-                const content = await readFile(filePath);
-                key = await readStateKey(filePath);
-                requireCurrentRead(state, key, filePath, content);
-                original = content.toString("utf8");
-              }
-            } catch (error) {
-              if (!(error instanceof Error) || !("code" in error) || error.code !== "ENOENT") {
-                throw error;
-              }
-            }
-            signal?.throwIfAborted();
-            await mkdir(dirname(filePath), { recursive: true });
-            await writeFile(filePath, params.content, "utf8");
-            const snapshot = snapshotOf(params.content);
-            // 新建文件：writeFile 之后 realpath 才能解析；覆盖写则复用上面的 key
-            const resolvedKey = key ?? (await readStateKey(filePath));
-            state.reads.set(resolvedKey, snapshot);
-            const diff = generateDiffString(original ?? "", params.content);
-            const text =
-              original === undefined
-                ? `File created successfully at: ${filePath}`
-                : `The file ${filePath} has been updated successfully.`;
-            signal?.throwIfAborted();
-            const {
-              text: diagnosticText,
-              errorCount,
-              warningCount,
-            } = await getService().lspDiagnosticsForFile(filePath, ctx.cwd, {
-              notify: (message, level) => ctx.ui.notify(message, level),
-            });
-            return [
-              text,
-              {
-                diff: diff.diff,
-                patch: generateUnifiedPatch(filePath, original ?? "", params.content),
-                firstChangedLine: diff.firstChangedLine,
-                reads: { [resolvedKey]: snapshot },
-              },
-              diagnosticText,
-              errorCount,
-              warningCount,
-            ];
+      const [message, details, diagnostics] = await withFileMutationQueue<
+        [string, FileToolDetails, DiagnosticReport]
+      >(filePath, async () => {
+        let original: string | undefined;
+        let key: string | undefined;
+        try {
+          const value = await stat(filePath);
+          if (value.isFile()) {
+            const content = await readFile(filePath);
+            key = await readStateKey(filePath);
+            requireCurrentRead(state, key, filePath, content);
+            original = content.toString("utf8");
+          }
+        } catch (error) {
+          if (!(error instanceof Error) || !("code" in error) || error.code !== "ENOENT") {
+            throw error;
+          }
+        }
+        signal?.throwIfAborted();
+        await mkdir(dirname(filePath), { recursive: true });
+        await writeFile(filePath, params.content, "utf8");
+        const snapshot = snapshotOf(params.content);
+        // 新建文件：writeFile 之后 realpath 才能解析；覆盖写则复用上面的 key
+        const resolvedKey = key ?? (await readStateKey(filePath));
+        state.reads.set(resolvedKey, snapshot);
+        const diff = generateDiffString(original ?? "", params.content);
+        const text =
+          original === undefined
+            ? `File created successfully at: ${filePath}`
+            : `The file ${filePath} has been updated successfully.`;
+        signal?.throwIfAborted();
+        const diagnostics = await getService().lspDiagnosticsForFile(filePath, ctx.cwd, {
+          notify: (message, level) => ctx.ui.notify(message, level),
+        });
+        return [
+          text,
+          {
+            diff: diff.diff,
+            patch: generateUnifiedPatch(filePath, original ?? "", params.content),
+            firstChangedLine: diff.firstChangedLine,
+            reads: { [resolvedKey]: snapshot },
           },
-        );
+          diagnostics,
+        ];
+      });
 
-      const text = appendLspDiagnosticText(message, diagnosticText, errorCount);
+      const text = appendLspDiagnosticText(message, diagnostics.text);
       return {
         content: [{ type: "text" as const, text }],
         details: {
           ...details,
-          pendant: { subtitle: formatSubtitlePath(ctx.cwd, filePath, errorCount, warningCount) },
+          pendant: {
+            subtitle: formatSubtitlePath(
+              ctx.cwd,
+              filePath,
+              diagnostics.errorCount,
+              diagnostics.warningCount,
+            ),
+          },
         },
       };
     },

package/src/gh-readonly.ts

@@ -425,6 +425,11 @@ export function stepsDetail(
 // 缓存检查避免重复网络请求。闭包状态不与其他扩展共享，key 无需全局前缀。
 const seq = createSeqState();
 
+/** Absolute path of the raw job log cache file written by `getJobLog`. */
+export function jobLogPath(runId: string, jobId: number): string {
+  return join(homedir(), ".cache", "pi", "ci-logs", runId, `${jobId}.log`);
+}
+
 async function getJobLog(
   runId: string,
   jobId: number,
@@ -433,8 +438,8 @@ async function getJobLog(
   cwd: string | undefined,
   input?: unknown,
 ): Promise<string> {
-  const cacheDir = join(homedir(), ".cache", "pi", "ci-logs", runId);
-  const cacheFile = join(cacheDir, `${jobId}.log`);
+  const cacheFile = jobLogPath(runId, jobId);
+  const cacheDir = dirname(cacheFile);
 
   // 同一 runId:jobId 的请求串行执行：后一个进入时缓存已写入，直接命中缓存，
   // 不会重复发网络请求；串行也保证不会有两个并发写同一 cache 文件。
@@ -682,6 +687,15 @@ export function stripAnsi(text: string): string {
   return text.replace(/^\uFEFF/, "").replaceAll(ANSI_RE, "");
 }
 
+/**
+ * Appended to `read-github-ci-logs` content so the raw log file path reaches
+ * the model: tool `details` is not part of the LLM context, so the path has to
+ * live in the returned text.
+ */
+function rawLogNotice(path: string): string {
+  return `[Raw job log (whole job): ${path}]`;
+}
+
 export interface StepLogParams {
   runId: string;
   job?: string;
@@ -772,6 +786,7 @@ export async function renderStepLog(
   });
 
   const rawLog = await fetchJobLog(targetJob.id);
+  const rawLogFile = jobLogPath(params.runId, targetJob.id);
 
   const stepLog = extractStepFromLog(rawLog, stepNum, targetJob.steps);
   if (stepLog === null) {
@@ -779,10 +794,10 @@ export async function renderStepLog(
       content: [
         {
           type: "text",
-          text: `Could not extract step ${stepNum} from job "${targetJob.name}" logs. The log may be malformed or empty. Try fetching without \`step\` to see the full job log.`,
+          text: `Could not extract step ${stepNum} from job "${targetJob.name}" logs. The log may be malformed or empty — read the raw file instead.\n\n${rawLogNotice(rawLogFile)}`,
         },
       ],
-      details: {},
+      details: { rawLogFile },
     };
   }
 
@@ -792,11 +807,12 @@ export async function renderStepLog(
   if (full) {
     const fullLines = clean.split("\n").length;
     return {
-      content: [{ type: "text", text: clean }],
+      content: [{ type: "text", text: `${clean}\n\n${rawLogNotice(rawLogFile)}` }],
       details: {
         summary: `Step ${stepNum} — ${targetJob.name} / ${found.name}: complete output (${fullLines} lines)`,
         truncated: false,
         full: true,
+        rawLogFile,
         job: {
           name: targetJob.name,
           conclusion: targetJob.conclusion,
@@ -818,10 +834,10 @@ export async function renderStepLog(
         content: [
           {
             type: "text",
-            text: `Offset ${offset} exceeds step log length (${totalLines} lines).`,
+            text: `Offset ${offset} exceeds step log length (${totalLines} lines).\n\n${rawLogNotice(rawLogFile)}`,
           },
         ],
-        details: {},
+        details: { rawLogFile },
       };
     }
     logToShow = clean
@@ -837,10 +853,11 @@ export async function renderStepLog(
   const shownLines = text.split("\n").length;
 
   return {
-    content: [{ type: "text", text }],
+    content: [{ type: "text", text: `${text}\n\n${rawLogNotice(rawLogFile)}` }],
     details: {
       summary: `Step ${stepNum} — ${targetJob.name} / ${found.name}: ${shownLines} of ${totalLines} lines${tr ? " (truncated)" : ""}`,
       truncated: tr,
+      rawLogFile,
       job: {
         name: targetJob.name,
         conclusion: targetJob.conclusion,
@@ -869,15 +886,18 @@ export interface JobLogsStep {
 
 export interface JobLogsOutput {
   name: string;
+  /** Absolute path of this job's raw log on disk, set when its log was fetched. */
+  log_file?: string;
   steps: JobLogsStep[];
 }
 
 /**
  * Render the result of `read-github-ci-logs` without a `step`: a JSON array of
- * jobs `[{ name, steps: [{ name, output? }] }]`. Every step is listed by name;
- * only failed steps carry an `output` (their log as plain text). `job` is an
- * optional filter; `offset`/`limit` control the size of each `output` text.
- * Pure — no network, no `gh`.
+ * jobs `[{ name, log_file?, steps: [{ name, output? }] }]`. Every step is listed
+ * by name; only failed steps carry an `output` (their log as plain text).
+ * `log_file` is the raw on-disk log path for jobs whose log was fetched. `job`
+ * is an optional filter; `offset`/`limit` control the size of each `output`
+ * text. Pure — no network, no `gh`.
  */
 export async function renderJobLogs(
   params: JobLogsParams,
@@ -961,7 +981,11 @@ export async function renderJobLogs(
       }
     }
 
-    output.push({ name: j.name, steps });
+    output.push({
+      name: j.name,
+      ...(rawLog !== null && { log_file: jobLogPath(params.runId, j.id) }),
+      steps,
+    });
   }
 
   const totalJobs = output.length;
@@ -1856,7 +1880,7 @@ export default function ghReadonlyTools(pi: ExtensionAPI) {
     name: "read-github-ci-logs",
     label: "GitHub CI Logs",
     description:
-      "Get CI logs from a GitHub Actions workflow run. Without step: returns a JSON array of jobs [{name, steps:[{name, output?}]}] where every step is listed by name and failed steps carry their log as plain text in `output`. With step (requires job): returns that step's complete log as plain text. offset/limit control the size of every expanded output. Use run_id from list-github-workflow-runs. Note: queued jobs have no logs yet; use watch-github-run to wait for completion. Set full=true for complete untruncated outputs (every step when step is omitted; caution: very large outputs consume a lot of LLM context). Set output_file=/path to write the complete log to a file instead of returning it (requires job when the run has multiple jobs); the tool returns the file path to read.",
+      "Get CI logs from a GitHub Actions workflow run. Without step: returns a JSON array of jobs [{name, log_file?, steps:[{name, output?}]}] where every step is listed by name and failed steps carry their log as plain text in `output`. With step (requires job): returns that step's complete log as plain text. Every fetched job log is also saved raw (timestamps and ANSI kept) to a local file whose path is reported — `log_file` on the job, or a trailing `[Raw job log (whole job): <path>]` line with step — so read/grep that file for the complete, untruncated log. offset/limit control the size of every expanded output. Use run_id from list-github-workflow-runs. Note: queued jobs have no logs yet; use watch-github-run to wait for completion. Set full=true for complete untruncated outputs (every step when step is omitted; caution: very large outputs consume a lot of LLM context). Set output_file=/path to write the complete cleaned log to this file instead of returning it (requires job when the run has multiple jobs); the tool returns the file path to read.",
     promptSnippet: "Read GitHub CI logs",
     parameters: Type.Object({
       run_id: Type.Union([Type.Number(), Type.String()], { description: "Workflow run ID" }),

package/src/lib/lsp/diagnostic.ts

@@ -17,6 +17,20 @@ function severityOf(diagnostic: Diagnostic): number {
   return diagnostic.severity ?? 1;
 }
 
+/** 单个文件的诊断报告；计数覆盖该文件全部 ERROR / WARN，不受每文件上限影响。 */
+export interface DiagnosticReport {
+  /** 空字符串表示没有 ERROR / WARN。 */
+  text: string;
+  errorCount: number;
+  warningCount: number;
+}
+
+export const EMPTY_DIAGNOSTIC_REPORT: DiagnosticReport = {
+  text: "",
+  errorCount: 0,
+  warningCount: 0,
+};
+
 export function prettyDiagnostic(diagnostic: Diagnostic): string {
   const severity = SEVERITY_LABELS[severityOf(diagnostic)] ?? "ERROR";
   const line = diagnostic.range.start.line + 1;
@@ -27,25 +41,38 @@ export function prettyDiagnostic(diagnostic: Diagnostic): string {
   return `${severity} [${line}:${col}] ${message}`;
 }
 
-/** 返回空字符串表示没有 ERROR / WARN。ERROR 在前，WARN 在后。 */
-export function report(file: string, issues: Diagnostic[]): string {
+function pluralize(count: number, noun: string): string {
+  return `${count} ${noun}${count === 1 ? "" : "s"}`;
+}
+
+/** 未列出部分的构成：`3 errors, 4 warnings`（为 0 的一类省略；单复数按数量）。 */
+function hiddenSummary(hidden: Diagnostic[]): string {
+  const errors = hidden.filter((item) => severityOf(item) === 1).length;
+  const warnings = hidden.length - errors;
+  return [
+    ...(errors > 0 ? [pluralize(errors, "error")] : []),
+    ...(warnings > 0 ? [pluralize(warnings, "warning")] : []),
+  ].join(", ");
+}
+
+/** 返回空 text 表示没有 ERROR / WARN。ERROR 在前，WARN 在后，超出上限的部分只报数量。 */
+export function report(file: string, issues: Diagnostic[]): DiagnosticReport {
   const errors = issues.filter((item) => severityOf(item) === 1);
   const warnings = issues.filter((item) => item.severity === 2);
   const relevant = [...errors, ...warnings];
-  if (relevant.length === 0) return "";
+  if (relevant.length === 0) return EMPTY_DIAGNOSTIC_REPORT;
   const limited = relevant.slice(0, MAX_PER_FILE);
-  const more = relevant.length - MAX_PER_FILE;
-  const suffix = more > 0 ? `\n... and ${more} more` : "";
-  return `<diagnostics file="${file}">\n${limited.map((d) => prettyDiagnostic(d)).join("\n")}${suffix}\n</diagnostics>`;
+  const hidden = relevant.slice(MAX_PER_FILE);
+  const suffix = hidden.length > 0 ? `\n... and ${hiddenSummary(hidden)}` : "";
+  return {
+    text: `<diagnostics file="${file}">\n${limited.map((d) => prettyDiagnostic(d)).join("\n")}${suffix}\n</diagnostics>`,
+    errorCount: errors.length,
+    warningCount: warnings.length,
+  };
 }
 
 /** 把 LSP 诊断块接到工具成功文案后面；无诊断时原样返回。 */
-export function appendLspDiagnosticText(
-  message: string,
-  diagnosticText: string,
-  errorCount: number,
-): string {
-  if (!diagnosticText) return message;
-  const kind = errorCount > 0 ? "errors" : "warnings";
-  return `${message}\n\nLSP ${kind} detected in this file:\n${diagnosticText}`;
+export function appendLspDiagnosticText(message: string, diagnosticText: string): string {
+  if (diagnosticText === "") return message;
+  return `${message}\n\nLSP diagnostics detected in this file\n${diagnosticText}`;
 }

package/src/lib/lsp/lsp.ts

@@ -43,7 +43,7 @@ import {
   WATCH_KIND_CREATE,
   WATCH_KIND_DELETE,
 } from "./client.js";
-import { report } from "./diagnostic.js";
+import { type DiagnosticReport, EMPTY_DIAGNOSTIC_REPORT, report } from "./diagnostic.js";
 import {
   createAdapters,
   matchesInclude,
@@ -398,12 +398,12 @@ export interface LspService {
     options?: LspRequestOptions,
   ): Promise<void>;
   diagnostics(): Promise<Record<string, Diagnostic[]>>;
-  /** read / edit / write 用：等待文档诊断并返回该文件的 ERROR / WARN 报告与数量。 */
+  /** read / edit / write 用：等待文档诊断并返回该文件的 ERROR / WARN 报告与计数。 */
   lspDiagnosticsForFile(
     file: string,
     cwd: string,
     options?: LspRequestOptions,
-  ): Promise<{ text: string; errorCount: number; warningCount: number }>;
+  ): Promise<DiagnosticReport>;
   /**
    * 符号重命名：只面向 kind 为 "language" 的服务器（linter 不参与符号级
    * 功能）；多 client 按配置顺序取第一个成功结果，全部失败时抛聚合错误。
@@ -929,15 +929,12 @@ export function createLspService(
     file: string,
     cwd: string,
     options?: LspRequestOptions,
-  ): Promise<{ text: string; errorCount: number; warningCount: number }> {
-    if (options?.signal?.aborted) return { text: "", errorCount: 0, warningCount: 0 };
+  ): Promise<DiagnosticReport> {
+    if (options?.signal?.aborted) return EMPTY_DIAGNOSTIC_REPORT;
     await touchFile(file, cwd, "document", options);
     const all = await diagnostics();
     const normalized = normalize(file);
-    const issues = all[normalized] ?? [];
-    const errorCount = issues.filter((item) => (item.severity ?? 1) === 1).length;
-    const warningCount = issues.filter((item) => item.severity === 2).length;
-    return { text: report(normalized, issues), errorCount, warningCount };
+    return report(normalized, all[normalized] ?? []);
   }
 
   /** 符号重命名：只面向 language 类服务器；按配置顺序取第一个成功结果。 */

package/src/opencode/files.ts

@@ -432,15 +432,13 @@ function registerReadTool(pi: ExtensionAPI, getService: () => LspService): void
       }
 
       // opencode: 与 edit / write 同一条驻留路径：didOpen 后等待该文件的诊断并报告
-      const {
-        text: diagnosticText,
-        errorCount,
-        warningCount,
-      } = await getService().lspDiagnosticsForFile(absolutePath, ctx.cwd, { signal });
+      const diagnostics = await getService().lspDiagnosticsForFile(absolutePath, ctx.cwd, {
+        signal,
+      });
       content = [
         {
           type: "text",
-          text: appendLspDiagnosticText(outputText, diagnosticText, errorCount),
+          text: appendLspDiagnosticText(outputText, diagnostics.text),
         },
       ];
 
@@ -449,7 +447,12 @@ function registerReadTool(pi: ExtensionAPI, getService: () => LspService): void
         details: {
           ...details,
           pendant: {
-            subtitle: formatSubtitlePath(ctx.cwd, absolutePath, errorCount, warningCount),
+            subtitle: formatSubtitlePath(
+              ctx.cwd,
+              absolutePath,
+              diagnostics.errorCount,
+              diagnostics.warningCount,
+            ),
           },
         },
       };
@@ -503,8 +506,9 @@ function registerEditTool(pi: ExtensionAPI, getService: () => LspService): void
         change: { oldText: oldString, newText: newString, replaceAll },
       });
 
-      const [message, details, diagnosticText, errorCount, warningCount] =
-        await withFileMutationQueue(absolutePath, async () => {
+      const [message, details, diagnostics] = await withFileMutationQueue(
+        absolutePath,
+        async () => {
           signal?.throwIfAborted();
 
           // opencode: 前置校验，先于空 oldString 分支
@@ -529,17 +533,13 @@ function registerEditTool(pi: ExtensionAPI, getService: () => LspService): void
             await mkdir(dirname(absolutePath), { recursive: true });
             signal?.throwIfAborted();
             await writeFile(absolutePath, newString, "utf8");
-            const {
-              text: diagnosticText,
-              errorCount,
-              warningCount,
-            } = await getService().lspDiagnosticsForFile(absolutePath, ctx.cwd, { signal });
+            const diagnostics = await getService().lspDiagnosticsForFile(absolutePath, ctx.cwd, {
+              signal,
+            });
             return [
               "Edit applied successfully.",
               { diff: "", patch: "", firstChangedLine: 0 },
-              diagnosticText,
-              errorCount,
-              warningCount,
+              diagnostics,
             ] as const;
           }
 
@@ -571,27 +571,29 @@ function registerEditTool(pi: ExtensionAPI, getService: () => LspService): void
           const diffNew = normalizeToLF(applied.contentNew);
           const diffResult = generateDiffString(diffOld, diffNew);
           const patch = generateUnifiedPatch(filePath, diffOld, diffNew);
-          const {
-            text: diagnosticText,
-            errorCount,
-            warningCount,
-          } = await getService().lspDiagnosticsForFile(absolutePath, ctx.cwd, { signal });
+          const diagnostics = await getService().lspDiagnosticsForFile(absolutePath, ctx.cwd, {
+            signal,
+          });
           return [
             "Edit applied successfully.",
             { diff: diffResult.diff, patch, firstChangedLine: diffResult.firstChangedLine },
-            diagnosticText,
-            errorCount,
-            warningCount,
+            diagnostics,
           ] as const;
-        });
+        },
+      );
 
-      const text = appendLspDiagnosticText(message, diagnosticText, errorCount);
+      const text = appendLspDiagnosticText(message, diagnostics.text);
       return {
         content: [{ type: "text" as const, text }],
         details: {
           ...details,
           pendant: {
-            subtitle: formatSubtitlePath(ctx.cwd, absolutePath, errorCount, warningCount),
+            subtitle: formatSubtitlePath(
+              ctx.cwd,
+              absolutePath,
+              diagnostics.errorCount,
+              diagnostics.warningCount,
+            ),
           },
         },
       };
@@ -647,8 +649,9 @@ function registerWriteTool(pi: ExtensionAPI, getService: () => LspService): void
       });
       const dir = dirname(absolutePath);
 
-      const [message, details, diagnosticText, errorCount, warningCount] =
-        await withFileMutationQueue(absolutePath, async () => {
+      const [message, details, diagnostics] = await withFileMutationQueue(
+        absolutePath,
+        async () => {
           signal?.throwIfAborted();
 
           // opencode: desiredBom = source.bom || next.bom —— 保留原文件 BOM，
@@ -671,28 +674,26 @@ function registerWriteTool(pi: ExtensionAPI, getService: () => LspService): void
           await mkdir(dir, { recursive: true });
           signal?.throwIfAborted();
           await writeFile(absolutePath, desiredBom + nextText, "utf8");
-          const {
-            text: diagnosticText,
-            errorCount,
-            warningCount,
-          } = await getService().lspDiagnosticsForFile(absolutePath, ctx.cwd, { signal });
+          const diagnostics = await getService().lspDiagnosticsForFile(absolutePath, ctx.cwd, {
+            signal,
+          });
 
-          return [
-            "Wrote file successfully.",
-            {},
-            diagnosticText,
-            errorCount,
-            warningCount,
-          ] as const;
-        });
+          return ["Wrote file successfully.", {}, diagnostics] as const;
+        },
+      );
 
-      const text = appendLspDiagnosticText(message, diagnosticText, errorCount);
+      const text = appendLspDiagnosticText(message, diagnostics.text);
       return {
         content: [{ type: "text" as const, text }],
         details: {
           ...details,
           pendant: {
-            subtitle: formatSubtitlePath(ctx.cwd, absolutePath, errorCount, warningCount),
+            subtitle: formatSubtitlePath(
+              ctx.cwd,
+              absolutePath,
+              diagnostics.errorCount,
+              diagnostics.warningCount,
+            ),
           },
         },
       };