@blumintinc/eslint-plugin-blumint 1.20.73 → 1.20.74

package/lib/index.js

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

package/lib/rules/global-const-style.js

@@ -3,6 +3,26 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const utils_1 = require("@typescript-eslint/utils");
 const createRule_1 = require("../utils/createRule");
 const isUpperSnakeCase = (str) => /^[A-Z][A-Z0-9_]*$/.test(str);
+/**
+ * Converts an identifier to UPPER_SNAKE_CASE by splitting on case *boundaries*.
+ *
+ * Idempotence is a correctness requirement, not a nicety: `--fix` re-lints its
+ * own output up to ten times per file, and a sibling rule can rewrite the same
+ * identifier in between (`enforce-react-type-naming` lowercases it), so a
+ * converter that re-separates what it already separated compounds every pass
+ * and writes an ever-growing, corrupted identifier into source (Issue #1605).
+ * Splitting on boundaries also keeps acronym runs intact, so `HTTPServer` reads
+ * as `HTTP_SERVER` rather than `H_T_T_P_SERVER`.
+ *
+ * The leading underscore is dropped because `_PRIVATE_THING` fails
+ * `isUpperSnakeCase`, which would leave the rule demanding a rename it can
+ * never satisfy.
+ */
+const toUpperSnakeCase = (name) => name
+    .replace(/([a-z0-9])([A-Z])/g, '$1_$2')
+    .replace(/([A-Z]+)([A-Z][a-z])/g, '$1_$2')
+    .toUpperCase()
+    .replace(/^_/, '');
 // Jest mock handles produced by an `as` cast to a `jest.Mock*` type are
 // stateful test doubles that are reassigned/mutated through
 // `.mockImplementation()`, `.mockReturnValue()`, etc. They are not immutable
@@ -328,10 +348,7 @@ exports.default = (0, createRule_1.createRule)({
                     // the `mockedX` idiom is intentional. The exemption gates only this
                     // rename check — the `as const` logic above is untouched.
                     if (!isUpperSnakeCase(name) && !isJestMockCast(init)) {
-                        const newName = name
-                            .replace(/([A-Z])/g, '_$1')
-                            .toUpperCase()
-                            .replace(/^_/, '');
+                        const newName = toUpperSnakeCase(name);
                         const idNode = declaration.id;
                         context.report({
                             node: declaration,

package/package.json

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

package/release-manifest.json

@@ -1,4 +1,18 @@
 [
+  {
+    "version": "1.20.74",
+    "date": "2026-08-02T06:07:00.009Z",
+    "rules": [
+      {
+        "name": "global-const-style",
+        "changeType": "fix",
+        "issues": [
+          1605
+        ],
+        "summary": "split the rename on case boundaries so it is idempotent (closes #1605)"
+      }
+    ]
+  },
   {
     "version": "1.20.73",
     "date": "2026-08-02T05:14:39.301Z",