@blumintinc/eslint-plugin-blumint 1.20.95 → 1.20.97

package/lib/utils/harvestRuleTesterCases.js

@@ -0,0 +1,168 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || function (mod) {
+    if (mod && mod.__esModule) return mod;
+    var result = {};
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    __setModuleDefault(result, mod);
+    return result;
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.harvestRuleTesterCases = void 0;
+const fs = __importStar(require("fs"));
+const os = __importStar(require("os"));
+const path = __importStar(require("path"));
+const sharedTesters = __importStar(require("./ruleTester"));
+const TESTS_DIR = path.join(__dirname, '..', 'tests');
+/**
+ * Only suites that import the shared tester module can declare a case, since
+ * `src/tests/no-local-rule-tester.test.ts` forbids a locally-built tester. That
+ * makes the import a sound admission test rather than a heuristic, and it is
+ * what keeps this affordable: the ~28 files it excludes are the meta-suites
+ * (`fixer-type-safety`, `docs-examples-conformance`, `rule-crash-robustness`,
+ * this guard's own siblings) which run full corpus sweeps at module scope and
+ * cost more to load than every rule suite combined.
+ *
+ * Matching the import rather than a `ruleTesterTs.run(` call site is
+ * deliberate: `prefer-next-dynamic.test.ts` aliases the tester
+ * (`const jsx = ruleTesterJsx`) before calling `run`, so a call-site pattern
+ * drops it.
+ */
+const IMPORTS_SHARED_TESTER = /from\s+'\.\.\/utils\/ruleTester'/;
+/**
+ * Jest registers a test for every `describe`/`it` a loaded module calls, so
+ * loading 271 suites inside a suite would graft their entire test list onto
+ * this one. Neutralizing the registrars for the duration of the load keeps the
+ * captured declarations and discards the registrations.
+ *
+ * `describe` bodies still execute — several suites call `run` inside one, and
+ * skipping the body would drop those cases — but everything that would register
+ * or assert becomes a no-op.
+ */
+const REGISTRAR_GLOBALS = [
+    'it',
+    'test',
+    'beforeEach',
+    'afterEach',
+    'beforeAll',
+    'afterAll',
+];
+/* eslint-disable @typescript-eslint/no-explicit-any */
+const asAny = (value) => value;
+const noop = () => undefined;
+/** `it.each(rows)(name, fn)` is a call chain, so the stub needs the same shape. */
+const withEach = (fn) => {
+    fn.each = () => () => undefined;
+    fn.only = fn;
+    fn.skip = fn;
+    fn.todo = noop;
+    fn.failing = fn;
+    fn.concurrent = fn;
+    return fn;
+};
+function harvestRuleTesterCases() {
+    const suites = [];
+    const failures = [];
+    let currentFile = '';
+    let filesLoaded = 0;
+    let filesSkipped = 0;
+    const testerEntries = Object.entries(sharedTesters).filter(([, value]) => typeof asAny(value)?.run === 'function');
+    const originalRun = new Map();
+    for (const [key, tester] of testerEntries) {
+        originalRun.set(key, tester.run);
+        tester.run = (name, rule, tests) => {
+            const bag = asAny(tests) || {};
+            suites.push({
+                name,
+                tester: key,
+                file: currentFile,
+                rule,
+                valid: bag.valid || [],
+                invalid: bag.invalid || [],
+            });
+            return undefined;
+        };
+    }
+    const globalScope = global;
+    const savedGlobals = new Map();
+    const stub = (key, value) => {
+        savedGlobals.set(key, globalScope[key]);
+        globalScope[key] = value;
+    };
+    stub('describe', withEach((_name, body) => {
+        if (typeof body === 'function')
+            body();
+    }));
+    for (const key of REGISTRAR_GLOBALS)
+        stub(key, withEach(noop));
+    /**
+     * A handful of suites write fixture files at module scope, and
+     * `test-file-location-enforcement` writes them under
+     * `path.join(process.cwd(), '.cursor/tmp/...')` — a path it also `rmSync`s in
+     * an `afterAll` that the stubs above turn into a no-op. Loading it from the
+     * real working directory would therefore race that suite when it runs
+     * concurrently in another worker (its cleanup landing between this harvest's
+     * `mkdirSync` and `writeFileSync` throws ENOENT) and would strand fixtures in
+     * the repo when it does not. Pointing the working directory at a private
+     * scratch root for the duration of the load sends every cwd-derived write
+     * somewhere no other worker can see, and it is removed below.
+     *
+     * The paths the suites *record* are unaffected: they are made relative to the
+     * same cwd they were built from, so the harvested filenames come out
+     * identical either way.
+     */
+    const scratchRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'blumint-harvest-'));
+    const realCwd = process.cwd();
+    process.chdir(scratchRoot);
+    try {
+        const files = fs
+            .readdirSync(TESTS_DIR)
+            .filter((file) => file.endsWith('.test.ts'))
+            .sort();
+        for (const file of files) {
+            const fullPath = path.join(TESTS_DIR, file);
+            if (!IMPORTS_SHARED_TESTER.test(fs.readFileSync(fullPath, 'utf8'))) {
+                filesSkipped++;
+                continue;
+            }
+            currentFile = file;
+            try {
+                require(fullPath);
+                filesLoaded++;
+            }
+            catch (error) {
+                failures.push(`${file}: ${error?.message}`);
+            }
+        }
+    }
+    finally {
+        process.chdir(realCwd);
+        fs.rmSync(scratchRoot, { recursive: true, force: true });
+        for (const [key, tester] of testerEntries) {
+            const original = originalRun.get(key);
+            if (original)
+                tester.run = original;
+        }
+        for (const [key, value] of savedGlobals)
+            globalScope[key] = value;
+    }
+    return { suites, failures, filesLoaded, filesSkipped };
+}
+exports.harvestRuleTesterCases = harvestRuleTesterCases;
+/* eslint-enable @typescript-eslint/no-explicit-any */
+//# sourceMappingURL=harvestRuleTesterCases.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@blumintinc/eslint-plugin-blumint",
-  "version": "1.20.95",
+  "version": "1.20.97",
   "description": "Custom eslint rules for use within BluMint",
   "author": {
     "name": "Brodie McGuire",

package/release-manifest.json

@@ -1,4 +1,155 @@
 [
+  {
+    "version": "1.20.97",
+    "date": "2026-08-04T08:50:14.394Z",
+    "rules": [
+      {
+        "name": "enforce-boolean-naming-prefixes",
+        "changeType": "fix",
+        "issues": [
+          1690,
+          1691
+        ],
+        "summary": "infer a callee's return from its body when the annotation is absent (closes #1691); accept a digit or $ fused onto an UPPER_SNAKE prefix (closes #1690)"
+      },
+      {
+        "name": "enforce-positive-naming",
+        "changeType": "fix",
+        "issues": [
+          1692
+        ],
+        "summary": "decline when a function's returns yield no verdict (closes #1692)"
+      }
+    ]
+  },
+  {
+    "version": "1.20.96",
+    "date": "2026-08-04T07:23:58.552Z",
+    "rules": [
+      {
+        "name": "enforce-css-media-queries",
+        "changeType": "fix",
+        "issues": [
+          1675
+        ],
+        "summary": "exempt capability queries and report once per usage (closes #1675)"
+      },
+      {
+        "name": "enforce-exported-function-types",
+        "changeType": "fix",
+        "issues": [
+          1677,
+          1686
+        ],
+        "summary": "read destructured props params and resolve memo(Identifier) components (closes #1686); see arrow and memo/forwardRef component shapes, autofix the type export (closes #1677)"
+      },
+      {
+        "name": "enforce-firestore-doc-ref-generic",
+        "changeType": "fix",
+        "issues": [
+          1678
+        ],
+        "summary": "resolve type-alias generics beside interfaces (closes #1678)"
+      },
+      {
+        "name": "enforce-mui-rounded-icons",
+        "changeType": "fix",
+        "issues": [
+          1674
+        ],
+        "summary": "rename the binding with the path it retargets (closes #1674)"
+      },
+      {
+        "name": "generic-starts-with-t",
+        "changeType": "fix",
+        "issues": [
+          1676
+        ],
+        "summary": "exempt module augmentations where the name is upstream-owned (closes #1676)"
+      },
+      {
+        "name": "global-const-style",
+        "changeType": "fix",
+        "issues": [
+          1681
+        ],
+        "summary": "exempt function-expression components and unwrap satisfies/non-null wrappers (closes #1681)"
+      },
+      {
+        "name": "no-always-true-false-conditions",
+        "changeType": "fix",
+        "issues": [
+          1682
+        ],
+        "summary": "resolve any const object in optional chains and climb assertion wrappers (closes #1682)"
+      },
+      {
+        "name": "no-unnecessary-verb-suffix",
+        "changeType": "fix",
+        "issues": [
+          1679
+        ],
+        "summary": "read intersection-alias contracts constituent by constituent (closes #1679)"
+      },
+      {
+        "name": "parallelize-loop-awaits",
+        "changeType": "fix",
+        "issues": [
+          1687,
+          1688
+        ],
+        "summary": "cover do-while, dedupe nested-loop reports, stop property keys vetoing (closes #1688); exempt test files, condition-coupled loops, and for-await-of (closes #1687)"
+      },
+      {
+        "name": "prefer-getter-over-parameterless-method",
+        "changeType": "fix",
+        "issues": [
+          1684
+        ],
+        "summary": "skip methods bound by a heritage clause (closes #1684)"
+      },
+      {
+        "name": "prefer-type-alias-over-typeof-constant",
+        "changeType": "fix",
+        "issues": [
+          1680
+        ],
+        "summary": "exempt only canonical alias derivations, not nested use sites (closes #1680)"
+      },
+      {
+        "name": "require-https-error",
+        "changeType": "fix",
+        "issues": [
+          1685
+        ],
+        "summary": "name the codebase-owned HttpsError module, not a placeholder package (closes #1685)"
+      },
+      {
+        "name": "use-custom-link",
+        "changeType": "fix",
+        "issues": [
+          1673
+        ],
+        "summary": "exempt the wrapper's implementation files (closes #1673)"
+      },
+      {
+        "name": "use-custom-memo",
+        "changeType": "fix",
+        "issues": [
+          1671
+        ],
+        "summary": "exempt the wrapper module the rule points at (closes #1671)"
+      },
+      {
+        "name": "use-custom-router",
+        "changeType": "fix",
+        "issues": [
+          1672
+        ],
+        "summary": "exempt the wrapper module the rule points at (closes #1672)"
+      }
+    ]
+  },
   {
     "version": "1.20.95",
     "date": "2026-08-03T23:44:33.733Z",