@webpieces/eslint-rules 0.0.1 → 0.2.113

package/src/rules/max-file-lines.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"max-file-lines.js","sourceRoot":"","sources":["../../../../../../packages/tooling/eslint-rules/src/rules/max-file-lines.ts"],"names":[],"mappings":";AAAA;;;;;;;;GAQG;;AAGH,+CAAyB;AACzB,mDAA6B;AAC7B,wCAAqC;AAMrC,MAAM,gBAAgB,GAAG;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CAkJxB,CAAC;AAEF,uDAAuD;AACvD,IAAI,cAAc,GAAG,KAAK,CAAC;AAE3B,SAAS,gBAAgB,CAAC,OAAyB;IAC/C,MAAM,QAAQ,GAAG,OAAO,CAAC,QAAQ,IAAI,OAAO,CAAC,WAAW,EAAE,CAAC;IAC3D,IAAI,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;IAEjC,gDAAgD;IAChD,OAAO,GAAG,KAAK,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC;QAC/B,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,cAAc,CAAC,CAAC;QAC/C,IAAI,EAAE,CAAC,UAAU,CAAC,OAAO,CAAC,EAAE,CAAC;YACzB,8DAA8D;YAC9D,IAAI,CAAC;gBACD,MAAM,GAAG,GAAG,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,OAAO,CAAC,CAAC,CAAC;gBAC1D,IAAI,GAAG,CAAC,UAAU,IAAI,GAAG,CAAC,IAAI,KAAK,cAAc,EAAE,CAAC;oBAChD,OAAO,GAAG,CAAC;gBACf,CAAC;YACL,CAAC;YAAC,OAAO,GAAY,EAAE,CAAC;gBACpB,6BAA6B;gBAC7B,KAAK,GAAG,CAAC,CAAC,yCAAyC;YACvD,CAAC;QACL,CAAC;QACD,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;IAC5B,CAAC;IACD,OAAO,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC,WAAW;AACrC,CAAC;AAED,SAAS,aAAa,CAAC,OAAe,EAAE,OAAe;IACnD,8DAA8D;IAC9D,IAAI,CAAC;QACD,EAAE,CAAC,SAAS,CAAC,IAAI,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QACzD,EAAE,CAAC,aAAa,CAAC,OAAO,EAAE,OAAO,EAAE,OAAO,CAAC,CAAC;QAC5C,OAAO,IAAI,CAAC;IAChB,CAAC;IAAC,OAAO,GAAY,EAAE,CAAC;QACpB,MAAM,KAAK,GAAG,IAAA,iBAAO,EAAC,GAAG,CAAC,CAAC;QAC3B,OAAO,CAAC,IAAI,CAAC,0CAA0C,OAAO,EAAE,EAAE,KAAK,CAAC,CAAC;QACzE,OAAO,KAAK,CAAC;IACjB,CAAC;AACL,CAAC;AAED,SAAS,aAAa,CAAC,OAAyB;IAC5C,IAAI,cAAc;QAAE,OAAO,CAAC,6CAA6C;IAEzE,MAAM,aAAa,GAAG,gBAAgB,CAAC,OAAO,CAAC,CAAC;IAChD,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,aAAa,EAAE,YAAY,EAAE,aAAa,EAAE,uBAAuB,CAAC,CAAC;IAE/F,IAAI,aAAa,CAAC,OAAO,EAAE,gBAAgB,CAAC,EAAE,CAAC;QAC3C,cAAc,GAAG,IAAI,CAAC;IAC1B,CAAC;AACL,CAAC;AAED,MAAM,IAAI,GAAoB;IAC1B,IAAI,EAAE;QACF,IAAI,EAAE,YAAY;QAClB,IAAI,EAAE;YACF,WAAW,EAAE,6BAA6B;YAC1C,QAAQ,EAAE,gBAAgB;YAC1B,WAAW,EAAE,KAAK;YAClB,GAAG,EAAE,4CAA4C;SACpD;QACD,QAAQ,EAAE;YACN,OAAO,EACH,4HAA4H;SACnI;QACD,OAAO,EAAE,SAAS;QAClB,MAAM,EAAE;YACJ;gBACI,IAAI,EAAE,QAAQ;gBACd,UAAU,EAAE;oBACR,GAAG,EAAE;wBACD,IAAI,EAAE,SAAS;wBACf,OAAO,EAAE,CAAC;qBACb;iBACJ;gBACD,oBAAoB,EAAE,KAAK;aAC9B;SACJ;KACJ;IAED,MAAM,CAAC,OAAyB;QAC5B,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC,CAAiC,CAAC;QACnE,MAAM,QAAQ,GAAG,OAAO,EAAE,GAAG,IAAI,GAAG,CAAC;QAErC,OAAO;YACH,0FAA0F;YAC1F,OAAO,CAAC,IAAS;gBACb,aAAa,CAAC,OAAO,CAAC,CAAC;gBAEvB,MAAM,UAAU,GAAG,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,aAAa,EAAE,CAAC;gBACjE,MAAM,KAAK,GAAG,UAAU,CAAC,KAAK,CAAC;gBAC/B,MAAM,SAAS,GAAG,KAAK,CAAC,MAAM,CAAC;gBAE/B,IAAI,SAAS,GAAG,QAAQ,EAAE,CAAC;oBACvB,OAAO,CAAC,MAAM,CAAC;wBACX,IAAI;wBACJ,SAAS,EAAE,SAAS;wBACpB,IAAI,EAAE;4BACF,MAAM,EAAE,MAAM,CAAC,SAAS,CAAC;4BACzB,GAAG,EAAE,MAAM,CAAC,QAAQ,CAAC;yBACxB;qBACJ,CAAC,CAAC;gBACP,CAAC;YACL,CAAC;SACJ,CAAC;IACN,CAAC;CACJ,CAAC;AAEF,iBAAS,IAAI,CAAC","sourcesContent":["/**\n * ESLint rule to enforce maximum file length\n *\n * Enforces a configurable maximum line count for files.\n * Default: 700 lines\n *\n * Configuration:\n * '@webpieces/max-file-lines': ['error', { max: 700 }]\n */\n\nimport type { Rule } from 'eslint';\nimport * as fs from 'fs';\nimport * as path from 'path';\nimport { toError } from '../toError';\n\ninterface FileLinesOptions {\n    max: number;\n}\n\nconst FILE_DOC_CONTENT = `# AI Agent Instructions: File Too Long\n\n**READ THIS FILE to fix files that are too long**\n\n## Core Principle\nFiles should contain a SINGLE COHESIVE UNIT.\n- One class per file (Java convention)\n- If class is too large, extract child responsibilities\n- Use dependency injection to compose functionality\n\n## Command: Reduce File Size\n\n### Step 1: Check for Multiple Classes\nIf the file contains multiple classes, **SEPARATE each class into its own file**.\n\n\\`\\`\\`typescript\n// BAD: UserController.ts (multiple classes)\nexport class UserController { /* ... */ }\nexport class UserValidator { /* ... */ }\nexport class UserNotifier { /* ... */ }\n\n// GOOD: Three separate files\n// UserController.ts\nexport class UserController { /* ... */ }\n\n// UserValidator.ts\nexport class UserValidator { /* ... */ }\n\n// UserNotifier.ts\nexport class UserNotifier { /* ... */ }\n\\`\\`\\`\n\n### Step 2: Extract Child Responsibilities (if single class is too large)\n\n#### Pattern: Create New Service Class with Dependency Injection\n\n\\`\\`\\`typescript\n// BAD: UserController.ts (800 lines, single class)\n@provideSingleton()\n@Controller()\nexport class UserController {\n  // 200 lines: CRUD operations\n  // 300 lines: validation logic\n  // 200 lines: notification logic\n  // 100 lines: analytics logic\n}\n\n// GOOD: Extract validation service\n// 1. Create UserValidationService.ts\n@provideSingleton()\nexport class UserValidationService {\n  validateUserData(data: UserData): ValidationResult {\n    // 300 lines of validation logic moved here\n  }\n\n  validateEmail(email: string): boolean { /* ... */ }\n  validatePassword(password: string): boolean { /* ... */ }\n}\n\n// 2. Inject into UserController.ts\n@provideSingleton()\n@Controller()\nexport class UserController {\n  constructor(\n    @inject(TYPES.UserValidationService)\n    private validator: UserValidationService\n  ) {}\n\n  async createUser(data: UserData): Promise<User> {\n    const validation = this.validator.validateUserData(data);\n    if (!validation.isValid) {\n      throw new ValidationError(validation.errors);\n    }\n    // ... rest of logic\n  }\n}\n\\`\\`\\`\n\n## AI Agent Action Steps\n\n1. **ANALYZE** the file structure:\n   - Count classes (if >1, separate immediately)\n   - Identify logical responsibilities within single class\n\n2. **IDENTIFY** \"child code\" to extract:\n   - Validation logic -> ValidationService\n   - Notification logic -> NotificationService\n   - Data transformation -> TransformerService\n   - External API calls -> ApiService\n   - Business rules -> RulesEngine\n\n3. **CREATE** new service file(s):\n   - Start with temporary name: \\`XXXX.ts\\` or \\`ChildService.ts\\`\n   - Add \\`@provideSingleton()\\` decorator\n   - Move child methods to new class\n\n4. **UPDATE** dependency injection:\n   - Add to \\`TYPES\\` constants (if using symbol-based DI)\n   - Inject new service into original class constructor\n   - Replace direct method calls with \\`this.serviceName.method()\\`\n\n5. **RENAME** extracted file:\n   - Read the extracted code to understand its purpose\n   - Rename \\`XXXX.ts\\` to logical name (e.g., \\`UserValidationService.ts\\`)\n\n6. **VERIFY** file sizes:\n   - Original file should now be <700 lines\n   - Each extracted file should be <700 lines\n   - If still too large, extract more services\n\n## Examples of Child Responsibilities to Extract\n\n| If File Contains | Extract To | Pattern |\n|-----------------|------------|---------|\n| Validation logic (200+ lines) | \\`XValidator.ts\\` or \\`XValidationService.ts\\` | Singleton service |\n| Notification logic (150+ lines) | \\`XNotifier.ts\\` or \\`XNotificationService.ts\\` | Singleton service |\n| Data transformation (200+ lines) | \\`XTransformer.ts\\` | Singleton service |\n| External API calls (200+ lines) | \\`XApiClient.ts\\` | Singleton service |\n| Complex business rules (300+ lines) | \\`XRulesEngine.ts\\` | Singleton service |\n| Database queries (200+ lines) | \\`XRepository.ts\\` | Singleton service |\n\n## WebPieces Dependency Injection Pattern\n\n\\`\\`\\`typescript\n// 1. Define service with @provideSingleton\nimport { provideSingleton } from '@webpieces/http-routing';\n\n@provideSingleton()\nexport class MyService {\n  doSomething(): void { /* ... */ }\n}\n\n// 2. Inject into consumer\nimport { inject } from 'inversify';\nimport { TYPES } from './types';\n\n@provideSingleton()\n@Controller()\nexport class MyController {\n  constructor(\n    @inject(TYPES.MyService) private service: MyService\n  ) {}\n}\n\\`\\`\\`\n\nRemember: Find the \"child code\" and pull it down into a new class. Once moved, the code's purpose becomes clear, making it easy to rename to a logical name.\n`;\n\n// Module-level flag to prevent redundant file creation\nlet fileDocCreated = false;\n\nfunction getWorkspaceRoot(context: Rule.RuleContext): string {\n    const filename = context.filename || context.getFilename();\n    let dir = path.dirname(filename);\n\n    // Walk up directory tree to find workspace root\n    while (dir !== path.dirname(dir)) {\n        const pkgPath = path.join(dir, 'package.json');\n        if (fs.existsSync(pkgPath)) {\n            // eslint-disable-next-line @webpieces/no-unmanaged-exceptions\n            try {\n                const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf-8'));\n                if (pkg.workspaces || pkg.name === 'webpieces-ts') {\n                    return dir;\n                }\n            } catch (err: unknown) {\n                //const error = toError(err);\n                void err; // Continue searching if JSON parse fails\n            }\n        }\n        dir = path.dirname(dir);\n    }\n    return process.cwd(); // Fallback\n}\n\nfunction ensureDocFile(docPath: string, content: string): boolean {\n    // eslint-disable-next-line @webpieces/no-unmanaged-exceptions\n    try {\n        fs.mkdirSync(path.dirname(docPath), { recursive: true });\n        fs.writeFileSync(docPath, content, 'utf-8');\n        return true;\n    } catch (err: unknown) {\n        const error = toError(err);\n        console.warn(`[webpieces] Could not create doc file: ${docPath}`, error);\n        return false;\n    }\n}\n\nfunction ensureFileDoc(context: Rule.RuleContext): void {\n    if (fileDocCreated) return; // Performance: only create once per lint run\n\n    const workspaceRoot = getWorkspaceRoot(context);\n    const docPath = path.join(workspaceRoot, '.webpieces', 'instruct-ai', 'webpieces.filesize.md');\n\n    if (ensureDocFile(docPath, FILE_DOC_CONTENT)) {\n        fileDocCreated = true;\n    }\n}\n\nconst rule: Rule.RuleModule = {\n    meta: {\n        type: 'suggestion',\n        docs: {\n            description: 'Enforce maximum file length',\n            category: 'Best Practices',\n            recommended: false,\n            url: 'https://github.com/deanhiller/webpieces-ts',\n        },\n        messages: {\n            tooLong:\n                'AI Agent: READ .webpieces/instruct-ai/webpieces.filesize.md for fix instructions. File has {{actual}} lines (max: {{max}})',\n        },\n        fixable: undefined,\n        schema: [\n            {\n                type: 'object',\n                properties: {\n                    max: {\n                        type: 'integer',\n                        minimum: 1,\n                    },\n                },\n                additionalProperties: false,\n            },\n        ],\n    },\n\n    create(context: Rule.RuleContext): Rule.RuleListener {\n        const options = context.options[0] as FileLinesOptions | undefined;\n        const maxLines = options?.max ?? 700;\n\n        return {\n            // webpieces-disable no-any-unknown -- ESTree AST nodes require any for dynamic properties\n            Program(node: any): void {\n                ensureFileDoc(context);\n\n                const sourceCode = context.sourceCode || context.getSourceCode();\n                const lines = sourceCode.lines;\n                const lineCount = lines.length;\n\n                if (lineCount > maxLines) {\n                    context.report({\n                        node,\n                        messageId: 'tooLong',\n                        data: {\n                            actual: String(lineCount),\n                            max: String(maxLines),\n                        },\n                    });\n                }\n            },\n        };\n    },\n};\n\nexport = rule;\n"]}

package/src/rules/max-method-lines.d.ts

@@ -0,0 +1,12 @@
+/**
+ * ESLint rule to enforce maximum method length
+ *
+ * Enforces a configurable maximum line count for methods, functions, and arrow functions.
+ * Default: 70 lines
+ *
+ * Configuration:
+ * '@webpieces/max-method-lines': ['error', { max: 70 }]
+ */
+import type { Rule } from 'eslint';
+declare const rule: Rule.RuleModule;
+export = rule;

package/src/rules/{max-method-lines.ts → max-method-lines.js}

@@ -1,3 +1,4 @@
+"use strict";
 /**
  * ESLint rule to enforce maximum method length
  *
@@ -7,44 +8,10 @@
  * Configuration:
  * '@webpieces/max-method-lines': ['error', { max: 70 }]
  */
-
-import type { Rule } from 'eslint';
-import * as fs from 'fs';
-import * as path from 'path';
-import { toError } from '../toError';
-
-interface MethodLinesOptions {
-    max: number;
-}
-
-// webpieces-disable no-any-unknown -- ESTree AST nodes require any for dynamic properties
-interface FunctionNode {
-    type:
-        | 'FunctionDeclaration'
-        | 'FunctionExpression'
-        | 'ArrowFunctionExpression'
-        | 'MethodDefinition';
-    // webpieces-disable no-any-unknown -- ESTree AST dynamic body
-    body?: any;
-    loc?: {
-        start: { line: number };
-        end: { line: number };
-    };
-    key?: {
-        name?: string;
-    };
-    id?: {
-        name?: string;
-    };
-    // webpieces-disable no-any-unknown -- ESTree AST nodes require any for dynamic properties
-    [key: string]: any;
-}
-
-interface CheckerContext {
-    context: Rule.RuleContext;
-    maxLines: number;
-}
-
+const tslib_1 = require("tslib");
+const fs = tslib_1.__importStar(require("fs"));
+const path = tslib_1.__importStar(require("path"));
+const toError_1 = require("../toError");
 const METHOD_DOC_CONTENT = `# AI Agent Instructions: Method Too Long
 
 **READ THIS FILE to fix methods that are too long**
@@ -143,14 +110,11 @@ if (isAdminWithWriteAccess(user)) {
 
 Remember: Methods should read like a table of contents. Each line should be a "chapter title" (method call) that describes what happens, not how it happens.
 `;
-
 // Module-level flag to prevent redundant file creation
 let methodDocCreated = false;
-
-function getWorkspaceRoot(context: Rule.RuleContext): string {
+function getWorkspaceRoot(context) {
     const filename = context.filename || context.getFilename();
     let dir = path.dirname(filename);
-
     while (dir !== path.dirname(dir)) {
         const pkgPath = path.join(dir, 'package.json');
         if (fs.existsSync(pkgPath)) {
@@ -160,7 +124,8 @@ function getWorkspaceRoot(context: Rule.RuleContext): string {
                 if (pkg.workspaces || pkg.name === 'webpieces-ts') {
                     return dir;
                 }
-            } catch (err: unknown) {
+            }
+            catch (err) {
                 //const error = toError(err);
                 void err;
             }
@@ -169,33 +134,30 @@ function getWorkspaceRoot(context: Rule.RuleContext): string {
     }
     return process.cwd();
 }
-
-function ensureDocFile(docPath: string, content: string): boolean {
+function ensureDocFile(docPath, content) {
     // eslint-disable-next-line @webpieces/no-unmanaged-exceptions
     try {
         fs.mkdirSync(path.dirname(docPath), { recursive: true });
         fs.writeFileSync(docPath, content, 'utf-8');
         return true;
-    } catch (err: unknown) {
-        const error = toError(err);
+    }
+    catch (err) {
+        const error = (0, toError_1.toError)(err);
         console.warn(`[webpieces] Could not create doc file: ${docPath}`, error);
         return false;
     }
 }
-
-function ensureMethodDoc(context: Rule.RuleContext): void {
+function ensureMethodDoc(context) {
     const workspaceRoot = getWorkspaceRoot(context);
     const docPath = path.join(workspaceRoot, '.webpieces', 'instruct-ai', 'webpieces.methods.md');
-
     // Check if file exists AND flag is true - if both, skip
-    if (methodDocCreated && fs.existsSync(docPath)) return;
-
+    if (methodDocCreated && fs.existsSync(docPath))
+        return;
     if (ensureDocFile(docPath, METHOD_DOC_CONTENT)) {
         methodDocCreated = true;
     }
 }
-
-function getFunctionName(funcNode: FunctionNode): string {
+function getFunctionName(funcNode) {
     if (funcNode.type === 'FunctionDeclaration' && funcNode.id?.name) {
         return funcNode.id.name;
     }
@@ -204,9 +166,8 @@ function getFunctionName(funcNode: FunctionNode): string {
     }
     return 'anonymous';
 }
-
 // webpieces-disable no-any-unknown -- ESTree AST nodes require any for dynamic properties
-function reportTooLong(ctx: CheckerContext, node: any, name: string, lineCount: number): void {
+function reportTooLong(ctx, node, name, lineCount) {
     ctx.context.report({
         node,
         messageId: 'tooLong',
@@ -217,42 +178,34 @@ function reportTooLong(ctx: CheckerContext, node: any, name: string, lineCount:
         },
     });
 }
-
 // webpieces-disable no-any-unknown -- ESTree AST nodes require any for dynamic properties
-function checkFunctionNode(ctx: CheckerContext, node: any): void {
+function checkFunctionNode(ctx, node) {
     ensureMethodDoc(ctx.context);
-    const funcNode = node as FunctionNode;
-
+    const funcNode = node;
     // Skip function expressions inside method definitions
     if (funcNode.type === 'FunctionExpression' && funcNode['parent']?.type === 'MethodDefinition') {
         return;
     }
-
-    if (!funcNode.loc || !funcNode.body) return;
-
+    if (!funcNode.loc || !funcNode.body)
+        return;
     const name = getFunctionName(funcNode);
     const lineCount = funcNode.loc.end.line - funcNode.loc.start.line + 1;
-
     if (lineCount > ctx.maxLines) {
         reportTooLong(ctx, funcNode, name, lineCount);
     }
 }
-
 // webpieces-disable no-any-unknown -- ESTree AST nodes require any for dynamic properties
-function checkMethodNode(ctx: CheckerContext, node: any): void {
+function checkMethodNode(ctx, node) {
     ensureMethodDoc(ctx.context);
-
-    if (!node.loc || !node.value) return;
-
+    if (!node.loc || !node.value)
+        return;
     const name = node.key?.name || 'anonymous';
     const lineCount = node.loc.end.line - node.loc.start.line + 1;
-
     if (lineCount > ctx.maxLines) {
         reportTooLong(ctx, node, name, lineCount);
     }
 }
-
-const rule: Rule.RuleModule = {
+const rule = {
     meta: {
         type: 'suggestion',
         docs: {
@@ -262,8 +215,7 @@ const rule: Rule.RuleModule = {
             url: 'https://github.com/deanhiller/webpieces-ts',
         },
         messages: {
-            tooLong:
-                'AI Agent: READ .webpieces/instruct-ai/webpieces.methods.md for fix instructions. Method "{{name}}" has {{actual}} lines (max: {{max}})',
+            tooLong: 'AI Agent: READ .webpieces/instruct-ai/webpieces.methods.md for fix instructions. Method "{{name}}" has {{actual}} lines (max: {{max}})',
         },
         fixable: undefined,
         schema: [
@@ -279,11 +231,9 @@ const rule: Rule.RuleModule = {
             },
         ],
     },
-
-    create(context: Rule.RuleContext): Rule.RuleListener {
-        const options = context.options[0] as MethodLinesOptions | undefined;
-        const ctx: CheckerContext = { context, maxLines: options?.max ?? 70 };
-
+    create(context) {
+        const options = context.options[0];
+        const ctx = { context, maxLines: options?.max ?? 70 };
         return {
             FunctionDeclaration: (node) => checkFunctionNode(ctx, node),
             FunctionExpression: (node) => checkFunctionNode(ctx, node),
@@ -292,5 +242,5 @@ const rule: Rule.RuleModule = {
         };
     },
 };
-
-export = rule;
+module.exports = rule;
+//# sourceMappingURL=max-method-lines.js.map

package/src/rules/max-method-lines.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"max-method-lines.js","sourceRoot":"","sources":["../../../../../../packages/tooling/eslint-rules/src/rules/max-method-lines.ts"],"names":[],"mappings":";AAAA;;;;;;;;GAQG;;AAGH,+CAAyB;AACzB,mDAA6B;AAC7B,wCAAqC;AAkCrC,MAAM,kBAAkB,GAAG;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CAiG1B,CAAC;AAEF,uDAAuD;AACvD,IAAI,gBAAgB,GAAG,KAAK,CAAC;AAE7B,SAAS,gBAAgB,CAAC,OAAyB;IAC/C,MAAM,QAAQ,GAAG,OAAO,CAAC,QAAQ,IAAI,OAAO,CAAC,WAAW,EAAE,CAAC;IAC3D,IAAI,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;IAEjC,OAAO,GAAG,KAAK,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC;QAC/B,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,cAAc,CAAC,CAAC;QAC/C,IAAI,EAAE,CAAC,UAAU,CAAC,OAAO,CAAC,EAAE,CAAC;YACzB,8DAA8D;YAC9D,IAAI,CAAC;gBACD,MAAM,GAAG,GAAG,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,OAAO,CAAC,CAAC,CAAC;gBAC1D,IAAI,GAAG,CAAC,UAAU,IAAI,GAAG,CAAC,IAAI,KAAK,cAAc,EAAE,CAAC;oBAChD,OAAO,GAAG,CAAC;gBACf,CAAC;YACL,CAAC;YAAC,OAAO,GAAY,EAAE,CAAC;gBACpB,6BAA6B;gBAC7B,KAAK,GAAG,CAAC;YACb,CAAC;QACL,CAAC;QACD,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;IAC5B,CAAC;IACD,OAAO,OAAO,CAAC,GAAG,EAAE,CAAC;AACzB,CAAC;AAED,SAAS,aAAa,CAAC,OAAe,EAAE,OAAe;IACnD,8DAA8D;IAC9D,IAAI,CAAC;QACD,EAAE,CAAC,SAAS,CAAC,IAAI,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QACzD,EAAE,CAAC,aAAa,CAAC,OAAO,EAAE,OAAO,EAAE,OAAO,CAAC,CAAC;QAC5C,OAAO,IAAI,CAAC;IAChB,CAAC;IAAC,OAAO,GAAY,EAAE,CAAC;QACpB,MAAM,KAAK,GAAG,IAAA,iBAAO,EAAC,GAAG,CAAC,CAAC;QAC3B,OAAO,CAAC,IAAI,CAAC,0CAA0C,OAAO,EAAE,EAAE,KAAK,CAAC,CAAC;QACzE,OAAO,KAAK,CAAC;IACjB,CAAC;AACL,CAAC;AAED,SAAS,eAAe,CAAC,OAAyB;IAC9C,MAAM,aAAa,GAAG,gBAAgB,CAAC,OAAO,CAAC,CAAC;IAChD,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,aAAa,EAAE,YAAY,EAAE,aAAa,EAAE,sBAAsB,CAAC,CAAC;IAE9F,wDAAwD;IACxD,IAAI,gBAAgB,IAAI,EAAE,CAAC,UAAU,CAAC,OAAO,CAAC;QAAE,OAAO;IAEvD,IAAI,aAAa,CAAC,OAAO,EAAE,kBAAkB,CAAC,EAAE,CAAC;QAC7C,gBAAgB,GAAG,IAAI,CAAC;IAC5B,CAAC;AACL,CAAC;AAED,SAAS,eAAe,CAAC,QAAsB;IAC3C,IAAI,QAAQ,CAAC,IAAI,KAAK,qBAAqB,IAAI,QAAQ,CAAC,EAAE,EAAE,IAAI,EAAE,CAAC;QAC/D,OAAO,QAAQ,CAAC,EAAE,CAAC,IAAI,CAAC;IAC5B,CAAC;IACD,IAAI,QAAQ,CAAC,IAAI,KAAK,oBAAoB,IAAI,QAAQ,CAAC,EAAE,EAAE,IAAI,EAAE,CAAC;QAC9D,OAAO,QAAQ,CAAC,EAAE,CAAC,IAAI,CAAC;IAC5B,CAAC;IACD,OAAO,WAAW,CAAC;AACvB,CAAC;AAED,0FAA0F;AAC1F,SAAS,aAAa,CAAC,GAAmB,EAAE,IAAS,EAAE,IAAY,EAAE,SAAiB;IAClF,GAAG,CAAC,OAAO,CAAC,MAAM,CAAC;QACf,IAAI;QACJ,SAAS,EAAE,SAAS;QACpB,IAAI,EAAE;YACF,IAAI;YACJ,MAAM,EAAE,MAAM,CAAC,SAAS,CAAC;YACzB,GAAG,EAAE,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC;SAC5B;KACJ,CAAC,CAAC;AACP,CAAC;AAED,0FAA0F;AAC1F,SAAS,iBAAiB,CAAC,GAAmB,EAAE,IAAS;IACrD,eAAe,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;IAC7B,MAAM,QAAQ,GAAG,IAAoB,CAAC;IAEtC,sDAAsD;IACtD,IAAI,QAAQ,CAAC,IAAI,KAAK,oBAAoB,IAAI,QAAQ,CAAC,QAAQ,CAAC,EAAE,IAAI,KAAK,kBAAkB,EAAE,CAAC;QAC5F,OAAO;IACX,CAAC;IAED,IAAI,CAAC,QAAQ,CAAC,GAAG,IAAI,CAAC,QAAQ,CAAC,IAAI;QAAE,OAAO;IAE5C,MAAM,IAAI,GAAG,eAAe,CAAC,QAAQ,CAAC,CAAC;IACvC,MAAM,SAAS,GAAG,QAAQ,CAAC,GAAG,CAAC,GAAG,CAAC,IAAI,GAAG,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,GAAG,CAAC,CAAC;IAEtE,IAAI,SAAS,GAAG,GAAG,CAAC,QAAQ,EAAE,CAAC;QAC3B,aAAa,CAAC,GAAG,EAAE,QAAQ,EAAE,IAAI,EAAE,SAAS,CAAC,CAAC;IAClD,CAAC;AACL,CAAC;AAED,0FAA0F;AAC1F,SAAS,eAAe,CAAC,GAAmB,EAAE,IAAS;IACnD,eAAe,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;IAE7B,IAAI,CAAC,IAAI,CAAC,GAAG,IAAI,CAAC,IAAI,CAAC,KAAK;QAAE,OAAO;IAErC,MAAM,IAAI,GAAG,IAAI,CAAC,GAAG,EAAE,IAAI,IAAI,WAAW,CAAC;IAC3C,MAAM,SAAS,GAAG,IAAI,CAAC,GAAG,CAAC,GAAG,CAAC,IAAI,GAAG,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,GAAG,CAAC,CAAC;IAE9D,IAAI,SAAS,GAAG,GAAG,CAAC,QAAQ,EAAE,CAAC;QAC3B,aAAa,CAAC,GAAG,EAAE,IAAI,EAAE,IAAI,EAAE,SAAS,CAAC,CAAC;IAC9C,CAAC;AACL,CAAC;AAED,MAAM,IAAI,GAAoB;IAC1B,IAAI,EAAE;QACF,IAAI,EAAE,YAAY;QAClB,IAAI,EAAE;YACF,WAAW,EAAE,+BAA+B;YAC5C,QAAQ,EAAE,gBAAgB;YAC1B,WAAW,EAAE,KAAK;YAClB,GAAG,EAAE,4CAA4C;SACpD;QACD,QAAQ,EAAE;YACN,OAAO,EACH,wIAAwI;SAC/I;QACD,OAAO,EAAE,SAAS;QAClB,MAAM,EAAE;YACJ;gBACI,IAAI,EAAE,QAAQ;gBACd,UAAU,EAAE;oBACR,GAAG,EAAE;wBACD,IAAI,EAAE,SAAS;wBACf,OAAO,EAAE,CAAC;qBACb;iBACJ;gBACD,oBAAoB,EAAE,KAAK;aAC9B;SACJ;KACJ;IAED,MAAM,CAAC,OAAyB;QAC5B,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC,CAAmC,CAAC;QACrE,MAAM,GAAG,GAAmB,EAAE,OAAO,EAAE,QAAQ,EAAE,OAAO,EAAE,GAAG,IAAI,EAAE,EAAE,CAAC;QAEtE,OAAO;YACH,mBAAmB,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,iBAAiB,CAAC,GAAG,EAAE,IAAI,CAAC;YAC3D,kBAAkB,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,iBAAiB,CAAC,GAAG,EAAE,IAAI,CAAC;YAC1D,uBAAuB,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,iBAAiB,CAAC,GAAG,EAAE,IAAI,CAAC;YAC/D,gBAAgB,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,eAAe,CAAC,GAAG,EAAE,IAAI,CAAC;SACzD,CAAC;IACN,CAAC;CACJ,CAAC;AAEF,iBAAS,IAAI,CAAC","sourcesContent":["/**\n * ESLint rule to enforce maximum method length\n *\n * Enforces a configurable maximum line count for methods, functions, and arrow functions.\n * Default: 70 lines\n *\n * Configuration:\n * '@webpieces/max-method-lines': ['error', { max: 70 }]\n */\n\nimport type { Rule } from 'eslint';\nimport * as fs from 'fs';\nimport * as path from 'path';\nimport { toError } from '../toError';\n\ninterface MethodLinesOptions {\n    max: number;\n}\n\n// webpieces-disable no-any-unknown -- ESTree AST nodes require any for dynamic properties\ninterface FunctionNode {\n    type:\n        | 'FunctionDeclaration'\n        | 'FunctionExpression'\n        | 'ArrowFunctionExpression'\n        | 'MethodDefinition';\n    // webpieces-disable no-any-unknown -- ESTree AST dynamic body\n    body?: any;\n    loc?: {\n        start: { line: number };\n        end: { line: number };\n    };\n    key?: {\n        name?: string;\n    };\n    id?: {\n        name?: string;\n    };\n    // webpieces-disable no-any-unknown -- ESTree AST nodes require any for dynamic properties\n    [key: string]: any;\n}\n\ninterface CheckerContext {\n    context: Rule.RuleContext;\n    maxLines: number;\n}\n\nconst METHOD_DOC_CONTENT = `# AI Agent Instructions: Method Too Long\n\n**READ THIS FILE to fix methods that are too long**\n\n## Core Principle\nEvery method should read like a TABLE OF CONTENTS of a book.\n- Each method call is a \"chapter\"\n- When you dive into a method, you find another table of contents\n- Keeping methods under 70 lines is achievable with proper extraction\n\n## Command: Extract Code into Named Methods\n\n### Pattern 1: Extract Loop Bodies\n\\`\\`\\`typescript\n// BAD: 50 lines embedded in loop\nfor (const order of orders) {\n  // 20 lines of validation logic\n  // 15 lines of processing logic\n  // 10 lines of notification logic\n}\n\n// GOOD: Extracted to named methods\nfor (const order of orders) {\n  validateOrder(order);\n  processOrderItems(order);\n  sendNotifications(order);\n}\n\\`\\`\\`\n\n### Pattern 2: Try-Catch Wrapper for Exception Handling\n\\`\\`\\`typescript\n// GOOD: Separates success path from error handling\nasync function handleRequest(req: Request): Promise<Response> {\n  // eslint-disable-next-line @webpieces/no-unmanaged-exceptions\n  try {\n    return await executeRequest(req);\n  } catch (err: unknown) {\n    const error = toError(err);\n    return createErrorResponse(error);\n  }\n}\n\\`\\`\\`\n\n### Pattern 3: Sequential Method Calls (Table of Contents)\n\\`\\`\\`typescript\n// GOOD: Self-documenting steps\nfunction processOrder(order: Order): void {\n  validateOrderData(order);\n  calculateTotals(order);\n  applyDiscounts(order);\n  processPayment(order);\n  updateInventory(order);\n  sendConfirmation(order);\n}\n\\`\\`\\`\n\n### Pattern 4: Separate Data Object Creation\n\\`\\`\\`typescript\n// BAD: 15 lines of inline object creation\ndoSomething({ field1: ..., field2: ..., field3: ..., /* 15 more fields */ });\n\n// GOOD: Extract to factory method\nconst request = createRequestObject(data);\ndoSomething(request);\n\\`\\`\\`\n\n### Pattern 5: Extract Inline Logic to Named Functions\n\\`\\`\\`typescript\n// BAD: Complex inline logic\nif (user.role === 'admin' && user.permissions.includes('write') && !user.suspended) {\n  // 30 lines of admin logic\n}\n\n// GOOD: Extract to named methods\nif (isAdminWithWriteAccess(user)) {\n  performAdminOperation(user);\n}\n\\`\\`\\`\n\n## AI Agent Action Steps\n\n1. **IDENTIFY** the long method in the error message\n2. **READ** the method to understand its logical sections\n3. **EXTRACT** logical units into separate methods with descriptive names\n4. **REPLACE** inline code with method calls\n5. **VERIFY** each extracted method is <70 lines\n6. **TEST** that functionality remains unchanged\n\n## Examples of \"Logical Units\" to Extract\n- Validation logic -> \\`validateX()\\`\n- Data transformation -> \\`transformXToY()\\`\n- API calls -> \\`fetchXFromApi()\\`\n- Object creation -> \\`createX()\\`\n- Loop bodies -> \\`processItem()\\`\n- Error handling -> \\`handleXError()\\`\n\nRemember: Methods should read like a table of contents. Each line should be a \"chapter title\" (method call) that describes what happens, not how it happens.\n`;\n\n// Module-level flag to prevent redundant file creation\nlet methodDocCreated = false;\n\nfunction getWorkspaceRoot(context: Rule.RuleContext): string {\n    const filename = context.filename || context.getFilename();\n    let dir = path.dirname(filename);\n\n    while (dir !== path.dirname(dir)) {\n        const pkgPath = path.join(dir, 'package.json');\n        if (fs.existsSync(pkgPath)) {\n            // eslint-disable-next-line @webpieces/no-unmanaged-exceptions\n            try {\n                const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf-8'));\n                if (pkg.workspaces || pkg.name === 'webpieces-ts') {\n                    return dir;\n                }\n            } catch (err: unknown) {\n                //const error = toError(err);\n                void err;\n            }\n        }\n        dir = path.dirname(dir);\n    }\n    return process.cwd();\n}\n\nfunction ensureDocFile(docPath: string, content: string): boolean {\n    // eslint-disable-next-line @webpieces/no-unmanaged-exceptions\n    try {\n        fs.mkdirSync(path.dirname(docPath), { recursive: true });\n        fs.writeFileSync(docPath, content, 'utf-8');\n        return true;\n    } catch (err: unknown) {\n        const error = toError(err);\n        console.warn(`[webpieces] Could not create doc file: ${docPath}`, error);\n        return false;\n    }\n}\n\nfunction ensureMethodDoc(context: Rule.RuleContext): void {\n    const workspaceRoot = getWorkspaceRoot(context);\n    const docPath = path.join(workspaceRoot, '.webpieces', 'instruct-ai', 'webpieces.methods.md');\n\n    // Check if file exists AND flag is true - if both, skip\n    if (methodDocCreated && fs.existsSync(docPath)) return;\n\n    if (ensureDocFile(docPath, METHOD_DOC_CONTENT)) {\n        methodDocCreated = true;\n    }\n}\n\nfunction getFunctionName(funcNode: FunctionNode): string {\n    if (funcNode.type === 'FunctionDeclaration' && funcNode.id?.name) {\n        return funcNode.id.name;\n    }\n    if (funcNode.type === 'FunctionExpression' && funcNode.id?.name) {\n        return funcNode.id.name;\n    }\n    return 'anonymous';\n}\n\n// webpieces-disable no-any-unknown -- ESTree AST nodes require any for dynamic properties\nfunction reportTooLong(ctx: CheckerContext, node: any, name: string, lineCount: number): void {\n    ctx.context.report({\n        node,\n        messageId: 'tooLong',\n        data: {\n            name,\n            actual: String(lineCount),\n            max: String(ctx.maxLines),\n        },\n    });\n}\n\n// webpieces-disable no-any-unknown -- ESTree AST nodes require any for dynamic properties\nfunction checkFunctionNode(ctx: CheckerContext, node: any): void {\n    ensureMethodDoc(ctx.context);\n    const funcNode = node as FunctionNode;\n\n    // Skip function expressions inside method definitions\n    if (funcNode.type === 'FunctionExpression' && funcNode['parent']?.type === 'MethodDefinition') {\n        return;\n    }\n\n    if (!funcNode.loc || !funcNode.body) return;\n\n    const name = getFunctionName(funcNode);\n    const lineCount = funcNode.loc.end.line - funcNode.loc.start.line + 1;\n\n    if (lineCount > ctx.maxLines) {\n        reportTooLong(ctx, funcNode, name, lineCount);\n    }\n}\n\n// webpieces-disable no-any-unknown -- ESTree AST nodes require any for dynamic properties\nfunction checkMethodNode(ctx: CheckerContext, node: any): void {\n    ensureMethodDoc(ctx.context);\n\n    if (!node.loc || !node.value) return;\n\n    const name = node.key?.name || 'anonymous';\n    const lineCount = node.loc.end.line - node.loc.start.line + 1;\n\n    if (lineCount > ctx.maxLines) {\n        reportTooLong(ctx, node, name, lineCount);\n    }\n}\n\nconst rule: Rule.RuleModule = {\n    meta: {\n        type: 'suggestion',\n        docs: {\n            description: 'Enforce maximum method length',\n            category: 'Best Practices',\n            recommended: false,\n            url: 'https://github.com/deanhiller/webpieces-ts',\n        },\n        messages: {\n            tooLong:\n                'AI Agent: READ .webpieces/instruct-ai/webpieces.methods.md for fix instructions. Method \"{{name}}\" has {{actual}} lines (max: {{max}})',\n        },\n        fixable: undefined,\n        schema: [\n            {\n                type: 'object',\n                properties: {\n                    max: {\n                        type: 'integer',\n                        minimum: 1,\n                    },\n                },\n                additionalProperties: false,\n            },\n        ],\n    },\n\n    create(context: Rule.RuleContext): Rule.RuleListener {\n        const options = context.options[0] as MethodLinesOptions | undefined;\n        const ctx: CheckerContext = { context, maxLines: options?.max ?? 70 };\n\n        return {\n            FunctionDeclaration: (node) => checkFunctionNode(ctx, node),\n            FunctionExpression: (node) => checkFunctionNode(ctx, node),\n            ArrowFunctionExpression: (node) => checkFunctionNode(ctx, node),\n            MethodDefinition: (node) => checkMethodNode(ctx, node),\n        };\n    },\n};\n\nexport = rule;\n"]}

package/src/rules/no-json-property-primitive-type.d.ts

@@ -0,0 +1,17 @@
+/**
+ * ESLint rule: no-json-property-primitive-type
+ *
+ * Bans @JsonProperty({ type: String }), @JsonProperty({ type: Number }),
+ * and @JsonProperty({ type: Boolean }).
+ *
+ * These pass the TypeScript build but break production deserialization.
+ * The typescript-json-serializer `type` option expects class constructors,
+ * not JavaScript primitive constructors.
+ *
+ * Correct usage:
+ *   @JsonProperty()                      - for primitive arrays (string[], number[], boolean[])
+ *   @JsonProperty({ type: MyDtoClass })  - for class types only
+ */
+import type { Rule } from 'eslint';
+declare const rule: Rule.RuleModule;
+export = rule;

package/src/rules/no-json-property-primitive-type.js

@@ -0,0 +1,57 @@
+"use strict";
+/**
+ * ESLint rule: no-json-property-primitive-type
+ *
+ * Bans @JsonProperty({ type: String }), @JsonProperty({ type: Number }),
+ * and @JsonProperty({ type: Boolean }).
+ *
+ * These pass the TypeScript build but break production deserialization.
+ * The typescript-json-serializer `type` option expects class constructors,
+ * not JavaScript primitive constructors.
+ *
+ * Correct usage:
+ *   @JsonProperty()                      - for primitive arrays (string[], number[], boolean[])
+ *   @JsonProperty({ type: MyDtoClass })  - for class types only
+ */
+const BANNED_PRIMITIVES = ['String', 'Number', 'Boolean'];
+const rule = {
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Ban @JsonProperty({ type: String/Number/Boolean }) — breaks production deserialization',
+        },
+        messages: {
+            noPrimitiveType: '@JsonProperty({ type: {{ primitive }} }) breaks production deserialization. ' +
+                'For primitive arrays (string[], number[], boolean[]), use @JsonProperty() with ' +
+                'no type parameter. The type option is only for class types: ' +
+                '@JsonProperty({ type: MyDtoClass }).',
+        },
+        schema: [],
+    },
+    create(context) {
+        return {
+            CallExpression(node) {
+                if (node.callee.name !== 'JsonProperty')
+                    return;
+                const arg = node.arguments[0];
+                if (!arg || arg.type !== 'ObjectExpression')
+                    return;
+                const objArg = arg;
+                for (const prop of objArg.properties) {
+                    if (prop.key?.name === 'type' &&
+                        prop.value?.type === 'Identifier' &&
+                        BANNED_PRIMITIVES.includes(prop.value.name)) {
+                        context.report({
+                            // webpieces-disable no-any-unknown -- ESTree AST cast for ESLint report
+                            node: prop,
+                            messageId: 'noPrimitiveType',
+                            data: { primitive: prop.value.name },
+                        });
+                    }
+                }
+            },
+        };
+    },
+};
+module.exports = rule;
+//# sourceMappingURL=no-json-property-primitive-type.js.map

package/src/rules/no-json-property-primitive-type.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-json-property-primitive-type.js","sourceRoot":"","sources":["../../../../../../packages/tooling/eslint-rules/src/rules/no-json-property-primitive-type.ts"],"names":[],"mappings":";AAAA;;;;;;;;;;;;;GAaG;AA2BH,MAAM,iBAAiB,GAAG,CAAC,QAAQ,EAAE,QAAQ,EAAE,SAAS,CAAC,CAAC;AAE1D,MAAM,IAAI,GAAoB;IAC1B,IAAI,EAAE;QACF,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACF,WAAW,EACP,wFAAwF;SAC/F;QACD,QAAQ,EAAE;YACN,eAAe,EACX,8EAA8E;gBAC9E,iFAAiF;gBACjF,8DAA8D;gBAC9D,sCAAsC;SAC7C;QACD,MAAM,EAAE,EAAE;KACb;IACD,MAAM,CAAC,OAAyB;QAC5B,OAAO;YACH,cAAc,CAAC,IAAwB;gBACnC,IAAI,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,cAAc;oBAAE,OAAO;gBAChD,MAAM,GAAG,GAAG,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC;gBAC9B,IAAI,CAAC,GAAG,IAAI,GAAG,CAAC,IAAI,KAAK,kBAAkB;oBAAE,OAAO;gBACpD,MAAM,MAAM,GAAG,GAA2B,CAAC;gBAC3C,KAAK,MAAM,IAAI,IAAI,MAAM,CAAC,UAAU,EAAE,CAAC;oBACnC,IACI,IAAI,CAAC,GAAG,EAAE,IAAI,KAAK,MAAM;wBACzB,IAAI,CAAC,KAAK,EAAE,IAAI,KAAK,YAAY;wBACjC,iBAAiB,CAAC,QAAQ,CAAC,IAAI,CAAC,KAAK,CAAC,IAAK,CAAC,EAC9C,CAAC;wBACC,OAAO,CAAC,MAAM,CAAC;4BACX,wEAAwE;4BACxE,IAAI,EAAE,IAA4B;4BAClC,SAAS,EAAE,iBAAiB;4BAC5B,IAAI,EAAE,EAAE,SAAS,EAAE,IAAI,CAAC,KAAK,CAAC,IAAK,EAAE;yBACxC,CAAC,CAAC;oBACP,CAAC;gBACL,CAAC;YACL,CAAC;SACJ,CAAC;IACN,CAAC;CACJ,CAAC;AAEF,iBAAS,IAAI,CAAC","sourcesContent":["/**\n * ESLint rule: no-json-property-primitive-type\n *\n * Bans @JsonProperty({ type: String }), @JsonProperty({ type: Number }),\n * and @JsonProperty({ type: Boolean }).\n *\n * These pass the TypeScript build but break production deserialization.\n * The typescript-json-serializer `type` option expects class constructors,\n * not JavaScript primitive constructors.\n *\n * Correct usage:\n *   @JsonProperty()                      - for primitive arrays (string[], number[], boolean[])\n *   @JsonProperty({ type: MyDtoClass })  - for class types only\n */\n\nimport type { Rule } from 'eslint';\n\n// webpieces-disable no-any-unknown -- ESTree AST node interfaces require any for dynamic properties\ninterface CallExpressionNode {\n    type: 'CallExpression';\n    // webpieces-disable no-any-unknown -- ESTree AST dynamic callee\n    callee: { name?: string; [key: string]: any };\n    // webpieces-disable no-any-unknown -- ESTree AST dynamic arguments array\n    arguments: any[];\n    // webpieces-disable no-any-unknown -- ESTree AST index signature\n    [key: string]: any;\n}\n\ninterface ObjectExpressionNode {\n    type: 'ObjectExpression';\n    properties: PropertyNode[];\n}\n\ninterface PropertyNode {\n    key?: { name?: string };\n    value?: { type?: string; name?: string };\n    // webpieces-disable no-any-unknown -- ESTree AST index signature\n    [key: string]: any;\n}\n\nconst BANNED_PRIMITIVES = ['String', 'Number', 'Boolean'];\n\nconst rule: Rule.RuleModule = {\n    meta: {\n        type: 'problem',\n        docs: {\n            description:\n                'Ban @JsonProperty({ type: String/Number/Boolean }) — breaks production deserialization',\n        },\n        messages: {\n            noPrimitiveType:\n                '@JsonProperty({ type: {{ primitive }} }) breaks production deserialization. ' +\n                'For primitive arrays (string[], number[], boolean[]), use @JsonProperty() with ' +\n                'no type parameter. The type option is only for class types: ' +\n                '@JsonProperty({ type: MyDtoClass }).',\n        },\n        schema: [],\n    },\n    create(context: Rule.RuleContext): Rule.RuleListener {\n        return {\n            CallExpression(node: CallExpressionNode): void {\n                if (node.callee.name !== 'JsonProperty') return;\n                const arg = node.arguments[0];\n                if (!arg || arg.type !== 'ObjectExpression') return;\n                const objArg = arg as ObjectExpressionNode;\n                for (const prop of objArg.properties) {\n                    if (\n                        prop.key?.name === 'type' &&\n                        prop.value?.type === 'Identifier' &&\n                        BANNED_PRIMITIVES.includes(prop.value.name!)\n                    ) {\n                        context.report({\n                            // webpieces-disable no-any-unknown -- ESTree AST cast for ESLint report\n                            node: prop as unknown as Rule.Node,\n                            messageId: 'noPrimitiveType',\n                            data: { primitive: prop.value.name! },\n                        });\n                    }\n                }\n            },\n        };\n    },\n};\n\nexport = rule;\n"]}

package/src/rules/no-mat-cell-def.d.ts

@@ -0,0 +1,15 @@
+/**
+ * ESLint rule: no-mat-cell-def
+ *
+ * Bans *matCellDef and *matHeaderCellDef in Angular HTML templates.
+ * New files should use the div-grid table pattern instead of mat-table.
+ *
+ * Works with @angular-eslint/template-parser AST where structural directives
+ * (*matCellDef) are desugared into Template nodes with templateAttrs[].
+ *
+ * NOTE: This rule only works when files are parsed with @angular-eslint/template-parser.
+ * It is intended for Angular HTML template files (**.html).
+ */
+import type { Rule } from 'eslint';
+declare const rule: Rule.RuleModule;
+export = rule;

package/src/rules/{no-mat-cell-def.ts → no-mat-cell-def.js}

@@ -1,3 +1,4 @@
+"use strict";
 /**
  * ESLint rule: no-mat-cell-def
  *
@@ -10,36 +11,22 @@
  * NOTE: This rule only works when files are parsed with @angular-eslint/template-parser.
  * It is intended for Angular HTML template files (**.html).
  */
-
-import type { Rule } from 'eslint';
-
-// webpieces-disable no-any-unknown -- Angular template AST node interfaces
-// These interfaces represent the Template node shape from @angular-eslint/template-parser.
-// We define them inline since the parser is not a dependency of this plugin.
-interface AngularTemplateNode {
-    templateAttrs?: Array<{ name: string }>;
-    // webpieces-disable no-any-unknown -- ESTree AST index signature
-    [key: string]: any;
-}
-
 const BANNED_DIRECTIVES = ['matCellDef', 'matHeaderCellDef'];
-
-const rule: Rule.RuleModule = {
+const rule = {
     meta: {
         type: 'problem',
         docs: {
             description: 'Ban *matCellDef and *matHeaderCellDef — use div-grid tables instead',
         },
         messages: {
-            noMatCellDef:
-                '*{{ directive }} is banned in new files. Use the div-grid table pattern instead. ' +
+            noMatCellDef: '*{{ directive }} is banned in new files. Use the div-grid table pattern instead. ' +
                 'Div-grid tables are inherently type-safe with @for loops + strictTemplates.',
         },
         schema: [],
     },
-    create(context: Rule.RuleContext): Rule.RuleListener {
+    create(context) {
         return {
-            Template(node: AngularTemplateNode): void {
+            Template(node) {
                 // Structural directives (*matCellDef) are desugared into Template nodes.
                 // The directive name appears in node.templateAttrs as either a
                 // BoundAttribute or TextAttribute.
@@ -48,7 +35,7 @@ const rule: Rule.RuleModule = {
                     if (BANNED_DIRECTIVES.includes(attr.name)) {
                         context.report({
                             // webpieces-disable no-any-unknown -- ESTree AST cast for ESLint report
-                            node: node as unknown as Rule.Node,
+                            node: node,
                             messageId: 'noMatCellDef',
                             data: { directive: attr.name },
                         });
@@ -58,5 +45,5 @@ const rule: Rule.RuleModule = {
         };
     },
 };
-
-export = rule;
+module.exports = rule;
+//# sourceMappingURL=no-mat-cell-def.js.map

package/src/rules/no-mat-cell-def.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-mat-cell-def.js","sourceRoot":"","sources":["../../../../../../packages/tooling/eslint-rules/src/rules/no-mat-cell-def.ts"],"names":[],"mappings":";AAAA;;;;;;;;;;;GAWG;AAaH,MAAM,iBAAiB,GAAG,CAAC,YAAY,EAAE,kBAAkB,CAAC,CAAC;AAE7D,MAAM,IAAI,GAAoB;IAC1B,IAAI,EAAE;QACF,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACF,WAAW,EAAE,qEAAqE;SACrF;QACD,QAAQ,EAAE;YACN,YAAY,EACR,mFAAmF;gBACnF,6EAA6E;SACpF;QACD,MAAM,EAAE,EAAE;KACb;IACD,MAAM,CAAC,OAAyB;QAC5B,OAAO;YACH,QAAQ,CAAC,IAAyB;gBAC9B,yEAAyE;gBACzE,+DAA+D;gBAC/D,mCAAmC;gBACnC,MAAM,KAAK,GAAG,IAAI,CAAC,aAAa,IAAI,EAAE,CAAC;gBACvC,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;oBACvB,IAAI,iBAAiB,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;wBACxC,OAAO,CAAC,MAAM,CAAC;4BACX,wEAAwE;4BACxE,IAAI,EAAE,IAA4B;4BAClC,SAAS,EAAE,cAAc;4BACzB,IAAI,EAAE,EAAE,SAAS,EAAE,IAAI,CAAC,IAAI,EAAE;yBACjC,CAAC,CAAC;oBACP,CAAC;gBACL,CAAC;YACL,CAAC;SACJ,CAAC;IACN,CAAC;CACJ,CAAC;AAEF,iBAAS,IAAI,CAAC","sourcesContent":["/**\n * ESLint rule: no-mat-cell-def\n *\n * Bans *matCellDef and *matHeaderCellDef in Angular HTML templates.\n * New files should use the div-grid table pattern instead of mat-table.\n *\n * Works with @angular-eslint/template-parser AST where structural directives\n * (*matCellDef) are desugared into Template nodes with templateAttrs[].\n *\n * NOTE: This rule only works when files are parsed with @angular-eslint/template-parser.\n * It is intended for Angular HTML template files (**.html).\n */\n\nimport type { Rule } from 'eslint';\n\n// webpieces-disable no-any-unknown -- Angular template AST node interfaces\n// These interfaces represent the Template node shape from @angular-eslint/template-parser.\n// We define them inline since the parser is not a dependency of this plugin.\ninterface AngularTemplateNode {\n    templateAttrs?: Array<{ name: string }>;\n    // webpieces-disable no-any-unknown -- ESTree AST index signature\n    [key: string]: any;\n}\n\nconst BANNED_DIRECTIVES = ['matCellDef', 'matHeaderCellDef'];\n\nconst rule: Rule.RuleModule = {\n    meta: {\n        type: 'problem',\n        docs: {\n            description: 'Ban *matCellDef and *matHeaderCellDef — use div-grid tables instead',\n        },\n        messages: {\n            noMatCellDef:\n                '*{{ directive }} is banned in new files. Use the div-grid table pattern instead. ' +\n                'Div-grid tables are inherently type-safe with @for loops + strictTemplates.',\n        },\n        schema: [],\n    },\n    create(context: Rule.RuleContext): Rule.RuleListener {\n        return {\n            Template(node: AngularTemplateNode): void {\n                // Structural directives (*matCellDef) are desugared into Template nodes.\n                // The directive name appears in node.templateAttrs as either a\n                // BoundAttribute or TextAttribute.\n                const attrs = node.templateAttrs || [];\n                for (const attr of attrs) {\n                    if (BANNED_DIRECTIVES.includes(attr.name)) {\n                        context.report({\n                            // webpieces-disable no-any-unknown -- ESTree AST cast for ESLint report\n                            node: node as unknown as Rule.Node,\n                            messageId: 'noMatCellDef',\n                            data: { directive: attr.name },\n                        });\n                    }\n                }\n            },\n        };\n    },\n};\n\nexport = rule;\n"]}

package/src/rules/no-unmanaged-exceptions.d.ts

@@ -0,0 +1,22 @@
+/**
+ * ESLint rule to discourage try-catch blocks outside test files
+ *
+ * Works alongside catch-error-pattern rule:
+ * - catch-error-pattern: Enforces HOW to handle exceptions (with toError())
+ * - no-unmanaged-exceptions: Enforces WHERE try-catch is allowed (tests only by default)
+ *
+ * Philosophy: Exceptions should bubble to global error handlers where they are logged
+ * with traceId and stored for debugging via /debugLocal and /debugCloud endpoints.
+ * Local try-catch blocks break this architecture and create blind spots in production.
+ *
+ * Auto-allowed in:
+ * - Test files (.test.ts, .spec.ts, __tests__/)
+ *
+ * Requires eslint-disable comment in:
+ * - Retry loops with exponential backoff
+ * - Batch processing where partial failure is expected
+ * - Resource cleanup (with approval)
+ */
+import type { Rule } from 'eslint';
+declare const rule: Rule.RuleModule;
+export = rule;