dsh-plugin-t-expert 0.2.7 → 0.2.10

package/lib/command.js

@@ -1,3 +1,4 @@
+// @ts-check
 /**
  * `/t` —— T专家 的小队短命令。
  *
@@ -161,22 +162,37 @@ 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: "<小队名|别名> <目标>" },
     handler(invocation) {
       if (hasEngine() !== true) {
         const detail = typeof options.engineError === "function" ? options.engineError() : "";
+        // 中性表述：不指路日志。旧文案让用户去看 `[t-team]` error 行，但「引擎已调度、工具却没注册」
+        // 这条路径下那句话**永远不会打印**（lib/index.js 的 engineReady 探到假只会补 warn），
+        // 于是指引恒定失效（N-5）。
         const text = detail === ""
-          ? "团队引擎未就绪：T专家 内置的团队引擎没有挂载成功（查看 DSH 日志里的 [t-team] 行）。"
+          ? "团队引擎未就绪：T专家 内置的团队引擎没有注册成功（它的工具没有出现在工具表里）。"
           : `团队引擎挂载失败：${detail}`;
         // 命令结果行在部分 DSH 版本不渲染，所以连错误也交给队长转述一次。
         invocation.agent.followup(createUserMessage({
@@ -225,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,8 @@ 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}」已被内置或自建专家占用",
   "error.customDivisionInvalid": "分类目录名只能用 a-z、0-9、连字符与点，且以字母或数字开头（中文请填在「显示名」里）：\"{division}\"",
@@ -65,6 +68,8 @@ 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",
   "error.categoryExists": "Division \"{division}\" already exists",
@@ -128,8 +133,49 @@ export function localized(expert, locale) {
   };
 }
 
-/** list_t_experts 的文本渲染。 */
-export function renderList(locale, { query, groups, total, allDivisions }) {
+/**
+ * 把执行期名册分组投影成渲染期就绪的形状。
+ *
+ * 为什么在这里做：`output.render(args, value)` 必须是 `(args, value)` 的纯函数
+ * ——同一个 canonical value 在会话日志里回放时必须渲染出同一段文本。而「用哪种语言」
+ * 是**执行期**才知道的事实（读宿主 locale 设置），所以它必须和分组一起进 canonical value，
+ * 不能在 render 里再读一次宿主。
+ *
+ * @param groups - `groupByDivision()` 的分组（带 label/labelEn 与 expert 条目）。
+ * @param locale - 执行期语言（"zh" | "en"）。
+ * @returns 只含渲染所需字段的分组数组。
+ */
+export function toRenderableGroups(groups, locale) {
+  return (Array.isArray(groups) ? groups : []).map((group) => ({
+    division: group.division,
+    label: locale === "en" ? (group.labelEn ?? group.label ?? group.division) : (group.label ?? group.division),
+    count: group.count ?? (group.experts ?? []).length,
+    experts: (group.experts ?? []).map((expert) => {
+      const { name, description } = localized(expert, locale);
+      return {
+        slug: expert.slug,
+        emoji: expert.emoji ?? "",
+        // 渲染期不再读侧车译文：名字与简介在执行期就按 locale 定好。
+        name,
+        description,
+      };
+    }),
+  }));
+}
+
+/**
+ * list_t_experts 的文本渲染（纯函数）。
+ *
+ * 语言只从 `value.locale` 取，不读宿主设置：同一个 `value` 永远渲染出同一段文本。
+ * @param args - 工具入参（只用来取 division 过滤词）。
+ * @param value - `list_t_experts.execute` 返回的 canonical value。
+ */
+export function renderList(_args, value) {
+  const query = typeof value.query === "string" ? value.query : "";
+  const locale = value.locale === "en" ? "en" : "zh";
+  const groups = Array.isArray(value.divisions) ? value.divisions : [];
+  const total = typeof value.total === "number" ? value.total : 0;
+  const allDivisions = Array.isArray(value.allDivisions) ? value.allDivisions : [];
   if (query !== "" && groups.length === 0) {
     return t(locale, "list.unknownDivision", { query, divisions: allDivisions.join(", ") });
   }
@@ -146,23 +192,44 @@ export function renderList(locale, { query, groups, total, allDivisions }) {
   } else {
     const group = groups[0];
     lines.push(t(locale, "list.headerOne", { division: `${group.label} (${group.division})`, count: group.count }));
-    for (const expert of group.experts) {
-      const { name, description } = localized(expert, locale);
+    for (const expert of group.experts ?? []) {
       // 带上 slug：模型可以直接拿它当召唤参数，避免只靠名字解析
-      lines.push(`- ${expert.emoji} ${name} (${expert.slug}) — ${description}`);
+      lines.push(`- ${expert.emoji} ${expert.name} (${expert.slug}) — ${expert.description}`);
     }
   }
   return lines.join("\n");
 }
 
-/** summon_experts 的文本渲染。 */
-export function renderSummonResults(locale, results) {
-  const ok = results.filter((r) => r.ok).length;
+/**
+ * 把一批召唤结果投影成渲染期就绪的形状。
+ * @param results - `summon_t_experts` 的结果项；`expert` 已是**执行期按 locale 定好的显示名**
+ *   （`summon_t_experts.execute` 负责解析，render 不再碰名册）。
+ * @param locale - 执行期语言（写进 value，供 render 取文案）。
+ */
+export function toRenderableSummonResults(results, locale) {
+  return (Array.isArray(results) ? results : []).map((item) => ({
+    expert: typeof item?.expert === "string" ? item.expert : "",
+    ok: item?.ok === true,
+    answer: item?.answer ?? "",
+    error: item?.error ?? "",
+  }));
+}
+
+/**
+ * summon_t_experts 的文本渲染（纯函数）。
+ * 语言只从 `value.locale` 取，专家名来自 `value.results[].expert`（执行期已本地化）。
+ * @param args - 工具入参（未使用）。
+ * @param value - `summon_t_experts.execute` 返回的 canonical value。
+ */
+export function renderSummonResults(_args, value) {
+  const locale = value.locale === "en" ? "en" : "zh";
+  const results = Array.isArray(value.results) ? value.results : [];
+  const ok = results.filter((item) => item.ok === true).length;
   const lines = [t(locale, "summon.batchHeader", { total: results.length, ok, failed: results.length - ok })];
   for (const item of results) {
-    lines.push(item.ok
-      ? t(locale, "summon.itemOk", { expert: item.expert, answer: item.answer })
-      : t(locale, "summon.itemFail", { expert: item.expert, error: item.error ?? "" }));
+    lines.push(item.ok === true
+      ? t(locale, "summon.itemOk", { expert: item.expert ?? "", answer: item.answer ?? "" })
+      : t(locale, "summon.itemFail", { expert: item.expert ?? "", error: item.error ?? "" }));
   }
   return lines.join("\n\n");
 }