dsh-plugin-tool-management 0.21.0 → 0.21.1

package/lib/context-inject-frame.js

@@ -48,11 +48,34 @@ export const DOMAIN_FRAME = {
         //     这是它做判断时要用的事实，不是反推。
         //   - 代价 ≈24 tok/轮，只在这一域。
         title: 'Current scene',
-        how: "A scene is the user's working mode: the memories, prompt, tools and personas you are given are already filtered to this one.",
+        how: "A scene is the user's working mode: the memories, prompt, tools and personas you are given are already filtered to it.",
         supersede: SUPERSEDE_NOTE,
     },
     memory: {
         title: 'Current memories',
+        // 2026-10-18（审查 #7）：记忆段截断时，折叠行 `- ... ({rest} more not listed)` 给了计数
+        // 但没给出口——被截掉的条目连名字都没列出来，模型既调不了 `memory_manager_read`（不知道
+        // 传什么名字），也不知道该用 `memory_manager_list` 去发现。另外三个域的溢出行都点名了
+        // 自己的 list 工具，唯独记忆段没有。
+        //
+        // 修法选了「frame `how` 条件点名」而不是「把 listToolVisible 管道接到记忆服务」：记忆 SWR
+        // 缓存是指纹驱动的（缓存渲染结果，不是缓存数据按 agent 重渲染），重构成技能目录那种方式
+        // 要动 4–5 个文件。而 `how` 已经能看 `{ text }`——MCP 就是这么按正文内容决定发不发的。
+        // `text.includes('more not listed')` 只在折叠行出现时命中（rest > 0）；rest = 0（清单列全）
+        // 时折叠行不出现，how 不发——那些条目有名字，模型调 `memory_manager_read` 就行。
+        // 工具被关掉（兼容页 / 人设 toolsDeny）时不点名——与另外三域同一纪律。
+        how: ({ text, toolHidden }) => {
+            // 折叠行 `more not listed` 和截断标记 `(Truncated:` 两种信号都要覆盖：
+            //   - 折叠行 = 清单部分列了 + 剩余没列 → 没名的那些需要 list 工具发现
+            //   - 标记 = 连清单都没放进来 → 全部没名，更需要 list 工具
+            //   - 清单列全（`## Not injected` + 所有名字，rest = 0）→ 两种信号都没有 → 不发 how：
+            //     模型用 `memory_manager_read` 按名字读就行
+            if (!text.includes('more not listed') && !text.includes('(Truncated:'))
+                return undefined;
+            if (toolHidden('memory_manager_list'))
+                return undefined;
+            return 'Not all memories fit here — see the rest with `memory_manager_list`.';
+        },
         supersede: SUPERSEDE_NOTE,
     },
     mcp: {
@@ -154,23 +177,31 @@ export const DOMAIN_FRAME = {
         // 说明**这些规则是要遵循的**。记忆域有授权语、场景域有定义，唯独效力最强的这一域空着 ——
         // 而官方那条行给的是**降权**措辞（"可作参考"），我们当初刻意没抄，于是等于什么都没说。
         // 现在补一句正面的：要遵循，且用户当场说的优先。
-        how: 'These are the instructions in force for this machine; follow them unless the user says otherwise in this conversation.',
+        // 措辞用 "right now" 不用 "for this machine"（2026-10-18 审查）：场景预设接管时，正文是
+        // 场景绑定的那份 AGENTS.md，不是机器全局的 —— "for this machine" 在那个态下是假话。
+        // "right now" 同时覆盖全局 AGENTS.md 与场景预设两种来源，由正文首行 `Instructions from:`
+        // 给出具体出处。
+        how: 'These are the instructions in force right now; follow them unless the user says otherwise in this conversation.',
         supersede: SUPERSEDE_NOTE,
     },
 };
 /**
- * `how` 行里**点名过的工具**（六个域里只有 skills / subagents 两个会点名）。
+ * `how` 行里**点名过的工具**（skills / subagents / memory 三个域会点名）。
  *
  * 为什么要有这份名单：一条 `how` 行是否成立，取决于它点名的工具在**这个 agent 手里**还在不在。
- * 名单放在这里而不是调用方 —— 与上面那两条 `how` 相邻，改一处不会漏另一处。
+ * 名单放在这里而不是调用方 —— 与上面那几条 `how` 相邻，改一处不会漏另一处。
  *
  * "不在手里"有两条路，都要算：
  *   ① 兼容页「模型工具表」把它关掉（用户全局开关，名单在 index.ts 的 `hiddenTools` 里）；
  *   ② **人设的工具限制**把它砍掉（`decideToolFilter` → 官方 `tools.restrict`，只作用于那份
  *      人设启动的子代理）。②此前没人管：目录照注、`how` 行照样点名一个模型手里没有的工具，
  *      等于诱导它去调一个不存在的名字（2026-10-07 修）。
+ *
+ * `memory_manager_list` 2026-10-18 加入（审查 #7）：记忆段截断折叠行不点名查询工具时，
+ * 被截掉的条目无名、模型无法发现它们。`how` 在折叠行出现时点名它，与技能 / 子智能体 /
+ * MCP 三个域的溢出行同构。
  */
-export const HOW_NAMED_TOOLS = ['skill_manager_read', 'subagent_manager_run'];
+export const HOW_NAMED_TOOLS = ['skill_manager_read', 'subagent_manager_run', 'memory_manager_list'];
 /** 域声明里没登记的 key（理论上到不了这里）：给一个不出错的通用框架。 */
 export const fallbackFrame = (label) => ({
     title: `Current ${label}`,

package/lib/context-inject.js

@@ -281,7 +281,7 @@ export function createContextInjector(deps) {
         // 的可见提示由兼容页那条「去填令牌」横幅承担。
         try {
             if (deps.tokenGateActive && deps.tokenGateActive()) {
-                log('令牌未验证：本轮对话被拒绝（在「工具 → 兼容」页填入访问令牌后恢复）');
+                log('令牌未验证：本轮对话被拒绝（在「工具管理 → 兼容」页填入访问令牌后恢复）');
                 return { kind: 'reject' };
             }
         }

package/lib/index.js

@@ -398,14 +398,19 @@ export default {
             // 退出回放时也不该去动它们，否则会改写用户对副本的显式策略记录。
             currentSkills: async () => (await skillScan()).states,
             applySkills: async (target) => {
-                // 三道过滤，缺一不可：
+                // 四道过滤，缺一不可：
                 //  1. 被同名覆盖的副本直接跳过——它们本来就不生效，写策略只会污染用户的记录。
                 //  2. 结构不完整的技能也跳过——core 对它们一律拒绝启停（error.skill.notLoadable），
                 //     不跳过就会抛成"应用失败"，**整个场景进不去**（老档案里的残留键能踩到）。
-                //  3. **已经处于目标状态的技能不再写**。`applySkills` 写的是显式策略记录，而目标
+                //  3. **已经不存在的技能也跳过**（用户 2026-10-18 报「启动的场景里有被删除的内容，
+                //     就导致没办法退出」）：快照 / 档案里记的是"进场景那一刻还在"的键，场景期间那条
+                //     技能被删掉之后，退出时按快照回放必然报 `技能不存在`，于是整次退出被这一条打断、
+                //     mode 停在原地 —— 用户被困在场景里。没有可写的对象，跳过才是对的。
+                //  4. **已经处于目标状态的技能不再写**。`applySkills` 写的是显式策略记录，而目标
                 //     状态是生效状态，两者不等价：对已经生效为「停用」的技能再 disable 一次，
                 //     只是把「没有记录」固化成「显式停用」，场景模式进出一次就会改写 state.json。
                 const { states, shadowed, unloadable } = await skillScan();
+                const skipped = [];
                 for (const [key, on] of Object.entries(target)) {
                     const i = key.indexOf('/');
                     if (i <= 0)
@@ -417,13 +422,29 @@ export default {
                     // 继续往下走会抛成"应用失败"，**整个场景就进不去了**（老档案里的残留键能踩到）。
                     if (unloadable.has(key))
                         continue;
+                    // 这次扫描里根本没有这个键 = 技能已被删除（或所在目录已消失）：没有可写的对象。
+                    if (!(key in states)) {
+                        skipped.push(`${key}（已不存在）`);
+                        continue;
+                    }
                     if (states[key] === on)
                         continue;
                     const root = key.slice(0, i);
                     const name = key.slice(i + 1);
                     const r = await skillsService.ops[on ? 'skill-enable' : 'skill-disable']({ root, name });
-                    if (r && r.ok === false)
+                    if (r && r.ok === false) {
+                        // 结构性失败（这一条现在解不开）只记警告并跳过 —— 与 applySkillSourceSwitches
+                        // 同一条口径：拿它中断整个进入 / 退出只会把用户困在场景里，而这一行与本次改动无关。
+                        const code = String((r && r.code) || '');
+                        if (code === 'error.skill.notFound' || code === 'error.skill.notLoadable' || code === 'error.skill.noFrontmatter') {
+                            skipped.push(`${key}（${r.error}）`);
+                            continue;
+                        }
                         throw new Error(`技能 ${key} 应用失败: ${r.error}`);
+                    }
+                }
+                if (skipped.length) {
+                    ctx.logger?.warn?.(`scene: 跳过无法应用的技能 —— ${skipped.join('、')}`);
                 }
             },
             // ── 服务器级 / 来源级启停（P5）────────────────────────────────────────────
@@ -705,7 +726,7 @@ export default {
         // 三份界面设置侧车（inject / scene / slash）拆到 ./settings-sidecars.ts：读走 3 秒 TTL 缓存，
         // 写共用上面那条串行写锁。工厂只在这里调用一次，三份缓存仍是「每次 apply 一份」。
         // 传的是 apply 作用域里那几个函数声明的引用（提升已初始化），不是快照值。
-        const { readInjectSettings, injectSettingsSync, injectNoticeOptions, injectSettingsOp, readSceneSettings, sceneSettingsOp, readSlashSettings, slashSettingsOp, } = createSettingsSidecars({
+        const { readInjectSettings, injectSettingsSync, injectNoticeOptions, injectSettingsOp, readSceneSettings, sceneSettingsOp, readSlashSettings, slashSettingsOp, readSidebarSettings, sidebarSettingsOp, } = createSettingsSidecars({
             ensurePaths,
             readJsonFile,
             writeJsonFile,
@@ -769,7 +790,7 @@ export default {
                     // 关掉或没配都是空串）而本次启动还没有人验过 ⇒ 这一步不放行，宿主把 turn 收成
                     // `blocked`。这是"没输入令牌就没法对话"的唯一真正落实 —— 客户端那条输入框锁定在
                     // 本宿主上无路可走（`ctx.conversation.blocks` 不存在，见 ContextInjectorDeps 注释）。
-                    // 只读 op 不受影响，所以「工具 → 兼容」页照常能打开、能填令牌 —— 解铃就在那里。
+                    // 只读 op 不受影响，所以「工具管理 → 兼容」页照常能打开、能填令牌 —— 解铃就在那里。
                     tokenGateActive: () => TOKEN !== '' && !acceptedThisBoot(),
                 });
                 contextInjectorLive = () => injector.live();
@@ -1675,6 +1696,9 @@ export default {
             // 斜杠命令入口的开关（读 / 写）：客户端 boot 时读一次决定挂不挂 `/` 菜单里的「工具」段，
             // 兼容页那两颗开关写一次。同 `scene-settings`：按写门禁但不冻结（见 slash-settings.ts）。
             'slash-settings': (args) => slashSettingsOp(args),
+            // 侧栏入口的开关（读 / 写）：客户端 boot 时读一次决定注不注册侧栏那一行，兼容页那颗
+            // 按钮写一次。同 `slash-settings`：按写门禁但不冻结（见 sidebar-settings.ts）。
+            'sidebar-settings': (args) => sidebarSettingsOp(args),
             // state-doctor：跨域**悬空引用**体检（只读，实现与理由见 src/ops/state-doctor.ts）。
             // 这里只做"把权威集合取来"这一件事 —— 某一域读失败就传 null，体检会把它记进
             // `skipped` 而不是当成"该域没有悬空项"（读不到 ≠ 不存在，报成后者就是骗人）。

package/lib/memories/snapshot.js

@@ -412,12 +412,12 @@ export function renderSceneMemory(probe, index, maxBytes) {
             return byteLen(marker) <= space ? marker : '';
         const rest = missed.length - lines.length;
         const fold = `- ... (${rest} more not listed)\n`;
-        // 清单已经列全时**不再加标记**：`## Not injected` 这个标题本身就说清了"有东西没进来"，
-        // 再补一句 `(Truncated: …)` 只是把同一件事说两遍。标记只在清单**自己也不全**（有没列出的
-        // 条目）时才有信息量 —— 注意判据是 `rest > 0`，不是"折叠行放得下"：折叠行放不下时清单
-        // 同样是残缺的，标记不能跟着一起消失（那会让残缺的清单看起来是全部）。
+        // 标记与折叠行的关系（2026-10-18 审查收口）：
+        //   - 清单列全（rest === 0）→ 不发标记也不发折叠行：`## Not injected` 标题已说清。
+        //   - 清单不全 + 折叠行放得下 → 只发折叠行（给了具体计数），不发标记（重复）。
+        //   - 清单不全 + 折叠行放不下 → 只发标记（折叠行是更具体的版本，但放不下时标记是唯一信号）。
         const folded = rest > 0 && used + byteLen(fold) <= space;
-        const tail = `${head}${lines.join('')}${folded ? fold : ''}${rest > 0 ? marker : ''}`;
+        const tail = `${head}${lines.join('')}${folded ? fold : ''}${rest > 0 && !folded ? marker : ''}`;
         return tail;
     };
     // ── ① 贪心：放得下就放，越界**跳过**（而不是整体停止）──────────────────
@@ -455,10 +455,18 @@ export function renderSceneMemory(probe, index, maxBytes) {
         tail = renderTail(missed, maxBytes - byteLen(body), truncatedNow);
     }
     let text = `${body}${tail}`.replace(/^\n+/, '');
-    // ③ 兜底：预算小到连标记都放不下时也宁可超出几个字节——"有记忆没注入"这件事
-    // 绝不能静默消失（原来单条超预算会让整段变成空串，模型端完全无痕）。
-    if (text === '' && (missed.length > 0 || probe.truncated))
-        text = `${TRUNCATION_MARKER}\n`;
+    // ③ 兜底：有记忆没注入时绝不能静默消失。
+    //   - 原来（2026-09-? 初版）只兜 text === '' 的情形：body 非空、tail 放不下时，
+    //     模型看到的是一段看起来完整的记忆段 —— 既没有 `## Not injected` 也没有标记，
+    //     被截掉的那些条目就**完全无痕**了。这是 2026-10-18 审查发现的 P1：body 的字节
+    //     核算（`used`）与 `byteLen(body)` 有几字节的误差（渲染时条目间的 `\n` / `\n\n`
+    //     没全进 `used`），于是 reserve 留的余量偶尔不够，标记放不下，3 兜底又不触发。
+    //   - 现在的判据：只要 missed 非空或发现阶段截断，而 text 里既没有标记也没有
+    //     `## Not injected` 标题，就强制追加标记 —— 宁可超预算几字节（原 ③ 注释的原则），
+    //     也不能让截断无痕。两样有一样就不追加（信号已经在）。
+    if ((missed.length > 0 || probe.truncated) && !text.includes(TRUNCATION_MARKER) && !text.includes(DROPPED_HEADING)) {
+        text = text === '' ? `${TRUNCATION_MARKER}\n` : `${text.replace(/\n+$/, '')}\n${TRUNCATION_MARKER}\n`;
+    }
     const scenes = [];
     for (const c of selected)
         if (!scenes.includes(c.scene))

package/lib/op-registry.js

@@ -243,6 +243,9 @@ export const OP_REGISTRY = Object.freeze({
     // 客户端 boot 时必问一次决定挂不挂 `/` 菜单，那一次不能要令牌，否则没配令牌的宿主上这个功能
     // 永远出不来。
     'slash-settings': { write: true, writeWhen: (a) => a.set === true },
+    // 侧栏入口的开关（侧车 `sidebar-settings.json`）：同 `slash-settings` —— 写侧车按写门禁、
+    // 不冻结；读侧是纯读（客户端 boot 时必问一次，那一次不能要令牌）。
+    'sidebar-settings': { write: true, writeWhen: (a) => a.set === true },
     // 清理 patch 备份：删磁盘文件（备份里含明文凭据副本），按写操作门禁。
     'backups-clean': { write: true },
     // ── 场景候选源（4）与内联（1）──────────────────────────────────────────────

package/lib/ops/trash.js

@@ -269,7 +269,18 @@ export function buildTrashOps(rc) {
         catch {
             return fail('error.rules.notFound', `回收站条目不存在：${id}`);
         }
-        return { ok: true, id, domain, files: await readTrashPayloadFiles(dir) };
+        // 清单也一并回给界面（用户 2026-10-18：「回收站的详细功能做的不够好，显示的内容不够全，
+        // 例如场景没有显示勾选了什么」）：场景的档案、记忆的场景 / 形态都在 `manifest.json` 里，
+        // 只回文件列表的话，详情页能看到的就只剩"有几个文件"。
+        // 读不到 / 坏了就回 null —— 详情页按"只有文件"那一档渲染，不因为清单缺了就整页报错。
+        let manifest = null;
+        try {
+            manifest = JSON.parse(await readFile(join(dir, 'manifest.json'), 'utf8'));
+        }
+        catch {
+            manifest = null;
+        }
+        return { ok: true, id, domain, manifest, files: await readTrashPayloadFiles(dir) };
     }
     return { rulesRemove, rulesRestore, rulesTrashList, rulesTrashRemove, trashPreview };
 }

package/lib/settings-sidecars.js

@@ -10,6 +10,7 @@ import { hubPath } from './hub.js';
 import { DEFAULT_INJECT_SETTINGS, normalizeInjectSettings } from './context-inject.js';
 import { normalizeSceneSettings } from './scene-settings.js';
 import { normalizeSlashSettings } from './slash-settings.js';
+import { normalizeSidebarSettings } from './sidebar-settings.js';
 export function createSettingsSidecars(deps) {
     // 注入设置（侧车 `inject-settings.json`，界面在「兼容」页）：
     //   - underSuppressingPresets：压制型预设（persona complete / 关运行时上下文，如极简）
@@ -131,6 +132,40 @@ export function createSettingsSidecars(deps) {
             return { ok: true, settings: next };
         });
     }
+    // 侧栏入口的开关（侧车 `sidebar-settings.json`）：只一项 —— 侧栏里挂不挂本插件那一行。
+    // 与 `slash-settings` 同规格：入口开关，不冻结；读侧不带 `set` 是纯读（客户端 boot 时
+    // 必问一次决定注不注册，那一次不能要令牌，否则没配令牌的宿主上这个入口永远出不来）。
+    const SIDEBAR_SETTINGS_FILE = 'sidebar-settings.json';
+    const SIDEBAR_SETTINGS_TTL_MS = 3000;
+    let sidebarSettingsCache = null;
+    async function readSidebarSettings(force = false) {
+        if (sidebarSettingsCache && !force && Date.now() - sidebarSettingsCache.at < SIDEBAR_SETTINGS_TTL_MS)
+            return sidebarSettingsCache.value;
+        await deps.ensurePaths();
+        const raw = await deps.readJsonFile(hubPath(SIDEBAR_SETTINGS_FILE));
+        const value = normalizeSidebarSettings(raw);
+        sidebarSettingsCache = { at: Date.now(), value };
+        return value;
+    }
+    async function sidebarSettingsOp(args) {
+        const current = await readSidebarSettings();
+        if (!args || args.set !== true)
+            return { ok: true, settings: current };
+        const next = normalizeSidebarSettings({
+            enabled: typeof args.enabled === 'boolean' ? args.enabled : current.enabled,
+        });
+        await deps.ensurePaths();
+        return deps.withWriteLock(async () => {
+            try {
+                await deps.writeJsonFile(hubPath(SIDEBAR_SETTINGS_FILE), next);
+            }
+            catch (e) {
+                return { ok: false, error: '设置保存失败: ' + deps.message(e) };
+            }
+            sidebarSettingsCache = { at: Date.now(), value: next };
+            return { ok: true, settings: next };
+        });
+    }
     return {
         readInjectSettings,
         injectSettingsSync,
@@ -140,5 +175,7 @@ export function createSettingsSidecars(deps) {
         sceneSettingsOp,
         readSlashSettings,
         slashSettingsOp,
+        readSidebarSettings,
+        sidebarSettingsOp,
     };
 }

package/lib/sidebar-settings.js

@@ -0,0 +1,13 @@
+// 侧栏入口的开关（侧车 `sidebar-settings.json`，按钮在「兼容」页页头）。
+//
+// 与 `slash-settings.ts` 同一档：它是个**入口开关**，不是管理动作 —— 所以不进场景冻结
+// （锁着场景的人照样能把它关掉），也不参与任何门禁判定。
+//
+// 为什么存服务端而不是 localStorage：与其余入口开关同一个家，换浏览器也在；存本地就
+// 只有那个浏览器认，而这条开关的用途正是"这台机器上我不想在侧栏看到它"。
+export const DEFAULT_SIDEBAR_SETTINGS = { enabled: true };
+/** 把任意输入夹成合法设置：读不到 / 形状不对一律按默认开启。 */
+export function normalizeSidebarSettings(raw) {
+    const obj = (raw && typeof raw === 'object' ? raw : {});
+    return { enabled: typeof obj.enabled === 'boolean' ? obj.enabled : DEFAULT_SIDEBAR_SETTINGS.enabled };
+}

package/locale/en.json

@@ -0,0 +1,6 @@
+{
+  "meta": {
+    "title": "Tools Management",
+    "description": "One settings panel for MCP, skills, memories, subagents, prompts and archived sessions; compose scenes to switch the whole environment in one move, and the agent can drive this plugin for you."
+  }
+}

package/locale/zh.json

@@ -0,0 +1,6 @@
+{
+  "meta": {
+    "title": "工具管理",
+    "description": "一个设置面板，统一管理 MCP、技能、记忆、子智能体、提示词和归档会话；可自定义搭配场景，一个场景即可切换整个环境；Agent也能帮你操作这个插件。"
+  }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-plugin-tool-management",
-  "version": "0.21.0",
+  "version": "0.21.1",
   "description": "一个设置面板，统一管理 MCP、技能、记忆、子智能体、提示词和归档会话；可自定义搭配场景，一个场景即可切换整个环境；Agent也能帮你操作这个插件。",
   "repository": {
     "type": "git",
@@ -12,10 +12,12 @@
     ".": "./lib/index.js",
     "./client": "./lib/client.js",
     "./workspace": "./lib/sessions/workspace.js",
+    "./locale/*.json": "./locale/*.json",
     "./package.json": "./package.json"
   },
   "files": [
     "lib",
+    "locale",
     "cordis.patch.yml",
     "README.md",
     "README_EN.md",
@@ -67,6 +69,7 @@
       ]
     }
   },
+  "icon": "./docs/images/icon.png",
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.2",
     "@deepseek-ai/dsh-agent-preset-registry": ">=0.1.7-rc.2 <0.2.0-rc.1 || >=0.2.0-rc.1 <0.2.1-rc.2 || >=0.2.1-rc.2 <0.2.2",