@dadado/agent-kit-cli 5.3.0 → 5.4.0

package/dashboard/README.md

@@ -0,0 +1,25 @@
+# Mission Control runtime
+
+The dashboard the CLI serves. Plain ES modules, no build step: `serve.mjs` reads this directory
+directly, and `packages/cli/dashboard/` is a generated copy (gitignored, produced by
+`scripts/sync-cli-dashboard.mjs` at build/prepack time so the npm tarball can ship it).
+
+| Path | Role |
+| --- | --- |
+| `serve.mjs` | HTTP server, auth gate, SSE |
+| `start.mjs` / `start-broadcast.mjs` | loopback and LAN entry points, per-workspace port allocation |
+| `dashboard-data.mjs` | snapshot builder for the panel |
+| `dashboard.html`, `open.html` | panel and share shell |
+| `lib/*.mjs` | guards, semantic model, live refresh, browser open, terminal snapshot |
+| `lib/guards.d.mts` | hand-written types consumed by the CLI package (parity is pinned by a test) |
+
+## Where the tests live
+
+Tests for these modules are in `packages/cli/src/dashboard/*.test.ts`, not next to the source. The
+CLI package owns the only test runner in the workspace (vitest), and those suites import the `.mjs`
+files directly (`../../../../dashboard/lib/...`) so there is one implementation under test rather
+than a copy. Lint and format are covered from the repository root (`pnpm lint` checks `dashboard/**`
+before it fans out to the workspace packages).
+
+`dashboard.html` is outside Biome's scope; CSS/HTML-only changes are covered by
+`packages/cli/src/dashboard/plugin-ux-validation.test.ts` instead.

package/dashboard/lib/guards.mjs

@@ -47,12 +47,12 @@ export function escapePerlDoubleQuoted(value) {
 
 /**
  * Resolve the repository root Mission Control should snapshot.
- * @param {NodeJS.ProcessEnv | Record<string, string | undefined>} [env]
+ * @param {NodeJS.ProcessEnv | Record<string, string | undefined> | undefined} env - `undefined` falls back to `process.env`
  * @param {string} kitRoot - absolute path to the kit tree (parent of `dashboard/`)
  * @returns {string} absolute snapshot root
  */
-export function resolveSnapshotRepoRoot(env = process.env, kitRoot) {
-  const raw = env?.[REPO_ROOT_ENV];
+export function resolveSnapshotRepoRoot(env, kitRoot) {
+  const raw = (env ?? process.env)?.[REPO_ROOT_ENV];
   if (typeof raw === "string" && raw.trim()) {
     return resolve(raw.trim());
   }
@@ -400,7 +400,7 @@ export function tokensMatch(a, b) {
 /**
  * Resolve bind + token gate for Mission Control serve.
  * Non-loopback bind requires a valid MISSION_CONTROL_TOKEN (no warn-only 0.0.0.0).
- * @param {NodeJS.ProcessEnv | Record<string, string | undefined>} [env]
+ * @param {NodeJS.ProcessEnv | Record<string, string | undefined> | undefined} env - `undefined` falls back to `process.env`
  * @returns
  *   | { ok: true, host: string, tokenRequired: boolean, token: string | null, broadcast: boolean }
  *   | { ok: false, error: string }

package/dashboard/lib/triage-heading.mjs

@@ -11,8 +11,7 @@
  *   ## Follow-up plan  (also "Followup plan")
  *   ## Residuals plan
  */
-export const TRIAGE_HEADING_RE =
-  /^#{2,6}\s+(?:Triage note|Follow-?up plan|Residuals plan)\b/im;
+export const TRIAGE_HEADING_RE = /^#{2,6}\s+(?:Triage note|Follow-?up plan|Residuals plan)\b/im;
 
 /** True when markdown carries a durable triage heading. */
 export function hasTriageHeading(text) {

package/dist/index.js

@@ -214,7 +214,9 @@ var KNOWN_SHIPPED_OVERLAY_HASHES = /* @__PURE__ */ new Set([
   "73cd50bac290b84df6245d3647e686dee8ec3b0434736d08fdba6e30d342ef0a",
   "61b635ea8a08171062bd329a4615d7426eb787eb796271aa882f55f39db56810",
   "86afbea8f64de68a79ad5e374f3132bdbe2582b94321fb3d438314838e36c776",
-  "65cc1c0293b145e48ed73ad0ca9ab33cbba5ed834a5bfb31f195ed30c2f143df"
+  "65cc1c0293b145e48ed73ad0ca9ab33cbba5ed834a5bfb31f195ed30c2f143df",
+  "f04fcfe31354d1b09aeb256a17e4aab91c98ea48a5cff25e4a0281af3cfb289f",
+  "34e559ad9036d93d9cbc96d394bc2bbeb10ce50ead00d58a914158ae19daa76c"
 ]);
 
 // src/lifecycle/paths.ts
@@ -1088,17 +1090,38 @@ async function findPack(rootDir, packId) {
 }
 
 // src/registry/install.ts
-import { readFile as readFile5 } from "fs/promises";
+import { readFile as readFile5, readdir as readdir3 } from "fs/promises";
 import path8 from "path";
+function skillTargetDir(skillPath, skillId) {
+  const category = skillPath.includes("/core/") ? "core" : "community";
+  return path8.posix.join(".cursor", "skills", category, skillId);
+}
+async function skillFileTargets(registryRoot, skillPath, skillId) {
+  const targetDir = skillTargetDir(skillPath, skillId);
+  const pair = (rel) => ({
+    sourceRel: path8.posix.join(skillPath, rel),
+    targetRel: path8.posix.join(targetDir, rel)
+  });
+  let companions = [];
+  try {
+    const dirAbs = resolveContained(registryRoot, skillPath);
+    const entries = await readdir3(dirAbs, { withFileTypes: true, recursive: true });
+    companions = entries.filter((entry) => entry.isFile()).map((entry) => {
+      const parent = path8.relative(dirAbs, entry.parentPath ?? dirAbs);
+      return path8.join(parent, entry.name).split(path8.sep).join("/");
+    }).filter((rel) => rel !== "SKILL.md" && !rel.split("/").some((seg) => seg.startsWith("."))).sort();
+  } catch {
+    companions = [];
+  }
+  return [pair("SKILL.md"), ...companions.map(pair)];
+}
 function targetForMember(member) {
   switch (member.kind) {
-    case "skill": {
-      const category = member.source.includes("/core/") ? "core" : "community";
+    case "skill":
       return {
         sourceRel: path8.posix.join(member.source, "SKILL.md"),
-        targetRel: path8.posix.join(".cursor", "skills", category, member.id, "SKILL.md")
+        targetRel: path8.posix.join(skillTargetDir(member.source, member.id), "SKILL.md")
       };
-    }
     case "rule":
       return {
         sourceRel: member.source,
@@ -1139,19 +1162,22 @@ function targetForMember(member) {
 }
 async function installSkill(registryRoot, projectRoot, skill, options = {}) {
   const stats = emptyStats();
-  const category = skill.path.includes("/core/") ? "core" : "community";
-  const sourceRel = path8.posix.join(skill.path, "SKILL.md");
-  const targetRel = path8.posix.join(".cursor", "skills", category, skill.id, "SKILL.md");
   const managedHashes = await loadManagedHashLedger(projectRoot);
-  const outcome = await copyRegistryFile(
+  for (const { sourceRel, targetRel } of await skillFileTargets(
     registryRoot,
-    projectRoot,
-    sourceRel,
-    targetRel,
-    options.protectedGlobs ?? [],
-    { managedHashes, persistManagedHashes: false }
-  );
-  recordOutcome(stats, targetRel, outcome);
+    skill.path,
+    skill.id
+  )) {
+    const outcome = await copyRegistryFile(
+      registryRoot,
+      projectRoot,
+      sourceRel,
+      targetRel,
+      options.protectedGlobs ?? [],
+      { managedHashes, persistManagedHashes: false }
+    );
+    recordOutcome(stats, targetRel, outcome);
+  }
   await saveManagedHashLedger(projectRoot, managedHashes);
   return stats;
 }
@@ -1185,16 +1211,18 @@ async function installPack(registryRoot, projectRoot, packId, options = {}) {
   const managedHashes = await loadManagedHashLedger(projectRoot);
   const copyOpts = { managedHashes, persistManagedHashes: false };
   for (const member of packManifest.members) {
-    const { sourceRel, targetRel } = targetForMember(member);
-    const outcome = await copyRegistryFile(
-      registryRoot,
-      projectRoot,
-      sourceRel,
-      targetRel,
-      protectedGlobs,
-      copyOpts
-    );
-    recordOutcome(stats, targetRel, outcome);
+    const pairs = member.kind === "skill" ? await skillFileTargets(registryRoot, member.source, member.id) : [targetForMember(member)];
+    for (const { sourceRel, targetRel } of pairs) {
+      const outcome = await copyRegistryFile(
+        registryRoot,
+        projectRoot,
+        sourceRel,
+        targetRel,
+        protectedGlobs,
+        copyOpts
+      );
+      recordOutcome(stats, targetRel, outcome);
+    }
   }
   await saveManagedHashLedger(projectRoot, managedHashes);
   return stats;
@@ -1620,8 +1648,10 @@ async function buildRegistryPathMap(registryRoot, manifest) {
   for (const packId of manifest.packs ?? []) {
     const pack = await loadPackManifest(registryRoot, packId);
     for (const member of pack.members) {
-      const { sourceRel, targetRel } = packMemberTargets(member);
-      map.set(targetRel.split(path9.sep).join("/"), sourceRel.split(path9.sep).join("/"));
+      const pairs = member.kind === "skill" ? await skillFileTargets(registryRoot, member.source, member.id) : [packMemberTargets(member)];
+      for (const { sourceRel, targetRel } of pairs) {
+        map.set(targetRel.split(path9.sep).join("/"), sourceRel.split(path9.sep).join("/"));
+      }
     }
   }
   if ((manifest.skills ?? []).length > 0) {
@@ -1630,25 +1660,28 @@ async function buildRegistryPathMap(registryRoot, manifest) {
     for (const id of manifest.skills ?? []) {
       const skill = pool.find((s) => s.id === id);
       if (!skill) continue;
-      const category = skill.path.includes("/core/") ? "core" : "community";
-      const sourceRel = path9.posix.join(skill.path, "SKILL.md");
-      const targetRel = path9.posix.join(".cursor", "skills", category, skill.id, "SKILL.md");
-      map.set(targetRel, sourceRel);
+      for (const { sourceRel, targetRel } of await skillFileTargets(
+        registryRoot,
+        skill.path,
+        skill.id
+      )) {
+        map.set(targetRel, sourceRel);
+      }
     }
   }
   return map;
 }
 function guessRegistryPath(projectRel) {
   const p = projectRel.split(path9.sep).join("/");
-  if (p.startsWith(".cursor/skills/") && p.endsWith("/SKILL.md")) {
+  if (p.startsWith(".cursor/skills/")) {
     const rest = p.slice(".cursor/skills/".length);
     const parts = rest.split("/");
-    if (parts.length === 3 && (parts[0] === "core" || parts[0] === "community")) {
+    if (parts.length >= 3 && (parts[0] === "core" || parts[0] === "community")) {
       return path9.posix.join("registry/skills", rest);
     }
     const skillId = parts[0];
-    if (parts.length === 2 && skillId) {
-      return path9.posix.join("registry/skills", "community", skillId, "SKILL.md");
+    if (parts.length >= 2 && skillId) {
+      return path9.posix.join("registry/skills", "community", rest);
     }
   }
   if (p.startsWith(".cursor/rules/")) {
@@ -2751,8 +2784,10 @@ async function diffAgainstRegistry(registryRoot, projectRoot, manifest) {
   for (const packId of manifest.packs ?? []) {
     const pack = await loadPackManifest(registryRoot, packId);
     for (const member of pack.members) {
-      const { sourceRel, targetRel } = packMemberTargets(member);
-      await pushUnique(sourceRel, targetRel);
+      const pairs = member.kind === "skill" ? await skillFileTargets(registryRoot, member.source, member.id) : [packMemberTargets(member)];
+      for (const { sourceRel, targetRel } of pairs) {
+        await pushUnique(sourceRel, targetRel);
+      }
     }
   }
   if ((manifest.skills ?? []).length > 0) {
@@ -2764,10 +2799,13 @@ async function diffAgainstRegistry(registryRoot, projectRoot, manifest) {
         entries.push({ path: `skill:${id}`, status: "missing-registry" });
         continue;
       }
-      const category = skill.path.includes("/core/") ? "core" : "community";
-      const sourceRel = path13.posix.join(skill.path, "SKILL.md");
-      const targetRel = path13.posix.join(".cursor", "skills", category, skill.id, "SKILL.md");
-      await pushUnique(sourceRel, targetRel);
+      for (const { sourceRel, targetRel } of await skillFileTargets(
+        registryRoot,
+        skill.path,
+        skill.id
+      )) {
+        await pushUnique(sourceRel, targetRel);
+      }
     }
   }
   return entries;
@@ -2843,7 +2881,7 @@ import { defineCommand as defineCommand7 } from "citty";
 
 // src/invariants/hooks-health.ts
 import { execFile as execFile2 } from "child_process";
-import { constants as constants2, access as access5, readFile as readFile9, readdir as readdir3, stat as stat2 } from "fs/promises";
+import { constants as constants2, access as access5, readFile as readFile9, readdir as readdir4, stat as stat2 } from "fs/promises";
 import path14 from "path";
 import { promisify as promisify2 } from "util";
 var execFileAsync2 = promisify2(execFile2);
@@ -2927,7 +2965,7 @@ async function assessGitHooksInstallDrift(rootDir) {
   const installHint = "Install or refresh with: `cp git-hooks/<name> .git/hooks/<name> && chmod +x .git/hooks/<name>` (see git-hooks/README.md)";
   let names = [...GIT_HOOK_CANONICAL_NAMES];
   try {
-    const listed = await readdir3(canonicalDir);
+    const listed = await readdir4(canonicalDir);
     const fromDisk = listed.filter(
       (n) => GIT_HOOK_CANONICAL_NAMES.includes(n)
     );
@@ -4772,7 +4810,7 @@ var guardCommand = defineCommand8({
 
 // src/commands/handoff.ts
 import { spawn as spawn4 } from "child_process";
-import { readFile as readFile13, readdir as readdir4, writeFile as writeFile6 } from "fs/promises";
+import { readFile as readFile13, readdir as readdir5, writeFile as writeFile6 } from "fs/promises";
 import path25 from "path";
 import { defineCommand as defineCommand9 } from "citty";
 function parsePlanFrontmatter(raw) {
@@ -4791,7 +4829,7 @@ function parsePlanFrontmatter(raw) {
 }
 async function findActivePlan(plansDir) {
   if (!await fileExists(plansDir)) return null;
-  const files = (await readdir4(plansDir)).filter((f) => f.endsWith(".plan.md")).sort().reverse();
+  const files = (await readdir5(plansDir)).filter((f) => f.endsWith(".plan.md")).sort().reverse();
   for (const file of files) {
     const raw = await readFile13(path25.join(plansDir, file), "utf8");
     const fm = parsePlanFrontmatter(raw);
@@ -6425,7 +6463,7 @@ import { defineCommand as defineCommand13 } from "citty";
 
 // src/invariants/monitors-untriaged.ts
 import { execFile as execFile5 } from "child_process";
-import { readFile as readFile17, readdir as readdir5, stat as stat3 } from "fs/promises";
+import { readFile as readFile17, readdir as readdir6, stat as stat3 } from "fs/promises";
 import path34 from "path";
 import { promisify as promisify5 } from "util";
 
@@ -6450,7 +6488,7 @@ function hasOpenGaps(content) {
 }
 async function listMonitorFiles(memoryDir) {
   try {
-    const names = await readdir5(memoryDir);
+    const names = await readdir6(memoryDir);
     return names.filter((n) => n.startsWith("plan-monitor-") && n.endsWith(".md")).sort();
   } catch {
     return [];
@@ -6858,7 +6896,7 @@ function createPersonaBannerPrinter(persona) {
 }
 
 // src/plan-loop/plan-state.ts
-import { readFile as readFile18, readdir as readdir6 } from "fs/promises";
+import { readFile as readFile18, readdir as readdir7 } from "fs/promises";
 import path38 from "path";
 function countPendingTodos(raw) {
   const lines = raw.split(/\r?\n/);
@@ -6886,7 +6924,7 @@ function countPendingTodos(raw) {
 }
 async function findActivePlanFile(plansDir) {
   if (!await fileExists(plansDir)) return null;
-  const files = (await readdir6(plansDir)).filter((f) => f.endsWith(".plan.md")).sort();
+  const files = (await readdir7(plansDir)).filter((f) => f.endsWith(".plan.md")).sort();
   return files[0] ? path38.join(plansDir, files[0]) : null;
 }
 async function readPlan(planPath) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dadado/agent-kit-cli",
-  "version": "5.3.0",
+  "version": "5.4.0",
   "description": "Agent Kit CLI: HITL framework install and tooling for AI-assisted IDEs (rules, skills, plan/handoff, context).",
   "license": "PolyForm-Noncommercial-1.0.0",
   "type": "module",