@webpieces/ai-hook-rules 0.3.185 → 0.3.187

package/src/core/rules/pr-merge-cleanup.js

@@ -1,18 +1,21 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.PrMergeCleanupRule = void 0;
 const child_process_1 = require("child_process");
 const types_1 = require("../types");
-const prMergeCleanup = {
-    name: 'pr-merge-cleanup',
-    description: 'After merging a PR, require switching to main, pulling, and deleting the local branch.',
-    scope: 'bash',
-    files: [],
-    defaultOptions: {},
-    fixHint: [
+const rule_base_1 = require("../rule-base");
+function truncate(s) {
+    const MAX = 120;
+    return s.length <= MAX ? s : s.slice(0, MAX) + '…';
+}
+class PrMergeCleanupRule extends rule_base_1.BashRuleBase {
+    constructor(config) { super(config, 'pr-merge-cleanup'); }
+    description = 'After merging a PR, require switching to main, pulling, and deleting the local branch.';
+    fixHint = [
         'After merging a PR you must clean up the local branch.',
         'Run this combined command instead:',
         '  gh pr merge --squash && git checkout main && git pull && git branch -d <current-branch>',
-    ],
+    ];
     check(ctx) {
         if (!/gh\s+pr\s+merge/.test(ctx.command))
             return [];
@@ -29,11 +32,7 @@ const prMergeCleanup = {
                 'Run this combined command instead:',
                 `  gh pr merge --squash && git checkout main && git pull && git branch -d ${currentBranch}`,
             ].join('\n'))];
-    },
-};
-function truncate(s) {
-    const MAX = 120;
-    return s.length <= MAX ? s : s.slice(0, MAX) + '…';
+    }
 }
-exports.default = prMergeCleanup;
+exports.PrMergeCleanupRule = PrMergeCleanupRule;
 //# sourceMappingURL=pr-merge-cleanup.js.map

package/src/core/rules/pr-merge-cleanup.js.map

@@ -1 +1 @@
-{"version":3,"file":"pr-merge-cleanup.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/pr-merge-cleanup.ts"],"names":[],"mappings":";;AAAA,iDAAyC;AAEzC,oCAA0C;AAE1C,MAAM,cAAc,GAAa;IAC7B,IAAI,EAAE,kBAAkB;IACxB,WAAW,EAAE,wFAAwF;IACrG,KAAK,EAAE,MAAM;IACb,KAAK,EAAE,EAAE;IACT,cAAc,EAAE,EAAE;IAClB,OAAO,EAAE;QACL,wDAAwD;QACxD,oCAAoC;QACpC,2FAA2F;KAC9F;IAED,KAAK,CAAC,GAAgB;QAClB,IAAI,CAAC,iBAAiB,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC;YAAE,OAAO,EAAE,CAAC;QAEpD,MAAM,WAAW,GAAG,gCAAgC,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QACvE,MAAM,SAAS,GAAG,sBAAsB,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QAE3D,IAAI,WAAW,IAAI,SAAS;YAAE,OAAO,EAAE,CAAC;QAExC,MAAM,aAAa,GAAG,IAAA,wBAAQ,EAAC,iCAAiC,EAAE;YAC9D,GAAG,EAAE,GAAG,CAAC,aAAa;YACtB,QAAQ,EAAE,MAAM;SACnB,CAAC,CAAC,IAAI,EAAE,CAAC;QAEV,OAAO,CAAC,IAAI,iBAAC,CACT,CAAC,EACD,QAAQ,CAAC,GAAG,CAAC,OAAO,CAAC,EACrB;gBACI,2EAA2E;gBAC3E,oCAAoC;gBACpC,4EAA4E,aAAa,EAAE;aAC9F,CAAC,IAAI,CAAC,IAAI,CAAC,CACf,CAAC,CAAC;IACP,CAAC;CACJ,CAAC;AAEF,SAAS,QAAQ,CAAC,CAAS;IACvB,MAAM,GAAG,GAAG,GAAG,CAAC;IAChB,OAAO,CAAC,CAAC,MAAM,IAAI,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,GAAG,GAAG,CAAC;AACvD,CAAC;AAED,kBAAe,cAAc,CAAC","sourcesContent":["import { execSync } from 'child_process';\nimport type { BashRule, BashContext, Violation } from '../types';\nimport { Violation as V } from '../types';\n\nconst prMergeCleanup: BashRule = {\n    name: 'pr-merge-cleanup',\n    description: 'After merging a PR, require switching to main, pulling, and deleting the local branch.',\n    scope: 'bash',\n    files: [],\n    defaultOptions: {},\n    fixHint: [\n        'After merging a PR you must clean up the local branch.',\n        'Run this combined command instead:',\n        '  gh pr merge --squash && git checkout main && git pull && git branch -d <current-branch>',\n    ],\n\n    check(ctx: BashContext): readonly Violation[] {\n        if (!/gh\\s+pr\\s+merge/.test(ctx.command)) return [];\n\n        const hasCheckout = /git\\s+(checkout|switch)\\s+main/.test(ctx.command);\n        const hasDelete = /git\\s+branch\\s+-[dD]/.test(ctx.command);\n\n        if (hasCheckout && hasDelete) return [];\n\n        const currentBranch = execSync('git rev-parse --abbrev-ref HEAD', {\n            cwd: ctx.workspaceRoot,\n            encoding: 'utf8',\n        }).trim();\n\n        return [new V(\n            1,\n            truncate(ctx.command),\n            [\n                '[pr-merge-cleanup] After merging a PR you must clean up the local branch.',\n                'Run this combined command instead:',\n                `  gh pr merge --squash && git checkout main && git pull && git branch -d ${currentBranch}`,\n            ].join('\\n'),\n        )];\n    },\n};\n\nfunction truncate(s: string): string {\n    const MAX = 120;\n    return s.length <= MAX ? s : s.slice(0, MAX) + '…';\n}\n\nexport default prMergeCleanup;\n"]}
+{"version":3,"file":"pr-merge-cleanup.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/pr-merge-cleanup.ts"],"names":[],"mappings":";;;AAAA,iDAAyC;AAKzC,oCAA0C;AAC1C,4CAA4C;AAE5C,SAAS,QAAQ,CAAC,CAAS;IACvB,MAAM,GAAG,GAAG,GAAG,CAAC;IAChB,OAAO,CAAC,CAAC,MAAM,IAAI,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,GAAG,GAAG,CAAC;AACvD,CAAC;AAED,MAAa,kBAAmB,SAAQ,wBAAkC;IACtE,YAAY,MAA4B,IAAI,KAAK,CAAC,MAAM,EAAE,kBAAkB,CAAC,CAAC,CAAC,CAAC;IAEvE,WAAW,GAAG,wFAAwF,CAAC;IACvG,OAAO,GAAG;QACf,wDAAwD;QACxD,oCAAoC;QACpC,2FAA2F;KAC9F,CAAC;IAEF,KAAK,CAAC,GAAgB;QAClB,IAAI,CAAC,iBAAiB,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC;YAAE,OAAO,EAAE,CAAC;QAEpD,MAAM,WAAW,GAAG,gCAAgC,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QACvE,MAAM,SAAS,GAAG,sBAAsB,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QAE3D,IAAI,WAAW,IAAI,SAAS;YAAE,OAAO,EAAE,CAAC;QAExC,MAAM,aAAa,GAAG,IAAA,wBAAQ,EAAC,iCAAiC,EAAE;YAC9D,GAAG,EAAE,GAAG,CAAC,aAAa;YACtB,QAAQ,EAAE,MAAM;SACnB,CAAC,CAAC,IAAI,EAAE,CAAC;QAEV,OAAO,CAAC,IAAI,iBAAC,CACT,CAAC,EACD,QAAQ,CAAC,GAAG,CAAC,OAAO,CAAC,EACrB;gBACI,2EAA2E;gBAC3E,oCAAoC;gBACpC,4EAA4E,aAAa,EAAE;aAC9F,CAAC,IAAI,CAAC,IAAI,CAAC,CACf,CAAC,CAAC;IACP,CAAC;CACJ;AAjCD,gDAiCC","sourcesContent":["import { execSync } from 'child_process';\n\nimport { PrMergeCleanupConfig } from '@webpieces/rules-config';\n\nimport type { BashContext, Violation } from '../types';\nimport { Violation as V } from '../types';\nimport { BashRuleBase } from '../rule-base';\n\nfunction truncate(s: string): string {\n    const MAX = 120;\n    return s.length <= MAX ? s : s.slice(0, MAX) + '…';\n}\n\nexport class PrMergeCleanupRule extends BashRuleBase<PrMergeCleanupConfig> {\n    constructor(config: PrMergeCleanupConfig) { super(config, 'pr-merge-cleanup'); }\n\n    readonly description = 'After merging a PR, require switching to main, pulling, and deleting the local branch.';\n    readonly fixHint = [\n        'After merging a PR you must clean up the local branch.',\n        'Run this combined command instead:',\n        '  gh pr merge --squash && git checkout main && git pull && git branch -d <current-branch>',\n    ];\n\n    check(ctx: BashContext): readonly Violation[] {\n        if (!/gh\\s+pr\\s+merge/.test(ctx.command)) return [];\n\n        const hasCheckout = /git\\s+(checkout|switch)\\s+main/.test(ctx.command);\n        const hasDelete = /git\\s+branch\\s+-[dD]/.test(ctx.command);\n\n        if (hasCheckout && hasDelete) return [];\n\n        const currentBranch = execSync('git rev-parse --abbrev-ref HEAD', {\n            cwd: ctx.workspaceRoot,\n            encoding: 'utf8',\n        }).trim();\n\n        return [new V(\n            1,\n            truncate(ctx.command),\n            [\n                '[pr-merge-cleanup] After merging a PR you must clean up the local branch.',\n                'Run this combined command instead:',\n                `  gh pr merge --squash && git checkout main && git pull && git branch -d ${currentBranch}`,\n            ].join('\\n'),\n        )];\n    }\n}\n"]}

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

@@ -1,3 +1,10 @@
-import type { EditRule } from '../types';
-declare const requireReturnTypeRule: EditRule;
-export default requireReturnTypeRule;
+import { RequireReturnTypeConfig } from '@webpieces/rules-config';
+import type { EditContext, Violation } from '../types';
+import { EditRuleBase } from '../rule-base';
+export declare class RequireReturnTypeRule extends EditRuleBase<RequireReturnTypeConfig> {
+    constructor(config: RequireReturnTypeConfig);
+    readonly description = "Every function and method must declare its return type.";
+    readonly files: string[];
+    readonly fixHint: string[];
+    check(ctx: EditContext): readonly Violation[];
+}

package/src/core/rules/require-return-type.js

@@ -1,7 +1,9 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-const types_1 = require("../types");
+exports.RequireReturnTypeRule = void 0;
 const rules_config_1 = require("@webpieces/rules-config");
+const types_1 = require("../types");
+const rule_base_1 = require("../rule-base");
 // Matches function/method signatures that don't have `: ReturnType` before the `{` body opener.
 // Pattern: function name(<params>) { — missing `: Type` between `)` and `{`
 const FUNC_DECL_MISSING = /\bfunction\s+\w+\s*(?:<[^>]*>)?\s*\([^)]*\)\s*\{/;
@@ -13,16 +15,26 @@ const ARROW_MISSING = /\bconst\s+\w+\s*=\s*(?:async\s+)?(?:<[^>]*>)?\s*\([^)]*\)
 const HAS_RETURN_TYPE = /\)\s*:\s*\S/;
 // Skip constructors, getters, setters, and control flow keywords
 const SKIP_PATTERN = /\b(?:constructor|get\s+\w+|set\s+\w+|if|else|while|for|switch|catch|return)\s*\(/;
-const requireReturnTypeRule = {
-    name: 'require-return-type',
-    description: 'Every function and method must declare its return type.',
-    scope: 'edit',
-    files: ['**/*.ts', '**/*.tsx'],
-    defaultOptions: {},
-    fixHint: [
+function isMissingReturnType(line) {
+    if (SKIP_PATTERN.test(line))
+        return false;
+    const isFuncLike = FUNC_DECL_MISSING.test(line) ||
+        METHOD_MISSING.test(line) ||
+        ARROW_MISSING.test(line);
+    if (!isFuncLike)
+        return false;
+    if (HAS_RETURN_TYPE.test(line))
+        return false;
+    return true;
+}
+class RequireReturnTypeRule extends rule_base_1.EditRuleBase {
+    constructor(config) { super(config, 'require-return-type'); }
+    description = 'Every function and method must declare its return type.';
+    files = ['**/*.ts', '**/*.tsx'];
+    fixHint = [
         'Add return type: function foo(x: T): ReturnType { ... }',
         '// webpieces-disable require-return-type -- <reason>',
-    ],
+    ];
     check(ctx) {
         const violations = [];
         for (let i = 0; i < ctx.strippedLines.length; i += 1) {
@@ -35,19 +47,7 @@ const requireReturnTypeRule = {
             violations.push(new types_1.Violation(lineNum, ctx.lines[i].trim(), 'Missing return type annotation.'));
         }
         return violations;
-    },
-};
-function isMissingReturnType(line) {
-    if (SKIP_PATTERN.test(line))
-        return false;
-    const isFuncLike = FUNC_DECL_MISSING.test(line) ||
-        METHOD_MISSING.test(line) ||
-        ARROW_MISSING.test(line);
-    if (!isFuncLike)
-        return false;
-    if (HAS_RETURN_TYPE.test(line))
-        return false;
-    return true;
+    }
 }
-exports.default = requireReturnTypeRule;
+exports.RequireReturnTypeRule = RequireReturnTypeRule;
 //# sourceMappingURL=require-return-type.js.map

package/src/core/rules/require-return-type.js.map

@@ -1 +1 @@
-{"version":3,"file":"require-return-type.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/require-return-type.ts"],"names":[],"mappings":";;AACA,oCAA0C;AAC1C,0DAAqD;AAErD,gGAAgG;AAChG,4EAA4E;AAC5E,MAAM,iBAAiB,GAAG,kDAAkD,CAAC;AAE7E,8EAA8E;AAC9E,MAAM,cAAc,GAAG,yDAAyD,CAAC;AAEjF,qFAAqF;AACrF,MAAM,aAAa,GAAG,gEAAgE,CAAC;AAEvF,qEAAqE;AACrE,MAAM,eAAe,GAAG,aAAa,CAAC;AAEtC,iEAAiE;AACjE,MAAM,YAAY,GAAG,kFAAkF,CAAC;AAExG,MAAM,qBAAqB,GAAa;IACpC,IAAI,EAAE,qBAAqB;IAC3B,WAAW,EAAE,yDAAyD;IACtE,KAAK,EAAE,MAAM;IACb,KAAK,EAAE,CAAC,SAAS,EAAE,UAAU,CAAC;IAC9B,cAAc,EAAE,EAAE;IAClB,OAAO,EAAE;QACL,yDAAyD;QACzD,sDAAsD;KACzD;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,mBAAmB,CAAC,QAAQ,CAAC;gBAAE,SAAS;YAC7C,MAAM,OAAO,GAAG,CAAC,GAAG,CAAC,CAAC;YACtB,IAAI,GAAG,CAAC,cAAc,CAAC,OAAO,EAAE,yBAAU,CAAC,mBAAmB,CAAC;gBAAE,SAAS;YAC1E,UAAU,CAAC,IAAI,CAAC,IAAI,iBAAC,CACjB,OAAO,EACP,GAAG,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,EACnB,iCAAiC,CACpC,CAAC,CAAC;QACP,CAAC;QACD,OAAO,UAAU,CAAC;IACtB,CAAC;CACJ,CAAC;AAEF,SAAS,mBAAmB,CAAC,IAAY;IACrC,IAAI,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC;QAAE,OAAO,KAAK,CAAC;IAC1C,MAAM,UAAU,GACZ,iBAAiB,CAAC,IAAI,CAAC,IAAI,CAAC;QAC5B,cAAc,CAAC,IAAI,CAAC,IAAI,CAAC;QACzB,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC7B,IAAI,CAAC,UAAU;QAAE,OAAO,KAAK,CAAC;IAC9B,IAAI,eAAe,CAAC,IAAI,CAAC,IAAI,CAAC;QAAE,OAAO,KAAK,CAAC;IAC7C,OAAO,IAAI,CAAC;AAChB,CAAC;AAED,kBAAe,qBAAqB,CAAC","sourcesContent":["import type { EditRule, EditContext, Violation } from '../types';\nimport { Violation as V } from '../types';\nimport { RULE_NAMES } from '@webpieces/rules-config';\n\n// Matches function/method signatures that don't have `: ReturnType` before the `{` body opener.\n// Pattern: function name(<params>) { — missing `: Type` between `)` and `{`\nconst FUNC_DECL_MISSING = /\\bfunction\\s+\\w+\\s*(?:<[^>]*>)?\\s*\\([^)]*\\)\\s*\\{/;\n\n// Matches class method signatures: indented, optional async, name(<params>) {\nconst METHOD_MISSING = /^\\s{2,}(?:async\\s+)?\\w+\\s*(?:<[^>]*>)?\\s*\\([^)]*\\)\\s*\\{/;\n\n// Arrow function: const name = (async)? (<params>) => — missing `: Type` before `=>`\nconst ARROW_MISSING = /\\bconst\\s+\\w+\\s*=\\s*(?:async\\s+)?(?:<[^>]*>)?\\s*\\([^)]*\\)\\s*=>/;\n\n// Lines that have ): ReturnType before { or => — these are COMPLIANT\nconst HAS_RETURN_TYPE = /\\)\\s*:\\s*\\S/;\n\n// Skip constructors, getters, setters, and control flow keywords\nconst SKIP_PATTERN = /\\b(?:constructor|get\\s+\\w+|set\\s+\\w+|if|else|while|for|switch|catch|return)\\s*\\(/;\n\nconst requireReturnTypeRule: EditRule = {\n    name: 'require-return-type',\n    description: 'Every function and method must declare its return type.',\n    scope: 'edit',\n    files: ['**/*.ts', '**/*.tsx'],\n    defaultOptions: {},\n    fixHint: [\n        'Add return type: function foo(x: T): ReturnType { ... }',\n        '// webpieces-disable require-return-type -- <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 (!isMissingReturnType(stripped)) continue;\n            const lineNum = i + 1;\n            if (ctx.isLineDisabled(lineNum, RULE_NAMES.REQUIRE_RETURN_TYPE)) continue;\n            violations.push(new V(\n                lineNum,\n                ctx.lines[i].trim(),\n                'Missing return type annotation.',\n            ));\n        }\n        return violations;\n    },\n};\n\nfunction isMissingReturnType(line: string): boolean {\n    if (SKIP_PATTERN.test(line)) return false;\n    const isFuncLike =\n        FUNC_DECL_MISSING.test(line) ||\n        METHOD_MISSING.test(line) ||\n        ARROW_MISSING.test(line);\n    if (!isFuncLike) return false;\n    if (HAS_RETURN_TYPE.test(line)) return false;\n    return true;\n}\n\nexport default requireReturnTypeRule;\n"]}
+{"version":3,"file":"require-return-type.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/require-return-type.ts"],"names":[],"mappings":";;;AAAA,0DAA8E;AAG9E,oCAA0C;AAC1C,4CAA4C;AAE5C,gGAAgG;AAChG,4EAA4E;AAC5E,MAAM,iBAAiB,GAAG,kDAAkD,CAAC;AAE7E,8EAA8E;AAC9E,MAAM,cAAc,GAAG,yDAAyD,CAAC;AAEjF,qFAAqF;AACrF,MAAM,aAAa,GAAG,gEAAgE,CAAC;AAEvF,qEAAqE;AACrE,MAAM,eAAe,GAAG,aAAa,CAAC;AAEtC,iEAAiE;AACjE,MAAM,YAAY,GAAG,kFAAkF,CAAC;AAExG,SAAS,mBAAmB,CAAC,IAAY;IACrC,IAAI,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC;QAAE,OAAO,KAAK,CAAC;IAC1C,MAAM,UAAU,GACZ,iBAAiB,CAAC,IAAI,CAAC,IAAI,CAAC;QAC5B,cAAc,CAAC,IAAI,CAAC,IAAI,CAAC;QACzB,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC7B,IAAI,CAAC,UAAU;QAAE,OAAO,KAAK,CAAC;IAC9B,IAAI,eAAe,CAAC,IAAI,CAAC,IAAI,CAAC;QAAE,OAAO,KAAK,CAAC;IAC7C,OAAO,IAAI,CAAC;AAChB,CAAC;AAED,MAAa,qBAAsB,SAAQ,wBAAqC;IAC5E,YAAY,MAA+B,IAAI,KAAK,CAAC,MAAM,EAAE,qBAAqB,CAAC,CAAC,CAAC,CAAC;IAE7E,WAAW,GAAG,yDAAyD,CAAC;IAC/D,KAAK,GAAG,CAAC,SAAS,EAAE,UAAU,CAAC,CAAC;IACzC,OAAO,GAAG;QACf,yDAAyD;QACzD,sDAAsD;KACzD,CAAC;IAEF,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,mBAAmB,CAAC,QAAQ,CAAC;gBAAE,SAAS;YAC7C,MAAM,OAAO,GAAG,CAAC,GAAG,CAAC,CAAC;YACtB,IAAI,GAAG,CAAC,cAAc,CAAC,OAAO,EAAE,yBAAU,CAAC,mBAAmB,CAAC;gBAAE,SAAS;YAC1E,UAAU,CAAC,IAAI,CAAC,IAAI,iBAAC,CACjB,OAAO,EACP,GAAG,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,EACnB,iCAAiC,CACpC,CAAC,CAAC;QACP,CAAC;QACD,OAAO,UAAU,CAAC;IACtB,CAAC;CACJ;AAzBD,sDAyBC","sourcesContent":["import { RequireReturnTypeConfig, RULE_NAMES } from '@webpieces/rules-config';\n\nimport type { EditContext, Violation } from '../types';\nimport { Violation as V } from '../types';\nimport { EditRuleBase } from '../rule-base';\n\n// Matches function/method signatures that don't have `: ReturnType` before the `{` body opener.\n// Pattern: function name(<params>) { — missing `: Type` between `)` and `{`\nconst FUNC_DECL_MISSING = /\\bfunction\\s+\\w+\\s*(?:<[^>]*>)?\\s*\\([^)]*\\)\\s*\\{/;\n\n// Matches class method signatures: indented, optional async, name(<params>) {\nconst METHOD_MISSING = /^\\s{2,}(?:async\\s+)?\\w+\\s*(?:<[^>]*>)?\\s*\\([^)]*\\)\\s*\\{/;\n\n// Arrow function: const name = (async)? (<params>) => — missing `: Type` before `=>`\nconst ARROW_MISSING = /\\bconst\\s+\\w+\\s*=\\s*(?:async\\s+)?(?:<[^>]*>)?\\s*\\([^)]*\\)\\s*=>/;\n\n// Lines that have ): ReturnType before { or => — these are COMPLIANT\nconst HAS_RETURN_TYPE = /\\)\\s*:\\s*\\S/;\n\n// Skip constructors, getters, setters, and control flow keywords\nconst SKIP_PATTERN = /\\b(?:constructor|get\\s+\\w+|set\\s+\\w+|if|else|while|for|switch|catch|return)\\s*\\(/;\n\nfunction isMissingReturnType(line: string): boolean {\n    if (SKIP_PATTERN.test(line)) return false;\n    const isFuncLike =\n        FUNC_DECL_MISSING.test(line) ||\n        METHOD_MISSING.test(line) ||\n        ARROW_MISSING.test(line);\n    if (!isFuncLike) return false;\n    if (HAS_RETURN_TYPE.test(line)) return false;\n    return true;\n}\n\nexport class RequireReturnTypeRule extends EditRuleBase<RequireReturnTypeConfig> {\n    constructor(config: RequireReturnTypeConfig) { super(config, 'require-return-type'); }\n\n    readonly description = 'Every function and method must declare its return type.';\n    override readonly files = ['**/*.ts', '**/*.tsx'];\n    readonly fixHint = [\n        'Add return type: function foo(x: T): ReturnType { ... }',\n        '// webpieces-disable require-return-type -- <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 (!isMissingReturnType(stripped)) continue;\n            const lineNum = i + 1;\n            if (ctx.isLineDisabled(lineNum, RULE_NAMES.REQUIRE_RETURN_TYPE)) continue;\n            violations.push(new V(\n                lineNum,\n                ctx.lines[i].trim(),\n                'Missing return type annotation.',\n            ));\n        }\n        return violations;\n    }\n}\n"]}

package/src/core/rules/throw-cause-required.d.ts

@@ -1,3 +1,10 @@
-import type { EditRule } from '../types';
-declare const throwCauseRequiredRule: EditRule;
-export default throwCauseRequiredRule;
+import { ThrowCauseRequiredConfig } from '@webpieces/rules-config';
+import type { EditContext, Violation } from '../types';
+import { EditRuleBase } from '../rule-base';
+export declare class ThrowCauseRequiredRule extends EditRuleBase<ThrowCauseRequiredConfig> {
+    constructor(config: ThrowCauseRequiredConfig);
+    readonly description = "When rethrowing with added context, chain the original exception: throw new Error(\"msg\", { cause: error })";
+    readonly files: string[];
+    readonly fixHint: string[];
+    check(ctx: EditContext): readonly Violation[];
+}

package/src/core/rules/throw-cause-required.js

@@ -1,7 +1,9 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-const types_1 = require("../types");
+exports.ThrowCauseRequiredRule = void 0;
 const rules_config_1 = require("@webpieces/rules-config");
+const types_1 = require("../types");
+const rule_base_1 = require("../rule-base");
 const instruct_ai_writer_1 = require("../instruct-ai-writer");
 /**
  * Matches: throw new SomeClass(
@@ -15,18 +17,16 @@ const ERROR_MESSAGE_PATTERN = /\berror\d*\.message\b/;
  * Matches: cause: error or cause: error2 etc. (the good pattern)
  */
 const CAUSE_PATTERN = /\bcause\s*:\s*error\d*\b/;
-const throwCauseRequiredRule = {
-    name: 'throw-cause-required',
-    description: 'When rethrowing with added context, chain the original exception: throw new Error("msg", { cause: error })',
-    scope: 'edit',
-    files: ['**/*.ts', '**/*.tsx'],
-    defaultOptions: {},
-    fixHint: [
+class ThrowCauseRequiredRule extends rule_base_1.EditRuleBase {
+    constructor(config) { super(config, 'throw-cause-required'); }
+    description = 'When rethrowing with added context, chain the original exception: throw new Error("msg", { cause: error })';
+    files = ['**/*.ts', '**/*.tsx'];
+    fixHint = [
         'Option 1 — Remove the try-catch entirely. Letting the original exception bubble is usually the best option.',
         'Option 2 — throw new Error("add more info here", { cause: error }); chains the error and preserves the full stack trace.',
         'Option 3 — throw new SpecificError("add info here", { cause: error }); e.g. new InformAiError("what happened for AI", { cause: error }).',
         '[Only if disableAllowed:true] Option 4 — // webpieces-disable throw-cause-required -- <reason>',
-    ],
+    ];
     check(ctx) {
         const violations = [];
         const lines = ctx.strippedLines;
@@ -46,7 +46,7 @@ const throwCauseRequiredRule = {
         if (violations.length > 0)
             (0, instruct_ai_writer_1.writeTemplateIfMissing)(ctx.workspaceRoot, 'webpieces.exceptions.md');
         return violations;
-    },
-};
-exports.default = throwCauseRequiredRule;
+    }
+}
+exports.ThrowCauseRequiredRule = ThrowCauseRequiredRule;
 //# sourceMappingURL=throw-cause-required.js.map

package/src/core/rules/throw-cause-required.js.map

@@ -1 +1 @@
-{"version":3,"file":"throw-cause-required.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/throw-cause-required.ts"],"names":[],"mappings":";;AACA,oCAA0C;AAC1C,0DAAqD;AACrD,8DAA+D;AAE/D;;GAEG;AACH,MAAM,iBAAiB,GAAG,0BAA0B,CAAC;AAErD;;GAEG;AACH,MAAM,qBAAqB,GAAG,uBAAuB,CAAC;AAEtD;;GAEG;AACH,MAAM,aAAa,GAAG,0BAA0B,CAAC;AAEjD,MAAM,sBAAsB,GAAa;IACrC,IAAI,EAAE,sBAAsB;IAC5B,WAAW,EAAE,4GAA4G;IACzH,KAAK,EAAE,MAAM;IACb,KAAK,EAAE,CAAC,SAAS,EAAE,UAAU,CAAC;IAC9B,cAAc,EAAE,EAAE;IAClB,OAAO,EAAE;QACL,6GAA6G;QAC7G,0HAA0H;QAC1H,0IAA0I;QAC1I,gGAAgG;KACnG;IAED,KAAK,CAAC,GAAgB;QAClB,MAAM,UAAU,GAAQ,EAAE,CAAC;QAC3B,MAAM,KAAK,GAAG,GAAG,CAAC,aAAa,CAAC;QAEhC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,IAAI,CAAC,EAAE,CAAC;YACvC,MAAM,QAAQ,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;YAC1B,IAAI,CAAC,iBAAiB,CAAC,IAAI,CAAC,QAAQ,CAAC;gBAAE,SAAS;YAChD,IAAI,CAAC,qBAAqB,CAAC,IAAI,CAAC,QAAQ,CAAC;gBAAE,SAAS;YACpD,IAAI,aAAa,CAAC,IAAI,CAAC,QAAQ,CAAC;gBAAE,SAAS;YAE3C,MAAM,OAAO,GAAG,CAAC,GAAG,CAAC,CAAC;YACtB,IAAI,GAAG,CAAC,cAAc,CAAC,OAAO,EAAE,yBAAU,CAAC,oBAAoB,CAAC;gBAAE,SAAS;YAE3E,UAAU,CAAC,IAAI,CAAC,IAAI,iBAAC,CACjB,OAAO,EACP,GAAG,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,EACnB,iGAAiG,CACpG,CAAC,CAAC;QACP,CAAC;QAED,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,kBAAe,sBAAsB,CAAC","sourcesContent":["import type { EditRule, EditContext, Violation } from '../types';\nimport { Violation as V } from '../types';\nimport { RULE_NAMES } from '@webpieces/rules-config';\nimport { writeTemplateIfMissing } from '../instruct-ai-writer';\n\n/**\n * Matches: throw new SomeClass(\n */\nconst THROW_NEW_PATTERN = /\\bthrow\\s+new\\s+\\w+\\s*\\(/;\n\n/**\n * Matches: error.message or error2.message or error3.message\n */\nconst ERROR_MESSAGE_PATTERN = /\\berror\\d*\\.message\\b/;\n\n/**\n * Matches: cause: error or cause: error2 etc. (the good pattern)\n */\nconst CAUSE_PATTERN = /\\bcause\\s*:\\s*error\\d*\\b/;\n\nconst throwCauseRequiredRule: EditRule = {\n    name: 'throw-cause-required',\n    description: 'When rethrowing with added context, chain the original exception: throw new Error(\"msg\", { cause: error })',\n    scope: 'edit',\n    files: ['**/*.ts', '**/*.tsx'],\n    defaultOptions: {},\n    fixHint: [\n        'Option 1 — Remove the try-catch entirely. Letting the original exception bubble is usually the best option.',\n        'Option 2 — throw new Error(\"add more info here\", { cause: error }); chains the error and preserves the full stack trace.',\n        'Option 3 — throw new SpecificError(\"add info here\", { cause: error }); e.g. new InformAiError(\"what happened for AI\", { cause: error }).',\n        '[Only if disableAllowed:true] Option 4 — // webpieces-disable throw-cause-required -- <reason>',\n    ],\n\n    check(ctx: EditContext): readonly Violation[] {\n        const violations: V[] = [];\n        const lines = ctx.strippedLines;\n\n        for (let i = 0; i < lines.length; i += 1) {\n            const stripped = lines[i];\n            if (!THROW_NEW_PATTERN.test(stripped)) continue;\n            if (!ERROR_MESSAGE_PATTERN.test(stripped)) continue;\n            if (CAUSE_PATTERN.test(stripped)) continue;\n\n            const lineNum = i + 1;\n            if (ctx.isLineDisabled(lineNum, RULE_NAMES.THROW_CAUSE_REQUIRED)) continue;\n\n            violations.push(new V(\n                lineNum,\n                ctx.lines[i].trim(),\n                'Rethrowing with error.message loses the original stack trace. Use { cause: error } to chain it.',\n            ));\n        }\n\n        if (violations.length > 0) writeTemplateIfMissing(ctx.workspaceRoot, 'webpieces.exceptions.md');\n        return violations;\n    },\n};\n\nexport default throwCauseRequiredRule;\n"]}
+{"version":3,"file":"throw-cause-required.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/throw-cause-required.ts"],"names":[],"mappings":";;;AAAA,0DAA+E;AAG/E,oCAA0C;AAC1C,4CAA4C;AAC5C,8DAA+D;AAE/D;;GAEG;AACH,MAAM,iBAAiB,GAAG,0BAA0B,CAAC;AAErD;;GAEG;AACH,MAAM,qBAAqB,GAAG,uBAAuB,CAAC;AAEtD;;GAEG;AACH,MAAM,aAAa,GAAG,0BAA0B,CAAC;AAEjD,MAAa,sBAAuB,SAAQ,wBAAsC;IAC9E,YAAY,MAAgC,IAAI,KAAK,CAAC,MAAM,EAAE,sBAAsB,CAAC,CAAC,CAAC,CAAC;IAE/E,WAAW,GAAG,4GAA4G,CAAC;IAClH,KAAK,GAAG,CAAC,SAAS,EAAE,UAAU,CAAC,CAAC;IACzC,OAAO,GAAG;QACf,6GAA6G;QAC7G,0HAA0H;QAC1H,0IAA0I;QAC1I,gGAAgG;KACnG,CAAC;IAEF,KAAK,CAAC,GAAgB;QAClB,MAAM,UAAU,GAAQ,EAAE,CAAC;QAC3B,MAAM,KAAK,GAAG,GAAG,CAAC,aAAa,CAAC;QAEhC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,IAAI,CAAC,EAAE,CAAC;YACvC,MAAM,QAAQ,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;YAC1B,IAAI,CAAC,iBAAiB,CAAC,IAAI,CAAC,QAAQ,CAAC;gBAAE,SAAS;YAChD,IAAI,CAAC,qBAAqB,CAAC,IAAI,CAAC,QAAQ,CAAC;gBAAE,SAAS;YACpD,IAAI,aAAa,CAAC,IAAI,CAAC,QAAQ,CAAC;gBAAE,SAAS;YAE3C,MAAM,OAAO,GAAG,CAAC,GAAG,CAAC,CAAC;YACtB,IAAI,GAAG,CAAC,cAAc,CAAC,OAAO,EAAE,yBAAU,CAAC,oBAAoB,CAAC;gBAAE,SAAS;YAE3E,UAAU,CAAC,IAAI,CAAC,IAAI,iBAAC,CACjB,OAAO,EACP,GAAG,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,EACnB,iGAAiG,CACpG,CAAC,CAAC;QACP,CAAC;QAED,IAAI,UAAU,CAAC,MAAM,GAAG,CAAC;YAAE,IAAA,2CAAsB,EAAC,GAAG,CAAC,aAAa,EAAE,yBAAyB,CAAC,CAAC;QAChG,OAAO,UAAU,CAAC;IACtB,CAAC;CACJ;AAnCD,wDAmCC","sourcesContent":["import { ThrowCauseRequiredConfig, RULE_NAMES } from '@webpieces/rules-config';\n\nimport type { EditContext, Violation } from '../types';\nimport { Violation as V } from '../types';\nimport { EditRuleBase } from '../rule-base';\nimport { writeTemplateIfMissing } from '../instruct-ai-writer';\n\n/**\n * Matches: throw new SomeClass(\n */\nconst THROW_NEW_PATTERN = /\\bthrow\\s+new\\s+\\w+\\s*\\(/;\n\n/**\n * Matches: error.message or error2.message or error3.message\n */\nconst ERROR_MESSAGE_PATTERN = /\\berror\\d*\\.message\\b/;\n\n/**\n * Matches: cause: error or cause: error2 etc. (the good pattern)\n */\nconst CAUSE_PATTERN = /\\bcause\\s*:\\s*error\\d*\\b/;\n\nexport class ThrowCauseRequiredRule extends EditRuleBase<ThrowCauseRequiredConfig> {\n    constructor(config: ThrowCauseRequiredConfig) { super(config, 'throw-cause-required'); }\n\n    readonly description = 'When rethrowing with added context, chain the original exception: throw new Error(\"msg\", { cause: error })';\n    override readonly files = ['**/*.ts', '**/*.tsx'];\n    readonly fixHint = [\n        'Option 1 — Remove the try-catch entirely. Letting the original exception bubble is usually the best option.',\n        'Option 2 — throw new Error(\"add more info here\", { cause: error }); chains the error and preserves the full stack trace.',\n        'Option 3 — throw new SpecificError(\"add info here\", { cause: error }); e.g. new InformAiError(\"what happened for AI\", { cause: error }).',\n        '[Only if disableAllowed:true] Option 4 — // webpieces-disable throw-cause-required -- <reason>',\n    ];\n\n    check(ctx: EditContext): readonly Violation[] {\n        const violations: V[] = [];\n        const lines = ctx.strippedLines;\n\n        for (let i = 0; i < lines.length; i += 1) {\n            const stripped = lines[i];\n            if (!THROW_NEW_PATTERN.test(stripped)) continue;\n            if (!ERROR_MESSAGE_PATTERN.test(stripped)) continue;\n            if (CAUSE_PATTERN.test(stripped)) continue;\n\n            const lineNum = i + 1;\n            if (ctx.isLineDisabled(lineNum, RULE_NAMES.THROW_CAUSE_REQUIRED)) continue;\n\n            violations.push(new V(\n                lineNum,\n                ctx.lines[i].trim(),\n                'Rethrowing with error.message loses the original stack trace. Use { cause: error } to chain it.',\n            ));\n        }\n\n        if (violations.length > 0) writeTemplateIfMissing(ctx.workspaceRoot, 'webpieces.exceptions.md');\n        return violations;\n    }\n}\n"]}

package/src/core/rules/validate-ts-in-src.d.ts

@@ -1,3 +1,14 @@
-import type { FileRule } from '../types';
-declare const validateTsInSrcRule: FileRule;
-export default validateTsInSrcRule;
+import { ValidateTsInSrcConfig } from '@webpieces/rules-config';
+import type { FileContext, Violation } from '../types';
+import { FileRuleBase } from '../rule-base';
+export declare class ValidateTsInSrcRule extends FileRuleBase<ValidateTsInSrcConfig> {
+    constructor(config: ValidateTsInSrcConfig);
+    readonly description = "Every .ts file must belong to a project's src/ directory.";
+    readonly files: string[];
+    readonly defaultOptions: {
+        excludePaths: string[];
+        allowedRootFiles: string[];
+    };
+    readonly fixHint: string[];
+    check(ctx: FileContext): readonly Violation[];
+}

package/src/core/rules/validate-ts-in-src.js

@@ -1,10 +1,12 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.ValidateTsInSrcRule = void 0;
 const tslib_1 = require("tslib");
 const fs = tslib_1.__importStar(require("fs"));
 const path = tslib_1.__importStar(require("path"));
 const rules_config_1 = require("@webpieces/rules-config");
 const types_1 = require("../types");
+const rule_base_1 = require("../rule-base");
 const DEFAULT_EXCLUDE_PATHS = [
     'node_modules', 'dist', '.nx', '.git',
     '**/*.d.ts', '**/jest.config.ts',
@@ -19,28 +21,23 @@ function findProjectRoot(filePath, workspaceRoot) {
     }
     return null;
 }
-const validateTsInSrcRule = {
-    name: 'validate-ts-in-src',
-    description: 'Every .ts file must belong to a project\'s src/ directory.',
-    scope: 'file',
-    files: ['**/*.ts', '**/*.tsx'],
-    defaultOptions: {
+class ValidateTsInSrcRule extends rule_base_1.FileRuleBase {
+    constructor(config) { super(config, 'validate-ts-in-src'); }
+    description = 'Every .ts file must belong to a project\'s src/ directory.';
+    files = ['**/*.ts', '**/*.tsx'];
+    defaultOptions = {
         excludePaths: DEFAULT_EXCLUDE_PATHS,
         allowedRootFiles: DEFAULT_ALLOWED_ROOT_FILES,
-    },
-    fixHint: [
+    };
+    fixHint = [
         'Move the file into an existing project\'s src/ directory, or create a new project with project.json that owns the directory.',
         'Add a dir or glob (e.g. "**/codegen.ts") to validate-ts-in-src.excludePaths in webpieces.config.json',
-    ],
+    ];
     check(ctx) {
         if (ctx.tool !== 'Write')
             return [];
-        const excludePaths = Array.isArray(ctx.options['excludePaths'])
-            ? ctx.options['excludePaths']
-            : DEFAULT_EXCLUDE_PATHS;
-        const allowedRootFiles = Array.isArray(ctx.options['allowedRootFiles'])
-            ? ctx.options['allowedRootFiles']
-            : DEFAULT_ALLOWED_ROOT_FILES;
+        const excludePaths = this.config.excludePaths ?? DEFAULT_EXCLUDE_PATHS;
+        const allowedRootFiles = this.config.allowedRootFiles ?? DEFAULT_ALLOWED_ROOT_FILES;
         // Holistic exclusion (Layer 1 + Layer 2): bare dir names + globs.
         if ((0, rules_config_1.isPathExcluded)(ctx.relativePath, excludePaths))
             return [];
@@ -57,7 +54,7 @@ const validateTsInSrcRule = {
             return [new types_1.Violation(1, ctx.relativePath, `File is inside project \`${projectName}\` but outside its src/ directory. Move it into src/.`)];
         }
         return [];
-    },
-};
-exports.default = validateTsInSrcRule;
+    }
+}
+exports.ValidateTsInSrcRule = ValidateTsInSrcRule;
 //# sourceMappingURL=validate-ts-in-src.js.map

package/src/core/rules/validate-ts-in-src.js.map

@@ -1 +1 @@
-{"version":3,"file":"validate-ts-in-src.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/validate-ts-in-src.ts"],"names":[],"mappings":";;;AAAA,+CAAyB;AACzB,mDAA6B;AAE7B,0DAAyD;AAGzD,oCAA0C;AAE1C,MAAM,qBAAqB,GAAG;IAC1B,cAAc,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM;IACrC,WAAW,EAAE,mBAAmB;CACnC,CAAC;AACF,MAAM,0BAA0B,GAAG,CAAC,eAAe,CAAC,CAAC;AAErD,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,mBAAmB,GAAa;IAClC,IAAI,EAAE,oBAAoB;IAC1B,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,sGAAsG;KACzG;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,kEAAkE;QAClE,IAAI,IAAA,6BAAc,EAAC,GAAG,CAAC,YAAY,EAAE,YAAY,CAAC;YAAE,OAAO,EAAE,CAAC;QAE9D,MAAM,QAAQ,GAAG,GAAG,CAAC,YAAY,CAAC,KAAK,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAClD,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC,IAAI,gBAAgB,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,IAAI,CAAC;YAAE,OAAO,EAAE,CAAC;QAEzF,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,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,mBAAmB,CAAC","sourcesContent":["import * as fs from 'fs';\nimport * as path from 'path';\n\nimport { isPathExcluded } from '@webpieces/rules-config';\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    '**/*.d.ts', '**/jest.config.ts',\n];\nconst DEFAULT_ALLOWED_ROOT_FILES = ['jest.setup.ts'];\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 validateTsInSrcRule: FileRule = {\n    name: 'validate-ts-in-src',\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 a dir or glob (e.g. \"**/codegen.ts\") to validate-ts-in-src.excludePaths in webpieces.config.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        // Holistic exclusion (Layer 1 + Layer 2): bare dir names + globs.\n        if (isPathExcluded(ctx.relativePath, excludePaths)) return [];\n\n        const relParts = ctx.relativePath.split(path.sep);\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        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 validateTsInSrcRule;\n"]}
+{"version":3,"file":"validate-ts-in-src.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/validate-ts-in-src.ts"],"names":[],"mappings":";;;;AAAA,+CAAyB;AACzB,mDAA6B;AAE7B,0DAAgF;AAGhF,oCAA0C;AAC1C,4CAA4C;AAE5C,MAAM,qBAAqB,GAAG;IAC1B,cAAc,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM;IACrC,WAAW,EAAE,mBAAmB;CACnC,CAAC;AACF,MAAM,0BAA0B,GAAG,CAAC,eAAe,CAAC,CAAC;AAErD,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,MAAa,mBAAoB,SAAQ,wBAAmC;IACxE,YAAY,MAA6B,IAAI,KAAK,CAAC,MAAM,EAAE,oBAAoB,CAAC,CAAC,CAAC,CAAC;IAE1E,WAAW,GAAG,4DAA4D,CAAC;IAClE,KAAK,GAAG,CAAC,SAAS,EAAE,UAAU,CAAC,CAAC;IAChC,cAAc,GAAG;QAC/B,YAAY,EAAE,qBAAqB;QACnC,gBAAgB,EAAE,0BAA0B;KAC/C,CAAC;IACO,OAAO,GAAG;QACf,8HAA8H;QAC9H,sGAAsG;KACzG,CAAC;IAEF,KAAK,CAAC,GAAgB;QAClB,IAAI,GAAG,CAAC,IAAI,KAAK,OAAO;YAAE,OAAO,EAAE,CAAC;QAEpC,MAAM,YAAY,GAAG,IAAI,CAAC,MAAM,CAAC,YAAY,IAAI,qBAAqB,CAAC;QACvE,MAAM,gBAAgB,GAAG,IAAI,CAAC,MAAM,CAAC,gBAAgB,IAAI,0BAA0B,CAAC;QAEpF,kEAAkE;QAClE,IAAI,IAAA,6BAAc,EAAC,GAAG,CAAC,YAAY,EAAE,YAAY,CAAC;YAAE,OAAO,EAAE,CAAC;QAE9D,MAAM,QAAQ,GAAG,GAAG,CAAC,YAAY,CAAC,KAAK,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAClD,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC,IAAI,gBAAgB,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,IAAI,CAAC;YAAE,OAAO,EAAE,CAAC;QAEzF,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,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;AAhDD,kDAgDC","sourcesContent":["import * as fs from 'fs';\nimport * as path from 'path';\n\nimport { ValidateTsInSrcConfig, isPathExcluded } from '@webpieces/rules-config';\n\nimport type { FileContext, Violation } from '../types';\nimport { Violation as V } from '../types';\nimport { FileRuleBase } from '../rule-base';\n\nconst DEFAULT_EXCLUDE_PATHS = [\n    'node_modules', 'dist', '.nx', '.git',\n    '**/*.d.ts', '**/jest.config.ts',\n];\nconst DEFAULT_ALLOWED_ROOT_FILES = ['jest.setup.ts'];\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\nexport class ValidateTsInSrcRule extends FileRuleBase<ValidateTsInSrcConfig> {\n    constructor(config: ValidateTsInSrcConfig) { super(config, 'validate-ts-in-src'); }\n\n    readonly description = 'Every .ts file must belong to a project\\'s src/ directory.';\n    override readonly files = ['**/*.ts', '**/*.tsx'];\n    override readonly defaultOptions = {\n        excludePaths: DEFAULT_EXCLUDE_PATHS,\n        allowedRootFiles: DEFAULT_ALLOWED_ROOT_FILES,\n    };\n    readonly 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 a dir or glob (e.g. \"**/codegen.ts\") to validate-ts-in-src.excludePaths in webpieces.config.json',\n    ];\n\n    check(ctx: FileContext): readonly Violation[] {\n        if (ctx.tool !== 'Write') return [];\n\n        const excludePaths = this.config.excludePaths ?? DEFAULT_EXCLUDE_PATHS;\n        const allowedRootFiles = this.config.allowedRootFiles ?? DEFAULT_ALLOWED_ROOT_FILES;\n\n        // Holistic exclusion (Layer 1 + Layer 2): bare dir names + globs.\n        if (isPathExcluded(ctx.relativePath, excludePaths)) return [];\n\n        const relParts = ctx.relativePath.split(path.sep);\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        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"]}

package/src/core/runner.js

@@ -6,36 +6,35 @@ const tslib_1 = require("tslib");
 const path = tslib_1.__importStar(require("path"));
 const rules_config_1 = require("@webpieces/rules-config");
 const build_context_1 = require("./build-context");
-const load_config_1 = require("./load-config");
 const load_rules_1 = require("./load-rules");
 const to_error_1 = require("./to-error");
 const report_1 = require("./report");
 const types_1 = require("./types");
+const CONFIG_MISSING_REPORT = 'webpieces.config.json not found.\n' +
+    'Tell the human: run `./node_modules/.bin/wp-setup-ai-hooks` to initialize the project configuration.\n' +
+    'Do not proceed until the human has done this.';
 function run(toolKind, input, cwd) {
     return runInternal(toolKind, input, cwd);
 }
 function runInternal(toolKind, input, cwd) {
-    const config = (0, load_config_1.loadConfig)(cwd);
-    if (!config.configPath) {
-        return new types_1.BlockedResult('webpieces.config.json not found.\n' +
-            'Tell the human: run `./node_modules/.bin/wp-setup-ai-hooks` to initialize the project configuration.\n' +
-            'Do not proceed until the human has done this.');
-    }
-    const workspaceRoot = path.dirname(config.configPath);
+    const loaded = (0, rules_config_1.loadWebpiecesRulesConfig)(cwd);
+    if (!loaded)
+        return new types_1.BlockedResult(CONFIG_MISSING_REPORT);
+    const workspaceRoot = path.dirname(loaded.configPath);
     // Always allow edits to webpieces.config.json — it's the fix target when out of sync
-    if (path.resolve(input.filePath) === path.resolve(config.configPath)) {
+    if (path.resolve(input.filePath) === path.resolve(loaded.configPath)) {
         return null;
     }
-    const rules = (0, load_rules_1.loadRules)(config, workspaceRoot);
+    const rules = (0, load_rules_1.loadRules)(loaded.config, workspaceRoot);
     if (rules.length === 0)
         return null;
-    const outOfSync = checkConfigSync(rules, config);
+    const outOfSync = checkConfigSync(rules, loaded.config);
     if (outOfSync)
         return outOfSync;
     const contexts = (0, build_context_1.buildContexts)(toolKind, input, workspaceRoot);
     const relativePath = path.relative(workspaceRoot, input.filePath);
-    const editGroups = runEditRules(rules, contexts.editContexts, config);
-    const fileGroups = runFileRules(rules, contexts.fileContext, config);
+    const editGroups = runEditRules(rules, contexts.editContexts);
+    const fileGroups = runFileRules(rules, contexts.fileContext);
     const allGroups = [...editGroups, ...fileGroups];
     if (allGroups.length === 0)
         return null;
@@ -46,28 +45,30 @@ function runBash(command, cwd) {
     return runBashInternal(command, cwd);
 }
 function runBashInternal(command, cwd) {
-    const config = (0, load_config_1.loadConfig)(cwd);
-    if (!config.configPath) {
-        return new types_1.BlockedResult('webpieces.config.json not found.\n' +
-            'Tell the human: run `./node_modules/.bin/wp-setup-ai-hooks` to initialize the project configuration.\n' +
-            'Do not proceed until the human has done this.');
-    }
-    const workspaceRoot = path.dirname(config.configPath);
-    const rules = (0, load_rules_1.loadRules)(config, workspaceRoot);
+    const loaded = (0, rules_config_1.loadWebpiecesRulesConfig)(cwd);
+    if (!loaded)
+        return new types_1.BlockedResult(CONFIG_MISSING_REPORT);
+    const workspaceRoot = path.dirname(loaded.configPath);
+    const rules = (0, load_rules_1.loadRules)(loaded.config, workspaceRoot);
     if (rules.length === 0)
         return null;
-    const outOfSync = checkConfigSync(rules, config);
+    const outOfSync = checkConfigSync(rules, loaded.config);
     if (outOfSync)
         return outOfSync;
     const ctx = (0, build_context_1.buildBashContext)(command, workspaceRoot);
-    const groups = runBashRules(rules, ctx, config);
+    const groups = runBashRules(rules, ctx);
     if (groups.length === 0)
         return null;
     const report = (0, report_1.formatReport)('<bash>', groups);
     return new types_1.BlockedResult(report);
 }
+// The set of rule names explicitly present in webpieces.config.json (every key except rulesDir).
+function configuredRuleNames(config) {
+    return new Set(Object.keys(config).filter((k) => k !== 'rulesDir'));
+}
 function checkConfigSync(rules, config) {
-    const unconfiguredRules = rules.filter((r) => !config.userConfiguredRuleNames.has(r.name));
+    const configured = configuredRuleNames(config);
+    const unconfiguredRules = rules.filter((r) => !configured.has(r.name));
     if (unconfiguredRules.length === 0)
         return null;
     const lines = [
@@ -112,17 +113,20 @@ function runRuleCheck(rule, ctx) {
         return [new types_1.Violation(0, '', `Rule '${rule.name}' crashed: ${error.message}`)];
     }
 }
-function runBashRules(rules, bashContext, config) {
+function ruleMatchesFile(rule, relativePath) {
+    for (const pattern of rule.files) {
+        if ((0, load_rules_1.globMatches)(pattern, relativePath))
+            return true;
+    }
+    return false;
+}
+function runBashRules(rules, bashContext) {
     const groups = [];
     for (const rule of rules) {
         if (rule.scope !== 'bash')
             continue;
-        const ruleConfig = config.rules.get(rule.name);
-        if (!ruleConfig || ruleConfig.isOff)
-            continue;
-        if (isRuleSkipped(ruleConfig))
+        if (!rule.shouldRun())
             continue;
-        bashContext.options = mergeOptions(rule.defaultOptions, ruleConfig);
         const vs = runRuleCheck(rule, bashContext);
         if (vs.length > 0) {
             groups.push(new types_1.RuleGroup(rule.name, rule.description, [...rule.fixHint], [...vs]));
@@ -130,46 +134,17 @@ function runBashRules(rules, bashContext, config) {
     }
     return groups;
 }
-// Every rule honors the universal escape hatches: skip while on a named branch
-// (ignoreRuleWhileOnBranch) or until an epoch passes (ignoreModifiedUntilEpoch).
-function isRuleSkipped(ruleConfig) {
-    return (0, rules_config_1.shouldSkipRule)(ruleConfig.options['ignoreModifiedUntilEpoch'], ruleConfig.options['ignoreRuleWhileOnBranch']).skip;
-}
-function ruleMatchesFile(rule, relativePath) {
-    for (const pattern of rule.files) {
-        if ((0, load_rules_1.globMatches)(pattern, relativePath))
-            return true;
-    }
-    return false;
-}
-function mergeOptions(defaultOptions, ruleConfig) {
-    // webpieces-disable no-any-unknown -- building an options bag from opaque RuleOptions
-    const out = {};
-    for (const key of Object.keys(defaultOptions))
-        out[key] = defaultOptions[key];
-    for (const key of Object.keys(ruleConfig.options)) {
-        // 'mode' is the framework-level on/off switch, not a rule option.
-        if (key === 'mode')
-            continue;
-        out[key] = ruleConfig.options[key];
-    }
-    return out;
-}
-function runEditRules(rules, editContexts, config) {
+function runEditRules(rules, editContexts) {
     const groups = [];
     for (const rule of rules) {
         if (rule.scope !== 'edit')
             continue;
-        const ruleConfig = config.rules.get(rule.name);
-        if (!ruleConfig || ruleConfig.isOff)
-            continue;
-        if (isRuleSkipped(ruleConfig))
+        if (!rule.shouldRun())
             continue;
         const allViolations = [];
         for (const ctx of editContexts) {
             if (!ruleMatchesFile(rule, ctx.relativePath))
                 continue;
-            ctx.options = mergeOptions(rule.defaultOptions, ruleConfig);
             const vs = runRuleCheck(rule, ctx);
             for (const v of vs) {
                 const copy = new types_1.Violation(v.line, v.snippet, v.message);
@@ -184,19 +159,15 @@ function runEditRules(rules, editContexts, config) {
     }
     return groups;
 }
-function runFileRules(rules, fileContext, config) {
+function runFileRules(rules, fileContext) {
     const groups = [];
     for (const rule of rules) {
         if (rule.scope !== 'file')
             continue;
-        const ruleConfig = config.rules.get(rule.name);
-        if (!ruleConfig || ruleConfig.isOff)
-            continue;
-        if (isRuleSkipped(ruleConfig))
+        if (!rule.shouldRun())
             continue;
         if (!ruleMatchesFile(rule, fileContext.relativePath))
             continue;
-        fileContext.options = mergeOptions(rule.defaultOptions, ruleConfig);
         const vs = runRuleCheck(rule, fileContext);
         if (vs.length > 0) {
             groups.push(new types_1.RuleGroup(rule.name, rule.description, [...rule.fixHint], [...vs]));