vigiles 5.1.0 → 5.2.0

package/dist/test-coverage.js

@@ -29,6 +29,20 @@ exports.formatUntestedReport = formatUntestedReport;
 const node_fs_1 = require("node:fs");
 const node_path_1 = require("node:path");
 const glob_1 = require("glob");
+const layout_js_1 = require("./adapters/claude-code/layout.js");
+/**
+ * The two on-disk locations a surface dir can occupy: the plugin-root form
+ * (`skills/…`) and the materialized form (`.claude/skills/…`) — derived from the
+ * layout so a non-Claude-Code harness (its own `materializeRoot` / surface dir)
+ * is discovered without hard-coding `.claude`. An empty `dir` (a harness lacking
+ * the surface, e.g. Codex subagents) yields no globs.
+ */
+function surfaceGlobs(dir, leaf, materializeRoot) {
+    if (!dir)
+        return [];
+    const matForm = materializeRoot ? `${materializeRoot}/${dir}` : dir;
+    return [...new Set([`${dir}/${leaf}`, `${matForm}/${leaf}`])];
+}
 // ---------------------------------------------------------------------------
 // Internals
 // ---------------------------------------------------------------------------
@@ -58,12 +72,9 @@ function read(path) {
         return "";
     }
 }
-function discoverSkills(basePath, ignore) {
+function discoverSkills(basePath, ignore, layout) {
     const out = [];
-    const found = (0, glob_1.globSync)(["skills/*/SKILL.md", ".claude/skills/*/SKILL.md"], {
-        cwd: basePath,
-        ignore,
-    });
+    const found = (0, glob_1.globSync)(surfaceGlobs(layout.skillDir, "*/SKILL.md", layout.materializeRoot), { cwd: basePath, ignore });
     for (const path of found.sort()) {
         const name = (0, node_path_1.basename)((0, node_path_1.dirname)(path));
         const content = read((0, node_path_1.join)(basePath, path));
@@ -71,18 +82,15 @@ function discoverSkills(basePath, ignore) {
             kind: "skill",
             path,
             name,
-            tokens: [`skills/${name}`, `:${name}`],
+            tokens: [`${layout.skillDir}/${name}`, `:${name}`],
             ignored: content.includes(IGNORE_MARKER),
         });
     }
     return out;
 }
-function discoverAgents(basePath, ignore) {
+function discoverAgents(basePath, ignore, layout) {
     const out = [];
-    const found = (0, glob_1.globSync)(["agents/*.md", ".claude/agents/*.md"], {
-        cwd: basePath,
-        ignore,
-    });
+    const found = (0, glob_1.globSync)(surfaceGlobs(layout.agentDir, "*.md", layout.materializeRoot), { cwd: basePath, ignore });
     for (const path of found.sort()) {
         if (path.endsWith(".spec.ts"))
             continue;
@@ -100,7 +108,7 @@ function discoverAgents(basePath, ignore) {
     return out;
 }
 /** Hook-script paths referenced from a manifest's `hooks` block (file hooks only). */
-function hookScripts(basePath, manifest) {
+function hookScripts(basePath, manifest, pluginRootToken) {
     if (!(0, node_fs_1.existsSync)((0, node_path_1.join)(basePath, manifest)))
         return [];
     let hooks;
@@ -114,25 +122,31 @@ function hookScripts(basePath, manifest) {
     if (hooks === undefined)
         return [];
     const text = JSON.stringify(hooks);
+    // "${CLAUDE_PLUGIN_ROOT}" → unbraced "$CLAUDE_PLUGIN_ROOT" — strip the harness's
+    // own token (both forms) so the path is checkable relative to the plugin root.
+    const unbraced = pluginRootToken.replace(/^\$\{(.+)\}$/, "$$$1");
     const scripts = new Set();
     for (const m of text.matchAll(SCRIPT_RE)) {
         const rel = m[0]
-            .replace("${CLAUDE_PLUGIN_ROOT}/", "")
-            .replace(/^\$\{CLAUDE_PLUGIN_ROOT\}/, "")
+            .replaceAll(pluginRootToken, "")
+            .replaceAll(unbraced, "")
+            .replace(/^\/+/, "")
             .replace(/^\.\//, "");
         if ((0, node_fs_1.existsSync)((0, node_path_1.join)(basePath, rel)))
             scripts.add(rel);
     }
     return [...scripts];
 }
-function discoverHooks(basePath) {
+function discoverHooks(basePath, layout) {
     const scripts = new Set();
-    for (const m of [
-        ".claude-plugin/plugin.json",
-        ".claude/settings.json",
-        ".claude/settings.local.json",
-    ]) {
-        for (const s of hookScripts(basePath, m))
+    // The harness's manifest + settings (and a `.local` settings sibling, a CC
+    // convention that's harmless to probe elsewhere).
+    const localSettings = layout.settingsPath.replace(/(\.[^./]+)$/, ".local$1");
+    const manifests = [
+        ...new Set([layout.manifestPath, layout.settingsPath, localSettings]),
+    ];
+    for (const m of manifests) {
+        for (const s of hookScripts(basePath, m, layout.pluginRootToken))
             scripts.add(s);
     }
     return [...scripts].sort().map((path) => ({
@@ -185,15 +199,16 @@ function isCovered(surface, tests) {
  */
 function findUntestedSurfaces(options = {}) {
     const basePath = options.basePath ?? process.cwd();
+    const layout = options.layout ?? layout_js_1.claudeCodeLayout;
     const ignore = [...DEFAULT_IGNORE, ...(options.exclude ?? [])];
     const globs = options.testGlobs ?? DEFAULT_TEST_GLOBS;
     const surfaces = [];
     if (options.skills !== false)
-        surfaces.push(...discoverSkills(basePath, ignore));
+        surfaces.push(...discoverSkills(basePath, ignore, layout));
     if (options.agents !== false)
-        surfaces.push(...discoverAgents(basePath, ignore));
+        surfaces.push(...discoverAgents(basePath, ignore, layout));
     if (options.hooks !== false)
-        surfaces.push(...discoverHooks(basePath));
+        surfaces.push(...discoverHooks(basePath, layout));
     // Every skill/agent/hook is held to the requirement — only an explicit
     // `vigiles:ignore-test` marker exempts a surface (a visible, deliberate skip).
     const considered = surfaces.filter((s) => !s.ignored);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vigiles",
-  "version": "5.1.0",
+  "version": "5.2.0",
   "description": "Lint & test the harness your AI agent runs on — verify the references in your CLAUDE.md / AGENTS.md and test that your hooks and skills actually work.",
   "keywords": [
     "claude-code",
@@ -128,6 +128,7 @@
     "glob": "^13.0.6",
     "js-yaml": "^4.1.0",
     "minimatch": "^10.0.1",
+    "mvdan-sh": "^0.10.1",
     "ts-essentials": "^10.2.1",
     "typescript": "^5.9.3"
   }

package/skills/{migrate-to-spec → adopt-spec}/SKILL.md

@@ -1,13 +1,13 @@
 ---
-name: migrate-to-spec
-description: Convert an existing hand-written CLAUDE.md into a typed .spec.ts file for incremental adoption
+name: adopt-spec
+description: Adopt a typed .spec.ts for an existing hand-written CLAUDE.md — start from the file you already have, non-destructively
 disable-model-invocation: true
 argument-hint: <path to CLAUDE.md, defaults to CLAUDE.md>
 ---
 
-Convert an existing hand-written CLAUDE.md (or AGENTS.md) into a typed `CLAUDE.md.spec.ts` file. This is the incremental adoption path — you keep your existing instruction file as the starting point and get type safety going forward.
+Start a typed `CLAUDE.md.spec.ts` from an existing hand-written CLAUDE.md (or AGENTS.md). This is the non-destructive adoption path — you keep your existing instruction file as the starting point and get type safety going forward.
 
-> **Don't need full TypeScript?** A typed spec is the deepest commitment level. If the user only wants verified rules without a build step, point them at markdown mode first: inline `<!-- vigiles:enforce ... -->` comments (Level 0) or a `vigiles:` YAML frontmatter block with `vigiles generate-schema` for editor autocomplete (Level 1). Both are verified by `vigiles lint` with the same engine as a spec. See `docs/markdown-mode.md`. Migrate to a spec only when they want compiler-grade guarantees.
+> **Don't need full TypeScript?** A typed spec is the deepest commitment level. If the user only wants verified rules without a build step, point them at markdown mode first: inline `<!-- vigiles:enforce ... -->` comments (Level 0) or a `vigiles:` YAML frontmatter block with `vigiles generate-schema` for editor autocomplete (Level 1). Both are verified by `vigiles lint` with the same engine as a spec. See `docs/markdown-mode.md`. Adopt a spec only when they want compiler-grade guarantees.
 
 ## Instructions
 

package/skills/edit-spec/SKILL.md

@@ -27,7 +27,7 @@ Look for spec files in the repo root:
 - Any `*.spec.ts` matching instruction files
 
 If no spec exists: if there's a hand-written `CLAUDE.md`, suggest the
-`migrate-to-spec` skill; otherwise suggest `npx vigiles init` to scaffold one.
+`adopt-spec` skill; otherwise suggest `npx vigiles init` to scaffold one.
 
 ### Step 2: Read and Understand the Spec