@webpieces/eslint-rules 0.0.1 → 0.2.114

package/src/rules/{no-unmanaged-exceptions.ts → no-unmanaged-exceptions.js}

@@ -1,3 +1,4 @@
+"use strict";
 /**
  * ESLint rule to discourage try-catch blocks outside test files
  *
@@ -17,45 +18,32 @@
  * - Batch processing where partial failure is expected
  * - Resource cleanup (with approval)
  */
-
-import type { Rule } from 'eslint';
-import * as fs from 'fs';
-import * as path from 'path';
-import { toError } from '../toError';
-
-// webpieces-disable no-any-unknown -- ESTree AST node interface
-interface TryStatementNode {
-    handler?: unknown;
-}
-
+const tslib_1 = require("tslib");
+const fs = tslib_1.__importStar(require("fs"));
+const path = tslib_1.__importStar(require("path"));
 /**
  * Determines if a file is a test file based on naming conventions
  * Test files are auto-allowed to use try-catch blocks
  */
-function isTestFile(filename: string): boolean {
+function isTestFile(filename) {
     const normalizedPath = filename.toLowerCase();
-
     // Check file extensions
     if (normalizedPath.endsWith('.test.ts') || normalizedPath.endsWith('.spec.ts')) {
         return true;
     }
-
     // Check directory names (cross-platform)
     if (normalizedPath.includes('/__tests__/') || normalizedPath.includes('\\__tests__\\')) {
         return true;
     }
-
     return false;
 }
-
 /**
  * Finds the workspace root by walking up the directory tree
  * Looks for package.json with workspaces or name === 'webpieces-ts'
  */
-function getWorkspaceRoot(context: Rule.RuleContext): string {
+function getWorkspaceRoot(context) {
     const filename = context.filename || context.getFilename();
     let dir = path.dirname(filename);
-
     // Walk up directory tree
     for (let i = 0; i < 10; i++) {
         const pkgPath = path.join(dir, 'package.json');
@@ -67,46 +55,43 @@ 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; // Invalid JSON, keep searching
             }
         }
-
         const parentDir = path.dirname(dir);
-        if (parentDir === dir) break; // Reached filesystem root
+        if (parentDir === dir)
+            break; // Reached filesystem root
         dir = parentDir;
     }
-
     // Fallback: return current directory
     return process.cwd();
 }
-
 /**
  * Ensures a documentation file exists at the given path
  * Creates parent directories if needed
  */
-function ensureDocFile(docPath: string, content: string): boolean {
+function ensureDocFile(docPath, content) {
     // eslint-disable-next-line @webpieces/no-unmanaged-exceptions
     try {
         const dir = path.dirname(docPath);
         if (!fs.existsSync(dir)) {
             fs.mkdirSync(dir, { recursive: true });
         }
-
         // Only write if file doesn't exist or is empty
         if (!fs.existsSync(docPath) || fs.readFileSync(docPath, 'utf-8').trim() === '') {
             fs.writeFileSync(docPath, content, 'utf-8');
         }
-
         return true;
-    } catch (err: unknown) {
+    }
+    catch (err) {
         //const error = toError(err);
         void err; // Silently fail - don't break linting if file creation fails
         return false;
     }
 }
-
 /**
  * Ensures the exception documentation markdown file exists
  * Only creates file once per lint run using module-level flag
@@ -114,39 +99,34 @@ function ensureDocFile(docPath: string, content: string): boolean {
  * Reads from the template file packaged with @webpieces/webpieces-rules
  * and copies it to .webpieces/instruct-ai/ for AI agents to read.
  */
-function ensureExceptionDoc(context: Rule.RuleContext): void {
-    if (exceptionDocCreated) return;
-
+function ensureExceptionDoc(context) {
+    if (exceptionDocCreated)
+        return;
     const workspaceRoot = getWorkspaceRoot(context);
     const docPath = path.join(workspaceRoot, '.webpieces', 'instruct-ai', 'webpieces.exceptions.md');
-
     // Read from the template file packaged with the npm module
     // Path: from eslint-plugin/rules/ -> ../../templates/
     const templatePath = path.join(__dirname, '..', '..', 'templates', 'webpieces.exceptions.md');
-
-    let content: string;
+    let content;
     // eslint-disable-next-line @webpieces/no-unmanaged-exceptions
     try {
         content = fs.readFileSync(templatePath, 'utf-8');
-    } catch (err: unknown) {
+    }
+    catch (err) {
         //const error = toError(err);
         void err;
         // Fallback message if template not found (shouldn't happen in published package)
         content = `# Exception Documentation Not Found\n\nTemplate file not found at: ${templatePath}\n\nPlease ensure @webpieces/webpieces-rules is properly installed.`;
     }
-
     if (ensureDocFile(docPath, content)) {
         exceptionDocCreated = true;
     }
 }
-
 // Module-level flag to prevent redundant markdown file creation
 let exceptionDocCreated = false;
-
 // NOTE: Documentation content moved to templates/webpieces.exceptions.md
 // The ensureExceptionDoc function reads from that file at runtime.
-
-const rule: Rule.RuleModule = {
+const rule = {
     meta: {
         type: 'problem',
         docs: {
@@ -156,39 +136,34 @@ const rule: Rule.RuleModule = {
             url: 'https://github.com/deanhiller/webpieces-ts/blob/main/CLAUDE.md#exception-handling-philosophy',
         },
         messages: {
-            noUnmanagedExceptions:
-                'AI Agent: READ .webpieces/instruct-ai/webpieces.exceptions.md for context. Try-catch blocks are discouraged - use global error handlers instead. Only allowed in test files or with eslint-disable comment.',
+            noUnmanagedExceptions: 'AI Agent: READ .webpieces/instruct-ai/webpieces.exceptions.md for context. Try-catch blocks are discouraged - use global error handlers instead. Only allowed in test files or with eslint-disable comment.',
         },
         fixable: undefined,
         schema: [],
     },
-
-    create(context: Rule.RuleContext): Rule.RuleListener {
+    create(context) {
         return {
             // webpieces-disable no-any-unknown -- ESLint visitor callback parameter type
-            TryStatement(node: unknown): void {
+            TryStatement(node) {
                 // Skip try..finally blocks (no catch handler, no exception handling)
                 // webpieces-disable no-any-unknown -- ESTree AST node type assertion
-                if (!(node as TryStatementNode).handler) {
+                if (!node.handler) {
                     return;
                 }
-
                 // Auto-allow in test files
                 const filename = context.filename || context.getFilename();
                 if (isTestFile(filename)) {
                     return;
                 }
-
                 // Has catch block outside test file - report violation
                 ensureExceptionDoc(context);
                 context.report({
-                    node: node as Rule.Node,
+                    node: node,
                     messageId: 'noUnmanagedExceptions',
                 });
             },
         };
     },
 };
-
-export = rule;
-
+module.exports = rule;
+//# sourceMappingURL=no-unmanaged-exceptions.js.map

package/src/rules/no-unmanaged-exceptions.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-unmanaged-exceptions.js","sourceRoot":"","sources":["../../../../../../packages/tooling/eslint-rules/src/rules/no-unmanaged-exceptions.ts"],"names":[],"mappings":";AAAA;;;;;;;;;;;;;;;;;;GAkBG;;AAGH,+CAAyB;AACzB,mDAA6B;AAQ7B;;;GAGG;AACH,SAAS,UAAU,CAAC,QAAgB;IAChC,MAAM,cAAc,GAAG,QAAQ,CAAC,WAAW,EAAE,CAAC;IAE9C,wBAAwB;IACxB,IAAI,cAAc,CAAC,QAAQ,CAAC,UAAU,CAAC,IAAI,cAAc,CAAC,QAAQ,CAAC,UAAU,CAAC,EAAE,CAAC;QAC7E,OAAO,IAAI,CAAC;IAChB,CAAC;IAED,yCAAyC;IACzC,IAAI,cAAc,CAAC,QAAQ,CAAC,aAAa,CAAC,IAAI,cAAc,CAAC,QAAQ,CAAC,eAAe,CAAC,EAAE,CAAC;QACrF,OAAO,IAAI,CAAC;IAChB,CAAC;IAED,OAAO,KAAK,CAAC;AACjB,CAAC;AAED;;;GAGG;AACH,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,yBAAyB;IACzB,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC;QAC1B,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,sCAAsC;gBACtC,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,+BAA+B;YAC7C,CAAC;QACL,CAAC;QAED,MAAM,SAAS,GAAG,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;QACpC,IAAI,SAAS,KAAK,GAAG;YAAE,MAAM,CAAC,0BAA0B;QACxD,GAAG,GAAG,SAAS,CAAC;IACpB,CAAC;IAED,qCAAqC;IACrC,OAAO,OAAO,CAAC,GAAG,EAAE,CAAC;AACzB,CAAC;AAED;;;GAGG;AACH,SAAS,aAAa,CAAC,OAAe,EAAE,OAAe;IACnD,8DAA8D;IAC9D,IAAI,CAAC;QACD,MAAM,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC;QAClC,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;YACtB,EAAE,CAAC,SAAS,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAC3C,CAAC;QAED,+CAA+C;QAC/C,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,OAAO,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE,CAAC;YAC7E,EAAE,CAAC,aAAa,CAAC,OAAO,EAAE,OAAO,EAAE,OAAO,CAAC,CAAC;QAChD,CAAC;QAED,OAAO,IAAI,CAAC;IAChB,CAAC;IAAC,OAAO,GAAY,EAAE,CAAC;QACpB,6BAA6B;QAC7B,KAAK,GAAG,CAAC,CAAC,6DAA6D;QACvE,OAAO,KAAK,CAAC;IACjB,CAAC;AACL,CAAC;AAED;;;;;;GAMG;AACH,SAAS,kBAAkB,CAAC,OAAyB;IACjD,IAAI,mBAAmB;QAAE,OAAO;IAEhC,MAAM,aAAa,GAAG,gBAAgB,CAAC,OAAO,CAAC,CAAC;IAChD,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,aAAa,EAAE,YAAY,EAAE,aAAa,EAAE,yBAAyB,CAAC,CAAC;IAEjG,2DAA2D;IAC3D,sDAAsD;IACtD,MAAM,YAAY,GAAG,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,IAAI,EAAE,IAAI,EAAE,WAAW,EAAE,yBAAyB,CAAC,CAAC;IAE9F,IAAI,OAAe,CAAC;IACpB,8DAA8D;IAC9D,IAAI,CAAC;QACD,OAAO,GAAG,EAAE,CAAC,YAAY,CAAC,YAAY,EAAE,OAAO,CAAC,CAAC;IACrD,CAAC;IAAC,OAAO,GAAY,EAAE,CAAC;QACpB,6BAA6B;QAC7B,KAAK,GAAG,CAAC;QACT,iFAAiF;QACjF,OAAO,GAAG,sEAAsE,YAAY,qEAAqE,CAAC;IACtK,CAAC;IAED,IAAI,aAAa,CAAC,OAAO,EAAE,OAAO,CAAC,EAAE,CAAC;QAClC,mBAAmB,GAAG,IAAI,CAAC;IAC/B,CAAC;AACL,CAAC;AAED,gEAAgE;AAChE,IAAI,mBAAmB,GAAG,KAAK,CAAC;AAEhC,yEAAyE;AACzE,mEAAmE;AAEnE,MAAM,IAAI,GAAoB;IAC1B,IAAI,EAAE;QACF,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACF,WAAW,EAAE,4EAA4E;YACzF,QAAQ,EAAE,gBAAgB;YAC1B,WAAW,EAAE,IAAI;YACjB,GAAG,EAAE,8FAA8F;SACtG;QACD,QAAQ,EAAE;YACN,qBAAqB,EACjB,6MAA6M;SACpN;QACD,OAAO,EAAE,SAAS;QAClB,MAAM,EAAE,EAAE;KACb;IAED,MAAM,CAAC,OAAyB;QAC5B,OAAO;YACH,6EAA6E;YAC7E,YAAY,CAAC,IAAa;gBACtB,qEAAqE;gBACrE,qEAAqE;gBACrE,IAAI,CAAE,IAAyB,CAAC,OAAO,EAAE,CAAC;oBACtC,OAAO;gBACX,CAAC;gBAED,2BAA2B;gBAC3B,MAAM,QAAQ,GAAG,OAAO,CAAC,QAAQ,IAAI,OAAO,CAAC,WAAW,EAAE,CAAC;gBAC3D,IAAI,UAAU,CAAC,QAAQ,CAAC,EAAE,CAAC;oBACvB,OAAO;gBACX,CAAC;gBAED,uDAAuD;gBACvD,kBAAkB,CAAC,OAAO,CAAC,CAAC;gBAC5B,OAAO,CAAC,MAAM,CAAC;oBACX,IAAI,EAAE,IAAiB;oBACvB,SAAS,EAAE,uBAAuB;iBACrC,CAAC,CAAC;YACP,CAAC;SACJ,CAAC;IACN,CAAC;CACJ,CAAC;AAEF,iBAAS,IAAI,CAAC","sourcesContent":["/**\n * ESLint rule to discourage try-catch blocks outside test files\n *\n * Works alongside catch-error-pattern rule:\n * - catch-error-pattern: Enforces HOW to handle exceptions (with toError())\n * - no-unmanaged-exceptions: Enforces WHERE try-catch is allowed (tests only by default)\n *\n * Philosophy: Exceptions should bubble to global error handlers where they are logged\n * with traceId and stored for debugging via /debugLocal and /debugCloud endpoints.\n * Local try-catch blocks break this architecture and create blind spots in production.\n *\n * Auto-allowed in:\n * - Test files (.test.ts, .spec.ts, __tests__/)\n *\n * Requires eslint-disable comment in:\n * - Retry loops with exponential backoff\n * - Batch processing where partial failure is expected\n * - Resource cleanup (with approval)\n */\n\nimport type { Rule } from 'eslint';\nimport * as fs from 'fs';\nimport * as path from 'path';\nimport { toError } from '../toError';\n\n// webpieces-disable no-any-unknown -- ESTree AST node interface\ninterface TryStatementNode {\n    handler?: unknown;\n}\n\n/**\n * Determines if a file is a test file based on naming conventions\n * Test files are auto-allowed to use try-catch blocks\n */\nfunction isTestFile(filename: string): boolean {\n    const normalizedPath = filename.toLowerCase();\n\n    // Check file extensions\n    if (normalizedPath.endsWith('.test.ts') || normalizedPath.endsWith('.spec.ts')) {\n        return true;\n    }\n\n    // Check directory names (cross-platform)\n    if (normalizedPath.includes('/__tests__/') || normalizedPath.includes('\\\\__tests__\\\\')) {\n        return true;\n    }\n\n    return false;\n}\n\n/**\n * Finds the workspace root by walking up the directory tree\n * Looks for package.json with workspaces or name === 'webpieces-ts'\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\n    for (let i = 0; i < 10; i++) {\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                // Check if this is the root workspace\n                if (pkg.workspaces || pkg.name === 'webpieces-ts') {\n                    return dir;\n                }\n            } catch (err: unknown) {\n                //const error = toError(err);\n                void err; // Invalid JSON, keep searching\n            }\n        }\n\n        const parentDir = path.dirname(dir);\n        if (parentDir === dir) break; // Reached filesystem root\n        dir = parentDir;\n    }\n\n    // Fallback: return current directory\n    return process.cwd();\n}\n\n/**\n * Ensures a documentation file exists at the given path\n * Creates parent directories if needed\n */\nfunction ensureDocFile(docPath: string, content: string): boolean {\n    // eslint-disable-next-line @webpieces/no-unmanaged-exceptions\n    try {\n        const dir = path.dirname(docPath);\n        if (!fs.existsSync(dir)) {\n            fs.mkdirSync(dir, { recursive: true });\n        }\n\n        // Only write if file doesn't exist or is empty\n        if (!fs.existsSync(docPath) || fs.readFileSync(docPath, 'utf-8').trim() === '') {\n            fs.writeFileSync(docPath, content, 'utf-8');\n        }\n\n        return true;\n    } catch (err: unknown) {\n        //const error = toError(err);\n        void err; // Silently fail - don't break linting if file creation fails\n        return false;\n    }\n}\n\n/**\n * Ensures the exception documentation markdown file exists\n * Only creates file once per lint run using module-level flag\n *\n * Reads from the template file packaged with @webpieces/webpieces-rules\n * and copies it to .webpieces/instruct-ai/ for AI agents to read.\n */\nfunction ensureExceptionDoc(context: Rule.RuleContext): void {\n    if (exceptionDocCreated) return;\n\n    const workspaceRoot = getWorkspaceRoot(context);\n    const docPath = path.join(workspaceRoot, '.webpieces', 'instruct-ai', 'webpieces.exceptions.md');\n\n    // Read from the template file packaged with the npm module\n    // Path: from eslint-plugin/rules/ -> ../../templates/\n    const templatePath = path.join(__dirname, '..', '..', 'templates', 'webpieces.exceptions.md');\n\n    let content: string;\n    // eslint-disable-next-line @webpieces/no-unmanaged-exceptions\n    try {\n        content = fs.readFileSync(templatePath, 'utf-8');\n    } catch (err: unknown) {\n        //const error = toError(err);\n        void err;\n        // Fallback message if template not found (shouldn't happen in published package)\n        content = `# Exception Documentation Not Found\\n\\nTemplate file not found at: ${templatePath}\\n\\nPlease ensure @webpieces/webpieces-rules is properly installed.`;\n    }\n\n    if (ensureDocFile(docPath, content)) {\n        exceptionDocCreated = true;\n    }\n}\n\n// Module-level flag to prevent redundant markdown file creation\nlet exceptionDocCreated = false;\n\n// NOTE: Documentation content moved to templates/webpieces.exceptions.md\n// The ensureExceptionDoc function reads from that file at runtime.\n\nconst rule: Rule.RuleModule = {\n    meta: {\n        type: 'problem',\n        docs: {\n            description: 'Discourage try-catch blocks outside test files - use global error handlers',\n            category: 'Best Practices',\n            recommended: true,\n            url: 'https://github.com/deanhiller/webpieces-ts/blob/main/CLAUDE.md#exception-handling-philosophy',\n        },\n        messages: {\n            noUnmanagedExceptions:\n                'AI Agent: READ .webpieces/instruct-ai/webpieces.exceptions.md for context. Try-catch blocks are discouraged - use global error handlers instead. Only allowed in test files or with eslint-disable comment.',\n        },\n        fixable: undefined,\n        schema: [],\n    },\n\n    create(context: Rule.RuleContext): Rule.RuleListener {\n        return {\n            // webpieces-disable no-any-unknown -- ESLint visitor callback parameter type\n            TryStatement(node: unknown): void {\n                // Skip try..finally blocks (no catch handler, no exception handling)\n                // webpieces-disable no-any-unknown -- ESTree AST node type assertion\n                if (!(node as TryStatementNode).handler) {\n                    return;\n                }\n\n                // Auto-allow in test files\n                const filename = context.filename || context.getFilename();\n                if (isTestFile(filename)) {\n                    return;\n                }\n\n                // Has catch block outside test file - report violation\n                ensureExceptionDoc(context);\n                context.report({\n                    node: node as Rule.Node,\n                    messageId: 'noUnmanagedExceptions',\n                });\n            },\n        };\n    },\n};\n\nexport = rule;\n\n"]}

package/src/rules/require-typed-template.d.ts

@@ -0,0 +1,17 @@
+/**
+ * ESLint rule: require-typed-template
+ *
+ * Enforces that every <ng-template> with let- variables also has [templateClassType]
+ * to preserve type safety via TypedTemplateOutletDirective.
+ *
+ * Works with @angular-eslint/template-parser AST where:
+ *   - ng-template variables (let-xxx) appear in node.variables[]
+ *   - bound inputs ([templateClassType]) appear in node.inputs[]
+ *   - static attributes appear in node.attributes[]
+ *
+ * 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/{require-typed-template.ts → require-typed-template.js}

@@ -1,3 +1,4 @@
+"use strict";
 /**
  * ESLint rule: require-typed-template
  *
@@ -12,31 +13,14 @@
  * 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 {
-    tagName?: string;
-    variables?: Array<{ name: string }>;
-    inputs?: Array<{ name: string }>;
-    attributes?: Array<{ name: string }>;
-    // webpieces-disable no-any-unknown -- ESTree AST index signature
-    [key: string]: any;
-}
-
-const rule: Rule.RuleModule = {
+const rule = {
     meta: {
         type: 'problem',
         docs: {
-            description:
-                'Require [templateClassType] on ng-template elements that use let- variables',
+            description: 'Require [templateClassType] on ng-template elements that use let- variables',
         },
         messages: {
-            missingTypedTemplate:
-                'ng-template with let- variables must include ' +
+            missingTypedTemplate: 'ng-template with let- variables must include ' +
                 '[templateClassType]="YourDtoClass" to preserve type safety. ' +
                 'Fix: (1) Add [templateClassType]="YourDtoClass" to this ng-template, ' +
                 '(2) Add TypedTemplateOutletDirective to component imports array, ' +
@@ -45,30 +29,26 @@ const rule: Rule.RuleModule = {
         },
         schema: [],
     },
-    create(context: Rule.RuleContext): Rule.RuleListener {
+    create(context) {
         return {
-            Template(node: AngularTemplateNode): void {
+            Template(node) {
                 // Only match explicit <ng-template>, not desugared structural directives
                 // (*ngFor, *ngIf, etc.) which also produce Template AST nodes
                 if (node.tagName !== 'ng-template') {
                     return;
                 }
-
                 const hasLetVariables = node.variables && node.variables.length > 0;
                 if (!hasLetVariables) {
                     return;
                 }
-
-                const hasTemplateClassType =
-                    (node.inputs &&
-                        node.inputs.some((input) => input.name === 'templateClassType')) ||
+                const hasTemplateClassType = (node.inputs &&
+                    node.inputs.some((input) => input.name === 'templateClassType')) ||
                     (node.attributes &&
                         node.attributes.some((attr) => attr.name === 'templateClassType'));
-
                 if (!hasTemplateClassType) {
                     context.report({
                         // webpieces-disable no-any-unknown -- ESTree AST cast for ESLint report
-                        node: node as unknown as Rule.Node,
+                        node: node,
                         messageId: 'missingTypedTemplate',
                     });
                 }
@@ -76,5 +56,5 @@ const rule: Rule.RuleModule = {
         };
     },
 };
-
-export = rule;
+module.exports = rule;
+//# sourceMappingURL=require-typed-template.js.map

package/src/rules/require-typed-template.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"require-typed-template.js","sourceRoot":"","sources":["../../../../../../packages/tooling/eslint-rules/src/rules/require-typed-template.ts"],"names":[],"mappings":";AAAA;;;;;;;;;;;;;GAaG;AAgBH,MAAM,IAAI,GAAoB;IAC1B,IAAI,EAAE;QACF,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACF,WAAW,EACP,6EAA6E;SACpF;QACD,QAAQ,EAAE;YACN,oBAAoB,EAChB,+CAA+C;gBAC/C,8DAA8D;gBAC9D,uEAAuE;gBACvE,mEAAmE;gBACnE,4EAA4E;gBAC5E,8CAA8C;SACrD;QACD,MAAM,EAAE,EAAE;KACb;IACD,MAAM,CAAC,OAAyB;QAC5B,OAAO;YACH,QAAQ,CAAC,IAAyB;gBAC9B,yEAAyE;gBACzE,8DAA8D;gBAC9D,IAAI,IAAI,CAAC,OAAO,KAAK,aAAa,EAAE,CAAC;oBACjC,OAAO;gBACX,CAAC;gBAED,MAAM,eAAe,GAAG,IAAI,CAAC,SAAS,IAAI,IAAI,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,CAAC;gBACpE,IAAI,CAAC,eAAe,EAAE,CAAC;oBACnB,OAAO;gBACX,CAAC;gBAED,MAAM,oBAAoB,GACtB,CAAC,IAAI,CAAC,MAAM;oBACR,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,IAAI,KAAK,mBAAmB,CAAC,CAAC;oBACpE,CAAC,IAAI,CAAC,UAAU;wBACZ,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,IAAI,KAAK,mBAAmB,CAAC,CAAC,CAAC;gBAE3E,IAAI,CAAC,oBAAoB,EAAE,CAAC;oBACxB,OAAO,CAAC,MAAM,CAAC;wBACX,wEAAwE;wBACxE,IAAI,EAAE,IAA4B;wBAClC,SAAS,EAAE,sBAAsB;qBACpC,CAAC,CAAC;gBACP,CAAC;YACL,CAAC;SACJ,CAAC;IACN,CAAC;CACJ,CAAC;AAEF,iBAAS,IAAI,CAAC","sourcesContent":["/**\n * ESLint rule: require-typed-template\n *\n * Enforces that every <ng-template> with let- variables also has [templateClassType]\n * to preserve type safety via TypedTemplateOutletDirective.\n *\n * Works with @angular-eslint/template-parser AST where:\n *   - ng-template variables (let-xxx) appear in node.variables[]\n *   - bound inputs ([templateClassType]) appear in node.inputs[]\n *   - static attributes appear in node.attributes[]\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    tagName?: string;\n    variables?: Array<{ name: string }>;\n    inputs?: Array<{ name: string }>;\n    attributes?: Array<{ name: string }>;\n    // webpieces-disable no-any-unknown -- ESTree AST index signature\n    [key: string]: any;\n}\n\nconst rule: Rule.RuleModule = {\n    meta: {\n        type: 'problem',\n        docs: {\n            description:\n                'Require [templateClassType] on ng-template elements that use let- variables',\n        },\n        messages: {\n            missingTypedTemplate:\n                'ng-template with let- variables must include ' +\n                '[templateClassType]=\"YourDtoClass\" to preserve type safety. ' +\n                'Fix: (1) Add [templateClassType]=\"YourDtoClass\" to this ng-template, ' +\n                '(2) Add TypedTemplateOutletDirective to component imports array, ' +\n                '(3) Expose the DTO class: protected readonly YourDtoClass = YourDtoClass. ' +\n                'See @fuse/directives/typed-template-outlet/.',\n        },\n        schema: [],\n    },\n    create(context: Rule.RuleContext): Rule.RuleListener {\n        return {\n            Template(node: AngularTemplateNode): void {\n                // Only match explicit <ng-template>, not desugared structural directives\n                // (*ngFor, *ngIf, etc.) which also produce Template AST nodes\n                if (node.tagName !== 'ng-template') {\n                    return;\n                }\n\n                const hasLetVariables = node.variables && node.variables.length > 0;\n                if (!hasLetVariables) {\n                    return;\n                }\n\n                const hasTemplateClassType =\n                    (node.inputs &&\n                        node.inputs.some((input) => input.name === 'templateClassType')) ||\n                    (node.attributes &&\n                        node.attributes.some((attr) => attr.name === 'templateClassType'));\n\n                if (!hasTemplateClassType) {\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: 'missingTypedTemplate',\n                    });\n                }\n            },\n        };\n    },\n};\n\nexport = rule;\n"]}

package/src/toError.d.ts

@@ -0,0 +1,5 @@
+/**
+ * Lightweight duplicate of @webpieces/core-util toError for use in eslint-plugin.
+ * eslint-plugin is Level 0 and cannot depend on core-util (also Level 0).
+ */
+export declare function toError(err: unknown): Error;

package/src/{toError.ts → toError.js}

@@ -1,17 +1,18 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.toError = toError;
 /**
  * Lightweight duplicate of @webpieces/core-util toError for use in eslint-plugin.
  * eslint-plugin is Level 0 and cannot depend on core-util (also Level 0).
  */
 // webpieces-disable no-any-unknown -- toError intentionally accepts unknown to safely convert any thrown value to Error
-export function toError(err: unknown): Error {
+function toError(err) {
     if (err instanceof Error) {
         return err;
     }
-
     if (err && typeof err === 'object') {
         if ('message' in err) {
             const error = new Error(String(err.message));
-
             if ('stack' in err && typeof err.stack === 'string') {
                 error.stack = err.stack;
             }
@@ -20,17 +21,17 @@ export function toError(err: unknown): Error {
             }
             return error;
         }
-
         // eslint-disable-next-line @webpieces/no-unmanaged-exceptions -- must handle circular references without recursion
         try {
             return new Error(`Non-Error object thrown: ${JSON.stringify(err)}`);
-        } catch (err: unknown) {
+        }
+        catch (err) {
             //const error = toError(err);
             void err;
             return new Error('Non-Error object thrown (unable to stringify)');
         }
     }
-
     const message = err == null ? 'Null or undefined thrown' : String(err);
     return new Error(message);
 }
+//# sourceMappingURL=toError.js.map

package/src/toError.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"toError.js","sourceRoot":"","sources":["../../../../../packages/tooling/eslint-rules/src/toError.ts"],"names":[],"mappings":";;AAKA,0BA8BC;AAnCD;;;GAGG;AACH,wHAAwH;AACxH,SAAgB,OAAO,CAAC,GAAY;IAChC,IAAI,GAAG,YAAY,KAAK,EAAE,CAAC;QACvB,OAAO,GAAG,CAAC;IACf,CAAC;IAED,IAAI,GAAG,IAAI,OAAO,GAAG,KAAK,QAAQ,EAAE,CAAC;QACjC,IAAI,SAAS,IAAI,GAAG,EAAE,CAAC;YACnB,MAAM,KAAK,GAAG,IAAI,KAAK,CAAC,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC;YAE7C,IAAI,OAAO,IAAI,GAAG,IAAI,OAAO,GAAG,CAAC,KAAK,KAAK,QAAQ,EAAE,CAAC;gBAClD,KAAK,CAAC,KAAK,GAAG,GAAG,CAAC,KAAK,CAAC;YAC5B,CAAC;YACD,IAAI,MAAM,IAAI,GAAG,IAAI,OAAO,GAAG,CAAC,IAAI,KAAK,QAAQ,EAAE,CAAC;gBAChD,KAAK,CAAC,IAAI,GAAG,GAAG,CAAC,IAAI,CAAC;YAC1B,CAAC;YACD,OAAO,KAAK,CAAC;QACjB,CAAC;QAED,mHAAmH;QACnH,IAAI,CAAC;YACD,OAAO,IAAI,KAAK,CAAC,4BAA4B,IAAI,CAAC,SAAS,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;QACxE,CAAC;QAAC,OAAO,GAAY,EAAE,CAAC;YACpB,6BAA6B;YAC7B,KAAK,GAAG,CAAC;YACT,OAAO,IAAI,KAAK,CAAC,+CAA+C,CAAC,CAAC;QACtE,CAAC;IACL,CAAC;IAED,MAAM,OAAO,GAAG,GAAG,IAAI,IAAI,CAAC,CAAC,CAAC,0BAA0B,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;IACvE,OAAO,IAAI,KAAK,CAAC,OAAO,CAAC,CAAC;AAC9B,CAAC","sourcesContent":["/**\n * Lightweight duplicate of @webpieces/core-util toError for use in eslint-plugin.\n * eslint-plugin is Level 0 and cannot depend on core-util (also Level 0).\n */\n// webpieces-disable no-any-unknown -- toError intentionally accepts unknown to safely convert any thrown value to Error\nexport function toError(err: unknown): Error {\n    if (err instanceof Error) {\n        return err;\n    }\n\n    if (err && typeof err === 'object') {\n        if ('message' in err) {\n            const error = new Error(String(err.message));\n\n            if ('stack' in err && typeof err.stack === 'string') {\n                error.stack = err.stack;\n            }\n            if ('name' in err && typeof err.name === 'string') {\n                error.name = err.name;\n            }\n            return error;\n        }\n\n        // eslint-disable-next-line @webpieces/no-unmanaged-exceptions -- must handle circular references without recursion\n        try {\n            return new Error(`Non-Error object thrown: ${JSON.stringify(err)}`);\n        } catch (err: unknown) {\n            //const error = toError(err);\n            void err;\n            return new Error('Non-Error object thrown (unable to stringify)');\n        }\n    }\n\n    const message = err == null ? 'Null or undefined thrown' : String(err);\n    return new Error(message);\n}\n"]}

package/.webpieces/instruct-ai/webpieces.exceptions.md

@@ -1,5 +0,0 @@
-# Exception Documentation Not Found
-
-Template file not found at: /Users/deanhiller/openclaw/personal/webpieces-ts30/packages/tooling/eslint-plugin/templates/webpieces.exceptions.md
-
-Please ensure @webpieces/dev-config is properly installed.

package/.webpieces/instruct-ai/webpieces.filesize.md

@@ -1,146 +0,0 @@
-# AI Agent Instructions: File Too Long
-
-**READ THIS FILE to fix files that are too long**
-
-## Core Principle
-Files should contain a SINGLE COHESIVE UNIT.
-- One class per file (Java convention)
-- If class is too large, extract child responsibilities
-- Use dependency injection to compose functionality
-
-## Command: Reduce File Size
-
-### Step 1: Check for Multiple Classes
-If the file contains multiple classes, **SEPARATE each class into its own file**.
-
-```typescript
-// BAD: UserController.ts (multiple classes)
-export class UserController { /* ... */ }
-export class UserValidator { /* ... */ }
-export class UserNotifier { /* ... */ }
-
-// GOOD: Three separate files
-// UserController.ts
-export class UserController { /* ... */ }
-
-// UserValidator.ts
-export class UserValidator { /* ... */ }
-
-// UserNotifier.ts
-export class UserNotifier { /* ... */ }
-```
-
-### Step 2: Extract Child Responsibilities (if single class is too large)
-
-#### Pattern: Create New Service Class with Dependency Injection
-
-```typescript
-// BAD: UserController.ts (800 lines, single class)
-@provideSingleton()
-@Controller()
-export class UserController {
-  // 200 lines: CRUD operations
-  // 300 lines: validation logic
-  // 200 lines: notification logic
-  // 100 lines: analytics logic
-}
-
-// GOOD: Extract validation service
-// 1. Create UserValidationService.ts
-@provideSingleton()
-export class UserValidationService {
-  validateUserData(data: UserData): ValidationResult {
-    // 300 lines of validation logic moved here
-  }
-
-  validateEmail(email: string): boolean { /* ... */ }
-  validatePassword(password: string): boolean { /* ... */ }
-}
-
-// 2. Inject into UserController.ts
-@provideSingleton()
-@Controller()
-export class UserController {
-  constructor(
-    @inject(TYPES.UserValidationService)
-    private validator: UserValidationService
-  ) {}
-
-  async createUser(data: UserData): Promise<User> {
-    const validation = this.validator.validateUserData(data);
-    if (!validation.isValid) {
-      throw new ValidationError(validation.errors);
-    }
-    // ... rest of logic
-  }
-}
-```
-
-## AI Agent Action Steps
-
-1. **ANALYZE** the file structure:
-   - Count classes (if >1, separate immediately)
-   - Identify logical responsibilities within single class
-
-2. **IDENTIFY** "child code" to extract:
-   - Validation logic -> ValidationService
-   - Notification logic -> NotificationService
-   - Data transformation -> TransformerService
-   - External API calls -> ApiService
-   - Business rules -> RulesEngine
-
-3. **CREATE** new service file(s):
-   - Start with temporary name: `XXXX.ts` or `ChildService.ts`
-   - Add `@provideSingleton()` decorator
-   - Move child methods to new class
-
-4. **UPDATE** dependency injection:
-   - Add to `TYPES` constants (if using symbol-based DI)
-   - Inject new service into original class constructor
-   - Replace direct method calls with `this.serviceName.method()`
-
-5. **RENAME** extracted file:
-   - Read the extracted code to understand its purpose
-   - Rename `XXXX.ts` to logical name (e.g., `UserValidationService.ts`)
-
-6. **VERIFY** file sizes:
-   - Original file should now be <700 lines
-   - Each extracted file should be <700 lines
-   - If still too large, extract more services
-
-## Examples of Child Responsibilities to Extract
-
-| If File Contains | Extract To | Pattern |
-|-----------------|------------|---------|
-| Validation logic (200+ lines) | `XValidator.ts` or `XValidationService.ts` | Singleton service |
-| Notification logic (150+ lines) | `XNotifier.ts` or `XNotificationService.ts` | Singleton service |
-| Data transformation (200+ lines) | `XTransformer.ts` | Singleton service |
-| External API calls (200+ lines) | `XApiClient.ts` | Singleton service |
-| Complex business rules (300+ lines) | `XRulesEngine.ts` | Singleton service |
-| Database queries (200+ lines) | `XRepository.ts` | Singleton service |
-
-## WebPieces Dependency Injection Pattern
-
-```typescript
-// 1. Define service with @provideSingleton
-import { provideSingleton } from '@webpieces/http-routing';
-
-@provideSingleton()
-export class MyService {
-  doSomething(): void { /* ... */ }
-}
-
-// 2. Inject into consumer
-import { inject } from 'inversify';
-import { TYPES } from './types';
-
-@provideSingleton()
-@Controller()
-export class MyController {
-  constructor(
-    @inject(TYPES.MyService) private service: MyService
-  ) {}
-}
-```
-
-Remember: 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.

package/.webpieces/instruct-ai/webpieces.methods.md

@@ -1,97 +0,0 @@
-# AI Agent Instructions: Method Too Long
-
-**READ THIS FILE to fix methods that are too long**
-
-## Core Principle
-Every method should read like a TABLE OF CONTENTS of a book.
-- Each method call is a "chapter"
-- When you dive into a method, you find another table of contents
-- Keeping methods under 70 lines is achievable with proper extraction
-
-## Command: Extract Code into Named Methods
-
-### Pattern 1: Extract Loop Bodies
-```typescript
-// BAD: 50 lines embedded in loop
-for (const order of orders) {
-  // 20 lines of validation logic
-  // 15 lines of processing logic
-  // 10 lines of notification logic
-}
-
-// GOOD: Extracted to named methods
-for (const order of orders) {
-  validateOrder(order);
-  processOrderItems(order);
-  sendNotifications(order);
-}
-```
-
-### Pattern 2: Try-Catch Wrapper for Exception Handling
-```typescript
-// GOOD: Separates success path from error handling
-async function handleRequest(req: Request): Promise<Response> {
-  // eslint-disable-next-line @webpieces/no-unmanaged-exceptions
-  try {
-    return await executeRequest(req);
-  } catch (err: unknown) {
-    const error = toError(err);
-    return createErrorResponse(error);
-  }
-}
-```
-
-### Pattern 3: Sequential Method Calls (Table of Contents)
-```typescript
-// GOOD: Self-documenting steps
-function processOrder(order: Order): void {
-  validateOrderData(order);
-  calculateTotals(order);
-  applyDiscounts(order);
-  processPayment(order);
-  updateInventory(order);
-  sendConfirmation(order);
-}
-```
-
-### Pattern 4: Separate Data Object Creation
-```typescript
-// BAD: 15 lines of inline object creation
-doSomething({ field1: ..., field2: ..., field3: ..., /* 15 more fields */ });
-
-// GOOD: Extract to factory method
-const request = createRequestObject(data);
-doSomething(request);
-```
-
-### Pattern 5: Extract Inline Logic to Named Functions
-```typescript
-// BAD: Complex inline logic
-if (user.role === 'admin' && user.permissions.includes('write') && !user.suspended) {
-  // 30 lines of admin logic
-}
-
-// GOOD: Extract to named methods
-if (isAdminWithWriteAccess(user)) {
-  performAdminOperation(user);
-}
-```
-
-## AI Agent Action Steps
-
-1. **IDENTIFY** the long method in the error message
-2. **READ** the method to understand its logical sections
-3. **EXTRACT** logical units into separate methods with descriptive names
-4. **REPLACE** inline code with method calls
-5. **VERIFY** each extracted method is <70 lines
-6. **TEST** that functionality remains unchanged
-
-## Examples of "Logical Units" to Extract
-- Validation logic -> `validateX()`
-- Data transformation -> `transformXToY()`
-- API calls -> `fetchXFromApi()`
-- Object creation -> `createX()`
-- Loop bodies -> `processItem()`
-- Error handling -> `handleXError()`
-
-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.