@blumintinc/eslint-plugin-blumint 1.20.35 → 1.20.37

package/lib/index.js

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

package/lib/rules/enforce-unique-cursor-headers.js

@@ -6,6 +6,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.enforceUniqueCursorHeaders = exports.RULE_NAME = void 0;
 const path_1 = __importDefault(require("path"));
 const minimatch_1 = require("minimatch");
+const utils_1 = require("@typescript-eslint/utils");
 const createRule_1 = require("../utils/createRule");
 const DEFAULT_OPTIONS = {
     requiredPatterns: ['**/*.{ts,tsx,js,jsx}'],
@@ -212,6 +213,62 @@ const analyzeHeaderGroups = (candidateGroups, options) => {
         });
     return { primaryHeader, duplicateGroups, splitHeaderGroups };
 };
+/**
+ * A template made of nothing but comments is safe to prepend to any file;
+ * anything else (bare text, trailing code) would be inserted as source and can
+ * leave the file unparseable.
+ */
+const isCommentOnly = (template) => template
+    .replace(/\/\*[\s\S]*?\*\//gu, '')
+    .replace(/\/\/[^\n]*/gu, '')
+    .trim().length === 0;
+const toTemplateComment = (template, raw, offset) => {
+    const isBlock = raw.startsWith('/*');
+    const precedingText = template.slice(0, offset);
+    const startLine = precedingText.split('\n').length;
+    const startColumn = offset - (precedingText.lastIndexOf('\n') + 1);
+    const rawLines = raw.split('\n');
+    const lastLine = rawLines[rawLines.length - 1];
+    return {
+        type: isBlock ? utils_1.AST_TOKEN_TYPES.Block : utils_1.AST_TOKEN_TYPES.Line,
+        /** Header detection normalizes the raw comment value, so drop only the delimiters. */
+        value: isBlock ? raw.slice(2, -2) : raw.slice(2),
+        range: [offset, offset + raw.length],
+        loc: {
+            start: { line: startLine, column: startColumn },
+            end: {
+                line: startLine + rawLines.length - 1,
+                column: rawLines.length > 1 ? lastLine.length : startColumn + raw.length,
+            },
+        },
+    };
+};
+const parseTemplateComments = (template) => {
+    const commentPattern = /\/\*[\s\S]*?\*\/|\/\/[^\n]*/gu;
+    const comments = [];
+    for (let match = commentPattern.exec(template); match !== null; match = commentPattern.exec(template)) {
+        comments.push(toTemplateComment(template, match[0], match.index));
+    }
+    return comments;
+};
+/**
+ * A fixer must remove its own trigger. Inserting a template the rule itself
+ * would not accept as a header leaves `missingHeader` reported, so ESLint's fix
+ * loop re-inserts the template on every pass until it hits the pass ceiling.
+ *
+ * Whether a template is acceptable is not a property of its text alone: the
+ * rule groups comments first and demands that a single group carry every tag,
+ * so `allowSplitHeaders` and block adjacency decide the answer. Running the
+ * template through the same grouping the rule applies to real files keeps the
+ * fixer convergent by construction instead of by a second approximation.
+ */
+const canTemplateSatisfyRule = (template, options, excludedAtDirectives) => {
+    if (options.requiredTags.length === 0 || !isCommentOnly(template)) {
+        return false;
+    }
+    const candidateGroups = collectHeaderGroups(parseTemplateComments(template), options, excludedAtDirectives);
+    return (analyzeHeaderGroups(candidateGroups, options).primaryHeader !== undefined);
+};
 const computeHeaderInsertion = (sourceText, headerTemplate) => {
     const headerText = buildHeaderInsertionText(headerTemplate);
     if (!sourceText.startsWith('#!')) {
@@ -225,8 +282,7 @@ const computeHeaderInsertion = (sourceText, headerTemplate) => {
     /** Insert after shebang line */
     return { index: shebangNewlineIndex + 1, text: headerText };
 };
-const reportMissingHeader = (context, fileName, sourceText, options) => {
-    const template = options.headerTemplate;
+const reportMissingHeader = (context, fileName, sourceText, options, template) => {
     context.report({
         loc: { line: 1, column: 0 },
         messageId: 'missingHeader',
@@ -331,6 +387,10 @@ exports.enforceUniqueCursorHeaders = (0, createRule_1.createRule)({
         const fileName = context.getFilename();
         const matchPath = fileName.split(path_1.default.sep).join('/');
         const excludedAtDirectives = new Set(options.excludedAtDirectives);
+        const fixableTemplate = options.headerTemplate !== null &&
+            canTemplateSatisfyRule(options.headerTemplate, options, excludedAtDirectives)
+            ? options.headerTemplate
+            : null;
         if (fileName === '<input>') {
             return {};
         }
@@ -353,7 +413,7 @@ exports.enforceUniqueCursorHeaders = (0, createRule_1.createRule)({
                 const candidateGroups = collectHeaderGroups(topComments, options, excludedAtDirectives);
                 const { primaryHeader, duplicateGroups, splitHeaderGroups } = analyzeHeaderGroups(candidateGroups, options);
                 if (!primaryHeader) {
-                    reportMissingHeader(context, fileName, sourceText, options);
+                    reportMissingHeader(context, fileName, sourceText, options, fixableTemplate);
                     return;
                 }
                 reportDuplicates(context, duplicateGroups, sourceText);

package/lib/rules/no-margin-properties.d.ts

@@ -1,7 +1 @@
-type Options = [
-    {
-        autofix?: boolean;
-    }
-];
-export declare const noMarginProperties: import("@typescript-eslint/utils/dist/ts-eslint/Rule").RuleModule<"noMarginProperties", Options, import("@typescript-eslint/utils/dist/ts-eslint/Rule").RuleListener>;
-export {};
+export declare const noMarginProperties: import("@typescript-eslint/utils/dist/ts-eslint/Rule").RuleModule<"noMarginProperties", [], import("@typescript-eslint/utils/dist/ts-eslint/Rule").RuleListener>;

package/lib/rules/no-margin-properties.js

@@ -42,27 +42,14 @@ exports.noMarginProperties = (0, createRule_1.createRule)({
             description: 'Prevent margin properties (margin, marginLeft, marginRight, marginTop, marginBottom, mx, my, etc.) in MUI styling because margins fight container-controlled spacing, double gutters, and misaligned breakpoints; keep spacing centralized with padding, gap, or spacing props instead.',
             recommended: 'error',
         },
-        schema: [
-            {
-                type: 'object',
-                properties: {
-                    autofix: {
-                        type: 'boolean',
-                        default: false,
-                    },
-                },
-                additionalProperties: false,
-            },
-        ],
+        schema: [],
         messages: {
             noMarginProperties: 'Margin property "{{property}}" in MUI styling fights container-controlled spacing (Stack/Grid spacing, gap, responsive gutters) and produces double gutters, misalignment, and overflow as layouts shift. Keep spacing inside the component with padding or let the parent handle separation via gap/spacing so layout remains predictable.',
         },
     },
-    defaultOptions: [{ autofix: false }],
+    defaultOptions: [],
     create(context) {
         const seenNodes = new WeakSet();
-        // Note: autofix option is available but not currently implemented
-        // Future implementation can use: const { autofix = false } = _options;
         function checkProperty(propertyName) {
             const normalizedName = normalizePropertyName(propertyName);
             return MARGIN_PROPERTIES.has(normalizedName);

package/package.json

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

package/release-manifest.json

@@ -1,4 +1,32 @@
 [
+  {
+    "version": "1.20.37",
+    "date": "2026-07-30T18:07:02.175Z",
+    "rules": [
+      {
+        "name": "enforce-unique-cursor-headers",
+        "changeType": "fix",
+        "issues": [
+          1461
+        ],
+        "summary": "only autofix a template that satisfies the rule (closes #1461)"
+      }
+    ]
+  },
+  {
+    "version": "1.20.36",
+    "date": "2026-07-30T17:05:52.383Z",
+    "rules": [
+      {
+        "name": "no-margin-properties",
+        "changeType": "fix",
+        "issues": [
+          1460
+        ],
+        "summary": "remove the inert autofix option (closes #1460)"
+      }
+    ]
+  },
   {
     "version": "1.20.35",
     "date": "2026-07-30T16:38:05.368Z",