pi-nested-skills 0.4.1 → 0.5.0

package/README.md

@@ -11,7 +11,7 @@ A Pi extension that discovers skills recursively and provides convenient aliases
 - Keeps Pi's native skill loader responsible for frontmatter validation and skill-body expansion; this package does not duplicate `read` compatibility or skill expansion.
 - Extends the native slash-command completion list and preserves built-in Pi command suggestions.
 - Provides `/skills` to list discovered aliases.
-- Provides `/config:nested-skills` to edit the skill roots through the normal TUI input; use `reset` for the default `skills` root.
+- Provides `/config:nested-skills` to edit the skill roots in a TUI settings panel; use `reset` for the default `skills` root.
 - Contributes discovered skill files through Pi's `resources_discover` event.
 - Uses `pi-extensions-i18n` for all user-visible messages.
 
@@ -44,7 +44,7 @@ Start from [`config.example.json`](./config.example.json) and write it to:
 
 A relative root is resolved relative to the Pi agent directory. Each direct child of a root is treated as a skill package, and every visible descendant containing `SKILL.md` is discovered. A root containing `SKILL.md` itself is also accepted as one package.
 
-Use `/config:nested-skills` to change the roots in the normal TUI input. Enter comma-separated paths, or use `reset` for the default root. Run `/reload` after saving.
+Use `/config:nested-skills` to change the roots in a TUI settings panel. Changes are saved and applied immediately, so the skill index is rebuilt without `/reload`. Press Enter on the roots row to edit the comma-separated list, or run `reset` for the default root.
 
 ## Aliases and native expansion
 

package/README.zh-CN.md

@@ -11,7 +11,7 @@ Pi 的嵌套技能扩展：递归发现多级 `SKILL.md`，并为嵌套技能提
 - 保留 Pi 原生技能加载器负责 frontmatter 校验和正文展开；不重复实现 `read` 兼容层或技能展开。
 - 将嵌套技能补全合并到 Pi 原生斜杠命令补全中，不影响内置命令。
 - 提供 `/skills` 列出发现到的全部别名。
-- 提供 `/config:nested-skills` 通过常规 TUI 输入配置技能根目录；输入 `reset` 恢复默认的 `skills` 根目录。
+- 提供 `/config:nested-skills` 通过 TUI 配置面板配置技能根目录；执行 `reset` 恢复默认的 `skills` 根目录。
 - 通过 Pi 的 `resources_discover` 事件注册发现到的技能文件。
 - 所有用户可见文案都通过 `pi-extensions-i18n` 提供中英文版本。
 
@@ -44,7 +44,7 @@ pi install npm:pi-nested-skills
 
 相对路径以 Pi agent 目录为基准。根目录的每个直接子目录视为一个技能包，扩展会递归发现其中所有可见目录下的 `SKILL.md`。如果根目录本身包含 `SKILL.md`，也会把它作为一个技能包处理。
 
-使用 `/config:nested-skills` 在 TUI 输入框中修改配置，多个目录用逗号分隔；输入 `reset` 恢复默认目录。保存后执行 `/reload`。
+使用 `/config:nested-skills` 在 TUI 配置面板中修改技能根目录，改动即时保存并生效，会自动重建技能索引，不需要 `/reload`。在根目录那行回车编辑逗号分隔的列表，或执行 `reset` 恢复默认目录。
 
 ## 别名与原生展开
 

package/SKILL.md

@@ -21,7 +21,7 @@ extensions/pi-nested-skills/config.json
 
 优先复制包内 [`config.example.json`](./config.example.json)，再修改 `skillRoots`。路径可以是绝对路径、`~/` 路径，或相对于 Pi agent 目录的路径。
 
-也可以使用 `/config:nested-skills` 修改配置；命令会在 TUI 输入框中接收逗号分隔的目录，输入 `reset` 恢复默认值。保存后执行 `/reload`。
+也可以使用 `/config:nested-skills` 打开 TUI 配置面板修改配置；在技能根目录那行回车编辑逗号分隔的目录，执行 `reset` 恢复默认值。改动即时保存并生效，会自动重建技能索引，不需要 `/reload`。
 
 别名形式为 `/技能包:子目录.子目录`，例如 `/development:code-reviewer` 或 `/design:icons.favicon`。也支持 `/skill:技能包.子目录`；扩展会把它转换为 Pi 原生 `/skill:<name>`，由 Pi 负责技能正文展开。
 

package/locales/index.json

@@ -31,12 +31,8 @@
     "zh-CN": "配置嵌套技能根目录",
     "en-US": "Configure nested skill roots"
   },
-  "configCommandInput": {
-    "zh-CN": "输入技能根目录（多个目录用逗号分隔）",
-    "en-US": "Enter skill roots (separate multiple roots with commas)"
-  },
   "configCommandUsage": {
-    "zh-CN": "用法：/config:nested-skills（打开配置输入）或 /config:nested-skills reset（恢复默认值）",
+    "zh-CN": "用法：/config:nested-skills（打开配置面板）或 /config:nested-skills reset（恢复默认值）",
     "en-US": "Usage: /config:nested-skills (open settings) or /config:nested-skills reset (restore defaults)"
   },
   "configCommandInteractiveOnly": {
@@ -44,11 +40,35 @@
     "en-US": "Nested skill configuration requires the TUI; run this command in an interactive Pi session."
   },
   "configCommandSaved": {
-    "zh-CN": "嵌套技能配置已保存：{path}。请执行 /reload 使配置生效。",
-    "en-US": "Nested skill configuration saved to {path}. Run /reload to apply it."
+    "zh-CN": "嵌套技能配置已保存：{path}。",
+    "en-US": "Nested skill configuration saved to {path}."
   },
   "configCommandInvalid": {
     "zh-CN": "嵌套技能配置无效：{error}",
     "en-US": "Invalid nested skill configuration: {error}"
+  },
+  "configSaveFailed": {
+    "zh-CN": "嵌套技能配置保存失败：{error}",
+    "en-US": "Failed to save nested skill configuration: {error}"
+  },
+  "configMenuTitle": {
+    "zh-CN": "嵌套技能配置",
+    "en-US": "Nested skill settings"
+  },
+  "configLabelSkillRoots": {
+    "zh-CN": "技能根目录",
+    "en-US": "Skill roots"
+  },
+  "configDescSkillRoots": {
+    "zh-CN": "逗号分隔的目录列表，递归扫描其中的 SKILL.md。",
+    "en-US": "Comma-separated directories scanned recursively for SKILL.md files."
+  },
+  "configInputPrompt": {
+    "zh-CN": "输入",
+    "en-US": "Input"
+  },
+  "configValueUnset": {
+    "zh-CN": "未设置",
+    "en-US": "unset"
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-nested-skills",
-  "version": "0.4.1",
+  "version": "0.5.0",
   "description": "Pi extension for recursive nested-skill discovery and aliases",
   "type": "module",
   "main": "./index.ts",
@@ -62,7 +62,7 @@
     "@earendil-works/pi-tui": ">=0.80.0"
   },
   "dependencies": {
-    "pi-extensions-i18n": "^0.7.0"
+    "pi-extensions-i18n": "^0.9.0"
   },
   "devDependencies": {
     "@earendil-works/pi-coding-agent": "0.85.1",

package/src/config-panel.ts

@@ -0,0 +1,180 @@
+/**
+ * pi-nested-skills 的 TUI 配置面板。
+ *
+ * 原来用一次 `ctx.ui.input` 让用户手输根目录，看不到当前值也看不出哪项能改。
+ * 这里改用 Pi 自带 `SettingsList`：左边字段名、右边当前值、选中项下方给一行说明；
+ * 技能根目录是字符串数组，面板上以逗号分隔显示为一行，回车打开预填输入框编辑整串，
+ * 提交后按逗号切分回数组。改一项立刻写盘并同步运行期配置，Esc 关闭即可生效。
+ */
+
+import {
+  getSettingsListTheme,
+  type ExtensionCommandContext,
+} from "@earendil-works/pi-coding-agent";
+import {
+  Container,
+  Input,
+  SettingsList,
+  Text,
+  type Component,
+  type SettingItem,
+} from "@earendil-works/pi-tui";
+import { parseConfig, type NestedSkillsConfig } from "./config.ts";
+import { i18n } from "./i18n.ts";
+
+/** 面板字段名，同时用作 `SettingsList` 的行 id。 */
+export const PANEL_FIELD = {
+  skillRoots: "skillRoots",
+} as const;
+
+/** 面板字段名的联合类型。 */
+export type PanelFieldId = (typeof PANEL_FIELD)[keyof typeof PANEL_FIELD];
+
+/** 面板覆盖的配置字段名，测试用它核对没有字段漏在面板外。 */
+export const PANEL_FIELD_IDS: readonly string[] = [PANEL_FIELD.skillRoots];
+
+/** 根目录数组在面板上的一行文本：逗号加空格分隔，与命令行输入保持一致。 */
+const ROOT_DISPLAY_SEPARATOR = ", ";
+
+/** 根目录数组在输入框里按逗号切分。 */
+const ROOT_INPUT_SEPARATOR = ",";
+
+/** 面板项：目前只有字符串数组一种取值方式。 */
+interface PanelFieldSpec {
+  /** 字段名。 */
+  id: PanelFieldId;
+  /** 字段标签的本地化 key。 */
+  labelKey: string;
+  /** 说明行的本地化 key。 */
+  descriptionKey: string;
+}
+
+/** 面板顺序。 */
+const PANEL_FIELDS: readonly PanelFieldSpec[] = [
+  { id: PANEL_FIELD.skillRoots, labelKey: "configLabelSkillRoots", descriptionKey: "configDescSkillRoots" },
+];
+
+/** 根目录数组转面板上的一行文本。 */
+export function rootsToText(roots: readonly string[]): string {
+  return roots.join(ROOT_DISPLAY_SEPARATOR);
+}
+
+/** 面板上的一行文本转根目录数组：按逗号切分、去空白、丢掉空项。 */
+export function textToRoots(text: string): string[] {
+  return text
+    .split(ROOT_INPUT_SEPARATOR)
+    .map((root) => root.trim())
+    .filter(Boolean);
+}
+
+/** 当前配置里某个字段的取值。 */
+function readField(config: NestedSkillsConfig, id: PanelFieldId): string {
+  switch (id) {
+    case PANEL_FIELD.skillRoots:
+      return rootsToText(config.skillRoots);
+  }
+}
+
+/** 自由文本二级菜单：输入框预填当前值，Enter 提交、Esc 取消。 */
+function createTextSubmenu(
+  value: string,
+  placeholder: string,
+  done: (value?: string) => void,
+): Component {
+  const input = new Input({ prompt: `${i18n.t("configInputPrompt")} `, placeholder });
+  input.setValue(value);
+  input.focused = true;
+  input.onSubmit = (submitted) => done(submitted);
+  input.onEscape = () => done(undefined);
+  return {
+    /** 一行可编辑输入。 */
+    render: (width) => input.render(width),
+    /** 输入框没有共享缓存。 */
+    invalidate: () => input.invalidate(),
+    /** 所有按键都归输入框。 */
+    handleInput: (data) => input.handleInput(data),
+  };
+}
+
+/** 把当前配置转成 Pi 设置列表的条目。 */
+export function buildSettingItems(config: NestedSkillsConfig): SettingItem[] {
+  return PANEL_FIELDS.map((field) => {
+    const label = i18n.t(field.labelKey);
+    const description = i18n.t(field.descriptionKey);
+    const stored = readField(config, field.id);
+    return {
+      id: field.id,
+      label,
+      description,
+      currentValue: stored || i18n.t("configValueUnset"),
+      // SettingsList 的 submenu 用 done 回传输入；这里编辑整串再切分回数组。
+      submenu: (_currentLabel, done) => createTextSubmenu(stored, description, done),
+    };
+  });
+}
+
+/**
+ * 把面板返回的文本写回配置；无法识别时返回 undefined，由调用方保持不变。
+ *
+ * 结果统一过一遍 parseConfig，写盘的值仍满足现有校验规则（只能是字符串数组），
+ * 面板与校验不会各走各的。空输入得到空数组，面板显示为「未设置」。
+ */
+export function applyPanelChange(
+  config: NestedSkillsConfig,
+  id: string,
+  value: string,
+): NestedSkillsConfig | undefined {
+  const field = PANEL_FIELDS.find((candidate) => candidate.id === id);
+  if (!field) return undefined;
+  try {
+    return parseConfig({ skillRoots: textToRoots(value) });
+  } catch {
+    // 输入格式非法时保持原值，面板不写入坏配置。
+    return undefined;
+  }
+}
+
+/** 面板与宿主之间的接口；配置读写与生效动作都由入口注入。 */
+export interface ConfigPanelHandlers {
+  /** 读取当前配置。 */
+  getConfig(): NestedSkillsConfig;
+  /** 某一项被改动后调用，由入口负责保存、同步运行期配置并重绘。 */
+  onChange(config: NestedSkillsConfig): void;
+}
+
+/** 打开配置面板；用户改动即时生效，Esc 关闭。 */
+export async function openConfigPanel(
+  ctx: ExtensionCommandContext,
+  handlers: ConfigPanelHandlers,
+): Promise<void> {
+  await ctx.ui.custom<void>((tui, theme, _keybindings, done) => {
+    const container = new Container();
+    container.addChild(new Text(theme.fg("accent", theme.bold(i18n.t("configMenuTitle"))), 1, 1));
+
+    const settingsList = new SettingsList(
+      buildSettingItems(handlers.getConfig()),
+      PANEL_FIELDS.length,
+      getSettingsListTheme(),
+      (id, newValue) => {
+        const next = applyPanelChange(handlers.getConfig(), id, newValue);
+        if (next) handlers.onChange(next);
+      },
+      () => done(undefined),
+    );
+    container.addChild(settingsList);
+
+    // ctx.ui.custom 要求返回 Component：整体布局交给 Container，键盘转给设置列表，
+    // 列表改了数据后必须 requestRender，否则画面停在上一帧。
+    return {
+      /** 面板整体按 Container 布局渲染。 */
+      render: (width) => container.render(width),
+      /** 无本地缓存，交给 Container 清理。 */
+      invalidate: () => container.invalidate(),
+      /** 键盘输入交给设置列表，并触发一次重绘。 */
+      handleInput: (data) => {
+        settingsList.handleInput(data);
+        tui.requestRender();
+      },
+    };
+  });
+}

package/src/i18n.ts

@@ -0,0 +1,3 @@
+import { createTranslator, loadCatalog } from "pi-extensions-i18n";
+
+export const i18n = createTranslator(loadCatalog(new URL("../locales/index.json", import.meta.url)));

package/src/index.ts

@@ -6,24 +6,22 @@ import type {
 import { fuzzyFilter } from "@earendil-works/pi-tui";
 import type { AutocompleteItem, AutocompleteProvider } from "@earendil-works/pi-tui";
 import {
-  createTranslator,
-  loadCatalog,
+  NOTICE_TAG_COLOR,
   installNoticeRenderer,
   notifyWithSource,
   type NoticeColor,
   type NoticeLevel,
   type NoticeSource,
 } from "pi-extensions-i18n";
-import { loadConfig, parseConfig, saveConfig, type LoadedNestedSkillsConfig } from "./config.ts";
+import { loadConfig, parseConfig, saveConfig, type LoadedNestedSkillsConfig, type NestedSkillsConfig } from "./config.ts";
+import { openConfigPanel } from "./config-panel.ts";
+import { i18n } from "./i18n.ts";
 import { scanSkillRoots, type NestedSkill, type SkillScanResult } from "./skills.ts";
 
-const messages = loadCatalog(new URL("../locales/index.json", import.meta.url));
-const i18n = createTranslator(messages);
-
 /** 本扩展的提示标签；短且唯一，便于在会话里定位来源。 */
 const NOTICE_TAG = "skills";
-/** 提示标签颜色；与其它扩展错开，避免看起来像同一条消息。 */
-const NOTICE_COLOR: NoticeColor = "success";
+/** 提示标签颜色：所有扩展统一用弱化色，来源靠 tag 文本区分，不靠颜色。 */
+const NOTICE_COLOR: NoticeColor = NOTICE_TAG_COLOR;
 /** 本扩展的提示来源。 */
 const NOTICE_SOURCE: NoticeSource = { tag: NOTICE_TAG, color: NOTICE_COLOR };
 
@@ -32,8 +30,6 @@ const COMMAND_ALIASES = [COMMAND_NAME] as const;
 const CONFIG_COMMAND_ALIASES = ["config:nested-skills", "nested-skills-config", "pi-nested-skills-config"] as const;
 const CONFIG_RESET_COMMAND = "reset";
 const DEFAULT_CONFIG_ROOT = "skills";
-const ROOT_INPUT_SEPARATOR = ",";
-const ROOT_DISPLAY_SEPARATOR = ", ";
 const NOTICE_WARNING: NoticeLevel = "warning";
 const NOTICE_INFO: NoticeLevel = "info";
 const NOTICE_ERROR: NoticeLevel = "error";
@@ -222,8 +218,45 @@ function formatSkillsMessage(index: SkillIndex): string {
   return lines.join("\n");
 }
 
-/** 注册配置命令，通过 TUI 输入逗号分隔的技能根目录。 */
-function registerConfigCommand(pi: ExtensionAPI): void {
+/**
+ * 运行期状态：配置和扫描索引都放在这里。
+ *
+ * 配置面板改完根目录后直接就地重建，让 resources_discover、输入转换、补全和 /skills
+ * 都看到新索引；否则用户改完配置看起来没反应，只能 /reload。
+ */
+interface RuntimeState {
+  loaded: LoadedNestedSkillsConfig;
+  index: SkillIndex;
+}
+
+/** 按根目录重新扫描并重建索引，同时刷新运行期配置。 */
+function rebuildIndex(state: RuntimeState, config: NestedSkillsConfig): void {
+  state.loaded = { config, source: "file", warnings: [], explicit: true };
+  state.index = buildSkillIndex(scanSkillRoots(config.skillRoots));
+}
+
+/** 保存一项配置并令其立即生效；写盘失败也要把新配置用在本次会话里。 */
+function applyConfig(state: RuntimeState, config: NestedSkillsConfig, ctx: ExtensionCommandContext): void {
+  rebuildIndex(state, config);
+  try {
+    saveConfig(config);
+  } catch (error) {
+    notify(ctx, i18n.t("configSaveFailed", {
+      error: error instanceof Error ? error.message : String(error),
+    }), NOTICE_ERROR);
+  }
+}
+
+/** 打开 TUI 配置面板；每改一项立即写盘并重建技能索引。 */
+async function openSkillsConfigPanel(state: RuntimeState, ctx: ExtensionCommandContext): Promise<void> {
+  await openConfigPanel(ctx, {
+    getConfig: () => state.loaded.config,
+    onChange: (config) => applyConfig(state, config, ctx),
+  });
+}
+
+/** 注册配置命令：无参数打开面板，`reset` 恢复默认值。 */
+function registerConfigCommand(pi: ExtensionAPI, state: RuntimeState): void {
   const command = {
     description: i18n.t("configCommandDescription"),
     getArgumentCompletions: () => [{ value: CONFIG_RESET_COMMAND, label: CONFIG_RESET_COMMAND }],
@@ -235,7 +268,9 @@ function registerConfigCommand(pi: ExtensionAPI): void {
       }
       if (argument === CONFIG_RESET_COMMAND) {
         try {
-          const path = saveConfig(parseConfig({ skillRoots: [DEFAULT_CONFIG_ROOT] }));
+          const config = parseConfig({ skillRoots: [DEFAULT_CONFIG_ROOT] });
+          const path = saveConfig(config);
+          rebuildIndex(state, loadConfig().config);
           notify(ctx, i18n.t("configCommandSaved", { path }), NOTICE_INFO);
         } catch (error) {
           notify(ctx, i18n.t("configCommandInvalid", {
@@ -248,33 +283,18 @@ function registerConfigCommand(pi: ExtensionAPI): void {
         notify(ctx, i18n.t("configCommandInteractiveOnly"), NOTICE_WARNING);
         return;
       }
-
-      const current = loadConfig().config;
-      const input = await ctx.ui.input(
-        i18n.t("configCommandInput"),
-        current.skillRoots.join(ROOT_DISPLAY_SEPARATOR),
-      );
-      if (input === undefined) return;
-      try {
-        const roots = input.split(ROOT_INPUT_SEPARATOR).map((root) => root.trim()).filter(Boolean);
-        const path = saveConfig(parseConfig({ skillRoots: roots }));
-        notify(ctx, i18n.t("configCommandSaved", { path }), NOTICE_INFO);
-      } catch (error) {
-        notify(ctx, i18n.t("configCommandInvalid", {
-          error: error instanceof Error ? error.message : String(error),
-        }), NOTICE_ERROR);
-      }
+      await openSkillsConfigPanel(state, ctx);
     },
   };
   for (const name of CONFIG_COMMAND_ALIASES) pi.registerCommand(name, command);
 }
 
-function registerSkillsCommand(pi: ExtensionAPI, index: SkillIndex): void {
+function registerSkillsCommand(pi: ExtensionAPI, state: RuntimeState): void {
   const command = {
     description: i18n.t("commandDescription"),
-    getArgumentCompletions: (): AutocompleteItem[] => commandItems(index),
+    getArgumentCompletions: (): AutocompleteItem[] => commandItems(state.index),
     handler: async (_args: string, _ctx: SkillCommandContext) => {
-      await pi.sendUserMessage(formatSkillsMessage(index), { deliverAs: "followUp" });
+      await pi.sendUserMessage(formatSkillsMessage(state.index), { deliverAs: "followUp" });
     },
   };
 
@@ -307,18 +327,20 @@ export default function nestedSkillsExtension(pi: ExtensionAPI): void {
   // 提示画成会话区里的带底色消息块；渲染器在本包这个模块实例里注册一次。
   installNoticeRenderer(pi);
   const loaded = loadConfig();
-  const scan = scanSkillRoots(loaded.config.skillRoots);
-  const index = buildSkillIndex(scan);
+  const state: RuntimeState = {
+    loaded,
+    index: buildSkillIndex(scanSkillRoots(loaded.config.skillRoots)),
+  };
 
   pi.on("resources_discover", () => ({
     // 每个 SKILL.md 单独交给原生 loader，绕过“父目录含 SKILL.md 后停止递归”的规则，
     // 同时保留 Pi 原生的 frontmatter 校验、正文展开和资源来源信息。
-    skillPaths: index.skillPaths,
+    skillPaths: state.index.skillPaths,
   }));
 
   pi.on("input", (event) => {
     if (!INPUT_SOURCES.has(event.source)) return { action: "continue" as const };
-    const transformed = transformSkillInput(event.text, index);
+    const transformed = transformSkillInput(event.text, state.index);
     if (!transformed) return { action: "continue" as const };
     return {
       action: "transform" as const,
@@ -328,13 +350,14 @@ export default function nestedSkillsExtension(pi: ExtensionAPI): void {
   });
 
   pi.on("session_start", (_event, ctx) => {
-    notifyDiagnostics(ctx, loaded, index);
-    if (index.aliases.size === 0) return;
-    ctx.ui.addAutocompleteProvider((current) => createAutocompleteProvider(current, index));
+    notifyDiagnostics(ctx, state.loaded, state.index);
+    if (state.index.aliases.size === 0) return;
+    // 每次会话开始都按当前索引重新注册，配置面板改了根目录后补全才能跟上。
+    ctx.ui.addAutocompleteProvider((current) => createAutocompleteProvider(current, state.index));
   });
 
-  registerConfigCommand(pi);
-  registerSkillsCommand(pi, index);
+  registerConfigCommand(pi, state);
+  registerSkillsCommand(pi, state);
 }
 
 export { buildSkillIndex, createAutocompleteProvider, formatSkillsMessage };