dsh-oh-my-terminal 0.2.2 → 0.2.3

package/lib/index.js

@@ -1014,9 +1014,16 @@ function createSettingsBridgeRoutes(settings, namespace, runtimeProfiles) {
 }
 
 // src/settings/index.ts
-function registerSettingsIntegration(ctx, runtimeProfiles) {
+function registerSettingsIntegration(ctx, schema, runtimeProfiles) {
   ctx.inject(["settings"], (sctx) => {
     sctx.effect(() => sctx.settings.configure({ auto: false }, ctx.fiber));
+    const registrar = sctx.settings;
+    try {
+      registrar.register(SETTINGS_NS, schema);
+    } catch (error) {
+      const message = error instanceof Error ? error.message : String(error);
+      if (!message.includes("already registered")) throw error;
+    }
   });
   ctx.inject(["webServer", "settings"], (sctx) => {
     sctx.effect(() => {
@@ -1277,7 +1284,7 @@ function apply(ctx, config) {
       upgradeDisposers
     })
   });
-  registerSettingsIntegration(ctx, cachedProfiles);
+  registerSettingsIntegration(ctx, Config, cachedProfiles);
   ctx.effect(() => {
     return () => {
       disposeRoute();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-oh-my-terminal",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "description": "DSH Web GUI 底部终端面板插件——node-pty 驱动的多标签交互式终端（Windows ConPTY / POSIX openpty）",
   "type": "module",
   "license": "Apache-2.0",

package/src/index.ts

@@ -521,9 +521,10 @@ export function apply(ctx: Context, config: Config): void {
   });
 
   // —— settings 集成（可选，软探测）——
-  // 传入 cachedProfiles：让 describe 端点返回运行时合并探测结果后的完整配置表，
-  // 而非配置系统持久化值（可能是空串），设置页面才能看到自动探测到的终端项
-  registerSettingsIntegration(ctx, cachedProfiles);
+  // 传入 Config schema：auto=false 下命名空间需显式注册（见 settings/index.ts），
+  // 缺注册则设置卡片静默不渲染；cachedProfiles 让 describe 端点返回运行时合并
+  // 探测结果后的完整配置表，而非配置系统持久化值（可能是空串）
+  registerSettingsIntegration(ctx, Config, cachedProfiles);
 
   // —— 插件销毁清理 ——
   ctx.effect(() => {

package/src/settings/index.ts

@@ -37,14 +37,18 @@ export { createSettingsBridgeRoutes } from './bridge.js';
  * 逐一调用清理。
  *
  * @param ctx - cordis 上下文（插件主入口的 apply 参数）
+ * @param schema - 插件 Config 的 schemastery schema，用于注册 settings 命名空间。
+ *   auto=false 下服务不会自动注册，缺了这一步 describe/mutate 恒报「命名空间
+ *   不存在」，前端配置卡片静默不渲染（无报错、无入口）
  * @param runtimeProfiles - 运行时终端配置表（探测+合并后的完整数组），传给 bridge
  *   供 describe 端点覆盖持久化的 terminalProfiles 值
  */
 export function registerSettingsIntegration(
   ctx: Context,
+  schema: unknown,
   runtimeProfiles?: unknown[],
 ): void {
-  // 1. 配置 settings 服务为非自动模式——软探测，服务不存在时不激活
+  // 1. 注册 settings 命名空间 + 关闭自动模式——软探测，服务不存在时不激活
   ctx.inject(['settings'], (sctx) => {
     // settings.configure({ auto: false }, fiber) 关闭自动模式：
     // - auto=true：settings 服务自动从 profile patch 读配置并写回插件 Config
@@ -53,6 +57,23 @@ export function registerSettingsIntegration(
     //
     // fiber 参数绑定配置生命周期到当前 effect——effect 销毁时配置自动清理
     sctx.effect(() => sctx.settings.configure({ auto: false }, ctx.fiber));
+
+    // auto=false 同时关闭了命名空间的自动注册——必须在此显式 register，否则
+    // describe/mutate 恒报「命名空间不存在」，前端配置卡片静默不渲染。重复注册
+    // （热重载、重复 apply）抛 "already registered"，作为幂等情形吞掉；其余注册
+    // 失败如实上抛，不把失败包装成成功。
+    /** settings 服务注册面的最小鸭子类型（对齐官方 SettingsProvider.register） */
+    interface SettingsRegistrar {
+      /** 注册命名空间 schema；重复注册抛 "already registered" */
+      register(ns: string, schema: unknown): unknown;
+    }
+    const registrar = sctx.settings as unknown as SettingsRegistrar;
+    try {
+      registrar.register(SETTINGS_NS, schema);
+    } catch (error) {
+      const message = error instanceof Error ? error.message : String(error);
+      if (!message.includes('already registered')) throw error;
+    }
   });
 
   // 2. 注册 settings bridge 路由（describe + mutate）——软探测，服务不存在时不激活

package/tests/unit/settings-integration.test.ts

@@ -0,0 +1,99 @@
+/**
+ * @file settings 集成注册单元测试
+ * @description 回归锁定 registerSettingsIntegration 的命名空间注册契约：
+ *              auto=false 下 settings 服务不会自动注册命名空间，插件必须显式
+ *              register(SETTINGS_NS, schema)，否则 describe/mutate 恒报
+ *              「命名空间不存在」——前端配置卡片静默不渲染（无报错、无入口）。
+ *              本测试用假 settings 服务断言：注册确实发生且 schema 原样透传、
+ *              重复注册幂等吞掉、其余注册失败如实上抛（不把失败包装成成功）。
+ */
+
+import { describe, it } from 'node:test';
+import assert from 'node:assert/strict';
+import type { Context } from '@deepseek-ai/cordis';
+import { registerSettingsIntegration, SETTINGS_NS } from '../../src/settings/index.js';
+
+/** register 调用记录 */
+interface RegisterCall {
+  /** 被注册的命名空间 */
+  ns: string;
+  /** 传入的 schema 引用 */
+  schema: unknown;
+}
+
+/** 假 settings 服务：只实现集成代码用到的 configure / register / describe / mutate */
+function makeFakeSettings(options: { failRegisterWith?: Error } = {}): {
+  settings: Record<string, unknown>;
+  registerCalls: RegisterCall[];
+  configureCalls: unknown[];
+} {
+  const registerCalls: RegisterCall[] = [];
+  const configureCalls: unknown[] = [];
+  const settings = {
+    configure(value: unknown): void {
+      configureCalls.push(value);
+    },
+    register(ns: string, schema: unknown): unknown {
+      if (options.failRegisterWith !== undefined) throw options.failRegisterWith;
+      registerCalls.push({ ns, schema });
+      return {
+        get: () => ({}),
+        watch: () => (): void => undefined,
+        update: async (): Promise<void> => undefined,
+        replace: async (): Promise<void> => undefined,
+      };
+    },
+    describe: async (): Promise<unknown> => ({ ok: true }),
+    mutate: async (): Promise<unknown> => ({ ok: true }),
+  };
+  return { settings, registerCalls, configureCalls };
+}
+
+/** 假 cordis 上下文：inject 直接回调，effect 立即执行并回收 cleanup */
+function makeFakeContext(settings: unknown): Context {
+  const ctx = {
+    fiber: {},
+    inject(_deps: string[], callback: (sctx: Record<string, unknown>) => void): void {
+      callback({
+        settings,
+        webServer: { register: () => (): void => undefined },
+        effect: (fn: () => unknown): void => {
+          const returned = fn();
+          if (typeof returned === 'function') returned();
+        },
+      });
+    },
+  };
+  return ctx as unknown as Context;
+}
+
+describe('settings 集成注册', () => {
+  it('settings 服务就位时显式注册命名空间并透传 schema', () => {
+    const { settings, registerCalls, configureCalls } = makeFakeSettings();
+    const schema = { marker: 'config-schema' };
+    registerSettingsIntegration(makeFakeContext(settings), schema, []);
+    assert.equal(registerCalls.length, 1);
+    assert.equal(registerCalls[0]?.ns, SETTINGS_NS);
+    assert.equal(registerCalls[0]?.schema, schema);
+    assert.deepEqual(configureCalls, [{ auto: false }]);
+  });
+
+  it('重复注册视为幂等情形，吞掉 already registered 不上抛', () => {
+    const { settings } = makeFakeSettings({
+      failRegisterWith: new Error('settings namespace "terminal-panel" is already registered'),
+    });
+    assert.doesNotThrow(() => {
+      registerSettingsIntegration(makeFakeContext(settings), { marker: 'x' });
+    });
+  });
+
+  it('其他注册失败如实上抛，不把失败包装成成功', () => {
+    const { settings } = makeFakeSettings({
+      failRegisterWith: new Error('settings namespace "terminal-panel" 存储段损坏'),
+    });
+    assert.throws(
+      () => registerSettingsIntegration(makeFakeContext(settings), { marker: 'x' }),
+      /存储段损坏/u,
+    );
+  });
+});