@hncode/hncode 0.4.0 → 0.4.2

package/README.md

@@ -16,7 +16,8 @@ A cyan-blue terminal user interface (TUI) coding agent inspired by [Kimi Code](h
 - **Prompt Caching**: Explicit cache breakpoints for the stable prefix (`/cache`)
 - **MCP-style Toolbelt**: Read, Write, Edit, Bash, Glob, Grep, TodoList, WebSearch, etc.
 - **MCP Client**: Connect real Model Context Protocol servers (stdio + HTTP) and use their tools
-- **Skills**: Package reusable prompts as Markdown (`/import-skill`, `/skill:<name>` with tab-completion)
+- **Skills**: Package reusable prompts as Markdown (`/import-skill`, `/skill:<name>` with tab-completion, plus a manager that installs them from the repo)
+- **Skills & plugins manager**: `/skills` and `/plugins` browse what is installed *and* what the repo offers, with descriptions, and install/remove in place
 - **Shell Hooks**: Run your own commands at lifecycle events (`PreToolUse`, `PostToolUse`, …)
 - **Git Workflow**: `/git`, `/commit`, `/branch`, `/worktree`, `/pr` plus a `Git` tool for the agent
 - **Plan Review**: Review what a plan will touch, edit it, then approve before any file changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hncode/hncode",
-  "version": "0.4.0",
+  "version": "0.4.2",
   "description": "hncode — A cyan-blue TUI AI coding agent with zero runtime dependencies.",
   "type": "module",
   "main": "src/index.js",
@@ -16,8 +16,8 @@
     "package.json"
   ],
   "scripts": {
-    "preinstall": "node bin/preinstall.js",
-    "install": "node bin/install.js",
+    "preinstall": "node bin/preinstall.cjs",
+    "install": "node bin/install.cjs",
     "i18n": "node tools/sync-i18n.mjs",
     "i18n:check": "node tools/sync-i18n.mjs --check"
   },

package/src/plugin.js

@@ -40,6 +40,49 @@ const registeredHooks = {};
 const registeredConfig = {};
 const loadedPlugins = [];
 
+// ---------------------------------------------------------------------------
+// Plugin notices — what loaded, what failed, and anything a plugin reports.
+//
+// A plugin author's instinct is console.log/error, and that used to be the only channel.
+// The TUI owns the screen, so those messages went to a terminal the user cannot see while
+// the app runs: a plugin that failed to load simply did not appear, with no explanation
+// anywhere. Collecting every message and letting the TUI render it puts both the success
+// and the failure where the user is already looking.
+// ---------------------------------------------------------------------------
+const pendingNotices = [];
+
+/** Record one notice. `level` is 'info' | 'warn' | 'error'. */
+function notice(level, text) {
+  pendingNotices.push({ level, text: String(text == null ? '' : text) });
+}
+
+/** Take and clear the pending notices — the TUI drains this after loading. */
+export function takePluginNotices() {
+  const out = pendingNotices.slice();
+  pendingNotices.length = 0;
+  return out;
+}
+
+function fmtArgs(args) {
+  return args.map((a) => {
+    if (typeof a === 'string') return a;
+    if (a instanceof Error) return a.message;
+    try { return JSON.stringify(a); } catch { return String(a); }
+  }).join(' ');
+}
+
+/**
+ * A console-like object handed to `install(api)` as `api.log`, so a plugin can report
+ * without knowing anything about the UI. Everything reaches the chat AND mirrors to the
+ * terminal, so a plugin author watching stderr still sees it.
+ */
+export const pluginLogger = {
+  log: (...a) => { notice('info', fmtArgs(a)); console.log('[hncode-plugin]', ...a); },
+  info: (...a) => { notice('info', fmtArgs(a)); console.log('[hncode-plugin]', ...a); },
+  warn: (...a) => { notice('warn', fmtArgs(a)); console.warn('[hncode-plugin]', ...a); },
+  error: (...a) => { notice('error', fmtArgs(a)); console.error('[hncode-plugin]', ...a); },
+};
+
 const API = {
   // Register a new tool (same shape as built-in tool specs).
   // Returns an unregister function.
@@ -106,8 +149,15 @@ const API = {
   get hooks() { return { ...registeredHooks }; },
   get config() { return { ...registeredConfig }; },
   get plugins() { return [...loadedPlugins]; },
+
+  // Reporting. `api.log.info('…')` reaches the CHAT, which is the only channel a user
+  // can actually see while the TUI owns the screen; it also mirrors to the terminal for
+  // a developer watching it. See pluginLogger.
+  log: pluginLogger,
 };
 
+// The agent context is injected lazily; plugins that need it read api.ctx
+
 // The agent context is injected lazily; plugins that need it read api.ctx
 // during their hook callbacks.
 let _ctx = null;
@@ -133,20 +183,47 @@ export async function loadPlugins(dir) {
   for (const k of Object.keys(registeredHooks)) delete registeredHooks[k];
   for (const k of Object.keys(registeredConfig)) delete registeredConfig[k];
   loadedPlugins.length = 0;
+  // Notices belong to THIS run too; a second load must not replay the first run's.
+  pendingNotices.length = 0;
 
-  const files = fs.readdirSync(pluginDir)
-    .filter((f) => f.endsWith('.js') || f.endsWith('.mjs'))
-    .sort();
+  // Two layouts, because both are in the wild and the install path only produces one:
+  //   plugins/foo.js            — a single-file plugin
+  //   plugins/foo/index.js      — a plugin directory (this is what /plugins install
+  //                               writes, and what a plugin with helper files needs)
+  // Only the flat form used to be scanned, so every plugin installed through the
+  // manager was written to disk correctly and then silently never loaded.
+  const files = [];
+  for (const e of fs.readdirSync(pluginDir, { withFileTypes: true }).sort((a, b) => a.name.localeCompare(b.name))) {
+    if (e.name.startsWith('.')) continue;
+    if (e.isFile() && (e.name.endsWith('.js') || e.name.endsWith('.mjs'))) {
+      files.push({ id: e.name, full: path.join(pluginDir, e.name) });
+      continue;
+    }
+    if (!e.isDirectory()) continue;
+    const dir = path.join(pluginDir, e.name);
+    // A directory's entry point: index.js/.mjs, else the first .js/.mjs beside it (a
+    // directory may legitimately be named differently from its file).
+    let entries = [];
+    try { entries = fs.readdirSync(dir); } catch { continue; }
+    const entry = ['index.js', 'index.mjs'].find((n) => entries.includes(n))
+      || entries.filter((n) => n.endsWith('.js') || n.endsWith('.mjs')).sort()[0];
+    if (!entry) continue;
+    files.push({ id: `${e.name}/${entry}`, full: path.join(dir, entry) });
+  }
 
-  for (const f of files) {
-    const full = path.join(pluginDir, f);
+  for (const { id: f, full } of files) {
     try {
       const mod = await import(pathToFileURL(full).href);
+      // `install(api)` may be the named export or live on the default export.
       let installFn;
       if (mod && typeof mod.install === 'function') installFn = mod.install;
       else if (mod && mod.default && typeof mod.default.install === 'function') installFn = mod.default.install;
 
-      if (typeof installFn !== 'function') {
+if (typeof installFn !== 'function') {
+        // Report it to the chat as well: a file that is present but registers nothing is
+        // the most confusing failure of all — the user sees the file on disk and nothing
+        // in the UI, with no hint that the export is missing.
+        notice('error', `${f}: no install(api) function found — skipped`);
         // eslint-disable-next-line no-console
         console.error(`[hncode-plugin] ${f}: no install(api) function found, skipped.`);
         continue;
@@ -161,18 +238,35 @@ export async function loadPlugins(dir) {
       const def = (mod && mod.default) || {};
       const defMeta = typeof def === 'string' ? { name: def } : def;
       const namedMeta = (mod && mod.meta) || {};
+      // Fall back to the DIRECTORY name for a directory plugin, so a plugin at
+      // plugins/secret-guard/index.js with no metadata shows as `secret-guard`, not
+      // `secret-guard/index.js`.
+      const fallback = f.includes('/') ? f.split('/')[0] : f.replace(/\.m?js$/, '');
       const plugin = {
         id: f,
-        name: defMeta.name || defMeta.meta?.name || namedMeta.name || namedMeta.meta?.name || f,
+        name: defMeta.name || defMeta.meta?.name || namedMeta.name || namedMeta.meta?.name || fallback,
         version: defMeta.version || defMeta.meta?.version || namedMeta.version || namedMeta.meta?.version || '0.0.0',
       };
-      loadedPlugins.push(plugin);
+      // install(api) FIRST, then record it. Pushing before the call reported a plugin as
+      // loaded even when its install() threw — /plugins would list it under Loaded while
+      // none of its tools existed.
       installFn(API);
+      loadedPlugins.push(plugin);
+      // The notice goes to the chat; the console line stays for a developer watching a
+      // terminal, matching the failure path below.
+      notice('info', `loaded ${plugin.name}${plugin.version && plugin.version !== '0.0.0' ? ' v' + plugin.version : ''}`);
       // eslint-disable-next-line no-console
       console.log(`[hncode-plugin] loaded: ${plugin.name}`);
     } catch (e) {
+      // Report it BOTH ways. stderr is the developer's channel (a plugin author running
+      // hncode from a terminal), but the TUI owns the screen, so a user who installs a
+      // plugin through /plugins never sees it — their plugin simply does not appear and
+      // nothing says why. `notice` feeds the chat; the console line stays for a developer
+      // watching a terminal.
+      const msg = String((e && e.message) || e);
+      notice('error', `${f} failed to load: ${msg}`);
       // eslint-disable-next-line no-console
-      console.error(`[hncode-plugin] ${f}: ${e.message}`);
+      console.error(`[hncode-plugin] ${f}: ${msg}`);
     }
   }
 
@@ -212,5 +306,6 @@ export function _resetPlugins() {
   for (const k of Object.keys(registeredHooks)) delete registeredHooks[k];
   for (const k of Object.keys(registeredConfig)) delete registeredConfig[k];
   loadedPlugins.length = 0;
+  pendingNotices.length = 0;
   _ctx = null;
 }

package/src/registry-browser.js

@@ -0,0 +1,299 @@
+// RegistryBrowser — the full-screen /skills and /plugins manager.
+//
+// Modelled on kimi-code's PluginsPanelComponent (a tabbed list where every row has a
+// DESCRIPTION under its label) and on this project's own /tasks browser: pure
+// rendering plus key handling, returning ACTION STRINGS to the caller so all state
+// stays in one place.
+//
+//   ────────────────────────────────────────────────────────────────
+//    Skills
+//    Tab switch · ↑↓ select · Enter install/remove · Esc close
+//
+//    [Installed (1)]  Available (3)
+//
+//    ❯ demo                          installed
+//        A demo skill
+//      review-pr                     install
+//        Review a pull request for bugs
+//    ────────────────────────────────────────────────────────────────
+//
+// The description line is the reason this exists instead of the plain list panel: a
+// skill or plugin is meaningless without knowing what it does, and the picker's inline
+// `sub` truncates on a narrow terminal.
+//
+// Tabs differ per kind, because the two halves mean different things:
+//   TABS_LOCAL  = what is on disk here (installed skills / plugins on disk)
+//   TABS_REMOTE = what the repo offers, with an `installed` badge when present
+
+import { C } from './colors.js';
+import { visualWidth } from './term.js';
+
+const ELLIPSIS = '\u2026';
+const ESC = '\x1b';
+const MIN_WIDTH = 44;
+const MIN_HEIGHT = 8;
+
+function visWidth(s) { return visualWidth(String(s).replace(/\x1b\[[0-9;?]*[a-zA-Z]/g, '')); }
+
+function truncateToWidth(s, width) {
+  const str = String(s);
+  let out = '';
+  let n = 0;
+  let i = 0;
+  while (i < str.length) {
+    if (str[i] === ESC) {
+      const m = /^\x1b\[[0-9;?]*[a-zA-Z]/.exec(str.slice(i));
+      if (m) { out += m[0]; i += m[0].length; continue; }
+    }
+    const cp = str.codePointAt(i);
+    const ch = String.fromCodePoint(cp);
+    const cw = visWidth(ch);
+    if (n + cw > width - 1) return out + ELLIPSIS + ' '.repeat(Math.max(0, width - n - 1));
+    n += cw;
+    out += ch;
+    i += cp > 0xffff ? 2 : 1;
+  }
+  return out;
+}
+
+function fitExactly(line, width) {
+  const w = visWidth(line);
+  if (w === width) return line;
+  if (w > width) return truncateToWidth(line, width);
+  return line + ' '.repeat(width - w);
+}
+
+/** Plain-text wrap so a long description keeps its own indented lines. */
+function wrapText(text, width) {
+  const words = String(text || '').trim().split(/\s+/).filter(Boolean);
+  const out = [];
+  let cur = '';
+  for (const w of words) {
+    const next = cur ? cur + ' ' + w : w;
+    if (visWidth(next) <= width) { cur = next; continue; }
+    if (cur) out.push(cur);
+    cur = visWidth(w) <= width ? w : w.slice(0, Math.max(1, width));
+  }
+  if (cur) out.push(cur);
+  return out.length ? out : [''];
+}
+
+// Tabs. `id` is what the key handler switches on; `label` is shown with the count.
+export const SKILL_TABS = [
+  { id: 'installed', label: 'Installed' },
+  { id: 'available', label: 'Available' },
+];
+export const PLUGIN_TABS = [
+  { id: 'loaded', label: 'Loaded' },
+  { id: 'ondisk', label: 'On disk' },
+  { id: 'available', label: 'Available' },
+];
+
+/** The rows for one tab. Each row: { label, description, status, statusTone, name, action }. */
+export function rowsForTab(state, tabId) {
+  const d = state.data || {};
+  if (state.kind === 'skills') {
+    if (tabId === 'installed') {
+      return (d.installed || []).map((s) => ({
+        name: s.name,
+        label: s.name,
+        description: s.description || '(no description)',
+        status: 'installed',
+        statusTone: 'dim',
+        action: 'remove-skill',
+      }));
+    }
+    const local = new Set((d.installed || []).map((s) => s.name));
+    return (d.remote || []).map((r) => ({
+      name: r.name,
+      label: r.name,
+      description: r.description || (r.readable === false ? '(no SKILL.md — cannot install)' : '(no description)'),
+      status: local.has(r.name) ? 'installed' : 'install',
+      statusTone: local.has(r.name) ? 'dim' : 'primary',
+      action: local.has(r.name) ? 'reinstall-skill' : 'install-skill',
+      disabled: r.readable === false,
+    }));
+  }
+  // plugins
+  if (tabId === 'loaded') {
+    return (d.loaded || []).map((p) => ({
+      name: p.name,
+      label: p.name,
+      description: `v${p.version || '0.0.0'} · ${p.id}`,
+      status: 'loaded',
+      statusTone: 'ok',
+      action: 'none',
+    }));
+  }
+  if (tabId === 'ondisk') {
+    const loadedNames = new Set((d.loaded || []).map((p) => p.name));
+    return (d.onDisk || []).map((p) => ({
+      name: p.name,
+      label: p.name + (p.kind === 'dir' ? '/' : ''),
+      description: loadedNames.has(p.name)
+        ? 'loaded in this session'
+        : 'not loaded — restart hncode to load it',
+      status: loadedNames.has(p.name) ? 'loaded' : 'restart',
+      statusTone: loadedNames.has(p.name) ? 'ok' : 'warn',
+      action: 'remove-plugin',
+    }));
+  }
+  const local = new Set((d.onDisk || []).map((p) => p.name));
+  return (d.remote || []).map((r) => ({
+    name: r.name,
+    label: r.name,
+    description: local.has(r.name) ? 'already installed locally' : 'available in the repository',
+    status: local.has(r.name) ? 'installed' : 'install',
+    statusTone: local.has(r.name) ? 'dim' : 'primary',
+    action: local.has(r.name) ? 'reinstall-plugin' : 'install-plugin',
+  }));
+}
+
+function statusColor(tone) {
+  if (tone === 'ok') return C.green;
+  if (tone === 'warn') return C.orange;
+  if (tone === 'dim') return C.gray;
+  return C.blue;
+}
+
+export function renderRegistryBrowser(state, cols, rows) {
+  if (cols < MIN_WIDTH || rows < MIN_HEIGHT) {
+    const lines = [fitExactly(C.red + 'Terminal too small (need >= ' + MIN_WIDTH + ' x ' + MIN_HEIGHT + ')' + C.reset, cols)];
+    for (let i = 1; i < rows; i++) lines.push(' '.repeat(cols));
+    return lines;
+  }
+  const title = state.kind === 'skills' ? ' Skills' : ' Plugins';
+  const tabs = state.kind === 'skills' ? SKILL_TABS : PLUGIN_TABS;
+  const tabIdx = Math.max(0, tabs.findIndex((t) => t.id === state.tab));
+  const tab = tabs[tabIdx] || tabs[0];
+  const all = rowsForTab(state, tab.id);
+  const sel = Math.max(0, Math.min(all.length - 1, state.selectedIndex || 0));
+
+  const out = [];
+  const rule = C.border + '\u2500'.repeat(cols) + C.reset;
+  out.push(rule);
+  out.push(fitExactly(C.cyan + C.bold + title + C.reset, cols));
+  // The key hints stay visible while the fetch runs: the panel is usable during it
+  // (the local tabs are already loaded), so replacing the hints with a status message
+  // would hide the keys exactly when the user is most likely to press them.
+  const hint = ' Tab switch \u00b7 \u2191\u2193 select \u00b7 Enter install/remove \u00b7 Esc close';
+  out.push(fitExactly(C.gray + hint + (state.busy ? C.gray + `  ${state.busy}` : '') + C.reset, cols));
+  out.push('');
+
+  // Tab strip, with the count of each tab's rows.
+  // Tab strip, with the count of each tab's rows.
+  out.push(tabs.map((t) => {
+    const n = rowsForTab(state, t.id).length;
+    return t.id === tab.id
+      ? C.cyan + C.bold + `[${t.label} (${n})]` + C.reset
+      : C.gray + ` ${t.label} (${n}) ` + C.reset;
+  }).join(' '));
+  out.push('');
+
+  // Body: each row is a label line plus its wrapped description, so the footer must be
+  // sized against the ROWS, not the lines. A row is 2+ lines tall.
+  const listTop = out.length;
+  const bodyRows = Math.max(1, rows - listTop - 3);      // leave room for the footer
+  if (!all.length) {
+    // The message must describe THIS tab. A failed remote fetch used to win over
+    // everything, so "Loaded" and "On disk" — which are read straight off the local
+    // filesystem and cannot fail from the network — reported "fetch failed" whenever
+    // the repo lookup did. That reads as "your local plugins are broken", which is
+    // both wrong and alarming. Only the Available tab can talk about the fetch.
+    const isRemoteTab = tab.id === 'available';
+    // "Loading" only applies to the tab that is actually waiting on the fetch; a local
+    // tab is empty because there is genuinely nothing there, and saying "Loading" for
+    // data that is already on screen reads as a stall.
+    const msg = (isRemoteTab && state.busy) ? '  Loading\u2026'
+      : (isRemoteTab && state.data && state.data.remoteError)
+        ? '  Could not fetch from the repo: ' + state.data.remoteError
+        : isRemoteTab ? '  Nothing published in the repo yet.'
+          : '  Nothing here.';
+    out.push(C.gray + msg + C.reset);
+  }
+  for (let i = 0; i < all.length; i++) {
+    const r = all[i];
+    const isSel = i === sel;
+    const ptr = isSel ? C.cyan + '\u276f ' + C.reset : '  ';
+    const nameStyle = isSel ? (C.cyan + C.bold) : C.white;
+    const badge = r.status ? '  ' + statusColor(r.statusTone) + r.status + C.reset : '';
+    const dis = r.disabled ? C.gray : '';
+    out.push(fitExactly(ptr + nameStyle + dis + r.label + C.reset + badge, cols));
+    for (const line of wrapText(r.description, Math.max(8, cols - 6))) {
+      out.push(C.gray + '    ' + line + C.reset);
+    }
+  }
+  // A fetch failure has to show even when the LOCAL half has rows, or the Available
+  // tab silently looks empty and the user cannot tell why.
+  if (all.length && state.data && state.data.remoteError && tab.id === 'available') {
+    out.push('');
+    out.push(C.orange + '  Could not fetch the list: ' + state.data.remoteError + C.reset);
+  }
+  // The last action's outcome (installed / removed / failed) sits above the hint line.
+  if (state.flash) {
+    out.push('');
+    out.push(C.cyan + '  ' + state.flash + C.reset);
+  }
+
+  // Footer: what the highlighted row would DO, so the effect of Enter is never a
+  // surprise. A row already installed says "reinstall", because that is what Enter
+  // does there (and it is the only way to pick up a newer version).
+  const cur = all[sel];
+  const action = !cur ? ''
+    : cur.disabled ? 'Enter does nothing \u2014 this entry has no SKILL.md'
+      : cur.action === 'reinstall-skill' || cur.action === 'reinstall-plugin'
+        ? 'Enter reinstalls ' + cur.name + ' (replaces the local copy)'
+        : cur.action === 'install-skill' || cur.action === 'install-plugin'
+          ? 'Enter installs ' + cur.name + ' from the repository'
+          : cur.action.startsWith('remove') ? 'Enter removes the local copy of ' + cur.name
+            : 'loaded plugin \u2014 switch to the On disk tab to remove it';
+  while (out.length < rows - 2) out.push('');
+  out.push(fitExactly(C.gray + ' ' + action + C.reset, cols));
+  out.push(rule);
+  return out.slice(0, rows);
+}
+
+/**
+ * One key. Returns an action string for the caller:
+ * 'close' | 'switchTab' | 'select' | 'install' | 'remove' | 'reload' | null
+ */
+export function handleRegistryBrowserKey(state, t) {
+  const tabs = state.kind === 'skills' ? SKILL_TABS : PLUGIN_TABS;
+  const all = rowsForTab(state, state.tab);
+  const ch = t.ch;
+
+  if (t.key === 'escape' || ch === 'q' || ch === 'Q') return 'close';
+  if (t.key === 'tab') {
+    const i = Math.max(0, tabs.findIndex((x) => x.id === state.tab));
+    state.tab = tabs[(i + 1) % tabs.length].id;
+    state.selectedIndex = 0;
+    return 'switchTab';
+  }
+  if (t.key === 'up' || ch === 'k') {
+    state.selectedIndex = Math.max(0, (state.selectedIndex || 0) - 1);
+    return 'select';
+  }
+  if (t.key === 'down' || ch === 'j') {
+    state.selectedIndex = Math.min(Math.max(0, all.length - 1), (state.selectedIndex || 0) + 1);
+    return 'select';
+  }
+  if (t.key === 'home') { state.selectedIndex = 0; return 'select'; }
+  if (t.key === 'end') { state.selectedIndex = Math.max(0, all.length - 1); return 'select'; }
+  if (ch === 'r' || ch === 'R') return 'reload';
+  if (t.key === 'enter' || t.key === 'space' || ch === ' ') {
+    const row = all[state.selectedIndex || 0];
+    if (!row || row.disabled) return null;
+    // `installed` on the Available tab still reinstalls: that is how you pick up a
+    // newer version of something you already have, and refusing it would leave no way
+    // to update.
+    if (row.action.startsWith('install')) return 'install';
+    if (row.action.startsWith('remove')) return 'remove';
+    return null;
+  }
+  if (ch === 'd' || ch === 'D') {
+    const row = all[state.selectedIndex || 0];
+    if (row && row.action.startsWith('remove')) return 'remove';
+    return null;
+  }
+  return null;
+}