@trim21/personal-pi-extensions 0.1.20261012000609 → 0.1.20261012003405

package/README.md

@@ -475,7 +475,7 @@ GitHub 只读工具集。取数全部走 octokit REST（`src/lib/github-reads.ts
 - **CI**：`read-github-ci-logs`、`list-github-workflow-runs`、`get-github-workflow-jobs`、`wait-github-pr-checks`、`wait-github-commit-checks`、`watch-github-run`
 - **仓库 / 发布**：`read-github-repo`、`list-github-releases`、`read-github-release`、`download-github-release-assets`
 
-`list-github-issues` / `list-github-prs` 带 `keywords` 时走 GitHub search API，不带 `keywords` 时走 REST 列表端点（state 缺省只搜 open，`state: "all"` 不加 state qualifier 覆盖 open + closed），输出 TSV，默认列 `number,state,title,labels,updatedAt`，可用 `fields` 白名单指定列。`wait-github-pr-checks` / `wait-github-commit-checks` 30 秒轮询、600 秒截止：任一 fail 即返回，全部 pass / skipped 才算通过，超时返回快照不抛错；检查快照还包含「已结束但没有 check run」的 check suite——fork PR 上等待维护者批准的 workflow 只存在于这一层，遇到它立即返回 `awaiting_approval`（不会自行开始，轮询无意义），条目名字取自对应的 workflow run。
+`list-github-issues` / `list-github-prs` 带 `keywords` 时走 GitHub search API，不带 `keywords` 时走 REST 列表端点（state 缺省只搜 open，`state: "all"` 不加 state qualifier 覆盖 open + closed），输出 TSV，固定列 `number,state,title,labels,updatedAt`（未给 `repo` 时最前面多一列 `repo`），默认列之外的字段从结构化载荷 `items` 里取。`wait-github-pr-checks` / `wait-github-commit-checks` 30 秒轮询、600 秒截止：任一 fail 即返回，全部 pass / skipped 才算通过，超时返回快照不抛错；检查快照还包含「已结束但没有 check run」的 check suite——fork PR 上等待维护者批准的 workflow 只存在于这一层，遇到它立即返回 `awaiting_approval`（不会自行开始，轮询无意义），条目名字取自对应的 workflow run。
 
 出网代理：`~/.pi/agent/proxy.json`（`{ "proxy": "http://127.0.0.1:7890", "noProxy": "localhost" }`），缺省字段回退 `HTTPS_PROXY` / `HTTP_PROXY` / `ALL_PROXY` / `NO_PROXY` 环境变量。
 

package/package.json

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

package/src/gh/base.ts

@@ -334,8 +334,9 @@ export function withPendant<T extends ToolResult, IdKey extends string = never>(
 }
 
 /**
- * `list-github-issues` / `list-github-prs` 共用的一套过滤参数，只有 `state` 与 `fields`
- * 的说明文案按 kind 不同。
+ * `list-github-issues` / `list-github-prs` 共用的一套过滤参数，只有 `state` 的说明文案按
+ * kind 不同。文本列是固定的（见 `renderHits`），因此没有列选择参数：默认列之外的字段
+ * 从结构化载荷里取。
  */
 export function listFilterParameters(kind: "issue" | "pr") {
   const isPr = kind === "pr";
@@ -364,11 +365,6 @@ export function listFilterParameters(kind: "issue" | "pr") {
     ),
     milestone: Type.Optional(Type.String({ description: "Filter by milestone" })),
     limit: Type.Optional(Type.Number({ description: "Max results (default 30, max 100)" })),
-    fields: Type.Optional(
-      Type.String({
-        description: `Comma-separated columns for the rows (default: number,state,title,labels,updatedAt; adds repo when no repo given). Valid: number,state,title,url,author,labels,milestone,assignees,comments,repo,createdAt,updatedAt,closedAt${isPr ? ",mergedAt" : ""}`,
-      }),
-    ),
   });
 }
 
@@ -381,8 +377,6 @@ export interface ListFilters {
   assignee?: string;
   milestone?: string;
   limit?: number;
-  /** Comma-separated field names for the keyword-search result rows. */
-  fields?: string;
 }
 
 /**
@@ -415,11 +409,11 @@ export async function browseList(
   return { repo, hits: normalizeRestList(items, repo) };
 }
 
-/** 行列表的文本：两条分支共用一套 TSV 渲染；空结果是成功结果。 */
+/** 行列表的文本：两条分支共用一套 TSV 渲染（列为固定默认列）；空结果是成功结果。 */
 export function renderHitList(
   kind: "issue" | "pr",
   hits: SearchHit[],
-  options: { repo?: string; fields?: string },
+  options: { repo?: string },
 ): string {
   if (hits.length === 0) {
     return `(no matching ${kind === "issue" ? "issues" : "pull requests"})`;

package/src/gh/tools/list-issues.ts

@@ -26,7 +26,7 @@ async function listIssues(
   const { repo, hits } = params.keywords
     ? { repo: params.repo, hits: await gh.search.search("issue", params) }
     : await browseList(gh, "issue", params, { cwd: ctx.cwd, signal, input: params });
-  const text = renderHitList("issue", hits, { repo, fields: params.fields });
+  const text = renderHitList("issue", hits, { repo });
   const result = toToolResult(text, params);
   // 文本可能被行数/字节预算截断，载荷给全部命中
   return withStructuredResult(withPendant(result, params), { text, items: hits });

package/src/gh/tools/list-prs.ts

@@ -26,7 +26,7 @@ async function listPrs(
   const { repo, hits } = params.keywords
     ? { repo: params.repo, hits: await gh.search.search("pr", params) }
     : await browseList(gh, "pr", params, { cwd: ctx.cwd, signal, input: params });
-  const text = renderHitList("pr", hits, { repo, fields: params.fields });
+  const text = renderHitList("pr", hits, { repo });
   const result = toToolResult(text, params);
   // 文本可能被行数/字节预算截断，载荷给全部命中
   return withStructuredResult(withPendant(result, params), { text, items: hits });

package/src/lib/github.ts

@@ -146,42 +146,19 @@ function quoteQualifier(value: string): string {
   return `"${value.replaceAll('"', String.raw`\"`)}"`;
 }
 
-const FIELD_EXTRACTORS: Record<string, (hit: SearchHit) => string> = {
-  number: (h) => String(h.number),
-  state: (h) => h.state,
-  title: (h) => h.title,
-  url: (h) => h.url,
-  repo: (h) => h.repo,
-  author: (h) => h.author,
-  labels: (h) => h.labels.join(","),
-  milestone: (h) => h.milestone,
-  assignees: (h) => h.assignees.join(","),
-  comments: (h) => String(h.comments),
-  createdAt: (h) => h.createdAt,
-  updatedAt: (h) => h.updatedAt,
-  closedAt: (h) => h.closedAt,
-  mergedAt: (h) => h.mergedAt,
-};
-
-export const SEARCH_FIELDS: readonly string[] = Object.keys(FIELD_EXTRACTORS);
-
-/** Render search hits as tab-separated rows; one row per hit, one column per field. */
-export function renderHits(hits: SearchHit[], options: { repo?: string; fields?: string }): string {
-  const requested = options.fields
-    ? options.fields
-        .split(",")
-        .map((f) => f.trim())
-        .filter(Boolean)
-    : options.repo
-      ? ["number", "state", "title", "labels", "updatedAt"]
-      : ["repo", "number", "state", "title", "labels", "updatedAt"];
-  const unknown = requested.filter((f) => !(f in FIELD_EXTRACTORS));
-  if (unknown.length > 0) {
-    throw new Error(
-      `unknown field${unknown.length > 1 ? "s" : ""}: ${unknown.join(", ")} (valid: ${SEARCH_FIELDS.join(", ")})`,
-    );
-  }
-  return hits.map((hit) => requested.map((f) => FIELD_EXTRACTORS[f](hit)).join("\t")).join("\n");
+/**
+ * Render search hits as tab-separated rows: one row per hit, fixed columns
+ * `number,state,title,labels,updatedAt`, plus a leading `repo` column when no
+ * repo is given (cross-repo search). Other fields of a hit reach the caller
+ * through the structured payload, not through the text.
+ */
+export function renderHits(hits: SearchHit[], options: { repo?: string }): string {
+  return hits
+    .map((hit) => {
+      const columns = [hit.number, hit.state, hit.title, hit.labels.join(","), hit.updatedAt];
+      return (options.repo ? columns : [hit.repo, ...columns]).join("\t");
+    })
+    .join("\n");
 }
 
 function toDate(iso: string | null | undefined): string {