@zaofan/dsh-qqbot 1.10.35 → 1.10.36

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zaofan/dsh-qqbot",
-  "version": "1.10.35",
+  "version": "1.10.36",
   "description": "QQ Bot IM channel plugin for deepseek-harness (dsh)",
   "type": "module",
   "main": "./entry.js",

package/settings-host.js

@@ -243,6 +243,9 @@ function viewOf(settings, ns) {
  * 位置：{DSH_HOME}/qqbot-settings/<instanceNs>.json
  */
 const OWN_SETTINGS_DIR = join(DSH_HOME_DIR, 'qqbot-settings');
+/** appSecret 的回显掩码：前端看到它 / 空值 = 保留原值（不覆盖）。
+ *  2026-10-25 提到模块级：/read 与 /accounts 都要用。 */
+const SECRET_MASK = '********';
 
 /**
  * 路径按 profile 隔离（2026-10-11；规则必须与 `src/features/ns-guard.ts` 的 `nsPathsOf()` 保持一致，改一处同步另一处）：
@@ -258,36 +261,36 @@ const OWN_SETTINGS_DIR = join(DSH_HOME_DIR, 'qqbot-settings');
  * 现在与 `src/shared/utils.ts` 的 `resolveProfileName()` **同规则**：环境变量 → `PROFILE_ROOT` 目录名
  * （要求其父目录名为 `profiles`，挡住源码 checkout 目录）→ 落 `_default`。改一处务必同步另一处。
  */
-function resolveProfileName() {
-  // ── 按维护手册 §「profile 定位的正解：回写环境变量，别改推断逻辑」 ──
-  //    宿主的 profileContext 是权威答案，src/index.ts 的 apply() 已经把它回写到：
-  //      process.env.DSH_PROFILE / DSH_PROFILE_DIR
-  //      globalThis.__dshProfileName / __dshProfileContext
-  //    这里只要**读**它，别自己猜路径。
-
-  // ① 环境变量（apply 阶段回写，最直接）
-  const env = String(process.env.DSH_PROFILE || '').trim();
-  if (env) return env;
-
-  // ② apply() 落下的全局变量 —— **这是关键补的一层**：
-  //    settings-host.js 有可能在 apply() 之前就被加载（模块顶层求值），
-  //    那时环境变量还没回写；但真正用到 profile 名是在请求处理阶段（惰性化后），
-  //    那时 __dshProfileName 早就有了。
-  try {
-    const g = globalThis.__dshProfileName;
-    if (typeof g === 'string' && g.trim()) return g.trim();
-  } catch { /* 下一层 */ }
-
-  // ③ 老宿主没有 profileContext 服务 → 只能从 PROFILE_ROOT 目录名反推
-  try {
-    if (basename(dirname(PROFILE_ROOT)) === 'profiles') {
-      const n = basename(PROFILE_ROOT);
-      if (n && n !== 'profiles') return n;
-    }
-  } catch { /* 放弃 */ }
-
-  return '';
-}
+function resolveProfileName() {
+  // ── 按维护手册 §「profile 定位的正解：回写环境变量，别改推断逻辑」 ──
+  //    宿主的 profileContext 是权威答案，src/index.ts 的 apply() 已经把它回写到：
+  //      process.env.DSH_PROFILE / DSH_PROFILE_DIR
+  //      globalThis.__dshProfileName / __dshProfileContext
+  //    这里只要**读**它，别自己猜路径。
+
+  // ① 环境变量（apply 阶段回写，最直接）
+  const env = String(process.env.DSH_PROFILE || '').trim();
+  if (env) return env;
+
+  // ② apply() 落下的全局变量 —— **这是关键补的一层**：
+  //    settings-host.js 有可能在 apply() 之前就被加载（模块顶层求值），
+  //    那时环境变量还没回写；但真正用到 profile 名是在请求处理阶段（惰性化后），
+  //    那时 __dshProfileName 早就有了。
+  try {
+    const g = globalThis.__dshProfileName;
+    if (typeof g === 'string' && g.trim()) return g.trim();
+  } catch { /* 下一层 */ }
+
+  // ③ 老宿主没有 profileContext 服务 → 只能从 PROFILE_ROOT 目录名反推
+  try {
+    if (basename(dirname(PROFILE_ROOT)) === 'profiles') {
+      const n = basename(PROFILE_ROOT);
+      if (n && n !== 'profiles') return n;
+    }
+  } catch { /* 放弃 */ }
+
+  return '';
+}
 /**
  * 当前 profile 对应的存档子目录名。
  *
@@ -297,12 +300,12 @@ function resolveProfileName() {
  *   （真实存档在 qqbot-settings/desktop/xxx.json）⇒ 凭据读不到 ⇒ nsBot() null。
  *   改成每次调用时现算（开销是几次字符串/目录读，API 请求级，可忽略）。
  */
-function nsProfileDir() {
-  // 惰性求值（issue #20 的核心修复）：原来这是个模块顶层 const，
-  // 算的时候 apply() 还没回写 DSH_PROFILE → 永远落 _default。
-  const n = String(resolveProfileName()).trim().replace(/[^A-Za-z0-9_-]/g, '_');
-  if (n) return n;
-  return '_default';
+function nsProfileDir() {
+  // 惰性求值（issue #20 的核心修复）：原来这是个模块顶层 const，
+  // 算的时候 apply() 还没回写 DSH_PROFILE → 永远落 _default。
+  const n = String(resolveProfileName()).trim().replace(/[^A-Za-z0-9_-]/g, '_');
+  if (n) return n;
+  return '_default';
 }
 function safeNsOf(ns) {
   return String(ns || NS).replace(/[^A-Za-z0-9_-]/g, '_') || 'im-qqbot';
@@ -579,6 +582,15 @@ export function apply(ctx) {
     const own = readOwnSettings(ns);
     // 自有存储优先：它就是用户在设置页保存的那份
     const value = Object.assign({}, base.value || {}, own);
+    // ── 2026-10-25 修「保存一次 appSecret 就变成旧值」──────────────────────
+    //   原来把 appSecret 明文一起回给前端。value = Object.assign(base.value, own)，
+    //   而 base 来自 cordis.patch.yml（**历史旧值**，2026-09-24 起就不再写它了）。
+    //   一旦 own 里缺 appSecret，patch 的旧值就露出来；前端把它当"当前值"，
+    //   下次保存原样提交回来 → 盖掉用户存的那份 → 每次保存都被顶回去。
+    //   修法：这个接口永远只回掩码，前端已有的「掩码/空 = 保留原值」逻辑会正确处理。
+    if (value && typeof value === 'object' && value.appSecret) {
+      value.appSecret = SECRET_MASK;
+    }
     // 2026-10-11: 顺手把"这个槽是谁在写"告诉前端 —— 被**别的 profile**占用时设置页能一眼看到（判据同 writeOwnSettings）
     const w = readNsWriter(ns);
     const conflict = nsConflictOf(ns);
@@ -1246,7 +1258,7 @@ export function apply(ctx) {
   // 账号列表(多账号: 每条带 settings ns 与数据目录, 供二级 UI 按账号读写)
   // 借鉴 zhengjy01/dsh-qqbot-panel(2026-09-06): appSecret 只回显掩码(不泄露明文),
   // 并带 hasSecret 供前端区分"已保存"与"真空"; 保存时空值/掩码 = 保留原 secret。
-  const SECRET_MASK = '********';
+
   route(ctx, 'GET', '/api/qqbot-settings/accounts', async (_req, res) => {
     try {
       const { bots, hasFile } = parsePatch();
@@ -2188,6 +2200,18 @@ export function apply(ctx) {
   function mergeOwnIntoCfg(cfg, nsLike) {
     let own = {};
     try { own = readOwnSettings(String(nsLike)) || {}; } catch { own = {}; }
+    // ── 2026-10-25 修「保存一次密钥就变」的另一半 ──────────────────────
+    //   patch 里的 appSecret 是**历史旧值**（2026-09-24 起已不再写 patch）。
+    //   原来自有存储缺 appSecret 时就一直回退到 patch ⇒ patch 一旦被写脏，
+    //   用户怎么保存都会被它顶回去。改成：**只在自有存储没有时继承一次并落盘**，
+    //   之后永远以自有存储为准。
+    try {
+      if ((!own || !own.appSecret) && cfg && cfg.appSecret) {
+        const seeded = Object.assign({}, own, { appSecret: String(cfg.appSecret) });
+        writeOwnSettings(String(nsLike), seeded);
+        own = seeded;
+      }
+    } catch { /* 继承失败不影响本次读取 */ }
     const pick = (k) => {
       const o = own[k];
       if (o !== undefined && o !== '') return o;      // ← 自有存储优先（用户最新保存的）