@webpieces/ai-hook-rules 0.0.1 → 0.2.113

package/src/core/rules/{file-location.ts → file-location.js}

@@ -1,34 +1,32 @@
-import * as fs from 'fs';
-import * as path from 'path';
-
-import type { FileRule, FileContext, Violation } from '../types';
-import { Violation as V } from '../types';
-
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const tslib_1 = require("tslib");
+const fs = tslib_1.__importStar(require("fs"));
+const path = tslib_1.__importStar(require("path"));
+const types_1 = require("../types");
 const DEFAULT_EXCLUDE_PATHS = [
     'node_modules', 'dist', '.nx', '.git',
     'architecture', 'tmp', 'scripts',
 ];
 const DEFAULT_ALLOWED_ROOT_FILES = ['jest.setup.ts'];
-
-function isNodeModulesDir(name: string): boolean {
+function isNodeModulesDir(name) {
     return name === 'node_modules' || name.startsWith('node_modules_');
 }
-
-function shouldSkipDir(name: string, excludePaths: readonly string[]): boolean {
-    if (isNodeModulesDir(name)) return true;
+function shouldSkipDir(name, excludePaths) {
+    if (isNodeModulesDir(name))
+        return true;
     return excludePaths.indexOf(name) >= 0;
 }
-
-function findProjectRoot(filePath: string, workspaceRoot: string): string | null {
+function findProjectRoot(filePath, workspaceRoot) {
     let dir = path.dirname(filePath);
     while (dir !== workspaceRoot && dir.startsWith(workspaceRoot)) {
-        if (fs.existsSync(path.join(dir, 'project.json'))) return dir;
+        if (fs.existsSync(path.join(dir, 'project.json')))
+            return dir;
         dir = path.dirname(dir);
     }
     return null;
 }
-
-const fileLocationRule: FileRule = {
+const fileLocationRule = {
     name: 'file-location',
     description: 'Every .ts file must belong to a project\'s src/ directory.',
     scope: 'file',
@@ -41,47 +39,35 @@ const fileLocationRule: FileRule = {
         'Move the file into an existing project\'s src/ directory, or create a new project with project.json that owns the directory.',
         'Add the dir to file-location.excludePaths in webpieces.ai-hooks.json',
     ],
-
-    check(ctx: FileContext): readonly Violation[] {
-        if (ctx.tool !== 'Write') return [];
-
+    check(ctx) {
+        if (ctx.tool !== 'Write')
+            return [];
         const excludePaths = Array.isArray(ctx.options['excludePaths'])
-            ? ctx.options['excludePaths'] as string[]
+            ? ctx.options['excludePaths']
             : DEFAULT_EXCLUDE_PATHS;
         const allowedRootFiles = Array.isArray(ctx.options['allowedRootFiles'])
-            ? ctx.options['allowedRootFiles'] as string[]
+            ? ctx.options['allowedRootFiles']
             : DEFAULT_ALLOWED_ROOT_FILES;
-
         const relParts = ctx.relativePath.split(path.sep);
         const topDir = relParts[0];
-
-        if (topDir && shouldSkipDir(topDir, excludePaths)) return [];
-        if (relParts.length === 1 && allowedRootFiles.indexOf(relParts[0]) >= 0) return [];
-
+        if (topDir && shouldSkipDir(topDir, excludePaths))
+            return [];
+        if (relParts.length === 1 && allowedRootFiles.indexOf(relParts[0]) >= 0)
+            return [];
         const projectRoot = findProjectRoot(ctx.filePath, ctx.workspaceRoot);
-
         if (!projectRoot) {
-            return [new V(
-                1,
-                ctx.relativePath,
-                'File is not inside any Nx project. Move it into a project\'s src/ directory.',
-            )];
+            return [new types_1.Violation(1, ctx.relativePath, 'File is not inside any Nx project. Move it into a project\'s src/ directory.')];
         }
-
         const relToProject = path.relative(projectRoot, ctx.filePath);
         const fileName = path.basename(ctx.filePath);
-        if (fileName === 'jest.config.ts') return [];
+        if (fileName === 'jest.config.ts')
+            return [];
         if (!relToProject.startsWith('src' + path.sep) && relToProject !== 'src') {
             const projectName = path.relative(ctx.workspaceRoot, projectRoot);
-            return [new V(
-                1,
-                ctx.relativePath,
-                `File is inside project \`${projectName}\` but outside its src/ directory. Move it into src/.`,
-            )];
+            return [new types_1.Violation(1, ctx.relativePath, `File is inside project \`${projectName}\` but outside its src/ directory. Move it into src/.`)];
         }
-
         return [];
     },
 };
-
-export default fileLocationRule;
+exports.default = fileLocationRule;
+//# sourceMappingURL=file-location.js.map

package/src/core/rules/file-location.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"file-location.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/file-location.ts"],"names":[],"mappings":";;;AAAA,+CAAyB;AACzB,mDAA6B;AAG7B,oCAA0C;AAE1C,MAAM,qBAAqB,GAAG;IAC1B,cAAc,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM;IACrC,cAAc,EAAE,KAAK,EAAE,SAAS;CACnC,CAAC;AACF,MAAM,0BAA0B,GAAG,CAAC,eAAe,CAAC,CAAC;AAErD,SAAS,gBAAgB,CAAC,IAAY;IAClC,OAAO,IAAI,KAAK,cAAc,IAAI,IAAI,CAAC,UAAU,CAAC,eAAe,CAAC,CAAC;AACvE,CAAC;AAED,SAAS,aAAa,CAAC,IAAY,EAAE,YAA+B;IAChE,IAAI,gBAAgB,CAAC,IAAI,CAAC;QAAE,OAAO,IAAI,CAAC;IACxC,OAAO,YAAY,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC3C,CAAC;AAED,SAAS,eAAe,CAAC,QAAgB,EAAE,aAAqB;IAC5D,IAAI,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;IACjC,OAAO,GAAG,KAAK,aAAa,IAAI,GAAG,CAAC,UAAU,CAAC,aAAa,CAAC,EAAE,CAAC;QAC5D,IAAI,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,cAAc,CAAC,CAAC;YAAE,OAAO,GAAG,CAAC;QAC9D,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;IAC5B,CAAC;IACD,OAAO,IAAI,CAAC;AAChB,CAAC;AAED,MAAM,gBAAgB,GAAa;IAC/B,IAAI,EAAE,eAAe;IACrB,WAAW,EAAE,4DAA4D;IACzE,KAAK,EAAE,MAAM;IACb,KAAK,EAAE,CAAC,SAAS,EAAE,UAAU,CAAC;IAC9B,cAAc,EAAE;QACZ,YAAY,EAAE,qBAAqB;QACnC,gBAAgB,EAAE,0BAA0B;KAC/C;IACD,OAAO,EAAE;QACL,8HAA8H;QAC9H,sEAAsE;KACzE;IAED,KAAK,CAAC,GAAgB;QAClB,IAAI,GAAG,CAAC,IAAI,KAAK,OAAO;YAAE,OAAO,EAAE,CAAC;QAEpC,MAAM,YAAY,GAAG,KAAK,CAAC,OAAO,CAAC,GAAG,CAAC,OAAO,CAAC,cAAc,CAAC,CAAC;YAC3D,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,cAAc,CAAa;YACzC,CAAC,CAAC,qBAAqB,CAAC;QAC5B,MAAM,gBAAgB,GAAG,KAAK,CAAC,OAAO,CAAC,GAAG,CAAC,OAAO,CAAC,kBAAkB,CAAC,CAAC;YACnE,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,kBAAkB,CAAa;YAC7C,CAAC,CAAC,0BAA0B,CAAC;QAEjC,MAAM,QAAQ,GAAG,GAAG,CAAC,YAAY,CAAC,KAAK,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAClD,MAAM,MAAM,GAAG,QAAQ,CAAC,CAAC,CAAC,CAAC;QAE3B,IAAI,MAAM,IAAI,aAAa,CAAC,MAAM,EAAE,YAAY,CAAC;YAAE,OAAO,EAAE,CAAC;QAC7D,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC,IAAI,gBAAgB,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;YAAE,OAAO,EAAE,CAAC;QAEnF,MAAM,WAAW,GAAG,eAAe,CAAC,GAAG,CAAC,QAAQ,EAAE,GAAG,CAAC,aAAa,CAAC,CAAC;QAErE,IAAI,CAAC,WAAW,EAAE,CAAC;YACf,OAAO,CAAC,IAAI,iBAAC,CACT,CAAC,EACD,GAAG,CAAC,YAAY,EAChB,8EAA8E,CACjF,CAAC,CAAC;QACP,CAAC;QAED,MAAM,YAAY,GAAG,IAAI,CAAC,QAAQ,CAAC,WAAW,EAAE,GAAG,CAAC,QAAQ,CAAC,CAAC;QAC9D,MAAM,QAAQ,GAAG,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC;QAC7C,IAAI,QAAQ,KAAK,gBAAgB;YAAE,OAAO,EAAE,CAAC;QAC7C,IAAI,CAAC,YAAY,CAAC,UAAU,CAAC,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,IAAI,YAAY,KAAK,KAAK,EAAE,CAAC;YACvE,MAAM,WAAW,GAAG,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,aAAa,EAAE,WAAW,CAAC,CAAC;YAClE,OAAO,CAAC,IAAI,iBAAC,CACT,CAAC,EACD,GAAG,CAAC,YAAY,EAChB,4BAA4B,WAAW,uDAAuD,CACjG,CAAC,CAAC;QACP,CAAC;QAED,OAAO,EAAE,CAAC;IACd,CAAC;CACJ,CAAC;AAEF,kBAAe,gBAAgB,CAAC","sourcesContent":["import * as fs from 'fs';\nimport * as path from 'path';\n\nimport type { FileRule, FileContext, Violation } from '../types';\nimport { Violation as V } from '../types';\n\nconst DEFAULT_EXCLUDE_PATHS = [\n    'node_modules', 'dist', '.nx', '.git',\n    'architecture', 'tmp', 'scripts',\n];\nconst DEFAULT_ALLOWED_ROOT_FILES = ['jest.setup.ts'];\n\nfunction isNodeModulesDir(name: string): boolean {\n    return name === 'node_modules' || name.startsWith('node_modules_');\n}\n\nfunction shouldSkipDir(name: string, excludePaths: readonly string[]): boolean {\n    if (isNodeModulesDir(name)) return true;\n    return excludePaths.indexOf(name) >= 0;\n}\n\nfunction findProjectRoot(filePath: string, workspaceRoot: string): string | null {\n    let dir = path.dirname(filePath);\n    while (dir !== workspaceRoot && dir.startsWith(workspaceRoot)) {\n        if (fs.existsSync(path.join(dir, 'project.json'))) return dir;\n        dir = path.dirname(dir);\n    }\n    return null;\n}\n\nconst fileLocationRule: FileRule = {\n    name: 'file-location',\n    description: 'Every .ts file must belong to a project\\'s src/ directory.',\n    scope: 'file',\n    files: ['**/*.ts', '**/*.tsx'],\n    defaultOptions: {\n        excludePaths: DEFAULT_EXCLUDE_PATHS,\n        allowedRootFiles: DEFAULT_ALLOWED_ROOT_FILES,\n    },\n    fixHint: [\n        'Move the file into an existing project\\'s src/ directory, or create a new project with project.json that owns the directory.',\n        'Add the dir to file-location.excludePaths in webpieces.ai-hooks.json',\n    ],\n\n    check(ctx: FileContext): readonly Violation[] {\n        if (ctx.tool !== 'Write') return [];\n\n        const excludePaths = Array.isArray(ctx.options['excludePaths'])\n            ? ctx.options['excludePaths'] as string[]\n            : DEFAULT_EXCLUDE_PATHS;\n        const allowedRootFiles = Array.isArray(ctx.options['allowedRootFiles'])\n            ? ctx.options['allowedRootFiles'] as string[]\n            : DEFAULT_ALLOWED_ROOT_FILES;\n\n        const relParts = ctx.relativePath.split(path.sep);\n        const topDir = relParts[0];\n\n        if (topDir && shouldSkipDir(topDir, excludePaths)) return [];\n        if (relParts.length === 1 && allowedRootFiles.indexOf(relParts[0]) >= 0) return [];\n\n        const projectRoot = findProjectRoot(ctx.filePath, ctx.workspaceRoot);\n\n        if (!projectRoot) {\n            return [new V(\n                1,\n                ctx.relativePath,\n                'File is not inside any Nx project. Move it into a project\\'s src/ directory.',\n            )];\n        }\n\n        const relToProject = path.relative(projectRoot, ctx.filePath);\n        const fileName = path.basename(ctx.filePath);\n        if (fileName === 'jest.config.ts') return [];\n        if (!relToProject.startsWith('src' + path.sep) && relToProject !== 'src') {\n            const projectName = path.relative(ctx.workspaceRoot, projectRoot);\n            return [new V(\n                1,\n                ctx.relativePath,\n                `File is inside project \\`${projectName}\\` but outside its src/ directory. Move it into src/.`,\n            )];\n        }\n\n        return [];\n    },\n};\n\nexport default fileLocationRule;\n"]}

package/src/core/rules/index.d.ts

@@ -0,0 +1 @@
+export declare const builtInRuleNames: readonly string[];

package/src/core/rules/{index.ts → index.js}

@@ -1,4 +1,7 @@
-export const builtInRuleNames: readonly string[] = [
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.builtInRuleNames = void 0;
+exports.builtInRuleNames = [
     'no-any-unknown',
     'no-implicit-any',
     'max-file-lines',
@@ -9,3 +12,4 @@ export const builtInRuleNames: readonly string[] = [
     'catch-error-pattern',
     'no-shell-substitution',
 ];
+//# sourceMappingURL=index.js.map

package/src/core/rules/index.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/index.ts"],"names":[],"mappings":";;;AAAa,QAAA,gBAAgB,GAAsB;IAC/C,gBAAgB;IAChB,iBAAiB;IACjB,gBAAgB;IAChB,eAAe;IACf,gBAAgB;IAChB,qBAAqB;IACrB,yBAAyB;IACzB,qBAAqB;IACrB,uBAAuB;CAC1B,CAAC","sourcesContent":["export const builtInRuleNames: readonly string[] = [\n    'no-any-unknown',\n    'no-implicit-any',\n    'max-file-lines',\n    'file-location',\n    'no-destructure',\n    'require-return-type',\n    'no-unmanaged-exceptions',\n    'catch-error-pattern',\n    'no-shell-substitution',\n];\n"]}

package/src/core/rules/max-file-lines.d.ts

@@ -0,0 +1,3 @@
+import type { FileRule } from '../types';
+declare const maxFileLinesRule: FileRule;
+export default maxFileLinesRule;

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

@@ -1,14 +1,13 @@
-import * as fs from 'fs';
-import * as path from 'path';
-
-import type { FileRule, FileContext, Violation } from '../types';
-import { Violation as V } from '../types';
-
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const tslib_1 = require("tslib");
+const fs = tslib_1.__importStar(require("fs"));
+const path = tslib_1.__importStar(require("path"));
+const types_1 = require("../types");
 const DEFAULT_LIMIT = 900;
 const INSTRUCT_DIR = '.webpieces/instruct-ai';
 const INSTRUCT_FILE = 'webpieces.filesize.md';
-
-const maxFileLinesRule: FileRule = {
+const maxFileLinesRule = {
     name: 'max-file-lines',
     description: 'Cap file length at a configured line limit.',
     scope: 'file',
@@ -18,29 +17,24 @@ const maxFileLinesRule: FileRule = {
         'READ .webpieces/instruct-ai/webpieces.filesize.md for step-by-step refactoring guidance.',
         '// eslint-disable-next-line @webpieces/max-file-lines  (also suppresses the eslint rule)',
     ],
-
-    check(ctx: FileContext): readonly Violation[] {
+    check(ctx) {
         const limit = typeof ctx.options['limit'] === 'number'
-            ? ctx.options['limit'] as number
+            ? ctx.options['limit']
             : DEFAULT_LIMIT;
-        if (ctx.projectedFileLines <= limit) return [];
+        if (ctx.projectedFileLines <= limit)
+            return [];
         writeInstructionFile(ctx.workspaceRoot);
-        return [new V(
-            1,
-            `(projected ${String(ctx.projectedFileLines)} lines)`,
-            `File will be ${String(ctx.projectedFileLines)} lines, exceeding the ${String(limit)}-line limit. See .webpieces/instruct-ai/webpieces.filesize.md for detailed refactoring instructions.`,
-        )];
+        return [new types_1.Violation(1, `(projected ${String(ctx.projectedFileLines)} lines)`, `File will be ${String(ctx.projectedFileLines)} lines, exceeding the ${String(limit)}-line limit. See .webpieces/instruct-ai/webpieces.filesize.md for detailed refactoring instructions.`)];
     },
 };
-
-function writeInstructionFile(workspaceRoot: string): void {
+function writeInstructionFile(workspaceRoot) {
     const dir = path.join(workspaceRoot, INSTRUCT_DIR);
     const filePath = path.join(dir, INSTRUCT_FILE);
-    if (fs.existsSync(filePath)) return;
+    if (fs.existsSync(filePath))
+        return;
     fs.mkdirSync(dir, { recursive: true });
     fs.writeFileSync(filePath, FILESIZE_DOC_CONTENT);
 }
-
 // eslint-disable-next-line @webpieces/max-file-lines
 const FILESIZE_DOC_CONTENT = `# AI Agent Instructions: File Too Long
 
@@ -133,5 +127,5 @@ If refactoring is genuinely not feasible, add a disable comment:
 
 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.
 `;
-
-export default maxFileLinesRule;
+exports.default = maxFileLinesRule;
+//# sourceMappingURL=max-file-lines.js.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"max-file-lines.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/max-file-lines.ts"],"names":[],"mappings":";;;AAAA,+CAAyB;AACzB,mDAA6B;AAG7B,oCAA0C;AAE1C,MAAM,aAAa,GAAG,GAAG,CAAC;AAC1B,MAAM,YAAY,GAAG,wBAAwB,CAAC;AAC9C,MAAM,aAAa,GAAG,uBAAuB,CAAC;AAE9C,MAAM,gBAAgB,GAAa;IAC/B,IAAI,EAAE,gBAAgB;IACtB,WAAW,EAAE,6CAA6C;IAC1D,KAAK,EAAE,MAAM;IACb,KAAK,EAAE,CAAC,SAAS,EAAE,UAAU,CAAC;IAC9B,cAAc,EAAE,EAAE,KAAK,EAAE,aAAa,EAAE;IACxC,OAAO,EAAE;QACL,0FAA0F;QAC1F,0FAA0F;KAC7F;IAED,KAAK,CAAC,GAAgB;QAClB,MAAM,KAAK,GAAG,OAAO,GAAG,CAAC,OAAO,CAAC,OAAO,CAAC,KAAK,QAAQ;YAClD,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,OAAO,CAAW;YAChC,CAAC,CAAC,aAAa,CAAC;QACpB,IAAI,GAAG,CAAC,kBAAkB,IAAI,KAAK;YAAE,OAAO,EAAE,CAAC;QAC/C,oBAAoB,CAAC,GAAG,CAAC,aAAa,CAAC,CAAC;QACxC,OAAO,CAAC,IAAI,iBAAC,CACT,CAAC,EACD,cAAc,MAAM,CAAC,GAAG,CAAC,kBAAkB,CAAC,SAAS,EACrD,gBAAgB,MAAM,CAAC,GAAG,CAAC,kBAAkB,CAAC,yBAAyB,MAAM,CAAC,KAAK,CAAC,sGAAsG,CAC7L,CAAC,CAAC;IACP,CAAC;CACJ,CAAC;AAEF,SAAS,oBAAoB,CAAC,aAAqB;IAC/C,MAAM,GAAG,GAAG,IAAI,CAAC,IAAI,CAAC,aAAa,EAAE,YAAY,CAAC,CAAC;IACnD,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,aAAa,CAAC,CAAC;IAC/C,IAAI,EAAE,CAAC,UAAU,CAAC,QAAQ,CAAC;QAAE,OAAO;IACpC,EAAE,CAAC,SAAS,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IACvC,EAAE,CAAC,aAAa,CAAC,QAAQ,EAAE,oBAAoB,CAAC,CAAC;AACrD,CAAC;AAED,qDAAqD;AACrD,MAAM,oBAAoB,GAAG;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CA0F5B,CAAC;AAEF,kBAAe,gBAAgB,CAAC","sourcesContent":["import * as fs from 'fs';\nimport * as path from 'path';\n\nimport type { FileRule, FileContext, Violation } from '../types';\nimport { Violation as V } from '../types';\n\nconst DEFAULT_LIMIT = 900;\nconst INSTRUCT_DIR = '.webpieces/instruct-ai';\nconst INSTRUCT_FILE = 'webpieces.filesize.md';\n\nconst maxFileLinesRule: FileRule = {\n    name: 'max-file-lines',\n    description: 'Cap file length at a configured line limit.',\n    scope: 'file',\n    files: ['**/*.ts', '**/*.tsx'],\n    defaultOptions: { limit: DEFAULT_LIMIT },\n    fixHint: [\n        'READ .webpieces/instruct-ai/webpieces.filesize.md for step-by-step refactoring guidance.',\n        '// eslint-disable-next-line @webpieces/max-file-lines  (also suppresses the eslint rule)',\n    ],\n\n    check(ctx: FileContext): readonly Violation[] {\n        const limit = typeof ctx.options['limit'] === 'number'\n            ? ctx.options['limit'] as number\n            : DEFAULT_LIMIT;\n        if (ctx.projectedFileLines <= limit) return [];\n        writeInstructionFile(ctx.workspaceRoot);\n        return [new V(\n            1,\n            `(projected ${String(ctx.projectedFileLines)} lines)`,\n            `File will be ${String(ctx.projectedFileLines)} lines, exceeding the ${String(limit)}-line limit. See .webpieces/instruct-ai/webpieces.filesize.md for detailed refactoring instructions.`,\n        )];\n    },\n};\n\nfunction writeInstructionFile(workspaceRoot: string): void {\n    const dir = path.join(workspaceRoot, INSTRUCT_DIR);\n    const filePath = path.join(dir, INSTRUCT_FILE);\n    if (fs.existsSync(filePath)) return;\n    fs.mkdirSync(dir, { recursive: true });\n    fs.writeFileSync(filePath, FILESIZE_DOC_CONTENT);\n}\n\n// eslint-disable-next-line @webpieces/max-file-lines\nconst FILESIZE_DOC_CONTENT = `# AI Agent Instructions: File Too Long\n\n**READ THIS FILE to fix files that are too long**\n\n## Core Principle\n\nWith **stateless systems + dependency injection, refactor is trivial**.\nPick a method or a few and move to new class XXXXX, then inject XXXXX\ninto all users of those methods via the constructor.\nDelete those methods from original class.\n\n**99% of files can be less than the configured max lines of code.**\n\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### 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}\n\n// GOOD: Extract validation service\n// 1. Create UserValidationService.ts\n@provideSingleton()\nexport class UserValidationService {\n  validateUserData(data: UserData): ValidationResult { /* ... */ }\n  validateEmail(email: 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\\`\\`\\`\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   - Add \\`@provideSingleton()\\` decorator\n   - Move child methods to new class\n\n4. **UPDATE** dependency injection:\n   - Inject new service into original class constructor\n   - Replace direct method calls with \\`this.serviceName.method()\\`\n\n5. **VERIFY** file sizes:\n   - Original file should now be under the limit\n   - Each extracted file should be under the limit\n\n## Escape Hatch\n\nIf refactoring is genuinely not feasible, add a disable comment:\n\n\\`\\`\\`typescript\n// eslint-disable-next-line @webpieces/max-file-lines\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\nexport default maxFileLinesRule;\n"]}

package/src/core/rules/no-any-unknown.d.ts

@@ -0,0 +1,3 @@
+import type { EditRule } from '../types';
+declare const noAnyRule: EditRule;
+export default noAnyRule;

package/src/core/rules/no-any-unknown.js

@@ -0,0 +1,30 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const types_1 = require("../types");
+const ANY_PATTERN = /(?::\s*any\b|\bas\s+any\b|<any>|any\[\]|Array<any>|Promise<any>|Map<[^,<>]+,\s*any\s*>|Record<[^,<>]+,\s*any\s*>|Set<any>)/;
+const noAnyRule = {
+    name: 'no-any-unknown',
+    description: 'Disallow the `any` keyword. Use concrete types or interfaces.',
+    scope: 'edit',
+    files: ['**/*.ts', '**/*.tsx'],
+    defaultOptions: {},
+    fixHint: [
+        'Prefer: interface MyData { ... }   or   class MyData { ... }',
+        '// webpieces-disable no-any-unknown -- <one-line reason>',
+    ],
+    check(ctx) {
+        const violations = [];
+        for (let i = 0; i < ctx.strippedLines.length; i += 1) {
+            const stripped = ctx.strippedLines[i];
+            if (!ANY_PATTERN.test(stripped))
+                continue;
+            const lineNum = i + 1;
+            if (ctx.isLineDisabled(lineNum, 'no-any-unknown'))
+                continue;
+            violations.push(new types_1.Violation(lineNum, ctx.lines[i].trim(), '`any` erases type information. Use a concrete type, an interface, or `unknown` with type guards.'));
+        }
+        return violations;
+    },
+};
+exports.default = noAnyRule;
+//# sourceMappingURL=no-any-unknown.js.map

package/src/core/rules/no-any-unknown.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-any-unknown.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/no-any-unknown.ts"],"names":[],"mappings":";;AACA,oCAA0C;AAE1C,MAAM,WAAW,GACb,4HAA4H,CAAC;AAEjI,MAAM,SAAS,GAAa;IACxB,IAAI,EAAE,gBAAgB;IACtB,WAAW,EAAE,+DAA+D;IAC5E,KAAK,EAAE,MAAM;IACb,KAAK,EAAE,CAAC,SAAS,EAAE,UAAU,CAAC;IAC9B,cAAc,EAAE,EAAE;IAClB,OAAO,EAAE;QACL,8DAA8D;QAC9D,0DAA0D;KAC7D;IAED,KAAK,CAAC,GAAgB;QAClB,MAAM,UAAU,GAAQ,EAAE,CAAC;QAC3B,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,GAAG,CAAC,aAAa,CAAC,MAAM,EAAE,CAAC,IAAI,CAAC,EAAE,CAAC;YACnD,MAAM,QAAQ,GAAG,GAAG,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC;YACtC,IAAI,CAAC,WAAW,CAAC,IAAI,CAAC,QAAQ,CAAC;gBAAE,SAAS;YAC1C,MAAM,OAAO,GAAG,CAAC,GAAG,CAAC,CAAC;YACtB,IAAI,GAAG,CAAC,cAAc,CAAC,OAAO,EAAE,gBAAgB,CAAC;gBAAE,SAAS;YAC5D,UAAU,CAAC,IAAI,CAAC,IAAI,iBAAC,CACjB,OAAO,EACP,GAAG,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,EACnB,kGAAkG,CACrG,CAAC,CAAC;QACP,CAAC;QACD,OAAO,UAAU,CAAC;IACtB,CAAC;CACJ,CAAC;AAEF,kBAAe,SAAS,CAAC","sourcesContent":["import type { EditRule, EditContext, Violation } from '../types';\nimport { Violation as V } from '../types';\n\nconst ANY_PATTERN =\n    /(?::\\s*any\\b|\\bas\\s+any\\b|<any>|any\\[\\]|Array<any>|Promise<any>|Map<[^,<>]+,\\s*any\\s*>|Record<[^,<>]+,\\s*any\\s*>|Set<any>)/;\n\nconst noAnyRule: EditRule = {\n    name: 'no-any-unknown',\n    description: 'Disallow the `any` keyword. Use concrete types or interfaces.',\n    scope: 'edit',\n    files: ['**/*.ts', '**/*.tsx'],\n    defaultOptions: {},\n    fixHint: [\n        'Prefer: interface MyData { ... }   or   class MyData { ... }',\n        '// webpieces-disable no-any-unknown -- <one-line reason>',\n    ],\n\n    check(ctx: EditContext): readonly Violation[] {\n        const violations: V[] = [];\n        for (let i = 0; i < ctx.strippedLines.length; i += 1) {\n            const stripped = ctx.strippedLines[i];\n            if (!ANY_PATTERN.test(stripped)) continue;\n            const lineNum = i + 1;\n            if (ctx.isLineDisabled(lineNum, 'no-any-unknown')) continue;\n            violations.push(new V(\n                lineNum,\n                ctx.lines[i].trim(),\n                '`any` erases type information. Use a concrete type, an interface, or `unknown` with type guards.',\n            ));\n        }\n        return violations;\n    },\n};\n\nexport default noAnyRule;\n"]}

package/src/core/rules/no-destructure.d.ts

@@ -0,0 +1,3 @@
+import type { EditRule } from '../types';
+declare const noDestructureRule: EditRule;
+export default noDestructureRule;

package/src/core/rules/{no-destructure.ts → no-destructure.js}

@@ -1,9 +1,8 @@
-import type { EditRule, EditContext, Violation } from '../types';
-import { Violation as V } from '../types';
-
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const types_1 = require("../types");
 const VARIABLE_DESTRUCTURE = /\b(?:const|let|var)\s*\{/;
-
-const noDestructureRule: EditRule = {
+const noDestructureRule = {
     name: 'no-destructure',
     description: 'Disallow destructuring patterns. Assign the whole result and pass it around or access properties explicitly.',
     scope: 'edit',
@@ -13,22 +12,19 @@ const noDestructureRule: EditRule = {
         'Instead of: const { x, y } = methodCall(); prefer const obj = methodCall(); then pass obj to other methods or use obj.x',
         '// webpieces-disable no-destructure -- <reason>',
     ],
-
-    check(ctx: EditContext): readonly Violation[] {
-        const violations: V[] = [];
+    check(ctx) {
+        const violations = [];
         for (let i = 0; i < ctx.strippedLines.length; i += 1) {
             const stripped = ctx.strippedLines[i];
-            if (!VARIABLE_DESTRUCTURE.test(stripped)) continue;
+            if (!VARIABLE_DESTRUCTURE.test(stripped))
+                continue;
             const lineNum = i + 1;
-            if (ctx.isLineDisabled(lineNum, 'no-destructure')) continue;
-            violations.push(new V(
-                lineNum,
-                ctx.lines[i].trim(),
-                'Destructuring pattern. Assign the whole result instead: const obj = methodCall(); then pass obj around or use obj.x',
-            ));
+            if (ctx.isLineDisabled(lineNum, 'no-destructure'))
+                continue;
+            violations.push(new types_1.Violation(lineNum, ctx.lines[i].trim(), 'Destructuring pattern. Assign the whole result instead: const obj = methodCall(); then pass obj around or use obj.x'));
         }
         return violations;
     },
 };
-
-export default noDestructureRule;
+exports.default = noDestructureRule;
+//# sourceMappingURL=no-destructure.js.map

package/src/core/rules/no-destructure.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-destructure.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/no-destructure.ts"],"names":[],"mappings":";;AACA,oCAA0C;AAE1C,MAAM,oBAAoB,GAAG,0BAA0B,CAAC;AAExD,MAAM,iBAAiB,GAAa;IAChC,IAAI,EAAE,gBAAgB;IACtB,WAAW,EAAE,8GAA8G;IAC3H,KAAK,EAAE,MAAM;IACb,KAAK,EAAE,CAAC,SAAS,EAAE,UAAU,CAAC;IAC9B,cAAc,EAAE,EAAE,aAAa,EAAE,IAAI,EAAE;IACvC,OAAO,EAAE;QACL,yHAAyH;QACzH,iDAAiD;KACpD;IAED,KAAK,CAAC,GAAgB;QAClB,MAAM,UAAU,GAAQ,EAAE,CAAC;QAC3B,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,GAAG,CAAC,aAAa,CAAC,MAAM,EAAE,CAAC,IAAI,CAAC,EAAE,CAAC;YACnD,MAAM,QAAQ,GAAG,GAAG,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC;YACtC,IAAI,CAAC,oBAAoB,CAAC,IAAI,CAAC,QAAQ,CAAC;gBAAE,SAAS;YACnD,MAAM,OAAO,GAAG,CAAC,GAAG,CAAC,CAAC;YACtB,IAAI,GAAG,CAAC,cAAc,CAAC,OAAO,EAAE,gBAAgB,CAAC;gBAAE,SAAS;YAC5D,UAAU,CAAC,IAAI,CAAC,IAAI,iBAAC,CACjB,OAAO,EACP,GAAG,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,EACnB,qHAAqH,CACxH,CAAC,CAAC;QACP,CAAC;QACD,OAAO,UAAU,CAAC;IACtB,CAAC;CACJ,CAAC;AAEF,kBAAe,iBAAiB,CAAC","sourcesContent":["import type { EditRule, EditContext, Violation } from '../types';\nimport { Violation as V } from '../types';\n\nconst VARIABLE_DESTRUCTURE = /\\b(?:const|let|var)\\s*\\{/;\n\nconst noDestructureRule: EditRule = {\n    name: 'no-destructure',\n    description: 'Disallow destructuring patterns. Assign the whole result and pass it around or access properties explicitly.',\n    scope: 'edit',\n    files: ['**/*.ts', '**/*.tsx'],\n    defaultOptions: { allowTopLevel: true },\n    fixHint: [\n        'Instead of: const { x, y } = methodCall(); prefer const obj = methodCall(); then pass obj to other methods or use obj.x',\n        '// webpieces-disable no-destructure -- <reason>',\n    ],\n\n    check(ctx: EditContext): readonly Violation[] {\n        const violations: V[] = [];\n        for (let i = 0; i < ctx.strippedLines.length; i += 1) {\n            const stripped = ctx.strippedLines[i];\n            if (!VARIABLE_DESTRUCTURE.test(stripped)) continue;\n            const lineNum = i + 1;\n            if (ctx.isLineDisabled(lineNum, 'no-destructure')) continue;\n            violations.push(new V(\n                lineNum,\n                ctx.lines[i].trim(),\n                'Destructuring pattern. Assign the whole result instead: const obj = methodCall(); then pass obj around or use obj.x',\n            ));\n        }\n        return violations;\n    },\n};\n\nexport default noDestructureRule;\n"]}

package/src/core/rules/no-implicit-any.d.ts

@@ -0,0 +1,3 @@
+import type { EditRule } from '../types';
+declare const noImplicitAnyRule: EditRule;
+export default noImplicitAnyRule;

package/src/core/rules/{no-implicit-any.ts → no-implicit-any.js}

@@ -1,41 +1,46 @@
-import type { EditRule, EditContext, Violation } from '../types';
-import { Violation as V } from '../types';
-
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const types_1 = require("../types");
 const ARROW_PARAMS_RE = /\(([^()]*)\)\s*=>/g;
 const FN_DECL_PARAMS_RE = /\bfunction\s*[\w$]*\s*\(([^()]*)\)/g;
-
-function firstUntypedParam(paramsStr: string): string | null {
-    if (paramsStr.includes('{') || paramsStr.includes('[')) return null;
+function firstUntypedParam(paramsStr) {
+    if (paramsStr.includes('{') || paramsStr.includes('['))
+        return null;
     const parts = paramsStr.split(',').map((p) => p.trim()).filter((p) => p.length > 0);
     for (const part of parts) {
-        if (part.startsWith('...')) continue;
-        if (part.includes(':')) continue;
-        if (part.includes('=')) continue;
-        if (part === 'this') continue;
-        if (/^[a-zA-Z_$][\w$]*$/.test(part)) return part;
+        if (part.startsWith('...'))
+            continue;
+        if (part.includes(':'))
+            continue;
+        if (part.includes('='))
+            continue;
+        if (part === 'this')
+            continue;
+        if (/^[a-zA-Z_$][\w$]*$/.test(part))
+            return part;
     }
     return null;
 }
-
-function findOffender(line: string): string | null {
+function findOffender(line) {
     ARROW_PARAMS_RE.lastIndex = 0;
-    let m: RegExpExecArray | null = ARROW_PARAMS_RE.exec(line);
+    let m = ARROW_PARAMS_RE.exec(line);
     while (m !== null) {
         const bad = firstUntypedParam(m[1]);
-        if (bad) return bad;
+        if (bad)
+            return bad;
         m = ARROW_PARAMS_RE.exec(line);
     }
     FN_DECL_PARAMS_RE.lastIndex = 0;
     m = FN_DECL_PARAMS_RE.exec(line);
     while (m !== null) {
         const bad = firstUntypedParam(m[1]);
-        if (bad) return bad;
+        if (bad)
+            return bad;
         m = FN_DECL_PARAMS_RE.exec(line);
     }
     return null;
 }
-
-const noImplicitAnyRule: EditRule = {
+const noImplicitAnyRule = {
     name: 'no-implicit-any',
     description: 'Disallow function parameters without explicit type annotations (implicit-any).',
     scope: 'edit',
@@ -45,23 +50,20 @@ const noImplicitAnyRule: EditRule = {
         'Add explicit types: (x: string) => ...   or   function foo(x: number)',
         '// webpieces-disable no-implicit-any -- <one-line reason>',
     ],
-
-    check(ctx: EditContext): readonly Violation[] {
-        const violations: V[] = [];
+    check(ctx) {
+        const violations = [];
         for (let i = 0; i < ctx.strippedLines.length; i += 1) {
             const stripped = ctx.strippedLines[i];
             const lineNum = i + 1;
-            if (ctx.isLineDisabled(lineNum, 'no-implicit-any')) continue;
+            if (ctx.isLineDisabled(lineNum, 'no-implicit-any'))
+                continue;
             const offender = findOffender(stripped);
-            if (!offender) continue;
-            violations.push(new V(
-                lineNum,
-                ctx.lines[i].trim(),
-                `Parameter "${offender}" has no type annotation. Add an explicit type to avoid implicit-any.`,
-            ));
+            if (!offender)
+                continue;
+            violations.push(new types_1.Violation(lineNum, ctx.lines[i].trim(), `Parameter "${offender}" has no type annotation. Add an explicit type to avoid implicit-any.`));
         }
         return violations;
     },
 };
-
-export default noImplicitAnyRule;
+exports.default = noImplicitAnyRule;
+//# sourceMappingURL=no-implicit-any.js.map

package/src/core/rules/no-implicit-any.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-implicit-any.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/no-implicit-any.ts"],"names":[],"mappings":";;AACA,oCAA0C;AAE1C,MAAM,eAAe,GAAG,oBAAoB,CAAC;AAC7C,MAAM,iBAAiB,GAAG,qCAAqC,CAAC;AAEhE,SAAS,iBAAiB,CAAC,SAAiB;IACxC,IAAI,SAAS,CAAC,QAAQ,CAAC,GAAG,CAAC,IAAI,SAAS,CAAC,QAAQ,CAAC,GAAG,CAAC;QAAE,OAAO,IAAI,CAAC;IACpE,MAAM,KAAK,GAAG,SAAS,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;IACpF,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACvB,IAAI,IAAI,CAAC,UAAU,CAAC,KAAK,CAAC;YAAE,SAAS;QACrC,IAAI,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC;YAAE,SAAS;QACjC,IAAI,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC;YAAE,SAAS;QACjC,IAAI,IAAI,KAAK,MAAM;YAAE,SAAS;QAC9B,IAAI,oBAAoB,CAAC,IAAI,CAAC,IAAI,CAAC;YAAE,OAAO,IAAI,CAAC;IACrD,CAAC;IACD,OAAO,IAAI,CAAC;AAChB,CAAC;AAED,SAAS,YAAY,CAAC,IAAY;IAC9B,eAAe,CAAC,SAAS,GAAG,CAAC,CAAC;IAC9B,IAAI,CAAC,GAA2B,eAAe,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC3D,OAAO,CAAC,KAAK,IAAI,EAAE,CAAC;QAChB,MAAM,GAAG,GAAG,iBAAiB,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;QACpC,IAAI,GAAG;YAAE,OAAO,GAAG,CAAC;QACpB,CAAC,GAAG,eAAe,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACnC,CAAC;IACD,iBAAiB,CAAC,SAAS,GAAG,CAAC,CAAC;IAChC,CAAC,GAAG,iBAAiB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACjC,OAAO,CAAC,KAAK,IAAI,EAAE,CAAC;QAChB,MAAM,GAAG,GAAG,iBAAiB,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;QACpC,IAAI,GAAG;YAAE,OAAO,GAAG,CAAC;QACpB,CAAC,GAAG,iBAAiB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACrC,CAAC;IACD,OAAO,IAAI,CAAC;AAChB,CAAC;AAED,MAAM,iBAAiB,GAAa;IAChC,IAAI,EAAE,iBAAiB;IACvB,WAAW,EAAE,gFAAgF;IAC7F,KAAK,EAAE,MAAM;IACb,KAAK,EAAE,CAAC,SAAS,EAAE,UAAU,CAAC;IAC9B,cAAc,EAAE,EAAE;IAClB,OAAO,EAAE;QACL,uEAAuE;QACvE,2DAA2D;KAC9D;IAED,KAAK,CAAC,GAAgB;QAClB,MAAM,UAAU,GAAQ,EAAE,CAAC;QAC3B,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,GAAG,CAAC,aAAa,CAAC,MAAM,EAAE,CAAC,IAAI,CAAC,EAAE,CAAC;YACnD,MAAM,QAAQ,GAAG,GAAG,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC;YACtC,MAAM,OAAO,GAAG,CAAC,GAAG,CAAC,CAAC;YACtB,IAAI,GAAG,CAAC,cAAc,CAAC,OAAO,EAAE,iBAAiB,CAAC;gBAAE,SAAS;YAC7D,MAAM,QAAQ,GAAG,YAAY,CAAC,QAAQ,CAAC,CAAC;YACxC,IAAI,CAAC,QAAQ;gBAAE,SAAS;YACxB,UAAU,CAAC,IAAI,CAAC,IAAI,iBAAC,CACjB,OAAO,EACP,GAAG,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,EACnB,cAAc,QAAQ,uEAAuE,CAChG,CAAC,CAAC;QACP,CAAC;QACD,OAAO,UAAU,CAAC;IACtB,CAAC;CACJ,CAAC;AAEF,kBAAe,iBAAiB,CAAC","sourcesContent":["import type { EditRule, EditContext, Violation } from '../types';\nimport { Violation as V } from '../types';\n\nconst ARROW_PARAMS_RE = /\\(([^()]*)\\)\\s*=>/g;\nconst FN_DECL_PARAMS_RE = /\\bfunction\\s*[\\w$]*\\s*\\(([^()]*)\\)/g;\n\nfunction firstUntypedParam(paramsStr: string): string | null {\n    if (paramsStr.includes('{') || paramsStr.includes('[')) return null;\n    const parts = paramsStr.split(',').map((p) => p.trim()).filter((p) => p.length > 0);\n    for (const part of parts) {\n        if (part.startsWith('...')) continue;\n        if (part.includes(':')) continue;\n        if (part.includes('=')) continue;\n        if (part === 'this') continue;\n        if (/^[a-zA-Z_$][\\w$]*$/.test(part)) return part;\n    }\n    return null;\n}\n\nfunction findOffender(line: string): string | null {\n    ARROW_PARAMS_RE.lastIndex = 0;\n    let m: RegExpExecArray | null = ARROW_PARAMS_RE.exec(line);\n    while (m !== null) {\n        const bad = firstUntypedParam(m[1]);\n        if (bad) return bad;\n        m = ARROW_PARAMS_RE.exec(line);\n    }\n    FN_DECL_PARAMS_RE.lastIndex = 0;\n    m = FN_DECL_PARAMS_RE.exec(line);\n    while (m !== null) {\n        const bad = firstUntypedParam(m[1]);\n        if (bad) return bad;\n        m = FN_DECL_PARAMS_RE.exec(line);\n    }\n    return null;\n}\n\nconst noImplicitAnyRule: EditRule = {\n    name: 'no-implicit-any',\n    description: 'Disallow function parameters without explicit type annotations (implicit-any).',\n    scope: 'edit',\n    files: ['**/*.ts', '**/*.tsx'],\n    defaultOptions: {},\n    fixHint: [\n        'Add explicit types: (x: string) => ...   or   function foo(x: number)',\n        '// webpieces-disable no-implicit-any -- <one-line reason>',\n    ],\n\n    check(ctx: EditContext): readonly Violation[] {\n        const violations: V[] = [];\n        for (let i = 0; i < ctx.strippedLines.length; i += 1) {\n            const stripped = ctx.strippedLines[i];\n            const lineNum = i + 1;\n            if (ctx.isLineDisabled(lineNum, 'no-implicit-any')) continue;\n            const offender = findOffender(stripped);\n            if (!offender) continue;\n            violations.push(new V(\n                lineNum,\n                ctx.lines[i].trim(),\n                `Parameter \"${offender}\" has no type annotation. Add an explicit type to avoid implicit-any.`,\n            ));\n        }\n        return violations;\n    },\n};\n\nexport default noImplicitAnyRule;\n"]}

package/src/core/rules/no-shell-substitution.d.ts

@@ -0,0 +1,3 @@
+import type { BashRule } from '../types';
+declare const noShellSubstitutionRule: BashRule;
+export default noShellSubstitutionRule;

package/src/core/rules/no-shell-substitution.js

@@ -0,0 +1,54 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const types_1 = require("../types");
+const FIX_HINT = [
+    'Shell substitutions trigger Claude Code "simple_expansion" permission prompts that interrupt the user.',
+    'Instead:',
+    '  • Build payload files with Write, then: node script.js < /path/to/payload',
+    '  • Use Read, Grep, or Glob instead of piping shell output through $(...)',
+    '  • Write a small script file with Write and execute it: bash /path/to/script.sh',
+];
+const noShellSubstitutionRule = {
+    name: 'no-shell-substitution',
+    description: 'Reject Bash commands containing shell substitutions ($(...), backticks, $VAR).',
+    scope: 'bash',
+    files: [],
+    defaultOptions: {},
+    fixHint: FIX_HINT,
+    check(ctx) {
+        const scanned = stripSingleQuoted(ctx.command);
+        const violations = [];
+        if (/\$\(/.test(scanned)) {
+            violations.push(new types_1.Violation(1, truncate(ctx.command), 'Command contains `$(...)` command substitution.'));
+        }
+        if (hasUnescapedBacktick(scanned)) {
+            violations.push(new types_1.Violation(1, truncate(ctx.command), 'Command contains backtick command substitution.'));
+        }
+        if (/\$\{[A-Za-z_][A-Za-z0-9_]*\}/.test(scanned) || hasBareVarExpansion(scanned)) {
+            violations.push(new types_1.Violation(1, truncate(ctx.command), 'Command contains `$VAR` or `${VAR}` variable expansion.'));
+        }
+        return violations;
+    },
+};
+function stripSingleQuoted(cmd) {
+    return cmd.replace(/'[^']*'/g, "''");
+}
+function hasUnescapedBacktick(cmd) {
+    for (let i = 0; i < cmd.length; i += 1) {
+        if (cmd[i] === '`' && (i === 0 || cmd[i - 1] !== '\\'))
+            return true;
+    }
+    return false;
+}
+function hasBareVarExpansion(cmd) {
+    const re = /(^|[^\\])\$([A-Za-z_][A-Za-z0-9_]*)/g;
+    return re.test(cmd);
+}
+function truncate(s) {
+    const MAX = 120;
+    if (s.length <= MAX)
+        return s;
+    return s.slice(0, MAX) + '…';
+}
+exports.default = noShellSubstitutionRule;
+//# sourceMappingURL=no-shell-substitution.js.map

package/src/core/rules/no-shell-substitution.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-shell-substitution.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/no-shell-substitution.ts"],"names":[],"mappings":";;AACA,oCAA0C;AAE1C,MAAM,QAAQ,GAAsB;IAChC,wGAAwG;IACxG,UAAU;IACV,6EAA6E;IAC7E,2EAA2E;IAC3E,kFAAkF;CACrF,CAAC;AAEF,MAAM,uBAAuB,GAAa;IACtC,IAAI,EAAE,uBAAuB;IAC7B,WAAW,EAAE,gFAAgF;IAC7F,KAAK,EAAE,MAAM;IACb,KAAK,EAAE,EAAE;IACT,cAAc,EAAE,EAAE;IAClB,OAAO,EAAE,QAAQ;IAEjB,KAAK,CAAC,GAAgB;QAClB,MAAM,OAAO,GAAG,iBAAiB,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QAC/C,MAAM,UAAU,GAAgB,EAAE,CAAC;QAEnC,IAAI,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC;YACvB,UAAU,CAAC,IAAI,CAAC,IAAI,iBAAC,CACjB,CAAC,EACD,QAAQ,CAAC,GAAG,CAAC,OAAO,CAAC,EACrB,iDAAiD,CACpD,CAAC,CAAC;QACP,CAAC;QACD,IAAI,oBAAoB,CAAC,OAAO,CAAC,EAAE,CAAC;YAChC,UAAU,CAAC,IAAI,CAAC,IAAI,iBAAC,CACjB,CAAC,EACD,QAAQ,CAAC,GAAG,CAAC,OAAO,CAAC,EACrB,iDAAiD,CACpD,CAAC,CAAC;QACP,CAAC;QACD,IAAI,8BAA8B,CAAC,IAAI,CAAC,OAAO,CAAC,IAAI,mBAAmB,CAAC,OAAO,CAAC,EAAE,CAAC;YAC/E,UAAU,CAAC,IAAI,CAAC,IAAI,iBAAC,CACjB,CAAC,EACD,QAAQ,CAAC,GAAG,CAAC,OAAO,CAAC,EACrB,yDAAyD,CAC5D,CAAC,CAAC;QACP,CAAC;QACD,OAAO,UAAU,CAAC;IACtB,CAAC;CACJ,CAAC;AAEF,SAAS,iBAAiB,CAAC,GAAW;IAClC,OAAO,GAAG,CAAC,OAAO,CAAC,UAAU,EAAE,IAAI,CAAC,CAAC;AACzC,CAAC;AAED,SAAS,oBAAoB,CAAC,GAAW;IACrC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,GAAG,CAAC,MAAM,EAAE,CAAC,IAAI,CAAC,EAAE,CAAC;QACrC,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,GAAG,IAAI,CAAC,CAAC,KAAK,CAAC,IAAI,GAAG,CAAC,CAAC,GAAG,CAAC,CAAC,KAAK,IAAI,CAAC;YAAE,OAAO,IAAI,CAAC;IACxE,CAAC;IACD,OAAO,KAAK,CAAC;AACjB,CAAC;AAED,SAAS,mBAAmB,CAAC,GAAW;IACpC,MAAM,EAAE,GAAG,sCAAsC,CAAC;IAClD,OAAO,EAAE,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;AACxB,CAAC;AAED,SAAS,QAAQ,CAAC,CAAS;IACvB,MAAM,GAAG,GAAG,GAAG,CAAC;IAChB,IAAI,CAAC,CAAC,MAAM,IAAI,GAAG;QAAE,OAAO,CAAC,CAAC;IAC9B,OAAO,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,GAAG,GAAG,CAAC;AACjC,CAAC;AAED,kBAAe,uBAAuB,CAAC","sourcesContent":["import type { BashRule, BashContext, Violation } from '../types';\nimport { Violation as V } from '../types';\n\nconst FIX_HINT: readonly string[] = [\n    'Shell substitutions trigger Claude Code \"simple_expansion\" permission prompts that interrupt the user.',\n    'Instead:',\n    '  • Build payload files with Write, then: node script.js < /path/to/payload',\n    '  • Use Read, Grep, or Glob instead of piping shell output through $(...)',\n    '  • Write a small script file with Write and execute it: bash /path/to/script.sh',\n];\n\nconst noShellSubstitutionRule: BashRule = {\n    name: 'no-shell-substitution',\n    description: 'Reject Bash commands containing shell substitutions ($(...), backticks, $VAR).',\n    scope: 'bash',\n    files: [],\n    defaultOptions: {},\n    fixHint: FIX_HINT,\n\n    check(ctx: BashContext): readonly Violation[] {\n        const scanned = stripSingleQuoted(ctx.command);\n        const violations: Violation[] = [];\n\n        if (/\\$\\(/.test(scanned)) {\n            violations.push(new V(\n                1,\n                truncate(ctx.command),\n                'Command contains `$(...)` command substitution.',\n            ));\n        }\n        if (hasUnescapedBacktick(scanned)) {\n            violations.push(new V(\n                1,\n                truncate(ctx.command),\n                'Command contains backtick command substitution.',\n            ));\n        }\n        if (/\\$\\{[A-Za-z_][A-Za-z0-9_]*\\}/.test(scanned) || hasBareVarExpansion(scanned)) {\n            violations.push(new V(\n                1,\n                truncate(ctx.command),\n                'Command contains `$VAR` or `${VAR}` variable expansion.',\n            ));\n        }\n        return violations;\n    },\n};\n\nfunction stripSingleQuoted(cmd: string): string {\n    return cmd.replace(/'[^']*'/g, \"''\");\n}\n\nfunction hasUnescapedBacktick(cmd: string): boolean {\n    for (let i = 0; i < cmd.length; i += 1) {\n        if (cmd[i] === '`' && (i === 0 || cmd[i - 1] !== '\\\\')) return true;\n    }\n    return false;\n}\n\nfunction hasBareVarExpansion(cmd: string): boolean {\n    const re = /(^|[^\\\\])\\$([A-Za-z_][A-Za-z0-9_]*)/g;\n    return re.test(cmd);\n}\n\nfunction truncate(s: string): string {\n    const MAX = 120;\n    if (s.length <= MAX) return s;\n    return s.slice(0, MAX) + '…';\n}\n\nexport default noShellSubstitutionRule;\n"]}

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

@@ -0,0 +1,3 @@
+import type { EditRule } from '../types';
+declare const noUnmanagedExceptionsRule: EditRule;
+export default noUnmanagedExceptionsRule;

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

@@ -1,13 +1,11 @@
-import type { EditRule, EditContext, Violation } from '../types';
-import { Violation as V } from '../types';
-import { writeTemplateIfMissing } from '../instruct-ai-writer';
-
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const types_1 = require("../types");
+const instruct_ai_writer_1 = require("../instruct-ai-writer");
 const TRY_PATTERN = /\btry\s*\{/;
-
 // Both webpieces-disable and the existing ESLint directive suppress this rule
 const DISABLE_PATTERN = /@webpieces\/no-unmanaged-exceptions|webpieces-disable\s+(?:[\w-]+,\s*)*no-unmanaged-exceptions/;
-
-const noUnmanagedExceptionsRule: EditRule = {
+const noUnmanagedExceptionsRule = {
     name: 'no-unmanaged-exceptions',
     description: 'try/catch is generally not allowed. Only allowed in chokepoints (filter, globalErrorHandler) or other rare locations.',
     scope: 'edit',
@@ -19,30 +17,29 @@ const noUnmanagedExceptionsRule: EditRule = {
         '// webpieces-disable no-unmanaged-exceptions -- <reason>',
         'When try/catch IS used (after disabling), the catch block MUST use: catch (err: unknown) { const error = toError(err); ... } or //const error = toError(err); to explicitly ignore.',
     ],
-
-    check(ctx: EditContext): readonly Violation[] {
-        const violations: V[] = [];
+    check(ctx) {
+        const violations = [];
         for (let i = 0; i < ctx.strippedLines.length; i += 1) {
             const stripped = ctx.strippedLines[i];
-            if (!TRY_PATTERN.test(stripped)) continue;
+            if (!TRY_PATTERN.test(stripped))
+                continue;
             const lineNum = i + 1;
-            if (ctx.isLineDisabled(lineNum, 'no-unmanaged-exceptions')) continue;
-            if (hasPrecedingDisable(ctx.lines, i)) continue;
-            violations.push(new V(
-                lineNum,
-                ctx.lines[i].trim(),
-                'try/catch is generally not allowed. It is only allowed in chokepoints (filter, globalErrorHandler) or other rare locations.',
-            ));
+            if (ctx.isLineDisabled(lineNum, 'no-unmanaged-exceptions'))
+                continue;
+            if (hasPrecedingDisable(ctx.lines, i))
+                continue;
+            violations.push(new types_1.Violation(lineNum, ctx.lines[i].trim(), 'try/catch is generally not allowed. It is only allowed in chokepoints (filter, globalErrorHandler) or other rare locations.'));
         }
-        if (violations.length > 0) writeTemplateIfMissing(ctx.workspaceRoot, 'webpieces.exceptions.md');
+        if (violations.length > 0)
+            (0, instruct_ai_writer_1.writeTemplateIfMissing)(ctx.workspaceRoot, 'webpieces.exceptions.md');
         return violations;
     },
 };
-
-function hasPrecedingDisable(lines: readonly string[], idx: number): boolean {
-    if (idx === 0) return false;
+function hasPrecedingDisable(lines, idx) {
+    if (idx === 0)
+        return false;
     const prevLine = lines[idx - 1];
     return DISABLE_PATTERN.test(prevLine);
 }
-
-export default noUnmanagedExceptionsRule;
+exports.default = noUnmanagedExceptionsRule;
+//# sourceMappingURL=no-unmanaged-exceptions.js.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"no-unmanaged-exceptions.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/no-unmanaged-exceptions.ts"],"names":[],"mappings":";;AACA,oCAA0C;AAC1C,8DAA+D;AAE/D,MAAM,WAAW,GAAG,YAAY,CAAC;AAEjC,8EAA8E;AAC9E,MAAM,eAAe,GAAG,gGAAgG,CAAC;AAEzH,MAAM,yBAAyB,GAAa;IACxC,IAAI,EAAE,yBAAyB;IAC/B,WAAW,EAAE,uHAAuH;IACpI,KAAK,EAAE,MAAM;IACb,KAAK,EAAE,CAAC,SAAS,EAAE,UAAU,CAAC;IAC9B,cAAc,EAAE,EAAE;IAClB,OAAO,EAAE;QACL,4GAA4G;QAC5G,qIAAqI;QACrI,0DAA0D;QAC1D,qLAAqL;KACxL;IAED,KAAK,CAAC,GAAgB;QAClB,MAAM,UAAU,GAAQ,EAAE,CAAC;QAC3B,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,GAAG,CAAC,aAAa,CAAC,MAAM,EAAE,CAAC,IAAI,CAAC,EAAE,CAAC;YACnD,MAAM,QAAQ,GAAG,GAAG,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC;YACtC,IAAI,CAAC,WAAW,CAAC,IAAI,CAAC,QAAQ,CAAC;gBAAE,SAAS;YAC1C,MAAM,OAAO,GAAG,CAAC,GAAG,CAAC,CAAC;YACtB,IAAI,GAAG,CAAC,cAAc,CAAC,OAAO,EAAE,yBAAyB,CAAC;gBAAE,SAAS;YACrE,IAAI,mBAAmB,CAAC,GAAG,CAAC,KAAK,EAAE,CAAC,CAAC;gBAAE,SAAS;YAChD,UAAU,CAAC,IAAI,CAAC,IAAI,iBAAC,CACjB,OAAO,EACP,GAAG,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,EACnB,6HAA6H,CAChI,CAAC,CAAC;QACP,CAAC;QACD,IAAI,UAAU,CAAC,MAAM,GAAG,CAAC;YAAE,IAAA,2CAAsB,EAAC,GAAG,CAAC,aAAa,EAAE,yBAAyB,CAAC,CAAC;QAChG,OAAO,UAAU,CAAC;IACtB,CAAC;CACJ,CAAC;AAEF,SAAS,mBAAmB,CAAC,KAAwB,EAAE,GAAW;IAC9D,IAAI,GAAG,KAAK,CAAC;QAAE,OAAO,KAAK,CAAC;IAC5B,MAAM,QAAQ,GAAG,KAAK,CAAC,GAAG,GAAG,CAAC,CAAC,CAAC;IAChC,OAAO,eAAe,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;AAC1C,CAAC;AAED,kBAAe,yBAAyB,CAAC","sourcesContent":["import type { EditRule, EditContext, Violation } from '../types';\nimport { Violation as V } from '../types';\nimport { writeTemplateIfMissing } from '../instruct-ai-writer';\n\nconst TRY_PATTERN = /\\btry\\s*\\{/;\n\n// Both webpieces-disable and the existing ESLint directive suppress this rule\nconst DISABLE_PATTERN = /@webpieces\\/no-unmanaged-exceptions|webpieces-disable\\s+(?:[\\w-]+,\\s*)*no-unmanaged-exceptions/;\n\nconst noUnmanagedExceptionsRule: EditRule = {\n    name: 'no-unmanaged-exceptions',\n    description: 'try/catch is generally not allowed. Only allowed in chokepoints (filter, globalErrorHandler) or other rare locations.',\n    scope: 'edit',\n    files: ['**/*.ts', '**/*.tsx'],\n    defaultOptions: {},\n    fixHint: [\n        'VERY IMPORTANT: READ .webpieces/instruct-ai/webpieces.exceptions.md to understand why and how to fix this!',\n        'Exceptions should bubble to a chokepoint (filter in node.js, globalErrorHandler in Angular). Most code should NOT catch exceptions.',\n        '// webpieces-disable no-unmanaged-exceptions -- <reason>',\n        'When try/catch IS used (after disabling), the catch block MUST use: catch (err: unknown) { const error = toError(err); ... } or //const error = toError(err); to explicitly ignore.',\n    ],\n\n    check(ctx: EditContext): readonly Violation[] {\n        const violations: V[] = [];\n        for (let i = 0; i < ctx.strippedLines.length; i += 1) {\n            const stripped = ctx.strippedLines[i];\n            if (!TRY_PATTERN.test(stripped)) continue;\n            const lineNum = i + 1;\n            if (ctx.isLineDisabled(lineNum, 'no-unmanaged-exceptions')) continue;\n            if (hasPrecedingDisable(ctx.lines, i)) continue;\n            violations.push(new V(\n                lineNum,\n                ctx.lines[i].trim(),\n                'try/catch is generally not allowed. It is only allowed in chokepoints (filter, globalErrorHandler) or other rare locations.',\n            ));\n        }\n        if (violations.length > 0) writeTemplateIfMissing(ctx.workspaceRoot, 'webpieces.exceptions.md');\n        return violations;\n    },\n};\n\nfunction hasPrecedingDisable(lines: readonly string[], idx: number): boolean {\n    if (idx === 0) return false;\n    const prevLine = lines[idx - 1];\n    return DISABLE_PATTERN.test(prevLine);\n}\n\nexport default noUnmanagedExceptionsRule;\n"]}

package/src/core/rules/require-return-type.d.ts

@@ -0,0 +1,3 @@
+import type { EditRule } from '../types';
+declare const requireReturnTypeRule: EditRule;
+export default requireReturnTypeRule;