npm - @webpieces/dev-config - Versions diffs - 0.2.94 → 0.2.95 - Mend

@webpieces/dev-config 0.2.94 → 0.2.95

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (29) hide show

package/eslint-plugin/__tests__/no-unmanaged-exceptions.test.ts CHANGED Viewed

@@ -31,7 +31,7 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
             code: `
                 try {
                     await operation();
-                } catch (err: any) {
+                } catch (err: unknown) {
                     const error = toError(err);
                     expect(error).toBeDefined();
                 }
@@ -43,7 +43,7 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
                 try {
                     await controller.save(request);
                     fail('Should have thrown');
-                } catch (err: any) {
+                } catch (err: unknown) {
                     const error = toError(err);
                     expect(error.message).toContain('Invalid');
                 }
@@ -56,7 +56,7 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
             code: `
                 try {
                     await operation();
-                } catch (err: any) {
+                } catch (err: unknown) {
                     const error = toError(err);
                 }
             `,
@@ -68,7 +68,7 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
                     try {
                         await service.process();
                         fail();
-                    } catch (err: any) {
+                    } catch (err: unknown) {
                         const error = toError(err);
                         expect(error).toBeDefined();
                     }
@@ -82,7 +82,7 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
             code: `
                 try {
                     await operation();
-                } catch (err: any) {
+                } catch (err: unknown) {
                     const error = toError(err);
                 }
             `,
@@ -92,7 +92,7 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
             code: `
                 try {
                     await runIntegrationTest();
-                } catch (err: any) {
+                } catch (err: unknown) {
                     const error = toError(err);
                     console.error(error);
                 }
@@ -103,7 +103,7 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
             code: `
                 try {
                     const result = performAction();
-                } catch (err: any) {
+                } catch (err: unknown) {
                     const error = toError(err);
                 }
             `,
@@ -158,7 +158,7 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
             code: `
                 try {
                     await operation();
-                } catch (err: any) {
+                } catch (err: unknown) {
                     const error = toError(err);
                 }
             `,
@@ -172,7 +172,7 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
                         try {
                             const result = await this.service.save(request);
                             return result;
-                        } catch (err: any) {
+                        } catch (err: unknown) {
                             const error = toError(err);
                             throw error;
                         }
@@ -190,7 +190,7 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
             code: `
                 try {
                     await this.database.query(sql);
-                } catch (err: any) {
+                } catch (err: unknown) {
                     const error = toError(err);
                 }
             `,
@@ -204,7 +204,7 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
                         try {
                             const user = await this.db.findOne({ id });
                             return user;
-                        } catch (err: any) {
+                        } catch (err: unknown) {
                             const error = toError(err);
                             console.error('Failed to fetch user:', error);
                             return null;
@@ -223,7 +223,7 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
             code: `
                 try {
                     return JSON.parse(body);
-                } catch (err: any) {
+                } catch (err: unknown) {
                     const error = toError(err);
                 }
             `,
@@ -236,7 +236,7 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
                     async filter(meta: MethodMeta, next: NextFilter): Promise<Action> {
                         try {
                             return await next.execute();
-                        } catch (err: any) {
+                        } catch (err: unknown) {
                             const error = toError(err);
                             console.error('Filter error:', error);
                             throw error;
@@ -257,7 +257,7 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
                     try {
                         const response = await fetch(url);
                         return await response.json();
-                    } catch (err: any) {
+                    } catch (err: unknown) {
                         const error = toError(err);
                         throw new Error(\`Fetch failed: \${error.message}\`);
                     }
@@ -275,7 +275,7 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
                 async function operation1() {
                     try {
                         await doSomething();
-                    } catch (err: any) {
+                    } catch (err: unknown) {
                         const error = toError(err);
                     }
                 }
@@ -283,7 +283,7 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
                 async function operation2() {
                     try {
                         await doSomethingElse();
-                    } catch (err: any) {
+                    } catch (err: unknown) {
                         const error = toError(err);
                     }
                 }
@@ -303,10 +303,10 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
                 try {
                     try {
                         await operation();
-                    } catch (err2: any) {
+                    } catch (err2: unknown) {
                         const error2 = toError(err2);
                     }
-                } catch (err: any) {
+                } catch (err: unknown) {
                     const error = toError(err);
                 }
             `,
@@ -325,7 +325,7 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
                 const handler = async () => {
                     try {
                         await doSomething();
-                    } catch (err: any) {
+                    } catch (err: unknown) {
                         const error = toError(err);
                     }
                 };
@@ -343,7 +343,7 @@ ruleTester.run('no-unmanaged-exceptions', rule, {
                     process(data: Data): Result {
                         try {
                             return this.performProcessing(data);
-                        } catch (err: any) {
+                        } catch (err: unknown) {
                             const error = toError(err);
                             return null;
                         }

package/eslint-plugin/rules/catch-error-pattern.d.ts CHANGED Viewed

@@ -2,9 +2,9 @@
  * ESLint rule to enforce standardized catch block error handling patterns
  *
  * Enforces three approved patterns:
- * 1. Standard: catch (err: any) { const error = toError(err); }
- * 2. Ignored: catch (err: any) { //const error = toError(err); }
- * 3. Nested: catch (err2: any) { const error2 = toError(err2); }
+ * 1. Standard: catch (err: unknown) { const error = toError(err); }
+ * 2. Ignored: catch (err: unknown) { //const error = toError(err); }
+ * 3. Nested: catch (err2: unknown) { const error2 = toError(err2); }
  */
 import type { Rule } from 'eslint';
 declare const rule: Rule.RuleModule;

package/eslint-plugin/rules/catch-error-pattern.js CHANGED Viewed

@@ -3,10 +3,84 @@
  * ESLint rule to enforce standardized catch block error handling patterns
  *
  * Enforces three approved patterns:
- * 1. Standard: catch (err: any) { const error = toError(err); }
- * 2. Ignored: catch (err: any) { //const error = toError(err); }
- * 3. Nested: catch (err2: any) { const error2 = toError(err2); }
+ * 1. Standard: catch (err: unknown) { const error = toError(err); }
+ * 2. Ignored: catch (err: unknown) { //const error = toError(err); }
+ * 3. Nested: catch (err2: unknown) { const error2 = toError(err2); }
  */
+function validateParamName(context, param, expectedParamName) {
+    if (param.type === 'Identifier' && param.name !== expectedParamName) {
+        context.report({
+            node: param,
+            messageId: 'wrongParameterName',
+            data: { actual: param.name },
+        });
+    }
+}
+function validateTypeAnnotation(context, param, expectedParamName) {
+    if (!param.typeAnnotation ||
+        !param.typeAnnotation.typeAnnotation ||
+        param.typeAnnotation.typeAnnotation.type !== 'TSUnknownKeyword') {
+        context.report({
+            node: param,
+            messageId: 'missingTypeAnnotation',
+            data: { param: param.name || expectedParamName },
+        });
+    }
+}
+function hasIgnoreComment(catchNode, sourceCode, expectedVarName, actualParamName) {
+    const catchBlockStart = catchNode.body.range[0];
+    const catchBlockEnd = catchNode.body.range[1];
+    const catchBlockText = sourceCode.text.substring(catchBlockStart, catchBlockEnd);
+    const ignorePattern = new RegExp(`//\\s*const\\s+${expectedVarName}\\s*=\\s*toError\\(${actualParamName}\\)`);
+    return ignorePattern.test(catchBlockText);
+}
+function reportMissingToError(context,
+// webpieces-disable no-any-unknown -- ESLint AST node param requires any type
+node, paramName) {
+    context.report({
+        node,
+        messageId: 'missingToError',
+        data: { param: paramName },
+    });
+}
+function validateToErrorCall(context,
+// webpieces-disable no-any-unknown -- ESLint AST node param requires any type
+firstStatement, expectedParamName, expectedVarName, actualParamName) {
+    if (firstStatement.type !== 'VariableDeclaration') {
+        reportMissingToError(context, firstStatement, expectedParamName);
+        return;
+    }
+    const varDecl = firstStatement;
+    const declaration = varDecl.declarations[0];
+    if (!declaration) {
+        reportMissingToError(context, firstStatement, expectedParamName);
+        return;
+    }
+    if (declaration.id.type !== 'Identifier' || declaration.id.name !== expectedVarName) {
+        context.report({
+            node: declaration.id,
+            messageId: 'wrongVariableName',
+            data: { expected: expectedVarName, actual: declaration.id.name || 'unknown' },
+        });
+        return;
+    }
+    if (!declaration.init || declaration.init.type !== 'CallExpression') {
+        reportMissingToError(context, declaration.init || declaration, expectedParamName);
+        return;
+    }
+    const callExpr = declaration.init;
+    const callee = callExpr.callee;
+    if (callee.type !== 'Identifier' || callee.name !== 'toError') {
+        reportMissingToError(context, callee, expectedParamName);
+        return;
+    }
+    const args = callExpr.arguments;
+    if (args.length !== 1 ||
+        args[0].type !== 'Identifier' ||
+        args[0].name !== actualParamName) {
+        reportMissingToError(context, callExpr, actualParamName);
+    }
+}
 const rule = {
     meta: {
         type: 'problem',
@@ -19,7 +93,7 @@ const rule = {
         messages: {
             missingToError: 'Catch block must call toError({{param}}) as first statement or comment it out to explicitly ignore errors',
             wrongVariableName: 'Error variable must be named "{{expected}}", got "{{actual}}"',
-            missingTypeAnnotation: 'Catch parameter must be typed as "any": catch ({{param}}: any)',
+            missingTypeAnnotation: 'Catch parameter must be typed as "unknown": catch ({{param}}: unknown)',
             wrongParameterName: 'Catch parameter must be named "err" (or "err2", "err3" for nested catches), got "{{actual}}"',
             toErrorNotFirst: 'toError({{param}}) must be the first statement in the catch block',
         },
@@ -27,22 +101,17 @@ const rule = {
         schema: [],
     },
     create(context) {
-        // Track nesting depth for err, err2, err3, etc.
         const catchStack = [];
         return {
             CatchClause(node) {
                 const catchNode = node;
-                // Calculate depth (1-based: first catch is depth 1)
                 const depth = catchStack.length + 1;
                 catchStack.push(catchNode);
-                // Build expected names based on depth
                 const suffix = depth === 1 ? '' : String(depth);
                 const expectedParamName = 'err' + suffix;
                 const expectedVarName = 'error' + suffix;
-                // Get the catch parameter
                 const param = catchNode.param;
                 if (!param) {
-                    // No parameter - unusual but technically valid (though not our pattern)
                     context.report({
                         node: catchNode,
                         messageId: 'missingTypeAnnotation',
@@ -50,141 +119,19 @@ const rule = {
                     });
                     return;
                 }
-                // Track the actual parameter name for validation (may differ from expected)
                 const actualParamName = param.type === 'Identifier' ? param.name : expectedParamName;
-                // RULE 1: Parameter must be named correctly (err, err2, err3, etc.)
-                if (param.type === 'Identifier' && param.name !== expectedParamName) {
-                    context.report({
-                        node: param,
-                        messageId: 'wrongParameterName',
-                        data: {
-                            actual: param.name,
-                        },
-                    });
-                }
-                // RULE 2: Must have type annotation ": any"
-                if (!param.typeAnnotation ||
-                    !param.typeAnnotation.typeAnnotation ||
-                    param.typeAnnotation.typeAnnotation.type !== 'TSAnyKeyword') {
-                    context.report({
-                        node: param,
-                        messageId: 'missingTypeAnnotation',
-                        data: {
-                            param: param.name || expectedParamName,
-                        },
-                    });
-                }
-                // RULE 3: Check first statement in catch block
-                const body = catchNode.body.body;
+                validateParamName(context, param, expectedParamName);
+                validateTypeAnnotation(context, param, expectedParamName);
                 const sourceCode = context.sourceCode || context.getSourceCode();
-                // IMPORTANT: Check for commented ignore pattern FIRST (before checking if body is empty)
-                // This allows Pattern 2 (empty catch with only comment) to be valid
-                // Look for: //const error = toError(err);
-                const catchBlockStart = catchNode.body.range[0];
-                const catchBlockEnd = catchNode.body.range[1];
-                const catchBlockText = sourceCode.text.substring(catchBlockStart, catchBlockEnd);
-                const ignorePattern = new RegExp(`//\\s*const\\s+${expectedVarName}\\s*=\\s*toError\\(${actualParamName}\\)`);
-                if (ignorePattern.test(catchBlockText)) {
-                    // Pattern 2: Explicitly ignored - valid!
+                if (hasIgnoreComment(catchNode, sourceCode, expectedVarName, actualParamName)) {
                     return;
                 }
-                // Now check if body is empty (after checking for commented pattern)
+                const body = catchNode.body.body;
                 if (body.length === 0) {
-                    // Empty catch block without comment - not allowed
-                    context.report({
-                        node: catchNode.body,
-                        messageId: 'missingToError',
-                        data: {
-                            param: expectedParamName,
-                        },
-                    });
-                    return;
-                }
-                const firstStatement = body[0];
-                // Check if first statement is: const error = toError(err)
-                if (firstStatement.type !== 'VariableDeclaration') {
-                    context.report({
-                        node: firstStatement,
-                        messageId: 'missingToError',
-                        data: {
-                            param: expectedParamName,
-                        },
-                    });
-                    return;
-                }
-                const varDecl = firstStatement;
-                const declaration = varDecl.declarations[0];
-                if (!declaration) {
-                    context.report({
-                        node: firstStatement,
-                        messageId: 'missingToError',
-                        data: {
-                            param: expectedParamName,
-                        },
-                    });
-                    return;
-                }
-                // Check variable name
-                if (declaration.id.type !== 'Identifier' ||
-                    declaration.id.name !== expectedVarName) {
-                    context.report({
-                        node: declaration.id,
-                        messageId: 'wrongVariableName',
-                        data: {
-                            expected: expectedVarName,
-                            actual: declaration.id.name || 'unknown',
-                        },
-                    });
-                    return;
-                }
-                // Check initialization: toError(err)
-                if (!declaration.init) {
-                    context.report({
-                        node: declaration,
-                        messageId: 'missingToError',
-                        data: {
-                            param: expectedParamName,
-                        },
-                    });
-                    return;
-                }
-                if (declaration.init.type !== 'CallExpression') {
-                    context.report({
-                        node: declaration.init,
-                        messageId: 'missingToError',
-                        data: {
-                            param: expectedParamName,
-                        },
-                    });
-                    return;
-                }
-                const callExpr = declaration.init;
-                const callee = callExpr.callee;
-                if (callee.type !== 'Identifier' || callee.name !== 'toError') {
-                    context.report({
-                        node: callee,
-                        messageId: 'missingToError',
-                        data: {
-                            param: expectedParamName,
-                        },
-                    });
-                    return;
-                }
-                // Check argument: must be the catch parameter (use actual param name)
-                const args = callExpr.arguments;
-                if (args.length !== 1 ||
-                    args[0].type !== 'Identifier' ||
-                    args[0].name !== actualParamName) {
-                    context.report({
-                        node: callExpr,
-                        messageId: 'missingToError',
-                        data: {
-                            param: actualParamName,
-                        },
-                    });
+                    reportMissingToError(context, catchNode.body, expectedParamName);
                     return;
                 }
-                // All checks passed! ✅
+                validateToErrorCall(context, body[0], expectedParamName, expectedVarName, actualParamName);
             },
             'CatchClause:exit'() {
                 catchStack.pop();

package/eslint-plugin/rules/catch-error-pattern.js.map CHANGED Viewed

	@@ -1 +1 @@
1	- {"version":3,"file":"catch-error-pattern.js","sourceRoot":"","sources":["../../../../../../packages/tooling/dev-config/eslint-plugin/rules/catch-error-pattern.ts"],"names":[],"mappings":";AAAA;;;;;;;GAOG;AAqDH,MAAM,IAAI,GAAoB;IAC1B,IAAI,EAAE;QACF,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACF,WAAW,EAAE,0DAA0D;YACvE,QAAQ,EAAE,gBAAgB;YAC1B,WAAW,EAAE,IAAI;YACjB,GAAG,EAAE,gGAAgG;SACxG;QACD,QAAQ,EAAE;YACN,cAAc,EACV,2GAA2G;YAC/G,iBAAiB,EAAE,+DAA+D;YAClF,qBAAqB,EAAE,gEAAgE;YACvF,kBAAkB,EACd,8FAA8F;YAClG,eAAe,EAAE,mEAAmE;SACvF;QACD,OAAO,EAAE,SAAS;QAClB,MAAM,EAAE,EAAE;KACb;IAED,MAAM,CAAC,OAAyB;QAC5B,gDAAgD;QAChD,MAAM,UAAU,GAAsB,EAAE,CAAC;QAEzC,OAAO;YACH,WAAW,CAAC,IAAS;gBACjB,MAAM,SAAS,GAAG,IAAuB,CAAC;gBAE1C,oDAAoD;gBACpD,MAAM,KAAK,GAAG,UAAU,CAAC,MAAM,GAAG,CAAC,CAAC;gBACpC,UAAU,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;gBAE3B,sCAAsC;gBACtC,MAAM,MAAM,GAAG,KAAK,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;gBAChD,MAAM,iBAAiB,GAAG,KAAK,GAAG,MAAM,CAAC;gBACzC,MAAM,eAAe,GAAG,OAAO,GAAG,MAAM,CAAC;gBAEzC,0BAA0B;gBAC1B,MAAM,KAAK,GAAG,SAAS,CAAC,KAAK,CAAC;gBAC9B,IAAI,CAAC,KAAK,EAAE,CAAC;oBACT,wEAAwE;oBACxE,OAAO,CAAC,MAAM,CAAC;wBACX,IAAI,EAAE,SAAS;wBACf,SAAS,EAAE,uBAAuB;wBAClC,IAAI,EAAE,EAAE,KAAK,EAAE,iBAAiB,EAAE;qBACrC,CAAC,CAAC;oBACH,OAAO;gBACX,CAAC;gBAED,4EAA4E;gBAC5E,MAAM,eAAe,GACjB,KAAK,CAAC,IAAI,KAAK,YAAY,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,iBAAiB,CAAC;gBAEjE,oEAAoE;gBACpE,IAAI,KAAK,CAAC,IAAI,KAAK,YAAY,IAAI,KAAK,CAAC,IAAI,KAAK,iBAAiB,EAAE,CAAC;oBAClE,OAAO,CAAC,MAAM,CAAC;wBACX,IAAI,EAAE,KAAK;wBACX,SAAS,EAAE,oBAAoB;wBAC/B,IAAI,EAAE;4BACF,MAAM,EAAE,KAAK,CAAC,IAAI;yBACrB;qBACJ,CAAC,CAAC;gBACP,CAAC;gBAED,4CAA4C;gBAC5C,IACI,CAAC,KAAK,CAAC,cAAc;oBACrB,CAAC,KAAK,CAAC,cAAc,CAAC,cAAc;oBACpC,KAAK,CAAC,cAAc,CAAC,cAAc,CAAC,IAAI,KAAK,cAAc,EAC7D,CAAC;oBACC,OAAO,CAAC,MAAM,CAAC;wBACX,IAAI,EAAE,KAAK;wBACX,SAAS,EAAE,uBAAuB;wBAClC,IAAI,EAAE;4BACF,KAAK,EAAE,KAAK,CAAC,IAAI,IAAI,iBAAiB;yBACzC;qBACJ,CAAC,CAAC;gBACP,CAAC;gBAED,+CAA+C;gBAC/C,MAAM,IAAI,GAAG,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC;gBACjC,MAAM,UAAU,GAAG,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,aAAa,EAAE,CAAC;gBAEjE,yFAAyF;gBACzF,oEAAoE;gBACpE,0CAA0C;gBAC1C,MAAM,eAAe,GAAG,SAAS,CAAC,IAAI,CAAC,KAAM,CAAC,CAAC,CAAC,CAAC;gBACjD,MAAM,aAAa,GAAG,SAAS,CAAC,IAAI,CAAC,KAAM,CAAC,CAAC,CAAC,CAAC;gBAC/C,MAAM,cAAc,GAAG,UAAU,CAAC,IAAI,CAAC,SAAS,CAAC,eAAe,EAAE,aAAa,CAAC,CAAC;gBAEjF,MAAM,aAAa,GAAG,IAAI,MAAM,CAC5B,kBAAkB,eAAe,sBAAsB,eAAe,KAAK,CAC9E,CAAC;gBAEF,IAAI,aAAa,CAAC,IAAI,CAAC,cAAc,CAAC,EAAE,CAAC;oBACrC,yCAAyC;oBACzC,OAAO;gBACX,CAAC;gBAED,oEAAoE;gBACpE,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;oBACpB,kDAAkD;oBAClD,OAAO,CAAC,MAAM,CAAC;wBACX,IAAI,EAAE,SAAS,CAAC,IAAI;wBACpB,SAAS,EAAE,gBAAgB;wBAC3B,IAAI,EAAE;4BACF,KAAK,EAAE,iBAAiB;yBAC3B;qBACJ,CAAC,CAAC;oBACH,OAAO;gBACX,CAAC;gBAED,MAAM,cAAc,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC;gBAE/B,0DAA0D;gBAC1D,IAAI,cAAc,CAAC,IAAI,KAAK,qBAAqB,EAAE,CAAC;oBAChD,OAAO,CAAC,MAAM,CAAC;wBACX,IAAI,EAAE,cAAc;wBACpB,SAAS,EAAE,gBAAgB;wBAC3B,IAAI,EAAE;4BACF,KAAK,EAAE,iBAAiB;yBAC3B;qBACJ,CAAC,CAAC;oBACH,OAAO;gBACX,CAAC;gBAED,MAAM,OAAO,GAAG,cAAyC,CAAC;gBAC1D,MAAM,WAAW,GAAG,OAAO,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;gBAC5C,IAAI,CAAC,WAAW,EAAE,CAAC;oBACf,OAAO,CAAC,MAAM,CAAC;wBACX,IAAI,EAAE,cAAc;wBACpB,SAAS,EAAE,gBAAgB;wBAC3B,IAAI,EAAE;4BACF,KAAK,EAAE,iBAAiB;yBAC3B;qBACJ,CAAC,CAAC;oBACH,OAAO;gBACX,CAAC;gBAED,sBAAsB;gBACtB,IACI,WAAW,CAAC,EAAE,CAAC,IAAI,KAAK,YAAY;oBACpC,WAAW,CAAC,EAAE,CAAC,IAAI,KAAK,eAAe,EACzC,CAAC;oBACC,OAAO,CAAC,MAAM,CAAC;wBACX,IAAI,EAAE,WAAW,CAAC,EAAE;wBACpB,SAAS,EAAE,mBAAmB;wBAC9B,IAAI,EAAE;4BACF,QAAQ,EAAE,eAAe;4BACzB,MAAM,EAAE,WAAW,CAAC,EAAE,CAAC,IAAI,IAAI,SAAS;yBAC3C;qBACJ,CAAC,CAAC;oBACH,OAAO;gBACX,CAAC;gBAED,qCAAqC;gBACrC,IAAI,CAAC,WAAW,CAAC,IAAI,EAAE,CAAC;oBACpB,OAAO,CAAC,MAAM,CAAC;wBACX,IAAI,EAAE,WAAW;wBACjB,SAAS,EAAE,gBAAgB;wBAC3B,IAAI,EAAE;4BACF,KAAK,EAAE,iBAAiB;yBAC3B;qBACJ,CAAC,CAAC;oBACH,OAAO;gBACX,CAAC;gBAED,IAAI,WAAW,CAAC,IAAI,CAAC,IAAI,KAAK,gBAAgB,EAAE,CAAC;oBAC7C,OAAO,CAAC,MAAM,CAAC;wBACX,IAAI,EAAE,WAAW,CAAC,IAAI;wBACtB,SAAS,EAAE,gBAAgB;wBAC3B,IAAI,EAAE;4BACF,KAAK,EAAE,iBAAiB;yBAC3B;qBACJ,CAAC,CAAC;oBACH,OAAO;gBACX,CAAC;gBAED,MAAM,QAAQ,GAAG,WAAW,CAAC,IAA0B,CAAC;gBACxD,MAAM,MAAM,GAAG,QAAQ,CAAC,MAAM,CAAC;gBAC/B,IAAI,MAAM,CAAC,IAAI,KAAK,YAAY,IAAI,MAAM,CAAC,IAAI,KAAK,SAAS,EAAE,CAAC;oBAC5D,OAAO,CAAC,MAAM,CAAC;wBACX,IAAI,EAAE,MAAM;wBACZ,SAAS,EAAE,gBAAgB;wBAC3B,IAAI,EAAE;4BACF,KAAK,EAAE,iBAAiB;yBAC3B;qBACJ,CAAC,CAAC;oBACH,OAAO;gBACX,CAAC;gBAED,sEAAsE;gBACtE,MAAM,IAAI,GAAG,QAAQ,CAAC,SAAS,CAAC;gBAChC,IACI,IAAI,CAAC,MAAM,KAAK,CAAC;oBACjB,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,KAAK,YAAY;oBAC5B,IAAI,CAAC,CAAC,CAAoB,CAAC,IAAI,KAAK,eAAe,EACtD,CAAC;oBACC,OAAO,CAAC,MAAM,CAAC;wBACX,IAAI,EAAE,QAAQ;wBACd,SAAS,EAAE,gBAAgB;wBAC3B,IAAI,EAAE;4BACF,KAAK,EAAE,eAAe;yBACzB;qBACJ,CAAC,CAAC;oBACH,OAAO;gBACX,CAAC;gBAED,uBAAuB;YAC3B,CAAC;YAED,kBAAkB;gBACd,UAAU,CAAC,GAAG,EAAE,CAAC;YACrB,CAAC;SACJ,CAAC;IACN,CAAC;CACJ,CAAC;AAEF,iBAAS,IAAI,CAAC","sourcesContent":["/*\n ESLint rule to enforce standardized catch block error handling patterns\n \n Enforces three approved patterns:\n * 1. Standard: catch (err: any) { const error = toError(err); }\n * 2. Ignored: catch (err: any) { //const error = toError(err); }\n * 3. Nested: catch (err2: any) { const error2 = toError(err2); }\n /\n\nimport type { Rule } from 'eslint';\n\n// Using any for ESTree nodes to avoid complex type gymnastics\n// ESLint rules work with dynamic AST nodes anyway\ninterface CatchClauseNode {\n type: 'CatchClause';\n param?: IdentifierNode \| null;\n body: BlockStatementNode;\n [key: string]: any;\n}\n\ninterface IdentifierNode {\n type: 'Identifier';\n name: string;\n typeAnnotation?: TypeAnnotationNode;\n [key: string]: any;\n}\n\ninterface TypeAnnotationNode {\n typeAnnotation?: {\n type: string;\n };\n}\n\ninterface BlockStatementNode {\n type: 'BlockStatement';\n body: any[];\n range: [number, number];\n [key: string]: any;\n}\n\ninterface VariableDeclarationNode {\n type: 'VariableDeclaration';\n declarations: VariableDeclaratorNode[];\n [key: string]: any;\n}\n\ninterface VariableDeclaratorNode {\n type: 'VariableDeclarator';\n id: IdentifierNode;\n init?: CallExpressionNode \| null;\n [key: string]: any;\n}\n\ninterface CallExpressionNode {\n type: 'CallExpression';\n callee: IdentifierNode;\n arguments: any[];\n [key: string]: any;\n}\n\nconst rule: Rule.RuleModule = {\n meta: {\n type: 'problem',\n docs: {\n description: 'Enforce standardized catch block error handling patterns',\n category: 'Best Practices',\n recommended: true,\n url: 'https://github.com/deanhiller/webpieces-ts/blob/main/claude.patterns.md#error-handling-pattern',\n },\n messages: {\n missingToError:\n 'Catch block must call toError({{param}}) as first statement or comment it out to explicitly ignore errors',\n wrongVariableName: 'Error variable must be named \"{{expected}}\", got \"{{actual}}\"',\n missingTypeAnnotation: 'Catch parameter must be typed as \"any\": catch ({{param}}: any)',\n wrongParameterName:\n 'Catch parameter must be named \"err\" (or \"err2\", \"err3\" for nested catches), got \"{{actual}}\"',\n toErrorNotFirst: 'toError({{param}}) must be the first statement in the catch block',\n },\n fixable: undefined,\n schema: [],\n },\n\n create(context: Rule.RuleContext): Rule.RuleListener {\n // Track nesting depth for err, err2, err3, etc.\n const catchStack: CatchClauseNode[] = [];\n\n return {\n CatchClause(node: any): void {\n const catchNode = node as CatchClauseNode;\n\n // Calculate depth (1-based: first catch is depth 1)\n const depth = catchStack.length + 1;\n catchStack.push(catchNode);\n\n // Build expected names based on depth\n const suffix = depth === 1 ? '' : String(depth);\n const expectedParamName = 'err' + suffix;\n const expectedVarName = 'error' + suffix;\n\n // Get the catch parameter\n const param = catchNode.param;\n if (!param) {\n // No parameter - unusual but technically valid (though not our pattern)\n context.report({\n node: catchNode,\n messageId: 'missingTypeAnnotation',\n data: { param: expectedParamName },\n });\n return;\n }\n\n // Track the actual parameter name for validation (may differ from expected)\n const actualParamName =\n param.type === 'Identifier' ? param.name : expectedParamName;\n\n // RULE 1: Parameter must be named correctly (err, err2, err3, etc.)\n if (param.type === 'Identifier' && param.name !== expectedParamName) {\n context.report({\n node: param,\n messageId: 'wrongParameterName',\n data: {\n actual: param.name,\n },\n });\n }\n\n // RULE 2: Must have type annotation \": any\"\n if (\n !param.typeAnnotation \|\|\n !param.typeAnnotation.typeAnnotation \|\|\n param.typeAnnotation.typeAnnotation.type !== 'TSAnyKeyword'\n ) {\n context.report({\n node: param,\n messageId: 'missingTypeAnnotation',\n data: {\n param: param.name \|\| expectedParamName,\n },\n });\n }\n\n // RULE 3: Check first statement in catch block\n const body = catchNode.body.body;\n const sourceCode = context.sourceCode \|\| context.getSourceCode();\n\n // IMPORTANT: Check for commented ignore pattern FIRST (before checking if body is empty)\n // This allows Pattern 2 (empty catch with only comment) to be valid\n // Look for: //const error = toError(err);\n const catchBlockStart = catchNode.body.range![0];\n const catchBlockEnd = catchNode.body.range![1];\n const catchBlockText = sourceCode.text.substring(catchBlockStart, catchBlockEnd);\n\n const ignorePattern = new RegExp(\n `//\\\\sconst\\\\s+${expectedVarName}\\\\s=\\\\stoError\\\$${actualParamName}\\\$`,\n );\n\n if (ignorePattern.test(catchBlockText)) {\n // Pattern 2: Explicitly ignored - valid!\n return;\n }\n\n // Now check if body is empty (after checking for commented pattern)\n if (body.length === 0) {\n // Empty catch block without comment - not allowed\n context.report({\n node: catchNode.body,\n messageId: 'missingToError',\n data: {\n param: expectedParamName,\n },\n });\n return;\n }\n\n const firstStatement = body[0];\n\n // Check if first statement is: const error = toError(err)\n if (firstStatement.type !== 'VariableDeclaration') {\n context.report({\n node: firstStatement,\n messageId: 'missingToError',\n data: {\n param: expectedParamName,\n },\n });\n return;\n }\n\n const varDecl = firstStatement as VariableDeclarationNode;\n const declaration = varDecl.declarations[0];\n if (!declaration) {\n context.report({\n node: firstStatement,\n messageId: 'missingToError',\n data: {\n param: expectedParamName,\n },\n });\n return;\n }\n\n // Check variable name\n if (\n declaration.id.type !== 'Identifier' \|\|\n declaration.id.name !== expectedVarName\n ) {\n context.report({\n node: declaration.id,\n messageId: 'wrongVariableName',\n data: {\n expected: expectedVarName,\n actual: declaration.id.name \|\| 'unknown',\n },\n });\n return;\n }\n\n // Check initialization: toError(err)\n if (!declaration.init) {\n context.report({\n node: declaration,\n messageId: 'missingToError',\n data: {\n param: expectedParamName,\n },\n });\n return;\n }\n\n if (declaration.init.type !== 'CallExpression') {\n context.report({\n node: declaration.init,\n messageId: 'missingToError',\n data: {\n param: expectedParamName,\n },\n });\n return;\n }\n\n const callExpr = declaration.init as CallExpressionNode;\n const callee = callExpr.callee;\n if (callee.type !== 'Identifier' \|\| callee.name !== 'toError') {\n context.report({\n node: callee,\n messageId: 'missingToError',\n data: {\n param: expectedParamName,\n },\n });\n return;\n }\n\n // Check argument: must be the catch parameter (use actual param name)\n const args = callExpr.arguments;\n if (\n args.length !== 1 \|\|\n args[0].type !== 'Identifier' \|\|\n (args[0] as IdentifierNode).name !== actualParamName\n ) {\n context.report({\n node: callExpr,\n messageId: 'missingToError',\n data: {\n param: actualParamName,\n },\n });\n return;\n }\n\n // All checks passed! ✅\n },\n\n 'CatchClause:exit'(): void {\n catchStack.pop();\n },\n };\n },\n};\n\nexport = rule;\n"]}
1	+ {"version":3,"file":"catch-error-pattern.js","sourceRoot":"","sources":["../../../../../../packages/tooling/dev-config/eslint-plugin/rules/catch-error-pattern.ts"],"names":[],"mappings":";AAAA;;;;;;;GAOG;AAqDH,SAAS,iBAAiB,CACtB,OAAyB,EACzB,KAAqB,EACrB,iBAAyB;IAEzB,IAAI,KAAK,CAAC,IAAI,KAAK,YAAY,IAAI,KAAK,CAAC,IAAI,KAAK,iBAAiB,EAAE,CAAC;QAClE,OAAO,CAAC,MAAM,CAAC;YACX,IAAI,EAAE,KAAK;YACX,SAAS,EAAE,oBAAoB;YAC/B,IAAI,EAAE,EAAE,MAAM,EAAE,KAAK,CAAC,IAAI,EAAE;SAC/B,CAAC,CAAC;IACP,CAAC;AACL,CAAC;AAED,SAAS,sBAAsB,CAC3B,OAAyB,EACzB,KAAqB,EACrB,iBAAyB;IAEzB,IACI,CAAC,KAAK,CAAC,cAAc;QACrB,CAAC,KAAK,CAAC,cAAc,CAAC,cAAc;QACpC,KAAK,CAAC,cAAc,CAAC,cAAc,CAAC,IAAI,KAAK,kBAAkB,EACjE,CAAC;QACC,OAAO,CAAC,MAAM,CAAC;YACX,IAAI,EAAE,KAAK;YACX,SAAS,EAAE,uBAAuB;YAClC,IAAI,EAAE,EAAE,KAAK,EAAE,KAAK,CAAC,IAAI,IAAI,iBAAiB,EAAE;SACnD,CAAC,CAAC;IACP,CAAC;AACL,CAAC;AAED,SAAS,gBAAgB,CACrB,SAA0B,EAC1B,UAA0C,EAC1C,eAAuB,EACvB,eAAuB;IAEvB,MAAM,eAAe,GAAG,SAAS,CAAC,IAAI,CAAC,KAAM,CAAC,CAAC,CAAC,CAAC;IACjD,MAAM,aAAa,GAAG,SAAS,CAAC,IAAI,CAAC,KAAM,CAAC,CAAC,CAAC,CAAC;IAC/C,MAAM,cAAc,GAAG,UAAU,CAAC,IAAI,CAAC,SAAS,CAAC,eAAe,EAAE,aAAa,CAAC,CAAC;IAEjF,MAAM,aAAa,GAAG,IAAI,MAAM,CAC5B,kBAAkB,eAAe,sBAAsB,eAAe,KAAK,CAC9E,CAAC;IAEF,OAAO,aAAa,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;AAC9C,CAAC;AAED,SAAS,oBAAoB,CACzB,OAAyB;AACzB,8EAA8E;AAC9E,IAAS,EACT,SAAiB;IAEjB,OAAO,CAAC,MAAM,CAAC;QACX,IAAI;QACJ,SAAS,EAAE,gBAAgB;QAC3B,IAAI,EAAE,EAAE,KAAK,EAAE,SAAS,EAAE;KAC7B,CAAC,CAAC;AACP,CAAC;AAED,SAAS,mBAAmB,CACxB,OAAyB;AACzB,8EAA8E;AAC9E,cAAmB,EACnB,iBAAyB,EACzB,eAAuB,EACvB,eAAuB;IAEvB,IAAI,cAAc,CAAC,IAAI,KAAK,qBAAqB,EAAE,CAAC;QAChD,oBAAoB,CAAC,OAAO,EAAE,cAAc,EAAE,iBAAiB,CAAC,CAAC;QACjE,OAAO;IACX,CAAC;IAED,MAAM,OAAO,GAAG,cAAyC,CAAC;IAC1D,MAAM,WAAW,GAAG,OAAO,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IAC5C,IAAI,CAAC,WAAW,EAAE,CAAC;QACf,oBAAoB,CAAC,OAAO,EAAE,cAAc,EAAE,iBAAiB,CAAC,CAAC;QACjE,OAAO;IACX,CAAC;IAED,IAAI,WAAW,CAAC,EAAE,CAAC,IAAI,KAAK,YAAY,IAAI,WAAW,CAAC,EAAE,CAAC,IAAI,KAAK,eAAe,EAAE,CAAC;QAClF,OAAO,CAAC,MAAM,CAAC;YACX,IAAI,EAAE,WAAW,CAAC,EAAE;YACpB,SAAS,EAAE,mBAAmB;YAC9B,IAAI,EAAE,EAAE,QAAQ,EAAE,eAAe,EAAE,MAAM,EAAE,WAAW,CAAC,EAAE,CAAC,IAAI,IAAI,SAAS,EAAE;SAChF,CAAC,CAAC;QACH,OAAO;IACX,CAAC;IAED,IAAI,CAAC,WAAW,CAAC,IAAI,IAAI,WAAW,CAAC,IAAI,CAAC,IAAI,KAAK,gBAAgB,EAAE,CAAC;QAClE,oBAAoB,CAAC,OAAO,EAAE,WAAW,CAAC,IAAI,IAAI,WAAW,EAAE,iBAAiB,CAAC,CAAC;QAClF,OAAO;IACX,CAAC;IAED,MAAM,QAAQ,GAAG,WAAW,CAAC,IAA0B,CAAC;IACxD,MAAM,MAAM,GAAG,QAAQ,CAAC,MAAM,CAAC;IAC/B,IAAI,MAAM,CAAC,IAAI,KAAK,YAAY,IAAI,MAAM,CAAC,IAAI,KAAK,SAAS,EAAE,CAAC;QAC5D,oBAAoB,CAAC,OAAO,EAAE,MAAM,EAAE,iBAAiB,CAAC,CAAC;QACzD,OAAO;IACX,CAAC;IAED,MAAM,IAAI,GAAG,QAAQ,CAAC,SAAS,CAAC;IAChC,IACI,IAAI,CAAC,MAAM,KAAK,CAAC;QACjB,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,KAAK,YAAY;QAC5B,IAAI,CAAC,CAAC,CAAoB,CAAC,IAAI,KAAK,eAAe,EACtD,CAAC;QACC,oBAAoB,CAAC,OAAO,EAAE,QAAQ,EAAE,eAAe,CAAC,CAAC;IAC7D,CAAC;AACL,CAAC;AAED,MAAM,IAAI,GAAoB;IAC1B,IAAI,EAAE;QACF,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACF,WAAW,EAAE,0DAA0D;YACvE,QAAQ,EAAE,gBAAgB;YAC1B,WAAW,EAAE,IAAI;YACjB,GAAG,EAAE,gGAAgG;SACxG;QACD,QAAQ,EAAE;YACN,cAAc,EACV,2GAA2G;YAC/G,iBAAiB,EAAE,+DAA+D;YAClF,qBAAqB,EAAE,wEAAwE;YAC/F,kBAAkB,EACd,8FAA8F;YAClG,eAAe,EAAE,mEAAmE;SACvF;QACD,OAAO,EAAE,SAAS;QAClB,MAAM,EAAE,EAAE;KACb;IAED,MAAM,CAAC,OAAyB;QAC5B,MAAM,UAAU,GAAsB,EAAE,CAAC;QAEzC,OAAO;YACH,WAAW,CAAC,IAAS;gBACjB,MAAM,SAAS,GAAG,IAAuB,CAAC;gBAC1C,MAAM,KAAK,GAAG,UAAU,CAAC,MAAM,GAAG,CAAC,CAAC;gBACpC,UAAU,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;gBAE3B,MAAM,MAAM,GAAG,KAAK,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;gBAChD,MAAM,iBAAiB,GAAG,KAAK,GAAG,MAAM,CAAC;gBACzC,MAAM,eAAe,GAAG,OAAO,GAAG,MAAM,CAAC;gBAEzC,MAAM,KAAK,GAAG,SAAS,CAAC,KAAK,CAAC;gBAC9B,IAAI,CAAC,KAAK,EAAE,CAAC;oBACT,OAAO,CAAC,MAAM,CAAC;wBACX,IAAI,EAAE,SAAS;wBACf,SAAS,EAAE,uBAAuB;wBAClC,IAAI,EAAE,EAAE,KAAK,EAAE,iBAAiB,EAAE;qBACrC,CAAC,CAAC;oBACH,OAAO;gBACX,CAAC;gBAED,MAAM,eAAe,GACjB,KAAK,CAAC,IAAI,KAAK,YAAY,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,iBAAiB,CAAC;gBAEjE,iBAAiB,CAAC,OAAO,EAAE,KAAK,EAAE,iBAAiB,CAAC,CAAC;gBACrD,sBAAsB,CAAC,OAAO,EAAE,KAAK,EAAE,iBAAiB,CAAC,CAAC;gBAE1D,MAAM,UAAU,GAAG,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,aAAa,EAAE,CAAC;gBACjE,IAAI,gBAAgB,CAAC,SAAS,EAAE,UAAU,EAAE,eAAe,EAAE,eAAe,CAAC,EAAE,CAAC;oBAC5E,OAAO;gBACX,CAAC;gBAED,MAAM,IAAI,GAAG,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC;gBACjC,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;oBACpB,oBAAoB,CAAC,OAAO,EAAE,SAAS,CAAC,IAAI,EAAE,iBAAiB,CAAC,CAAC;oBACjE,OAAO;gBACX,CAAC;gBAED,mBAAmB,CAAC,OAAO,EAAE,IAAI,CAAC,CAAC,CAAC,EAAE,iBAAiB,EAAE,eAAe,EAAE,eAAe,CAAC,CAAC;YAC/F,CAAC;YAED,kBAAkB;gBACd,UAAU,CAAC,GAAG,EAAE,CAAC;YACrB,CAAC;SACJ,CAAC;IACN,CAAC;CACJ,CAAC;AAEF,iBAAS,IAAI,CAAC","sourcesContent":["/*\n ESLint rule to enforce standardized catch block error handling patterns\n \n Enforces three approved patterns:\n * 1. Standard: catch (err: unknown) { const error = toError(err); }\n * 2. Ignored: catch (err: unknown) { //const error = toError(err); }\n * 3. Nested: catch (err2: unknown) { const error2 = toError(err2); }\n /\n\nimport type { Rule } from 'eslint';\n\n// Using any for ESTree nodes to avoid complex type gymnastics\n// ESLint rules work with dynamic AST nodes anyway\ninterface CatchClauseNode {\n type: 'CatchClause';\n param?: IdentifierNode \| null;\n body: BlockStatementNode;\n [key: string]: any;\n}\n\ninterface IdentifierNode {\n type: 'Identifier';\n name: string;\n typeAnnotation?: TypeAnnotationNode;\n [key: string]: any;\n}\n\ninterface TypeAnnotationNode {\n typeAnnotation?: {\n type: string;\n };\n}\n\ninterface BlockStatementNode {\n type: 'BlockStatement';\n body: any[];\n range: [number, number];\n [key: string]: any;\n}\n\ninterface VariableDeclarationNode {\n type: 'VariableDeclaration';\n declarations: VariableDeclaratorNode[];\n [key: string]: any;\n}\n\ninterface VariableDeclaratorNode {\n type: 'VariableDeclarator';\n id: IdentifierNode;\n init?: CallExpressionNode \| null;\n [key: string]: any;\n}\n\ninterface CallExpressionNode {\n type: 'CallExpression';\n callee: IdentifierNode;\n arguments: any[];\n [key: string]: any;\n}\n\nfunction validateParamName(\n context: Rule.RuleContext,\n param: IdentifierNode,\n expectedParamName: string,\n): void {\n if (param.type === 'Identifier' && param.name !== expectedParamName) {\n context.report({\n node: param,\n messageId: 'wrongParameterName',\n data: { actual: param.name },\n });\n }\n}\n\nfunction validateTypeAnnotation(\n context: Rule.RuleContext,\n param: IdentifierNode,\n expectedParamName: string,\n): void {\n if (\n !param.typeAnnotation \|\|\n !param.typeAnnotation.typeAnnotation \|\|\n param.typeAnnotation.typeAnnotation.type !== 'TSUnknownKeyword'\n ) {\n context.report({\n node: param,\n messageId: 'missingTypeAnnotation',\n data: { param: param.name \|\| expectedParamName },\n });\n }\n}\n\nfunction hasIgnoreComment(\n catchNode: CatchClauseNode,\n sourceCode: Rule.RuleContext['sourceCode'],\n expectedVarName: string,\n actualParamName: string,\n): boolean {\n const catchBlockStart = catchNode.body.range![0];\n const catchBlockEnd = catchNode.body.range![1];\n const catchBlockText = sourceCode.text.substring(catchBlockStart, catchBlockEnd);\n\n const ignorePattern = new RegExp(\n `//\\\\sconst\\\\s+${expectedVarName}\\\\s=\\\\stoError\\\$${actualParamName}\\\$`,\n );\n\n return ignorePattern.test(catchBlockText);\n}\n\nfunction reportMissingToError(\n context: Rule.RuleContext,\n // webpieces-disable no-any-unknown -- ESLint AST node param requires any type\n node: any,\n paramName: string,\n): void {\n context.report({\n node,\n messageId: 'missingToError',\n data: { param: paramName },\n });\n}\n\nfunction validateToErrorCall(\n context: Rule.RuleContext,\n // webpieces-disable no-any-unknown -- ESLint AST node param requires any type\n firstStatement: any,\n expectedParamName: string,\n expectedVarName: string,\n actualParamName: string,\n): void {\n if (firstStatement.type !== 'VariableDeclaration') {\n reportMissingToError(context, firstStatement, expectedParamName);\n return;\n }\n\n const varDecl = firstStatement as VariableDeclarationNode;\n const declaration = varDecl.declarations[0];\n if (!declaration) {\n reportMissingToError(context, firstStatement, expectedParamName);\n return;\n }\n\n if (declaration.id.type !== 'Identifier' \|\| declaration.id.name !== expectedVarName) {\n context.report({\n node: declaration.id,\n messageId: 'wrongVariableName',\n data: { expected: expectedVarName, actual: declaration.id.name \|\| 'unknown' },\n });\n return;\n }\n\n if (!declaration.init \|\| declaration.init.type !== 'CallExpression') {\n reportMissingToError(context, declaration.init \|\| declaration, expectedParamName);\n return;\n }\n\n const callExpr = declaration.init as CallExpressionNode;\n const callee = callExpr.callee;\n if (callee.type !== 'Identifier' \|\| callee.name !== 'toError') {\n reportMissingToError(context, callee, expectedParamName);\n return;\n }\n\n const args = callExpr.arguments;\n if (\n args.length !== 1 \|\|\n args[0].type !== 'Identifier' \|\|\n (args[0] as IdentifierNode).name !== actualParamName\n ) {\n reportMissingToError(context, callExpr, actualParamName);\n }\n}\n\nconst rule: Rule.RuleModule = {\n meta: {\n type: 'problem',\n docs: {\n description: 'Enforce standardized catch block error handling patterns',\n category: 'Best Practices',\n recommended: true,\n url: 'https://github.com/deanhiller/webpieces-ts/blob/main/claude.patterns.md#error-handling-pattern',\n },\n messages: {\n missingToError:\n 'Catch block must call toError({{param}}) as first statement or comment it out to explicitly ignore errors',\n wrongVariableName: 'Error variable must be named \"{{expected}}\", got \"{{actual}}\"',\n missingTypeAnnotation: 'Catch parameter must be typed as \"unknown\": catch ({{param}}: unknown)',\n wrongParameterName:\n 'Catch parameter must be named \"err\" (or \"err2\", \"err3\" for nested catches), got \"{{actual}}\"',\n toErrorNotFirst: 'toError({{param}}) must be the first statement in the catch block',\n },\n fixable: undefined,\n schema: [],\n },\n\n create(context: Rule.RuleContext): Rule.RuleListener {\n const catchStack: CatchClauseNode[] = [];\n\n return {\n CatchClause(node: any): void {\n const catchNode = node as CatchClauseNode;\n const depth = catchStack.length + 1;\n catchStack.push(catchNode);\n\n const suffix = depth === 1 ? '' : String(depth);\n const expectedParamName = 'err' + suffix;\n const expectedVarName = 'error' + suffix;\n\n const param = catchNode.param;\n if (!param) {\n context.report({\n node: catchNode,\n messageId: 'missingTypeAnnotation',\n data: { param: expectedParamName },\n });\n return;\n }\n\n const actualParamName =\n param.type === 'Identifier' ? param.name : expectedParamName;\n\n validateParamName(context, param, expectedParamName);\n validateTypeAnnotation(context, param, expectedParamName);\n\n const sourceCode = context.sourceCode \|\| context.getSourceCode();\n if (hasIgnoreComment(catchNode, sourceCode, expectedVarName, actualParamName)) {\n return;\n }\n\n const body = catchNode.body.body;\n if (body.length === 0) {\n reportMissingToError(context, catchNode.body, expectedParamName);\n return;\n }\n\n validateToErrorCall(context, body[0], expectedParamName, expectedVarName, actualParamName);\n },\n\n 'CatchClause:exit'(): void {\n catchStack.pop();\n },\n };\n },\n};\n\nexport = rule;\n"]}