zen-gitsync 2.17.12 → 2.17.14

package/src/ui/server/routes/localRepos.test.js

@@ -0,0 +1,151 @@
+// Copyright 2026 xz333221
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+//
+// src/ui/server/routes/localRepos.js 单元测试。
+//
+// 两条路由都只是"转发",所以断言也集中在**它们自己的取舍**上,而不是扫描逻辑
+// (那部分在 utils/localRepoScan.test.js):
+//   1. GET 必须把快照原样摊平到顶层 —— 前端就靠 `data.scanning` 决定要不要轮询
+//   2. POST 必须**不 await** 扫描:一 await 就会挂十几秒,前端拿不到 started
+//   3. POST 那条 promise 是有意"发射后不管"的 —— 它 reject 时不能变成进程级
+//      unhandledRejection(Node 15+ 默认直接崩服务)
+//
+// 不真起 Express:沿用 remoteRepos.test.js 的 mock app + mock req/res 做法。
+// 默认实现会真去遍历磁盘,所以两个注入口在所有用例里都换成假实现。
+import { test } from 'node:test'
+import assert from 'node:assert/strict'
+
+import { registerLocalReposRoutes } from './localRepos.js'
+
+/** 最小 express app mock:只实现 get + post 与 handler 调用 */
+function makeApp() {
+  const handlers = new Map()
+  return {
+    get(path, handler) { handlers.set(`GET ${path}`, handler) },
+    post(path, handler) { handlers.set(`POST ${path}`, handler) },
+    invoke(method, path) {
+      const key = `${method} ${path}`
+      const handler = handlers.get(key)
+      assert.ok(handler, `no handler for ${key}`)
+      const res = makeRes()
+      return handler({ body: {}, query: {}, method, path }, res, () => {}).then(() => res)
+    },
+  }
+}
+
+function makeRes() {
+  return {
+    statusCode: 200,
+    body: undefined,
+    status(code) { this.statusCode = code; return this },
+    json(payload) { this.body = payload; return this },
+  }
+}
+
+const SNAPSHOT = {
+  scanning: false,
+  scannedAt: 1758000000000,
+  scannedDirs: 8613,
+  roots: ['C:/', 'D:/'],
+  repos: { 'D:/w/zen-gitsync': 'git@github.com:xz333221/zen-gitsync.git' },
+  error: null,
+}
+
+function setup({ snapshot, refresh } = {}) {
+  const app = makeApp()
+  const calls = { get: 0, refresh: 0 }
+  registerLocalReposRoutes({
+    app,
+    getLocalReposImpl: async () => {
+      calls.get += 1
+      return snapshot === undefined ? SNAPSHOT : snapshot
+    },
+    refreshReposImpl: () => {
+      calls.refresh += 1
+      return refresh === undefined ? Promise.resolve(SNAPSHOT) : refresh()
+    },
+  })
+  return { app, calls }
+}
+
+// ── GET /api/local-repos ────────────────────────────────────────────────
+
+test('GET /api/local-repos: 快照摊平进响应顶层(前端靠 data.scanning 决定要不要轮询)', async () => {
+  const { app, calls } = setup()
+  const res = await app.invoke('GET', '/api/local-repos')
+
+  assert.equal(res.statusCode, 200)
+  assert.deepEqual(res.body, { success: true, ...SNAPSHOT })
+  // 关键字段必须在顶层,而不是嵌在 data / snapshot 里
+  assert.equal(res.body.scanning, false)
+  assert.equal(res.body.repos['D:/w/zen-gitsync'], 'git@github.com:xz333221/zen-gitsync.git')
+  assert.equal(calls.get, 1)
+  // GET 不该顺手触发重扫 —— 重扫是 POST 的职责
+  assert.equal(calls.refresh, 0)
+})
+
+test('GET /api/local-repos: 首次运行(还在扫)也回 200,只把 scanning 置真', async () => {
+  const { app } = setup({ snapshot: { ...SNAPSHOT, scanning: true, scannedAt: null, repos: {} } })
+  const res = await app.invoke('GET', '/api/local-repos')
+
+  assert.equal(res.statusCode, 200)
+  assert.equal(res.body.success, true)
+  assert.equal(res.body.scanning, true)
+  assert.deepEqual(res.body.repos, {})
+})
+
+// ── POST /api/local-repos/scan ──────────────────────────────────────────
+
+test('POST /api/local-repos/scan: 不 await 扫描 —— 扫描没结束就先回 started', async () => {
+  let release
+  const pending = new Promise((resolve) => { release = resolve })
+  const { app, calls } = setup({ refresh: () => pending })
+
+  // 若路由 await 了扫描,这一行会一直挂着(扫描默认要十几秒)
+  const res = await app.invoke('POST', '/api/local-repos/scan')
+
+  assert.equal(res.statusCode, 200)
+  assert.deepEqual(res.body, { success: true, started: true })
+  assert.equal(calls.refresh, 1, '应该确实把重扫踢起来了')
+
+  release(SNAPSHOT)
+  await pending
+})
+
+test('POST /api/local-repos/scan: 重扫失败不会变成进程级 unhandledRejection', async () => {
+  const rejections = []
+  const onRejection = (reason) => { rejections.push(reason) }
+  process.on('unhandledRejection', onRejection)
+  try {
+    const { app } = setup({ refresh: () => Promise.reject(new Error('盘拔了')) })
+    const res = await app.invoke('POST', '/api/local-repos/scan')
+
+    assert.equal(res.body.started, true, '后台失败不该影响这次响应')
+
+    // 让被吞掉的那个 rejection 有机会冒出来
+    await new Promise(resolve => setTimeout(resolve, 50))
+    assert.deepEqual(rejections, [], 'rejection 必须被路由自己 catch 掉')
+  } finally {
+    process.off('unhandledRejection', onRejection)
+  }
+})
+
+test('POST /api/local-repos/scan: 重复点刷新都转成一次重扫(并发语义由 localRepoScan 保证)', async () => {
+  const { app, calls } = setup()
+  await app.invoke('POST', '/api/local-repos/scan')
+  await app.invoke('POST', '/api/local-repos/scan')
+
+  // 路由层不去重 —— 它对调用方是"转达意图",真正的单飞在 refreshLocalRepos 里
+  assert.equal(calls.refresh, 2)
+})

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

@@ -108,6 +108,13 @@ ${isWin ? `- 当前是 Windows,以下 Unix 命令**不存在**,用了必定报"
   报"不是内部或外部命令" / command not found 时,**不要换写法反复重试**,更不要凭印象编仓库名或目录名。
   直接告诉用户未检测到该 CLI,可在「远程仓库」页一键安装/登录,或需要时重启服务让新装的 CLI 生效;
   若只是想回答本项目的问题,退回 \`git remote -v\`
+- 克隆仓库 / 添加远端时**优先用 SSH**(用户偏好,本机已配好密钥,实测走 https 会弹凭据窗口把任务打断):
+  · GitHub 用 \`git@github.com:owner/repo.git\`,Gitee 用 \`git@gitee.com:owner/repo.git\`
+  · 用户给的是 \`https://...\` 或 \`gh repo clone owner/repo\` 时,先换算成 SSH 地址再执行 ——
+    走 https 会弹 Git Credential Manager 让用户输账号密码,任务就停在半路等输入
+  · 已有仓库换协议:\`git remote set-url origin <ssh 地址>\`(先 \`git remote -v\` 看当前是什么)
+  · 只有 SSH 真的不可用(报 \`Permission denied (publickey)\` / \`Host key verification failed\`)才退回 https,
+    并用一句话说明这次走的是 https、配好密钥后可改回;两边都失败就停下来问用户,不要反复重试
 
 # 权限(用户已明确授权,无需反复征求同意)
 - 工作目录内:读写文件、执行命令等所有操作直接执行
@@ -180,6 +187,15 @@ ${isWin ? `- This is Windows. The following Unix commands do NOT exist here:
   repo or directory names. Tell the user the CLI was not found and point them to the "Remote
   repositories" page (one-click install / login), or restart the server so a freshly installed CLI
   becomes visible; if you only need to answer for this project, fall back to \`git remote -v\`
+- Cloning a repo or adding a remote: **prefer SSH** (the user's preference — keys are already set up,
+  and https has been observed to pop a credential window that stalls the task):
+  · GitHub → \`git@github.com:owner/repo.git\`; Gitee → \`git@gitee.com:owner/repo.git\`
+  · If the user hands you an \`https://...\` URL (or \`gh repo clone owner/repo\`), convert it to the SSH
+    form first — https pops up Git Credential Manager asking for a username/password and blocks the task
+  · Switching an existing repo: \`git remote set-url origin <ssh url>\` (check \`git remote -v\` first)
+  · Fall back to https only when SSH genuinely fails (\`Permission denied (publickey)\` /
+    \`Host key verification failed\`), and say in one line that this one used https and can go back to SSH
+    once the key is set up; if both fail, stop and ask the user — do not keep retrying
 
 # Permissions (explicitly granted by the user)
 - Inside the working directory: read/write files and run commands directly

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

@@ -19,9 +19,12 @@
 // 一概不在里面。于是用户问"我的项目有哪些"，Agent 只能就着当前目录猜 ——
 // 而数据其实全在本机，只是没人告诉它路径。
 //
-// 这里做两件事，缺一不可：
+// 这里做三件事，缺一不可：
 //   1. 给**摘要** —— 项目清单 + 各列任务计数（顺着看板口径，一眼看到进度）；
-//   2. 给**真相源文件路径** —— 要细节自己去读。
+//   2. 给**真相源文件路径** —— 要细节自己去读；
+//   3. 给**用户偏好** —— 克隆仓库优先 SSH。这一条不是"运行环境"的事实，而是工作台任务的
+//      Agent 是外部 CLI（claude / opencode），系统提示词不归我们写 —— 这里是唯一能把偏好
+//      递进去的地方。少这一句，任务里克隆仓库就会退回 https，弹凭据窗口把任务停住。
 // 刻意不把 tasks.json 全文塞进 prompt：几十条任务、每条还挂着执行输出，
 // 既烧 token，又会把用户真正要办的那句话淹掉。
 //
@@ -122,6 +125,17 @@ export function buildEnvContextBlock({
     TASK_COLUMNS.map(k => `${COLUMN_LABELS[k]} ${total[k] || 0}`).join(' / '),
   );
   lines.push('');
+  // 用户偏好（见文件头第 3 条）。放在"读文件"之前、"看板概览"之后 ——
+  // 它是**行为**约定，与上面的事实清单分开；压尾的那句仍是"去读文件"，
+  // 那是整块上下文里最需要被记住的动作。
+  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 (tasksFile) lines.push(`- ${tasksFile} —— 全部任务全文（标题/描述/子任务状态/报错/projectPath）`);
   if (jobsFile) lines.push(`- ${jobsFile} —— 历次执行记录（状态与输出）`);

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

@@ -176,3 +176,17 @@ test('项目行/合计的段数必须与 TASK_COLUMNS 一致，且每列都有
   // 两个任务都是待处理 → 第一段是 2，其余列是 0（顺序由 TASK_COLUMNS 决定，不是写死的下标）
   assert.equal(rowCounts[TASK_COLUMNS.indexOf('todo')], '2');
 });
+
+// ── 用户偏好：克隆仓库优先 SSH ────────────────────────────────────────
+//
+// 这条只对**工作台任务**起作用（执行器是外部 CLI claude / opencode，系统提示词不归我们写，
+// envContext 是唯一能把偏好递进去的地方）。断言钉住两个平台的地址都写全 —— 用户实际踩的
+// 那个坑就是 Gitee 走 https 弹了凭据窗口；只留 GitHub 一条等于只覆盖一半场景。
+test('注入块带上「克隆优先 SSH」这条用户偏好', () => {
+  const block = buildEnvContextBlock({ projects: [entry('D:\\ws\\a')], tasks: [], jobs: [], ...PATHS });
+  assert.match(block, /优先使用 SSH/);
+  assert.match(block, /git@github\.com:owner\/repo\.git/);
+  assert.match(block, /git@gitee\.com:owner\/repo\.git/);
+  // 退回条件也得在：没写清"什么时候可以退回"，模型会在 SSH 失败后来回重试
+  assert.match(block, /Permission denied \(publickey\)/);
+});

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

@@ -122,6 +122,15 @@ const FACTS = [
     en: [/Never ask the user for a token/],
     commands: [],
   },
+  {
+    // 用户明确提过的偏好：让 AI 克隆仓库时优先 SSH。走 https 会弹 Git Credential Manager，
+    // 把任务停在半路等人输账号密码（实测）。四个入口都得守同一条，否则从某个入口克隆
+    // 又会退回去走 https —— 这类"少写一条不报错"的失效方式正是本文件存在的理由。
+    what: '克隆 / 加远端优先 SSH（https 先换算，无密钥才退回）',
+    zh: [/优先用 SSH/, /git@github\.com:owner\/repo\.git/, /Permission denied \(publickey\)/],
+    en: [/prefer SSH/, /git@github\.com:owner\/repo\.git/, /Permission denied \(publickey\)/],
+    commands: ['git@github.com:owner/repo.git', 'git@gitee.com:owner/repo.git', 'git remote set-url origin'],
+  },
 ];
 
 test('四条提示词覆盖同样的事实，命令串逐字一致（跨语言对齐）', () => {

package/src/ui/server/utils/directoryGitState.js

@@ -288,6 +288,55 @@ export async function probeDirectoryGitStates(paths, options = {}) {
   return results;
 }
 
+/**
+ * 探测单个目录的 origin 远程地址。
+ *
+ * 用途:远程仓库列表要回答"这个仓库本地已经克隆过了吗" —— 判据只能是本地
+ * 某个目录的 origin 指向谁。口径与 probeDirectoryGitState 一致:一次 spawn、
+ * 超时即放弃、**不联网**(`git remote get-url` 只读 .git/config,不发请求)。
+ *
+ * 非仓库、目录不存在、没有名为 origin 的 remote 都是失败(退出码非 0)——
+ * 统一收敛成 null:对"是不是克隆来的"这个问题,这些情况的答案都是"不知道/不是",
+ * 调用方不需要区分。
+ */
+export async function probeDirectoryOrigin(dirPath, { timeoutMs = DEFAULT_PROBE_TIMEOUT_MS } = {}) {
+  try {
+    const { stdout } = await execFileAsync(
+      'git',
+      ['-C', dirPath, 'remote', 'get-url', 'origin'],
+      { timeout: timeoutMs, maxBuffer: MAX_BUFFER, windowsHide: true }
+    );
+    const url = String(stdout || '').trim();
+    return url || null;
+  } catch {
+    return null;
+  }
+}
+
+/**
+ * 批量探测 origin 地址。返回 `{ [调用方传入的原始路径]: url|null }`
+ * —— 键用原始字符串,前端可以直接 `origins[item.path]` 取。
+ *
+ * 为什么不做缓存(同一个文件里 git 状态就有 15s TTL):remote 是用户随时在改的
+ * 东西(刚加了 remote、刚把地址换成 SSH),缓存只会让「已克隆」这个标识慢半拍地
+ * 骗人;而调用方只有仓库列表挂载时那一次请求,重复扫的代价可以忽略。
+ */
+export async function probeDirectoryOrigins(paths, options = {}) {
+  const { timeoutMs = DEFAULT_PROBE_TIMEOUT_MS, concurrency = DEFAULT_CONCURRENCY } = options;
+  const list = Array.isArray(paths) ? paths.filter(p => typeof p === 'string' && p.trim()) : [];
+  // 同一个目录可能以不同写法出现在配置里,先按原始串去重再探(normalizeDirKey
+  // 会把 `D:/x` 与 `d:\x\` 合并,但这里要保持"键用原始串"的约定)
+  const unique = [...new Set(list)];
+  const results = {};
+
+  const probed = await runWithConcurrency(unique, concurrency, async (dir) => ({
+    dir,
+    url: await probeDirectoryOrigin(dir, { timeoutMs }),
+  }));
+  for (const { dir, url } of probed) results[dir] = url;
+  return results;
+}
+
 /** 仅供测试:清空缓存 */
 export function clearGitStateCache() {
   cache.clear();

package/src/ui/server/utils/directoryGitState.test.js

@@ -26,6 +26,8 @@ import {
   normalizeDirKey,
   probeDirectoryGitState,
   probeDirectoryGitStates,
+  probeDirectoryOrigin,
+  probeDirectoryOrigins,
   clearGitStateCache,
 } from './directoryGitState.js';
 
@@ -262,6 +264,51 @@ test('probeDirectoryGitState: 真实远端 → 读出上游分支与领先/落
   }
 });
 
+// ── origin 远程地址探测(远程仓库列表的「已克隆」徽标) ──────────────────
+test('probeDirectoryOrigin: 有 origin 读到地址,没有 / 非仓库 / 目录不存在都是 null', async () => {
+  const root = await fs.mkdtemp(path.join(os.tmpdir(), 'zen-origin-'));
+  const withOrigin = path.join(root, 'with-origin');
+  const noRemote = path.join(root, 'no-remote');
+  const plain = path.join(root, 'plain');
+  try {
+    await fs.mkdir(withOrigin);
+    await fs.mkdir(noRemote);
+    await fs.mkdir(plain);
+    await git(['init', '-q'], withOrigin);
+    await git(['init', '-q'], noRemote);
+    await git(['remote', 'add', 'origin', 'git@gitee.com:xz_web/xiangqi.git'], withOrigin);
+
+    assert.equal(await probeDirectoryOrigin(withOrigin), 'git@gitee.com:xz_web/xiangqi.git');
+    // 仓库但没配 origin / 根本不是仓库 / 目录不存在 —— 对"是不是克隆来的"
+    // 这三个答案都是"不知道",统一收敛成 null
+    assert.equal(await probeDirectoryOrigin(noRemote), null);
+    assert.equal(await probeDirectoryOrigin(plain), null);
+    assert.equal(await probeDirectoryOrigin(path.join(root, 'ghost')), null);
+  } finally {
+    await fs.rm(root, { recursive: true, force: true });
+  }
+});
+
+test('probeDirectoryOrigins: 键是调用方原始路径,空值与非字符串入参被过滤', async () => {
+  const root = await fs.mkdtemp(path.join(os.tmpdir(), 'zen-origins-'));
+  const repoDir = path.join(root, 'repo');
+  const plainDir = path.join(root, 'plain');
+  try {
+    await fs.mkdir(repoDir);
+    await fs.mkdir(plainDir);
+    await git(['init', '-q'], repoDir);
+    await git(['remote', 'add', 'origin', 'https://gitee.com/xz_web/xiangqi.git'], repoDir);
+
+    const results = await probeDirectoryOrigins([repoDir, plainDir, '', null, repoDir]);
+    // 重复路径只探一次但保留在结果里(键去重),空串 / null 不进结果
+    assert.deepEqual(Object.keys(results).sort(), [repoDir, plainDir].sort());
+    assert.equal(results[repoDir], 'https://gitee.com/xz_web/xiangqi.git');
+    assert.equal(results[plainDir], null);
+  } finally {
+    await fs.rm(root, { recursive: true, force: true });
+  }
+});
+
 test('probeDirectoryGitStates: 结果按调用方原始路径字符串为键,重复路径只探一次', async () => {  clearGitStateCache();
   const dir = await fs.mkdtemp(path.join(os.tmpdir(), 'zen-batch-'));
   try {