pi-skill-evolve 0.1.8 → 0.1.9

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

@@ -29,21 +29,6 @@ export const RANGE_LABEL: Record<RangeKey, string> = {
 /** 状态下拉的可选值：既有 run 终态，也有待处理页自己的两种（running / stale） */
 export const STATUS_OPTIONS = ["running", "stale", "reviewed", "applied", "rejected", "failed", "analyzed"] as const;
 
-/**
- * 状态下拉的中文标签（用户 2026-11-08：「这里的下拉应该是中文」）。
- * 值仍然是状态机原文（URL 与过滤逻辑不变），只改给人看的那一侧；
- * 与徽章文案保持同一套词——同一个概念全站只能有一个中文名。
- */
-export const STATUS_LABEL: Record<(typeof STATUS_OPTIONS)[number], string> = {
-  running: "复盘中",
-  stale: "卡死",
-  reviewed: "待入队",
-  applied: "已采纳",
-  rejected: "已拒绝",
-  failed: "失败",
-  analyzed: "待评审",
-};
-
 export interface ListFilter {
   skill: string;
   /** 空串 = 全部 */

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_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,16 +106,7 @@ ${run}</div>
 ${warn}${backlog}`;
 }
 
-/** 状态下拉：先「全部」，再固定七个状态（顺序与 STATUS_OPTIONS 一致，别在渲染层重排） */
-function statusSelect(current: string): string {
-  const options = [`<option value="">全部状态</option>`];
-  for (const status of STATUS_OPTIONS) {
-    // value 仍是状态机原文（过滤逻辑不变），只把展示换中文
-    options.push(`<option value="${status}"${status === current ? " selected" : ""}>${STATUS_LABEL[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>`,
@@ -123,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

@@ -1,18 +1,19 @@
 /**
- * 筛选栏文案的验收测试（用户 2026-11-08：「这里的下拉应该是中文, 但是这个搜索框是干嘛的?」）。
+ * 筛选栏的验收测试（用户 2026-11-08：「这个搜索框的技能和状态不用了吧, 技能不知道用来搜索什么,
+ * 状态应该在顶部 tab 包含了吧」）。
  *
- * 两条口径：
- * 1. 状态下拉的 **value 仍是状态机原文**（URL / 过滤逻辑一个字符都不改），只把给人看的一侧换中文；
- *    中文名与列表徽章必须同一套词，同一个概念全站只能有一个名字。
- * 2. 技能那个输入框是**筛选条件**，不是搜索框也不是下拉：placeholder 必须写成「技能名包含…」，
- *    让人一眼知道「填一部分就够」。
+ * 口径：
+ * 1. 表单**只剩「时间」**一个条件——技能看不出该填什么、状态与顶部 tab 重复，两个都撤掉；
+ * 2. 撤掉的是**输入控件**，不是参数：`?skill=` / `?status=` 仍由 `parseListQuery` 解析，
+ *    技能页点进来、旧书签、代码内部跳转都不会断；
+ * 3. 表单仍是 GET + 保留 `view`，否则一提交就跳出当前页。
  *
  * 纪律：纯字符串断言，不碰文件系统、不起服务。
  */
 import { test } from "node:test";
 import assert from "node:assert/strict";
 
-import { parseListQuery, STATUS_LABEL, STATUS_OPTIONS } from "../lib/web/filter.ts";
+import { parseListQuery } from "../lib/web/filter.ts";
 import { renderFilterForm } from "../lib/web/render-controls.ts";
 
 const FILTER = parseListQuery(new URLSearchParams(""));
@@ -22,36 +23,38 @@ function form(): string {
   return renderFilterForm("pending", FILTER, EXTRAS);
 }
 
-test("状态下拉每一项都显示中文，value 保持状态机原文", () => {
+test("筛选栏不再有技能输入框与状态下拉", () => {
   const html = form();
-  for (const status of STATUS_OPTIONS) {
-    assert.ok(html.includes(`value="${status}"`), `value 必须是 ${status}`);
-    assert.ok(html.includes(`>${STATUS_LABEL[status]}<`), `${status} 要显示成 ${STATUS_LABEL[status]}`);
-  }
+  assert.ok(!html.includes('name="skill"'), "技能搜索框已撤掉");
+  assert.ok(!html.includes('name="status"'), "状态下拉已撤掉（顶部 tab 就是状态）");
+  assert.ok(!html.includes("全部状态"));
+  assert.ok(!html.includes("全部技能"));
 });
 
-test("状态下拉里不出现英文状态词", () => {
+test("时间筛选还在，且三个档位齐全", () => {
   const html = form();
-  const select = html.slice(html.indexOf('name="status"'), html.indexOf("</select>", html.indexOf('name="status"')));
-  for (const status of STATUS_OPTIONS) {
-    assert.ok(!select.includes(`>${status}<`), `不应再出现英文项 >${status}<`);
+  assert.ok(html.includes('name="range"'));
+  for (const label of ["最近 7 天", "最近 30 天", "全部时间"]) {
+    assert.ok(html.includes(`>${label}<`), `缺时间档位：${label}`);
   }
 });
 
-test("中文标签覆盖七个可选状态，且没有空值", () => {
-  assert.equal(Object.keys(STATUS_LABEL).length, STATUS_OPTIONS.length);
-  for (const label of Object.values(STATUS_LABEL)) assert.ok(label.trim().length > 0);
+test("表单仍是 GET，且保留 view", () => {
+  const html = form();
+  assert.ok(html.includes('method="get"'));
+  assert.ok(html.includes('<input type="hidden" name="view" value="pending">'));
 });
 
-test("技能输入框的 placeholder 说明它是筛选条件", () => {
-  const html = form();
-  assert.ok(html.includes('name="skill"'));
-  assert.ok(html.includes('placeholder="技能名包含…"'), "要告诉人这里填的是「技能名的一部分」");
-  assert.ok(!html.includes('placeholder="全部技能"'), "旧 placeholder 让人误以为是静态字段");
+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("筛选表单仍然是 GET + 保留 view", () => {
-  const html = form();
-  assert.ok(html.includes('method="get"'));
-  assert.ok(html.includes('<input type="hidden" name="view" value="pending">'));
+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.8",
+  "version": "0.1.9",
   "description": "技能进化：采集会话中的技能曝光/加载，异步复盘、生成改进提案并独立评审，附本机 Web 管理面板",
   "keywords": [
     "pi-package",