pi-skill-evolve 0.0.0-stage → 0.1.0

package/extensions/skill-evolve/test/web-filter.test.ts

@@ -0,0 +1,212 @@
+/**
+ * 列表筛选 + 分页：query 语义、三页各用哪个字段、翻页链接保留筛选参数。
+ * 纯函数测试，不读盘、不起服务。
+ */
+import assert from "node:assert/strict";
+import test from "node:test";
+import type { ApprovalItem, DoneItem, PendingItem, RunReport, WebState } from "../lib/web/data.ts";
+import {
+  NO_FILTER,
+  PAGE_SIZE,
+  buildViewPage,
+  isDefaultFilter,
+  pageHref,
+  paginate,
+  parseListQuery,
+  parseView,
+  rangeStart,
+  type ListFilter,
+} from "../lib/web/filter.ts";
+
+const NOW = Date.UTC(2026, 10, 8, 12);
+const HOUR = 60 * 60 * 1000;
+const DAY = 24 * HOUR;
+
+function filter(overrides: Partial<ListFilter> = {}): ListFilter {
+  return { ...NO_FILTER, ...overrides };
+}
+
+function pending(runId: string, kind: PendingItem["kind"], skill: string, at: number | null): PendingItem {
+  return { runId, kind, skill, at, detail: `${runId} 的详情` };
+}
+
+function done(runId: string, status: string, skill: string, at: number | null): DoneItem {
+  return { runId, status, source: "run", level: "auto", skill, at, detail: `${runId} 的详情` };
+}
+
+function report(runId: string, status: string, skill: string | null, startedAt: number | null): RunReport {
+  return {
+    runId,
+    status,
+    trigger: "manual",
+    startedAt,
+    findings: { total: 0, rows: [] },
+    proposal: skill === null ? null : { type: "skill", skill, added: 1, removed: 0 },
+    review: null,
+    decision: null,
+    verify: null,
+  };
+}
+
+function approval(runId: string, skill: string): ApprovalItem {
+  return { runId, skill, level: "approval", targets: [], createdAt: NOW, proposal: null, review: null, detail: "" };
+}
+
+function state(parts: Partial<WebState> = {}): WebState {
+  return {
+    pending: [],
+    approvals: [],
+    applied: [],
+    done: [],
+    reports: [],
+    summary: { reviewing: 0, stale: 0, pendingApprovals: 0, appliedTotal: 0, rollbackRate: 0 },
+    ...parts,
+  };
+}
+
+test("parseListQuery：缺省 → 全部技能 / 全部状态 / 7d / 第 1 页", () => {
+  assert.deepEqual(parseListQuery(new URLSearchParams()), { skill: "", status: "", range: "7d", page: 1 });
+  assert.equal(parseListQuery(new URLSearchParams("skill=%20demo%20")).skill, "demo", "技能名去空格");
+});
+
+test("parseListQuery：未知 range 与非法 page 一律回落默认（筛选不报错）", () => {
+  assert.equal(parseListQuery(new URLSearchParams("range=nope")).range, "7d");
+  assert.equal(parseListQuery(new URLSearchParams("range=all")).range, "all");
+  for (const page of ["0", "-3", "abc", ""]) {
+    assert.equal(parseListQuery(new URLSearchParams(`page=${page}`)).page, 1, `page=${page} 归 1`);
+  }
+  assert.equal(parseListQuery(new URLSearchParams("page=2")).page, 2);
+});
+
+test("paginate：25 条时 page=2 取第 21-25 条，越界夹到最后一页", () => {
+  const rows = Array.from({ length: 25 }, (_, index) => index + 1);
+  const second = paginate(rows, 2);
+  assert.equal(second.rows[0], 21);
+  assert.equal(second.rows.at(-1), 25);
+  assert.equal(second.page, 2);
+  assert.equal(second.pages, 2);
+  assert.equal(second.total, 25);
+  assert.equal(paginate(rows, 99).page, 2, "页码越界夹到最后一页而不是空列表");
+  assert.equal(paginate(rows, 0).page, 1);
+  const empty = paginate([], 1);
+  assert.deepEqual([empty.page, empty.pages, empty.total], [1, 1, 0], "空列表也要有 1 页，否则页脚消失");
+  assert.equal(PAGE_SIZE, 20);
+});
+
+test("buildViewPage(pending)：skill 精确匹配 + kind→status 映射 + range 过滤", () => {
+  const sample = state({
+    pending: [
+      pending("run-1", "stale", "demo", NOW - HOUR),
+      pending("run-2", "approval", "demo", NOW - 2 * HOUR),
+      pending("run-3", "stale", "demo2", NOW - HOUR),
+      pending("run-4", "stale", "demo", NOW - 8 * DAY),
+      pending("run-5", "stale", "demo", null),
+    ],
+  });
+  const bySkill = buildViewPage(sample, "pending", filter({ skill: "demo", range: "all" }), NOW);
+  assert.deepEqual(
+    bySkill.rows.map((item) => item.runId),
+    ["run-1", "run-2", "run-4", "run-5"],
+    "skill=demo 不能把 demo2 带进来",
+  );
+  const byStatus = buildViewPage(sample, "pending", filter({ status: "approval", range: "all" }), NOW);
+  assert.deepEqual(
+    byStatus.rows.map((item) => item.runId),
+    ["run-2"],
+    "待批的 status 就是 approval",
+  );
+  const recent = buildViewPage(sample, "pending", filter({ range: "24h" }), NOW);
+  assert.deepEqual(
+    recent.rows.map((item) => item.runId),
+    ["run-1", "run-2", "run-3"],
+    "at 为 null 的（run-5）不冒充「最近 24 小时」，8 天前的（run-4）也被挡掉",
+  );
+});
+
+test("buildViewPage(done)：status / skill / range 三个条件叠加", () => {
+  // 已采纳已经单独成 tab，所以 done 里只会有回滚 / 失败 / 否决 / 被顶替这四种终态
+  const sample = state({
+    done: [
+      done("run-a", "rolled-back", "demo", NOW - HOUR),
+      done("run-b", "rejected", "demo", NOW - HOUR),
+      done("run-c", "rolled-back", "other", NOW - HOUR),
+      done("run-d", "rolled-back", "demo", NOW - 10 * DAY),
+    ],
+  });
+  const rolledBack = buildViewPage(sample, "done", filter({ status: "rolled-back", skill: "demo", range: "7d" }), NOW);
+  assert.deepEqual(
+    rolledBack.rows.map((item) => item.runId),
+    ["run-a"],
+  );
+  const all = buildViewPage(sample, "done", filter(), NOW);
+  assert.equal(all.total, 4);
+});
+
+test("buildViewPage(reports)：技能名取自提案，时间用 startedAt", () => {
+  const sample = state({
+    reports: [
+      report("run-p", "analyzed", "demo", NOW - HOUR),
+      report("run-q", "failed", "other", NOW - HOUR),
+      report("run-r", "analyzed", null, NOW - HOUR),
+      report("run-s", "analyzed", "demo", NOW - 20 * DAY),
+    ],
+  });
+  const demo = buildViewPage(sample, "reports", filter({ skill: "demo", range: "7d" }), NOW);
+  assert.deepEqual(
+    demo.rows.map((item) => item.runId),
+    ["run-p"],
+  );
+  const noProposal = buildViewPage(sample, "reports", filter({ range: "all" }), NOW);
+  assert.equal(noProposal.total, 4, "没有提案的报告只在「全部技能」时命中");
+  const failed = buildViewPage(sample, "reports", filter({ status: "failed" }), NOW);
+  assert.deepEqual(
+    failed.rows.map((item) => item.runId),
+    ["run-q"],
+  );
+});
+
+test("buildViewPage(approvals)：不分页，rows 原样（待批本身很短）", () => {
+  const sample = state({ approvals: [approval("run-x", "demo"), approval("run-y", "other")] });
+  const page = buildViewPage(sample, "approvals", filter({ skill: "demo" }), NOW);
+  assert.equal(page.page, 1);
+  assert.equal(page.pages, 1);
+  assert.equal(page.total, 2, "待批页不受筛选影响（筛选表单只在三个列表页渲染）");
+});
+
+test("buildViewPage(done)：45 条时 page=2 拿到第 21-40 条", () => {
+  const rows = Array.from({ length: 45 }, (_, index) => done(`run-${index + 1}`, "failed", "demo", NOW - HOUR));
+  const page = buildViewPage(state({ done: rows }), "done", filter({ page: 2 }), NOW);
+  assert.equal(page.page, 2);
+  assert.equal(page.pages, 3);
+  assert.equal(page.total, 45);
+  assert.equal(page.rows.length, PAGE_SIZE);
+  assert.equal(page.rows[0]?.runId, "run-21");
+  assert.equal(page.rows.at(-1)?.runId, "run-40");
+});
+
+test("pageHref：保留 skill / status / range，空的不写进链接", () => {
+  const href = pageHref("done", filter({ skill: "demo", status: "applied", range: "30d", page: 2 }), 3);
+  const url = new URL(`http://x${href}`);
+  assert.equal(url.searchParams.get("view"), "done");
+  assert.equal(url.searchParams.get("skill"), "demo");
+  assert.equal(url.searchParams.get("status"), "applied");
+  assert.equal(url.searchParams.get("range"), "30d");
+  assert.equal(url.searchParams.get("page"), "3");
+  const bare = new URL(`http://x${pageHref("pending", filter(), 2)}`);
+  assert.equal(bare.searchParams.get("skill"), null);
+  assert.equal(bare.searchParams.get("status"), null);
+  assert.equal(bare.searchParams.get("page"), "2");
+});
+
+test("parseView 白名单 / isDefaultFilter / rangeStart", () => {
+  assert.equal(parseView("reports"), "reports");
+  assert.equal(parseView("done"), "done");
+  for (const bad of [null, "", "nope", "../x"]) assert.equal(parseView(bad), "pending", `parseView(${bad})`);
+  assert.equal(isDefaultFilter(filter({ range: "7d" })), true);
+  assert.equal(isDefaultFilter(filter({ range: "7d", page: 2 })), false);
+  assert.equal(isDefaultFilter(filter({ range: "7d", skill: "demo" })), false);
+  assert.equal(rangeStart("all", NOW), null);
+  assert.equal(rangeStart("24h", NOW), NOW - DAY);
+  assert.equal(rangeStart("7d", NOW), NOW - 7 * DAY);
+  assert.equal(rangeStart("30d", NOW), NOW - 30 * DAY);
+});

package/extensions/skill-evolve/test/web-panel-pid.test.ts

@@ -0,0 +1,97 @@
+/**
+ * 「面板记录」与「端口占用提示」：真监听 127.0.0.1（port=0 取随机端口），不起 node/pi 子进程。
+ *
+ * 为什么要单一个文件：这组用例验的是 `locks/web.json` 的写入/清除与占用文案判读，
+ * 与 HTTP 路由白名单无关；放回 web-server.test.ts 会把那个文件顶过 500 行闸门。
+ */
+import assert from "node:assert/strict";
+import * as fs from "node:fs";
+import * as os from "node:os";
+import * as path from "node:path";
+import test from "node:test";
+import { readWebPid, writeWebPid } from "../lib/web/panel-pid.ts";
+import { startWebPanel, type WebPanel, type WebPanelOptions } from "../lib/web/server.ts";
+
+/** 不存在的 pid：Linux 的 pid_max 远小于 INT32_MAX，这个号系统里不会有进程 */
+const DEAD_PID = 2_147_483_646;
+
+function workspace(): string {
+  const cwd = fs.mkdtempSync(path.join(os.tmpdir(), "skill-evolve-webpanelpid-"));
+  const evolutionDir = path.join(cwd, ".pi", "evolution");
+  fs.mkdirSync(path.join(evolutionDir, "runs"), { recursive: true });
+  return evolutionDir;
+}
+
+function pidFile(evolutionDir: string): string {
+  return path.join(evolutionDir, "locks", "web.json");
+}
+
+async function open(evolutionDir: string, extra: Partial<WebPanelOptions> = {}): Promise<WebPanel> {
+  const result = await startWebPanel({ evolutionDir, port: 0, ...extra });
+  assert.equal(result.ok, true, `面板应该起来：${result.ok ? "" : result.message}`);
+  if (!result.ok) throw new Error("unreachable");
+  return result.panel;
+}
+
+test("起监听即落 locks/web.json：pid 是本进程、url 带实际端口，close() 后消失", async () => {
+  const evolutionDir = workspace();
+  const panel = await open(evolutionDir);
+  try {
+    const record = readWebPid(evolutionDir);
+    assert.ok(record, "起监听之后必须有记录");
+    assert.equal(record.pid, process.pid);
+    // port: 0 时写的是内核分配的实际端口，不是请求值
+    assert.equal(record.port, panel.port);
+    assert.equal(record.url, panel.url);
+    assert.equal(Number.isFinite(record.startedAt), true);
+  } finally {
+    await panel.close();
+  }
+  assert.equal(fs.existsSync(pidFile(evolutionDir)), false, "close() 后记录要删掉");
+});
+
+test("close() 只删自己的记录：文件被换成别的 pid 时保留", async () => {
+  const evolutionDir = workspace();
+  const panel = await open(evolutionDir);
+  writeWebPid(evolutionDir, { pid: DEAD_PID, host: "127.0.0.1", port: panel.port, url: panel.url, startedAt: Date.now() });
+  await panel.close();
+  assert.equal(fs.existsSync(pidFile(evolutionDir)), true, "别人的记录不能被本实例抹掉");
+  assert.equal(readWebPid(evolutionDir)?.pid, DEAD_PID);
+});
+
+test("端口被占且记录活着：提示「已在 … 运行」带 pid，而不是让人去 kill", async () => {
+  const evolutionDir = workspace();
+  const first = await open(evolutionDir);
+  try {
+    const second = await startWebPanel({ evolutionDir, port: first.port });
+    assert.equal(second.ok, false);
+    if (second.ok) throw new Error("同端口不该起来第二份");
+    assert.equal(second.code, "EADDRINUSE");
+    assert.equal(second.message.includes(`已在 ${first.url} 运行`), true);
+    assert.equal(second.message.includes(`pid ${process.pid}`), true);
+    assert.match(second.message, /起于 \d{4}-\d{2}-\d{2}T/);
+    assert.match(second.message, /\/skill-evolve web --port 7799/);
+    assert.equal(second.message.includes("不要直接 kill"), false, "认出是本机面板就不该建议 kill 排查");
+  } finally {
+    await first.close();
+  }
+});
+
+test("端口被占但记录是死的：维持占用文案，提醒先 lsof 看 COMMAND 再动手", async () => {
+  const evolutionDir = workspace();
+  const first = await open(evolutionDir);
+  const port = first.port;
+  try {
+    // 记录换成已死 pid：说明占端口的不是本机面板记录的那个实例
+    writeWebPid(evolutionDir, { pid: DEAD_PID, host: "127.0.0.1", port, url: first.url, startedAt: Date.now() });
+    const second = await startWebPanel({ evolutionDir, port });
+    assert.equal(second.ok, false);
+    if (second.ok) throw new Error("同端口不该起来第二份");
+    assert.equal(second.message.includes("已在 "), false, "记录是死的就不能说面板在跑");
+    assert.match(second.message, /已被占用/);
+    assert.match(second.message, new RegExp(`lsof -nP -iTCP:${port} -sTCP:LISTEN`));
+    assert.match(second.message, /不要直接 kill/);
+  } finally {
+    await first.close();
+  }
+});

package/extensions/skill-evolve/test/web-render-run.test.ts

@@ -0,0 +1,132 @@
+/**
+ * run 详情页的渲染：概要 / diff 着色与截断 / 评审证据表 / 五闸 / 缺块空态。
+ * 直接喂 RunDetail 对象（不读盘）：这一层只负责拼 HTML，读盘由 run-detail.ts 的单测覆盖。
+ */
+import assert from "node:assert/strict";
+import test from "node:test";
+import type { WebState } from "../lib/web/data.ts";
+import type { RunDetail } from "../lib/web/run-detail.ts";
+import { renderRunPage } from "../lib/web/render-run.ts";
+
+const RUN_ID = "354df7db-1794133942";
+
+const DIFF_TEXT = [
+  "--- a/.pi/skills/demo/SKILL.md",
+  "+++ b/.pi/skills/demo/SKILL.md",
+  "@@ -1,2 +1,2 @@",
+  " ---",
+  "-description: 旧",
+  "+description: 新",
+].join("\n");
+
+function state(): WebState {
+  return {
+    pending: [],
+    approvals: [],
+    applied: [],
+    done: [],
+    reports: [],
+    summary: { reviewing: 1, stale: 0, pendingApprovals: 2, appliedTotal: 3, rollbackRate: 0.25 },
+  };
+}
+
+function detail(parts: Partial<RunDetail> = {}): RunDetail {
+  return {
+    runId: RUN_ID,
+    status: "rejected",
+    trigger: "auto",
+    skill: "demo",
+    startedAt: 1794133942865,
+    endedAt: 1794134158913,
+    pid: 65757,
+    hasChildLog: true,
+    diff: DIFF_TEXT,
+    candidateSkill: "demo",
+    candidateText: "---\nname: demo\n---\n# 改后\n",
+    review: { status: "rejected", verdict: "revise", action: "forbidden", reviewerModel: "deepseek/deepseek-flash" },
+    blocking: [
+      {
+        issue: "描述承诺大于正文能力",
+        file: ".pi/skills/demo/SKILL.md",
+        line: 3,
+        quote: "<script> 引用",
+      },
+    ],
+    gates: [
+      { gate: "gate0", name: "授权核对", ok: true, detail: "" },
+      { gate: "gate3", name: "复验", ok: false, detail: "复验失败" },
+    ],
+    verifyStatus: "applied",
+    findings: [{ skill: "demo", reason: "G", confidence: "high", lines: [578, 622] }],
+    findingsTotal: 1,
+    notes: "",
+    ...parts,
+  };
+}
+
+test("概要：runId / 状态 / 技能 / 触发 / 开始结束 / pid / child.log 一次列全", () => {
+  const html = renderRunPage(detail(), { state: state() });
+  assert.match(html, /<title>技能进化 · run 354df7db-1794133942<\/title>/);
+  assert.match(html, /<span class="k">runId<\/span><span class="v"><code>354df7db-1794133942<\/code><\/span>/);
+  assert.match(html, /<span class="k">状态<\/span><span class="v">rejected<\/span>/);
+  assert.match(html, /<span class="k">技能<\/span><span class="v">demo<\/span>/);
+  assert.match(html, /<span class="k">触发<\/span><span class="v">auto<\/span>/);
+  assert.match(html, /<span class="k">pid<\/span><span class="v">65757<\/span>/);
+  assert.match(html, /<a href="\/log\?runId=354df7db-1794133942">child\.log<\/a>/);
+  assert.match(html, /已采纳 <a href="\/\?view=applied">3<\/a>/, "详情页也带同一份状态栏");
+});
+
+test("diff：+ 绿底 / - 红底 / @@ 灰字三类 class 与两列行号都在", () => {
+  const html = renderRunPage(detail(), { state: state() });
+  assert.match(html, /<pre class="diff">/);
+  assert.match(html, /<span class="dl file">/);
+  assert.match(html, /<span class="dl hunk">/);
+  assert.match(html, /<span class="dl del"><span class="ln old">2<\/span><span class="ln new"> {1}<\/span>/);
+  assert.match(html, /<span class="dl add"><span class="ln old"> {1}<\/span><span class="ln new">2<\/span>/);
+  assert.match(html, /diff 共 6 行。/);
+  assert.equal(html.includes("完整 diff"), false, "没截断就不给下载链接");
+});
+
+test("diff 超 400 行：只渲前 400 行，并给 /diff 完整下载链接", () => {
+  const body = Array.from({ length: 500 }, (_unused, index) => ` line ${index + 1}`);
+  const diff = ["--- a/x", "+++ b/x", "@@ -1,500 +1,500 @@", ...body].join("\n");
+  const html = renderRunPage(detail({ diff }), { state: state() });
+  assert.match(html, /diff 共 503 行，页面只渲染前 400 行/);
+  assert.match(html, /<a class="btn" href="\/diff\?runId=354df7db-1794133942">完整 diff<\/a>/);
+  assert.equal(html.match(/class="dl /g)?.length, 400);
+});
+
+test("评审结论：verdict/action/模型 + blocking 表格（file:line 与引用），不是把 JSON 倒出来", () => {
+  const html = renderRunPage(detail(), { state: state() });
+  assert.match(html, /verdict <b class="fail">revise<\/b>/);
+  assert.match(html, /action <b>forbidden<\/b>/);
+  assert.match(html, /<code>deepseek\/deepseek-flash<\/code>/);
+  assert.match(html, /<td>描述承诺大于正文能力<\/td><td><code>\.pi\/skills\/demo\/SKILL\.md:3<\/code><\/td>/);
+  assert.match(html, /&lt;script&gt; 引用/, "引用原文也要转义");
+  assert.equal(html.includes('"blocking"'), false, "blocking 要做成表格，不许把 JSON 直接倒出来");
+});
+
+test("五闸：逐条 ok/fail 上色，失败那条带说明", () => {
+  const html = renderRunPage(detail(), { state: state() });
+  assert.match(html, /<td><code>gate0<\/code><\/td><td>授权核对<\/td><td class="ok">ok<\/td>/);
+  assert.match(html, /<td><code>gate3<\/code><\/td><td>复验<\/td><td class="fail">fail<\/td><td>复验失败<\/td>/);
+  assert.match(html, /验证状态：applied/);
+});
+
+test("缺块空态：diff / 评审 / 五闸 / 候选全文各说清缺的是哪个文件", () => {
+  const html = renderRunPage(
+    detail({ diff: null, review: null, gates: [], verifyStatus: null, candidateText: null, candidateSkill: null, findingsTotal: 0, findings: [] }),
+    { state: state() },
+  );
+  assert.match(html, /这个 run 没有 02-proposal\.diff/);
+  assert.match(html, /没有 03-review\.json：这一轮还没评审完，或被跳过。/);
+  assert.match(html, /没有 06-verify\.json：五闸未执行（这个 run 没被采纳）。/);
+  assert.match(html, /没有 candidate\/demo\/SKILL\.md/);
+  assert.match(html, /01-findings\.json 里没有条目/);
+});
+
+test("详情页不高亮任何 tab：点上去回不到这一页，highligh 就是在骗人", () => {
+  const html = renderRunPage(detail(), { state: state() });
+  const nav = html.slice(html.indexOf("<nav>"), html.indexOf("</nav>"));
+  assert.equal(nav.includes('class="active"'), false);
+});

package/extensions/skill-evolve/test/web-render-skills.test.ts

@@ -0,0 +1,134 @@
+/**
+ * 技能管理页与技能详情页的渲染：六列表格、加载率的「—」、两种遥测空态、历史提案表。
+ * 直接喂 SkillsView / SkillDetail（不读盘）。
+ */
+import assert from "node:assert/strict";
+import test from "node:test";
+import type { WebState } from "../lib/web/data.ts";
+import { renderSkillPage, renderSkillsPage } from "../lib/web/render-skills.ts";
+import type { SkillDetail, SkillRow, SkillsView } from "../lib/web/skills.ts";
+
+function state(): WebState {
+  return {
+    pending: [],
+    approvals: [],
+    applied: [],
+    done: [],
+    reports: [],
+    summary: { reviewing: 0, stale: 0, pendingApprovals: 0, appliedTotal: 0, rollbackRate: 0 },
+  };
+}
+
+function row(name: string, parts: Partial<SkillRow> = {}): SkillRow {
+  return {
+    name,
+    exposed: 0,
+    loaded: 0,
+    rate: null,
+    lastRunId: null,
+    lastRunAt: null,
+    hasPendingProposal: false,
+    hasBaseline: false,
+    missingCursorStub: false,
+    ...parts,
+  };
+}
+
+function view(rows: SkillRow[], parts: Partial<SkillsView> = {}): SkillsView {
+  return { days: 30, rows, telemetryOn: true, events: 10, ...parts };
+}
+
+function detail(parts: Partial<SkillDetail> = {}): SkillDetail {
+  return { name: "demo", days: 30, row: row("demo"), runs: [], telemetryOn: true, events: 10, ...parts };
+}
+
+test("技能列表：六列齐全，技能名与最近 run 都能点进详情", () => {
+  const html = renderSkillsPage(
+    view([
+      row("demo", {
+        exposed: 4,
+        loaded: 1,
+        rate: 0.25,
+        lastRunId: "354df7db-1794133942",
+        lastRunAt: 1794133942865,
+        hasPendingProposal: true,
+        missingCursorStub: true,
+      }),
+      row("quiet"),
+    ]),
+    state(),
+  );
+  assert.match(html, /<title>技能进化 · 技能管理<\/title>/);
+  for (const head of ["技能", "曝光", "加载", "加载率", "最近一次相关 run", "状态"]) {
+    assert.match(html, new RegExp(`<th>${head}</th>`));
+  }
+  assert.match(html, /<h2>技能（2 个，窗口 30 天）<\/h2>/);
+  assert.match(html, /<a href="\/\?view=skill&amp;name=demo&amp;days=30">demo<\/a><\/td><td>4<\/td><td>1<\/td><td>25%<\/td>/);
+  assert.match(html, /<a href="\/\?view=run&amp;runId=354df7db-1794133942">354df7db…3942<\/a>/);
+  assert.match(html, /<span class="tag warn">有待批提案<\/span>/);
+  assert.match(html, /<span class="tag">缺 \.cursor stub<\/span>/);
+  // 没曝光的技能写「—」而不是 0%：0% 会被当成「一百次曝光一次没加载」
+  assert.match(html, /<td><a href="\/\?view=skill&amp;name=quiet&amp;days=30">quiet<\/a><\/td><td>0<\/td><td>0<\/td><td>—<\/td>/);
+});
+
+test("空遥测两种说法：采集关着 vs 采集开着但窗口内没事件", () => {
+  const off = renderSkillsPage(view([row("demo")], { telemetryOn: false, events: 0 }), state());
+  assert.match(off, /没有遥测数据（采集关着）/);
+  assert.match(off, /曝光与加载都是 0 是因为没采集/);
+  // 采集关着时数值是「不知道」而不是 0：一屏 0 会被读成「这些技能没人用」
+  assert.match(off, /<td><a href="\/\?view=skill&amp;name=demo&amp;days=30">demo<\/a><\/td><td>—<\/td><td>—<\/td><td>—<\/td>/);
+  const on = renderSkillsPage(view([row("demo")], { telemetryOn: true, events: 0, days: 7 }), state());
+  assert.match(on, /最近 7 天没有遥测事件/);
+  assert.equal(on.includes("采集关着"), false);
+});
+
+test("窗口切换：当前窗口是字不是链接，另外两个可点", () => {
+  const html = renderSkillsPage(view([row("demo")], { days: 7 }), state());
+  assert.match(html, /<span class="on">近 7 天<\/span>/);
+  assert.match(html, /<a href="\/\?view=skills&amp;days=30">近 30 天<\/a>/);
+  assert.match(html, /近 90 天/);
+});
+
+test("技能详情：统计一行 + 历史提案表（run / 状态 / 时间 / 提案 / verdict / action）", () => {
+  const html = renderSkillPage(
+    detail({
+      row: row("demo", { exposed: 4, loaded: 2, rate: 0.5, hasBaseline: true }),
+      runs: [
+        {
+          runId: "354df7db-1794133942",
+          status: "rejected",
+          at: 1794133942865,
+          type: "improve",
+          added: 3,
+          removed: 1,
+          verdict: "revise",
+          action: "forbidden",
+        },
+      ],
+    }),
+    state(),
+  );
+  assert.match(html, /<title>技能进化 · 技能 demo<\/title>/);
+  assert.match(html, /曝光 <b>4<\/b> · 加载 <b>2<\/b> · 加载率 <b>50%<\/b> · 窗口 30 天/);
+  assert.match(html, /<span class="tag fail">存量违规<\/span>/);
+  assert.match(html, /<a href="\/\?view=run&amp;runId=354df7db-1794133942">354df7db…3942<\/a><\/td><td>rejected<\/td>/);
+  assert.match(html, /improve \+3\/-1<\/td><td>revise<\/td><td>forbidden<\/td>/);
+  assert.match(html, /<a href="\/\?view=skills">← 回技能管理<\/a>/);
+});
+
+test("技能详情空态：没有提案时说明是 runs/ 里没这个技能的提案，不是页面坏了", () => {
+  const html = renderSkillPage(detail(), state());
+  assert.match(html, /还没有针对 demo 的提案：runs\/ 里没有 skill=demo 的 02-proposal\.json。/);
+  const gone = renderSkillPage(detail({ name: "gone", row: null, runs: [] }), state());
+  assert.match(gone, /gone 不在 \.pi\/skills 目录里（可能已改名或删除）/);
+});
+
+test("技能详情也守同一条：采集关着时曝光/加载写「—」，不写 0", () => {
+  const html = renderSkillPage(detail({ telemetryOn: false, events: 0 }), state());
+  assert.match(html, /曝光 <b>—<\/b> · 加载 <b>—<\/b> · 加载率 <b>—<\/b> · 窗口 30 天/);
+});
+
+test("技能列表空目录：说清是工作区根路径推断错了，不是「没有技能」", () => {
+  const html = renderSkillsPage(view([], { events: 0 }), state());
+  assert.match(html, /\.pi\/skills 下没有技能目录/);
+});