@venturewild/workspace 0.6.0 → 0.6.2

package/server/src/skills.mjs

@@ -1,213 +1,213 @@
-// Skill discovery + the Powers-block curation store (§7/§8). There is NO native
-// "list commands" API in Claude Code, so we discover the user's skills the cleanest
-// way the design settled on: SCAN the skill directories and parse YAML frontmatter.
-//
-//   - project skills:  <workspace>/.claude/skills/<name>/SKILL.md
-//   - project commands: <workspace>/.claude/commands/<name>.md
-//   - user skills:     ~/.claude/skills/<name>/SKILL.md
-//
-// Precedence (per §2): project > user. The Powers block is USER-CURATED — we surface
-// what the user has (their own + a teammate's, both trusted by membership) with NO
-// VW-prescribed core forced. The "─ from Bazaar ─" zone stays empty/LOCKED until the
-// Shelf trust/provenance Class-D gate ships (a tap runs third-party instructions with
-// bypassPermissions), so this module never returns Bazaar skills.
-//
-// Invocation is NOT done here: a Powers tap injects `/<name>` as a chat message, which
-// Claude Code expands+runs headless (verified §2) — the same stdin path we already use.
-//
-// Curation (which skills are pinned/hidden + their order) lives under
-// ~/.wild-workspace/powers/ (CLAUDE.md #1 — never the synced repo, never localStorage).
-
-import fs from 'node:fs';
-import path from 'node:path';
-import os from 'node:os';
-
-// Bounds so a workspace stuffed with skills can't bloat the store/UI.
-const CAP = { skills: 80, name: 64, desc: 280 };
-
-// --- frontmatter parse -----------------------------------------------------
-// Minimal, dependency-free. We only need a few scalar keys; the description often
-// CONTAINS colons ("Usage: /foo"), so we split on the FIRST colon only. Never throws.
-
-export function parseFrontmatter(text = '') {
-  const out = {};
-  if (typeof text !== 'string') return out;
-  // Frontmatter is the block between the first two `---` fences at the top.
-  const m = text.match(/^?---\s*\r?\n([\s\S]*?)\r?\n---/);
-  if (!m) return out;
-  for (const rawLine of m[1].split(/\r?\n/)) {
-    const line = rawLine.trim();
-    if (!line || line.startsWith('#')) continue;
-    const i = line.indexOf(':');
-    if (i < 0) continue;
-    const key = line.slice(0, i).trim();
-    let val = line.slice(i + 1).trim();
-    // strip matching surrounding quotes
-    if ((val.startsWith('"') && val.endsWith('"')) || (val.startsWith("'") && val.endsWith("'"))) {
-      val = val.slice(1, -1);
-    }
-    out[key] = val;
-  }
-  return out;
-}
-
-function asBool(v, dflt) {
-  if (v === undefined) return dflt;
-  const s = String(v).trim().toLowerCase();
-  if (s === 'true' || s === 'yes' || s === '1') return true;
-  if (s === 'false' || s === 'no' || s === '0') return false;
-  return dflt;
-}
-
-function clip(s, n) {
-  return String(s || '').slice(0, n);
-}
-
-// Build one skill record from a parsed frontmatter + its fallback name.
-function toSkill(fm, fallbackName, source) {
-  const name = clip(fm.name || fallbackName, CAP.name).trim();
-  if (!name) return null;
-  return {
-    name,
-    description: clip(fm.description || '', CAP.desc).trim(),
-    source, //                       'project' | 'user'
-    // user-invocable defaults TRUE (most skills can be run as /name); only an explicit
-    // `user-invocable: false` opts out of being a tappable Power.
-    userInvocable: asBool(fm['user-invocable'], true),
-  };
-}
-
-function readFileSafe(file) {
-  try { return fs.readFileSync(file, 'utf8'); } catch { return null; }
-}
-
-function listDirs(dir) {
-  try {
-    return fs.readdirSync(dir, { withFileTypes: true })
-      .filter((e) => e.isDirectory())
-      .map((e) => e.name);
-  } catch { return []; }
-}
-
-function listFiles(dir, ext) {
-  try {
-    return fs.readdirSync(dir, { withFileTypes: true })
-      .filter((e) => e.isFile() && e.name.endsWith(ext))
-      .map((e) => e.name);
-  } catch { return []; }
-}
-
-// Scan one skills root (<root>/<name>/SKILL.md) → skill records.
-function scanSkillsRoot(root, source) {
-  const out = [];
-  for (const name of listDirs(root)) {
-    const text = readFileSafe(path.join(root, name, 'SKILL.md'));
-    if (text == null) continue;
-    const s = toSkill(parseFrontmatter(text), name, source);
-    if (s) out.push(s);
-  }
-  return out;
-}
-
-// Scan a commands root (<root>/<name>.md) → skill records.
-function scanCommandsRoot(root, source) {
-  const out = [];
-  for (const file of listFiles(root, '.md')) {
-    const text = readFileSafe(path.join(root, file));
-    if (text == null) continue;
-    const s = toSkill(parseFrontmatter(text), file.replace(/\.md$/, ''), source);
-    if (s) out.push(s);
-  }
-  return out;
-}
-
-/**
- * Discover the workspace's skills. Project entries take precedence over same-named
- * user entries (§2). Returns ONLY user-invocable skills (the Powers block taps run
- * them as /name), capped. Never throws — a missing dir just yields fewer skills.
- */
-export function discoverSkills({ workspaceDir = process.cwd(), home = os.homedir() } = {}) {
-  const project = [
-    ...scanSkillsRoot(path.join(workspaceDir, '.claude', 'skills'), 'project'),
-    ...scanCommandsRoot(path.join(workspaceDir, '.claude', 'commands'), 'project'),
-  ];
-  const user = scanSkillsRoot(path.join(home, '.claude', 'skills'), 'user');
-
-  // Dedup by name, project wins.
-  const byName = new Map();
-  for (const s of user) byName.set(s.name, s);
-  for (const s of project) byName.set(s.name, s); // overrides user
-  return [...byName.values()]
-    .filter((s) => s.userInvocable)
-    .sort((a, b) => a.name.localeCompare(b.name))
-    .slice(0, CAP.skills);
-}
-
-// --- curation store --------------------------------------------------------
-// The user's arrangement: `order` (pinned, in display order) + `hidden` (subtracted).
-// A discovered skill not in either is shown after the pinned ones (so a NEW skill
-// appears without the user having to add it — "add/subtract/arrange", subtract is opt-in).
-
-export function defaultPowersDir(env = process.env, home = os.homedir()) {
-  const base = env.WILD_WORKSPACE_GLOBAL_DIR || path.join(home, '.wild-workspace');
-  return path.join(base, 'powers');
-}
-
-function readJsonSafe(file, fallback) {
-  try { return JSON.parse(fs.readFileSync(file, 'utf8')); } catch { return fallback; }
-}
-
-export function createPowers({ baseDir, workspaceDir, home = os.homedir(), env = process.env } = {}) {
-  const dir = baseDir || defaultPowersDir(env, home);
-  const curationFile = path.join(dir, 'curation.json');
-
-  function getCuration() {
-    const v = readJsonSafe(curationFile, null);
-    const order = Array.isArray(v?.order) ? v.order.filter((s) => typeof s === 'string') : [];
-    const hidden = Array.isArray(v?.hidden) ? v.hidden.filter((s) => typeof s === 'string') : [];
-    return { order, hidden };
-  }
-
-  function saveCuration(raw = {}) {
-    const order = Array.isArray(raw.order)
-      ? [...new Set(raw.order.filter((s) => typeof s === 'string').map((s) => s.slice(0, CAP.name)))].slice(0, CAP.skills)
-      : [];
-    const hidden = Array.isArray(raw.hidden)
-      ? [...new Set(raw.hidden.filter((s) => typeof s === 'string').map((s) => s.slice(0, CAP.name)))].slice(0, CAP.skills)
-      : [];
-    const data = { order, hidden, ts: Date.now() };
-    try {
-      fs.mkdirSync(dir, { recursive: true });
-      const tmp = `${curationFile}.${process.pid}.tmp`;
-      fs.writeFileSync(tmp, JSON.stringify(data, null, 2));
-      fs.renameSync(tmp, curationFile);
-    } catch { /* read-only fs — degrade to not-persisted */ }
-    return data;
-  }
-
-  // The merged view the UI renders: discovered skills, ordered by curation, with a
-  // `hidden` flag. Pinned (in `order`) first, then the rest alphabetically.
-  function state() {
-    const discovered = discoverSkills({ workspaceDir, home });
-    const { order, hidden } = getCuration();
-    const hiddenSet = new Set(hidden);
-    const rank = new Map(order.map((n, i) => [n, i]));
-    const skills = discovered
-      .map((s) => ({ ...s, hidden: hiddenSet.has(s.name) }))
-      .sort((a, b) => {
-        const ra = rank.has(a.name) ? rank.get(a.name) : Infinity;
-        const rb = rank.has(b.name) ? rank.get(b.name) : Infinity;
-        if (ra !== rb) return ra - rb;
-        return a.name.localeCompare(b.name);
-      });
-    return {
-      skills,
-      curation: { order, hidden },
-      // The marketplace zone is specified but GATED (Class-D, not built) — always
-      // empty + locked here so the UI can render the "─ from Bazaar ─" boundary.
-      bazaar: { locked: true, skills: [] },
-    };
-  }
-
-  return { dir, curationFile, getCuration, saveCuration, state };
-}
+// Skill discovery + the Powers-block curation store (§7/§8). There is NO native
+// "list commands" API in Claude Code, so we discover the user's skills the cleanest
+// way the design settled on: SCAN the skill directories and parse YAML frontmatter.
+//
+//   - project skills:  <workspace>/.claude/skills/<name>/SKILL.md
+//   - project commands: <workspace>/.claude/commands/<name>.md
+//   - user skills:     ~/.claude/skills/<name>/SKILL.md
+//
+// Precedence (per §2): project > user. The Powers block is USER-CURATED — we surface
+// what the user has (their own + a teammate's, both trusted by membership) with NO
+// VW-prescribed core forced. The "─ from Bazaar ─" zone stays empty/LOCKED until the
+// Shelf trust/provenance Class-D gate ships (a tap runs third-party instructions with
+// bypassPermissions), so this module never returns Bazaar skills.
+//
+// Invocation is NOT done here: a Powers tap injects `/<name>` as a chat message, which
+// Claude Code expands+runs headless (verified §2) — the same stdin path we already use.
+//
+// Curation (which skills are pinned/hidden + their order) lives under
+// ~/.wild-workspace/powers/ (CLAUDE.md #1 — never the synced repo, never localStorage).
+
+import fs from 'node:fs';
+import path from 'node:path';
+import os from 'node:os';
+
+// Bounds so a workspace stuffed with skills can't bloat the store/UI.
+const CAP = { skills: 80, name: 64, desc: 280 };
+
+// --- frontmatter parse -----------------------------------------------------
+// Minimal, dependency-free. We only need a few scalar keys; the description often
+// CONTAINS colons ("Usage: /foo"), so we split on the FIRST colon only. Never throws.
+
+export function parseFrontmatter(text = '') {
+  const out = {};
+  if (typeof text !== 'string') return out;
+  // Frontmatter is the block between the first two `---` fences at the top.
+  const m = text.match(/^?---\s*\r?\n([\s\S]*?)\r?\n---/);
+  if (!m) return out;
+  for (const rawLine of m[1].split(/\r?\n/)) {
+    const line = rawLine.trim();
+    if (!line || line.startsWith('#')) continue;
+    const i = line.indexOf(':');
+    if (i < 0) continue;
+    const key = line.slice(0, i).trim();
+    let val = line.slice(i + 1).trim();
+    // strip matching surrounding quotes
+    if ((val.startsWith('"') && val.endsWith('"')) || (val.startsWith("'") && val.endsWith("'"))) {
+      val = val.slice(1, -1);
+    }
+    out[key] = val;
+  }
+  return out;
+}
+
+function asBool(v, dflt) {
+  if (v === undefined) return dflt;
+  const s = String(v).trim().toLowerCase();
+  if (s === 'true' || s === 'yes' || s === '1') return true;
+  if (s === 'false' || s === 'no' || s === '0') return false;
+  return dflt;
+}
+
+function clip(s, n) {
+  return String(s || '').slice(0, n);
+}
+
+// Build one skill record from a parsed frontmatter + its fallback name.
+function toSkill(fm, fallbackName, source) {
+  const name = clip(fm.name || fallbackName, CAP.name).trim();
+  if (!name) return null;
+  return {
+    name,
+    description: clip(fm.description || '', CAP.desc).trim(),
+    source, //                       'project' | 'user'
+    // user-invocable defaults TRUE (most skills can be run as /name); only an explicit
+    // `user-invocable: false` opts out of being a tappable Power.
+    userInvocable: asBool(fm['user-invocable'], true),
+  };
+}
+
+function readFileSafe(file) {
+  try { return fs.readFileSync(file, 'utf8'); } catch { return null; }
+}
+
+function listDirs(dir) {
+  try {
+    return fs.readdirSync(dir, { withFileTypes: true })
+      .filter((e) => e.isDirectory())
+      .map((e) => e.name);
+  } catch { return []; }
+}
+
+function listFiles(dir, ext) {
+  try {
+    return fs.readdirSync(dir, { withFileTypes: true })
+      .filter((e) => e.isFile() && e.name.endsWith(ext))
+      .map((e) => e.name);
+  } catch { return []; }
+}
+
+// Scan one skills root (<root>/<name>/SKILL.md) → skill records.
+function scanSkillsRoot(root, source) {
+  const out = [];
+  for (const name of listDirs(root)) {
+    const text = readFileSafe(path.join(root, name, 'SKILL.md'));
+    if (text == null) continue;
+    const s = toSkill(parseFrontmatter(text), name, source);
+    if (s) out.push(s);
+  }
+  return out;
+}
+
+// Scan a commands root (<root>/<name>.md) → skill records.
+function scanCommandsRoot(root, source) {
+  const out = [];
+  for (const file of listFiles(root, '.md')) {
+    const text = readFileSafe(path.join(root, file));
+    if (text == null) continue;
+    const s = toSkill(parseFrontmatter(text), file.replace(/\.md$/, ''), source);
+    if (s) out.push(s);
+  }
+  return out;
+}
+
+/**
+ * Discover the workspace's skills. Project entries take precedence over same-named
+ * user entries (§2). Returns ONLY user-invocable skills (the Powers block taps run
+ * them as /name), capped. Never throws — a missing dir just yields fewer skills.
+ */
+export function discoverSkills({ workspaceDir = process.cwd(), home = os.homedir() } = {}) {
+  const project = [
+    ...scanSkillsRoot(path.join(workspaceDir, '.claude', 'skills'), 'project'),
+    ...scanCommandsRoot(path.join(workspaceDir, '.claude', 'commands'), 'project'),
+  ];
+  const user = scanSkillsRoot(path.join(home, '.claude', 'skills'), 'user');
+
+  // Dedup by name, project wins.
+  const byName = new Map();
+  for (const s of user) byName.set(s.name, s);
+  for (const s of project) byName.set(s.name, s); // overrides user
+  return [...byName.values()]
+    .filter((s) => s.userInvocable)
+    .sort((a, b) => a.name.localeCompare(b.name))
+    .slice(0, CAP.skills);
+}
+
+// --- curation store --------------------------------------------------------
+// The user's arrangement: `order` (pinned, in display order) + `hidden` (subtracted).
+// A discovered skill not in either is shown after the pinned ones (so a NEW skill
+// appears without the user having to add it — "add/subtract/arrange", subtract is opt-in).
+
+export function defaultPowersDir(env = process.env, home = os.homedir()) {
+  const base = env.WILD_WORKSPACE_GLOBAL_DIR || path.join(home, '.wild-workspace');
+  return path.join(base, 'powers');
+}
+
+function readJsonSafe(file, fallback) {
+  try { return JSON.parse(fs.readFileSync(file, 'utf8')); } catch { return fallback; }
+}
+
+export function createPowers({ baseDir, workspaceDir, home = os.homedir(), env = process.env } = {}) {
+  const dir = baseDir || defaultPowersDir(env, home);
+  const curationFile = path.join(dir, 'curation.json');
+
+  function getCuration() {
+    const v = readJsonSafe(curationFile, null);
+    const order = Array.isArray(v?.order) ? v.order.filter((s) => typeof s === 'string') : [];
+    const hidden = Array.isArray(v?.hidden) ? v.hidden.filter((s) => typeof s === 'string') : [];
+    return { order, hidden };
+  }
+
+  function saveCuration(raw = {}) {
+    const order = Array.isArray(raw.order)
+      ? [...new Set(raw.order.filter((s) => typeof s === 'string').map((s) => s.slice(0, CAP.name)))].slice(0, CAP.skills)
+      : [];
+    const hidden = Array.isArray(raw.hidden)
+      ? [...new Set(raw.hidden.filter((s) => typeof s === 'string').map((s) => s.slice(0, CAP.name)))].slice(0, CAP.skills)
+      : [];
+    const data = { order, hidden, ts: Date.now() };
+    try {
+      fs.mkdirSync(dir, { recursive: true });
+      const tmp = `${curationFile}.${process.pid}.tmp`;
+      fs.writeFileSync(tmp, JSON.stringify(data, null, 2));
+      fs.renameSync(tmp, curationFile);
+    } catch { /* read-only fs — degrade to not-persisted */ }
+    return data;
+  }
+
+  // The merged view the UI renders: discovered skills, ordered by curation, with a
+  // `hidden` flag. Pinned (in `order`) first, then the rest alphabetically.
+  function state() {
+    const discovered = discoverSkills({ workspaceDir, home });
+    const { order, hidden } = getCuration();
+    const hiddenSet = new Set(hidden);
+    const rank = new Map(order.map((n, i) => [n, i]));
+    const skills = discovered
+      .map((s) => ({ ...s, hidden: hiddenSet.has(s.name) }))
+      .sort((a, b) => {
+        const ra = rank.has(a.name) ? rank.get(a.name) : Infinity;
+        const rb = rank.has(b.name) ? rank.get(b.name) : Infinity;
+        if (ra !== rb) return ra - rb;
+        return a.name.localeCompare(b.name);
+      });
+    return {
+      skills,
+      curation: { order, hidden },
+      // The marketplace zone is specified but GATED (Class-D, not built) — always
+      // empty + locked here so the UI can render the "─ from Bazaar ─" boundary.
+      bazaar: { locked: true, skills: [] },
+    };
+  }
+
+  return { dir, curationFile, getCuration, saveCuration, state };
+}