@blumintinc/eslint-plugin-blumint 1.19.25 → 1.19.26

package/lib/index.js

@@ -222,7 +222,7 @@ function noFrontendImportsFromFunctionsPatterns(pattern) {
 module.exports = {
     meta: {
         name: '@blumintinc/eslint-plugin-blumint',
-        version: '1.19.25',
+        version: '1.19.26',
     },
     parseOptions: {
         ecmaVersion: 2020,
@@ -267,7 +267,7 @@ module.exports = {
                 '@blumintinc/blumint/no-async-foreach': 'error',
                 '@blumintinc/blumint/no-console-error': [
                     'error',
-                    { allowWithUseAlertDialog: true },
+                    { allowWithUseAlertDialog: true, allowErrorInstanceArgument: true },
                 ],
                 '@blumintinc/blumint/no-conditional-literals-in-jsx': 'error',
                 '@blumintinc/blumint/no-filter-without-return': 'error',

package/lib/rules/no-console-error.d.ts

@@ -3,6 +3,7 @@ type Options = [
     {
         ignorePatterns?: string[];
         allowWithUseAlertDialog?: boolean;
+        allowErrorInstanceArgument?: boolean;
     }
 ];
 export declare const noConsoleError: TSESLint.RuleModule<"noConsoleError", Options, TSESLint.RuleListener>;

package/lib/rules/no-console-error.js

@@ -113,6 +113,18 @@ const getLocalNameFromPattern = (target) => {
     return null;
 };
 const findDeclaredVariableByName = (name, variables) => variables.find((variable) => variable.name === name);
+/**
+ * A `new <Something>Error(...)` argument is the structured-error handoff the
+ * rule exists to encourage: on the backend, `console.error(new HttpsError(...))`
+ * is the monitored pipeline (spyOnConsoleErrors reports the Error among the
+ * args), not a bypass of it. Detection stays purely syntactic — the rule is not
+ * type-aware and runs on files outside any `parserOptions.project` — so a bare
+ * identifier (`console.error(error)`) is deliberately excluded because syntax
+ * alone cannot prove it holds an Error.
+ */
+const isStructuredErrorArgument = (arg) => arg.type === utils_1.AST_NODE_TYPES.NewExpression &&
+    arg.callee.type === utils_1.AST_NODE_TYPES.Identifier &&
+    /Error$/.test(arg.callee.name);
 /**
  * Tracks aliases of `console` and `console.error` to detect indirect calls.
  */
@@ -451,6 +463,7 @@ exports.noConsoleError = (0, createRule_1.createRule)({
                         items: { type: 'string' },
                     },
                     allowWithUseAlertDialog: { type: 'boolean' },
+                    allowErrorInstanceArgument: { type: 'boolean' },
                 },
                 additionalProperties: false,
             },
@@ -467,6 +480,7 @@ exports.noConsoleError = (0, createRule_1.createRule)({
         if (shouldIgnoreFile(filename))
             return {};
         const allowWithUseAlertDialog = options.allowWithUseAlertDialog === true;
+        const allowErrorInstanceArgument = options.allowErrorInstanceArgument === true;
         const aliasTracker = new AliasTracker(context);
         const dialogTracker = new UseAlertDialogTracker(context);
         const pendingTracker = new PendingCallTracker();
@@ -511,6 +525,14 @@ exports.noConsoleError = (0, createRule_1.createRule)({
                 const scope = getScopeForNode(context, node);
                 if (!aliasTracker.isConsoleErrorCall(node, scope))
                     return;
+                // A structured-error argument is the sanctioned, monitored path, so the
+                // carve-out applies to every matched call shape (direct, aliased,
+                // destructured, computed) — unlike the useAlertDialog queueing below,
+                // which is restricted to the direct-global shape.
+                if (allowErrorInstanceArgument &&
+                    node.arguments.some(isStructuredErrorArgument)) {
+                    return;
+                }
                 // Queueing is limited to direct global `console.error(...)` calls because deferral relies on
                 // linear control flow. Aliased or destructured references can escape the current scope
                 // (stored, passed, invoked later), so they are reported immediately to avoid missing non-local calls.

package/package.json

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

package/release-manifest.json

@@ -1,4 +1,18 @@
 [
+  {
+    "version": "1.19.26",
+    "date": "2026-07-23T00:25:08.850Z",
+    "rules": [
+      {
+        "name": "no-console-error",
+        "changeType": "fix",
+        "issues": [
+          1333
+        ],
+        "summary": "allow structured Error argument via allowErrorInstanceArgument (closes #1333)"
+      }
+    ]
+  },
   {
     "version": "1.19.25",
     "date": "2026-07-22T21:39:17.787Z",