@blumintinc/eslint-plugin-blumint 1.20.192 → 1.20.194

package/lib/index.js

@@ -223,7 +223,7 @@ function noFrontendImportsFromFunctionsPatterns(pattern) {
 module.exports = {
     meta: {
         name: '@blumintinc/eslint-plugin-blumint',
-        version: '1.20.192',
+        version: '1.20.194',
     },
     parseOptions: {
         ecmaVersion: 2020,

package/lib/rules/enforce-memoize-getters.js

@@ -423,6 +423,19 @@ exports.enforceMemoizeGetters = (0, createRule_1.createRule)({
                 // breakage.
                 if (node.key.type === utils_1.AST_NODE_TYPES.PrivateIdentifier)
                     return;
+                // The same unwritable-remedy shape, one key form over: `Memoize`
+                // declares its `propertyKey` parameter as `string`, so decorating a
+                // symbol-keyed getter — `get [Symbol.iterator]()` — is TS1241, and the
+                // `--fix` edit turns compiling code into code that does not compile.
+                // A computed key that IS a string literal (`get ['name']()`) is an
+                // ordinary string key and stays in scope; anything else cannot be shown
+                // to be a string without the checker, and a false negative is the
+                // cheaper error here.
+                if (node.computed &&
+                    !(node.key.type === utils_1.AST_NODE_TYPES.Literal &&
+                        typeof node.key.value === 'string')) {
+                    return;
+                }
                 const classBody = node.parent;
                 // The same reasoning, one level out: under `experimentalDecorators` a
                 // decorator is rejected on EVERY member of a class EXPRESSION — TS1206

package/lib/rules/enforce-typescript-markdown-code-blocks.js

@@ -16,6 +16,10 @@ const MIN_FENCE_LENGTH = 3;
 const MAX_FENCE_INDENT_COLUMNS = 3;
 /** CommonMark advances a tab to the next multiple of four when measuring indent. */
 const TAB_STOP = 4;
+/** The three bullet characters CommonMark accepts for an unordered list item. */
+const BULLET_MARKERS = new Set(['-', '+', '*']);
+/** CommonMark caps an ordered list marker at nine digits before its delimiter. */
+const MAX_ORDERED_MARKER_DIGITS = 9;
 function splitLines(text) {
     const lines = [];
     let start = 0;
@@ -72,6 +76,106 @@ function readFence(line) {
         infoString: line.text.slice(offset + runLength),
     };
 }
+/**
+ * Reads the list marker at `offset`, including the whitespace that separates
+ * it from the item's content, or 0 when no marker starts there. CommonMark
+ * requires that separator, which is what keeps `*emphasis*` and the thematic
+ * break `***` from reading as list items.
+ */
+function readListMarker(text, offset) {
+    let cursor = offset;
+    const char = text[cursor];
+    if (char !== undefined && BULLET_MARKERS.has(char)) {
+        cursor += 1;
+    }
+    else {
+        let digits = 0;
+        while (digits < MAX_ORDERED_MARKER_DIGITS &&
+            text[cursor + digits] >= '0' &&
+            text[cursor + digits] <= '9') {
+            digits += 1;
+        }
+        const delimiter = text[cursor + digits];
+        if (digits === 0 || (delimiter !== '.' && delimiter !== ')')) {
+            return 0;
+        }
+        cursor += digits + 1;
+    }
+    if (text[cursor] !== ' ' && text[cursor] !== '\t') {
+        return 0;
+    }
+    while (text[cursor] === ' ' || text[cursor] === '\t') {
+        cursor += 1;
+    }
+    return cursor - offset;
+}
+/**
+ * Reads a fence opened on the same line as one or more list markers, as in
+ * "- ```ts". A list item's content begins after its marker, so the backticks
+ * open a block there exactly as they would at the head of a line.
+ *
+ * `readFence` cannot see one, because it stops at the first character that is
+ * neither a space nor a tab. Without this the scanner walks INTO such a block
+ * and mistakes its closing fence — which is nothing but spaces and backticks —
+ * for an opening one, appending a language to literal document content.
+ *
+ * The block is skipped whole rather than labeled: a fix would have to be
+ * written past the marker, and labeling a block this rule cannot delimit at
+ * the head of a line is the false-negative direction it deliberately keeps.
+ */
+function readListItemFence(line) {
+    let indentColumns = 0;
+    let offset = 0;
+    let sawMarker = false;
+    for (;;) {
+        while (offset < line.text.length) {
+            const char = line.text[offset];
+            if (char === ' ') {
+                indentColumns += 1;
+            }
+            else if (char === '\t') {
+                indentColumns += TAB_STOP - (indentColumns % TAB_STOP);
+            }
+            else {
+                break;
+            }
+            offset += 1;
+        }
+        if (indentColumns > MAX_FENCE_INDENT_COLUMNS) {
+            return null;
+        }
+        const markerLength = readListMarker(line.text, offset);
+        if (markerLength === 0) {
+            break;
+        }
+        offset += markerLength;
+        // The item's content starts a fresh indent budget, so a fence sitting at
+        // the content column is at indent zero however deep the nesting is.
+        indentColumns = 0;
+        sawMarker = true;
+    }
+    if (!sawMarker) {
+        return null;
+    }
+    const marker = line.text[offset];
+    if (marker !== BACKTICK && marker !== TILDE) {
+        return null;
+    }
+    let runLength = 0;
+    while (line.text[offset + runLength] === marker) {
+        runLength += 1;
+    }
+    if (runLength < MIN_FENCE_LENGTH) {
+        return null;
+    }
+    return {
+        runStart: line.start + offset,
+        marker,
+        runLength,
+        indent: line.text.slice(0, offset),
+        infoString: line.text.slice(offset + runLength),
+    };
+}
 /**
  * The line a block closes on, per CommonMark: a run of at least the opening
  * length, of the SAME marker, at a fence indent, with nothing but whitespace
@@ -131,7 +235,21 @@ exports.enforceTypescriptMarkdownCodeBlocks = (0, createRule_1.createRule)({
                     const openingLine = lines[index];
                     const fence = readFence(openingLine);
                     if (fence === null) {
-                        index += 1;
+                        const listFence = readListItemFence(openingLine);
+                        if (listFence === null) {
+                            index += 1;
+                            continue;
+                        }
+                        // A fence opened on a list marker's line is one this rule declines
+                        // to label, and declining to label a block means declining to read
+                        // it. Skipping it whole is what keeps its closing fence — spaces
+                        // and backticks, indistinguishable from an opener — out of the
+                        // walk.
+                        const listClosing = findFenceCloser(lines, index + 1, listFence);
+                        if (listClosing === null) {
+                            return;
+                        }
+                        index = listClosing.line + 1;
                         continue;
                     }
                     const closing = findFenceCloser(lines, index + 1, fence);

package/lib/rules/no-direct-function-state.d.ts

@@ -4,6 +4,6 @@ type Options = [
         functionPatterns?: string[];
     }
 ];
-type MessageIds = 'noDirectFunctionState' | 'noDirectFunctionStateAssertion';
+type MessageIds = 'noDirectFunctionState' | 'noDirectFunctionStateAssertion' | 'invalidFunctionPattern';
 export declare const noDirectFunctionState: TSESLint.RuleModule<MessageIds, Options, TSESLint.RuleListener>;
 export {};

package/lib/rules/no-direct-function-state.js

@@ -198,22 +198,33 @@ function isDefinitelySafeArg(argNode) {
     }
 }
 /**
- * Checks whether an identifier name matches any of the function-naming patterns
- * (e.g. onClose, handler, fn, callback).
+ * Compiles the configured function-naming patterns once, separating the ones
+ * that do not compile from the ones that do.
+ *
+ * Swallowing an uncompilable pattern makes the consumer's allowlist silently
+ * inert: the rule then reports the very code they wrote the pattern to exclude,
+ * with nothing anywhere saying why. Returning the rejects lets `create` report
+ * them, which is what the sibling pattern-compiling rules already do.
  */
-function matchesFunctionPattern(name, patterns) {
+function compileFunctionPatterns(patterns) {
+    const matchers = [];
+    const invalid = [];
     for (const pattern of patterns) {
         try {
-            const regex = new RegExp(`^${pattern}$`);
-            if (regex.test(name)) {
-                return true;
-            }
+            matchers.push(new RegExp(`^${pattern}$`));
         }
         catch {
-            // Ignore invalid regex patterns
+            invalid.push(pattern);
         }
     }
-    return false;
+    return { matchers, invalid };
+}
+/**
+ * Checks whether an identifier name matches any of the function-naming patterns
+ * (e.g. onClose, handler, fn, callback).
+ */
+function matchesFunctionPattern(name, matchers) {
+    return matchers.some((matcher) => matcher.test(name));
 }
 /**
  * Extracts the identifier name from an argument node for pattern matching.
@@ -296,12 +307,16 @@ exports.noDirectFunctionState = (0, createRule_1.createRule)({
                 'Why it matters: The function will be called with the previous state value and its return value stored — a silent bug with no error. ' +
                 'How to fix: Give the asserted value a name, then store that name through a thunk: const value = {{argText}}; {{setterName}}(() => value). ' +
                 'The assertion is hoisted out because a thunk that returned it would be an arrow returning a cast, which no-type-assertion-returns reports.',
+            invalidFunctionPattern: 'What\u2019s wrong: "{{pattern}}" in functionPatterns is not a valid regular expression, so it was dropped. ' +
+                'Why it matters: the rule silently stops honouring that entry, and reports the very code the pattern was written to exclude. ' +
+                'How to fix: correct the pattern in your ESLint configuration.',
         },
     },
     defaultOptions: [{ functionPatterns: DEFAULT_FUNCTION_PATTERNS }],
     create(context) {
         const options = context.options[0] ?? {};
         const functionPatterns = options.functionPatterns ?? DEFAULT_FUNCTION_PATTERNS;
+        const { matchers: functionPatternMatchers, invalid: invalidPatterns } = compileFunctionPatterns(functionPatterns);
         /**
          * Maps setter-variable names to whether the corresponding useState has
          * an explicit function type parameter. This is populated as we encounter
@@ -309,6 +324,15 @@ exports.noDirectFunctionState = (0, createRule_1.createRule)({
          */
         const setterFunctionTyped = new Map();
         return {
+            Program(node) {
+                for (const pattern of invalidPatterns) {
+                    context.report({
+                        node,
+                        messageId: 'invalidFunctionPattern',
+                        data: { pattern },
+                    });
+                }
+            },
             VariableDeclarator(node) {
                 // Look for `const [state, setter] = useState<T>(...)` or
                 // `const [state, setter] = React.useState<T>(...)`.
@@ -383,7 +407,8 @@ exports.noDirectFunctionState = (0, createRule_1.createRule)({
                 // No explicit function type. Fall back to heuristic: name pattern match
                 // or scope-level binding to a function.
                 const argName = getArgName(arg);
-                if (argName && matchesFunctionPattern(argName, functionPatterns)) {
+                if (argName &&
+                    matchesFunctionPattern(argName, functionPatternMatchers)) {
                     reportAndFix(node, arg, setterName, context);
                     return;
                 }

package/package.json

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

package/release-manifest.json

@@ -1,4 +1,40 @@
 [
+  {
+    "version": "1.20.194",
+    "date": "2026-08-30T09:11:52.116Z",
+    "rules": [
+      {
+        "name": "enforce-typescript-markdown-code-blocks",
+        "changeType": "fix",
+        "issues": [
+          2220
+        ],
+        "summary": "skip a fence opened on a list marker's line (closes #2220)"
+      }
+    ]
+  },
+  {
+    "version": "1.20.193",
+    "date": "2026-08-30T05:32:04.963Z",
+    "rules": [
+      {
+        "name": "enforce-memoize-getters",
+        "changeType": "fix",
+        "issues": [
+          2215
+        ],
+        "summary": "decline a getter whose computed key is not a string literal"
+      },
+      {
+        "name": "no-direct-function-state",
+        "changeType": "fix",
+        "issues": [
+          2218
+        ],
+        "summary": "report an uncompilable functionPatterns entry instead of dropping it (closes #2218)"
+      }
+    ]
+  },
   {
     "version": "1.20.192",
     "date": "2026-08-30T01:20:08.891Z",