@trim21/personal-pi-extensions 0.1.517 → 0.1.519

package/package.json

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

package/src/gh-readonly.ts

@@ -1134,86 +1134,30 @@ const PR_CHECKS_JSON_FIELDS = "name,state,bucket,startedAt,completedAt,link,work
 const CHECKS_POLL_INTERVAL_MS = 30_000;
 const CHECKS_WATCH_DEADLINE_MS = 600_000;
 
-function bucketIcon(bucket: string): string {
-  switch (bucket) {
-    case "pass": {
-      return "✅";
-    }
-    case "fail": {
-      return "❌";
-    }
-    case "skipping": {
-      return "⏭️";
-    }
-    case "cancel": {
-      return "🚫";
-    }
-    default: {
-      return "🔄";
-    }
-  }
-}
-
-function formatClock(ms: number): string {
-  const date = new Date(ms);
-  return [date.getHours(), date.getMinutes(), date.getSeconds()]
-    .map((n) => String(n).padStart(2, "0"))
-    .join(":");
-}
-
-function pad2(n: number): string {
-  return String(n).padStart(2, "0");
-}
-
-function formatDuration(totalSeconds: number): string {
-  const seconds = Math.floor(totalSeconds);
-  const h = Math.floor(seconds / 3600);
-  const m = Math.floor((seconds % 3600) / 60);
-  const s = seconds % 60;
-  if (h > 0) return `${h}h${pad2(m)}m${pad2(s)}s`;
-  if (m > 0) return `${m}m${pad2(s)}s`;
-  return `${s}s`;
-}
-
 /**
- * Render one polling round of `gh pr checks` like the GitHub web UI: check
- * name, workflow, status, start time and elapsed (total duration when the
- * check completed, time-so-far while still pending). Pure — no network.
+ * Render one polling round of `gh pr checks` as a compact bullet list of the
+ * checks still in flight: running ones first (`- [>]`), queued ones after
+ * (`- [ ]`). Completed checks are hidden — the header already reports the
+ * completion count. Pure — no network.
  */
-export function renderPrChecksTable(options: {
+export function renderPrChecksList(options: {
   prNumber: number | string;
   round: number;
   checks: readonly PrCheck[];
-  now: number;
 }): string {
-  const { prNumber, round, checks, now } = options;
+  const { prNumber, round, checks } = options;
   const completed = checks.filter((c) => c.bucket !== "pending").length;
 
-  const rows = checks.map((check) => {
+  const pending = checks.filter((c) => c.bucket === "pending");
+  const ordered = [...pending.filter((c) => c.startedAt), ...pending.filter((c) => !c.startedAt)];
+  const lines = ordered.map((check) => {
     const name = check.link ? `[${check.name}](${check.link})` : check.name;
-    let elapsed = "—";
-    if (check.startedAt) {
-      const startMs = Date.parse(check.startedAt);
-      const endMs = check.completedAt ? Date.parse(check.completedAt) : now;
-      elapsed = formatDuration(Math.max(0, (endMs - startMs) / 1000));
-    }
-    const cells = [
-      `${bucketIcon(check.bucket)} ${name}`,
-      check.workflow ?? "—",
-      check.bucket,
-      check.startedAt ? formatClock(Date.parse(check.startedAt)) : "—",
-      elapsed,
-    ];
-    return `| ${cells.join(" | ")} |`;
+    return `- [${check.startedAt ? ">" : " "}] ${name}`;
   });
-  const body = rows.length > 0 ? rows.join("\n") : "| _no checks reported_ | — | — | — | — |";
+  const body =
+    checks.length === 0 ? "- _no checks reported_" : lines.length > 0 ? lines.join("\n") : "";
 
-  return (
-    `### PR #${prNumber} checks — round ${round}: ${completed}/${checks.length} complete\n\n` +
-    `| Check | Workflow | Status | Started | Elapsed |\n` +
-    `|---|---|---|---|---|\n` +
-    body
-  );
+  return `### PR #${prNumber} checks — round ${round}: ${completed}/${checks.length} complete${body ? `\n\n${body}` : ""}`;
 }
 
 function sleepInterruptibly(ms: number, signal: AbortSignal | undefined): Promise<void> {
@@ -1248,8 +1192,9 @@ export interface PollPrChecksOptions {
 
 /**
  * Poll `gh pr checks --json` until no check is pending (or a fail-fast
- * failure, or the deadline), emitting a GitHub-UI-like table via `onUpdate`
- * each round. In JSON mode gh exits 0 whenever it could fetch the checks —
+ * failure, or the deadline), emitting a compact list of in-flight checks via
+ * `onUpdate` each round. In JSON mode gh exits 0 whenever it could fetch the
+ * checks —
  * completion is judged from the `bucket` field, not the exit code. A non-zero
  * exit (no checks reported, auth, network) is not fatal here: the caller
  * proceeds to the Actions API verification, which either produces the final
@@ -1282,9 +1227,7 @@ export async function pollPrChecks(options: PollPrChecksOptions): Promise<void>
     const checks: PrCheck[] = Value.Parse(Type.Array(prCheckSchema), JSON.parse(result.stdout));
 
     onUpdate?.({
-      content: [
-        { type: "text", text: renderPrChecksTable({ prNumber, round, checks, now: Date.now() }) },
-      ],
+      content: [{ type: "text", text: renderPrChecksList({ prNumber, round, checks }) }],
       details: {},
     });
 
@@ -1852,7 +1795,7 @@ export default function ghReadonlyTools(pi: ExtensionAPI) {
     label: "Watch GitHub PR Checks",
     description:
       "Watch CI status checks for a PR until they complete. Blocks until all checks finish or one fails. " +
-      "Each polling round streams a GitHub-UI-like check table (name, workflow, status, started, elapsed) via onUpdate. " +
+      "Each polling round streams a compact bullet list of the checks still in flight via onUpdate. " +
       "Use this when you need to wait for CI to complete and see the final result.",
     promptSnippet: "Watch and wait for GitHub PR CI checks to complete",
     parameters: Type.Object({

package/src/lib/lsp/client.ts

@@ -213,10 +213,39 @@ function toInspectLocations(result: DefinitionResult): InspectLocation[] {
   return locations;
 }
 
+/** stderr 尾部保留上限（字符）：足够容纳启动失败的最后报错，又不撑爆通知。 */
+const STDERR_TAIL_CHARS = 4_000;
+
+/** 展开 Error 的 cause 链为一条 message；流包装错误只包一层，逐层展开即可还原根因。 */
+function errorChainMessage(error: unknown): string {
+  if (!(error instanceof Error)) return String(error);
+  const parts: string[] = [];
+  let current: unknown = error;
+  while (current instanceof Error && current.message && !parts.includes(current.message)) {
+    parts.push(current.message);
+    current = current.cause;
+  }
+  if (typeof current === "string" || typeof current === "number") parts.push(String(current));
+  return parts.join(": ");
+}
+
+/** 拼装握手失败的完整原因：错误链 + 进程退出状态 + stderr 尾部。 */
+function describeStartupFailure(
+  error: unknown,
+  exitDescription: string | undefined,
+  stderrTail: string,
+): string {
+  const parts = [errorChainMessage(error)];
+  if (exitDescription) parts.push(`server ${exitDescription}`);
+  const stderr = stderrTail.trim();
+  if (stderr) parts.push(`stderr: ${stderr}`);
+  return parts.join("; ");
+}
+
 export class InitializeError extends Error {
   readonly serverID: string;
-  constructor(serverID: string, cause: unknown) {
-    super(`Failed to initialize LSP server ${serverID}`, { cause });
+  constructor(serverID: string, cause: unknown, detail?: string) {
+    super(`Failed to initialize LSP server ${serverID}${detail ? `: ${detail}` : ""}`, { cause });
     this.serverID = serverID;
   }
 }
@@ -443,11 +472,19 @@ export async function create(input: CreateInput): Promise<LspClient> {
     new StreamMessageReader(input.server.process.stdout),
     new StreamMessageWriter(input.server.process.stdin),
   );
-  input.server.process.stderr.resume();
+  // stderr 平时只在尾部保留少量内容（避免子进程大量输出撑爆内存）；
+  // 握手失败时随错误输出，服务器 panic / 参数错误等启动原因由此还原。
+  let stderrTail = "";
+  input.server.process.stderr.setEncoding("utf8");
+  input.server.process.stderr.on("data", (chunk: string) => {
+    stderrTail = (stderrTail + chunk).slice(-STDERR_TAIL_CHARS);
+  });
   /** 连接或服务器进程已关闭；pull 重试循环以此终止，避免无界等待。 */
   let connectionClosed = false;
-  input.server.process.once("exit", () => {
+  let exitDescription: string | undefined;
+  input.server.process.once("exit", (code, signal) => {
     connectionClosed = true;
+    exitDescription = code === null ? `killed by signal ${signal}` : `exited with code ${code}`;
   });
   connection.onDispose(() => {
     connectionClosed = true;
@@ -582,7 +619,11 @@ export async function create(input: CreateInput): Promise<LspClient> {
     connection.end();
     connection.dispose();
     await stopProcess(input.server.process);
-    throw new InitializeError(input.serverID, error);
+    throw new InitializeError(
+      input.serverID,
+      error,
+      describeStartupFailure(error, exitDescription, stderrTail),
+    );
   });
 
   const syncKind = getSyncKind(initialized.capabilities);