@prairielearn/eslint-plugin 4.0.0 → 4.2.0

package/dist/tests/aws-client-shared-config.test.js

@@ -1,16 +1,11 @@
-"use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-const rule_tester_1 = require("@typescript-eslint/rule-tester");
-const vitest_1 = require("vitest");
-const aws_client_shared_config_1 = __importDefault(require("../rules/aws-client-shared-config"));
-rule_tester_1.RuleTester.afterAll = vitest_1.afterAll;
-rule_tester_1.RuleTester.describe = vitest_1.describe;
-rule_tester_1.RuleTester.it = vitest_1.it;
-const ruleTester = new rule_tester_1.RuleTester();
-ruleTester.run('aws-client-shared-config', aws_client_shared_config_1.default, {
+import { RuleTester } from '@typescript-eslint/rule-tester';
+import { afterAll, describe, it } from 'vitest';
+import rule from '../rules/aws-client-shared-config.js';
+RuleTester.afterAll = afterAll;
+RuleTester.describe = describe;
+RuleTester.it = it;
+const ruleTester = new RuleTester();
+ruleTester.run('aws-client-shared-config', rule, {
     valid: [
         {
             code: "import { S3 } from '@aws-sdk/client-s3'; new S3(makeS3ClientConfig());",

package/dist/tests/aws-client-shared-config.test.js.map

@@ -1 +1 @@
-{"version":3,"file":"aws-client-shared-config.test.js","sourceRoot":"","sources":["../../src/tests/aws-client-shared-config.test.ts"],"names":[],"mappings":";;;;;AAAA,gEAA4D;AAC5D,mCAAgD;AAEhD,iGAAqD;AAErD,wBAAU,CAAC,QAAQ,GAAG,iBAAQ,CAAC;AAC/B,wBAAU,CAAC,QAAQ,GAAG,iBAAQ,CAAC;AAC/B,wBAAU,CAAC,EAAE,GAAG,WAAE,CAAC;AAEnB,MAAM,UAAU,GAAG,IAAI,wBAAU,EAAE,CAAC;AAEpC,UAAU,CAAC,GAAG,CAAC,0BAA0B,EAAE,kCAAI,EAAE;IAC/C,KAAK,EAAE;QACL;YACE,IAAI,EAAE,wEAAwE;SAC/E;QACD;YACE,IAAI,EAAE,oFAAoF;SAC3F;QACD;YACE,IAAI,EAAE,4EAA4E;SACnF;QACD;YACE,IAAI,EAAE,wFAAwF;SAC/F;QACD;YACE,IAAI,EAAE,gFAAgF;SACvF;KACF;IACD,OAAO,EAAE;QACP;YACE,IAAI,EAAE,2EAA2E;YACjF,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,gBAAgB,EAAE,CAAC;SAC1C;QACD;YACE,IAAI,EAAE,8EAA8E;YACpF,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,gBAAgB,EAAE,CAAC;SAC1C;QACD;YACE,IAAI,EAAE,mEAAmE;YACzE,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,gBAAgB,EAAE,CAAC;SAC1C;KACF;CACF,CAAC,CAAC","sourcesContent":["import { RuleTester } from '@typescript-eslint/rule-tester';\nimport { afterAll, describe, it } from 'vitest';\n\nimport rule from '../rules/aws-client-shared-config';\n\nRuleTester.afterAll = afterAll;\nRuleTester.describe = describe;\nRuleTester.it = it;\n\nconst ruleTester = new RuleTester();\n\nruleTester.run('aws-client-shared-config', rule, {\n  valid: [\n    {\n      code: \"import { S3 } from '@aws-sdk/client-s3'; new S3(makeS3ClientConfig());\",\n    },\n    {\n      code: \"import { S3Client } from '@aws-sdk/client-s3'; new S3Client(makeS3ClientConfig());\",\n    },\n    {\n      code: \"import { EC2 } from '@aws-sdk/client-ec2'; new EC2(makeAwsClientConfig());\",\n    },\n    {\n      code: \"import { EC2Client } from '@aws-sdk/client-ec2'; new EC2Client(makeAwsClientConfig());\",\n    },\n    {\n      code: \"import { EC2 } from '@aws-sdk/client-ec2'; new EC2(aws.makeAwsClientConfig());\",\n    },\n  ],\n  invalid: [\n    {\n      code: \"import { S3 } from '@aws-sdk/client-s3'; new S3({ region: 'us-east-2' });\",\n      errors: [{ messageId: 'improperConfig' }],\n    },\n    {\n      code: \"import { EC2 } from '@aws-sdk/client-ec2'; new EC2({ region: 'us-east-2' });\",\n      errors: [{ messageId: 'improperConfig' }],\n    },\n    {\n      code: \"import { S3 } from '@aws-sdk/client-s3'; new S3(wrongFunction());\",\n      errors: [{ messageId: 'improperConfig' }],\n    },\n  ],\n});\n"]}
+{"version":3,"file":"aws-client-shared-config.test.js","sourceRoot":"","sources":["../../src/tests/aws-client-shared-config.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,gCAAgC,CAAC;AAC5D,OAAO,EAAE,QAAQ,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAEhD,OAAO,IAAI,MAAM,sCAAsC,CAAC;AAExD,UAAU,CAAC,QAAQ,GAAG,QAAQ,CAAC;AAC/B,UAAU,CAAC,QAAQ,GAAG,QAAQ,CAAC;AAC/B,UAAU,CAAC,EAAE,GAAG,EAAE,CAAC;AAEnB,MAAM,UAAU,GAAG,IAAI,UAAU,EAAE,CAAC;AAEpC,UAAU,CAAC,GAAG,CAAC,0BAA0B,EAAE,IAAI,EAAE;IAC/C,KAAK,EAAE;QACL;YACE,IAAI,EAAE,wEAAwE;SAC/E;QACD;YACE,IAAI,EAAE,oFAAoF;SAC3F;QACD;YACE,IAAI,EAAE,4EAA4E;SACnF;QACD;YACE,IAAI,EAAE,wFAAwF;SAC/F;QACD;YACE,IAAI,EAAE,gFAAgF;SACvF;KACF;IACD,OAAO,EAAE;QACP;YACE,IAAI,EAAE,2EAA2E;YACjF,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,gBAAgB,EAAE,CAAC;SAC1C;QACD;YACE,IAAI,EAAE,8EAA8E;YACpF,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,gBAAgB,EAAE,CAAC;SAC1C;QACD;YACE,IAAI,EAAE,mEAAmE;YACzE,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,gBAAgB,EAAE,CAAC;SAC1C;KACF;CACF,CAAC,CAAC","sourcesContent":["import { RuleTester } from '@typescript-eslint/rule-tester';\nimport { afterAll, describe, it } from 'vitest';\n\nimport rule from '../rules/aws-client-shared-config.js';\n\nRuleTester.afterAll = afterAll;\nRuleTester.describe = describe;\nRuleTester.it = it;\n\nconst ruleTester = new RuleTester();\n\nruleTester.run('aws-client-shared-config', rule, {\n  valid: [\n    {\n      code: \"import { S3 } from '@aws-sdk/client-s3'; new S3(makeS3ClientConfig());\",\n    },\n    {\n      code: \"import { S3Client } from '@aws-sdk/client-s3'; new S3Client(makeS3ClientConfig());\",\n    },\n    {\n      code: \"import { EC2 } from '@aws-sdk/client-ec2'; new EC2(makeAwsClientConfig());\",\n    },\n    {\n      code: \"import { EC2Client } from '@aws-sdk/client-ec2'; new EC2Client(makeAwsClientConfig());\",\n    },\n    {\n      code: \"import { EC2 } from '@aws-sdk/client-ec2'; new EC2(aws.makeAwsClientConfig());\",\n    },\n  ],\n  invalid: [\n    {\n      code: \"import { S3 } from '@aws-sdk/client-s3'; new S3({ region: 'us-east-2' });\",\n      errors: [{ messageId: 'improperConfig' }],\n    },\n    {\n      code: \"import { EC2 } from '@aws-sdk/client-ec2'; new EC2({ region: 'us-east-2' });\",\n      errors: [{ messageId: 'improperConfig' }],\n    },\n    {\n      code: \"import { S3 } from '@aws-sdk/client-s3'; new S3(wrongFunction());\",\n      errors: [{ messageId: 'improperConfig' }],\n    },\n  ],\n});\n"]}

package/dist/tests/jsx-no-dollar-interpolation.test.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"jsx-no-dollar-interpolation.test.d.ts","sourceRoot":"","sources":["../../src/tests/jsx-no-dollar-interpolation.test.ts"],"names":[],"mappings":"","sourcesContent":["import { RuleTester } from '@typescript-eslint/rule-tester';\nimport { afterAll, describe, it } from 'vitest';\n\nimport rule from '../rules/jsx-no-dollar-interpolation';\n\nRuleTester.afterAll = afterAll;\nRuleTester.describe = describe;\nRuleTester.it = it;\n\nconst ruleTester = new RuleTester({\n  languageOptions: {\n    parserOptions: {\n      ecmaFeatures: {\n        jsx: true,\n      },\n    },\n  },\n});\n\nruleTester.run('jsx-no-dollar-interpolation', rule, {\n  valid: [\n    {\n      code: '<div>hello</div>',\n    },\n    {\n      code: '<div>100$</div>',\n    },\n    {\n      code: '<div>$100</div>',\n    },\n    {\n      code: '<div>$</div>',\n    },\n  ],\n  invalid: [\n    {\n      // eslint-disable-next-line no-template-curly-in-string\n      code: '<div>${message}</div>',\n      errors: [\n        {\n          messageId: 'dollarInterpolationNotAllowed',\n          line: 1,\n          column: 6,\n          endLine: 1,\n          endColumn: 16,\n        },\n      ],\n    },\n    {\n      code: '<div>$ {message}</div>',\n      errors: [\n        {\n          messageId: 'dollarInterpolationNotAllowed',\n          line: 1,\n          column: 6,\n          endLine: 1,\n          endColumn: 17,\n        },\n      ],\n    },\n  ],\n});\n"]}
+{"version":3,"file":"jsx-no-dollar-interpolation.test.d.ts","sourceRoot":"","sources":["../../src/tests/jsx-no-dollar-interpolation.test.ts"],"names":[],"mappings":"","sourcesContent":["import { RuleTester } from '@typescript-eslint/rule-tester';\nimport { afterAll, describe, it } from 'vitest';\n\nimport rule from '../rules/jsx-no-dollar-interpolation.js';\n\nRuleTester.afterAll = afterAll;\nRuleTester.describe = describe;\nRuleTester.it = it;\n\nconst ruleTester = new RuleTester({\n  languageOptions: {\n    parserOptions: {\n      ecmaFeatures: {\n        jsx: true,\n      },\n    },\n  },\n});\n\nruleTester.run('jsx-no-dollar-interpolation', rule, {\n  valid: [\n    {\n      code: '<div>hello</div>',\n    },\n    {\n      code: '<div>100$</div>',\n    },\n    {\n      code: '<div>$100</div>',\n    },\n    {\n      code: '<div>$</div>',\n    },\n  ],\n  invalid: [\n    {\n      // eslint-disable-next-line no-template-curly-in-string\n      code: '<div>${message}</div>',\n      errors: [\n        {\n          messageId: 'dollarInterpolationNotAllowed',\n          line: 1,\n          column: 6,\n          endLine: 1,\n          endColumn: 16,\n        },\n      ],\n    },\n    {\n      code: '<div>$ {message}</div>',\n      errors: [\n        {\n          messageId: 'dollarInterpolationNotAllowed',\n          line: 1,\n          column: 6,\n          endLine: 1,\n          endColumn: 17,\n        },\n      ],\n    },\n  ],\n});\n"]}

package/dist/tests/jsx-no-dollar-interpolation.test.js

@@ -1,15 +1,10 @@
-"use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-const rule_tester_1 = require("@typescript-eslint/rule-tester");
-const vitest_1 = require("vitest");
-const jsx_no_dollar_interpolation_1 = __importDefault(require("../rules/jsx-no-dollar-interpolation"));
-rule_tester_1.RuleTester.afterAll = vitest_1.afterAll;
-rule_tester_1.RuleTester.describe = vitest_1.describe;
-rule_tester_1.RuleTester.it = vitest_1.it;
-const ruleTester = new rule_tester_1.RuleTester({
+import { RuleTester } from '@typescript-eslint/rule-tester';
+import { afterAll, describe, it } from 'vitest';
+import rule from '../rules/jsx-no-dollar-interpolation.js';
+RuleTester.afterAll = afterAll;
+RuleTester.describe = describe;
+RuleTester.it = it;
+const ruleTester = new RuleTester({
     languageOptions: {
         parserOptions: {
             ecmaFeatures: {
@@ -18,7 +13,7 @@ const ruleTester = new rule_tester_1.RuleTester({
         },
     },
 });
-ruleTester.run('jsx-no-dollar-interpolation', jsx_no_dollar_interpolation_1.default, {
+ruleTester.run('jsx-no-dollar-interpolation', rule, {
     valid: [
         {
             code: '<div>hello</div>',

package/dist/tests/jsx-no-dollar-interpolation.test.js.map

@@ -1 +1 @@
-{"version":3,"file":"jsx-no-dollar-interpolation.test.js","sourceRoot":"","sources":["../../src/tests/jsx-no-dollar-interpolation.test.ts"],"names":[],"mappings":";;;;;AAAA,gEAA4D;AAC5D,mCAAgD;AAEhD,uGAAwD;AAExD,wBAAU,CAAC,QAAQ,GAAG,iBAAQ,CAAC;AAC/B,wBAAU,CAAC,QAAQ,GAAG,iBAAQ,CAAC;AAC/B,wBAAU,CAAC,EAAE,GAAG,WAAE,CAAC;AAEnB,MAAM,UAAU,GAAG,IAAI,wBAAU,CAAC;IAChC,eAAe,EAAE;QACf,aAAa,EAAE;YACb,YAAY,EAAE;gBACZ,GAAG,EAAE,IAAI;aACV;SACF;KACF;CACF,CAAC,CAAC;AAEH,UAAU,CAAC,GAAG,CAAC,6BAA6B,EAAE,qCAAI,EAAE;IAClD,KAAK,EAAE;QACL;YACE,IAAI,EAAE,kBAAkB;SACzB;QACD;YACE,IAAI,EAAE,iBAAiB;SACxB;QACD;YACE,IAAI,EAAE,iBAAiB;SACxB;QACD;YACE,IAAI,EAAE,cAAc;SACrB;KACF;IACD,OAAO,EAAE;QACP;YACE,uDAAuD;YACvD,IAAI,EAAE,uBAAuB;YAC7B,MAAM,EAAE;gBACN;oBACE,SAAS,EAAE,+BAA+B;oBAC1C,IAAI,EAAE,CAAC;oBACP,MAAM,EAAE,CAAC;oBACT,OAAO,EAAE,CAAC;oBACV,SAAS,EAAE,EAAE;iBACd;aACF;SACF;QACD;YACE,IAAI,EAAE,wBAAwB;YAC9B,MAAM,EAAE;gBACN;oBACE,SAAS,EAAE,+BAA+B;oBAC1C,IAAI,EAAE,CAAC;oBACP,MAAM,EAAE,CAAC;oBACT,OAAO,EAAE,CAAC;oBACV,SAAS,EAAE,EAAE;iBACd;aACF;SACF;KACF;CACF,CAAC,CAAC","sourcesContent":["import { RuleTester } from '@typescript-eslint/rule-tester';\nimport { afterAll, describe, it } from 'vitest';\n\nimport rule from '../rules/jsx-no-dollar-interpolation';\n\nRuleTester.afterAll = afterAll;\nRuleTester.describe = describe;\nRuleTester.it = it;\n\nconst ruleTester = new RuleTester({\n  languageOptions: {\n    parserOptions: {\n      ecmaFeatures: {\n        jsx: true,\n      },\n    },\n  },\n});\n\nruleTester.run('jsx-no-dollar-interpolation', rule, {\n  valid: [\n    {\n      code: '<div>hello</div>',\n    },\n    {\n      code: '<div>100$</div>',\n    },\n    {\n      code: '<div>$100</div>',\n    },\n    {\n      code: '<div>$</div>',\n    },\n  ],\n  invalid: [\n    {\n      // eslint-disable-next-line no-template-curly-in-string\n      code: '<div>${message}</div>',\n      errors: [\n        {\n          messageId: 'dollarInterpolationNotAllowed',\n          line: 1,\n          column: 6,\n          endLine: 1,\n          endColumn: 16,\n        },\n      ],\n    },\n    {\n      code: '<div>$ {message}</div>',\n      errors: [\n        {\n          messageId: 'dollarInterpolationNotAllowed',\n          line: 1,\n          column: 6,\n          endLine: 1,\n          endColumn: 17,\n        },\n      ],\n    },\n  ],\n});\n"]}
+{"version":3,"file":"jsx-no-dollar-interpolation.test.js","sourceRoot":"","sources":["../../src/tests/jsx-no-dollar-interpolation.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,gCAAgC,CAAC;AAC5D,OAAO,EAAE,QAAQ,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAEhD,OAAO,IAAI,MAAM,yCAAyC,CAAC;AAE3D,UAAU,CAAC,QAAQ,GAAG,QAAQ,CAAC;AAC/B,UAAU,CAAC,QAAQ,GAAG,QAAQ,CAAC;AAC/B,UAAU,CAAC,EAAE,GAAG,EAAE,CAAC;AAEnB,MAAM,UAAU,GAAG,IAAI,UAAU,CAAC;IAChC,eAAe,EAAE;QACf,aAAa,EAAE;YACb,YAAY,EAAE;gBACZ,GAAG,EAAE,IAAI;aACV;SACF;KACF;CACF,CAAC,CAAC;AAEH,UAAU,CAAC,GAAG,CAAC,6BAA6B,EAAE,IAAI,EAAE;IAClD,KAAK,EAAE;QACL;YACE,IAAI,EAAE,kBAAkB;SACzB;QACD;YACE,IAAI,EAAE,iBAAiB;SACxB;QACD;YACE,IAAI,EAAE,iBAAiB;SACxB;QACD;YACE,IAAI,EAAE,cAAc;SACrB;KACF;IACD,OAAO,EAAE;QACP;YACE,uDAAuD;YACvD,IAAI,EAAE,uBAAuB;YAC7B,MAAM,EAAE;gBACN;oBACE,SAAS,EAAE,+BAA+B;oBAC1C,IAAI,EAAE,CAAC;oBACP,MAAM,EAAE,CAAC;oBACT,OAAO,EAAE,CAAC;oBACV,SAAS,EAAE,EAAE;iBACd;aACF;SACF;QACD;YACE,IAAI,EAAE,wBAAwB;YAC9B,MAAM,EAAE;gBACN;oBACE,SAAS,EAAE,+BAA+B;oBAC1C,IAAI,EAAE,CAAC;oBACP,MAAM,EAAE,CAAC;oBACT,OAAO,EAAE,CAAC;oBACV,SAAS,EAAE,EAAE;iBACd;aACF;SACF;KACF;CACF,CAAC,CAAC","sourcesContent":["import { RuleTester } from '@typescript-eslint/rule-tester';\nimport { afterAll, describe, it } from 'vitest';\n\nimport rule from '../rules/jsx-no-dollar-interpolation.js';\n\nRuleTester.afterAll = afterAll;\nRuleTester.describe = describe;\nRuleTester.it = it;\n\nconst ruleTester = new RuleTester({\n  languageOptions: {\n    parserOptions: {\n      ecmaFeatures: {\n        jsx: true,\n      },\n    },\n  },\n});\n\nruleTester.run('jsx-no-dollar-interpolation', rule, {\n  valid: [\n    {\n      code: '<div>hello</div>',\n    },\n    {\n      code: '<div>100$</div>',\n    },\n    {\n      code: '<div>$100</div>',\n    },\n    {\n      code: '<div>$</div>',\n    },\n  ],\n  invalid: [\n    {\n      // eslint-disable-next-line no-template-curly-in-string\n      code: '<div>${message}</div>',\n      errors: [\n        {\n          messageId: 'dollarInterpolationNotAllowed',\n          line: 1,\n          column: 6,\n          endLine: 1,\n          endColumn: 16,\n        },\n      ],\n    },\n    {\n      code: '<div>$ {message}</div>',\n      errors: [\n        {\n          messageId: 'dollarInterpolationNotAllowed',\n          line: 1,\n          column: 6,\n          endLine: 1,\n          endColumn: 17,\n        },\n      ],\n    },\n  ],\n});\n"]}

package/dist/tests/no-current-target-in-callback.test.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"no-current-target-in-callback.test.d.ts","sourceRoot":"","sources":["../../src/tests/no-current-target-in-callback.test.ts"],"names":[],"mappings":"","sourcesContent":["import { RuleTester } from '@typescript-eslint/rule-tester';\nimport { afterAll, describe, it } from 'vitest';\n\nimport rule from '../rules/no-current-target-in-callback.js';\n\nRuleTester.afterAll = afterAll;\nRuleTester.describe = describe;\nRuleTester.it = it;\n\nconst ruleTester = new RuleTester({\n  languageOptions: {\n    parserOptions: {\n      ecmaFeatures: {\n        jsx: true,\n      },\n    },\n  },\n});\n\nruleTester.run('no-current-target-in-callback', rule, {\n  valid: [\n    // Direct access to currentTarget (no nesting) is fine\n    {\n      code: '<input onChange={(e) => console.log(e.currentTarget.value)} />',\n    },\n    // Destructuring at the handler level is the recommended pattern\n    {\n      code: '<input onChange={({ currentTarget }) => setChecks((c) => ({ ...c, value: currentTarget.checked }))} />',\n    },\n    // Using a local variable is fine\n    {\n      code: `<input onChange={(e) => {\n        const target = e.currentTarget;\n        setChecks((c) => ({ ...c, value: target.checked }));\n      }} />`,\n    },\n    // Non-event handlers are not checked\n    {\n      code: '<Component render={(e) => setFoo(() => e.currentTarget)} />',\n    },\n    // Direct setState without callback\n    {\n      code: '<input onChange={(e) => setChecks({ value: e.currentTarget.checked })} />',\n    },\n    // Event target (not currentTarget) - different issue, not covered by this rule\n    {\n      code: '<input onChange={(e) => setChecks((c) => ({ ...c, value: e.target.checked }))} />',\n    },\n  ],\n  invalid: [\n    // Basic case: accessing e.currentTarget inside setState callback\n    {\n      code: '<input onChange={(e) => setChecks((c) => ({ ...c, value: e.currentTarget.checked }))} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Multiple accesses\n    {\n      code: `<input onChange={(e) => setChecks((c) => ({\n        ...c,\n        value: e.currentTarget.checked,\n        name: e.currentTarget.name\n      }))} />`,\n      errors: [\n        { messageId: 'noCurrentTargetInCallback' },\n        { messageId: 'noCurrentTargetInCallback' },\n      ],\n    },\n    // Different event parameter names\n    {\n      code: '<input onChange={(event) => setChecks((c) => ({ ...c, value: event.currentTarget.checked }))} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    {\n      code: '<input onChange={(evt) => setChecks((c) => ({ ...c, value: evt.currentTarget.checked }))} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // onClick handler\n    {\n      code: '<button onClick={(e) => setFoo((prev) => ({ ...prev, clicked: e.currentTarget.id }))} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Nested arrow function in array method\n    {\n      code: '<input onChange={(e) => items.map(() => e.currentTarget.value)} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Function expression callback\n    {\n      code: '<input onChange={(e) => setChecks(function(c) { return { ...c, value: e.currentTarget.checked }; })} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Deeply nested\n    {\n      code: '<input onChange={(e) => outer(() => inner(() => e.currentTarget.value))} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // With block body\n    {\n      code: `<input onChange={(e) => {\n        setChecks((c) => {\n          return { ...c, value: e.currentTarget.checked };\n        });\n      }} />`,\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n  ],\n});\n"]}
+{"version":3,"file":"no-current-target-in-callback.test.d.ts","sourceRoot":"","sources":["../../src/tests/no-current-target-in-callback.test.ts"],"names":[],"mappings":"","sourcesContent":["import { RuleTester } from '@typescript-eslint/rule-tester';\nimport { afterAll, describe, it } from 'vitest';\n\nimport rule from '../rules/no-current-target-in-callback.js';\n\nRuleTester.afterAll = afterAll;\nRuleTester.describe = describe;\nRuleTester.it = it;\n\nconst ruleTester = new RuleTester({\n  languageOptions: {\n    parserOptions: {\n      ecmaFeatures: {\n        jsx: true,\n      },\n    },\n  },\n});\n\nruleTester.run('no-current-target-in-callback', rule, {\n  valid: [\n    // Direct access to currentTarget (no nesting) is fine\n    {\n      code: '<input onChange={(e) => console.log(e.currentTarget.value)} />',\n    },\n    // Destructuring at the handler level is the recommended pattern\n    {\n      code: '<input onChange={({ currentTarget }) => setChecks((c) => ({ ...c, value: currentTarget.checked }))} />',\n    },\n    // Using a local variable is fine\n    {\n      code: `<input onChange={(e) => {\n        const target = e.currentTarget;\n        setChecks((c) => ({ ...c, value: target.checked }));\n      }} />`,\n    },\n    // Non-event handlers are not checked\n    {\n      code: '<Component render={(e) => setFoo(() => e.currentTarget)} />',\n    },\n    // Synchronous nested callbacks are safe\n    {\n      code: '<input onChange={(e) => items.map(() => e.currentTarget.value)} />',\n    },\n    {\n      code: `const handlePaste = (e: React.ClipboardEvent<HTMLInputElement>) => {\n        fields.find((field) => field.name === e.currentTarget.name);\n      };`,\n    },\n    {\n      code: `const handlePaste = (e: React.ClipboardEvent<HTMLInputElement>) => {\n        const cursorPosition = run(() => e.currentTarget.selectionStart ?? 0);\n        return cursorPosition;\n      };`,\n    },\n    // Direct setState without callback\n    {\n      code: '<input onChange={(e) => setChecks({ value: e.currentTarget.checked })} />',\n    },\n    // Capturing currentTarget before await is safe\n    {\n      code: `const handleSubmit = async (e: React.SubmitEvent<HTMLFormElement>) => {\n        const form = e.currentTarget;\n        await trigger();\n        form.submit();\n      };`,\n    },\n    // Destructuring currentTarget before await is safe\n    {\n      code: `const handleSubmit = async ({ currentTarget }: React.SubmitEvent<HTMLFormElement>) => {\n        await trigger();\n        currentTarget.submit();\n      };`,\n    },\n    // Event target (not currentTarget) - different issue, not covered by this rule\n    {\n      code: '<input onChange={(e) => setChecks((c) => ({ ...c, value: e.target.checked }))} />',\n    },\n  ],\n  invalid: [\n    // Basic case: accessing e.currentTarget inside setState callback\n    {\n      code: '<input onChange={(e) => setChecks((c) => ({ ...c, value: e.currentTarget.checked }))} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Multiple accesses\n    {\n      code: `<input onChange={(e) => setChecks((c) => ({\n        ...c,\n        value: e.currentTarget.checked,\n        name: e.currentTarget.name\n      }))} />`,\n      errors: [\n        { messageId: 'noCurrentTargetInCallback' },\n        { messageId: 'noCurrentTargetInCallback' },\n      ],\n    },\n    // Different event parameter names\n    {\n      code: '<input onChange={(event) => setChecks((c) => ({ ...c, value: event.currentTarget.checked }))} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    {\n      code: '<input onChange={(evt) => setChecks((c) => ({ ...c, value: evt.currentTarget.checked }))} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // onClick handler\n    {\n      code: '<button onClick={(e) => setFoo((prev) => ({ ...prev, clicked: e.currentTarget.id }))} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Function expression callback\n    {\n      code: '<input onChange={(e) => setChecks(function(c) { return { ...c, value: e.currentTarget.checked }; })} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // With block body\n    {\n      code: `<input onChange={(e) => {\n        setChecks((c) => {\n          return { ...c, value: e.currentTarget.checked };\n        });\n      }} />`,\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Promise callbacks are deferred\n    {\n      code: `<input onChange={(e) => {\n        Promise.resolve().then(() => e.currentTarget.value);\n      }} />`,\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Timer callbacks are deferred\n    {\n      code: `<input onChange={(e) => {\n        setTimeout(() => e.currentTarget.value, 0);\n      }} />`,\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Access after await in a typed event handler\n    {\n      code: `const handleSubmit = async (e: React.SubmitEvent<HTMLFormElement>) => {\n        await trigger();\n        e.currentTarget.submit();\n      };`,\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Access after await in an inline JSX handler\n    {\n      code: `<form onSubmit={async (e) => {\n        await trigger();\n        e.currentTarget.submit();\n      }} />`,\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // react-hook-form handleSubmit callbacks are also after the original event boundary\n    {\n      code: `const onSubmit = handleSubmit((_data, e) => {\n        e.currentTarget.submit();\n      });`,\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n  ],\n});\n"]}

package/dist/tests/no-current-target-in-callback.test.js

@@ -1,15 +1,10 @@
-"use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-const rule_tester_1 = require("@typescript-eslint/rule-tester");
-const vitest_1 = require("vitest");
-const no_current_target_in_callback_js_1 = __importDefault(require("../rules/no-current-target-in-callback.js"));
-rule_tester_1.RuleTester.afterAll = vitest_1.afterAll;
-rule_tester_1.RuleTester.describe = vitest_1.describe;
-rule_tester_1.RuleTester.it = vitest_1.it;
-const ruleTester = new rule_tester_1.RuleTester({
+import { RuleTester } from '@typescript-eslint/rule-tester';
+import { afterAll, describe, it } from 'vitest';
+import rule from '../rules/no-current-target-in-callback.js';
+RuleTester.afterAll = afterAll;
+RuleTester.describe = describe;
+RuleTester.it = it;
+const ruleTester = new RuleTester({
     languageOptions: {
         parserOptions: {
             ecmaFeatures: {
@@ -18,7 +13,7 @@ const ruleTester = new rule_tester_1.RuleTester({
         },
     },
 });
-ruleTester.run('no-current-target-in-callback', no_current_target_in_callback_js_1.default, {
+ruleTester.run('no-current-target-in-callback', rule, {
     valid: [
         // Direct access to currentTarget (no nesting) is fine
         {
@@ -39,10 +34,40 @@ ruleTester.run('no-current-target-in-callback', no_current_target_in_callback_js
         {
             code: '<Component render={(e) => setFoo(() => e.currentTarget)} />',
         },
+        // Synchronous nested callbacks are safe
+        {
+            code: '<input onChange={(e) => items.map(() => e.currentTarget.value)} />',
+        },
+        {
+            code: `const handlePaste = (e: React.ClipboardEvent<HTMLInputElement>) => {
+        fields.find((field) => field.name === e.currentTarget.name);
+      };`,
+        },
+        {
+            code: `const handlePaste = (e: React.ClipboardEvent<HTMLInputElement>) => {
+        const cursorPosition = run(() => e.currentTarget.selectionStart ?? 0);
+        return cursorPosition;
+      };`,
+        },
         // Direct setState without callback
         {
             code: '<input onChange={(e) => setChecks({ value: e.currentTarget.checked })} />',
         },
+        // Capturing currentTarget before await is safe
+        {
+            code: `const handleSubmit = async (e: React.SubmitEvent<HTMLFormElement>) => {
+        const form = e.currentTarget;
+        await trigger();
+        form.submit();
+      };`,
+        },
+        // Destructuring currentTarget before await is safe
+        {
+            code: `const handleSubmit = async ({ currentTarget }: React.SubmitEvent<HTMLFormElement>) => {
+        await trigger();
+        currentTarget.submit();
+      };`,
+        },
         // Event target (not currentTarget) - different issue, not covered by this rule
         {
             code: '<input onChange={(e) => setChecks((c) => ({ ...c, value: e.target.checked }))} />',
@@ -80,21 +105,11 @@ ruleTester.run('no-current-target-in-callback', no_current_target_in_callback_js
             code: '<button onClick={(e) => setFoo((prev) => ({ ...prev, clicked: e.currentTarget.id }))} />',
             errors: [{ messageId: 'noCurrentTargetInCallback' }],
         },
-        // Nested arrow function in array method
-        {
-            code: '<input onChange={(e) => items.map(() => e.currentTarget.value)} />',
-            errors: [{ messageId: 'noCurrentTargetInCallback' }],
-        },
         // Function expression callback
         {
             code: '<input onChange={(e) => setChecks(function(c) { return { ...c, value: e.currentTarget.checked }; })} />',
             errors: [{ messageId: 'noCurrentTargetInCallback' }],
         },
-        // Deeply nested
-        {
-            code: '<input onChange={(e) => outer(() => inner(() => e.currentTarget.value))} />',
-            errors: [{ messageId: 'noCurrentTargetInCallback' }],
-        },
         // With block body
         {
             code: `<input onChange={(e) => {
@@ -104,6 +119,43 @@ ruleTester.run('no-current-target-in-callback', no_current_target_in_callback_js
       }} />`,
             errors: [{ messageId: 'noCurrentTargetInCallback' }],
         },
+        // Promise callbacks are deferred
+        {
+            code: `<input onChange={(e) => {
+        Promise.resolve().then(() => e.currentTarget.value);
+      }} />`,
+            errors: [{ messageId: 'noCurrentTargetInCallback' }],
+        },
+        // Timer callbacks are deferred
+        {
+            code: `<input onChange={(e) => {
+        setTimeout(() => e.currentTarget.value, 0);
+      }} />`,
+            errors: [{ messageId: 'noCurrentTargetInCallback' }],
+        },
+        // Access after await in a typed event handler
+        {
+            code: `const handleSubmit = async (e: React.SubmitEvent<HTMLFormElement>) => {
+        await trigger();
+        e.currentTarget.submit();
+      };`,
+            errors: [{ messageId: 'noCurrentTargetInCallback' }],
+        },
+        // Access after await in an inline JSX handler
+        {
+            code: `<form onSubmit={async (e) => {
+        await trigger();
+        e.currentTarget.submit();
+      }} />`,
+            errors: [{ messageId: 'noCurrentTargetInCallback' }],
+        },
+        // react-hook-form handleSubmit callbacks are also after the original event boundary
+        {
+            code: `const onSubmit = handleSubmit((_data, e) => {
+        e.currentTarget.submit();
+      });`,
+            errors: [{ messageId: 'noCurrentTargetInCallback' }],
+        },
     ],
 });
 //# sourceMappingURL=no-current-target-in-callback.test.js.map

package/dist/tests/no-current-target-in-callback.test.js.map

@@ -1 +1 @@
-{"version":3,"file":"no-current-target-in-callback.test.js","sourceRoot":"","sources":["../../src/tests/no-current-target-in-callback.test.ts"],"names":[],"mappings":";;;;;AAAA,gEAA4D;AAC5D,mCAAgD;AAEhD,iHAA6D;AAE7D,wBAAU,CAAC,QAAQ,GAAG,iBAAQ,CAAC;AAC/B,wBAAU,CAAC,QAAQ,GAAG,iBAAQ,CAAC;AAC/B,wBAAU,CAAC,EAAE,GAAG,WAAE,CAAC;AAEnB,MAAM,UAAU,GAAG,IAAI,wBAAU,CAAC;IAChC,eAAe,EAAE;QACf,aAAa,EAAE;YACb,YAAY,EAAE;gBACZ,GAAG,EAAE,IAAI;aACV;SACF;KACF;CACF,CAAC,CAAC;AAEH,UAAU,CAAC,GAAG,CAAC,+BAA+B,EAAE,0CAAI,EAAE;IACpD,KAAK,EAAE;QACL,sDAAsD;QACtD;YACE,IAAI,EAAE,gEAAgE;SACvE;QACD,gEAAgE;QAChE;YACE,IAAI,EAAE,wGAAwG;SAC/G;QACD,iCAAiC;QACjC;YACE,IAAI,EAAE;;;YAGA;SACP;QACD,qCAAqC;QACrC;YACE,IAAI,EAAE,6DAA6D;SACpE;QACD,mCAAmC;QACnC;YACE,IAAI,EAAE,2EAA2E;SAClF;QACD,+EAA+E;QAC/E;YACE,IAAI,EAAE,mFAAmF;SAC1F;KACF;IACD,OAAO,EAAE;QACP,iEAAiE;QACjE;YACE,IAAI,EAAE,0FAA0F;YAChG,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;QACD,oBAAoB;QACpB;YACE,IAAI,EAAE;;;;cAIE;YACR,MAAM,EAAE;gBACN,EAAE,SAAS,EAAE,2BAA2B,EAAE;gBAC1C,EAAE,SAAS,EAAE,2BAA2B,EAAE;aAC3C;SACF;QACD,kCAAkC;QAClC;YACE,IAAI,EAAE,kGAAkG;YACxG,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;QACD;YACE,IAAI,EAAE,8FAA8F;YACpG,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;QACD,kBAAkB;QAClB;YACE,IAAI,EAAE,0FAA0F;YAChG,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;QACD,wCAAwC;QACxC;YACE,IAAI,EAAE,oEAAoE;YAC1E,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;QACD,+BAA+B;QAC/B;YACE,IAAI,EAAE,yGAAyG;YAC/G,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;QACD,gBAAgB;QAChB;YACE,IAAI,EAAE,6EAA6E;YACnF,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;QACD,kBAAkB;QAClB;YACE,IAAI,EAAE;;;;YAIA;YACN,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;KACF;CACF,CAAC,CAAC","sourcesContent":["import { RuleTester } from '@typescript-eslint/rule-tester';\nimport { afterAll, describe, it } from 'vitest';\n\nimport rule from '../rules/no-current-target-in-callback.js';\n\nRuleTester.afterAll = afterAll;\nRuleTester.describe = describe;\nRuleTester.it = it;\n\nconst ruleTester = new RuleTester({\n  languageOptions: {\n    parserOptions: {\n      ecmaFeatures: {\n        jsx: true,\n      },\n    },\n  },\n});\n\nruleTester.run('no-current-target-in-callback', rule, {\n  valid: [\n    // Direct access to currentTarget (no nesting) is fine\n    {\n      code: '<input onChange={(e) => console.log(e.currentTarget.value)} />',\n    },\n    // Destructuring at the handler level is the recommended pattern\n    {\n      code: '<input onChange={({ currentTarget }) => setChecks((c) => ({ ...c, value: currentTarget.checked }))} />',\n    },\n    // Using a local variable is fine\n    {\n      code: `<input onChange={(e) => {\n        const target = e.currentTarget;\n        setChecks((c) => ({ ...c, value: target.checked }));\n      }} />`,\n    },\n    // Non-event handlers are not checked\n    {\n      code: '<Component render={(e) => setFoo(() => e.currentTarget)} />',\n    },\n    // Direct setState without callback\n    {\n      code: '<input onChange={(e) => setChecks({ value: e.currentTarget.checked })} />',\n    },\n    // Event target (not currentTarget) - different issue, not covered by this rule\n    {\n      code: '<input onChange={(e) => setChecks((c) => ({ ...c, value: e.target.checked }))} />',\n    },\n  ],\n  invalid: [\n    // Basic case: accessing e.currentTarget inside setState callback\n    {\n      code: '<input onChange={(e) => setChecks((c) => ({ ...c, value: e.currentTarget.checked }))} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Multiple accesses\n    {\n      code: `<input onChange={(e) => setChecks((c) => ({\n        ...c,\n        value: e.currentTarget.checked,\n        name: e.currentTarget.name\n      }))} />`,\n      errors: [\n        { messageId: 'noCurrentTargetInCallback' },\n        { messageId: 'noCurrentTargetInCallback' },\n      ],\n    },\n    // Different event parameter names\n    {\n      code: '<input onChange={(event) => setChecks((c) => ({ ...c, value: event.currentTarget.checked }))} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    {\n      code: '<input onChange={(evt) => setChecks((c) => ({ ...c, value: evt.currentTarget.checked }))} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // onClick handler\n    {\n      code: '<button onClick={(e) => setFoo((prev) => ({ ...prev, clicked: e.currentTarget.id }))} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Nested arrow function in array method\n    {\n      code: '<input onChange={(e) => items.map(() => e.currentTarget.value)} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Function expression callback\n    {\n      code: '<input onChange={(e) => setChecks(function(c) { return { ...c, value: e.currentTarget.checked }; })} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Deeply nested\n    {\n      code: '<input onChange={(e) => outer(() => inner(() => e.currentTarget.value))} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // With block body\n    {\n      code: `<input onChange={(e) => {\n        setChecks((c) => {\n          return { ...c, value: e.currentTarget.checked };\n        });\n      }} />`,\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n  ],\n});\n"]}
+{"version":3,"file":"no-current-target-in-callback.test.js","sourceRoot":"","sources":["../../src/tests/no-current-target-in-callback.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,gCAAgC,CAAC;AAC5D,OAAO,EAAE,QAAQ,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAEhD,OAAO,IAAI,MAAM,2CAA2C,CAAC;AAE7D,UAAU,CAAC,QAAQ,GAAG,QAAQ,CAAC;AAC/B,UAAU,CAAC,QAAQ,GAAG,QAAQ,CAAC;AAC/B,UAAU,CAAC,EAAE,GAAG,EAAE,CAAC;AAEnB,MAAM,UAAU,GAAG,IAAI,UAAU,CAAC;IAChC,eAAe,EAAE;QACf,aAAa,EAAE;YACb,YAAY,EAAE;gBACZ,GAAG,EAAE,IAAI;aACV;SACF;KACF;CACF,CAAC,CAAC;AAEH,UAAU,CAAC,GAAG,CAAC,+BAA+B,EAAE,IAAI,EAAE;IACpD,KAAK,EAAE;QACL,sDAAsD;QACtD;YACE,IAAI,EAAE,gEAAgE;SACvE;QACD,gEAAgE;QAChE;YACE,IAAI,EAAE,wGAAwG;SAC/G;QACD,iCAAiC;QACjC;YACE,IAAI,EAAE;;;YAGA;SACP;QACD,qCAAqC;QACrC;YACE,IAAI,EAAE,6DAA6D;SACpE;QACD,wCAAwC;QACxC;YACE,IAAI,EAAE,oEAAoE;SAC3E;QACD;YACE,IAAI,EAAE;;SAEH;SACJ;QACD;YACE,IAAI,EAAE;;;SAGH;SACJ;QACD,mCAAmC;QACnC;YACE,IAAI,EAAE,2EAA2E;SAClF;QACD,+CAA+C;QAC/C;YACE,IAAI,EAAE;;;;SAIH;SACJ;QACD,mDAAmD;QACnD;YACE,IAAI,EAAE;;;SAGH;SACJ;QACD,+EAA+E;QAC/E;YACE,IAAI,EAAE,mFAAmF;SAC1F;KACF;IACD,OAAO,EAAE;QACP,iEAAiE;QACjE;YACE,IAAI,EAAE,0FAA0F;YAChG,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;QACD,oBAAoB;QACpB;YACE,IAAI,EAAE;;;;cAIE;YACR,MAAM,EAAE;gBACN,EAAE,SAAS,EAAE,2BAA2B,EAAE;gBAC1C,EAAE,SAAS,EAAE,2BAA2B,EAAE;aAC3C;SACF;QACD,kCAAkC;QAClC;YACE,IAAI,EAAE,kGAAkG;YACxG,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;QACD;YACE,IAAI,EAAE,8FAA8F;YACpG,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;QACD,kBAAkB;QAClB;YACE,IAAI,EAAE,0FAA0F;YAChG,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;QACD,+BAA+B;QAC/B;YACE,IAAI,EAAE,yGAAyG;YAC/G,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;QACD,kBAAkB;QAClB;YACE,IAAI,EAAE;;;;YAIA;YACN,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;QACD,iCAAiC;QACjC;YACE,IAAI,EAAE;;YAEA;YACN,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;QACD,+BAA+B;QAC/B;YACE,IAAI,EAAE;;YAEA;YACN,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;QACD,8CAA8C;QAC9C;YACE,IAAI,EAAE;;;SAGH;YACH,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;QACD,8CAA8C;QAC9C;YACE,IAAI,EAAE;;;YAGA;YACN,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;QACD,oFAAoF;QACpF;YACE,IAAI,EAAE;;UAEF;YACJ,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,2BAA2B,EAAE,CAAC;SACrD;KACF;CACF,CAAC,CAAC","sourcesContent":["import { RuleTester } from '@typescript-eslint/rule-tester';\nimport { afterAll, describe, it } from 'vitest';\n\nimport rule from '../rules/no-current-target-in-callback.js';\n\nRuleTester.afterAll = afterAll;\nRuleTester.describe = describe;\nRuleTester.it = it;\n\nconst ruleTester = new RuleTester({\n  languageOptions: {\n    parserOptions: {\n      ecmaFeatures: {\n        jsx: true,\n      },\n    },\n  },\n});\n\nruleTester.run('no-current-target-in-callback', rule, {\n  valid: [\n    // Direct access to currentTarget (no nesting) is fine\n    {\n      code: '<input onChange={(e) => console.log(e.currentTarget.value)} />',\n    },\n    // Destructuring at the handler level is the recommended pattern\n    {\n      code: '<input onChange={({ currentTarget }) => setChecks((c) => ({ ...c, value: currentTarget.checked }))} />',\n    },\n    // Using a local variable is fine\n    {\n      code: `<input onChange={(e) => {\n        const target = e.currentTarget;\n        setChecks((c) => ({ ...c, value: target.checked }));\n      }} />`,\n    },\n    // Non-event handlers are not checked\n    {\n      code: '<Component render={(e) => setFoo(() => e.currentTarget)} />',\n    },\n    // Synchronous nested callbacks are safe\n    {\n      code: '<input onChange={(e) => items.map(() => e.currentTarget.value)} />',\n    },\n    {\n      code: `const handlePaste = (e: React.ClipboardEvent<HTMLInputElement>) => {\n        fields.find((field) => field.name === e.currentTarget.name);\n      };`,\n    },\n    {\n      code: `const handlePaste = (e: React.ClipboardEvent<HTMLInputElement>) => {\n        const cursorPosition = run(() => e.currentTarget.selectionStart ?? 0);\n        return cursorPosition;\n      };`,\n    },\n    // Direct setState without callback\n    {\n      code: '<input onChange={(e) => setChecks({ value: e.currentTarget.checked })} />',\n    },\n    // Capturing currentTarget before await is safe\n    {\n      code: `const handleSubmit = async (e: React.SubmitEvent<HTMLFormElement>) => {\n        const form = e.currentTarget;\n        await trigger();\n        form.submit();\n      };`,\n    },\n    // Destructuring currentTarget before await is safe\n    {\n      code: `const handleSubmit = async ({ currentTarget }: React.SubmitEvent<HTMLFormElement>) => {\n        await trigger();\n        currentTarget.submit();\n      };`,\n    },\n    // Event target (not currentTarget) - different issue, not covered by this rule\n    {\n      code: '<input onChange={(e) => setChecks((c) => ({ ...c, value: e.target.checked }))} />',\n    },\n  ],\n  invalid: [\n    // Basic case: accessing e.currentTarget inside setState callback\n    {\n      code: '<input onChange={(e) => setChecks((c) => ({ ...c, value: e.currentTarget.checked }))} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Multiple accesses\n    {\n      code: `<input onChange={(e) => setChecks((c) => ({\n        ...c,\n        value: e.currentTarget.checked,\n        name: e.currentTarget.name\n      }))} />`,\n      errors: [\n        { messageId: 'noCurrentTargetInCallback' },\n        { messageId: 'noCurrentTargetInCallback' },\n      ],\n    },\n    // Different event parameter names\n    {\n      code: '<input onChange={(event) => setChecks((c) => ({ ...c, value: event.currentTarget.checked }))} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    {\n      code: '<input onChange={(evt) => setChecks((c) => ({ ...c, value: evt.currentTarget.checked }))} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // onClick handler\n    {\n      code: '<button onClick={(e) => setFoo((prev) => ({ ...prev, clicked: e.currentTarget.id }))} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Function expression callback\n    {\n      code: '<input onChange={(e) => setChecks(function(c) { return { ...c, value: e.currentTarget.checked }; })} />',\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // With block body\n    {\n      code: `<input onChange={(e) => {\n        setChecks((c) => {\n          return { ...c, value: e.currentTarget.checked };\n        });\n      }} />`,\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Promise callbacks are deferred\n    {\n      code: `<input onChange={(e) => {\n        Promise.resolve().then(() => e.currentTarget.value);\n      }} />`,\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Timer callbacks are deferred\n    {\n      code: `<input onChange={(e) => {\n        setTimeout(() => e.currentTarget.value, 0);\n      }} />`,\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Access after await in a typed event handler\n    {\n      code: `const handleSubmit = async (e: React.SubmitEvent<HTMLFormElement>) => {\n        await trigger();\n        e.currentTarget.submit();\n      };`,\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // Access after await in an inline JSX handler\n    {\n      code: `<form onSubmit={async (e) => {\n        await trigger();\n        e.currentTarget.submit();\n      }} />`,\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n    // react-hook-form handleSubmit callbacks are also after the original event boundary\n    {\n      code: `const onSubmit = handleSubmit((_data, e) => {\n        e.currentTarget.submit();\n      });`,\n      errors: [{ messageId: 'noCurrentTargetInCallback' }],\n    },\n  ],\n});\n"]}

package/dist/tests/no-hydrate-reslocals.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=no-hydrate-reslocals.test.d.ts.map

package/dist/tests/no-hydrate-reslocals.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-hydrate-reslocals.test.d.ts","sourceRoot":"","sources":["../../src/tests/no-hydrate-reslocals.test.ts"],"names":[],"mappings":"","sourcesContent":["import { RuleTester } from '@typescript-eslint/rule-tester';\nimport { afterAll, describe, it } from 'vitest';\n\nimport rule from '../rules/no-hydrate-reslocals.js';\n\nRuleTester.afterAll = afterAll;\nRuleTester.describe = describe;\nRuleTester.it = it;\n\nconst ruleTester = new RuleTester({\n  languageOptions: {\n    parserOptions: {\n      ecmaFeatures: {\n        jsx: true,\n      },\n    },\n  },\n});\n\nruleTester.run('no-hydrate-reslocals', rule, {\n  valid: [\n    {\n      code: '<Hydrate><Child foo={1} bar=\"baz\" /></Hydrate>',\n    },\n    {\n      code: '<Hydrate><Child course={course} assessment={assessment} /></Hydrate>',\n    },\n    {\n      code: '<SomeOther resLocals={res.locals}><Child /></SomeOther>',\n    },\n    {\n      code: '<Hydrate><Child {...otherProps} /></Hydrate>',\n    },\n    {\n      code: '<Hydrate resLocals={foo}><Child /></Hydrate>',\n    },\n    {\n      code: 'hydrateHtml(<Child foo={1} bar=\"baz\" />)',\n    },\n    {\n      code: 'hydrateHtml(<Child {...otherProps} />)',\n    },\n    {\n      code: 'otherFunction(<Child resLocals={res.locals} />)',\n    },\n  ],\n  invalid: [\n    {\n      code: '<Hydrate><Child resLocals={res.locals} /></Hydrate>',\n      errors: [{ messageId: 'forbiddenProp', data: { name: 'resLocals' } }],\n    },\n    {\n      code: '<Hydrate><Child locals={res.locals} /></Hydrate>',\n      errors: [{ messageId: 'forbiddenProp', data: { name: 'locals' } }],\n    },\n    {\n      code: '<Hydrate><Child {...res.locals} /></Hydrate>',\n      errors: [{ messageId: 'forbiddenSpread' }],\n    },\n    {\n      code: '<Hydrate><Child {...resLocals} /></Hydrate>',\n      errors: [{ messageId: 'forbiddenSpread' }],\n    },\n    {\n      code: '<Hydrate><Child {...locals} /></Hydrate>',\n      errors: [{ messageId: 'forbiddenSpread' }],\n    },\n    {\n      code: '<Hydrate fullHeight><Child foo={1} resLocals={res.locals} /></Hydrate>',\n      errors: [{ messageId: 'forbiddenProp', data: { name: 'resLocals' } }],\n    },\n    {\n      code: 'hydrateHtml(<Child resLocals={res.locals} />)',\n      errors: [{ messageId: 'forbiddenProp', data: { name: 'resLocals' } }],\n    },\n    {\n      code: 'hydrateHtml(<Child locals={res.locals} />)',\n      errors: [{ messageId: 'forbiddenProp', data: { name: 'locals' } }],\n    },\n    {\n      code: 'hydrateHtml(<Child {...res.locals} />)',\n      errors: [{ messageId: 'forbiddenSpread' }],\n    },\n    {\n      code: 'hydrateHtml(<Child {...resLocals} />, { fullHeight: true })',\n      errors: [{ messageId: 'forbiddenSpread' }],\n    },\n  ],\n});\n"]}

package/dist/tests/no-hydrate-reslocals.test.js

@@ -0,0 +1,86 @@
+import { RuleTester } from '@typescript-eslint/rule-tester';
+import { afterAll, describe, it } from 'vitest';
+import rule from '../rules/no-hydrate-reslocals.js';
+RuleTester.afterAll = afterAll;
+RuleTester.describe = describe;
+RuleTester.it = it;
+const ruleTester = new RuleTester({
+    languageOptions: {
+        parserOptions: {
+            ecmaFeatures: {
+                jsx: true,
+            },
+        },
+    },
+});
+ruleTester.run('no-hydrate-reslocals', rule, {
+    valid: [
+        {
+            code: '<Hydrate><Child foo={1} bar="baz" /></Hydrate>',
+        },
+        {
+            code: '<Hydrate><Child course={course} assessment={assessment} /></Hydrate>',
+        },
+        {
+            code: '<SomeOther resLocals={res.locals}><Child /></SomeOther>',
+        },
+        {
+            code: '<Hydrate><Child {...otherProps} /></Hydrate>',
+        },
+        {
+            code: '<Hydrate resLocals={foo}><Child /></Hydrate>',
+        },
+        {
+            code: 'hydrateHtml(<Child foo={1} bar="baz" />)',
+        },
+        {
+            code: 'hydrateHtml(<Child {...otherProps} />)',
+        },
+        {
+            code: 'otherFunction(<Child resLocals={res.locals} />)',
+        },
+    ],
+    invalid: [
+        {
+            code: '<Hydrate><Child resLocals={res.locals} /></Hydrate>',
+            errors: [{ messageId: 'forbiddenProp', data: { name: 'resLocals' } }],
+        },
+        {
+            code: '<Hydrate><Child locals={res.locals} /></Hydrate>',
+            errors: [{ messageId: 'forbiddenProp', data: { name: 'locals' } }],
+        },
+        {
+            code: '<Hydrate><Child {...res.locals} /></Hydrate>',
+            errors: [{ messageId: 'forbiddenSpread' }],
+        },
+        {
+            code: '<Hydrate><Child {...resLocals} /></Hydrate>',
+            errors: [{ messageId: 'forbiddenSpread' }],
+        },
+        {
+            code: '<Hydrate><Child {...locals} /></Hydrate>',
+            errors: [{ messageId: 'forbiddenSpread' }],
+        },
+        {
+            code: '<Hydrate fullHeight><Child foo={1} resLocals={res.locals} /></Hydrate>',
+            errors: [{ messageId: 'forbiddenProp', data: { name: 'resLocals' } }],
+        },
+        {
+            code: 'hydrateHtml(<Child resLocals={res.locals} />)',
+            errors: [{ messageId: 'forbiddenProp', data: { name: 'resLocals' } }],
+        },
+        {
+            code: 'hydrateHtml(<Child locals={res.locals} />)',
+            errors: [{ messageId: 'forbiddenProp', data: { name: 'locals' } }],
+        },
+        {
+            code: 'hydrateHtml(<Child {...res.locals} />)',
+            errors: [{ messageId: 'forbiddenSpread' }],
+        },
+        {
+            code: 'hydrateHtml(<Child {...resLocals} />, { fullHeight: true })',
+            errors: [{ messageId: 'forbiddenSpread' }],
+        },
+    ],
+});
+//# sourceMappingURL=no-hydrate-reslocals.test.js.map

package/dist/tests/no-hydrate-reslocals.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-hydrate-reslocals.test.js","sourceRoot":"","sources":["../../src/tests/no-hydrate-reslocals.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,gCAAgC,CAAC;AAC5D,OAAO,EAAE,QAAQ,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAEhD,OAAO,IAAI,MAAM,kCAAkC,CAAC;AAEpD,UAAU,CAAC,QAAQ,GAAG,QAAQ,CAAC;AAC/B,UAAU,CAAC,QAAQ,GAAG,QAAQ,CAAC;AAC/B,UAAU,CAAC,EAAE,GAAG,EAAE,CAAC;AAEnB,MAAM,UAAU,GAAG,IAAI,UAAU,CAAC;IAChC,eAAe,EAAE;QACf,aAAa,EAAE;YACb,YAAY,EAAE;gBACZ,GAAG,EAAE,IAAI;aACV;SACF;KACF;CACF,CAAC,CAAC;AAEH,UAAU,CAAC,GAAG,CAAC,sBAAsB,EAAE,IAAI,EAAE;IAC3C,KAAK,EAAE;QACL;YACE,IAAI,EAAE,gDAAgD;SACvD;QACD;YACE,IAAI,EAAE,sEAAsE;SAC7E;QACD;YACE,IAAI,EAAE,yDAAyD;SAChE;QACD;YACE,IAAI,EAAE,8CAA8C;SACrD;QACD;YACE,IAAI,EAAE,8CAA8C;SACrD;QACD;YACE,IAAI,EAAE,0CAA0C;SACjD;QACD;YACE,IAAI,EAAE,wCAAwC;SAC/C;QACD;YACE,IAAI,EAAE,iDAAiD;SACxD;KACF;IACD,OAAO,EAAE;QACP;YACE,IAAI,EAAE,qDAAqD;YAC3D,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,eAAe,EAAE,IAAI,EAAE,EAAE,IAAI,EAAE,WAAW,EAAE,EAAE,CAAC;SACtE;QACD;YACE,IAAI,EAAE,kDAAkD;YACxD,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,eAAe,EAAE,IAAI,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE,EAAE,CAAC;SACnE;QACD;YACE,IAAI,EAAE,8CAA8C;YACpD,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,iBAAiB,EAAE,CAAC;SAC3C;QACD;YACE,IAAI,EAAE,6CAA6C;YACnD,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,iBAAiB,EAAE,CAAC;SAC3C;QACD;YACE,IAAI,EAAE,0CAA0C;YAChD,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,iBAAiB,EAAE,CAAC;SAC3C;QACD;YACE,IAAI,EAAE,wEAAwE;YAC9E,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,eAAe,EAAE,IAAI,EAAE,EAAE,IAAI,EAAE,WAAW,EAAE,EAAE,CAAC;SACtE;QACD;YACE,IAAI,EAAE,+CAA+C;YACrD,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,eAAe,EAAE,IAAI,EAAE,EAAE,IAAI,EAAE,WAAW,EAAE,EAAE,CAAC;SACtE;QACD;YACE,IAAI,EAAE,4CAA4C;YAClD,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,eAAe,EAAE,IAAI,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE,EAAE,CAAC;SACnE;QACD;YACE,IAAI,EAAE,wCAAwC;YAC9C,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,iBAAiB,EAAE,CAAC;SAC3C;QACD;YACE,IAAI,EAAE,6DAA6D;YACnE,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,iBAAiB,EAAE,CAAC;SAC3C;KACF;CACF,CAAC,CAAC","sourcesContent":["import { RuleTester } from '@typescript-eslint/rule-tester';\nimport { afterAll, describe, it } from 'vitest';\n\nimport rule from '../rules/no-hydrate-reslocals.js';\n\nRuleTester.afterAll = afterAll;\nRuleTester.describe = describe;\nRuleTester.it = it;\n\nconst ruleTester = new RuleTester({\n  languageOptions: {\n    parserOptions: {\n      ecmaFeatures: {\n        jsx: true,\n      },\n    },\n  },\n});\n\nruleTester.run('no-hydrate-reslocals', rule, {\n  valid: [\n    {\n      code: '<Hydrate><Child foo={1} bar=\"baz\" /></Hydrate>',\n    },\n    {\n      code: '<Hydrate><Child course={course} assessment={assessment} /></Hydrate>',\n    },\n    {\n      code: '<SomeOther resLocals={res.locals}><Child /></SomeOther>',\n    },\n    {\n      code: '<Hydrate><Child {...otherProps} /></Hydrate>',\n    },\n    {\n      code: '<Hydrate resLocals={foo}><Child /></Hydrate>',\n    },\n    {\n      code: 'hydrateHtml(<Child foo={1} bar=\"baz\" />)',\n    },\n    {\n      code: 'hydrateHtml(<Child {...otherProps} />)',\n    },\n    {\n      code: 'otherFunction(<Child resLocals={res.locals} />)',\n    },\n  ],\n  invalid: [\n    {\n      code: '<Hydrate><Child resLocals={res.locals} /></Hydrate>',\n      errors: [{ messageId: 'forbiddenProp', data: { name: 'resLocals' } }],\n    },\n    {\n      code: '<Hydrate><Child locals={res.locals} /></Hydrate>',\n      errors: [{ messageId: 'forbiddenProp', data: { name: 'locals' } }],\n    },\n    {\n      code: '<Hydrate><Child {...res.locals} /></Hydrate>',\n      errors: [{ messageId: 'forbiddenSpread' }],\n    },\n    {\n      code: '<Hydrate><Child {...resLocals} /></Hydrate>',\n      errors: [{ messageId: 'forbiddenSpread' }],\n    },\n    {\n      code: '<Hydrate><Child {...locals} /></Hydrate>',\n      errors: [{ messageId: 'forbiddenSpread' }],\n    },\n    {\n      code: '<Hydrate fullHeight><Child foo={1} resLocals={res.locals} /></Hydrate>',\n      errors: [{ messageId: 'forbiddenProp', data: { name: 'resLocals' } }],\n    },\n    {\n      code: 'hydrateHtml(<Child resLocals={res.locals} />)',\n      errors: [{ messageId: 'forbiddenProp', data: { name: 'resLocals' } }],\n    },\n    {\n      code: 'hydrateHtml(<Child locals={res.locals} />)',\n      errors: [{ messageId: 'forbiddenProp', data: { name: 'locals' } }],\n    },\n    {\n      code: 'hydrateHtml(<Child {...res.locals} />)',\n      errors: [{ messageId: 'forbiddenSpread' }],\n    },\n    {\n      code: 'hydrateHtml(<Child {...resLocals} />, { fullHeight: true })',\n      errors: [{ messageId: 'forbiddenSpread' }],\n    },\n  ],\n});\n"]}

package/dist/tests/require-trpc-permission-middleware.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=require-trpc-permission-middleware.test.d.ts.map

package/dist/tests/require-trpc-permission-middleware.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"require-trpc-permission-middleware.test.d.ts","sourceRoot":"","sources":["../../src/tests/require-trpc-permission-middleware.test.ts"],"names":[],"mappings":"","sourcesContent":["import { RuleTester } from '@typescript-eslint/rule-tester';\nimport { afterAll, describe, it } from 'vitest';\n\nimport rule from '../rules/require-trpc-permission-middleware.js';\n\nRuleTester.afterAll = afterAll;\nRuleTester.describe = describe;\nRuleTester.it = it;\n\nconst ruleTester = new RuleTester();\n\nruleTester.run('require-trpc-permission-middleware', rule, {\n  valid: [\n    {\n      code: 'const list = t.procedure.use(requireCoursePermissionEdit).query(async () => {});',\n    },\n    {\n      code: 'const m = t.procedure.use(requireCoursePermissionPreview).mutation(async () => {});',\n    },\n    {\n      code: `const m = t.procedure\n        .use(requireEnhancedAccessControl)\n        .use(requireCourseInstancePermissionView)\n        .query(async () => {});`,\n    },\n    {\n      code: 'const m = t.procedure.use(requireAdministrator).mutation(async () => {});',\n    },\n    {\n      code: `const m = t.procedure\n        .use(requireCourseInstancePermissionEdit)\n        .input(z.object({}))\n        .mutation(async () => {});`,\n    },\n    {\n      code: 'const list = t.procedure.use(requireCoursePermissionOwn).query(async () => {});',\n    },\n    // Not a t.procedure chain — should be ignored\n    {\n      code: 'const fn = somethingElse.query(async () => {});',\n    },\n    {\n      code: 'const fn = otherRouter.procedure.query(async () => {});',\n    },\n    // Intermediate procedure-base variable: permission middleware on the base\n    {\n      code: `\n        const protectedProcedure = t.procedure.use(requireCoursePermissionOwn);\n        const m = protectedProcedure.query(async () => {});\n      `,\n    },\n    // Intermediate procedure-base variable: permission added downstream\n    {\n      code: `\n        const featureProcedure = t.procedure.use(requireAiGradingFeature);\n        const m = featureProcedure.use(requireAdministrator).mutation(async () => {});\n      `,\n    },\n    // Imported procedure base — we can't follow across files, so don't fire\n    {\n      code: `\n        import { protectedProcedure } from './foo.js';\n        const m = protectedProcedure.query(async () => {});\n      `,\n      languageOptions: { parserOptions: { sourceType: 'module' } },\n    },\n  ],\n  invalid: [\n    {\n      code: 'const list = t.procedure.query(async () => {});',\n      errors: [{ messageId: 'missingPermissionMiddleware' }],\n    },\n    {\n      code: 'const m = t.procedure.input(z.object({})).mutation(async () => {});',\n      errors: [{ messageId: 'missingPermissionMiddleware' }],\n    },\n    // Feature gate alone is not a permission gate\n    {\n      code: 'const m = t.procedure.use(requireEnhancedAccessControl).mutation(async () => {});',\n      errors: [{ messageId: 'missingPermissionMiddleware' }],\n    },\n    {\n      code: 'const m = t.procedure.use(requireAiGradingFeature).mutation(async () => {});',\n      errors: [{ messageId: 'missingPermissionMiddleware' }],\n    },\n    {\n      code: `const m = t.procedure\n        .input(z.object({}))\n        .output(z.object({}))\n        .subscription(async () => {});`,\n      errors: [{ messageId: 'missingPermissionMiddleware' }],\n    },\n    // Intermediate procedure-base variable with only a feature gate is still a miss\n    {\n      code: `\n        const featureProcedure = t.procedure.use(requireAiGradingFeature);\n        const m = featureProcedure.query(async () => {});\n      `,\n      errors: [{ messageId: 'missingPermissionMiddleware' }],\n    },\n    {\n      code: `\n        const featureProcedure = t.procedure.use(requireAiGradingFeature);\n        const m = featureProcedure.input(z.object({})).mutation(async () => {});\n      `,\n      errors: [{ messageId: 'missingPermissionMiddleware' }],\n    },\n  ],\n});\n"]}

package/dist/tests/require-trpc-permission-middleware.test.js

@@ -0,0 +1,106 @@
+import { RuleTester } from '@typescript-eslint/rule-tester';
+import { afterAll, describe, it } from 'vitest';
+import rule from '../rules/require-trpc-permission-middleware.js';
+RuleTester.afterAll = afterAll;
+RuleTester.describe = describe;
+RuleTester.it = it;
+const ruleTester = new RuleTester();
+ruleTester.run('require-trpc-permission-middleware', rule, {
+    valid: [
+        {
+            code: 'const list = t.procedure.use(requireCoursePermissionEdit).query(async () => {});',
+        },
+        {
+            code: 'const m = t.procedure.use(requireCoursePermissionPreview).mutation(async () => {});',
+        },
+        {
+            code: `const m = t.procedure
+        .use(requireEnhancedAccessControl)
+        .use(requireCourseInstancePermissionView)
+        .query(async () => {});`,
+        },
+        {
+            code: 'const m = t.procedure.use(requireAdministrator).mutation(async () => {});',
+        },
+        {
+            code: `const m = t.procedure
+        .use(requireCourseInstancePermissionEdit)
+        .input(z.object({}))
+        .mutation(async () => {});`,
+        },
+        {
+            code: 'const list = t.procedure.use(requireCoursePermissionOwn).query(async () => {});',
+        },
+        // Not a t.procedure chain — should be ignored
+        {
+            code: 'const fn = somethingElse.query(async () => {});',
+        },
+        {
+            code: 'const fn = otherRouter.procedure.query(async () => {});',
+        },
+        // Intermediate procedure-base variable: permission middleware on the base
+        {
+            code: `
+        const protectedProcedure = t.procedure.use(requireCoursePermissionOwn);
+        const m = protectedProcedure.query(async () => {});
+      `,
+        },
+        // Intermediate procedure-base variable: permission added downstream
+        {
+            code: `
+        const featureProcedure = t.procedure.use(requireAiGradingFeature);
+        const m = featureProcedure.use(requireAdministrator).mutation(async () => {});
+      `,
+        },
+        // Imported procedure base — we can't follow across files, so don't fire
+        {
+            code: `
+        import { protectedProcedure } from './foo.js';
+        const m = protectedProcedure.query(async () => {});
+      `,
+            languageOptions: { parserOptions: { sourceType: 'module' } },
+        },
+    ],
+    invalid: [
+        {
+            code: 'const list = t.procedure.query(async () => {});',
+            errors: [{ messageId: 'missingPermissionMiddleware' }],
+        },
+        {
+            code: 'const m = t.procedure.input(z.object({})).mutation(async () => {});',
+            errors: [{ messageId: 'missingPermissionMiddleware' }],
+        },
+        // Feature gate alone is not a permission gate
+        {
+            code: 'const m = t.procedure.use(requireEnhancedAccessControl).mutation(async () => {});',
+            errors: [{ messageId: 'missingPermissionMiddleware' }],
+        },
+        {
+            code: 'const m = t.procedure.use(requireAiGradingFeature).mutation(async () => {});',
+            errors: [{ messageId: 'missingPermissionMiddleware' }],
+        },
+        {
+            code: `const m = t.procedure
+        .input(z.object({}))
+        .output(z.object({}))
+        .subscription(async () => {});`,
+            errors: [{ messageId: 'missingPermissionMiddleware' }],
+        },
+        // Intermediate procedure-base variable with only a feature gate is still a miss
+        {
+            code: `
+        const featureProcedure = t.procedure.use(requireAiGradingFeature);
+        const m = featureProcedure.query(async () => {});
+      `,
+            errors: [{ messageId: 'missingPermissionMiddleware' }],
+        },
+        {
+            code: `
+        const featureProcedure = t.procedure.use(requireAiGradingFeature);
+        const m = featureProcedure.input(z.object({})).mutation(async () => {});
+      `,
+            errors: [{ messageId: 'missingPermissionMiddleware' }],
+        },
+    ],
+});
+//# sourceMappingURL=require-trpc-permission-middleware.test.js.map

package/dist/tests/require-trpc-permission-middleware.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"require-trpc-permission-middleware.test.js","sourceRoot":"","sources":["../../src/tests/require-trpc-permission-middleware.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,gCAAgC,CAAC;AAC5D,OAAO,EAAE,QAAQ,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAEhD,OAAO,IAAI,MAAM,gDAAgD,CAAC;AAElE,UAAU,CAAC,QAAQ,GAAG,QAAQ,CAAC;AAC/B,UAAU,CAAC,QAAQ,GAAG,QAAQ,CAAC;AAC/B,UAAU,CAAC,EAAE,GAAG,EAAE,CAAC;AAEnB,MAAM,UAAU,GAAG,IAAI,UAAU,EAAE,CAAC;AAEpC,UAAU,CAAC,GAAG,CAAC,oCAAoC,EAAE,IAAI,EAAE;IACzD,KAAK,EAAE;QACL;YACE,IAAI,EAAE,kFAAkF;SACzF;QACD;YACE,IAAI,EAAE,qFAAqF;SAC5F;QACD;YACE,IAAI,EAAE;;;gCAGoB;SAC3B;QACD;YACE,IAAI,EAAE,2EAA2E;SAClF;QACD;YACE,IAAI,EAAE;;;mCAGuB;SAC9B;QACD;YACE,IAAI,EAAE,iFAAiF;SACxF;QACD,8CAA8C;QAC9C;YACE,IAAI,EAAE,iDAAiD;SACxD;QACD;YACE,IAAI,EAAE,yDAAyD;SAChE;QACD,0EAA0E;QAC1E;YACE,IAAI,EAAE;;;OAGL;SACF;QACD,oEAAoE;QACpE;YACE,IAAI,EAAE;;;OAGL;SACF;QACD,wEAAwE;QACxE;YACE,IAAI,EAAE;;;OAGL;YACD,eAAe,EAAE,EAAE,aAAa,EAAE,EAAE,UAAU,EAAE,QAAQ,EAAE,EAAE;SAC7D;KACF;IACD,OAAO,EAAE;QACP;YACE,IAAI,EAAE,iDAAiD;YACvD,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,6BAA6B,EAAE,CAAC;SACvD;QACD;YACE,IAAI,EAAE,qEAAqE;YAC3E,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,6BAA6B,EAAE,CAAC;SACvD;QACD,8CAA8C;QAC9C;YACE,IAAI,EAAE,mFAAmF;YACzF,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,6BAA6B,EAAE,CAAC;SACvD;QACD;YACE,IAAI,EAAE,8EAA8E;YACpF,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,6BAA6B,EAAE,CAAC;SACvD;QACD;YACE,IAAI,EAAE;;;uCAG2B;YACjC,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,6BAA6B,EAAE,CAAC;SACvD;QACD,gFAAgF;QAChF;YACE,IAAI,EAAE;;;OAGL;YACD,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,6BAA6B,EAAE,CAAC;SACvD;QACD;YACE,IAAI,EAAE;;;OAGL;YACD,MAAM,EAAE,CAAC,EAAE,SAAS,EAAE,6BAA6B,EAAE,CAAC;SACvD;KACF;CACF,CAAC,CAAC","sourcesContent":["import { RuleTester } from '@typescript-eslint/rule-tester';\nimport { afterAll, describe, it } from 'vitest';\n\nimport rule from '../rules/require-trpc-permission-middleware.js';\n\nRuleTester.afterAll = afterAll;\nRuleTester.describe = describe;\nRuleTester.it = it;\n\nconst ruleTester = new RuleTester();\n\nruleTester.run('require-trpc-permission-middleware', rule, {\n  valid: [\n    {\n      code: 'const list = t.procedure.use(requireCoursePermissionEdit).query(async () => {});',\n    },\n    {\n      code: 'const m = t.procedure.use(requireCoursePermissionPreview).mutation(async () => {});',\n    },\n    {\n      code: `const m = t.procedure\n        .use(requireEnhancedAccessControl)\n        .use(requireCourseInstancePermissionView)\n        .query(async () => {});`,\n    },\n    {\n      code: 'const m = t.procedure.use(requireAdministrator).mutation(async () => {});',\n    },\n    {\n      code: `const m = t.procedure\n        .use(requireCourseInstancePermissionEdit)\n        .input(z.object({}))\n        .mutation(async () => {});`,\n    },\n    {\n      code: 'const list = t.procedure.use(requireCoursePermissionOwn).query(async () => {});',\n    },\n    // Not a t.procedure chain — should be ignored\n    {\n      code: 'const fn = somethingElse.query(async () => {});',\n    },\n    {\n      code: 'const fn = otherRouter.procedure.query(async () => {});',\n    },\n    // Intermediate procedure-base variable: permission middleware on the base\n    {\n      code: `\n        const protectedProcedure = t.procedure.use(requireCoursePermissionOwn);\n        const m = protectedProcedure.query(async () => {});\n      `,\n    },\n    // Intermediate procedure-base variable: permission added downstream\n    {\n      code: `\n        const featureProcedure = t.procedure.use(requireAiGradingFeature);\n        const m = featureProcedure.use(requireAdministrator).mutation(async () => {});\n      `,\n    },\n    // Imported procedure base — we can't follow across files, so don't fire\n    {\n      code: `\n        import { protectedProcedure } from './foo.js';\n        const m = protectedProcedure.query(async () => {});\n      `,\n      languageOptions: { parserOptions: { sourceType: 'module' } },\n    },\n  ],\n  invalid: [\n    {\n      code: 'const list = t.procedure.query(async () => {});',\n      errors: [{ messageId: 'missingPermissionMiddleware' }],\n    },\n    {\n      code: 'const m = t.procedure.input(z.object({})).mutation(async () => {});',\n      errors: [{ messageId: 'missingPermissionMiddleware' }],\n    },\n    // Feature gate alone is not a permission gate\n    {\n      code: 'const m = t.procedure.use(requireEnhancedAccessControl).mutation(async () => {});',\n      errors: [{ messageId: 'missingPermissionMiddleware' }],\n    },\n    {\n      code: 'const m = t.procedure.use(requireAiGradingFeature).mutation(async () => {});',\n      errors: [{ messageId: 'missingPermissionMiddleware' }],\n    },\n    {\n      code: `const m = t.procedure\n        .input(z.object({}))\n        .output(z.object({}))\n        .subscription(async () => {});`,\n      errors: [{ messageId: 'missingPermissionMiddleware' }],\n    },\n    // Intermediate procedure-base variable with only a feature gate is still a miss\n    {\n      code: `\n        const featureProcedure = t.procedure.use(requireAiGradingFeature);\n        const m = featureProcedure.query(async () => {});\n      `,\n      errors: [{ messageId: 'missingPermissionMiddleware' }],\n    },\n    {\n      code: `\n        const featureProcedure = t.procedure.use(requireAiGradingFeature);\n        const m = featureProcedure.input(z.object({})).mutation(async () => {});\n      `,\n      errors: [{ messageId: 'missingPermissionMiddleware' }],\n    },\n  ],\n});\n"]}

package/dist/utils.js

@@ -1,6 +1,3 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.getAwsClientNamesFromImportDeclaration = getAwsClientNamesFromImportDeclaration;
 /**
  * Determines if the given identifier name corresponds to a client from the
  * given package.
@@ -29,7 +26,7 @@ function isIdentifierClient(identifierName, packageName) {
  *
  * then this function will return a set containing the strings "S3" and "S3Client".
  */
-function getAwsClientNamesFromImportDeclaration(node) {
+export function getAwsClientNamesFromImportDeclaration(node) {
     const clientNames = new Set();
     const importSource = node.source.value;
     if (importSource.startsWith('@aws-sdk/client-')) {

package/dist/utils.js.map

@@ -1 +1 @@
-{"version":3,"file":"utils.js","sourceRoot":"","sources":["../src/utils.ts"],"names":[],"mappings":";;;AAAA;;;GAGG;AACH,SAAS,kBAAkB,CAAC,cAAsB,EAAE,WAAmB,EAAW;IAChF,+DAA+D;IAC/D,IAAI,cAAc,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC;QACtC,OAAO,IAAI,CAAC;IACd,CAAC;IAED,8EAA8E;IAC9E,MAAM,UAAU,GAAG,WAAW,CAAC,OAAO,CAAC,kBAAkB,EAAE,EAAE,CAAC,CAAC;IAC/D,MAAM,iBAAiB,GAAG,UAAU,CAAC,UAAU,CAAC,GAAG,EAAE,EAAE,CAAC,CAAC,WAAW,EAAE,CAAC;IACvE,IAAI,cAAc,CAAC,WAAW,EAAE,KAAK,iBAAiB,EAAE,CAAC;QACvD,OAAO,IAAI,CAAC;IACd,CAAC;IAED,OAAO,KAAK,CAAC;AAAA,CACd;AAED;;;;;;;;;;GAUG;AACH,gDAAuD,IAAS,EAAE;IAChE,MAAM,WAAW,GAAG,IAAI,GAAG,EAAU,CAAC;IAEtC,MAAM,YAAY,GAAG,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC;IACvC,IAAI,YAAY,CAAC,UAAU,CAAC,kBAAkB,CAAC,EAAE,CAAC;QAChD,IAAI,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC,SAAc,EAAE,EAAE,CAAC;YAC1C,IAAI,SAAS,CAAC,IAAI,KAAK,iBAAiB,EAAE,CAAC;gBACzC,MAAM,aAAa,GAAG,SAAS,CAAC,QAAQ,CAAC,IAAI,CAAC;gBAC9C,IAAI,kBAAkB,CAAC,aAAa,EAAE,YAAY,CAAC,EAAE,CAAC;oBACpD,WAAW,CAAC,GAAG,CAAC,aAAa,CAAC,CAAC;gBACjC,CAAC;YACH,CAAC;QAAA,CACF,CAAC,CAAC;IACL,CAAC;IAED,OAAO,WAAW,CAAC;AAAA,CACpB","sourcesContent":["/**\n * Determines if the given identifier name corresponds to a client from the\n * given package.\n */\nfunction isIdentifierClient(identifierName: string, packageName: string): boolean {\n  // If the identifier ends with \"Client\", include it in the set.\n  if (identifierName.endsWith('Client')) {\n    return true;\n  }\n\n  // If the identifier matches the package name directly, include it in the set.\n  const clientName = packageName.replace('@aws-sdk/client-', '');\n  const packageIdentifier = clientName.replaceAll('-', '').toLowerCase();\n  if (identifierName.toLowerCase() === packageIdentifier) {\n    return true;\n  }\n\n  return false;\n}\n\n/**\n * Retrieves the names of AWS clients specified by the given import declaration.\n *\n * For instance, if the import declaration is:\n *\n * ```ts\n * import { S3, S3Client } from '@aws-sdk/client-s3';\n * ```\n *\n * then this function will return a set containing the strings \"S3\" and \"S3Client\".\n */\nexport function getAwsClientNamesFromImportDeclaration(node: any) {\n  const clientNames = new Set<string>();\n\n  const importSource = node.source.value;\n  if (importSource.startsWith('@aws-sdk/client-')) {\n    node.specifiers.forEach((specifier: any) => {\n      if (specifier.type === 'ImportSpecifier') {\n        const specifierName = specifier.imported.name;\n        if (isIdentifierClient(specifierName, importSource)) {\n          clientNames.add(specifierName);\n        }\n      }\n    });\n  }\n\n  return clientNames;\n}\n"]}
+{"version":3,"file":"utils.js","sourceRoot":"","sources":["../src/utils.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,SAAS,kBAAkB,CAAC,cAAsB,EAAE,WAAmB;IACrE,+DAA+D;IAC/D,IAAI,cAAc,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC;QACtC,OAAO,IAAI,CAAC;IACd,CAAC;IAED,8EAA8E;IAC9E,MAAM,UAAU,GAAG,WAAW,CAAC,OAAO,CAAC,kBAAkB,EAAE,EAAE,CAAC,CAAC;IAC/D,MAAM,iBAAiB,GAAG,UAAU,CAAC,UAAU,CAAC,GAAG,EAAE,EAAE,CAAC,CAAC,WAAW,EAAE,CAAC;IACvE,IAAI,cAAc,CAAC,WAAW,EAAE,KAAK,iBAAiB,EAAE,CAAC;QACvD,OAAO,IAAI,CAAC;IACd,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC;AAED;;;;;;;;;;GAUG;AACH,MAAM,UAAU,sCAAsC,CAAC,IAAS;IAC9D,MAAM,WAAW,GAAG,IAAI,GAAG,EAAU,CAAC;IAEtC,MAAM,YAAY,GAAG,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC;IACvC,IAAI,YAAY,CAAC,UAAU,CAAC,kBAAkB,CAAC,EAAE,CAAC;QAChD,IAAI,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC,SAAc,EAAE,EAAE;YACzC,IAAI,SAAS,CAAC,IAAI,KAAK,iBAAiB,EAAE,CAAC;gBACzC,MAAM,aAAa,GAAG,SAAS,CAAC,QAAQ,CAAC,IAAI,CAAC;gBAC9C,IAAI,kBAAkB,CAAC,aAAa,EAAE,YAAY,CAAC,EAAE,CAAC;oBACpD,WAAW,CAAC,GAAG,CAAC,aAAa,CAAC,CAAC;gBACjC,CAAC;YACH,CAAC;QACH,CAAC,CAAC,CAAC;IACL,CAAC;IAED,OAAO,WAAW,CAAC;AACrB,CAAC","sourcesContent":["/**\n * Determines if the given identifier name corresponds to a client from the\n * given package.\n */\nfunction isIdentifierClient(identifierName: string, packageName: string): boolean {\n  // If the identifier ends with \"Client\", include it in the set.\n  if (identifierName.endsWith('Client')) {\n    return true;\n  }\n\n  // If the identifier matches the package name directly, include it in the set.\n  const clientName = packageName.replace('@aws-sdk/client-', '');\n  const packageIdentifier = clientName.replaceAll('-', '').toLowerCase();\n  if (identifierName.toLowerCase() === packageIdentifier) {\n    return true;\n  }\n\n  return false;\n}\n\n/**\n * Retrieves the names of AWS clients specified by the given import declaration.\n *\n * For instance, if the import declaration is:\n *\n * ```ts\n * import { S3, S3Client } from '@aws-sdk/client-s3';\n * ```\n *\n * then this function will return a set containing the strings \"S3\" and \"S3Client\".\n */\nexport function getAwsClientNamesFromImportDeclaration(node: any) {\n  const clientNames = new Set<string>();\n\n  const importSource = node.source.value;\n  if (importSource.startsWith('@aws-sdk/client-')) {\n    node.specifiers.forEach((specifier: any) => {\n      if (specifier.type === 'ImportSpecifier') {\n        const specifierName = specifier.imported.name;\n        if (isIdentifierClient(specifierName, importSource)) {\n          clientNames.add(specifierName);\n        }\n      }\n    });\n  }\n\n  return clientNames;\n}\n"]}