@webpieces/ai-hook-rules 0.3.184 → 0.3.186

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

@@ -1,3 +1,13 @@
-import type { EditRule } from '../types';
-declare const noDestructureRule: EditRule;
-export default noDestructureRule;
+import { NoDestructureConfig } from '@webpieces/rules-config';
+import type { EditContext, Violation } from '../types';
+import { EditRuleBase } from '../rule-base';
+export declare class NoDestructureRule extends EditRuleBase<NoDestructureConfig> {
+    constructor(config: NoDestructureConfig);
+    readonly description = "Disallow destructuring patterns. Assign the whole result and pass it around or access properties explicitly.";
+    readonly files: string[];
+    readonly defaultOptions: {
+        allowTopLevel: boolean;
+    };
+    readonly fixHint: string[];
+    check(ctx: EditContext): readonly Violation[];
+}

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

@@ -1,17 +1,19 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.NoDestructureRule = void 0;
+const rules_config_1 = require("@webpieces/rules-config");
 const types_1 = require("../types");
+const rule_base_1 = require("../rule-base");
 const VARIABLE_DESTRUCTURE = /\b(?:const|let|var)\s*\{/;
-const noDestructureRule = {
-    name: 'no-destructure',
-    description: 'Disallow destructuring patterns. Assign the whole result and pass it around or access properties explicitly.',
-    scope: 'edit',
-    files: ['**/*.ts', '**/*.tsx'],
-    defaultOptions: { allowTopLevel: true },
-    fixHint: [
+class NoDestructureRule extends rule_base_1.EditRuleBase {
+    constructor(config) { super(config, 'no-destructure'); }
+    description = 'Disallow destructuring patterns. Assign the whole result and pass it around or access properties explicitly.';
+    files = ['**/*.ts', '**/*.tsx'];
+    defaultOptions = { allowTopLevel: true };
+    fixHint = [
         '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) {
         const violations = [];
         for (let i = 0; i < ctx.strippedLines.length; i += 1) {
@@ -19,12 +21,12 @@ const noDestructureRule = {
             if (!VARIABLE_DESTRUCTURE.test(stripped))
                 continue;
             const lineNum = i + 1;
-            if (ctx.isLineDisabled(lineNum, 'no-destructure'))
+            if (ctx.isLineDisabled(lineNum, rules_config_1.RULE_NAMES.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;
-    },
-};
-exports.default = noDestructureRule;
+    }
+}
+exports.NoDestructureRule = NoDestructureRule;
 //# sourceMappingURL=no-destructure.js.map

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

@@ -1 +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"]}
+{"version":3,"file":"no-destructure.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/no-destructure.ts"],"names":[],"mappings":";;;AAAA,0DAA0E;AAG1E,oCAA0C;AAC1C,4CAA4C;AAE5C,MAAM,oBAAoB,GAAG,0BAA0B,CAAC;AAExD,MAAa,iBAAkB,SAAQ,wBAAiC;IACpE,YAAY,MAA2B,IAAI,KAAK,CAAC,MAAM,EAAE,gBAAgB,CAAC,CAAC,CAAC,CAAC;IAEpE,WAAW,GAAG,8GAA8G,CAAC;IACpH,KAAK,GAAG,CAAC,SAAS,EAAE,UAAU,CAAC,CAAC;IAChC,cAAc,GAAG,EAAE,aAAa,EAAE,IAAI,EAAE,CAAC;IAClD,OAAO,GAAG;QACf,yHAAyH;QACzH,iDAAiD;KACpD,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,oBAAoB,CAAC,IAAI,CAAC,QAAQ,CAAC;gBAAE,SAAS;YACnD,MAAM,OAAO,GAAG,CAAC,GAAG,CAAC,CAAC;YACtB,IAAI,GAAG,CAAC,cAAc,CAAC,OAAO,EAAE,yBAAU,CAAC,cAAc,CAAC;gBAAE,SAAS;YACrE,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;AA1BD,8CA0BC","sourcesContent":["import { NoDestructureConfig, 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\nconst VARIABLE_DESTRUCTURE = /\\b(?:const|let|var)\\s*\\{/;\n\nexport class NoDestructureRule extends EditRuleBase<NoDestructureConfig> {\n    constructor(config: NoDestructureConfig) { super(config, 'no-destructure'); }\n\n    readonly description = 'Disallow destructuring patterns. Assign the whole result and pass it around or access properties explicitly.';\n    override readonly files = ['**/*.ts', '**/*.tsx'];\n    override readonly defaultOptions = { allowTopLevel: true };\n    readonly 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, RULE_NAMES.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"]}

package/src/core/rules/no-direct-main-update.d.ts

@@ -1,3 +1,9 @@
-import type { BashRule } from '../types';
-declare const noDirectMainUpdate: BashRule;
-export default noDirectMainUpdate;
+import { NoDirectMainUpdateConfig } from '@webpieces/rules-config';
+import type { BashContext, Violation } from '../types';
+import { BashRuleBase } from '../rule-base';
+export declare class NoDirectMainUpdateRule extends BashRuleBase<NoDirectMainUpdateConfig> {
+    constructor(config: NoDirectMainUpdateConfig);
+    readonly description = "Block direct git merge/rebase/pull from main on feature branches. Use the squash-update process instead.";
+    readonly fixHint: readonly string[];
+    check(ctx: BashContext): readonly Violation[];
+}

package/src/core/rules/no-direct-main-update.js

@@ -1,7 +1,9 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.NoDirectMainUpdateRule = void 0;
 const child_process_1 = require("child_process");
 const types_1 = require("../types");
+const rule_base_1 = require("../rule-base");
 const FIX_HINT = [
     "Run 'pnpm wp-git-update' to squash-update from main. This preserves the 3-point fork-point system (fork-point=A, feature-HEAD=B, main-HEAD=C) needed for clean PR diffs. See docs/git-workflow.md for details.",
 ];
@@ -10,13 +12,14 @@ const WRONG_UPDATE_PATTERNS = [
     /git\s+rebase\s+(origin\/main|main)\b/,
     /git\s+pull\s+origin\s+main\b/,
 ];
-const noDirectMainUpdate = {
-    name: 'no-direct-main-update',
-    description: 'Block direct git merge/rebase/pull from main on feature branches. Use the squash-update process instead.',
-    scope: 'bash',
-    files: [],
-    defaultOptions: {},
-    fixHint: FIX_HINT,
+function truncate(s) {
+    const MAX = 120;
+    return s.length <= MAX ? s : s.slice(0, MAX) + '…';
+}
+class NoDirectMainUpdateRule extends rule_base_1.BashRuleBase {
+    constructor(config) { super(config, 'no-direct-main-update'); }
+    description = 'Block direct git merge/rebase/pull from main on feature branches. Use the squash-update process instead.';
+    fixHint = FIX_HINT;
     check(ctx) {
         const matched = WRONG_UPDATE_PATTERNS.some((p) => p.test(ctx.command));
         if (!matched)
@@ -39,11 +42,7 @@ const noDirectMainUpdate = {
                 '  pnpm wp-git-update',
                 'See docs/git-workflow.md for details.',
             ].join('\n'))];
-    },
-};
-function truncate(s) {
-    const MAX = 120;
-    return s.length <= MAX ? s : s.slice(0, MAX) + '…';
+    }
 }
-exports.default = noDirectMainUpdate;
+exports.NoDirectMainUpdateRule = NoDirectMainUpdateRule;
 //# sourceMappingURL=no-direct-main-update.js.map

package/src/core/rules/no-direct-main-update.js.map

@@ -1 +1 @@
-{"version":3,"file":"no-direct-main-update.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/no-direct-main-update.ts"],"names":[],"mappings":";;AAAA,iDAAyC;AAEzC,oCAA0C;AAE1C,MAAM,QAAQ,GAAsB;IAChC,gNAAgN;CACnN,CAAC;AAEF,MAAM,qBAAqB,GAAa;IACpC,qCAAqC;IACrC,sCAAsC;IACtC,8BAA8B;CACjC,CAAC;AAEF,MAAM,kBAAkB,GAAa;IACjC,IAAI,EAAE,uBAAuB;IAC7B,WAAW,EAAE,0GAA0G;IACvH,KAAK,EAAE,MAAM;IACb,KAAK,EAAE,EAAE;IACT,cAAc,EAAE,EAAE;IAClB,OAAO,EAAE,QAAQ;IAEjB,KAAK,CAAC,GAAgB;QAClB,MAAM,OAAO,GAAG,qBAAqB,CAAC,IAAI,CAAC,CAAC,CAAS,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC;QAC/E,IAAI,CAAC,OAAO;YAAE,OAAO,EAAE,CAAC;QAExB,0GAA0G;QAC1G,IAAI,oCAAoC,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,EAAE,CAAC;YACzD,OAAO,EAAE,CAAC;QACd,CAAC;QAED,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,IAAI,aAAa,KAAK,MAAM,EAAE,CAAC;YAC3B,OAAO,EAAE,CAAC;QACd,CAAC;QAED,OAAO,CAAC,IAAI,iBAAC,CACT,CAAC,EACD,QAAQ,CAAC,GAAG,CAAC,OAAO,CAAC,EACrB;gBACI,4CAA4C,aAAa,eAAe;gBACxE,4CAA4C;gBAC5C,wCAAwC;gBACxC,sBAAsB;gBACtB,uCAAuC;aAC1C,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,kBAAkB,CAAC","sourcesContent":["import { execSync } from 'child_process';\nimport type { BashRule, BashContext, Violation } from '../types';\nimport { Violation as V } from '../types';\n\nconst FIX_HINT: readonly string[] = [\n    \"Run 'pnpm wp-git-update' to squash-update from main. This preserves the 3-point fork-point system (fork-point=A, feature-HEAD=B, main-HEAD=C) needed for clean PR diffs. See docs/git-workflow.md for details.\",\n];\n\nconst WRONG_UPDATE_PATTERNS: RegExp[] = [\n    /git\\s+merge\\s+(origin\\/main|main)\\b/,\n    /git\\s+rebase\\s+(origin\\/main|main)\\b/,\n    /git\\s+pull\\s+origin\\s+main\\b/,\n];\n\nconst noDirectMainUpdate: BashRule = {\n    name: 'no-direct-main-update',\n    description: 'Block direct git merge/rebase/pull from main on feature branches. Use the squash-update process instead.',\n    scope: 'bash',\n    files: [],\n    defaultOptions: {},\n    fixHint: FIX_HINT,\n\n    check(ctx: BashContext): readonly Violation[] {\n        const matched = WRONG_UPDATE_PATTERNS.some((p: RegExp) => p.test(ctx.command));\n        if (!matched) return [];\n\n        // Allow 'git checkout main && git pull origin main' — switching to main first is the recommended workflow\n        if (/git\\s+(?:checkout|switch)\\s+main\\b/.test(ctx.command)) {\n            return [];\n        }\n\n        const currentBranch = execSync('git rev-parse --abbrev-ref HEAD', {\n            cwd: ctx.workspaceRoot,\n            encoding: 'utf8',\n        }).trim();\n\n        if (currentBranch === 'main') {\n            return [];\n        }\n\n        return [new V(\n            1,\n            truncate(ctx.command),\n            [\n                `Direct merge/rebase from main on branch '${currentBranch}' is blocked.`,\n                'This breaks the 3-point fork-point system.',\n                'Use the squash-update process instead:',\n                '  pnpm wp-git-update',\n                'See docs/git-workflow.md for details.',\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 noDirectMainUpdate;\n"]}
+{"version":3,"file":"no-direct-main-update.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/no-direct-main-update.ts"],"names":[],"mappings":";;;AAAA,iDAAyC;AAKzC,oCAA0C;AAC1C,4CAA4C;AAE5C,MAAM,QAAQ,GAAsB;IAChC,gNAAgN;CACnN,CAAC;AAEF,MAAM,qBAAqB,GAAa;IACpC,qCAAqC;IACrC,sCAAsC;IACtC,8BAA8B;CACjC,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,MAAa,sBAAuB,SAAQ,wBAAsC;IAC9E,YAAY,MAAgC,IAAI,KAAK,CAAC,MAAM,EAAE,uBAAuB,CAAC,CAAC,CAAC,CAAC;IAEhF,WAAW,GAAG,0GAA0G,CAAC;IACzH,OAAO,GAAG,QAAQ,CAAC;IAE5B,KAAK,CAAC,GAAgB;QAClB,MAAM,OAAO,GAAG,qBAAqB,CAAC,IAAI,CAAC,CAAC,CAAS,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC;QAC/E,IAAI,CAAC,OAAO;YAAE,OAAO,EAAE,CAAC;QAExB,0GAA0G;QAC1G,IAAI,oCAAoC,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,EAAE,CAAC;YACzD,OAAO,EAAE,CAAC;QACd,CAAC;QAED,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,IAAI,aAAa,KAAK,MAAM,EAAE,CAAC;YAC3B,OAAO,EAAE,CAAC;QACd,CAAC;QAED,OAAO,CAAC,IAAI,iBAAC,CACT,CAAC,EACD,QAAQ,CAAC,GAAG,CAAC,OAAO,CAAC,EACrB;gBACI,4CAA4C,aAAa,eAAe;gBACxE,4CAA4C;gBAC5C,wCAAwC;gBACxC,sBAAsB;gBACtB,uCAAuC;aAC1C,CAAC,IAAI,CAAC,IAAI,CAAC,CACf,CAAC,CAAC;IACP,CAAC;CACJ;AApCD,wDAoCC","sourcesContent":["import { execSync } from 'child_process';\n\nimport { NoDirectMainUpdateConfig } from '@webpieces/rules-config';\n\nimport type { BashContext, Violation } from '../types';\nimport { Violation as V } from '../types';\nimport { BashRuleBase } from '../rule-base';\n\nconst FIX_HINT: readonly string[] = [\n    \"Run 'pnpm wp-git-update' to squash-update from main. This preserves the 3-point fork-point system (fork-point=A, feature-HEAD=B, main-HEAD=C) needed for clean PR diffs. See docs/git-workflow.md for details.\",\n];\n\nconst WRONG_UPDATE_PATTERNS: RegExp[] = [\n    /git\\s+merge\\s+(origin\\/main|main)\\b/,\n    /git\\s+rebase\\s+(origin\\/main|main)\\b/,\n    /git\\s+pull\\s+origin\\s+main\\b/,\n];\n\nfunction truncate(s: string): string {\n    const MAX = 120;\n    return s.length <= MAX ? s : s.slice(0, MAX) + '…';\n}\n\nexport class NoDirectMainUpdateRule extends BashRuleBase<NoDirectMainUpdateConfig> {\n    constructor(config: NoDirectMainUpdateConfig) { super(config, 'no-direct-main-update'); }\n\n    readonly description = 'Block direct git merge/rebase/pull from main on feature branches. Use the squash-update process instead.';\n    readonly fixHint = FIX_HINT;\n\n    check(ctx: BashContext): readonly Violation[] {\n        const matched = WRONG_UPDATE_PATTERNS.some((p: RegExp) => p.test(ctx.command));\n        if (!matched) return [];\n\n        // Allow 'git checkout main && git pull origin main' — switching to main first is the recommended workflow\n        if (/git\\s+(?:checkout|switch)\\s+main\\b/.test(ctx.command)) {\n            return [];\n        }\n\n        const currentBranch = execSync('git rev-parse --abbrev-ref HEAD', {\n            cwd: ctx.workspaceRoot,\n            encoding: 'utf8',\n        }).trim();\n\n        if (currentBranch === 'main') {\n            return [];\n        }\n\n        return [new V(\n            1,\n            truncate(ctx.command),\n            [\n                `Direct merge/rebase from main on branch '${currentBranch}' is blocked.`,\n                'This breaks the 3-point fork-point system.',\n                'Use the squash-update process instead:',\n                '  pnpm wp-git-update',\n                'See docs/git-workflow.md for details.',\n            ].join('\\n'),\n        )];\n    }\n}\n"]}

package/src/core/rules/no-edit-on-main.d.ts

@@ -1,3 +1,13 @@
-import type { FileRule } from '../types';
-declare const noEditOnMain: FileRule;
-export default noEditOnMain;
+import { NoEditOnMainConfig } from '@webpieces/rules-config';
+import type { FileContext, Violation } from '../types';
+import { FileRuleBase } from '../rule-base';
+export declare class NoEditOnMainRule extends FileRuleBase<NoEditOnMainConfig> {
+    constructor(config: NoEditOnMainConfig);
+    readonly description = "Block file edits when on the main branch \u2014 all work must happen on a feature branch.";
+    readonly files: string[];
+    readonly defaultOptions: {
+        branchNamingConvention: string;
+    };
+    readonly fixHint: string[];
+    check(ctx: FileContext): readonly Violation[];
+}

package/src/core/rules/no-edit-on-main.js

@@ -1,23 +1,24 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.NoEditOnMainRule = void 0;
 const child_process_1 = require("child_process");
 const types_1 = require("../types");
+const rule_base_1 = require("../rule-base");
 const to_error_1 = require("../to-error");
-const noEditOnMain = {
-    name: 'no-edit-on-main',
-    description: 'Block file edits when on the main branch — all work must happen on a feature branch.',
-    scope: 'file',
-    files: ['**/*'],
-    defaultOptions: {
+class NoEditOnMainRule extends rule_base_1.FileRuleBase {
+    constructor(config) { super(config, 'no-edit-on-main'); }
+    description = 'Block file edits when on the main branch — all work must happen on a feature branch.';
+    files = ['**/*'];
+    defaultOptions = {
         branchNamingConvention: '{whoami}/{featurename}',
-    },
-    fixHint: [
+    };
+    fixHint = [
         'You should not be working on main.',
         'Steps:',
         '  1. git pull origin main   ← get latest commits',
         '  2. git checkout -b <branch-name>   ← create feature branch',
         'Branch naming convention is defined in webpieces.config.json under no-edit-on-main.branchNamingConvention.',
-    ],
+    ];
     check(ctx) {
         // Only apply to files inside the workspace root
         if (ctx.relativePath.startsWith('..'))
@@ -38,15 +39,14 @@ const noEditOnMain = {
         }
         if (currentBranch !== 'main')
             return [];
-        const convention = ctx.options['branchNamingConvention']
-            ?? 'feature/<ticket-id>-<short-description>';
+        const convention = this.config.branchNamingConvention ?? '{whoami}/{featurename}';
         return [new types_1.Violation(1, ctx.relativePath, [
                 'You should not be working on main.',
                 'Do a `git pull origin main` to get latest, then create a feature branch based on the naming convention.',
                 `Branch naming convention (from webpieces.config.json): ${convention}`,
                 'Example: git checkout -b ' + convention.replace(/<[^>]+>/g, 'value'),
             ].join('\n'))];
-    },
-};
-exports.default = noEditOnMain;
+    }
+}
+exports.NoEditOnMainRule = NoEditOnMainRule;
 //# sourceMappingURL=no-edit-on-main.js.map

package/src/core/rules/no-edit-on-main.js.map

@@ -1 +1 @@
-{"version":3,"file":"no-edit-on-main.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/no-edit-on-main.ts"],"names":[],"mappings":";;AAAA,iDAAyC;AAEzC,oCAA0C;AAC1C,0CAAsC;AAEtC,MAAM,YAAY,GAAa;IAC3B,IAAI,EAAE,iBAAiB;IACvB,WAAW,EAAE,sFAAsF;IACnG,KAAK,EAAE,MAAM;IACb,KAAK,EAAE,CAAC,MAAM,CAAC;IACf,cAAc,EAAE;QACZ,sBAAsB,EAAE,wBAAwB;KACnD;IACD,OAAO,EAAE;QACL,oCAAoC;QACpC,QAAQ;QACR,kDAAkD;QAClD,8DAA8D;QAC9D,4GAA4G;KAC/G;IAED,KAAK,CAAC,GAAgB;QAClB,gDAAgD;QAChD,IAAI,GAAG,CAAC,YAAY,CAAC,UAAU,CAAC,IAAI,CAAC;YAAE,OAAO,EAAE,CAAC;QAEjD,IAAI,aAAqB,CAAC;QAC1B,8DAA8D;QAC9D,IAAI,CAAC;YACD,aAAa,GAAG,IAAA,wBAAQ,EAAC,iCAAiC,EAAE;gBACxD,GAAG,EAAE,GAAG,CAAC,aAAa;gBACtB,QAAQ,EAAE,MAAM;gBAChB,KAAK,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC;aAClC,CAAC,CAAC,IAAI,EAAE,CAAC;QACd,CAAC;QAAC,OAAO,GAAY,EAAE,CAAC;YACpB,MAAM,KAAK,GAAG,IAAA,kBAAO,EAAC,GAAG,CAAC,CAAC;YAC3B,KAAK,KAAK,CAAC;YACX,OAAO,EAAE,CAAC;QACd,CAAC;QAED,IAAI,aAAa,KAAK,MAAM;YAAE,OAAO,EAAE,CAAC;QAExC,MAAM,UAAU,GAAI,GAAG,CAAC,OAAO,CAAC,wBAAwB,CAAwB;eACzE,yCAAyC,CAAC;QAEjD,OAAO,CAAC,IAAI,iBAAC,CACT,CAAC,EACD,GAAG,CAAC,YAAY,EAChB;gBACI,oCAAoC;gBACpC,yGAAyG;gBACzG,0DAA0D,UAAU,EAAE;gBACtE,2BAA2B,GAAG,UAAU,CAAC,OAAO,CAAC,UAAU,EAAE,OAAO,CAAC;aACxE,CAAC,IAAI,CAAC,IAAI,CAAC,CACf,CAAC,CAAC;IACP,CAAC;CACJ,CAAC;AAEF,kBAAe,YAAY,CAAC","sourcesContent":["import { execSync } from 'child_process';\nimport type { FileRule, FileContext, Violation } from '../types';\nimport { Violation as V } from '../types';\nimport { toError } from '../to-error';\n\nconst noEditOnMain: FileRule = {\n    name: 'no-edit-on-main',\n    description: 'Block file edits when on the main branch — all work must happen on a feature branch.',\n    scope: 'file',\n    files: ['**/*'],\n    defaultOptions: {\n        branchNamingConvention: '{whoami}/{featurename}',\n    },\n    fixHint: [\n        'You should not be working on main.',\n        'Steps:',\n        '  1. git pull origin main   ← get latest commits',\n        '  2. git checkout -b <branch-name>   ← create feature branch',\n        'Branch naming convention is defined in webpieces.config.json under no-edit-on-main.branchNamingConvention.',\n    ],\n\n    check(ctx: FileContext): readonly Violation[] {\n        // Only apply to files inside the workspace root\n        if (ctx.relativePath.startsWith('..')) return [];\n\n        let currentBranch: string;\n        // eslint-disable-next-line @webpieces/no-unmanaged-exceptions\n        try {\n            currentBranch = execSync('git rev-parse --abbrev-ref HEAD', {\n                cwd: ctx.workspaceRoot,\n                encoding: 'utf8',\n                stdio: ['pipe', 'pipe', 'pipe'],\n            }).trim();\n        } catch (err: unknown) {\n            const error = toError(err);\n            void error;\n            return [];\n        }\n\n        if (currentBranch !== 'main') return [];\n\n        const convention = (ctx.options['branchNamingConvention'] as string | undefined)\n            ?? 'feature/<ticket-id>-<short-description>';\n\n        return [new V(\n            1,\n            ctx.relativePath,\n            [\n                'You should not be working on main.',\n                'Do a `git pull origin main` to get latest, then create a feature branch based on the naming convention.',\n                `Branch naming convention (from webpieces.config.json): ${convention}`,\n                'Example: git checkout -b ' + convention.replace(/<[^>]+>/g, 'value'),\n            ].join('\\n'),\n        )];\n    },\n};\n\nexport default noEditOnMain;\n"]}
+{"version":3,"file":"no-edit-on-main.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/no-edit-on-main.ts"],"names":[],"mappings":";;;AAAA,iDAAyC;AAKzC,oCAA0C;AAC1C,4CAA4C;AAC5C,0CAAsC;AAEtC,MAAa,gBAAiB,SAAQ,wBAAgC;IAClE,YAAY,MAA0B,IAAI,KAAK,CAAC,MAAM,EAAE,iBAAiB,CAAC,CAAC,CAAC,CAAC;IAEpE,WAAW,GAAG,sFAAsF,CAAC;IAC5F,KAAK,GAAG,CAAC,MAAM,CAAC,CAAC;IACjB,cAAc,GAAG;QAC/B,sBAAsB,EAAE,wBAAwB;KACnD,CAAC;IACO,OAAO,GAAG;QACf,oCAAoC;QACpC,QAAQ;QACR,kDAAkD;QAClD,8DAA8D;QAC9D,4GAA4G;KAC/G,CAAC;IAEF,KAAK,CAAC,GAAgB;QAClB,gDAAgD;QAChD,IAAI,GAAG,CAAC,YAAY,CAAC,UAAU,CAAC,IAAI,CAAC;YAAE,OAAO,EAAE,CAAC;QAEjD,IAAI,aAAqB,CAAC;QAC1B,8DAA8D;QAC9D,IAAI,CAAC;YACD,aAAa,GAAG,IAAA,wBAAQ,EAAC,iCAAiC,EAAE;gBACxD,GAAG,EAAE,GAAG,CAAC,aAAa;gBACtB,QAAQ,EAAE,MAAM;gBAChB,KAAK,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC;aAClC,CAAC,CAAC,IAAI,EAAE,CAAC;QACd,CAAC;QAAC,OAAO,GAAY,EAAE,CAAC;YACpB,MAAM,KAAK,GAAG,IAAA,kBAAO,EAAC,GAAG,CAAC,CAAC;YAC3B,KAAK,KAAK,CAAC;YACX,OAAO,EAAE,CAAC;QACd,CAAC;QAED,IAAI,aAAa,KAAK,MAAM;YAAE,OAAO,EAAE,CAAC;QAExC,MAAM,UAAU,GAAG,IAAI,CAAC,MAAM,CAAC,sBAAsB,IAAI,wBAAwB,CAAC;QAElF,OAAO,CAAC,IAAI,iBAAC,CACT,CAAC,EACD,GAAG,CAAC,YAAY,EAChB;gBACI,oCAAoC;gBACpC,yGAAyG;gBACzG,0DAA0D,UAAU,EAAE;gBACtE,2BAA2B,GAAG,UAAU,CAAC,OAAO,CAAC,UAAU,EAAE,OAAO,CAAC;aACxE,CAAC,IAAI,CAAC,IAAI,CAAC,CACf,CAAC,CAAC;IACP,CAAC;CACJ;AAjDD,4CAiDC","sourcesContent":["import { execSync } from 'child_process';\n\nimport { NoEditOnMainConfig } from '@webpieces/rules-config';\n\nimport type { FileContext, Violation } from '../types';\nimport { Violation as V } from '../types';\nimport { FileRuleBase } from '../rule-base';\nimport { toError } from '../to-error';\n\nexport class NoEditOnMainRule extends FileRuleBase<NoEditOnMainConfig> {\n    constructor(config: NoEditOnMainConfig) { super(config, 'no-edit-on-main'); }\n\n    readonly description = 'Block file edits when on the main branch — all work must happen on a feature branch.';\n    override readonly files = ['**/*'];\n    override readonly defaultOptions = {\n        branchNamingConvention: '{whoami}/{featurename}',\n    };\n    readonly fixHint = [\n        'You should not be working on main.',\n        'Steps:',\n        '  1. git pull origin main   ← get latest commits',\n        '  2. git checkout -b <branch-name>   ← create feature branch',\n        'Branch naming convention is defined in webpieces.config.json under no-edit-on-main.branchNamingConvention.',\n    ];\n\n    check(ctx: FileContext): readonly Violation[] {\n        // Only apply to files inside the workspace root\n        if (ctx.relativePath.startsWith('..')) return [];\n\n        let currentBranch: string;\n        // eslint-disable-next-line @webpieces/no-unmanaged-exceptions\n        try {\n            currentBranch = execSync('git rev-parse --abbrev-ref HEAD', {\n                cwd: ctx.workspaceRoot,\n                encoding: 'utf8',\n                stdio: ['pipe', 'pipe', 'pipe'],\n            }).trim();\n        } catch (err: unknown) {\n            const error = toError(err);\n            void error;\n            return [];\n        }\n\n        if (currentBranch !== 'main') return [];\n\n        const convention = this.config.branchNamingConvention ?? '{whoami}/{featurename}';\n\n        return [new V(\n            1,\n            ctx.relativePath,\n            [\n                'You should not be working on main.',\n                'Do a `git pull origin main` to get latest, then create a feature branch based on the naming convention.',\n                `Branch naming convention (from webpieces.config.json): ${convention}`,\n                'Example: git checkout -b ' + convention.replace(/<[^>]+>/g, 'value'),\n            ].join('\\n'),\n        )];\n    }\n}\n"]}

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

@@ -1,3 +1,10 @@
-import type { EditRule } from '../types';
-declare const noImplicitAnyRule: EditRule;
-export default noImplicitAnyRule;
+import { NoImplicitAnyConfig } from '@webpieces/rules-config';
+import type { EditContext, Violation } from '../types';
+import { EditRuleBase } from '../rule-base';
+export declare class NoImplicitAnyRule extends EditRuleBase<NoImplicitAnyConfig> {
+    constructor(config: NoImplicitAnyConfig);
+    readonly description = "Disallow function parameters without explicit type annotations (implicit-any).";
+    readonly files: string[];
+    readonly fixHint: string[];
+    check(ctx: EditContext): readonly Violation[];
+}

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

@@ -1,6 +1,9 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.NoImplicitAnyRule = void 0;
+const rules_config_1 = require("@webpieces/rules-config");
 const types_1 = require("../types");
+const rule_base_1 = require("../rule-base");
 const ARROW_PARAMS_RE = /\(([^()]*)\)\s*=>/g;
 const FN_DECL_PARAMS_RE = /\bfunction\s*[\w$]*\s*\(([^()]*)\)/g;
 function firstUntypedParam(paramsStr) {
@@ -40,22 +43,20 @@ function findOffender(line) {
     }
     return null;
 }
-const noImplicitAnyRule = {
-    name: 'no-implicit-any',
-    description: 'Disallow function parameters without explicit type annotations (implicit-any).',
-    scope: 'edit',
-    files: ['**/*.ts', '**/*.tsx'],
-    defaultOptions: {},
-    fixHint: [
+class NoImplicitAnyRule extends rule_base_1.EditRuleBase {
+    constructor(config) { super(config, 'no-implicit-any'); }
+    description = 'Disallow function parameters without explicit type annotations (implicit-any).';
+    files = ['**/*.ts', '**/*.tsx'];
+    fixHint = [
         'Add explicit types: (x: string) => ...   or   function foo(x: number)',
         '// webpieces-disable no-implicit-any -- <one-line reason>',
-    ],
+    ];
     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'))
+            if (ctx.isLineDisabled(lineNum, rules_config_1.RULE_NAMES.NO_IMPLICIT_ANY))
                 continue;
             const offender = findOffender(stripped);
             if (!offender)
@@ -63,7 +64,7 @@ const noImplicitAnyRule = {
             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;
-    },
-};
-exports.default = noImplicitAnyRule;
+    }
+}
+exports.NoImplicitAnyRule = NoImplicitAnyRule;
 //# sourceMappingURL=no-implicit-any.js.map

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

@@ -1 +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"]}
+{"version":3,"file":"no-implicit-any.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/no-implicit-any.ts"],"names":[],"mappings":";;;AAAA,0DAA0E;AAG1E,oCAA0C;AAC1C,4CAA4C;AAE5C,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,CAAS,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAS,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;IACpG,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,MAAa,iBAAkB,SAAQ,wBAAiC;IACpE,YAAY,MAA2B,IAAI,KAAK,CAAC,MAAM,EAAE,iBAAiB,CAAC,CAAC,CAAC,CAAC;IAErE,WAAW,GAAG,gFAAgF,CAAC;IACtF,KAAK,GAAG,CAAC,SAAS,EAAE,UAAU,CAAC,CAAC;IACzC,OAAO,GAAG;QACf,uEAAuE;QACvE,2DAA2D;KAC9D,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,MAAM,OAAO,GAAG,CAAC,GAAG,CAAC,CAAC;YACtB,IAAI,GAAG,CAAC,cAAc,CAAC,OAAO,EAAE,yBAAU,CAAC,eAAe,CAAC;gBAAE,SAAS;YACtE,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;AA1BD,8CA0BC","sourcesContent":["import { NoImplicitAnyConfig, 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\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: string) => p.trim()).filter((p: string) => 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\nexport class NoImplicitAnyRule extends EditRuleBase<NoImplicitAnyConfig> {\n    constructor(config: NoImplicitAnyConfig) { super(config, 'no-implicit-any'); }\n\n    readonly description = 'Disallow function parameters without explicit type annotations (implicit-any).';\n    override readonly files = ['**/*.ts', '**/*.tsx'];\n    readonly 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, RULE_NAMES.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"]}

package/src/core/rules/no-js-files.d.ts

@@ -1,3 +1,13 @@
-import type { FileRule } from '../types';
-declare const noJsFiles: FileRule;
-export default noJsFiles;
+import { NoJsFilesConfig } from '@webpieces/rules-config';
+import type { FileContext, Violation } from '../types';
+import { FileRuleBase } from '../rule-base';
+export declare class NoJsFilesRule extends FileRuleBase<NoJsFilesConfig> {
+    constructor(config: NoJsFilesConfig);
+    readonly description = "Disallow writing new .js/.jsx files. Use .ts/.tsx instead.";
+    readonly files: string[];
+    readonly defaultOptions: {
+        allowedPaths: never[];
+    };
+    readonly fixHint: string[];
+    check(ctx: FileContext): readonly Violation[];
+}

package/src/core/rules/no-js-files.js

@@ -1,29 +1,26 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.NoJsFilesRule = void 0;
 const rules_config_1 = require("@webpieces/rules-config");
 const types_1 = require("../types");
-const noJsFiles = {
-    name: 'no-js-files',
-    description: 'Disallow writing new .js/.jsx files. Use .ts/.tsx instead.',
-    scope: 'file',
-    files: ['**/*.js', '**/*.jsx'],
-    defaultOptions: {
-        allowedPaths: [],
-    },
-    fixHint: [
+const rule_base_1 = require("../rule-base");
+class NoJsFilesRule extends rule_base_1.FileRuleBase {
+    constructor(config) { super(config, 'no-js-files'); }
+    description = 'Disallow writing new .js/.jsx files. Use .ts/.tsx instead.';
+    files = ['**/*.js', '**/*.jsx'];
+    defaultOptions = { allowedPaths: [] };
+    fixHint = [
         'Write a .ts or .tsx file instead of .js/.jsx.',
         'If this path must be .js (e.g. a generated or legacy file), add it to no-js-files.allowedPaths in webpieces.config.json',
-    ],
+    ];
     check(ctx) {
         if (ctx.tool !== 'Write')
             return [];
-        const allowedPaths = Array.isArray(ctx.options['allowedPaths'])
-            ? ctx.options['allowedPaths']
-            : [];
+        const allowedPaths = this.config.allowedPaths ?? [];
         if ((0, rules_config_1.isPathExcluded)(ctx.relativePath, allowedPaths))
             return [];
         return [new types_1.Violation(1, ctx.relativePath, 'Writing .js/.jsx files is not allowed. Use .ts/.tsx instead.')];
-    },
-};
-exports.default = noJsFiles;
+    }
+}
+exports.NoJsFilesRule = NoJsFilesRule;
 //# sourceMappingURL=no-js-files.js.map

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

@@ -1 +1 @@
-{"version":3,"file":"no-js-files.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/no-js-files.ts"],"names":[],"mappings":";;AAAA,0DAAyD;AAGzD,oCAA0C;AAE1C,MAAM,SAAS,GAAa;IACxB,IAAI,EAAE,aAAa;IACnB,WAAW,EAAE,4DAA4D;IACzE,KAAK,EAAE,MAAM;IACb,KAAK,EAAE,CAAC,SAAS,EAAE,UAAU,CAAC;IAC9B,cAAc,EAAE;QACZ,YAAY,EAAE,EAAE;KACnB;IACD,OAAO,EAAE;QACL,+CAA+C;QAC/C,yHAAyH;KAC5H;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,EAAE,CAAC;QAET,IAAI,IAAA,6BAAc,EAAC,GAAG,CAAC,YAAY,EAAE,YAAY,CAAC;YAAE,OAAO,EAAE,CAAC;QAE9D,OAAO,CAAC,IAAI,iBAAC,CAAC,CAAC,EAAE,GAAG,CAAC,YAAY,EAAE,8DAA8D,CAAC,CAAC,CAAC;IACxG,CAAC;CACJ,CAAC;AAEF,kBAAe,SAAS,CAAC","sourcesContent":["import { isPathExcluded } from '@webpieces/rules-config';\n\nimport type { FileRule, FileContext, Violation } from '../types';\nimport { Violation as V } from '../types';\n\nconst noJsFiles: FileRule = {\n    name: 'no-js-files',\n    description: 'Disallow writing new .js/.jsx files. Use .ts/.tsx instead.',\n    scope: 'file',\n    files: ['**/*.js', '**/*.jsx'],\n    defaultOptions: {\n        allowedPaths: [],\n    },\n    fixHint: [\n        'Write a .ts or .tsx file instead of .js/.jsx.',\n        'If this path must be .js (e.g. a generated or legacy file), add it to no-js-files.allowedPaths in webpieces.config.json',\n    ],\n\n    check(ctx: FileContext): readonly Violation[] {\n        if (ctx.tool !== 'Write') return [];\n\n        const allowedPaths = Array.isArray(ctx.options['allowedPaths'])\n            ? ctx.options['allowedPaths'] as string[]\n            : [];\n\n        if (isPathExcluded(ctx.relativePath, allowedPaths)) return [];\n\n        return [new V(1, ctx.relativePath, 'Writing .js/.jsx files is not allowed. Use .ts/.tsx instead.')];\n    },\n};\n\nexport default noJsFiles;\n"]}
+{"version":3,"file":"no-js-files.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/no-js-files.ts"],"names":[],"mappings":";;;AAAA,0DAA0E;AAG1E,oCAA0C;AAC1C,4CAA4C;AAE5C,MAAa,aAAc,SAAQ,wBAA6B;IAC5D,YAAY,MAAuB,IAAI,KAAK,CAAC,MAAM,EAAE,aAAa,CAAC,CAAC,CAAC,CAAC;IAE7D,WAAW,GAAG,4DAA4D,CAAC;IAClE,KAAK,GAAG,CAAC,SAAS,EAAE,UAAU,CAAC,CAAC;IAChC,cAAc,GAAG,EAAE,YAAY,EAAE,EAAE,EAAE,CAAC;IAC/C,OAAO,GAAG;QACf,+CAA+C;QAC/C,yHAAyH;KAC5H,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,EAAE,CAAC;QAEpD,IAAI,IAAA,6BAAc,EAAC,GAAG,CAAC,YAAY,EAAE,YAAY,CAAC;YAAE,OAAO,EAAE,CAAC;QAE9D,OAAO,CAAC,IAAI,iBAAC,CAAC,CAAC,EAAE,GAAG,CAAC,YAAY,EAAE,8DAA8D,CAAC,CAAC,CAAC;IACxG,CAAC;CACJ;AApBD,sCAoBC","sourcesContent":["import { NoJsFilesConfig, isPathExcluded } from '@webpieces/rules-config';\n\nimport type { FileContext, Violation } from '../types';\nimport { Violation as V } from '../types';\nimport { FileRuleBase } from '../rule-base';\n\nexport class NoJsFilesRule extends FileRuleBase<NoJsFilesConfig> {\n    constructor(config: NoJsFilesConfig) { super(config, 'no-js-files'); }\n\n    readonly description = 'Disallow writing new .js/.jsx files. Use .ts/.tsx instead.';\n    override readonly files = ['**/*.js', '**/*.jsx'];\n    override readonly defaultOptions = { allowedPaths: [] };\n    readonly fixHint = [\n        'Write a .ts or .tsx file instead of .js/.jsx.',\n        'If this path must be .js (e.g. a generated or legacy file), add it to no-js-files.allowedPaths in webpieces.config.json',\n    ];\n\n    check(ctx: FileContext): readonly Violation[] {\n        if (ctx.tool !== 'Write') return [];\n\n        const allowedPaths = this.config.allowedPaths ?? [];\n\n        if (isPathExcluded(ctx.relativePath, allowedPaths)) return [];\n\n        return [new V(1, ctx.relativePath, 'Writing .js/.jsx files is not allowed. Use .ts/.tsx instead.')];\n    }\n}\n"]}

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

@@ -1,3 +1,9 @@
-import type { BashRule } from '../types';
-declare const noShellSubstitutionRule: BashRule;
-export default noShellSubstitutionRule;
+import { NoShellSubstitutionConfig } from '@webpieces/rules-config';
+import type { BashContext, Violation } from '../types';
+import { BashRuleBase } from '../rule-base';
+export declare class NoShellSubstitutionRule extends BashRuleBase<NoShellSubstitutionConfig> {
+    constructor(config: NoShellSubstitutionConfig);
+    readonly description = "Reject Bash commands containing shell substitutions ($(...), backticks, $VAR).";
+    readonly fixHint: readonly string[];
+    check(ctx: BashContext): readonly Violation[];
+}

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

@@ -1,33 +1,13 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.NoShellSubstitutionRule = void 0;
 const types_1 = require("../types");
+const rule_base_1 = require("../rule-base");
 const FIX_HINT = [
     'Build a payload file with the Write tool and pass it as stdin: node script.js < /path/to/payload.json',
     'Use the Read, Grep, or Glob tools directly instead of capturing output with $(...)',
     'Write a small script file with the Write tool and execute it directly: 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, "''");
 }
@@ -48,5 +28,24 @@ function truncate(s) {
         return s;
     return s.slice(0, MAX) + '…';
 }
-exports.default = noShellSubstitutionRule;
+class NoShellSubstitutionRule extends rule_base_1.BashRuleBase {
+    constructor(config) { super(config, 'no-shell-substitution'); }
+    description = 'Reject Bash commands containing shell substitutions ($(...), backticks, $VAR).';
+    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;
+    }
+}
+exports.NoShellSubstitutionRule = NoShellSubstitutionRule;
 //# sourceMappingURL=no-shell-substitution.js.map

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

@@ -1 +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,uGAAuG;IACvG,oFAAoF;IACpF,gGAAgG;CACnG,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    'Build a payload file with the Write tool and pass it as stdin: node script.js < /path/to/payload.json',\n    'Use the Read, Grep, or Glob tools directly instead of capturing output with $(...)',\n    'Write a small script file with the Write tool and execute it directly: 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"]}
+{"version":3,"file":"no-shell-substitution.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/ai-hook-rules/src/core/rules/no-shell-substitution.ts"],"names":[],"mappings":";;;AAGA,oCAA0C;AAC1C,4CAA4C;AAE5C,MAAM,QAAQ,GAAsB;IAChC,uGAAuG;IACvG,oFAAoF;IACpF,gGAAgG;CACnG,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,MAAa,uBAAwB,SAAQ,wBAAuC;IAChF,YAAY,MAAiC,IAAI,KAAK,CAAC,MAAM,EAAE,uBAAuB,CAAC,CAAC,CAAC,CAAC;IAEjF,WAAW,GAAG,gFAAgF,CAAC;IAC/F,OAAO,GAAG,QAAQ,CAAC;IAE5B,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;AAjCD,0DAiCC","sourcesContent":["import { NoShellSubstitutionConfig } from '@webpieces/rules-config';\n\nimport type { BashContext, Violation } from '../types';\nimport { Violation as V } from '../types';\nimport { BashRuleBase } from '../rule-base';\n\nconst FIX_HINT: readonly string[] = [\n    'Build a payload file with the Write tool and pass it as stdin: node script.js < /path/to/payload.json',\n    'Use the Read, Grep, or Glob tools directly instead of capturing output with $(...)',\n    'Write a small script file with the Write tool and execute it directly: bash /path/to/script.sh',\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 class NoShellSubstitutionRule extends BashRuleBase<NoShellSubstitutionConfig> {\n    constructor(config: NoShellSubstitutionConfig) { super(config, 'no-shell-substitution'); }\n\n    readonly description = 'Reject Bash commands containing shell substitutions ($(...), backticks, $VAR).';\n    readonly 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"]}

package/src/core/rules/no-symbol-di-tokens.d.ts

@@ -1,3 +1,13 @@
-import type { EditRule } from '../types';
-declare const noSymbolDiTokensRule: EditRule;
-export default noSymbolDiTokensRule;
+import { NoSymbolDiTokensConfig } from '@webpieces/rules-config';
+import type { EditContext, Violation } from '../types';
+import { EditRuleBase } from '../rule-base';
+export declare class NoSymbolDiTokensRule extends EditRuleBase<NoSymbolDiTokensConfig> {
+    constructor(config: NoSymbolDiTokensConfig);
+    readonly description = "Disallow Symbol() DI tokens outside explicitly configured paths. Use @provideSingleton() + inject-by-type instead.";
+    readonly files: string[];
+    readonly defaultOptions: {
+        allowedPaths: never[];
+    };
+    readonly fixHint: string[];
+    check(ctx: EditContext): readonly Violation[];
+}