zen-gitsync 2.17.28 → 2.17.29

package/src/ui/server/routes/workbench/agentMarketplace.test.js

@@ -37,7 +37,7 @@ process.env.HOME = sandboxHome;
 delete process.env.HOMEDRIVE;
 delete process.env.HOMEPATH;
 
-const { registerAgentMarketplaceRoutes } = await import('./agentMarketplace.js');
+const { registerAgentMarketplaceRoutes, pickSkillDir, collectSkillDirs } = await import('./agentMarketplace.js');
 const { AI_SKILLS_DIR, AI_MCP_FILE } = await import('../../../../paths.js');
 
 // ── 测试脚手架:把注册的 handler 抓出来直接调 ──────────────────
@@ -336,3 +336,70 @@ test('install: 项目不存在时拒绝,不会把 .claude/skills 建到奇怪的
   assert.equal(status, 400);
   assert.match(payload.error, /项目目录不存在/);
 });
+
+// ── 集合型仓库"装哪一个" ────────────────────────────────────────
+// 踩过的坑:Leonxlnx/taste-skill 的 skills/ 下第一个是 brandkit,
+// 旧逻辑"按目录顺序撞见第一个含 SKILL.md 的就装" → 点 taste-skill 装成 brandkit。
+// 卡片 id 仍是 github-Leonxlnx-taste-skill 所以显示"已安装",
+// 已安装列表里却是 brandkit 这个名字,用户根本对不上,只能怀疑没装成功。
+
+test('pickSkillDir: 候选里挑与条目同名的那个,不取目录顺序第一个', () => {
+  const candidates = [
+    { dir: 'T/skills/brandkit', rel: 'skills/brandkit' },
+    { dir: 'T/skills/taste-skill', rel: 'skills/taste-skill' },
+    { dir: 'T/skills/taste-skill-v1', rel: 'skills/taste-skill-v1' },
+  ];
+  assert.equal(pickSkillDir(candidates, { skillName: 'taste-skill' }).rel, 'skills/taste-skill');
+});
+
+test('pickSkillDir: 条目给了子目录时以子目录为准,名字对得上也不用', () => {
+  const candidates = [
+    { dir: 'T/skills/taste', rel: 'skills/taste' },
+    { dir: 'T/skills/taste-skill', rel: 'skills/taste-skill' },
+  ];
+  assert.equal(pickSkillDir(candidates, { subpath: 'skills/taste', skillName: 'taste-skill' }).rel, 'skills/taste');
+});
+
+test('pickSkillDir: 名字对不上且候选并列时返回 null(报错,而不是猜一个)', () => {
+  const candidates = [
+    { dir: 'T/alpha', rel: 'alpha' },
+    { dir: 'T/beta', rel: 'beta' },
+  ];
+  assert.equal(pickSkillDir(candidates, { skillName: 'whatever' }), null);
+});
+
+test('pickSkillDir: 短目录名不会被裸 includes 误判成变体', () => {
+  const only = [{ dir: 'T/a', rel: 'a' }];
+  // wantName='whatever' 包含子串 'a',裸 includes 会把它当 rank 3 命中
+  assert.equal(pickSkillDir([...only, { dir: 'T/b', rel: 'b' }], { skillName: 'whatever' }), null);
+  assert.equal(pickSkillDir(only, { skillName: 'whatever' }).rel, 'a');
+});
+
+test('pickSkillDir: 仓库根就一份 SKILL.md 时优先于没名字的深层目录', () => {
+  const candidates = [
+    { dir: 'T/sub/other', rel: 'sub/other' },
+    { dir: 'T', rel: '' },
+  ];
+  assert.equal(pickSkillDir(candidates, { skillName: 'anything' }).rel, '');
+});
+
+test('pickSkillDir: 没有候选时返回 null(调用方报"仓库里没有 SKILL.md")', () => {
+  assert.equal(pickSkillDir([], { skillName: 'x' }), null);
+});
+
+test('collectSkillDirs: 收两层以内的 SKILL.md,跳过 .git 与点目录', async () => {
+  const root = await fs.mkdtemp(path.join(os.tmpdir(), 'zen-mp-collect-'));
+  const write = async rel => {
+    await fs.mkdir(path.join(root, rel), { recursive: true });
+    await fs.writeFile(path.join(root, rel, 'SKILL.md'), `---\nname: ${path.basename(rel)}\n---\n`, 'utf8');
+  };
+  await write('skills/brandkit');
+  await write('skills/deep/nested/too-deep');
+  await write('.claude-plugin');
+  await fs.mkdir(path.join(root, '.git'), { recursive: true });
+  await fs.writeFile(path.join(root, '.git', 'SKILL.md'), 'not a skill', 'utf8');
+  await fs.writeFile(path.join(root, 'README.md'), '# x', 'utf8');
+
+  const rels = (await collectSkillDirs(root)).map(item => item.rel).sort();
+  assert.deepEqual(rels, ['skills/brandkit']);
+});

package/src/ui/server/routes/workbench/envContext.js

@@ -39,6 +39,25 @@
 // 不经过前端 $t()，也不会显示在界面上。反过来说，它也不该跟着界面语言变 ——
 // 中文指令配中文上下文，模型理解得更稳。
 //
+// ── 为什么续聊轮要精简（`compact`）────────────────────────────────────────
+//
+// 续接一轮对话时，prompt = [环境块][记忆块][用户这一轮说的话]。而续接走的是 CLI 自己的
+// 会话恢复（claude `--resume` / opencode `--session` / codex `exec resume`），上一轮那份
+// **完整**环境块已经在会话历史里 —— 整块再发一遍，信息上是纯重复。
+//
+// 实测（本机 jobs.json，2026-09-30）：一条续接轮的 prompt 是
+//   环境块 2118~2200 字符 + 记忆块 424 字符 + 用户原话 44 字符
+// 也就是 98% 是重发的背景。代价有两处：一是每轮多烧约两千 token，而且它在**会话尾部**
+// （不是前缀），命不中前缀缓存；二是 job.prompt 会被前端当成用户气泡渲染
+// （useWorkbenchSimpleConversation.ts）—— 聊天记录里每续一轮就多一份一模一样的项目清单，
+// 打开一条跑过多轮的任务，第一屏全是它（WorkbenchView.vue 里那条"钉到底部"的注释同源）。
+//
+// 但**不能整块砍掉**：块里有真正会变的东西（看板计数、正在执行几个、项目增减），
+// 而且会话被压缩/裁剪之后那块背景可能真的不在了。所以续聊轮发的是这个"精简刷新版"：
+// 只留 当前项目 + 合计概览 + 真相源路径，项目清单与用户偏好交给前文
+// （引导语里点名"克隆优先 SSH"，免得那条行为约定被彻底忘掉）。
+// 记忆块是同一套思路，见 taskRunner.js 的 `capture: !resumeSessionId`。
+//
 // 纯函数：只吃入参、不读文件（取数在 workbench/index.js，数据在快照里）。
 // 这样单测不用碰真实用户数据 —— 与 promptParts.js 同一个理由。
 
@@ -70,6 +89,8 @@ const columnLabels = TASK_COLUMNS.map(k => COLUMN_LABELS[k] || k);
  * @param {object}   [input.truthFiles] { tasksFile, jobsFile, orchestratorFile, configFile }
  *                                      路径口径的唯一来源见 workbench/shared.js 的 TRUTH_FILES
  * @param {number}   [input.maxProjects] 只影响列出条数，不影响合计统计
+ * @param {boolean}  [input.compact]  续聊轮的精简刷新版：只留 当前项目 + 合计概览 + 真相源，
+ *                                    项目清单与用户偏好交给会话前文（见文件头那一节）
  * @returns {string} 上下文块。**不返回 null** —— 见下面的注释。
  */
 export function buildEnvContextBlock({
@@ -77,6 +98,7 @@ export function buildEnvContextBlock({
   board = null,
   truthFiles = {},
   maxProjects = ENV_CONTEXT_MAX_PROJECTS,
+  compact = false,
 } = {}) {
   const list = Array.isArray(board?.projects) ? board.projects.filter(p => p && p.key) : [];
   const statsByKey = (board?.statsByKey && typeof board.statsByKey === 'object') ? board.statsByKey : {};
@@ -90,24 +112,33 @@ export function buildEnvContextBlock({
   const currentKey = canonicalProjectPath(currentProjectPath);
 
   const lines = [];
-  lines.push('[运行环境 · 由 zen-gitsync 多项目编排台自动注入，不是用户输入的内容]');
+  // 首行前缀 `[运行环境` 是探针的锚点（scripts/verify-workbench-env-context.mjs
+  // 断言 prompt.startsWith('[运行环境')），精简版也照旧以它开头。
+  lines.push(compact
+    ? '[运行环境 · 由 zen-gitsync 多项目编排台自动注入，不是用户输入的内容 · 续聊轮精简刷新，完整项目清单与用户偏好见本会话前文]'
+    : '[运行环境 · 由 zen-gitsync 多项目编排台自动注入，不是用户输入的内容]');
   lines.push('');
   lines.push(`当前任务所在项目（也是你的工作目录）: ${currentProjectPath || '(未指定)'}`);
   lines.push('');
 
   if (list.length > 0) {
-    const limit = Number.isFinite(maxProjects) && maxProjects > 0 ? Math.floor(maxProjects) : ENV_CONTEXT_MAX_PROJECTS;
-    const shown = list.slice(0, limit);
-    lines.push(`项目清单（共 ${list.length} 个；格式: 名称 | 路径 | ${columnLabels.join('/')}）:`);
-    for (const p of shown) {
-      const s = statsByKey[p.key] || {};
-      const mark = currentKey && p.key === currentKey ? '  ← 当前' : '';
-      lines.push(`- ${p.name || p.key} | ${p.path} | ${TASK_COLUMNS.map(k => s[k] || 0).join('/')}${mark}`);
-    }
-    if (list.length > shown.length) {
-      lines.push(`- …还有 ${list.length - shown.length} 个未列出（读下面的文件可以看全）`);
+    // 项目清单整段只在首轮发。它占这块的 90%（19 个项目 ≈ 1300 字符）却几乎不变，
+    // 续聊轮再发一遍等于把同一张表贴在对话里（见文件头）。
+    if (!compact) {
+      const limit = Number.isFinite(maxProjects) && maxProjects > 0 ? Math.floor(maxProjects) : ENV_CONTEXT_MAX_PROJECTS;
+      const shown = list.slice(0, limit);
+      lines.push(`项目清单（共 ${list.length} 个；格式: 名称 | 路径 | ${columnLabels.join('/')}）:`);
+      for (const p of shown) {
+        const s = statsByKey[p.key] || {};
+        const mark = currentKey && p.key === currentKey ? '  ← 当前' : '';
+        lines.push(`- ${p.name || p.key} | ${p.path} | ${TASK_COLUMNS.map(k => s[k] || 0).join('/')}${mark}`);
+      }
+      if (list.length > shown.length) {
+        lines.push(`- …还有 ${list.length - shown.length} 个未列出（读下面的文件可以看全）`);
+      }
+      lines.push('');
     }
-    lines.push('');
+    // 计数概览是这块里**真正会变**的部分，首轮与续聊轮都发
     lines.push(
       `看板任务概览：全部项目合计 ${total.total || 0} 条 —— ` +
       TASK_COLUMNS.map(k => `${COLUMN_LABELS[k]} ${total[k] || 0}`).join(' / ') +
@@ -122,27 +153,36 @@ export function buildEnvContextBlock({
     lines.push('');
   }
 
-  // 用户偏好（见文件头）。放在"读文件"之前、"看板概览"之后 ——
-  // 它是**行为**约定，与上面的事实清单分开；压尾的那句仍是"去读文件"，
-  // 那是整块上下文里最需要被记住的动作。
-  lines.push(
-    '用户偏好：克隆仓库 / 添加远端时**优先使用 SSH**（https 地址先换算成 ' +
-    'git@github.com:owner/repo.git，Gitee 同理 git@gitee.com:owner/repo.git —— ' +
-    '走 https 会弹凭据窗口，把任务停在半路等用户输账号密码）。' +
-    '只有 SSH 不可用（Permission denied (publickey) / Host key verification failed）' +
-    '才退回 https，并说明这次走的是 https。',
-  );
-  lines.push('');
-  lines.push('需要细节时直接读这些文件（本机绝对路径，你有读取权限，不必先问用户）:');
+  if (compact) {
+    // 引导语替掉整段用户偏好，但**点名**那条行为约定：偏好本身不重述（前文有），
+    // 可"有这么一条"得让模型看得见，否则克隆仓库时它会退回 https。
+    lines.push('需要完整项目清单 / 进展细节时（含「克隆优先 SSH」偏好），直接读这些文件（本机绝对路径，你有读取权限，不必先问用户）:');
+  } else {
+    // 用户偏好（见文件头）。放在"读文件"之前、"看板概览"之后 ——
+    // 它是**行为**约定，与上面的事实清单分开；压尾的那句仍是"去读文件"，
+    // 那是整块上下文里最需要被记住的动作。
+    lines.push(
+      '用户偏好：克隆仓库 / 添加远端时**优先使用 SSH**（https 地址先换算成 ' +
+      'git@github.com:owner/repo.git，Gitee 同理 git@gitee.com:owner/repo.git —— ' +
+      '走 https 会弹凭据窗口，把任务停在半路等用户输账号密码）。' +
+      '只有 SSH 不可用（Permission denied (publickey) / Host key verification failed）' +
+      '才退回 https，并说明这次走的是 https。',
+    );
+    lines.push('');
+    lines.push('需要细节时直接读这些文件（本机绝对路径，你有读取权限，不必先问用户）:');
+  }
   if (truthFiles.tasksFile) lines.push(`- ${truthFiles.tasksFile} —— 全部任务全文（标题/描述/子任务状态/报错/projectPath）`);
   if (truthFiles.jobsFile) lines.push(`- ${truthFiles.jobsFile} —— 历次执行记录（状态与输出）`);
   if (truthFiles.orchestratorFile) lines.push(`- ${truthFiles.orchestratorFile} —— 用户在调度台发过的指令流水`);
   if (truthFiles.configFile) lines.push(`- ${truthFiles.configFile} —— 应用配置（projects / recentDirectories）`);
-  lines.push('');
-  lines.push(
-    '用户问到"我有哪些项目""某件事进展如何"这类问题时，先读上面的文件再回答；' +
-    '不要凭当前目录猜测，也不要回答"我看不到/无法访问"。',
-  );
+  // 压尾那句"先读文件再回答"只在首轮讲：续聊轮上面那句引导语已经说了同一件事
+  if (!compact) {
+    lines.push('');
+    lines.push(
+      '用户问到"我有哪些项目""某件事进展如何"这类问题时，先读上面的文件再回答；' +
+      '不要凭当前目录猜测，也不要回答"我看不到/无法访问"。',
+    );
+  }
 
   return lines.join('\n');
 }

package/src/ui/server/routes/workbench/envContext.test.js

@@ -260,3 +260,71 @@ test('真相源清单来自唯一的 TRUTH_FILES（四个路径一个不落，
   assert.ok(block.includes(TRUTH_FILES.orchestratorFile));
   assert.match(block, /调度台发过的指令流水/);
 });
+
+// ── 续聊轮精简版（compact）──────────────────────────────────────────────
+//
+// 续接走的是 CLI 自己的会话恢复（claude --resume / opencode --session / codex exec resume），
+// 上一轮那份**完整**块已在会话历史里 —— 续聊轮再发一遍就是纯重复。实测一条续接轮 prompt：
+// 环境块 2118~2200 字符 + 记忆块 424 字符 + 用户原话 44 字符（本机 jobs.json，2026-09-30）。
+//
+// 这组守四件事：
+//   1. 项目清单真的不再发（它占这块 90%，且几乎不变）；
+//   2. 会变的那部分（合计概览、当前项目）还在 —— 精简不等于把刷新能力也砍了；
+//   3. 真相源路径照给：会话被压缩后靠它重新够得着数据；
+//   4. 首行锚点不变（scripts/verify-workbench-env-context.mjs 断言 startsWith('[运行环境')）。
+//
+// 数字口径一栏用"两种模式必须给出同一组计数"来守：精简是**裁剪**，不是另算一份。
+
+const richBoard = () => {
+  // 项目数刻意贴近真实规模（用户本机 19 个）：清单的"贵"是随项目数线性涨的，
+  // 只放两三个项目的夹具量不出"精简到底省了多少"（这条正是 compact 的存在理由）。
+  const recentDirs = ['D:\\ws\\zen-gitsync', 'D:\\ws\\article-generator'];
+  const tasks = [
+    { id: 't1', projectPath: 'D:\\ws\\zen-gitsync', type: 'simple' },
+    { id: 't2', projectPath: 'D:\\ws\\article-generator', type: 'simple' },
+    { id: 't3', projectPath: 'D:\\ws\\article-generator', type: 'simple' },
+  ];
+  for (let i = 0; i < 17; i += 1) recentDirs.push(`D:\\ws\\filler${i}`);
+  return boardOf({ recentDirs, tasks, jobs: [job('j1', 't1', 'done'), job('j2', 't2', 'running')] });
+};
+
+test('compact 不再发项目清单，但保留当前项目与合计概览', () => {
+  const board = richBoard();
+  const full = build({ currentProjectPath: 'D:\\ws\\zen-gitsync', board });
+  const compact = build({ currentProjectPath: 'D:\\ws\\zen-gitsync', board, compact: true });
+
+  // ① 清单行没了 —— 连"还有 N 个未列出"这种残余也不能留（模型会以为清单被截断）
+  assert.doesNotMatch(compact, /项目清单（共 \d+ 个/);
+  assert.doesNotMatch(compact, /^- .+ \| .+ \| \d+\/\d+\/\d+/m);
+  assert.doesNotMatch(compact, /还有 \d+ 个未列出/);
+  // 另一个项目（没在当前项目栏里出现过的）整体不该出现在精简块里
+  assert.ok(!compact.includes('D:\\ws\\article-generator'), '精简块不该再列其它项目');
+
+  // ② 会变的部分还在，而且数字与完整版**逐字相同**（同一次取数、两种裁剪）
+  const overview = full.match(/看板任务概览：全部项目合计 [^\n]+/)[0];
+  assert.ok(compact.includes(overview), `精简块应原样保留合计概览:\n${compact}`);
+  assert.match(compact, /全部项目合计 3 条/);
+  assert.match(compact, /正在执行 1 个/);
+  assert.match(compact, /当前任务所在项目（也是你的工作目录）: D:\\ws\\zen-gitsync/);
+
+  // ③ 真相源路径照给（会话被压缩后靠它重新够得着数据）
+  for (const p of Object.values(TRUTH_FILES)) assert.ok(compact.includes(p), `缺少路径 ${p}`);
+  // 用户偏好不重述，但要点名"有这么一条" —— 少这一句，克隆仓库时会退回 https
+  assert.match(compact, /克隆优先 SSH/);
+
+  // ④ 首行锚点（探针依赖）与完整版同前缀
+  assert.ok(compact.startsWith('[运行环境'));
+  assert.ok(full.startsWith('[运行环境'));
+
+  // ⑤ 真省：精简块不到完整版的一半
+  assert.ok(compact.length * 2 < full.length, `精简块应显著更短: compact=${compact.length} full=${full.length}`);
+});
+
+test('compact 在取不到 board 时也照给真相源（且不返回空）', () => {
+  const compact = build({ currentProjectPath: 'D:\\ws\\a', board: null, compact: true });
+  assert.ok(compact, '精简版同样不返回 null');
+  for (const p of Object.values(TRUTH_FILES)) assert.ok(compact.includes(p), `缺少路径 ${p}`);
+  assert.match(compact, /项目清单：本次未能取到/);
+  // 没有清单时更不该出现"清单行"的残影
+  assert.doesNotMatch(compact, /^- .+ \| .+ \| \d+\/\d+\/\d+/m);
+});