pi-skill-evolve 0.1.7 → 0.1.9

package/extensions/skill-evolve/lib/web/render-controls.ts

@@ -21,7 +21,7 @@ import {
   telemetryOffWhileEnabled,
 } from "../flags.ts";
 import type { WebView } from "./data.ts";
-import { RANGE_KEYS, RANGE_LABEL, STATUS_OPTIONS, type ListFilter, type ViewPage } from "./filter.ts";
+import { RANGE_KEYS, RANGE_LABEL, type ListFilter, type ViewPage } from "./filter.ts";
 import { listHref, type ListExtras } from "./list-query.ts";
 import { escapeHtml, formatStamp, formatTimeSeconds } from "./text.ts";
 import type { WebControls } from "./activity.ts";
@@ -106,15 +106,7 @@ ${run}</div>
 ${warn}${backlog}`;
 }
 
-/** 状态下拉：先「全部」，再固定七个状态（顺序与 STATUS_OPTIONS 一致，别在渲染层重排） */
-function statusSelect(current: string): string {
-  const options = [`<option value="">全部状态</option>`];
-  for (const status of STATUS_OPTIONS) {
-    options.push(`<option value="${status}"${status === current ? " selected" : ""}>${status}</option>`);
-  }
-  return options.join("");
-}
-
+/** 时间下拉：只有「看多久内」需要人手动换，状态与技能已在 tab 与技能页面各自解决了 */
 function rangeSelect(current: string): string {
   return RANGE_KEYS.map(
     (key) => `<option value="${key}"${key === current ? " selected" : ""}>${RANGE_LABEL[key]}</option>`,
@@ -122,16 +114,16 @@ function rangeSelect(current: string): string {
 }
 
 /**
- * 筛选表单：hidden view 保证 GET 提交后还在同一页；「重置」链回不带筛选的本页。
- * `open` / `live` 也一并带上：筛选不该顺手把「展开分组」「自动刷新」这两个开关清掉。
+ * 筛选表单：只留「时间」一个条件。
+ *   技能：每个列表页的行里已经有技能列，技能页可以按技能点进来，再摆一个模糊搜索框没人知道该填什么；
+ *   状态：顶部 tab 本身就是按状态分的（待处理 / 待审批 / 已采纳 / 已处理 / 复盘报告），重复选毫无意义。
+ * 两个参数（skill / status）仍由 `parseListQuery` 支持，旧链接与代码内部跳转不会断，只是不再出输入框。
  */
 export function renderFilterForm(view: WebView, filter: ListFilter, extras: ListExtras): string {
   const keep = extraInputs(extras);
   return `<form class="filter" method="get">
 <input type="hidden" name="view" value="${view}">
-${keep}<label>技能 <input type="text" name="skill" value="${escapeHtml(filter.skill)}" placeholder="全部技能" size="16"></label>
-<label>状态 <select name="status">${statusSelect(filter.status)}</select></label>
-<label>时间 <select name="range">${rangeSelect(filter.range)}</select></label>
+${keep}<label>时间 <select name="range">${rangeSelect(filter.range)}</select></label>
 <button type="submit">筛选</button>
 <a class="btn" href="${escapeHtml(listHref(view, filter, 1, extras))}">重置</a>
 </form>`;

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

@@ -0,0 +1,60 @@
+/**
+ * 筛选栏的验收测试（用户 2026-11-08：「这个搜索框的技能和状态不用了吧, 技能不知道用来搜索什么,
+ * 状态应该在顶部 tab 包含了吧」）。
+ *
+ * 口径：
+ * 1. 表单**只剩「时间」**一个条件——技能看不出该填什么、状态与顶部 tab 重复，两个都撤掉；
+ * 2. 撤掉的是**输入控件**，不是参数：`?skill=` / `?status=` 仍由 `parseListQuery` 解析，
+ *    技能页点进来、旧书签、代码内部跳转都不会断；
+ * 3. 表单仍是 GET + 保留 `view`，否则一提交就跳出当前页。
+ *
+ * 纪律：纯字符串断言，不碰文件系统、不起服务。
+ */
+import { test } from "node:test";
+import assert from "node:assert/strict";
+
+import { parseListQuery } from "../lib/web/filter.ts";
+import { renderFilterForm } from "../lib/web/render-controls.ts";
+
+const FILTER = parseListQuery(new URLSearchParams(""));
+const EXTRAS = { live: false, open: "all", only: false } as never;
+
+function form(): string {
+  return renderFilterForm("pending", FILTER, EXTRAS);
+}
+
+test("筛选栏不再有技能输入框与状态下拉", () => {
+  const html = form();
+  assert.ok(!html.includes('name="skill"'), "技能搜索框已撤掉");
+  assert.ok(!html.includes('name="status"'), "状态下拉已撤掉（顶部 tab 就是状态）");
+  assert.ok(!html.includes("全部状态"));
+  assert.ok(!html.includes("全部技能"));
+});
+
+test("时间筛选还在，且三个档位齐全", () => {
+  const html = form();
+  assert.ok(html.includes('name="range"'));
+  for (const label of ["最近 7 天", "最近 30 天", "全部时间"]) {
+    assert.ok(html.includes(`>${label}<`), `缺时间档位：${label}`);
+  }
+});
+
+test("表单仍是 GET，且保留 view", () => {
+  const html = form();
+  assert.ok(html.includes('method="get"'));
+  assert.ok(html.includes('<input type="hidden" name="view" value="pending">'));
+});
+
+test("参数没被删：URL 里的 skill / status 仍然生效", () => {
+  const parsed = parseListQuery(new URLSearchParams("skill=deploy-test&status=stale&range=30d"));
+  assert.equal(parsed.skill, "deploy-test");
+  assert.equal(parsed.status, "stale");
+  assert.equal(parsed.range, "30d");
+});
+
+test("技能页跳过来的链接照旧能带 skill 参数（只是不再有输入框）", () => {
+  const parsed = parseListQuery(new URLSearchParams("skill=ui-verify"));
+  const html = renderFilterForm("reports", parsed, EXTRAS);
+  assert.ok(!html.includes('name="skill"'));
+  assert.equal(parsed.skill, "ui-verify", "筛选逻辑要能吃到链接里的技能");
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-skill-evolve",
-  "version": "0.1.7",
+  "version": "0.1.9",
   "description": "技能进化：采集会话中的技能曝光/加载，异步复盘、生成改进提案并独立评审，附本机 Web 管理面板",
   "keywords": [
     "pi-package",