dsh-plugin-t-expert 0.2.9 → 0.2.12

package/lib/command.js

@@ -1,3 +1,4 @@
+// @ts-check
 /**
  * `/t` —— T专家 的小队短命令。
  *
@@ -161,14 +162,26 @@ export function renderTeamList(profiles) {
 
 /**
  * 注册 `/t`。
- * @param ctx 宿主上下文
- * @param options.teamsFile teams.json 路径（通常与名册同目录）
- * @param options.hasEngine 探测内置团队引擎是否在位的函数
+ * @param ctx 宿主上下文（需要 `effect` / `logger` 等完整能力）
+ * @param {object} options 注册选项
+ * @param {string} options.teamsFile teams.json 路径（通常与名册同目录）
+ * @param {() => boolean} options.hasEngine 探测内置团队引擎是否在位的函数
+ * @param {() => string} [options.engineError] 引擎不可用时的原因（报给用户）
+ * @param {object} [options.commands] 已解析好的 commands 服务（可选服务的时序修复，见下）
  */
 export function registerTeamCommand(ctx, options) {
   const teamsFile = options.teamsFile;
   const hasEngine = options.hasEngine;
-  ctx.effect(() => ctx.commands.register({
+  // ⚠️ 2026-09-13：`commands` 是**可选服务**，真实宿主里它**晚于本插件 apply** 才就绪；
+  // 而 `ctx.get("commands")` 在那一刻是 undefined，且 cordis 的 `provide` **不回溯通知**已跑过的代码
+  // —— 于是 `/t` 永远不会注册。调用方已用 `ctx.inject(["commands"], …)` 等到就绪后把服务传进来；
+  // 这里再兜底懒查一次（直连调用/测试），两条路都不做属性访问。
+  const commands = options.commands ?? ctx.get?.("commands");
+  if (typeof commands?.register !== "function") {
+    ctx.logger?.warn?.("[t-team] 没有可用的 commands 服务：/t 命令不注册。");
+    return;
+  }
+  ctx.effect(() => commands.register({
     name: COMMAND_NAME,
     description: "用 T专家 小队执行一个目标（底层走 T专家 内置的多智能体团队引擎）",
     input: { hint: "<小队名|别名> <目标>" },
@@ -228,4 +241,4 @@ export function registerTeamCommand(ctx, options) {
       };
     },
   }), "t-team: /t command");
-}
+}

package/lib/i18n.js

@@ -1,3 +1,4 @@
+// @ts-check
 /** T专家 host 侧文案与列表/结果渲染。 */
 
 export const NS = "t-team";
@@ -20,6 +21,7 @@ const ZH = {
   "error.specsTooMany": "一次最多召唤 {limit} 位专家",
   "error.specInvalid": "experts[{index}] 需要 expert 与 task 两个字段",
   "error.settingsMissing": "T专家 设置段尚未注册",
+  "error.settingsServiceMissing": "宿主没有 settings 服务：T专家 的「专家启停」不可用（名册与召唤相关能力需要它记录已启用专家）。这是宿主装配问题，不是配置错误。",
   "error.customUnknownSlug": "未知专家 slug：{slugs}（这些 slug 不在当前名册里；先用 list_t_experts 取准确的 slug）",
   "error.customSlugInvalid": "slug 只能用 a-z、0-9 与连字符，且以字母或数字开头：\"{slug}\"",
   "error.customSlugTaken": "slug「{slug}」已被内置或自建专家占用",
@@ -66,6 +68,7 @@ const EN = {
   "error.specsTooMany": "At most {limit} experts per call",
   "error.specInvalid": "experts[{index}] requires both expert and task",
   "error.settingsMissing": "T Expert settings section is not registered yet",
+  "error.settingsServiceMissing": "The host has no settings service: T Expert's enable/disable state is unavailable (the roster and summoning need it to remember which experts are enabled). This is a host wiring problem, not a configuration error.",
   "error.customUnknownSlug": "Unknown expert slug(s): {slugs} (not in the current roster; call list_t_experts for exact slugs)",
   "error.customSlugInvalid": "slug may only use a-z, 0-9 and hyphens, starting with a letter or digit: \"{slug}\"",
   "error.categoryOfficial": "\"{division}\" is an official division: its name and existence follow the built-in roster sync, so only custom divisions can be changed",
@@ -104,10 +107,23 @@ export function t(locale, key, params = {}) {
   return text;
 }
 
-/** 从宿主 locale 设置读语言，缺失或异常回退 zh。 */
+/**
+ * 从宿主 locale 设置读语言，缺失或异常回退 zh。
+ *
+ * ⚠️ `settings` 是**可选服务**，必须用 `ctx.get("settings")` 懒查，不要写成 `ctx.settings`
+ * 属性访问：它不在本插件的静态 `inject` 里，而宿主**没提供**这个服务时，属性访问会被
+ * cordis 抛 `cannot get property "settings" without inject`，只剩 try/catch 兜底、
+ * 静默退中文（2026-09-13 用真 cordis 4.0.2 实测：宿主**有**这个服务时属性访问读得到，
+ * 所以这类 bug 只在"服务不全"的宿主上现形，正常机器上不会露头）。
+ * 与 lib/index.js 里 workspaceRegistry / agents 的取值方式保持一致。
+ *
+ * 这里是**执行期**读取（工具调用那一刻），不是 apply 期探测，所以 `get` 是正确用法：
+ * 那一刻 settings 若存在一定已经就绪；不存在就退中文，不会挂起本插件。
+ */
 export function readLocale(ctx) {
   try {
-    const section = ctx.settings?.get?.("locale");
+    const settings = typeof ctx?.get === "function" ? ctx.get("settings") : undefined;
+    const section = settings?.get?.("locale");
     const preference = section?.preference;
     if (typeof preference === "string" && preference.toLowerCase().startsWith("en")) return "en";
   } catch {