pi-skill-evolve 0.1.3 → 0.1.4

package/extensions/skill-evolve/lib/command.ts

@@ -27,7 +27,7 @@ export const SUBCOMMANDS: readonly SubcommandItem[] = [
   { value: "dispatch", label: "dispatch", description: "只开/关自动复盘（on|off；会派生子进程，花 token）" },
   { value: "review", label: "review", description: "只开/关自动评审（on|off；提案后再起督导，再花一份 token）" },
   { value: "apply", label: "apply", description: "只开/关自动采纳（on|off；auto 提案自动落盘，可回滚）" },
-  { value: "on", label: "on", description: "复盘与评审一起打开（不动采集，也不动自动采纳）" },
+  { value: "on", label: "on", description: "只打开自动复盘（采集 / 评审 / 采纳都不动）" },
   { value: "off", label: "off", description: "四个开关（采集/复盘/评审/采纳）一起关闭" },
 ];
 

package/extensions/skill-evolve/lib/config.ts

@@ -1,9 +1,9 @@
 /**
  * 技能进化配置：唯一来源是 `.pi/evolution/config.json`。
  *
- * **默认只开自动评审，其余三个关**（采集 `telemetry:false` / 复盘 `enabled:false` / 自动采纳 `autoApply:false`，
- * 自动评审 `autoReview:true`）：别人 clone 下来不采集、不派生、不落盘，也就没有 token 开销 ——
- * `autoReview` 单独为 true 是安全的，因为它只在「复盘真的跑起来」之后才起作用（`enabled` 关着就永远走不到）。
+ * **四个开关默认全关**（采集 `telemetry` / 复盘 `enabled` / 评审 `autoReview` / 采纳 `autoApply` 都是 false，
+ * 2026-11-08 用户定「autoReview 默认也关」）：别人 clone 下来不采集、不派生、不评审、不落盘，零 token 开销。
+ * 要用就一条一条显式打开（`collect` / `dispatch` / `review` / `apply`），谁开谁负责。
  * 每一档花不花 token：
  * - 采集遥测：只写本地 jsonl，**不花 token**；
  * - 自动复盘：起子进程跑 agent，**花 token**；
@@ -76,9 +76,9 @@ export const DEFAULT_CONFIG: Readonly<EvolutionConfig> = Object.freeze({
   maxRuntimeMs: 20 * 60_000,
   perSessionCap: 20,
   telemetry: false,
-  // 自动评审默认开：设计原意就是「复盘 → 评审 → 采纳」一条流水线，评审只是流程的一部分，
-  // 而 enabled 关着时它根本不生效，所以默认 true 不会让 clone 下来的人白花 token。
-  autoReview: true,
+  // 自动评审默认关（2026-11-08 用户定）：评审本身也要起子进程再花一份 token，
+  // 不能让「打开复盘」顺手把 token 翻倍。要评审就显式 `/skill-evolve review on`。
+  autoReview: false,
   // 自动采纳默认关：它改的是仓库文件，必须人工先开一次；默认开等于替用户做决定。
   autoApply: false,
   ui: Object.freeze({ widget: false, entries: false }),
@@ -193,7 +193,7 @@ export async function saveConfigSwitch(
 ): Promise<EvolutionConfig> {
   const raw = readRawConfig(evolutionDir);
   const patch = on
-    ? { enabled: true, autoReview: true }
+    ? { enabled: true }
     : { enabled: false, telemetry: false, autoReview: false, autoApply: false };
   const next = normalizeConfig({ ...raw, ...patch });
   const file = configFile(evolutionDir);

package/extensions/skill-evolve/test/config.test.ts

@@ -17,11 +17,11 @@ function workspace(): string {
   return path.join(cwd, ".pi", "evolution");
 }
 
-test("DEFAULT_CONFIG：只开自动评审，采集 / 复盘 / 自动采纳都默认关", () => {
-  // 自动评审单独为 true 是安全的：enabled 关着时它永远走不到，不会替 clone 下来的人花 token
+test("DEFAULT_CONFIG：四个开关默认全关", () => {
+  // 四个默认全关：clone 下来不采集、不派生、不评审、不落盘（2026-11-08 用户定）
   assert.deepEqual(
     [DEFAULT_CONFIG.enabled, DEFAULT_CONFIG.telemetry, DEFAULT_CONFIG.autoReview, DEFAULT_CONFIG.autoApply],
-    [false, false, true, false],
+    [false, false, false, false],
   );
 });
 
@@ -30,14 +30,14 @@ test("loadConfig：没有 config.json 时只开自动评审，且不会顺手造
   const config = loadConfig(evolutionDir);
   assert.deepEqual(
     [config.enabled, config.telemetry, config.autoReview, config.autoApply],
-    [false, false, true, false],
+    [false, false, false, false],
   );
   assert.equal(fs.existsSync(configFile(evolutionDir)), false, "读配置不能有副作用");
 });
 
 test("normalizeConfig：空对象与缺字段都回落到默认（评审开、其余关）", () => {
   const empty = normalizeConfig({});
-  assert.deepEqual([empty.enabled, empty.telemetry, empty.autoReview, empty.autoApply], [false, false, true, false]);
+  assert.deepEqual([empty.enabled, empty.telemetry, empty.autoReview, empty.autoApply], [false, false, false, false]);
   const onlyEnabled = normalizeConfig({ enabled: true });
   assert.deepEqual(
     [onlyEnabled.enabled, onlyEnabled.telemetry, onlyEnabled.autoApply],
@@ -49,12 +49,12 @@ test("normalizeConfig：空对象与缺字段都回落到默认（评审开、
 test("normalizeConfig：autoReview / autoApply 写错类型回落到默认值", () => {
   // 手改配置写成字符串是常事：坏值只回退那一个字段，不能让整份配置失效
   const wrong = normalizeConfig({ autoReview: "off", autoApply: 1 });
-  assert.deepEqual([wrong.autoReview, wrong.autoApply], [true, false]);
+  assert.deepEqual([wrong.autoReview, wrong.autoApply], [false, false]);
   const explicit = normalizeConfig({ autoReview: false, autoApply: true, enabled: true });
   assert.deepEqual([explicit.enabled, explicit.autoReview, explicit.autoApply], [true, false, true]);
 });
 
-test("saveConfigSwitch：on 只开复盘 + 评审，采集与自动采纳保持原值", async () => {
+test("saveConfigSwitch：on 只开复盘，采集 / 评审 / 采纳都保持原值", async () => {
   const evolutionDir = workspace();
   fs.mkdirSync(evolutionDir, { recursive: true });
   fs.writeFileSync(
@@ -65,13 +65,13 @@ test("saveConfigSwitch：on 只开复盘 + 评审，采集与自动采纳保持
   const next = await saveConfigSwitch(evolutionDir, true, directQueue);
   assert.deepEqual(
     [next.enabled, next.autoReview, next.telemetry, next.autoApply, next.idleMs, next.perSessionCap],
-    [true, true, true, false, 1_234, 3],
+    [true, false, true, false, 1_234, 3],
   );
   const raw = JSON.parse(fs.readFileSync(configFile(evolutionDir), "utf8"));
   assert.deepEqual(
     [raw.enabled, raw.autoReview, raw.telemetry, raw.autoApply, raw.idleMs, raw.perSessionCap],
-    [true, true, true, false, 1_234, 3],
-    "写盘内容必须与返回值一致：采集关着不能被 on 顺手打开，自动采纳更不能被打开",
+    [true, false, true, false, 1_234, 3],
+    "写盘内容必须与返回值一致：on 只开复盘，采集 / 评审 / 采纳都保持原值",
   );
 });
 

package/extensions/skill-evolve/test/dispatch-supervisor.test.ts

@@ -68,7 +68,8 @@ function harness(
 }
 
 test("派发成功时同一轮就起评审督导，并把评审模型透传过去", async () => {
-  const h = harness({ reviewModel: () => "deepseek/deepseek-v3.2" });
+  // 评审默认关（2026-11-08），这里显式打开才测得到「起督导 + 透传模型」
+  const h = harness({ config: { autoReview: true }, reviewModel: () => "deepseek/deepseek-v3.2" });
   const outcome = await h.dispatch({ sessionId: "sess-1", skills: ["alpha"] });
   assert.equal(outcome.status, "dispatched");
   assert.equal(h.supervisorCalls.length, 1, "必须起且只起一个督导");

package/extensions/skill-evolve/test/evolution.test.ts

@@ -222,7 +222,7 @@ test("statusText / reportText：无数据也不崩，且带上盲区提示", asy
   const status = await h.evo.statusText();
   assert.equal(
     status.includes(
-      "采集：开（遥测写入 .pi/evolution/telemetry/）｜复盘：开（会派生 agent）｜评审：开（提案后自动起督导）｜采纳：关（auto 提案等人工批准）",
+      "采集：开（遥测写入 .pi/evolution/telemetry/）｜复盘：开（会派生 agent）｜评审：关（提案停在 proposed 等人工）｜采纳：关（auto 提案等人工批准）",
     ),
     true,
   );
@@ -257,7 +257,7 @@ test("reportText：按遥测给出曝光/加载/未命中", async () => {
   );
 });
 
-test("setEnabled：off 四个全关、on 只开复盘 + 评审（不动采集与自动采纳）", async () => {
+test("setEnabled：off 四个全关、on 只开复盘（不动采集 / 评审 / 采纳）", async () => {
   const h = harness();
   const message = await h.evo.setEnabled(false);
   assert.equal(message.includes("关闭"), true);
@@ -267,8 +267,8 @@ test("setEnabled：off 四个全关、on 只开复盘 + 评审（不动采集与
 
   await h.evo.setEnabled(true);
   const back = JSON.parse(fs.readFileSync(path.join(h.dir, ".pi", "evolution", "config.json"), "utf8"));
-  // 用户定的语义：on = 复盘 + 评审两开；采集要另外 collect on，自动采纳要另外 apply on
-  assert.deepEqual([back.enabled, back.autoReview, back.telemetry, back.autoApply], [true, true, false, false]);
+  // 用户定的语义（2026-11-08）：四个开关各管一段，on 只开自动复盘；评审要另外 review on
+  assert.deepEqual([back.enabled, back.autoReview, back.telemetry, back.autoApply], [true, false, false, false]);
 });
 
 test("setFlag：只切一个字段，并把当前四个开关的状态写进回执", async () => {
@@ -309,9 +309,9 @@ test("没有 config.json 时默认全关：不派生、不落遥测", async () =
   });
   const status = await evo.statusText();
   assert.equal(
-    status.includes("采集：关（不写遥测）｜复盘：关（不会派生 agent）｜评审：开（提案后自动起督导）｜采纳：关（auto 提案等人工批准）"),
+    status.includes("采集：关（不写遥测）｜复盘：关（不会派生 agent）｜评审：关（提案停在 proposed 等人工）｜采纳：关（auto 提案等人工批准）"),
     true,
-    "默认只开自动评审：enabled 关着时它永远走不到",
+    "默认四个全关：采集 / 复盘 / 评审 / 采纳都不动",
   );
   evo.recordAdvertised({ sessionId: "s1", cwd: dir, prompt: "x", skills: [skill("a")] });
   evo.settle({ sessionId: "s1" });

package/package.json

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