vigiles 32.1.1 → 33.1.0

package/dist/test-coverage-files.js

@@ -34,22 +34,7 @@ exports.findUntestedSurfacesInFiles = findUntestedSurfacesInFiles;
 const posix_path_js_1 = require("./posix-path.js");
 const layout_js_1 = require("./core/layout.js");
 const coverage_evidence_js_1 = require("./coverage-evidence.js");
-// Mirrors src/test-coverage.ts constants. VALUES are re-declared, never imported
-// — test-coverage.ts pulls in node:fs/glob, and this twin must stay browser-safe.
-const DEFAULT_TEST_SUFFIXES = [
-    ".harness.ts",
-    ".harness.mts",
-    ".harness.cts",
-    ".harness.js",
-    ".harness.mjs",
-    ".harness.cjs",
-    ".eval.ts",
-    ".eval.mts",
-    ".eval.cts",
-    ".eval.js",
-    ".eval.mjs",
-    ".eval.cjs",
-];
+const source_kinds_js_1 = require("./source-kinds.js");
 const IGNORE_MARKER = "vigiles:ignore-test";
 /** Mirror of the DEFAULT_IGNORE globs (root-anchored), as a key predicate. */
 function isIgnored(key) {
@@ -190,7 +175,7 @@ function discoverTests(files) {
     for (const path of Object.keys(files)) {
         if (isIgnored(path))
             continue;
-        if (DEFAULT_TEST_SUFFIXES.some((s) => path.endsWith(s))) {
+        if ((0, source_kinds_js_1.isVigilesTest)((0, posix_path_js_1.basename)(path))) {
             out.push((0, coverage_evidence_js_1.prepareTest)(path));
         }
     }

package/dist/test-coverage.js

@@ -78,6 +78,7 @@ const ts_runner_caps_js_1 = require("./ts-runner-caps.js");
 const glob_1 = require("glob");
 const glob_ignore_js_1 = require("./core/glob-ignore.js");
 const exclude_js_1 = require("./exclude.js");
+const source_kinds_js_1 = require("./source-kinds.js");
 const layout_js_1 = require("./core/layout.js");
 const coverage_evidence_js_1 = require("./coverage-evidence.js");
 const coverage_artifact_js_1 = require("./coverage-artifact.js");
@@ -98,9 +99,6 @@ function surfaceGlobs(dir, leaf, materializeRoot) {
 // ---------------------------------------------------------------------------
 // Internals
 // ---------------------------------------------------------------------------
-/** Every extension Node executes directly. `.mts`/`.cts` are real (TS 4.7+) and
- * Node 22 strips their types with no toolchain — measured, not assumed. */
-const RUNNABLE_EXTS = "{ts,mts,cts,js,mjs,cjs}";
 /**
  * 🔴 `*.test.*` USED TO BE HERE, AND REMOVING IT IS THE POINT.
  *
@@ -128,10 +126,11 @@ const RUNNABLE_EXTS = "{ts,mts,cts,js,mjs,cjs}";
  * for hooks/agents and dropping them for skills — because a rule with a per-kind
  * exception is what this file just spent a day removing.
  */
-const DEFAULT_TEST_GLOBS = [
-    `**/*.harness.${RUNNABLE_EXTS}`,
-    `**/*.eval.${RUNNABLE_EXTS}`,
-];
+// Read off the table in `source-kinds.ts` — the same one `vigiles test`,
+// `vigiles eval` and the hook/provider discovery classify by — so the patterns
+// that credit a test and the files a hook directory refuses to compile cannot
+// drift apart.
+const DEFAULT_TEST_GLOBS = source_kinds_js_1.DEFAULT_TEST_GLOBS;
 const DEFAULT_IGNORE = [
     "node_modules/**",
     "dist/**",
@@ -298,6 +297,10 @@ function discoverTests(basePath, globs, ignore) {
         cwd: basePath,
         ignore,
         dot: true,
+        // `glob` defaults this to TRUE on macOS and Windows, so `a.hook.HARNESS.mjs`
+        // was a test there and unclaimed on Linux — and the classifier is
+        // case-sensitive. One answer on every OS.
+        nocase: false,
     });
     // Prepared ONCE per file (comment-strip + declaration parse), not once per
     // (surface × file) pair — the matching below is quadratic by nature.
@@ -737,7 +740,7 @@ function staleRunNote(report) {
     ];
 }
 /** Names a default vitest/jest run collects — the suffixes vigiles will not use. */
-const FOREIGN_RUNNER_SUFFIX = /\.(test|spec)\.(ts|mts|cts|js|mjs|cjs)$/;
+const FOREIGN_RUNNER_SUFFIX = new RegExp(`\\.(test|spec)\\.(${source_kinds_js_1.RUNNABLE_EXTS.join("|")})$`);
 /**
  * The would-be colocated tests that only their NAME disqualifies.
  *

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vigiles",
-  "version": "32.1.1",
+  "version": "33.1.0",
   "description": "Audit, test and measure the harness your AI agent runs on — grade your CLAUDE.md / AGENTS.md, skills, subagents and hooks, run them against a scripted model, and measure whether they actually fire.",
   "keywords": [
     "claude-code",
@@ -67,6 +67,8 @@
     "hooks",
     "skills",
     "!skills/**/*.spec.ts",
+    "packages/eslint-config/index.mjs",
+    "packages/eslint-config/README.md",
     "README.md",
     "LICENSE"
   ],
@@ -75,7 +77,7 @@
     "build:core": "tsc",
     "test": "npm run build && vitest run",
     "coverage": "npm run build && vitest run --coverage",
-    "lint": "eslint src/",
+    "lint": "eslint src/ && node scripts/lint-ceilings.mjs",
     "fmt": "prettier --write .",
     "fmt:check": "prettier --check .",
     "test:unit": "npm run build && vitest run --project unit",
@@ -112,7 +114,8 @@
     "conventional-changelog-conventionalcommits": "9.3.1",
     "eslint": "^10.1.0",
     "eslint-import-resolver-typescript": "^4.4.5",
-    "eslint-plugin-boundaries": "^6.0.2",
+    "eslint-plugin-boundaries": "^7.2.0",
+    "eslint-plugin-functional": "^10.0.1",
     "eslint-plugin-sonarjs": "^4.0.2",
     "globals": "^17.4.0",
     "jest": "^30.4.2",

package/packages/eslint-config/README.md

@@ -0,0 +1,17 @@
+# @vigiles/eslint-config (private)
+
+The strict lint rules vigiles and paperlint share: size limits, functional
+rules, no type assertions, and `layers()`, the hexagonal boundary.
+
+**This is not part of vigiles' API.** It has no entry in vigiles' `exports`, no
+version of its own, and it can change in any release without notice. It ships
+inside the `vigiles` npm package only so that paperlint can import it by path:
+
+```js
+// PRIVATE file of vigiles, not a public API. If it moves, this import fails loudly.
+import { layers } from "./node_modules/vigiles/packages/eslint-config/index.mjs";
+```
+
+Nobody else should depend on it. The plugins it uses (eslint-plugin-boundaries,
+eslint-plugin-functional, eslint-plugin-sonarjs, typescript-eslint) are not
+bundled; the importing repo installs its own.

package/packages/eslint-config/index.mjs

@@ -0,0 +1,344 @@
+// @ts-check
+/**
+ * The strict lint rules vigiles and paperlint share.
+ *
+ * `.mjs`, not `.ts`: ESLint loads a config without a build step, and paperlint
+ * imports this file by path from `node_modules/vigiles/packages/eslint-config/`.
+ * That import is deliberate and private: vigiles' `exports` has no entry for this
+ * file, so nobody else should depend on it.
+ *
+ * The plugins are not bundled. Each repo installs eslint-plugin-boundaries,
+ * eslint-plugin-functional and eslint-plugin-sonarjs itself, and the imports
+ * below resolve to that copy.
+ *
+ * What each repo keeps for itself: its folder layout (passed to `layers`), its
+ * ceiling table for old functions (passed to `ceilings`) and its
+ * eslint-suppressions.json.
+ */
+import boundaries from "eslint-plugin-boundaries";
+import functional from "eslint-plugin-functional";
+import sonarjs from "eslint-plugin-sonarjs";
+
+/** @typedef {import("eslint").Linter.Config} Config */
+/** @typedef {import("eslint").Linter.RuleEntry} RuleEntry */
+/** @typedef {{ files: string[], ignores?: string[] }} Scope */
+
+/** One collection library: remeda. These would be a second one for the same job. */
+export const ONE_COLLECTION_LIBRARY = ["lodash", "lodash-es", "ramda"].map(
+  (name) => ({ name, message: "Use remeda." }),
+);
+
+/** `x as unknown as T` switches the type checker off. For `no-restricted-syntax`. */
+export const AS_UNKNOWN_AS = {
+  selector:
+    "TSAsExpression > TSAsExpression[typeAnnotation.type='TSUnknownKeyword']",
+  message:
+    "`as unknown as` switches the type checker off. Convert with a function, or fix the type.",
+};
+
+/** A function is one job; at thirty lines a second one starts to hide. */
+const MAX_LINES = { max: 30, skipComments: true, skipBlankLines: true };
+
+/**
+ * Limits on the size of a function. These rules report one finding per function
+ * however far over the limit it is, so a suppressed finding would let the
+ * function keep growing. Old functions get a ceiling instead (see `ceilings`).
+ */
+export const SIZE_LIMITS = {
+  complexity: 10,
+  "sonarjs/cognitive-complexity": 10,
+  "max-depth": 3,
+  "max-params": 4,
+  "max-lines-per-function": MAX_LINES.max,
+  "max-nested-callbacks": 3,
+};
+
+/** @typedef {keyof typeof SIZE_LIMITS} SizeRule */
+
+/**
+ * @param {string} rule
+ * @param {number} n
+ * @returns {RuleEntry}
+ */
+const sizeRule = (rule, n) =>
+  rule === "max-lines-per-function"
+    ? ["error", { ...MAX_LINES, max: n }]
+    : ["error", n];
+
+/**
+ * The strict rules for TypeScript source. Needs the @typescript-eslint plugin registered.
+ *
+ * @param {string[]} files
+ * @returns {Config}
+ */
+export const strictTypeScript = (files) => ({
+  files,
+  plugins: { sonarjs },
+  rules: {
+    ...Object.fromEntries(
+      Object.entries(SIZE_LIMITS).map(([rule, n]) => [rule, sizeRule(rule, n)]),
+    ),
+    "@typescript-eslint/no-explicit-any": "error",
+    // `x as T` is a claim the checker takes on trust. Parse or narrow instead.
+    "@typescript-eslint/consistent-type-assertions": [
+      "error",
+      { assertionStyle: "never" },
+    ],
+    "@typescript-eslint/switch-exhaustiveness-check": [
+      "error",
+      { considerDefaultExhaustiveForUnions: true },
+    ],
+    // Not set here: `no-restricted-imports` and `no-restricted-syntax`. Flat config
+    // replaces a rule's options per file group, so a repo that sets either for its
+    // own reasons would silently drop ours. Merge ONE_COLLECTION_LIBRARY and
+    // AS_UNKNOWN_AS into the repo's own blocks instead.
+  },
+});
+
+/**
+ * A test's `describe` callback is a list of cases, not a function to split.
+ *
+ * @param {string[]} files
+ * @returns {Config}
+ */
+export const testSizes = (files) => ({
+  files,
+  rules: { "max-lines-per-function": sizeRule("max-lines-per-function", 60) },
+});
+
+/**
+ * Per-file ceilings for functions written before the limits: each number is the
+ * file's measured maximum, so its functions may shrink and may not grow. Lower a
+ * number after a refactor; a file leaves the table once it is under the limits.
+ *
+ * @param {Record<string, Partial<Record<SizeRule, number>>>} table
+ * @returns {Config[]}
+ */
+export const ceilings = (table) =>
+  Object.entries(table).map(([file, max]) => ({
+    files: [file],
+    rules: Object.fromEntries(
+      Object.entries(max).map(([rule, n]) => [rule, sizeRule(rule, n)]),
+    ),
+  }));
+
+/**
+ * No `let`, no mutation, no loops, readonly parameters. Tests may mutate their fixtures.
+ *
+ * @param {Scope} scope
+ * @returns {Config}
+ */
+export const functionalCode = ({ files, ignores }) => ({
+  files,
+  ignores,
+  plugins: { functional },
+  rules: {
+    "functional/no-let": "error",
+    "functional/immutable-data": "error",
+    "functional/no-loop-statements": "error",
+    "functional/prefer-immutable-types": [
+      "error",
+      { enforcement: "ReadonlyShallow", ignoreInferredTypes: true },
+    ],
+  },
+});
+
+/** Node modules that ARE effects. Only an `io` file, a test or the root may import one. */
+export const IO_MODULES = [
+  "fs",
+  "fs/promises",
+  "child_process",
+  "os",
+  "net",
+  "http",
+  "https",
+  "worker_threads",
+  "readline",
+  "readline/promises",
+].flatMap((m) => [m, `node:${m}`]);
+
+/** Libraries the domain and ports may use: types, pure path arithmetic, hashing, collections. */
+export const PURE_LIBRARIES = [
+  "ts-essentials",
+  "remeda",
+  "path",
+  "node:path",
+  "crypto",
+  "node:crypto",
+];
+
+/** What app files may use besides: Node's pure helpers. */
+export const APP_LIBRARIES = [
+  ...PURE_LIBRARIES,
+  "util",
+  "node:util",
+  "url",
+  "node:url",
+  "module",
+  "node:module",
+];
+
+const ORIGINS = ["external", "core"];
+/** @param {string[]} source */
+const libraries = (source) => ({ to: { module: { origin: ORIGINS, source } } });
+/** @param {...string} types */
+const elements = (...types) => ({ element: { types: { anyOf: types } } });
+const APP = { file: { categories: "app" } };
+const ROOT = { file: { categories: "root" } };
+
+/** Who may import whom: the domain knows only itself, an adapter never another adapter. */
+/**
+ * @param {string[]} domainLibraries
+ * @param {string[]} appLibraries
+ */
+const knowledgePolicies = (domainLibraries, appLibraries) => [
+  { from: elements("domain"), allow: { to: elements("domain") } },
+  { from: elements("port"), allow: { to: elements("domain", "port") } },
+  { from: APP, allow: { to: [elements("domain", "port"), APP] } },
+  { from: elements("adapter"), allow: { to: elements("domain", "port") } },
+  { from: ROOT, allow: { to: [{ element: { type: "*" } }, APP, ROOT] } },
+  { from: elements("domain", "port"), allow: libraries(domainLibraries) },
+  { from: APP, allow: libraries(appLibraries) },
+  {
+    from: [elements("adapter"), ROOT],
+    allow: { to: { module: { origin: ORIGINS } } },
+  },
+  {
+    from: { file: { categories: "test" } },
+    allow: {
+      to: [
+        { element: { type: "*" } },
+        APP,
+        ROOT,
+        { module: { origin: ORIGINS } },
+      ],
+    },
+  },
+];
+
+/**
+ * Effects only in io files, tests and the root. Written as "nobody, then these
+ * three": a file with no category has `categories: null`, and a `noneOf` query
+ * never matches null, so the negative form would exempt every plain file.
+ */
+/** @param {string} ioPattern */
+const effectPolicies = (ioPattern) => [
+  {
+    disallow: { to: { module: { origin: "core", source: IO_MODULES } } },
+    message: `{{ dependency.source }} is I/O in a file that is not ${ioPattern}. Take a port instead, and do the I/O in an adapter's io file or the root.`,
+  },
+  {
+    from: { file: { categories: ["io", "test", "root"] } },
+    allow: { to: { module: { origin: "core", source: IO_MODULES } } },
+  },
+];
+
+/**
+ * @param {string} type
+ * @param {string | undefined} pattern
+ * @param {Record<string, unknown>} [extra]
+ */
+const element = (type, pattern, extra = {}) =>
+  pattern === undefined
+    ? []
+    : [{ type, pattern, partialMatch: false, ...extra }];
+
+/**
+ * Where a repo keeps each layer. Patterns are relative to `root`.
+ *
+ * @typedef {object} Layout
+ * @property {string[]} files       the files the rules apply to
+ * @property {string} root          the repository directory
+ * @property {string} domain        folder of the domain, e.g. "src/core"
+ * @property {string} [ports]       folder of the ports, if the repo has one
+ * @property {string} adapters      adapter folders, one element each, e.g. "src/adapters/*"
+ * @property {string} cliRoot       the composition root: the only non-test file that may wire anything to anything
+ * @property {string} app           the flat application files, e.g. "src/*.ts"
+ * @property {string} tests         test files
+ * @property {string} io            files allowed to do I/O, e.g. "src/adapters/*\/*.io.ts"
+ * @property {string[]} [domainLibraries]  what the domain and ports may import (default PURE_LIBRARIES)
+ * @property {string[]} [appLibraries]     what app files may import (default APP_LIBRARIES)
+ * @property {object} [resolver]    import/resolver settings (default: the TypeScript resolver)
+ * @property {object[]} [extraElements]  more boundaries elements, e.g. a folder of plain-JS modules
+ * @property {object[]} [extraAllows]    more allow-policies, checked before the effects rule
+ */
+
+/**
+ * The hexagonal layers, as one eslint-plugin-boundaries block. Every linted file
+ * must be a declared element or category, so a new file cannot sit outside the
+ * rules.
+ *
+ * `root` must be the repository directory. Without `boundaries/root-path` the
+ * plugin matches patterns against `process.cwd()`, and lint started from any
+ * other directory classifies nothing and passes.
+ *
+ * @param {Layout} o
+ * @returns {Config}
+ */
+export const layers = (o) => ({
+  files: o.files,
+  // @ts-expect-error The plugin's types describe an ES module whose `default` is
+  // the plugin; Node hands a default import of this CommonJS file the plugin
+  // itself (keys meta, rules, configs). Runtime is right, the declaration is not.
+  plugins: { boundaries },
+  settings: {
+    "boundaries/root-path": o.root,
+    "import/resolver": o.resolver ?? { typescript: { alwaysTryTypes: true } },
+    "boundaries/elements": [
+      ...element("port", o.ports),
+      ...element("domain", o.domain),
+      ...element("adapter", o.adapters, { capture: ["name"] }),
+      ...(o.extraElements ?? []),
+    ],
+    "boundaries/files": [
+      { category: "root", pattern: o.cliRoot, exclusive: true },
+      { category: "app", pattern: o.app },
+      { category: "test", pattern: o.tests },
+      { category: "io", pattern: o.io },
+    ],
+  },
+  rules: {
+    "boundaries/no-unknown-files": "error",
+    "boundaries/no-unknown-dependencies": "error",
+    "boundaries/dependencies": [
+      "error",
+      {
+        default: "disallow",
+        checkAllOrigins: true,
+        policies: [
+          ...knowledgePolicies(
+            o.domainLibraries ?? PURE_LIBRARIES,
+            o.appLibraries ?? APP_LIBRARIES,
+          ),
+          ...(o.extraAllows ?? []),
+          ...effectPolicies(o.io),
+        ],
+      },
+    ],
+  },
+});
+
+/**
+ * `process` and `fetch` are not imports, so `layers` cannot see them. Same scope as its effects rule.
+ *
+ * @param {Scope} scope
+ * @returns {Config}
+ */
+export const ioGlobals = ({ files, ignores }) => ({
+  files,
+  ignores,
+  rules: {
+    "no-restricted-globals": [
+      "error",
+      {
+        name: "process",
+        message:
+          "The environment is an input: the root reads it and passes a value in.",
+      },
+      {
+        name: "fetch",
+        message: "Network access belongs in an adapter's io file.",
+      },
+    ],
+  },
+});