pr-checkmate 3.2.3 → 3.3.0

package/dist/esm/core/checks/dependencies/npm-audit.js

@@ -4,8 +4,8 @@ const __dirname = __dirnameFn(__fileURLToPath(import.meta.url));
 import fs from 'node:fs';
 import path from 'node:path';
 import { execa } from '../../../utils/execa.js';
-import { logger, findOwnPackageRoot } from '../../../utils/index.js';
-import { isNpmProject, pass, fail, skip } from '../../types.js';
+import { findOwnPackageRoot, logger } from '../../../utils/index.js';
+import { fail, isNpmProject, pass, skip } from '../../types.js';
 const SEVERITY_ORDER = ['info', 'low', 'moderate', 'high', 'critical'];
 function isSeverityAtLeast(severity, level) {
     const sevIdx = SEVERITY_ORDER.indexOf(severity);

package/dist/esm/core/checks/git/diff-smell.js

@@ -1,6 +1,6 @@
 import { logger } from '../../../utils/index.js';
 import { pass, skip } from '../../types.js';
-import { bucketTotal, emptyBuckets, mergeSeverityBuckets, reportCountedFindings, scanDiffPatterns, reviewLines, } from '../findings.js';
+import { bucketTotal, emptyBuckets, mergeSeverityBuckets, reportCountedFindings, reviewLines, scanDiffPatterns, } from '../findings.js';
 /**
  * Smells that are only smells in live code.
  *

package/dist/esm/core/checks/languages/cpp-format.js

@@ -4,7 +4,7 @@ import { getSourcePaths } from '../../../config/index.js';
 import { execa } from '../../../utils/execa.js';
 import { logger, resolvePackageBin } from '../../../utils/index.js';
 import { resolveScopedTargetFiles } from '../../targets.js';
-import { pass, warn, skip } from '../../types.js';
+import { pass, skip, warn } from '../../types.js';
 /**
  * C/C++ source & header extensions clang-format understands. `.h` is intentionally
  * included even though it is ambiguous with C — clang-format handles both.

package/dist/esm/core/checks/languages/lint.js

@@ -1,9 +1,10 @@
-import { hasLocalConfig, getSourcePaths, getPackageConfigPath, IGNORE_PATTERNS, } from '../../../config/index.js';
+import path from 'node:path';
+import { getPackageConfigPath, getSourcePaths, hasLocalConfig, IGNORE_PATTERNS, } from '../../../config/index.js';
 import { execa } from '../../../utils/execa.js';
-import { logger, resolvePackageBin } from '../../../utils/index.js';
+import { logger, registeredToolPaths, resolvePackageBin } from '../../../utils/index.js';
 import { clientToolPaths } from '../../install/reuse.js';
 import { resolveScopedTargetFiles } from '../../targets.js';
-import { pass, fail, warn, skip } from '../../types.js';
+import { fail, pass, skip, warn } from '../../types.js';
 /**
  * Resolve the ESLint binary under the client's `install.reuseLocal` policy.
  *
@@ -73,10 +74,18 @@ export const lintCheck = {
             logger.info(`[ESLint]: Using pr-checkmate config (ESLint v${version}, ${configFile})`);
         }
         IGNORE_PATTERNS.forEach(p => args.push('--ignore-pattern', p));
+        // Our bundled config loads its plugins through these roots, and an ESLint
+        // older than 9 resolves legacy-config plugins relative to the path below — in
+        // both cases the plugins are in a fetched profile rather than beside us.
+        const toolRoots = registeredToolPaths();
+        if (!hasLocal && version < 9 && toolRoots.length > 0) {
+            args.push('--resolve-plugins-relative-to', toolRoots[0]);
+        }
         const result = await execa(process.execPath, [eslintBin, ...args], {
             cwd: ctx.cwd,
             stdio: 'inherit',
             reject: false,
+            env: { PR_CHECKMATE_TOOL_ROOTS: toolRoots.join(path.delimiter) },
         });
         if (result.exitCode === 0)
             return pass();

package/dist/esm/core/checks/languages/prettier.js

@@ -1,10 +1,10 @@
 import fs from 'node:fs';
 import path from 'node:path';
-import { getPackageConfigPath, hasLocalConfig, IGNORE_PATTERNS, getSourcePaths, } from '../../../config/index.js';
+import { getPackageConfigPath, getSourcePaths, hasLocalConfig, IGNORE_PATTERNS, } from '../../../config/index.js';
 import { execa } from '../../../utils/execa.js';
-import { logger, cleanupTempFile, resolvePackageBin } from '../../../utils/index.js';
+import { cleanupTempFile, logger, resolvePackageBin } from '../../../utils/index.js';
 import { resolveScopedTargetFiles } from '../../targets.js';
-import { pass, warn, skip } from '../../types.js';
+import { pass, skip, warn } from '../../types.js';
 function loadUserPrettierConfig(cwd) {
     const configPath = path.join(cwd, 'pr-checkmate.json');
     if (!fs.existsSync(configPath))

package/dist/esm/core/checks/languages/python-format.js

@@ -2,8 +2,8 @@ import fs from 'node:fs';
 import path from 'node:path';
 import { logger } from '../../../utils/index.js';
 import { resolveTargetFiles } from '../../targets.js';
-import { pass, warn, skip } from '../../types.js';
-import { PY_GLOBS, createRuffWorkspace } from './ruff-shared.js';
+import { pass, skip, warn } from '../../types.js';
+import { createRuffWorkspace, PY_GLOBS } from './ruff-shared.js';
 /**
  * Formats changed Python files with Ruff's formatter (WASM, no Python/pip needed).
  * Read-only in check mode (`warn` when files differ from formatted output); rewrites

package/dist/esm/core/checks/languages/python-lint.js

@@ -2,8 +2,8 @@ import fs from 'node:fs';
 import path from 'node:path';
 import { logger } from '../../../utils/index.js';
 import { resolveTargetFiles } from '../../targets.js';
-import { pass, warn, skip } from '../../types.js';
-import { PY_GLOBS, createRuffWorkspace } from './ruff-shared.js';
+import { pass, skip, warn } from '../../types.js';
+import { createRuffWorkspace, PY_GLOBS } from './ruff-shared.js';
 /**
  * Lints changed Python files with Ruff, bundled as a self-contained WASM module
  * (`@astral-sh/ruff-wasm-nodejs`) so it runs on a bare Node image with no Python

package/dist/esm/core/checks/languages/ruff-shared.js

@@ -1,5 +1,6 @@
 import { createRequire as __createRequire } from 'node:module';
 const require = __createRequire(import.meta.url);
+import { resolvePackageBin } from '../../../utils/index.js';
 export const PY_GLOBS = ['*.py'];
 /**
  * The rules we lint Python with — pinned here rather than left to Ruff's defaults.
@@ -26,7 +27,11 @@ const SELECTED_RULES = ['E4', 'E7', 'E9', 'F'];
  * singleton) so the WASM module loads only when a check runs.
  */
 export function createRuffWorkspace() {
-    const { Workspace, PositionEncoding } = require('@astral-sh/ruff-wasm-nodejs');
+    // Resolved rather than required by name: the package lives in the `python`
+    // profile, which is fetched into the cache directory when this repository
+    // needs it, and a bare specifier only searches our own tree.
+    const entry = resolvePackageBin('@astral-sh/ruff-wasm-nodejs', 'ruff_wasm.js');
+    const { Workspace, PositionEncoding } = require(entry);
     const settings = {
         ...Workspace.defaultSettings(),
         lint: { select: SELECTED_RULES },

package/dist/esm/core/checks/languages/typecheck.js

@@ -3,7 +3,7 @@ import path from 'node:path';
 import { execa } from '../../../utils/execa.js';
 import { logger, resolvePackageBin } from '../../../utils/index.js';
 import { clientToolPaths } from '../../install/reuse.js';
-import { pass, fail, warn, skip } from '../../types.js';
+import { fail, pass, skip, warn } from '../../types.js';
 /**
  * Prefer the consumer project's own TypeScript so its tsconfig is compiled by the
  * version it was written for; fall back to the compiler bundled with pr-checkmate.

package/dist/esm/core/checks/pr/commitlint.js

@@ -34,8 +34,10 @@ const DEFAULT_IGNORES = ['^Release\\b', '^chore\\(release\\):'];
 function buildTempConfig(types, ignores) {
     // Emit a CJS module so `ignores` can hold functions — commitlint requires
     // predicate functions there, which a JSON config cannot express.
-    // Resolve config-conventional via Node module resolution — robust to hoisting.
-    const configConventional = require.resolve('@commitlint/config-conventional');
+    // Resolved through the same roots as the binary, not by bare specifier: both
+    // live in the `commitlint` profile, which is fetched into the cache directory,
+    // and `require.resolve` alone only searches our own tree.
+    const configConventional = resolvePackageBin('@commitlint/config-conventional', 'lib/index.js');
     const content = `const patterns = ${JSON.stringify(ignores)}.map((p) => new RegExp(p, 'i'));
 module.exports = {
   extends: [${JSON.stringify(configConventional)}],

package/dist/esm/core/checks/pr/pr-body.js

@@ -1,6 +1,6 @@
 import fs from 'node:fs';
 import { logger } from '../../../utils/index.js';
-import { pass, warn, skip } from '../../types.js';
+import { pass, skip, warn } from '../../types.js';
 // Matches "Closes #123", "Fixes #45", "Resolves #7" or JIRA-style "PROJ-456".
 const TICKET_PATTERN = /(?:closes?|fixes?|resolves?)\s+#\d+|[A-Z][A-Z0-9]+-\d+/i;
 export const prBodyCheck = {

package/dist/esm/core/checks/pr/pr-size-check.js

@@ -1,6 +1,6 @@
 import { execa } from '../../../utils/execa.js';
 import { logger } from '../../../utils/index.js';
-import { pass, warn, skip } from '../../types.js';
+import { pass, skip, warn } from '../../types.js';
 const DEFAULT_MAX_FILES = 50;
 const DEFAULT_MAX_LINES = 1000;
 export const prSizeCheck = {

package/dist/esm/core/checks/pr/pr-title-lint.js

@@ -1,6 +1,6 @@
 import fs from 'node:fs';
 import { logger } from '../../../utils/index.js';
-import { pass, warn, skip } from '../../types.js';
+import { pass, skip, warn } from '../../types.js';
 const CONVENTIONAL_PREFIX = /^(feat|fix|docs|style|refactor|test|chore|perf|ci|build|revert)(\(.+\))?!?: .+/;
 export const prTitleLintCheck = {
     name: 'PR Title',

package/dist/esm/core/checks/quality/dead-code.js

@@ -4,7 +4,7 @@ import { DEAD_CODE_IGNORE_DEFAULTS } from '../../../config/constants.js';
 import { execa } from '../../../utils/execa.js';
 import { logger, resolvePackageBin } from '../../../utils/index.js';
 import { clientToolPaths } from '../../install/reuse.js';
-import { pass, warn, skip } from '../../types.js';
+import { pass, skip, warn } from '../../types.js';
 /**
  * Resolve ts-unused-exports, preferring the consumer project's own copy (so a
  * repo pinned to a different major keeps its own behaviour) and falling back to

package/dist/esm/core/checks/quality/duplicate-check.js

@@ -6,7 +6,7 @@ import { getSourcePaths } from '../../../config/index.js';
 import { execa } from '../../../utils/execa.js';
 import { logger, resolvePackageBin } from '../../../utils/index.js';
 import { resolveScopedTargetFiles } from '../../targets.js';
-import { pass, warn, skip } from '../../types.js';
+import { pass, skip, warn } from '../../types.js';
 import { logSample } from '../findings.js';
 /**
  * The files to hand jscpd, per language.

package/dist/esm/core/checks/quality/markdown-lint.js

@@ -4,7 +4,7 @@ import { pathToFileURL } from 'node:url';
 import { GENERATED_DOC_FILES, getPackageConfigPath, hasLocalConfig, } from '../../../config/index.js';
 import { logger, resolvePackageBin } from '../../../utils/index.js';
 import { resolveTargetFiles } from '../../targets.js';
-import { pass, warn, skip } from '../../types.js';
+import { pass, skip, warn } from '../../types.js';
 function loadUserMarkdownConfig(cwd) {
     const configPath = path.join(cwd, 'pr-checkmate.json');
     if (!fs.existsSync(configPath))

package/dist/esm/core/checks/quality/package-size.d.ts

@@ -1,6 +1,27 @@
 import { Check } from '../../types.js';
+/** The slice of `npm pack --dry-run --json` this check reads. */
+interface PackedFile {
+    path: string;
+    size: number;
+}
+interface PackManifest {
+    name: string;
+    version: string;
+    size: number;
+    unpackedSize: number;
+    entryCount: number;
+    files: PackedFile[];
+}
+/**
+ * The manifest out of whatever `npm pack --json` printed.
+ *
+ * npm 10 prints `[ {manifest} ]`; npm 11 prints `{ "<name>": {manifest} }`. Reading
+ * only the first form turned every npm 11 runner into "could not describe this
+ * package" and the check into a permanent skip.
+ */
+export declare function pickManifest(parsed: unknown): PackManifest | null;
 /** Why we could not read a published version. */
-export type QueryFailure = 'absent' | 'unauthenticated' | 'unreadable';
+type QueryFailure = 'absent' | 'unauthenticated' | 'unreadable';
 /**
  * Tell "never published" apart from "we were not let in".
  *
@@ -34,3 +55,4 @@ export declare function classifyQueryFailure(stderr: string): QueryFailure;
  * { "Package Size": "error" }` makes it a gate for a repository that wants one.
  */
 export declare const packageSizeCheck: Check;
+export {};

package/dist/esm/core/checks/quality/package-size.js

@@ -8,6 +8,22 @@ import { logSample } from '../findings.js';
 const DEFAULT_MAX_GROWTH = 20;
 /** How many of the files behind the growth are named. */
 const NAMED_FILES = 5;
+/**
+ * The manifest out of whatever `npm pack --json` printed.
+ *
+ * npm 10 prints `[ {manifest} ]`; npm 11 prints `{ "<name>": {manifest} }`. Reading
+ * only the first form turned every npm 11 runner into "could not describe this
+ * package" and the check into a permanent skip.
+ */
+export function pickManifest(parsed) {
+    const entries = Array.isArray(parsed)
+        ? parsed
+        : parsed !== null && typeof parsed === 'object'
+            ? Object.values(parsed)
+            : [];
+    const first = entries[0];
+    return first !== null && typeof first === 'object' ? first : null;
+}
 /**
  * Ask npm what a tarball would contain, without producing one.
  *
@@ -41,10 +57,9 @@ async function packManifest(cwd, spec, registry) {
     if (result.exitCode !== 0)
         return { failure: classifyQueryFailure(result.stderr ?? '') };
     try {
-        const parsed = JSON.parse(result.stdout);
-        if (Array.isArray(parsed) && parsed.length > 0) {
-            return { manifest: parsed[0] };
-        }
+        const manifest = pickManifest(JSON.parse(result.stdout));
+        if (manifest)
+            return { manifest };
     }
     catch {
         // Fall through: npm answered with something that is not a manifest.

package/dist/esm/core/checks/quality/spellcheck.js

@@ -1,10 +1,10 @@
 import fs from 'node:fs';
 import path from 'node:path';
-import { getPackageConfigPath, hasLocalConfig, IGNORE_PATTERNS, getSourcePaths, } from '../../../config/index.js';
+import { getPackageConfigPath, getSourcePaths, hasLocalConfig, IGNORE_PATTERNS, } from '../../../config/index.js';
 import { execa } from '../../../utils/execa.js';
-import { logger, cleanupTempFile, matchesAnyGlob, resolvePackageBin, } from '../../../utils/index.js';
+import { cleanupTempFile, logger, matchesAnyGlob, registeredToolPaths, resolvePackageBin, } from '../../../utils/index.js';
 import { resolveScopedTargetFiles } from '../../targets.js';
-import { pass, warn, skip } from '../../types.js';
+import { pass, skip, warn } from '../../types.js';
 function loadUserCspellConfig(cwd) {
     const configPath = path.join(cwd, 'pr-checkmate.json');
     if (!fs.existsSync(configPath))
@@ -193,12 +193,15 @@ export function dropMissingLanguages(language, missing) {
  */
 function resolveFrom(cwd) {
     return id => {
-        try {
-            return require.resolve(id, { paths: [cwd] });
-        }
-        catch {
-            return require.resolve(id);
+        // The client's copy first, then the fetched `spellcheck` profile (cspell's own
+        // dictionaries live there), then our tree.
+        for (const root of [cwd, ...registeredToolPaths()]) {
+            try {
+                return require.resolve(id, { paths: [root] });
+            }
+            catch { }
         }
+        return require.resolve(id);
     };
 }
 function createTempCspellConfig(cwd) {

package/dist/esm/core/checks/security/diff-security.js

@@ -1,7 +1,7 @@
 import { DIFF_SECURITY_IGNORE_DEFAULTS } from '../../../config/index.js';
 import { logger } from '../../../utils/index.js';
 import { skip } from '../../types.js';
-import { clientFindingOptions, emptyBuckets, mergeSeverityBuckets, reportSeverityBuckets, scanDiffPatterns, reviewLines, DEFAULT_DATAFLOW_WINDOW, } from '../findings.js';
+import { clientFindingOptions, DEFAULT_DATAFLOW_WINDOW, emptyBuckets, mergeSeverityBuckets, reportSeverityBuckets, reviewLines, scanDiffPatterns, } from '../findings.js';
 const MD5_LABEL = 'MD5 (weak hash — use sha256 or stronger)';
 const SHA1_LABEL = 'SHA-1 (weak hash — use sha256 or stronger)';
 const SECURITY_PATTERNS = [

package/dist/esm/core/checks/security/security.js

@@ -4,7 +4,7 @@ import path from 'node:path';
 import { getPackageConfigPath, hasLocalConfig } from '../../../config/index.js';
 import { execa } from '../../../utils/execa.js';
 import { logger, resolveGitRef, resolvePackageBin } from '../../../utils/index.js';
-import { pass, fail, warn, skip } from '../../types.js';
+import { fail, pass, skip, warn } from '../../types.js';
 /**
  * A HOME the scanner can actually write to.
  *

package/dist/esm/core/install/doctor.js

@@ -5,6 +5,7 @@ import { logger } from '../../utils/logger.js';
 import { packageVersion } from '../../utils/package-version.js';
 import { resolveCacheDir, stalePaths } from './cache.js';
 import { cachedProfiles, ensureTooling } from './ensure.js';
+import { grypeDbState } from './grype-db.js';
 import { profileSatisfiedLocally } from './prepare.js';
 import { planProfiles, PROFILES } from './profiles.js';
 // Re-exported: `doctor` has been the home of this since ticket 064, and the
@@ -135,6 +136,17 @@ export async function runDoctor(cwd = process.cwd()) {
         lines.push(`  ${missing.length} profile(s) not installed yet. They are fetched on the next`);
         lines.push('  run, or now with: npx pr-checkmate install');
     }
+    if (config.grypeScan?.enabled !== false) {
+        const db = await grypeDbState(cwd);
+        lines.push('');
+        lines.push('  Grype database (shared by all projects on this machine)');
+        if (db.state === 'missing-tool')
+            lines.push('    — grype is not installed (brew install grype)');
+        else if (db.state === 'ready')
+            lines.push(`    ✅ up to date${db.built ? ` (built ${db.built})` : ''}`);
+        else
+            lines.push(`    ⬇️  stale${db.built ? ` (built ${db.built})` : ''} — refresh: npx pr-checkmate install`);
+    }
     const stale = stalePaths(packageVersion(), config.install?.cacheDir);
     if (stale.length > 0) {
         lines.push('');

package/dist/esm/core/install/grype-db.d.ts

@@ -0,0 +1,25 @@
+type GrypeDbState = {
+    state: 'missing-tool';
+} | {
+    state: 'ready';
+    built?: string;
+} | {
+    state: 'stale';
+    built?: string;
+};
+/**
+ * Is grype's vulnerability database usable right now? grype refuses one older than
+ * 5 days (exit code 1 from `db status`). The database lives in grype's own cache,
+ * shared by every project on the machine, so one refresh serves them all.
+ */
+export declare function grypeDbState(cwd: string): Promise<GrypeDbState>;
+/**
+ * Download a fresh database when the current one is missing or stale.
+ *
+ * `timeoutMs` bounds the download: a check inside a run passes one so it can never
+ * look like a hung commit, `install` passes none because a person asked for exactly
+ * this and the download is the whole job. Returns why it is still unusable, or
+ * `null` when it is ready.
+ */
+export declare function ensureGrypeDb(cwd: string, timeoutMs?: number): Promise<string | null>;
+export {};

package/dist/esm/core/install/grype-db.js

@@ -0,0 +1,41 @@
+import { execa } from '../../utils/execa.js';
+import { logger } from '../../utils/logger.js';
+/** `Built:` line of `grype db status`, so `doctor` can say how old the data is. */
+const builtOf = (stdout) => typeof stdout === 'string' ? /^Built:\s*(.+)$/m.exec(stdout)?.[1]?.trim() : undefined;
+/**
+ * Is grype's vulnerability database usable right now? grype refuses one older than
+ * 5 days (exit code 1 from `db status`). The database lives in grype's own cache,
+ * shared by every project on the machine, so one refresh serves them all.
+ */
+export async function grypeDbState(cwd) {
+    const status = await execa('grype', ['db', 'status'], { cwd, reject: false, timeout: 30000 });
+    if (status.code === 'ENOENT')
+        return { state: 'missing-tool' };
+    const built = builtOf(status.stdout);
+    return status.exitCode === 0 ? { state: 'ready', built } : { state: 'stale', built };
+}
+/**
+ * Download a fresh database when the current one is missing or stale.
+ *
+ * `timeoutMs` bounds the download: a check inside a run passes one so it can never
+ * look like a hung commit, `install` passes none because a person asked for exactly
+ * this and the download is the whole job. Returns why it is still unusable, or
+ * `null` when it is ready.
+ */
+export async function ensureGrypeDb(cwd, timeoutMs) {
+    const current = await grypeDbState(cwd);
+    if (current.state === 'missing-tool')
+        return 'ENOENT';
+    if (current.state === 'ready')
+        return null;
+    logger.info('[Grype Scan]: vulnerability database missing or stale — updating it');
+    const update = await execa('grype', ['db', 'update'], {
+        cwd,
+        reject: false,
+        ...(timeoutMs ? { timeout: timeoutMs } : {}),
+    });
+    if (update.timedOut) {
+        return `database update exceeded ${timeoutMs}ms — run "grype db update" once, then retry`;
+    }
+    return update.exitCode === 0 ? null : 'database update failed (offline?) — run "grype db update"';
+}

package/dist/esm/core/install/prepare.js

@@ -1,7 +1,7 @@
 import { logger } from '../../utils/logger.js';
 import { clearToolPaths, packagePresent, registerToolPaths } from '../../utils/resolve-bin.js';
 import { ensureTooling } from './ensure.js';
-import { PROFILES, planProfiles } from './profiles.js';
+import { planProfiles, PROFILES } from './profiles.js';
 /**
  * Whether a profile needs fetching at all, or already ships inside the package.
  *

package/dist/esm/core/install/profiles.d.ts

@@ -3,7 +3,7 @@ import type { Language, LanguageUsage } from '../../utils/detect-language.js';
 /**
  * A named bundle of tooling, and the checks that stop working without it.
  *
- * The package ships ~233 MB of tooling, and no repository needs all of it: a
+ * The tooling totals ~233 MB, and no repository needs all of it: a
  * Python service pays for the whole JavaScript toolchain, a repository with
  * Spellcheck switched off still pays for cspell. Profiles are how that stops —
  * they name which slice of the tooling a given repository actually needs.

package/dist/esm/core/install/profiles.js

@@ -20,9 +20,9 @@ export const PROFILES = [
             'typescript',
             'eslint',
             'eslint-config-prettier',
-            'eslint-plugin-import-x',
             'eslint-plugin-jsdoc',
             'eslint-plugin-prettier',
+            'eslint-plugin-simple-import-sort',
             'eslint-plugin-sonarjs',
             '@typescript-eslint/eslint-plugin',
             '@typescript-eslint/parser',

package/dist/esm/core/install/reuse.js

@@ -5,15 +5,18 @@ import { ownPackageRoot } from '../../utils/resolve-bin.js';
 /**
  * The major version we declare for a tool, from our own manifest.
  *
- * Read from `dependencies` rather than from the installed tree: what matters is
+ * Read from the manifest rather than from the installed tree: what matters is
  * the version our configuration was written against, and a hoisted install can
  * resolve to something else entirely.
  */
 function ownMajor(pkgName) {
     try {
         const manifest = path.join(ownPackageRoot(), 'package.json');
-        const deps = JSON.parse(fs.readFileSync(manifest, 'utf-8')).dependencies;
-        const range = deps?.[pkgName];
+        const parsed = JSON.parse(fs.readFileSync(manifest, 'utf-8'));
+        // Profile tooling is declared in `devDependencies` — it is fetched on demand
+        // rather than installed with us — but it is still the version our config was
+        // written against, and the manifest keeps both.
+        const range = parsed.dependencies?.[pkgName] ?? parsed.devDependencies?.[pkgName];
         if (!range)
             return null;
         // `^10.6.0`, `>=3.0.0`, `~5.1` — the leading number is the major in every

package/dist/esm/core/run-checks.js

@@ -1,10 +1,10 @@
 import { isCiEnvironment, logger } from '../utils/index.js';
 import { packageVersion } from '../utils/package-version.js';
-import { Semaphore, resolveConcurrency, runPool, withTimeout } from '../utils/pool.js';
+import { resolveConcurrency, runPool, Semaphore, withTimeout } from '../utils/pool.js';
 import { clearToolPaths } from '../utils/resolve-bin.js';
 import { ResultCollector } from '../utils/result-collector.js';
 import { buildContext } from './context.js';
-import { prepareTooling, noTooling } from './install/prepare.js';
+import { noTooling, prepareTooling } from './install/prepare.js';
 import { REGISTRY } from './registry.js';
 import { report } from './reporters.js';
 import { PHASES, } from './types.js';

package/dist/esm/init.d.ts

@@ -30,14 +30,6 @@ export declare function setupWorkflow(root: string, mode: GateMode): void;
  * heart about what should block.
  */
 export declare const BLOCKING_BY_DEFAULT: readonly ["Security Scan", "Sensitive Files", "Diff Security", "Workflow Security", "Merge Conflict", "NPM Audit", "License Check", "Dependencies", "TypeScript", "ESLint", "YAML Lint"];
-/**
- * Every option the template documents that the file does not already carry, at
- * its default. Without this, options such as `vulnScan` (off by default) or
- * `licenseCheck` exist only in the docs, and nobody edits what they cannot see.
- * Comment keys travel with the option they describe, section headers with the
- * first option after them.
- */
-export declare function templateExtras(existing: Record<string, unknown>): Record<string, unknown>;
 export declare function buildInitialConfig(sourcePath: string, languages: string[]): Record<string, unknown>;
 /**
  * Initialize (or refresh) pr-checkmate configuration in the current project.

package/dist/esm/init.js

@@ -2,11 +2,11 @@ import { fileURLToPath as __fileURLToPath } from 'node:url';
 import { dirname as __dirnameFn } from 'node:path';
 const __dirname = __dirnameFn(__fileURLToPath(import.meta.url));
 import fs from 'fs';
-import path from 'path';
 import inquirer from 'inquirer';
-import { DEAD_CODE_IGNORE_DEFAULTS, DEFAULT_COMMANDS, DIFF_SECURITY_IGNORE_DEFAULTS, documentation, DOCS_URL, DUPLICATE_IGNORE_DEFAULTS, FULL_CONFIG_URL, GENERATED_DOC_FILES, IGNORED_DIRS, } from './config/index.js';
+import path from 'path';
+import { DEAD_CODE_IGNORE_DEFAULTS, DEFAULT_COMMANDS, DIFF_SECURITY_IGNORE_DEFAULTS, DOCS_URL, documentation, DUPLICATE_IGNORE_DEFAULTS, FULL_CONFIG_URL, GENERATED_DOC_FILES, IGNORED_DIRS, } from './config/index.js';
 import { DEFAULT_DATAFLOW_WINDOW } from './core/checks/findings.js';
-import { logger, detectLanguages, findOwnPackageRoot } from './utils/index.js';
+import { detectLanguages, findOwnPackageRoot, logger } from './utils/index.js';
 const CONFIG_FILE = path.resolve(process.cwd(), 'pr-checkmate.json');
 /**
  * Points editors at the bundled JSON schema. With the generated config reduced
@@ -286,7 +286,7 @@ const GRYPE_SCAN_SEED = {
  * Comment keys travel with the option they describe, section headers with the
  * first option after them.
  */
-export function templateExtras(existing) {
+function templateExtras(existing) {
     const templatePath = path.join(__dirname, 'config', 'pr-checkmate.template.json');
     if (!fs.existsSync(templatePath)) {
         return {};

package/dist/esm/utils/own-deps.js

@@ -3,6 +3,7 @@ import { dirname as __dirnameFn } from 'node:path';
 const __dirname = __dirnameFn(__fileURLToPath(import.meta.url));
 import fs from 'node:fs';
 import path from 'node:path';
+import { PROFILES } from '../core/install/profiles.js';
 const OWN_PACKAGE_NAME = 'pr-checkmate';
 const PACKAGE_JSON = 'package.json';
 let cachedClosure;
@@ -85,7 +86,13 @@ export function getOwnDependencyClosure() {
     }
     // BFS over the dependency graph, resolving each package relative to the parent
     // that declared it so nested (non-hoisted) installs resolve correctly.
-    const queue = readDependencyNames(root).map(name => ({
+    //
+    // Profile tooling is ours even though it no longer sits in `dependencies`: it is
+    // fetched on demand, so the roots are what we declare plus what the profiles
+    // name. Without the second half every fetched tool is "the client's" and its
+    // licence blocks the client for a dependency they cannot remove.
+    const roots = new Set([...readDependencyNames(root), ...PROFILES.flatMap(p => p.packages)]);
+    const queue = [...roots].map(name => ({
         name,
         fromDir: root,
     }));

package/dist/esm/utils/resolve-bin.d.ts

@@ -16,6 +16,8 @@ export interface ResolvedTool {
  * to a run from before there was anything to fetch.
  */
 export declare function registerToolPaths(dirs: string[]): void;
+/** The roots registered for this run, for tools that load plugins by themselves. */
+export declare function registeredToolPaths(): string[];
 /** Drop every registered profile root. For tests, and for a second run in-process. */
 export declare function clearToolPaths(): void;
 /** The root of the installed pr-checkmate package — where our own tooling lives. */

package/dist/esm/utils/resolve-bin.js

@@ -25,6 +25,10 @@ let profileRoots = [];
 export function registerToolPaths(dirs) {
     profileRoots = [...dirs];
 }
+/** The roots registered for this run, for tools that load plugins by themselves. */
+export function registeredToolPaths() {
+    return [...profileRoots];
+}
 /** Drop every registered profile root. For tests, and for a second run in-process. */
 export function clearToolPaths() {
     profileRoots = [];

package/eslint-default-rules.json

@@ -26,14 +26,8 @@
   "sonarjs/cognitive-complexity": ["warn", 30],
   "sonarjs/no-duplicate-string": ["warn", { "threshold": 3 }],
   "sonarjs/no-identical-functions": "warn",
-  "import/order": [
-    "warn",
-    {
-      "groups": ["builtin", "external", "internal", "parent", "sibling", "index"],
-      "alphabetize": { "order": "asc", "caseInsensitive": true }
-    }
-  ],
-  "import/no-duplicates": "error",
+  "simple-import-sort/imports": "warn",
+  "no-duplicate-imports": ["error", { "allowSeparateTypeImports": true }],
   "jsdoc/check-param-names": "warn",
   "jsdoc/check-tag-names": "warn",
   "jsdoc/check-types": "warn",