@blumintinc/eslint-plugin-blumint 1.18.9 → 1.18.11

package/lib/index.js

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

package/lib/rules/parallelize-async-operations.js

@@ -169,7 +169,43 @@ exports.parallelizeAsyncOperations = (0, createRule_1.createRule)({
          * "transactionCollector", etc. This may produce false positives for
          * unrelated managers, but errs on the side of safety.
          */
-        const COORDINATOR_PATTERN = /batch|manager|collector|transaction|tx|unitofwork|accumulator/i;
+        const COORDINATOR_PATTERN = /batch|manager|collector|transaction|tx|coordinator|unitofwork|accumulator|aggregator/i;
+        /**
+         * Matches guard/assertion callees by their leading verb. Anchored at the
+         * start so it fires on the callee's own verb (assertStartable,
+         * validateInput, ensureExists, requireAuth, checkAccess, verifyOwnership,
+         * guardAgainstX) rather than on an arbitrary substring elsewhere in the
+         * name.
+         */
+        const GUARD_PATTERN = /^(assert|ensure|require|validate|verify|guard|check)/i;
+        /**
+         * Extracts the callee's method name (the identifier bearing the leading
+         * verb) from an await expression argument. Handles both direct
+         * CallExpressions and optional-call ChainExpressions, and both bare
+         * identifier callees and member-expression callees.
+         */
+        function getCalleeMethodName(awaitExpr) {
+            let callExpr = null;
+            if (awaitExpr.argument.type === utils_1.AST_NODE_TYPES.CallExpression) {
+                callExpr = awaitExpr.argument;
+            }
+            else if (awaitExpr.argument.type === utils_1.AST_NODE_TYPES.ChainExpression &&
+                awaitExpr.argument.expression.type === utils_1.AST_NODE_TYPES.CallExpression) {
+                callExpr = awaitExpr.argument.expression;
+            }
+            if (!callExpr) {
+                return null;
+            }
+            const callee = callExpr.callee;
+            if (callee.type === utils_1.AST_NODE_TYPES.MemberExpression &&
+                callee.property.type === utils_1.AST_NODE_TYPES.Identifier) {
+                return callee.property.name;
+            }
+            if (callee.type === utils_1.AST_NODE_TYPES.Identifier) {
+                return callee.name;
+            }
+            return null;
+        }
         /**
          * Checks if there are dependencies between await expressions
          */
@@ -214,28 +250,34 @@ exports.parallelizeAsyncOperations = (0, createRule_1.createRule)({
                 }
                 // 3. Check for operations that might have side effects
                 // If any node has a side effect, we should not parallelize the sequence
-                let callExpr = null;
-                if (awaitExpr.argument.type === utils_1.AST_NODE_TYPES.CallExpression) {
-                    callExpr = awaitExpr.argument;
+                const methodName = getCalleeMethodName(awaitExpr);
+                if (methodName &&
+                    sideEffectPatterns.some((pattern) => pattern.test(methodName))) {
+                    return true;
                 }
-                else if (awaitExpr.argument.type === utils_1.AST_NODE_TYPES.ChainExpression &&
-                    awaitExpr.argument.expression.type === utils_1.AST_NODE_TYPES.CallExpression) {
-                    callExpr = awaitExpr.argument.expression;
+            }
+            // 4. Guard-then-side-effect ordering barrier. A discarded-result await
+            // whose callee reads as a guard/assertion (assert*, ensure*, validate*,
+            // ...) is a control-flow gate: it throws to abort the run when its
+            // precondition fails, so any await after it must run ONLY if the guard
+            // resolves. Promise.all invokes every operand eagerly, so it would fire
+            // the gated side effect even when the guard rejects. Treat the guard's
+            // presence (when something follows it) as a sequencing dependency that
+            // blocks parallelizing the whole run. Only discarded-result awaits
+            // (ExpressionStatements) qualify; `const ok = await validate(x)` has a
+            // variable and is handled by the data-dependency path above.
+            for (let i = 0; i < awaitNodes.length - 1; i++) {
+                const node = awaitNodes[i];
+                if (node.type !== utils_1.AST_NODE_TYPES.ExpressionStatement) {
+                    continue;
                 }
-                if (callExpr) {
-                    const callee = callExpr.callee;
-                    let methodName = null;
-                    if (callee.type === utils_1.AST_NODE_TYPES.MemberExpression &&
-                        callee.property.type === utils_1.AST_NODE_TYPES.Identifier) {
-                        methodName = callee.property.name;
-                    }
-                    else if (callee.type === utils_1.AST_NODE_TYPES.Identifier) {
-                        methodName = callee.name;
-                    }
-                    if (methodName &&
-                        sideEffectPatterns.some((pattern) => pattern.test(methodName))) {
-                        return true;
-                    }
+                const awaitExpr = getAwaitExpression(node);
+                if (!awaitExpr) {
+                    continue;
+                }
+                const methodName = getCalleeMethodName(awaitExpr);
+                if (methodName && GUARD_PATTERN.test(methodName)) {
+                    return true;
                 }
             }
             // If any node is a variable declaration with destructuring, consider it as having dependencies

package/package.json

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

package/release-manifest.json

@@ -1,4 +1,32 @@
 [
+  {
+    "version": "1.18.11",
+    "date": "2026-07-11T16:25:05.418Z",
+    "rules": [
+      {
+        "name": "parallelize-async-operations",
+        "changeType": "fix",
+        "issues": [
+          1284
+        ],
+        "summary": "treat throw-gated guard awaits as a sequencing barrier (closes #1284)"
+      }
+    ]
+  },
+  {
+    "version": "1.18.10",
+    "date": "2026-07-11T15:24:03.394Z",
+    "rules": [
+      {
+        "name": "parallelize-async-operations",
+        "changeType": "fix",
+        "issues": [
+          1283
+        ],
+        "summary": "restore coordinator to COORDINATOR_PATTERN (closes #1283)"
+      }
+    ]
+  },
   {
     "version": "1.18.9",
     "date": "2026-07-11T01:51:22.294Z",