zen-gitsync 2.17.10 → 2.17.12

package/src/ui/server/routes/remoteRepos.js

@@ -38,6 +38,7 @@ import { spawn, spawnSync } from 'node:child_process';
 import fs from 'node:fs/promises';
 import os from 'node:os';
 import path from 'node:path';
+import { stripVTControlCharacters } from 'node:util';
 import { asyncRoute, HttpError } from '../utils/asyncRoute.js';
 import { getToolInstallers, launchCommandInTerminal, parseVersionOutput, publicInstallerInfo } from './fileOpen.js';
 import { readRegistryPathDirs } from '../../../utils/shellPath.js';
@@ -226,14 +227,59 @@ export async function findCliExecutable(provider) {
 
 // ── CLI 调用 ────────────────────────────────────────────────────────────────
 
+/** 剥掉 ANSI 转义序列。CLI 一旦被"强制上色",我们拿到的就是带色码的文本 ——
+ *  那东西既不能 JSON.parse,也不能直接摆在界面上给用户看。 */
+export function stripAnsi(text) {
+  return stripVTControlCharacters(String(text ?? ''));
+}
+
+/**
+ * "强制上色"这一族环境变量。它们**优先级高于 NO_COLOR**:
+ * 实测(2026-09-22)只设 `CLICOLOR_FORCE=1` 时,gh 连我们显式传下去的
+ * `NO_COLOR=1` 都不理,`gh repo list --json` 会输出**高亮 + 折行**的 JSON,
+ * 首行正是 `\x1b[1;37m[`。
+ * 后果是三重的、而且每一条都不报错:
+ *   ① `parseRepoJson` 解不出来 → repos 空;
+ *   ② `gh auth status` 里的账号名被色码包住 → user 抠不出来(null);
+ *   ③ 界面上的"错误原因"变成 `\x1b[1;37m[\x1b[m` 这种乱码(用户截图来问的那种)。
+ * 所以不能只"设 NO_COLOR",必须把整族变量从子进程环境里删掉。
+ */
+const COLOR_FORCING_ENV_KEYS = ['CLICOLOR_FORCE', 'FORCE_COLOR', 'GH_FORCE_TTY', 'GITEE_FORCE_TTY'];
+
+/**
+ * 拼子进程环境:先删掉"强制上色"整族变量,再显式声明不要颜色与非交互。
+ *
+ * 为什么要按 key 遍历删除而不是 `delete env.CLICOLOR_FORCE`:
+ *   Windows 上环境变量名**不区分大小写**,而 `process.env` 的 key 保留原样 ——
+ *   `clicolor_force` / `gh_force_tty` 这种小写写法会被大写 delete 漏掉。
+ *   (`...overrides` 放在最后是刻意的:调用方显式指定的变量应当说话算数。)
+ */
+export function buildCliEnv(overrides = {}) {
+  const env = { ...process.env };
+  for (const key of Object.keys(env)) {
+    if (COLOR_FORCING_ENV_KEYS.includes(key.toUpperCase())) delete env[key];
+  }
+  return {
+    ...env,
+    NO_COLOR: '1',
+    CLICOLOR: '0',
+    GH_PROMPT_DISABLED: '1',
+    GH_NO_UPDATE_NOTIFIER: '1',
+    GITEE_NO_UPDATE_NOTIFIER: '1',
+    GIT_TERMINAL_PROMPT: '0',
+    ...overrides,
+  };
+}
+
 /**
  * 跑一条 CLI 命令并把 stdout 收全。
  *
  * 全程非交互:GH_PROMPT_DISABLED / GIT_TERMINAL_PROMPT=0 保证需要凭据时
  * **快速失败而不是挂住**(挂住会连带前端轮询一起卡死,与 directoryFetch.js
  * 里 fetch 的处理是同一个考虑)。
- * NO_COLOR + GITEE_NO_UPDATE_NOTIFIER:输出要拿来解析,不能混进 ANSI 色码,
- * 也不要让每次调用都去查一次更新。
+ * 颜色:见 buildCliEnv(删掉强制上色变量)+ finish() 里再兜一层 stripAnsi ——
+ * 万一将来某个 CLI 不认 NO_COLOR,我们也绝不会把色码当数据用。
+ * GITEE_NO_UPDATE_NOTIFIER:不要让每次调用都去查一次更新。
  *
  * Windows 上 npm 装出来的是 .cmd 垫片,不能直接 spawn,必须过一层 shell
  * (这里的 args 全部是服务端硬编码常量,不含任何用户输入)。
@@ -245,25 +291,19 @@ export function runCli(executable, args, { timeoutMs = CLI_TIMEOUT_MS, env = {}
       stdio: ['ignore', 'pipe', 'pipe'],
       windowsHide: true,
       shell: useShell,
-      env: {
-        ...process.env,
-        NO_COLOR: '1',
-        GH_PROMPT_DISABLED: '1',
-        GH_NO_UPDATE_NOTIFIER: '1',
-        GITEE_NO_UPDATE_NOTIFIER: '1',
-        GIT_TERMINAL_PROMPT: '0',
-        ...env,
-      },
+      env: buildCliEnv(env),
     });
 
     let stdout = '';
     let stderr = '';
     let done = false;
+    // 色码必须在**收全之后**再剥:按 chunk 剥会把跨 chunk 的转义序列切坏
+    // (前一个 chunk 停在 `\x1b[`,后一个 chunk 从 `1;37m` 开始)。
     const finish = (code, timedOut) => {
       if (done) return;
       done = true;
       try { child.kill('SIGKILL'); } catch {}
-      resolve({ code, stdout, stderr, timedOut: !!timedOut });
+      resolve({ code, stdout: stripAnsi(stdout), stderr: stripAnsi(stderr), timedOut: !!timedOut });
     };
 
     child.stdout.on('data', (chunk) => { stdout += chunk; });
@@ -277,12 +317,20 @@ export function runCli(executable, args, { timeoutMs = CLI_TIMEOUT_MS, env = {}
   });
 }
 
-/** 从 CLI 输出里挑一句能给用户看的话(gitee 把错误打在 stdout,gh 打在 stderr) */
-function firstMeaningfulLine(...texts) {
+/**
+ * 从 CLI 输出里挑一句能给用户看的话(gitee 把错误打在 stdout,gh 打在 stderr)。
+ *
+ * 跳过"剥掉色码后只剩标点/符号"的行:被高亮过的 JSON 压掉色码后可能只剩一个
+ * `[`,拿它当错误提示等于没说 —— 界面上就会出现用户完全看不懂的一行(实测踩到)。
+ */
+export function firstMeaningfulLine(...texts) {
   for (const text of texts) {
     if (!text) continue;
-    for (const line of String(text).split(/\r?\n/).map((s) => s.trim())) {
-      if (line && !/^\s*$/.test(line)) return line;
+    for (const raw of String(text).split(/\r?\n/)) {
+      const line = stripAnsi(raw).trim();
+      if (!line) continue;
+      if (!/[\p{L}\p{N}]/u.test(line)) continue;
+      return line;
     }
   }
   return '';
@@ -290,13 +338,19 @@ function firstMeaningfulLine(...texts) {
 
 /** 宽松 JSON 解析:容忍 CLI 在 JSON 前面打警告/更新提示行。
  *  只定位第一个 `[` 或 `{` 再整体交给 JSON.parse —— 不做任何字符串清洗,
- *  解析不出来就是 null,由调用方决定怎么降级。 */
+ *  解析不出来就是 null,由调用方决定怎么降级。
+ *
+ *  唯一"清洗"就是剥 ANSI:被强制上色的 `--json` 输出是**带色码的多行 JSON**,
+ *  色码不影响结构,剥掉就能正常解析(2026-09-22 实测:CLICOLOR_FORCE=1 时
+ *  gh 输出首行是 `\x1b[1;37m[`)。runCli 出口已经剥过一遍,这里再兜一层,
+ *  保证任何直接调用本函数的路径都不会被色码坑到。 */
 export function parseJsonLoose(text) {
   if (!text) return null;
-  const start = text.search(/[[{]/);
+  const plain = stripAnsi(text);
+  const start = plain.search(/[[{]/);
   if (start < 0) return null;
   try {
-    return JSON.parse(text.slice(start));
+    return JSON.parse(plain.slice(start));
   } catch {
     return null;
   }
@@ -387,6 +441,25 @@ async function detectCli(provider, executable, version) {
   };
 }
 
+/**
+ * 两个平台都要给的字段(前端排序 + 卡片元信息要用):
+ *   pushedAt      最近推送 —— 前端默认按它倒序排(最活跃的排前面)
+ *   createdAt     创建时间
+ *   forks         复刻数
+ *   defaultBranch 默认分支 —— 卡片上只在非 main/master 时显示
+ *   license       许可证
+ * 取不到时一律 null(不写 0),前端据此判断"这条信息没有",而不是显示一个假的 0。
+ */
+function commonRepoFields(source) {
+  return {
+    pushedAt: source.pushedAt || null,
+    createdAt: source.createdAt || null,
+    forks: typeof source.forks === 'number' ? source.forks : 0,
+    defaultBranch: source.defaultBranch || null,
+    license: source.license || null,
+  };
+}
+
 /** gh repo list 的 --json 字段 → 统一形状 */
 export function normalizeGithubRepos(raw) {
   return raw.map((r) => ({
@@ -399,6 +472,19 @@ export function normalizeGithubRepos(raw) {
     stars: typeof r.stargazerCount === 'number' ? r.stargazerCount : 0,
     updatedAt: r.updatedAt || null,
     url: r.url || '',
+    ...commonRepoFields({
+      pushedAt: r.pushedAt,
+      createdAt: r.createdAt,
+      forks: typeof r.forkCount === 'number' ? r.forkCount : undefined,
+      // defaultBranchRef 是对象({name}),没设默认分支时为 null
+      defaultBranch: r.defaultBranchRef?.name,
+      // gh 的 licenseInfo 只给 key/name/nickname(spdxId 在 list 里没有)。
+      // key === 'other' 表示"有这个 LICENSE 文件但 GitHub 认不出是哪一种",
+      // 显示成 "Other" 是纯噪音,直接当作没有。
+      license: r.licenseInfo && r.licenseInfo.key !== 'other'
+        ? (r.licenseInfo.name || r.licenseInfo.key)
+        : undefined,
+    }),
   }));
 }
 
@@ -415,6 +501,14 @@ export function normalizeGiteeRepos(raw) {
     stars: typeof r.stargazers_count === 'number' ? r.stargazers_count : 0,
     updatedAt: r.updated_at || null,
     url: r.html_url || (r.full_name ? `https://gitee.com/${r.full_name}` : ''),
+    ...commonRepoFields({
+      pushedAt: r.pushed_at,
+      createdAt: r.created_at,
+      forks: typeof r.forks_count === 'number' ? r.forks_count : undefined,
+      defaultBranch: r.default_branch,
+      // 未设置许可证时 gitee 给的是空字符串(实测),当成 null
+      license: r.license,
+    }),
   }));
 }
 
@@ -425,7 +519,9 @@ async function listRepos(provider, executable) {
     const res = await runCli(executable, [
       'repo', 'list',
       '--limit', String(GH_LIMIT),
-      '--json', 'name,nameWithOwner,description,isPrivate,isFork,primaryLanguage,stargazerCount,updatedAt,url',
+      // 字段要和 normalizeGithubRepos 一一对应:少一个不会报错,只会静默变成 null,
+      // 表现成"卡片上少了半行信息"。
+      '--json', 'name,nameWithOwner,description,isPrivate,isFork,primaryLanguage,stargazerCount,forkCount,defaultBranchRef,licenseInfo,updatedAt,pushedAt,createdAt,url',
     ]);
     if (res.timedOut) return { repos: [], error: 'gh repo list 超时(25 秒),请检查网络或代理设置' };
     const parsed = parseRepoJson(res.stdout);
@@ -443,7 +539,7 @@ async function listRepos(provider, executable) {
       'repo', 'list',
       '--limit', String(GITEE_PAGE_SIZE),
       '--page', String(page),
-      '--json=full_name,human_name,name,path,description,private,public,fork,language,stargazers_count,updated_at,html_url',
+      '--json=full_name,human_name,name,path,description,private,public,fork,language,stargazers_count,forks_count,default_branch,license,updated_at,pushed_at,created_at,html_url',
       '--no-tui',
     ]);
     if (res.timedOut) {

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

@@ -14,17 +14,23 @@
 //
 // src/ui/server/routes/remoteRepos.js 单元测试。
 //
-// 重点覆盖两处**实测踩过的坑**,它们都不会报错、只会静默给出错误结果:
+// 重点覆盖三处**实测踩过的坑**,它们都不会报错、只会静默给出错误结果:
 //   1. `gitee auth status` 未登录时退出码是 0 —— 用退出码判会永远认为已登录
 //   2. `parseRepoJson` 对纯对象(如 auth status 的 JSON)返回 null ——
 //      拿它去解登录态会让解析悄悄退化成文本匹配,登录后拿不到用户名
+//   3. 环境里带了"强制上色"变量(如 CLICOLOR_FORCE=1)时,CLI 的 --json 输出
+//      会带 ANSI 色码 —— JSON 解不出来、账号名抠不出来,界面上只剩 `\x1b[1;37m[`
 // 另外覆盖四条路由分支(未安装 / 未登录 / 已登录 / 拉取失败)与 provider 校验。
 //
 // 不真起 Express:沿用 branchStatus.test.js 的 mock app + mock req/res 做法。
 // CLI 调用通过 registerRemoteReposRoutes 的 *Impl 注入口换成假实现,不联网、不装 gh。
+// 唯独"色码"那几条必须真起子进程 —— 用 node 自己当被测 CLI(process.execPath),
+// 不依赖机器上有没有装 gh。
 import { test } from 'node:test'
 import assert from 'node:assert/strict'
 import {
+  buildCliEnv,
+  firstMeaningfulLine,
   interpretGiteeAuthStatus,
   interpretGithubAuthStatus,
   normalizeGiteeRepos,
@@ -33,8 +39,17 @@ import {
   parseRepoJson,
   pickUserName,
   registerRemoteReposRoutes,
+  runCli,
+  stripAnsi,
 } from './remoteRepos.js'
 
+const ESC = '\u001b'
+
+/** 用 node 自己当"被测 CLI":跑一段内联脚本,把 stdout 收回来。 */
+function runNode(code, options) {
+  return runCli(process.execPath, ['-e', code], options)
+}
+
 /** 最小 express app mock:只实现 get + post 与 handler 调用 */
 function makeApp() {
   const handlers = new Map()
@@ -193,6 +208,157 @@ test('normalizeGiteeRepos: private 缺失时用 public 反推;url 缺失时用 f
   assert.equal(explicit.isPrivate, true)
 })
 
+// ── 排序 / 卡片元信息用到的字段 ──────────────────────────────────────────────
+//
+// 前端默认按 pushedAt 倒序排,卡片第三行显示"最近推送 / Fork / 默认分支 / 许可证"。
+// 这些字段名两个平台各不相同,映射写错**不会报错** —— 只会静默变成 null,
+// 表现成"卡片上少半行字"或"排序乱掉",所以逐个钉住。
+test('normalizeGithubRepos: forkCount/defaultBranchRef/licenseInfo 映射到统一字段', () => {
+  const [repo] = normalizeGithubRepos([{
+    nameWithOwner: 'xz333221/zen-gitsync',
+    forkCount: 4,
+    defaultBranchRef: { name: 'develop' },
+    licenseInfo: { key: 'mit', name: 'MIT License' },
+    pushedAt: '2026-09-22T12:00:00Z',
+    createdAt: '2024-11-25T08:11:38Z',
+  }])
+  assert.equal(repo.forks, 4)
+  assert.equal(repo.defaultBranch, 'develop')
+  assert.equal(repo.license, 'MIT License')
+  assert.equal(repo.pushedAt, '2026-09-22T12:00:00Z')
+  assert.equal(repo.createdAt, '2024-11-25T08:11:38Z')
+
+  // 字段缺失时是 null / 0,不是 undefined —— 前端直接拿它做判断和比较
+  const [bare] = normalizeGithubRepos([{}])
+  assert.equal(bare.forks, 0)
+  assert.equal(bare.defaultBranch, null)
+  assert.equal(bare.license, null)
+  assert.equal(bare.pushedAt, null)
+})
+
+test('normalizeGithubRepos: licenseInfo.key=other(有 LICENSE 但认不出)不算许可证', () => {
+  // 实测:本仓库自己的 licenseInfo 就是 {key:'other', name:'Other'} ——
+  // 卡片上显示一个 "Other" 徽标是纯噪音,当成没有处理。
+  const [repo] = normalizeGithubRepos([{ nameWithOwner: 'a/b', licenseInfo: { key: 'other', name: 'Other' } }])
+  assert.equal(repo.license, null)
+})
+
+test('normalizeGiteeRepos: 未设置许可证时 gitee 给空串,要归一成 null', () => {
+  const [none] = normalizeGiteeRepos([{ full_name: 'flowdash/blog', license: '' }])
+  assert.equal(none.license, null)
+  assert.equal(none.forks, 0)
+  assert.equal(none.defaultBranch, null)
+
+  const [full] = normalizeGiteeRepos([{
+    full_name: 'flowdash/article-generator',
+    license: 'MIT',
+    forks_count: 2,
+    default_branch: 'develop',
+    pushed_at: '2026-09-02T01:08:48+08:00',
+    created_at: '2026-04-30T01:29:35+08:00',
+  }])
+  assert.equal(full.license, 'MIT')
+  assert.equal(full.forks, 2)
+  assert.equal(full.defaultBranch, 'develop')
+  assert.equal(full.pushedAt, '2026-09-02T01:08:48+08:00')
+  assert.equal(full.createdAt, '2026-04-30T01:29:35+08:00')
+})
+
+// ── 色码(ANSI)—— 强制上色的环境会把解析和提示一起搞坏 ───────────────────────
+//
+// 实测(2026-09-22):只要启动服务的那个终端里带着 CLICOLOR_FORCE=1,gh 就**不理**
+// 我们传下去的 NO_COLOR=1,`gh repo list --json` 会吐**高亮 + 折行**的 JSON:
+//     \x1b[1;37m[\x1b[m
+//       \x1b[1;37m{\x1b[m
+//         \x1b[1;34m"name"\x1b[m\x1b[1;37m:\x1b[m \x1b[32m"zen-gitsync"\x1b[m
+// 于是:① repos 空(JSON.parse 失败)② user 为 null(账号名被色码包住)
+// ③ 界面上的"错误原因"变成 `\x1b[1;37m[\x1b[m` 这种乱码。
+
+test('stripAnsi: 剥掉 SGR 与光标类转义序列', () => {
+  assert.equal(stripAnsi(`${ESC}[1;37m[${ESC}[m`), '[')
+  assert.equal(stripAnsi('plain text'), 'plain text')
+  assert.equal(stripAnsi(`${ESC}[2K${ESC}[1Aabc`), 'abc')
+  assert.equal(stripAnsi(null), '')
+})
+
+test('buildCliEnv: 继承来的"强制上色"变量一律清掉,并显式关色(Windows 上大小写不敏感)', () => {
+  process.env.CLICOLOR_FORCE = '1'
+  process.env.gh_force_tty = '100%'
+  process.env.FORCE_COLOR = '3'
+  try {
+    const env = buildCliEnv()
+    assert.equal(env.CLICOLOR_FORCE, undefined)
+    // 小写写法必须一起清 —— Windows 上环境变量名不区分大小写,
+    // 只 delete 大写那种写法会漏掉它,而漏掉就等于没修。
+    assert.equal(env.gh_force_tty, undefined)
+    assert.equal(env.GH_FORCE_TTY, undefined)
+    assert.equal(env.FORCE_COLOR, undefined)
+    assert.equal(env.NO_COLOR, '1')
+    assert.equal(env.CLICOLOR, '0')
+    // 其它环境变量照旧带下去(子进程要能拿到 PATH 等)
+    assert.equal(env.PATH, process.env.PATH)
+  } finally {
+    delete process.env.CLICOLOR_FORCE
+    delete process.env.gh_force_tty
+    delete process.env.FORCE_COLOR
+  }
+})
+
+test('runCli: 父进程带着强制上色变量时,子进程既看不到它、输出里的色码也被剥掉', async () => {
+  const code = 'process.stdout.write(String(process.env.CLICOLOR_FORCE) + "|" + String(process.env.NO_COLOR) + "|" + String(process.env.GH_FORCE_TTY))'
+  process.env.CLICOLOR_FORCE = '1'
+  process.env.GH_FORCE_TTY = '80'
+  try {
+    const res = await runNode(code)
+    assert.equal(res.code, 0)
+    assert.equal(res.stdout, 'undefined|1|undefined')
+  } finally {
+    delete process.env.CLICOLOR_FORCE
+    delete process.env.GH_FORCE_TTY
+  }
+
+  // 色码在出口处统一剥掉(按 chunk 剥会把跨 chunk 的转义序列切坏,所以必须在收全后剥)
+  const colored = await runNode(`process.stdout.write(${JSON.stringify(`${ESC}[1;37m[${ESC}[m\nabc\n`)})`)
+  assert.equal(colored.stdout, '[\nabc\n')
+
+  // 同样适用于 stderr —— 报错信息是要直接摆到界面上的
+  const coloredErr = await runNode(`process.stderr.write(${JSON.stringify(`${ESC}[31mHTTP 401: Bad credentials${ESC}[m\n`)})`)
+  assert.equal(coloredErr.stderr, 'HTTP 401: Bad credentials\n')
+})
+
+test('runCli + interpretGithubAuthStatus: 彩色 auth status 也能抠出账号名', async () => {
+  // 没有这一层修复时,账号名被 \x1b[1;37m 包住 → 正则匹配不到 → user 为 null,
+  // 界面显示"已登录 "后面空一格(users 看不到自己是谁)。
+  const output = `github.com\n  ${ESC}[32m✓${ESC}[m Logged in to github.com account ${ESC}[1;37mxz333221${ESC}[m (keyring)\n`
+  const res = await runNode(`process.stdout.write(${JSON.stringify(`${output}\n`)})`)
+  assert.deepEqual(interpretGithubAuthStatus(res.code, res.stdout), {
+    authenticated: true,
+    user: 'xz333221',
+  })
+})
+
+test('parseJsonLoose: 高亮过的多行 JSON 剥掉色码后照常解析(兜底,不依赖 runCli)', () => {
+  const colored = [
+    `${ESC}[1;37m[${ESC}[m`,
+    `  ${ESC}[1;37m{${ESC}[m`,
+    `    ${ESC}[1;34m"name"${ESC}[m${ESC}[1;37m:${ESC}[m ${ESC}[32m"zen-gitsync"${ESC}[m`,
+    `  ${ESC}[1;37m}${ESC}[m`,
+    `${ESC}[1;37m]${ESC}[m`,
+  ].join('\n')
+  assert.deepEqual(parseJsonLoose(colored), [{ name: 'zen-gitsync' }])
+  assert.deepEqual(parseRepoJson(colored), [{ name: 'zen-gitsync' }])
+})
+
+test('firstMeaningfulLine: 剥掉色码后只剩标点的行不算"能给用户看的话"', () => {
+  // 用户看到的那行乱码正是这一条:整段输出剥完色码只剩 [,拿它当错误提示等于没说
+  assert.equal(firstMeaningfulLine('', `${ESC}[1;37m[${ESC}[m`), '')
+  assert.equal(
+    firstMeaningfulLine(`${ESC}[1;37m[${ESC}[m`, 'HTTP 401: Bad credentials (https://api.github.com/graphql)'),
+    'HTTP 401: Bad credentials (https://api.github.com/graphql)',
+  )
+  assert.equal(firstMeaningfulLine(''), '')
+})
+
 // ── 路由分支 ────────────────────────────────────────────────────────────────
 
 test('GET /api/remote-repos: provider 非法 → 400', async () => {

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

@@ -298,19 +298,25 @@ export async function runAgentTurn({ session, model, userMessage, images = [], c
 
     const { content, toolCalls } = result;
 
+    // 推理内容必须原样带回历史。DeepSeek 系 thinking 模式下带 tool_calls 的 assistant
+    // 消息一旦缺 reasoning_content，下一轮回传就被上游 400 拒掉:
+    // "The `reasoning_content` in the thinking mode must be passed back to the API"。
+    // 口径与 CLI 侧 src/cli/ai/turn.js 的 assistant.reasoning_content 保持一致。
+    const withReasoning = msg => (result.reasoning ? { ...msg, reasoning_content: result.reasoning } : msg);
+
     // 无工具调用：本轮结束
     if (toolCalls.length === 0) {
-      session.messages.push({ role: 'assistant', content: content || null });
+      session.messages.push(withReasoning({ role: 'assistant', content: content || null }));
       send({ type: 'done', content: content || '' });
       return { aborted: false };
     }
 
     // 有工具调用：assistant(带 tool_calls)入历史
-    session.messages.push({
+    session.messages.push(withReasoning({
       role: 'assistant',
       content: content || null,
       tool_calls: toolCalls
-    });
+    }));
 
     // 逐个执行工具
     for (const tc of toolCalls) {

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

@@ -165,3 +165,19 @@ test('共享模块本身必须导出这些口径', () => {
     }
   }
 });
+
+test('两个入口都把推理内容写进 assistant 历史消息', () => {
+  // 上游要求（DeepSeek 系 thinking 模式 + tool calls）：回传的历史里，assistant 消息当初产出的
+  // reasoning_content 必须原样带回，否则下一轮请求被 400 拒掉：
+  //   The `reasoning_content` in the thinking mode must be passed back to the API.
+  // 传输层已经把它返回了（streamChatOnce 的 result.reasoning），漏的是"入历史"这一步 ——
+  // Web 侧曾经只解构 content/toolCalls，于是纯聊天没事、一调工具就挂，又是两个入口表现分叉。
+  // 只认代码行，注释里的提及不算（CLI 是 `assistant.reasoning_content = …`，Web 是 `reasoning_content: …`）。
+  for (const { label, file } of ENTRY_POINTS) {
+    const code = readFileSync(file, 'utf8')
+      .split(/\r?\n/)
+      .filter(line => !/^\s*(\/\/|\*|\/\*)/.test(line))
+      .join('\n');
+    assert.match(code, /reasoning_content/, `${label} 没有把推理内容写进 assistant 历史消息`);
+  }
+});

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

@@ -14,7 +14,7 @@
 //
 // 智能体对话 session 持久化层。
 //
-// 与 ai-split-sessions 不同，这里存储的是完整 agent 对话（含工具调用）。
+// 这里存储的是完整 agent 对话（含工具调用）—— 智能体面板专用，与任务执行链路的 job 记录互不相干。
 // 消息格式兼容 OpenAI chat completions：
 //   - { role: 'system', content: string }
 //   - { role: 'user', content: string | content_parts[] }
@@ -29,7 +29,7 @@ import path from 'path';
 import { logger } from './shared.js';
 import { AGENT_SESSIONS_DIR } from '../../../../paths.js';
 
-// 数据目录：与 ai-split-sessions 平级，定义见 src/paths.js
+// 数据目录定义见 src/paths.js
 export { AGENT_SESSIONS_DIR };
 // 历史遗留别名:此前这里有个"从 import.meta.url 推路径"的错误实现(Windows 下不可靠),
 // 后来补了个 _FIXED 版本两套并存。现在统一到 paths.js 的单一常量。

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

@@ -57,11 +57,11 @@ test('项目清单带名称、路径与三列计数，并标出当前项目', ()
   const projects = [entry('D:\\ws\\zen-gitsync'), entry('D:\\ws\\article-generator')];
   const tasks = [
     // 已完成：simple 任务跑过一次且 job 是 done
-    { id: 't1', projectPath: 'D:\\ws\\zen-gitsync', type: 'simple', subtasks: [] },
+    { id: 't1', projectPath: 'D:\\ws\\zen-gitsync', type: 'simple' },
     // 进行中：有 job 处于 running
-    { id: 't2', projectPath: 'D:\\ws\\article-generator', type: 'simple', subtasks: [] },
+    { id: 't2', projectPath: 'D:\\ws\\article-generator', type: 'simple' },
     // 待处理：没子任务也没执行过
-    { id: 't3', projectPath: 'D:\\ws\\article-generator', type: 'simple', subtasks: [] },
+    { id: 't3', projectPath: 'D:\\ws\\article-generator', type: 'simple' },
   ];
   const jobs = [job('j1', 't1', 'done'), job('j2', 't2', 'running')];
 
@@ -106,7 +106,7 @@ test('项目数超过上限时只列前 N 个，但合计仍按全量算', () =>
   for (let i = 0; i < total; i += 1) {
     const p = `D:\\ws\\p${i}`;
     projects.push(entry(p));
-    tasks.push({ id: `t${i}`, projectPath: p, type: 'simple', subtasks: [] });
+    tasks.push({ id: `t${i}`, projectPath: p, type: 'simple' });
   }
 
   const block = buildEnvContextBlock({
@@ -133,8 +133,8 @@ test('maxProjects 可覆盖，且非法值回落到默认上限', () => {
 test('没关联项目的任务不计入项目行，但计入合计', () => {
   const projects = [entry('D:\\ws\\a')];
   const tasks = [
-    { id: 't1', projectPath: 'D:\\ws\\a', type: 'simple', subtasks: [] },
-    { id: 't2', projectPath: '', type: 'simple', subtasks: [] },
+    { id: 't1', projectPath: 'D:\\ws\\a', type: 'simple' },
+    { id: 't2', projectPath: '', type: 'simple' },
   ];
   const block = buildEnvContextBlock({ projects, tasks, jobs: [], ...PATHS });
   // 项目行只认 key 匹配得上的那条（t1）→ 待处理 1
@@ -153,8 +153,8 @@ test('没关联项目的任务不计入项目行，但计入合计', () => {
 test('项目行/合计的段数必须与 TASK_COLUMNS 一致，且每列都有中文标签', () => {
   const projects = [entry('D:\\ws\\a')];
   const tasks = [
-    { id: 't1', projectPath: 'D:\\ws\\a', type: 'simple', subtasks: [] },
-    { id: 't2', projectPath: 'D:\\ws\\a', type: 'simple', subtasks: [] },
+    { id: 't1', projectPath: 'D:\\ws\\a', type: 'simple' },
+    { id: 't2', projectPath: 'D:\\ws\\a', type: 'simple' },
   ];
   const block = buildEnvContextBlock({ projects, tasks, jobs: [], ...PATHS });