@less-unsound/eslint-plugin 0.0.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/LICENSE +21 -0
- package/README.md +112 -0
- package/dist/cjs/index.js +355 -0
- package/dist/cjs/package.json +3 -0
- package/dist/cjs/rules/no-accidental-mutation.js +100 -0
- package/dist/cjs/rules/no-ambient-dom-globals.js +181 -0
- package/dist/cjs/rules/no-any.js +27 -0
- package/dist/cjs/rules/no-arguments-object.js +122 -0
- package/dist/cjs/rules/no-as-type-assertion.js +44 -0
- package/dist/cjs/rules/no-bigint.js +74 -0
- package/dist/cjs/rules/no-boolean-literal-compare.js +37 -0
- package/dist/cjs/rules/no-broad-types.js +50 -0
- package/dist/cjs/rules/no-buffer.js +146 -0
- package/dist/cjs/rules/no-class.js +39 -0
- package/dist/cjs/rules/no-comma-operator.js +27 -0
- package/dist/cjs/rules/no-conditional-type.js +27 -0
- package/dist/cjs/rules/no-default-parameter.js +71 -0
- package/dist/cjs/rules/no-enum.js +27 -0
- package/dist/cjs/rules/no-eval.js +33 -0
- package/dist/cjs/rules/no-export-default.js +45 -0
- package/dist/cjs/rules/no-for-in.js +27 -0
- package/dist/cjs/rules/no-for-let.js +39 -0
- package/dist/cjs/rules/no-function-constructor.js +84 -0
- package/dist/cjs/rules/no-function-keyword.js +48 -0
- package/dist/cjs/rules/no-generic-parameter-bounds.js +34 -0
- package/dist/cjs/rules/no-getter-setter.js +48 -0
- package/dist/cjs/rules/no-global-context.js +134 -0
- package/dist/cjs/rules/no-import-code-extension.js +70 -0
- package/dist/cjs/rules/no-import-rename.js +45 -0
- package/dist/cjs/rules/no-import-type.js +27 -0
- package/dist/cjs/rules/no-in-operator.js +27 -0
- package/dist/cjs/rules/no-indexed-access-type.js +27 -0
- package/dist/cjs/rules/no-indexed-object-type.js +53 -0
- package/dist/cjs/rules/no-inheritance.js +65 -0
- package/dist/cjs/rules/no-inline-tagged-object.js +95 -0
- package/dist/cjs/rules/no-intersection-type.js +27 -0
- package/dist/cjs/rules/no-json-static-method.js +158 -0
- package/dist/cjs/rules/no-keyof.js +27 -0
- package/dist/cjs/rules/no-let.js +45 -0
- package/dist/cjs/rules/no-mapped-object-type.js +27 -0
- package/dist/cjs/rules/no-never.js +27 -0
- package/dist/cjs/rules/no-non-null-assertion.js +27 -0
- package/dist/cjs/rules/no-null.js +36 -0
- package/dist/cjs/rules/no-object-as-map.js +138 -0
- package/dist/cjs/rules/no-object-spread.js +27 -0
- package/dist/cjs/rules/no-object-stringification-hook.js +101 -0
- package/dist/cjs/rules/no-optional-parameter.js +104 -0
- package/dist/cjs/rules/no-optional-property.js +30 -0
- package/dist/cjs/rules/no-overload.js +110 -0
- package/dist/cjs/rules/no-param-assign.js +94 -0
- package/dist/cjs/rules/no-prototype-access.js +79 -0
- package/dist/cjs/rules/no-proxy-reflect.js +112 -0
- package/dist/cjs/rules/no-reexport-outside-entrypoint.js +46 -0
- package/dist/cjs/rules/no-regular-expression.js +93 -0
- package/dist/cjs/rules/no-require.js +162 -0
- package/dist/cjs/rules/no-rest-parameter.js +38 -0
- package/dist/cjs/rules/no-switch.js +27 -0
- package/dist/cjs/rules/no-symbol.js +33 -0
- package/dist/cjs/rules/no-try-catch.js +30 -0
- package/dist/cjs/rules/no-ts-comment-directive.js +39 -0
- package/dist/cjs/rules/no-tuple.js +27 -0
- package/dist/cjs/rules/no-type-guard.js +27 -0
- package/dist/cjs/rules/no-typeof-operator.js +27 -0
- package/dist/cjs/rules/no-typeof-type.js +27 -0
- package/dist/cjs/rules/no-undefined-type.js +27 -0
- package/dist/cjs/rules/no-unknown.js +27 -0
- package/dist/cjs/rules/no-untagged-object-union.js +150 -0
- package/dist/cjs/rules/no-utility-type.js +57 -0
- package/dist/cjs/rules/no-var.js +27 -0
- package/dist/cjs/rules/no-void-operator.js +27 -0
- package/dist/cjs/rules/no-while.js +33 -0
- package/dist/cjs/rules/require-boolean-prefix.js +179 -0
- package/dist/cjs/rules/require-disjoint-union.js +228 -0
- package/dist/cjs/rules/require-function-verb.js +293 -0
- package/dist/cjs/rules/require-identifier-case.js +197 -0
- package/dist/cjs/rules/require-kebab-case-filename.js +45 -0
- package/dist/cjs/rules/require-named-union-branch.js +34 -0
- package/dist/cjs/rules/require-readonly-array.js +49 -0
- package/dist/cjs/rules/require-readonly-collections.js +37 -0
- package/dist/cjs/rules/require-readonly-object-fields.js +32 -0
- package/dist/cjs/rules/require-readonly-tuple.js +33 -0
- package/dist/cjs/rules/require-return-type.js +85 -0
- package/dist/cjs/rules/require-string-template-expression.js +55 -0
- package/dist/cjs/utils/create-rule.js +6 -0
- package/dist/cjs/utils/package-entrypoint.js +69 -0
- package/dist/cjs/utils/readonly.js +18 -0
- package/dist/cjs/utils/string-case.js +12 -0
- package/dist/cjs/utils/type-reference.js +35 -0
- package/dist/esm/index.d.ts +3568 -0
- package/dist/esm/index.js +352 -0
- package/dist/esm/rules/no-accidental-mutation.d.ts +3 -0
- package/dist/esm/rules/no-accidental-mutation.js +97 -0
- package/dist/esm/rules/no-ambient-dom-globals.d.ts +3 -0
- package/dist/esm/rules/no-ambient-dom-globals.js +178 -0
- package/dist/esm/rules/no-any.d.ts +3 -0
- package/dist/esm/rules/no-any.js +24 -0
- package/dist/esm/rules/no-arguments-object.d.ts +3 -0
- package/dist/esm/rules/no-arguments-object.js +119 -0
- package/dist/esm/rules/no-as-type-assertion.d.ts +3 -0
- package/dist/esm/rules/no-as-type-assertion.js +41 -0
- package/dist/esm/rules/no-bigint.d.ts +3 -0
- package/dist/esm/rules/no-bigint.js +71 -0
- package/dist/esm/rules/no-boolean-literal-compare.d.ts +3 -0
- package/dist/esm/rules/no-boolean-literal-compare.js +34 -0
- package/dist/esm/rules/no-broad-types.d.ts +3 -0
- package/dist/esm/rules/no-broad-types.js +47 -0
- package/dist/esm/rules/no-buffer.d.ts +3 -0
- package/dist/esm/rules/no-buffer.js +143 -0
- package/dist/esm/rules/no-class.d.ts +3 -0
- package/dist/esm/rules/no-class.js +36 -0
- package/dist/esm/rules/no-comma-operator.d.ts +3 -0
- package/dist/esm/rules/no-comma-operator.js +24 -0
- package/dist/esm/rules/no-conditional-type.d.ts +3 -0
- package/dist/esm/rules/no-conditional-type.js +24 -0
- package/dist/esm/rules/no-default-parameter.d.ts +3 -0
- package/dist/esm/rules/no-default-parameter.js +68 -0
- package/dist/esm/rules/no-enum.d.ts +3 -0
- package/dist/esm/rules/no-enum.js +24 -0
- package/dist/esm/rules/no-eval.d.ts +3 -0
- package/dist/esm/rules/no-eval.js +30 -0
- package/dist/esm/rules/no-export-default.d.ts +3 -0
- package/dist/esm/rules/no-export-default.js +42 -0
- package/dist/esm/rules/no-for-in.d.ts +3 -0
- package/dist/esm/rules/no-for-in.js +24 -0
- package/dist/esm/rules/no-for-let.d.ts +3 -0
- package/dist/esm/rules/no-for-let.js +36 -0
- package/dist/esm/rules/no-function-constructor.d.ts +3 -0
- package/dist/esm/rules/no-function-constructor.js +81 -0
- package/dist/esm/rules/no-function-keyword.d.ts +3 -0
- package/dist/esm/rules/no-function-keyword.js +45 -0
- package/dist/esm/rules/no-generic-parameter-bounds.d.ts +3 -0
- package/dist/esm/rules/no-generic-parameter-bounds.js +31 -0
- package/dist/esm/rules/no-getter-setter.d.ts +3 -0
- package/dist/esm/rules/no-getter-setter.js +45 -0
- package/dist/esm/rules/no-global-context.d.ts +3 -0
- package/dist/esm/rules/no-global-context.js +131 -0
- package/dist/esm/rules/no-import-code-extension.d.ts +4 -0
- package/dist/esm/rules/no-import-code-extension.js +67 -0
- package/dist/esm/rules/no-import-rename.d.ts +3 -0
- package/dist/esm/rules/no-import-rename.js +42 -0
- package/dist/esm/rules/no-import-type.d.ts +3 -0
- package/dist/esm/rules/no-import-type.js +24 -0
- package/dist/esm/rules/no-in-operator.d.ts +3 -0
- package/dist/esm/rules/no-in-operator.js +24 -0
- package/dist/esm/rules/no-indexed-access-type.d.ts +3 -0
- package/dist/esm/rules/no-indexed-access-type.js +24 -0
- package/dist/esm/rules/no-indexed-object-type.d.ts +3 -0
- package/dist/esm/rules/no-indexed-object-type.js +50 -0
- package/dist/esm/rules/no-inheritance.d.ts +3 -0
- package/dist/esm/rules/no-inheritance.js +62 -0
- package/dist/esm/rules/no-inline-tagged-object.d.ts +5 -0
- package/dist/esm/rules/no-inline-tagged-object.js +92 -0
- package/dist/esm/rules/no-intersection-type.d.ts +3 -0
- package/dist/esm/rules/no-intersection-type.js +24 -0
- package/dist/esm/rules/no-json-static-method.d.ts +3 -0
- package/dist/esm/rules/no-json-static-method.js +155 -0
- package/dist/esm/rules/no-keyof.d.ts +3 -0
- package/dist/esm/rules/no-keyof.js +24 -0
- package/dist/esm/rules/no-let.d.ts +3 -0
- package/dist/esm/rules/no-let.js +42 -0
- package/dist/esm/rules/no-mapped-object-type.d.ts +3 -0
- package/dist/esm/rules/no-mapped-object-type.js +24 -0
- package/dist/esm/rules/no-never.d.ts +3 -0
- package/dist/esm/rules/no-never.js +24 -0
- package/dist/esm/rules/no-non-null-assertion.d.ts +3 -0
- package/dist/esm/rules/no-non-null-assertion.js +24 -0
- package/dist/esm/rules/no-null.d.ts +3 -0
- package/dist/esm/rules/no-null.js +33 -0
- package/dist/esm/rules/no-object-as-map.d.ts +3 -0
- package/dist/esm/rules/no-object-as-map.js +135 -0
- package/dist/esm/rules/no-object-spread.d.ts +3 -0
- package/dist/esm/rules/no-object-spread.js +24 -0
- package/dist/esm/rules/no-object-stringification-hook.d.ts +3 -0
- package/dist/esm/rules/no-object-stringification-hook.js +98 -0
- package/dist/esm/rules/no-optional-parameter.d.ts +3 -0
- package/dist/esm/rules/no-optional-parameter.js +101 -0
- package/dist/esm/rules/no-optional-property.d.ts +3 -0
- package/dist/esm/rules/no-optional-property.js +27 -0
- package/dist/esm/rules/no-overload.d.ts +3 -0
- package/dist/esm/rules/no-overload.js +107 -0
- package/dist/esm/rules/no-param-assign.d.ts +3 -0
- package/dist/esm/rules/no-param-assign.js +91 -0
- package/dist/esm/rules/no-prototype-access.d.ts +3 -0
- package/dist/esm/rules/no-prototype-access.js +76 -0
- package/dist/esm/rules/no-proxy-reflect.d.ts +3 -0
- package/dist/esm/rules/no-proxy-reflect.js +109 -0
- package/dist/esm/rules/no-reexport-outside-entrypoint.d.ts +3 -0
- package/dist/esm/rules/no-reexport-outside-entrypoint.js +43 -0
- package/dist/esm/rules/no-regular-expression.d.ts +3 -0
- package/dist/esm/rules/no-regular-expression.js +90 -0
- package/dist/esm/rules/no-require.d.ts +3 -0
- package/dist/esm/rules/no-require.js +159 -0
- package/dist/esm/rules/no-rest-parameter.d.ts +3 -0
- package/dist/esm/rules/no-rest-parameter.js +35 -0
- package/dist/esm/rules/no-switch.d.ts +3 -0
- package/dist/esm/rules/no-switch.js +24 -0
- package/dist/esm/rules/no-symbol.d.ts +3 -0
- package/dist/esm/rules/no-symbol.js +30 -0
- package/dist/esm/rules/no-try-catch.d.ts +3 -0
- package/dist/esm/rules/no-try-catch.js +27 -0
- package/dist/esm/rules/no-ts-comment-directive.d.ts +3 -0
- package/dist/esm/rules/no-ts-comment-directive.js +36 -0
- package/dist/esm/rules/no-tuple.d.ts +3 -0
- package/dist/esm/rules/no-tuple.js +24 -0
- package/dist/esm/rules/no-type-guard.d.ts +3 -0
- package/dist/esm/rules/no-type-guard.js +24 -0
- package/dist/esm/rules/no-typeof-operator.d.ts +3 -0
- package/dist/esm/rules/no-typeof-operator.js +24 -0
- package/dist/esm/rules/no-typeof-type.d.ts +3 -0
- package/dist/esm/rules/no-typeof-type.js +24 -0
- package/dist/esm/rules/no-undefined-type.d.ts +3 -0
- package/dist/esm/rules/no-undefined-type.js +24 -0
- package/dist/esm/rules/no-unknown.d.ts +3 -0
- package/dist/esm/rules/no-unknown.js +24 -0
- package/dist/esm/rules/no-untagged-object-union.d.ts +3 -0
- package/dist/esm/rules/no-untagged-object-union.js +147 -0
- package/dist/esm/rules/no-utility-type.d.ts +3 -0
- package/dist/esm/rules/no-utility-type.js +54 -0
- package/dist/esm/rules/no-var.d.ts +3 -0
- package/dist/esm/rules/no-var.js +24 -0
- package/dist/esm/rules/no-void-operator.d.ts +3 -0
- package/dist/esm/rules/no-void-operator.js +24 -0
- package/dist/esm/rules/no-while.d.ts +3 -0
- package/dist/esm/rules/no-while.js +30 -0
- package/dist/esm/rules/require-boolean-prefix.d.ts +4 -0
- package/dist/esm/rules/require-boolean-prefix.js +176 -0
- package/dist/esm/rules/require-disjoint-union.d.ts +5 -0
- package/dist/esm/rules/require-disjoint-union.js +225 -0
- package/dist/esm/rules/require-function-verb.d.ts +4 -0
- package/dist/esm/rules/require-function-verb.js +290 -0
- package/dist/esm/rules/require-identifier-case.d.ts +4 -0
- package/dist/esm/rules/require-identifier-case.js +194 -0
- package/dist/esm/rules/require-kebab-case-filename.d.ts +4 -0
- package/dist/esm/rules/require-kebab-case-filename.js +42 -0
- package/dist/esm/rules/require-named-union-branch.d.ts +3 -0
- package/dist/esm/rules/require-named-union-branch.js +31 -0
- package/dist/esm/rules/require-readonly-array.d.ts +3 -0
- package/dist/esm/rules/require-readonly-array.js +46 -0
- package/dist/esm/rules/require-readonly-collections.d.ts +3 -0
- package/dist/esm/rules/require-readonly-collections.js +34 -0
- package/dist/esm/rules/require-readonly-object-fields.d.ts +3 -0
- package/dist/esm/rules/require-readonly-object-fields.js +29 -0
- package/dist/esm/rules/require-readonly-tuple.d.ts +3 -0
- package/dist/esm/rules/require-readonly-tuple.js +30 -0
- package/dist/esm/rules/require-return-type.d.ts +4 -0
- package/dist/esm/rules/require-return-type.js +79 -0
- package/dist/esm/rules/require-string-template-expression.d.ts +4 -0
- package/dist/esm/rules/require-string-template-expression.js +52 -0
- package/dist/esm/utils/create-rule.d.ts +4 -0
- package/dist/esm/utils/create-rule.js +3 -0
- package/dist/esm/utils/package-entrypoint.d.ts +2 -0
- package/dist/esm/utils/package-entrypoint.js +64 -0
- package/dist/esm/utils/readonly.d.ts +2 -0
- package/dist/esm/utils/readonly.js +14 -0
- package/dist/esm/utils/string-case.d.ts +3 -0
- package/dist/esm/utils/string-case.js +6 -0
- package/dist/esm/utils/type-reference.d.ts +2 -0
- package/dist/esm/utils/type-reference.js +31 -0
- package/docs/rules/no-accidental-mutation.md +39 -0
- package/docs/rules/no-ambient-dom-globals.md +37 -0
- package/docs/rules/no-any.md +23 -0
- package/docs/rules/no-arguments-object.md +35 -0
- package/docs/rules/no-as-type-assertion.md +26 -0
- package/docs/rules/no-bigint.md +31 -0
- package/docs/rules/no-boolean-literal-compare.md +31 -0
- package/docs/rules/no-broad-types.md +27 -0
- package/docs/rules/no-buffer.md +31 -0
- package/docs/rules/no-class.md +56 -0
- package/docs/rules/no-comma-operator.md +31 -0
- package/docs/rules/no-conditional-type.md +23 -0
- package/docs/rules/no-default-parameter.md +43 -0
- package/docs/rules/no-enum.md +31 -0
- package/docs/rules/no-eval.md +27 -0
- package/docs/rules/no-export-default.md +28 -0
- package/docs/rules/no-for-in.md +23 -0
- package/docs/rules/no-for-let.md +31 -0
- package/docs/rules/no-function-constructor.md +27 -0
- package/docs/rules/no-function-keyword.md +37 -0
- package/docs/rules/no-generic-parameter-bounds.md +25 -0
- package/docs/rules/no-getter-setter.md +55 -0
- package/docs/rules/no-global-context.md +50 -0
- package/docs/rules/no-import-code-extension.md +39 -0
- package/docs/rules/no-import-rename.md +31 -0
- package/docs/rules/no-import-type.md +25 -0
- package/docs/rules/no-in-operator.md +23 -0
- package/docs/rules/no-indexed-access-type.md +33 -0
- package/docs/rules/no-indexed-object-type.md +23 -0
- package/docs/rules/no-inheritance.md +31 -0
- package/docs/rules/no-inline-tagged-object.md +41 -0
- package/docs/rules/no-intersection-type.md +23 -0
- package/docs/rules/no-json-static-method.md +46 -0
- package/docs/rules/no-keyof.md +35 -0
- package/docs/rules/no-let.md +31 -0
- package/docs/rules/no-mapped-object-type.md +23 -0
- package/docs/rules/no-never.md +35 -0
- package/docs/rules/no-non-null-assertion.md +23 -0
- package/docs/rules/no-null.md +31 -0
- package/docs/rules/no-object-as-map.md +49 -0
- package/docs/rules/no-object-spread.md +31 -0
- package/docs/rules/no-object-stringification-hook.md +49 -0
- package/docs/rules/no-optional-parameter.md +39 -0
- package/docs/rules/no-optional-property.md +33 -0
- package/docs/rules/no-overload.md +29 -0
- package/docs/rules/no-param-assign.md +30 -0
- package/docs/rules/no-prototype-access.md +43 -0
- package/docs/rules/no-proxy-reflect.md +39 -0
- package/docs/rules/no-reexport-outside-entrypoint.md +35 -0
- package/docs/rules/no-regular-expression.md +36 -0
- package/docs/rules/no-require.md +29 -0
- package/docs/rules/no-rest-parameter.md +23 -0
- package/docs/rules/no-switch.md +32 -0
- package/docs/rules/no-symbol.md +31 -0
- package/docs/rules/no-try-catch.md +37 -0
- package/docs/rules/no-ts-comment-directive.md +25 -0
- package/docs/rules/no-tuple.md +29 -0
- package/docs/rules/no-type-guard.md +23 -0
- package/docs/rules/no-typeof-operator.md +23 -0
- package/docs/rules/no-typeof-type.md +23 -0
- package/docs/rules/no-undefined-type.md +30 -0
- package/docs/rules/no-unknown.md +23 -0
- package/docs/rules/no-untagged-object-union.md +25 -0
- package/docs/rules/no-utility-type.md +35 -0
- package/docs/rules/no-var.md +27 -0
- package/docs/rules/no-void-operator.md +27 -0
- package/docs/rules/no-while.md +33 -0
- package/docs/rules/require-boolean-prefix.md +31 -0
- package/docs/rules/require-disjoint-union.md +58 -0
- package/docs/rules/require-function-verb.md +83 -0
- package/docs/rules/require-identifier-case.md +54 -0
- package/docs/rules/require-kebab-case-filename.md +37 -0
- package/docs/rules/require-named-union-branch.md +35 -0
- package/docs/rules/require-readonly-array.md +35 -0
- package/docs/rules/require-readonly-collections.md +35 -0
- package/docs/rules/require-readonly-object-fields.md +46 -0
- package/docs/rules/require-readonly-tuple.md +27 -0
- package/docs/rules/require-return-type.md +37 -0
- package/docs/rules/require-string-template-expression.md +38 -0
- package/package.json +70 -0
|
@@ -0,0 +1,33 @@
|
|
|
1
|
+
import { createRule } from "../utils/create-rule.js";
|
|
2
|
+
const messageIds = Object.freeze({
|
|
3
|
+
forbiddenNull: "`null` is not allowed."
|
|
4
|
+
});
|
|
5
|
+
export const noNullRule = createRule({
|
|
6
|
+
name: "no-null",
|
|
7
|
+
meta: {
|
|
8
|
+
type: "problem",
|
|
9
|
+
docs: {
|
|
10
|
+
description: "Disallow both runtime `null` literals and the `null` type."
|
|
11
|
+
},
|
|
12
|
+
schema: [],
|
|
13
|
+
messages: messageIds
|
|
14
|
+
},
|
|
15
|
+
defaultOptions: [],
|
|
16
|
+
create: (context) => ({
|
|
17
|
+
Literal: (node) => {
|
|
18
|
+
if (node.value !== null) {
|
|
19
|
+
return;
|
|
20
|
+
}
|
|
21
|
+
context.report({
|
|
22
|
+
node,
|
|
23
|
+
messageId: "forbiddenNull"
|
|
24
|
+
});
|
|
25
|
+
},
|
|
26
|
+
TSNullKeyword: (node) => {
|
|
27
|
+
context.report({
|
|
28
|
+
node,
|
|
29
|
+
messageId: "forbiddenNull"
|
|
30
|
+
});
|
|
31
|
+
}
|
|
32
|
+
})
|
|
33
|
+
});
|
|
@@ -0,0 +1,135 @@
|
|
|
1
|
+
import { AST_NODE_TYPES } from "@typescript-eslint/utils";
|
|
2
|
+
import { createRule } from "../utils/create-rule.js";
|
|
3
|
+
const messageIds = Object.freeze({
|
|
4
|
+
forbiddenObjectAsMap: "Plain objects are not allowed as dynamic maps."
|
|
5
|
+
});
|
|
6
|
+
const scopeObjects = new WeakMap();
|
|
7
|
+
const getScopeNames = (scope) => {
|
|
8
|
+
const names = scopeObjects.get(scope);
|
|
9
|
+
if (names !== undefined) {
|
|
10
|
+
return names;
|
|
11
|
+
}
|
|
12
|
+
const created = new Map();
|
|
13
|
+
scopeObjects.set(scope, created);
|
|
14
|
+
return created;
|
|
15
|
+
};
|
|
16
|
+
const unwrapExpression = (node) => {
|
|
17
|
+
if (node.type === AST_NODE_TYPES.TSAsExpression) {
|
|
18
|
+
return unwrapExpression(node.expression);
|
|
19
|
+
}
|
|
20
|
+
if (node.type === AST_NODE_TYPES.TSSatisfiesExpression) {
|
|
21
|
+
return unwrapExpression(node.expression);
|
|
22
|
+
}
|
|
23
|
+
if (node.type === AST_NODE_TYPES.TSNonNullExpression) {
|
|
24
|
+
return unwrapExpression(node.expression);
|
|
25
|
+
}
|
|
26
|
+
return node;
|
|
27
|
+
};
|
|
28
|
+
const isNullLiteral = (node) => node.type === AST_NODE_TYPES.Literal && node.value === null;
|
|
29
|
+
const isObjectCreateNullCall = (node) => {
|
|
30
|
+
if (node.callee.type !== AST_NODE_TYPES.MemberExpression) {
|
|
31
|
+
return false;
|
|
32
|
+
}
|
|
33
|
+
if (node.callee.computed) {
|
|
34
|
+
return false;
|
|
35
|
+
}
|
|
36
|
+
if (node.callee.object.type !== AST_NODE_TYPES.Identifier) {
|
|
37
|
+
return false;
|
|
38
|
+
}
|
|
39
|
+
if (node.callee.object.name !== "Object") {
|
|
40
|
+
return false;
|
|
41
|
+
}
|
|
42
|
+
if (node.callee.property.type !== AST_NODE_TYPES.Identifier) {
|
|
43
|
+
return false;
|
|
44
|
+
}
|
|
45
|
+
if (node.callee.property.name !== "create") {
|
|
46
|
+
return false;
|
|
47
|
+
}
|
|
48
|
+
if (node.arguments.length !== 1) {
|
|
49
|
+
return false;
|
|
50
|
+
}
|
|
51
|
+
const [argument] = node.arguments;
|
|
52
|
+
if (argument === undefined || argument.type === AST_NODE_TYPES.SpreadElement) {
|
|
53
|
+
return false;
|
|
54
|
+
}
|
|
55
|
+
return isNullLiteral(argument);
|
|
56
|
+
};
|
|
57
|
+
const isPlainObjectExpression = (node) => {
|
|
58
|
+
const expression = unwrapExpression(node);
|
|
59
|
+
if (expression.type === AST_NODE_TYPES.ObjectExpression) {
|
|
60
|
+
return true;
|
|
61
|
+
}
|
|
62
|
+
if (expression.type !== AST_NODE_TYPES.CallExpression) {
|
|
63
|
+
return false;
|
|
64
|
+
}
|
|
65
|
+
return isObjectCreateNullCall(expression);
|
|
66
|
+
};
|
|
67
|
+
const isDynamicKey = (node) => {
|
|
68
|
+
if (!node.computed) {
|
|
69
|
+
return false;
|
|
70
|
+
}
|
|
71
|
+
return node.property.type !== AST_NODE_TYPES.Literal;
|
|
72
|
+
};
|
|
73
|
+
const markDeclaredVariable = (context, node, name, isPlainObject) => {
|
|
74
|
+
const scope = context.sourceCode.getScope(node);
|
|
75
|
+
const names = getScopeNames(scope);
|
|
76
|
+
const current = names.get(name) ?? false;
|
|
77
|
+
names.set(name, current || isPlainObject);
|
|
78
|
+
};
|
|
79
|
+
const isPlainObjectVariable = (context, node, name) => {
|
|
80
|
+
let scope = context.sourceCode.getScope(node);
|
|
81
|
+
while (scope !== null) {
|
|
82
|
+
const names = scopeObjects.get(scope);
|
|
83
|
+
if (names !== undefined && names.has(name)) {
|
|
84
|
+
return names.get(name) ?? false;
|
|
85
|
+
}
|
|
86
|
+
scope = scope.upper;
|
|
87
|
+
}
|
|
88
|
+
return false;
|
|
89
|
+
};
|
|
90
|
+
export const noObjectAsMapRule = createRule({
|
|
91
|
+
name: "no-object-as-map",
|
|
92
|
+
meta: {
|
|
93
|
+
type: "problem",
|
|
94
|
+
docs: {
|
|
95
|
+
description: "Disallow using plain objects as dynamic maps."
|
|
96
|
+
},
|
|
97
|
+
schema: [],
|
|
98
|
+
messages: messageIds
|
|
99
|
+
},
|
|
100
|
+
defaultOptions: [],
|
|
101
|
+
create: (context) => ({
|
|
102
|
+
VariableDeclarator: (node) => {
|
|
103
|
+
if (node.id.type !== AST_NODE_TYPES.Identifier) {
|
|
104
|
+
return;
|
|
105
|
+
}
|
|
106
|
+
const isPlainObject = node.init !== undefined &&
|
|
107
|
+
node.init !== null &&
|
|
108
|
+
isPlainObjectExpression(node.init);
|
|
109
|
+
markDeclaredVariable(context, node, node.id.name, isPlainObject);
|
|
110
|
+
},
|
|
111
|
+
MemberExpression: (node) => {
|
|
112
|
+
if (!isDynamicKey(node)) {
|
|
113
|
+
return;
|
|
114
|
+
}
|
|
115
|
+
const object = unwrapExpression(node.object);
|
|
116
|
+
if (object.type === AST_NODE_TYPES.ObjectExpression) {
|
|
117
|
+
context.report({
|
|
118
|
+
node,
|
|
119
|
+
messageId: "forbiddenObjectAsMap"
|
|
120
|
+
});
|
|
121
|
+
return;
|
|
122
|
+
}
|
|
123
|
+
if (object.type !== AST_NODE_TYPES.Identifier) {
|
|
124
|
+
return;
|
|
125
|
+
}
|
|
126
|
+
if (!isPlainObjectVariable(context, node, object.name)) {
|
|
127
|
+
return;
|
|
128
|
+
}
|
|
129
|
+
context.report({
|
|
130
|
+
node,
|
|
131
|
+
messageId: "forbiddenObjectAsMap"
|
|
132
|
+
});
|
|
133
|
+
}
|
|
134
|
+
})
|
|
135
|
+
});
|
|
@@ -0,0 +1,24 @@
|
|
|
1
|
+
import { createRule } from "../utils/create-rule.js";
|
|
2
|
+
const messageIds = Object.freeze({
|
|
3
|
+
forbiddenObjectSpread: "Object spread is not allowed."
|
|
4
|
+
});
|
|
5
|
+
export const noObjectSpreadRule = createRule({
|
|
6
|
+
name: "no-object-spread",
|
|
7
|
+
meta: {
|
|
8
|
+
type: "problem",
|
|
9
|
+
docs: {
|
|
10
|
+
description: "Disallow object spread in object literals."
|
|
11
|
+
},
|
|
12
|
+
schema: [],
|
|
13
|
+
messages: messageIds
|
|
14
|
+
},
|
|
15
|
+
defaultOptions: [],
|
|
16
|
+
create: (context) => ({
|
|
17
|
+
"ObjectExpression > SpreadElement": (node) => {
|
|
18
|
+
context.report({
|
|
19
|
+
node,
|
|
20
|
+
messageId: "forbiddenObjectSpread"
|
|
21
|
+
});
|
|
22
|
+
}
|
|
23
|
+
})
|
|
24
|
+
});
|
|
@@ -0,0 +1,98 @@
|
|
|
1
|
+
import { AST_NODE_TYPES } from "@typescript-eslint/utils";
|
|
2
|
+
import { createRule } from "../utils/create-rule.js";
|
|
3
|
+
const forbiddenNames = new Set(["toJSON", "toString"]);
|
|
4
|
+
const messageIds = Object.freeze({
|
|
5
|
+
forbiddenObjectStringificationHook: "Runtime `.toString` and `.toJSON` are not allowed."
|
|
6
|
+
});
|
|
7
|
+
const getPropertyName = (node) => {
|
|
8
|
+
if (node.type === AST_NODE_TYPES.Identifier) {
|
|
9
|
+
return node.name;
|
|
10
|
+
}
|
|
11
|
+
if (node.type === AST_NODE_TYPES.Literal && typeof node.value === "string") {
|
|
12
|
+
return node.value;
|
|
13
|
+
}
|
|
14
|
+
if (node.type === AST_NODE_TYPES.TemplateLiteral &&
|
|
15
|
+
node.expressions.length === 0 &&
|
|
16
|
+
node.quasis.length === 1) {
|
|
17
|
+
return node.quasis[0]?.value.cooked ?? undefined;
|
|
18
|
+
}
|
|
19
|
+
return undefined;
|
|
20
|
+
};
|
|
21
|
+
const isWriteTarget = (node) => {
|
|
22
|
+
const parent = node.parent;
|
|
23
|
+
if (parent === null) {
|
|
24
|
+
return false;
|
|
25
|
+
}
|
|
26
|
+
if (parent.type === AST_NODE_TYPES.AssignmentExpression) {
|
|
27
|
+
return parent.left === node;
|
|
28
|
+
}
|
|
29
|
+
return false;
|
|
30
|
+
};
|
|
31
|
+
export const noObjectStringificationHookRule = createRule({
|
|
32
|
+
name: "no-object-stringification-hook",
|
|
33
|
+
meta: {
|
|
34
|
+
type: "problem",
|
|
35
|
+
docs: {
|
|
36
|
+
description: "Disallow runtime `toString` and `toJSON` hooks on objects and classes."
|
|
37
|
+
},
|
|
38
|
+
schema: [],
|
|
39
|
+
messages: messageIds
|
|
40
|
+
},
|
|
41
|
+
defaultOptions: [],
|
|
42
|
+
create: (context) => {
|
|
43
|
+
const report = (node) => {
|
|
44
|
+
context.report({
|
|
45
|
+
node,
|
|
46
|
+
messageId: "forbiddenObjectStringificationHook"
|
|
47
|
+
});
|
|
48
|
+
};
|
|
49
|
+
return {
|
|
50
|
+
AssignmentExpression: (node) => {
|
|
51
|
+
if (node.left.type !== AST_NODE_TYPES.MemberExpression) {
|
|
52
|
+
return;
|
|
53
|
+
}
|
|
54
|
+
const name = getPropertyName(node.left.property);
|
|
55
|
+
if (name === undefined || !forbiddenNames.has(name)) {
|
|
56
|
+
return;
|
|
57
|
+
}
|
|
58
|
+
report(node.left.property);
|
|
59
|
+
},
|
|
60
|
+
MethodDefinition: (node) => {
|
|
61
|
+
const name = getPropertyName(node.key);
|
|
62
|
+
if (name === undefined || !forbiddenNames.has(name)) {
|
|
63
|
+
return;
|
|
64
|
+
}
|
|
65
|
+
report(node.key);
|
|
66
|
+
},
|
|
67
|
+
MemberExpression: (node) => {
|
|
68
|
+
if (isWriteTarget(node)) {
|
|
69
|
+
return;
|
|
70
|
+
}
|
|
71
|
+
const name = getPropertyName(node.property);
|
|
72
|
+
if (name === undefined || !forbiddenNames.has(name)) {
|
|
73
|
+
return;
|
|
74
|
+
}
|
|
75
|
+
report(node.property);
|
|
76
|
+
},
|
|
77
|
+
ObjectExpression: (node) => {
|
|
78
|
+
for (const property of node.properties) {
|
|
79
|
+
if (property.type !== AST_NODE_TYPES.Property) {
|
|
80
|
+
continue;
|
|
81
|
+
}
|
|
82
|
+
const name = getPropertyName(property.key);
|
|
83
|
+
if (name === undefined || !forbiddenNames.has(name)) {
|
|
84
|
+
continue;
|
|
85
|
+
}
|
|
86
|
+
report(property.key);
|
|
87
|
+
}
|
|
88
|
+
},
|
|
89
|
+
PropertyDefinition: (node) => {
|
|
90
|
+
const name = getPropertyName(node.key);
|
|
91
|
+
if (name === undefined || !forbiddenNames.has(name)) {
|
|
92
|
+
return;
|
|
93
|
+
}
|
|
94
|
+
report(node.key);
|
|
95
|
+
}
|
|
96
|
+
};
|
|
97
|
+
}
|
|
98
|
+
});
|
|
@@ -0,0 +1,101 @@
|
|
|
1
|
+
import { AST_NODE_TYPES } from "@typescript-eslint/utils";
|
|
2
|
+
import { createRule } from "../utils/create-rule.js";
|
|
3
|
+
const messageIds = Object.freeze({
|
|
4
|
+
forbiddenOptionalParameter: "Optional parameters are not allowed."
|
|
5
|
+
});
|
|
6
|
+
const isOptionalIdentifier = (node) => node.type === AST_NODE_TYPES.Identifier && node.optional;
|
|
7
|
+
const reportOptionalParameters = (report, params) => {
|
|
8
|
+
params.forEach((param) => {
|
|
9
|
+
if (!isOptionalIdentifier(param)) {
|
|
10
|
+
return;
|
|
11
|
+
}
|
|
12
|
+
report(param);
|
|
13
|
+
});
|
|
14
|
+
};
|
|
15
|
+
export const noOptionalParameterRule = createRule({
|
|
16
|
+
name: "no-optional-parameter",
|
|
17
|
+
meta: {
|
|
18
|
+
type: "problem",
|
|
19
|
+
docs: {
|
|
20
|
+
description: "Disallow optional parameters marked with `?` in functions and constructor parameter properties."
|
|
21
|
+
},
|
|
22
|
+
schema: [],
|
|
23
|
+
messages: messageIds
|
|
24
|
+
},
|
|
25
|
+
defaultOptions: [],
|
|
26
|
+
create: (context) => ({
|
|
27
|
+
ArrowFunctionExpression: (node) => {
|
|
28
|
+
reportOptionalParameters((param) => {
|
|
29
|
+
context.report({
|
|
30
|
+
node: param,
|
|
31
|
+
messageId: "forbiddenOptionalParameter"
|
|
32
|
+
});
|
|
33
|
+
}, node.params);
|
|
34
|
+
},
|
|
35
|
+
FunctionDeclaration: (node) => {
|
|
36
|
+
reportOptionalParameters((param) => {
|
|
37
|
+
context.report({
|
|
38
|
+
node: param,
|
|
39
|
+
messageId: "forbiddenOptionalParameter"
|
|
40
|
+
});
|
|
41
|
+
}, node.params);
|
|
42
|
+
},
|
|
43
|
+
FunctionExpression: (node) => {
|
|
44
|
+
reportOptionalParameters((param) => {
|
|
45
|
+
context.report({
|
|
46
|
+
node: param,
|
|
47
|
+
messageId: "forbiddenOptionalParameter"
|
|
48
|
+
});
|
|
49
|
+
}, node.params);
|
|
50
|
+
},
|
|
51
|
+
TSCallSignatureDeclaration: (node) => {
|
|
52
|
+
reportOptionalParameters((param) => {
|
|
53
|
+
context.report({
|
|
54
|
+
node: param,
|
|
55
|
+
messageId: "forbiddenOptionalParameter"
|
|
56
|
+
});
|
|
57
|
+
}, node.params);
|
|
58
|
+
},
|
|
59
|
+
TSConstructSignatureDeclaration: (node) => {
|
|
60
|
+
reportOptionalParameters((param) => {
|
|
61
|
+
context.report({
|
|
62
|
+
node: param,
|
|
63
|
+
messageId: "forbiddenOptionalParameter"
|
|
64
|
+
});
|
|
65
|
+
}, node.params);
|
|
66
|
+
},
|
|
67
|
+
TSDeclareFunction: (node) => {
|
|
68
|
+
reportOptionalParameters((param) => {
|
|
69
|
+
context.report({
|
|
70
|
+
node: param,
|
|
71
|
+
messageId: "forbiddenOptionalParameter"
|
|
72
|
+
});
|
|
73
|
+
}, node.params);
|
|
74
|
+
},
|
|
75
|
+
TSFunctionType: (node) => {
|
|
76
|
+
reportOptionalParameters((param) => {
|
|
77
|
+
context.report({
|
|
78
|
+
node: param,
|
|
79
|
+
messageId: "forbiddenOptionalParameter"
|
|
80
|
+
});
|
|
81
|
+
}, node.params);
|
|
82
|
+
},
|
|
83
|
+
TSMethodSignature: (node) => {
|
|
84
|
+
reportOptionalParameters((param) => {
|
|
85
|
+
context.report({
|
|
86
|
+
node: param,
|
|
87
|
+
messageId: "forbiddenOptionalParameter"
|
|
88
|
+
});
|
|
89
|
+
}, node.params);
|
|
90
|
+
},
|
|
91
|
+
TSParameterProperty: (node) => {
|
|
92
|
+
if (!isOptionalIdentifier(node.parameter)) {
|
|
93
|
+
return;
|
|
94
|
+
}
|
|
95
|
+
context.report({
|
|
96
|
+
node: node.parameter,
|
|
97
|
+
messageId: "forbiddenOptionalParameter"
|
|
98
|
+
});
|
|
99
|
+
}
|
|
100
|
+
})
|
|
101
|
+
});
|
|
@@ -0,0 +1,27 @@
|
|
|
1
|
+
import { createRule } from "../utils/create-rule.js";
|
|
2
|
+
const messageIds = Object.freeze({
|
|
3
|
+
forbiddenOptionalProperty: "Optional object fields are not allowed."
|
|
4
|
+
});
|
|
5
|
+
export const noOptionalPropertyRule = createRule({
|
|
6
|
+
name: "no-optional-property",
|
|
7
|
+
meta: {
|
|
8
|
+
type: "problem",
|
|
9
|
+
docs: {
|
|
10
|
+
description: "Disallow optional object fields marked with `?`."
|
|
11
|
+
},
|
|
12
|
+
schema: [],
|
|
13
|
+
messages: messageIds
|
|
14
|
+
},
|
|
15
|
+
defaultOptions: [],
|
|
16
|
+
create: (context) => ({
|
|
17
|
+
TSPropertySignature: (node) => {
|
|
18
|
+
if (!node.optional) {
|
|
19
|
+
return;
|
|
20
|
+
}
|
|
21
|
+
context.report({
|
|
22
|
+
node,
|
|
23
|
+
messageId: "forbiddenOptionalProperty"
|
|
24
|
+
});
|
|
25
|
+
}
|
|
26
|
+
})
|
|
27
|
+
});
|
|
@@ -0,0 +1,107 @@
|
|
|
1
|
+
import { AST_NODE_TYPES } from "@typescript-eslint/utils";
|
|
2
|
+
import { createRule } from "../utils/create-rule.js";
|
|
3
|
+
const messageIds = Object.freeze({
|
|
4
|
+
forbiddenOverload: "Overloads are not allowed."
|
|
5
|
+
});
|
|
6
|
+
const getText = (sourceText, range) => sourceText.slice(range[0], range[1]);
|
|
7
|
+
const getTopLevelEntries = (body) => {
|
|
8
|
+
const entries = [];
|
|
9
|
+
body.forEach((statement) => {
|
|
10
|
+
if (statement.type !== AST_NODE_TYPES.FunctionDeclaration &&
|
|
11
|
+
statement.type !== AST_NODE_TYPES.TSDeclareFunction) {
|
|
12
|
+
return;
|
|
13
|
+
}
|
|
14
|
+
if (statement.id === null) {
|
|
15
|
+
return;
|
|
16
|
+
}
|
|
17
|
+
entries.push({
|
|
18
|
+
name: statement.id.name,
|
|
19
|
+
node: statement
|
|
20
|
+
});
|
|
21
|
+
});
|
|
22
|
+
return entries;
|
|
23
|
+
};
|
|
24
|
+
const getClassEntries = (sourceText, body) => {
|
|
25
|
+
const entries = [];
|
|
26
|
+
body.forEach((member) => {
|
|
27
|
+
if (member.type !== AST_NODE_TYPES.MethodDefinition) {
|
|
28
|
+
return;
|
|
29
|
+
}
|
|
30
|
+
if (member.kind === "get" || member.kind === "set") {
|
|
31
|
+
return;
|
|
32
|
+
}
|
|
33
|
+
entries.push({
|
|
34
|
+
name: getText(sourceText, member.key.range),
|
|
35
|
+
node: member
|
|
36
|
+
});
|
|
37
|
+
});
|
|
38
|
+
return entries;
|
|
39
|
+
};
|
|
40
|
+
const getTypeEntries = (sourceText, body) => {
|
|
41
|
+
const entries = [];
|
|
42
|
+
body.forEach((member) => {
|
|
43
|
+
if (member.type === AST_NODE_TYPES.TSMethodSignature) {
|
|
44
|
+
entries.push({
|
|
45
|
+
name: getText(sourceText, member.key.range),
|
|
46
|
+
node: member
|
|
47
|
+
});
|
|
48
|
+
return;
|
|
49
|
+
}
|
|
50
|
+
if (member.type === AST_NODE_TYPES.TSCallSignatureDeclaration) {
|
|
51
|
+
entries.push({
|
|
52
|
+
name: "(call)",
|
|
53
|
+
node: member
|
|
54
|
+
});
|
|
55
|
+
return;
|
|
56
|
+
}
|
|
57
|
+
if (member.type === AST_NODE_TYPES.TSConstructSignatureDeclaration) {
|
|
58
|
+
entries.push({
|
|
59
|
+
name: "(construct)",
|
|
60
|
+
node: member
|
|
61
|
+
});
|
|
62
|
+
}
|
|
63
|
+
});
|
|
64
|
+
return entries;
|
|
65
|
+
};
|
|
66
|
+
export const noOverloadRule = createRule({
|
|
67
|
+
name: "no-overload",
|
|
68
|
+
meta: {
|
|
69
|
+
type: "problem",
|
|
70
|
+
docs: {
|
|
71
|
+
description: "Disallow function and method overloads."
|
|
72
|
+
},
|
|
73
|
+
schema: [],
|
|
74
|
+
messages: messageIds
|
|
75
|
+
},
|
|
76
|
+
defaultOptions: [],
|
|
77
|
+
create: (context) => {
|
|
78
|
+
const sourceText = context.sourceCode.getText();
|
|
79
|
+
const reportDuplicates = (entries) => {
|
|
80
|
+
const seen = new Set();
|
|
81
|
+
entries.forEach((entry) => {
|
|
82
|
+
if (seen.has(entry.name)) {
|
|
83
|
+
context.report({
|
|
84
|
+
node: entry.node,
|
|
85
|
+
messageId: "forbiddenOverload"
|
|
86
|
+
});
|
|
87
|
+
return;
|
|
88
|
+
}
|
|
89
|
+
seen.add(entry.name);
|
|
90
|
+
});
|
|
91
|
+
};
|
|
92
|
+
return {
|
|
93
|
+
Program: (node) => {
|
|
94
|
+
reportDuplicates(getTopLevelEntries(node.body));
|
|
95
|
+
},
|
|
96
|
+
ClassBody: (node) => {
|
|
97
|
+
reportDuplicates(getClassEntries(sourceText, node.body));
|
|
98
|
+
},
|
|
99
|
+
TSInterfaceBody: (node) => {
|
|
100
|
+
reportDuplicates(getTypeEntries(sourceText, node.body));
|
|
101
|
+
},
|
|
102
|
+
TSTypeLiteral: (node) => {
|
|
103
|
+
reportDuplicates(getTypeEntries(sourceText, node.members));
|
|
104
|
+
}
|
|
105
|
+
};
|
|
106
|
+
}
|
|
107
|
+
});
|
|
@@ -0,0 +1,91 @@
|
|
|
1
|
+
import { AST_NODE_TYPES } from "@typescript-eslint/utils";
|
|
2
|
+
import { createRule } from "../utils/create-rule.js";
|
|
3
|
+
const messageIds = Object.freeze({
|
|
4
|
+
forbiddenParamAssign: "Assigning to function parameters is not allowed."
|
|
5
|
+
});
|
|
6
|
+
const getPatternNames = (node) => {
|
|
7
|
+
if (node.type === AST_NODE_TYPES.Identifier) {
|
|
8
|
+
return [node.name];
|
|
9
|
+
}
|
|
10
|
+
if (node.type === AST_NODE_TYPES.AssignmentPattern) {
|
|
11
|
+
return getPatternNames(node.left);
|
|
12
|
+
}
|
|
13
|
+
if (node.type === AST_NODE_TYPES.RestElement) {
|
|
14
|
+
return getPatternNames(node.argument);
|
|
15
|
+
}
|
|
16
|
+
if (node.type === AST_NODE_TYPES.ArrayPattern) {
|
|
17
|
+
return node.elements.flatMap((element) => {
|
|
18
|
+
if (element === null) {
|
|
19
|
+
return [];
|
|
20
|
+
}
|
|
21
|
+
return getPatternNames(element);
|
|
22
|
+
});
|
|
23
|
+
}
|
|
24
|
+
if (node.type === AST_NODE_TYPES.ObjectPattern) {
|
|
25
|
+
return node.properties.flatMap((property) => {
|
|
26
|
+
if (property.type === AST_NODE_TYPES.RestElement) {
|
|
27
|
+
return getPatternNames(property.argument);
|
|
28
|
+
}
|
|
29
|
+
return getPatternNames(property.value);
|
|
30
|
+
});
|
|
31
|
+
}
|
|
32
|
+
return [];
|
|
33
|
+
};
|
|
34
|
+
const getParamNames = (params) => new Set(params.flatMap((param) => getPatternNames(param)));
|
|
35
|
+
const getEnclosingFunction = (node) => {
|
|
36
|
+
let parent = node.parent;
|
|
37
|
+
while (parent !== undefined) {
|
|
38
|
+
if (parent.type === AST_NODE_TYPES.ArrowFunctionExpression ||
|
|
39
|
+
parent.type === AST_NODE_TYPES.FunctionDeclaration ||
|
|
40
|
+
parent.type === AST_NODE_TYPES.FunctionExpression) {
|
|
41
|
+
return parent;
|
|
42
|
+
}
|
|
43
|
+
parent = parent.parent;
|
|
44
|
+
}
|
|
45
|
+
return undefined;
|
|
46
|
+
};
|
|
47
|
+
const isAssignedParam = (node) => {
|
|
48
|
+
const fn = getEnclosingFunction(node);
|
|
49
|
+
if (fn === undefined) {
|
|
50
|
+
return false;
|
|
51
|
+
}
|
|
52
|
+
return getParamNames(fn.params).has(node.name);
|
|
53
|
+
};
|
|
54
|
+
export const noParamAssignRule = createRule({
|
|
55
|
+
name: "no-param-assign",
|
|
56
|
+
meta: {
|
|
57
|
+
type: "problem",
|
|
58
|
+
docs: {
|
|
59
|
+
description: "Disallow assigning to function parameters unless mutability is required for performance."
|
|
60
|
+
},
|
|
61
|
+
schema: [],
|
|
62
|
+
messages: messageIds
|
|
63
|
+
},
|
|
64
|
+
defaultOptions: [],
|
|
65
|
+
create: (context) => ({
|
|
66
|
+
AssignmentExpression: (node) => {
|
|
67
|
+
if (node.left.type !== AST_NODE_TYPES.Identifier) {
|
|
68
|
+
return;
|
|
69
|
+
}
|
|
70
|
+
if (!isAssignedParam(node.left)) {
|
|
71
|
+
return;
|
|
72
|
+
}
|
|
73
|
+
context.report({
|
|
74
|
+
node: node.left,
|
|
75
|
+
messageId: "forbiddenParamAssign"
|
|
76
|
+
});
|
|
77
|
+
},
|
|
78
|
+
UpdateExpression: (node) => {
|
|
79
|
+
if (node.argument.type !== AST_NODE_TYPES.Identifier) {
|
|
80
|
+
return;
|
|
81
|
+
}
|
|
82
|
+
if (!isAssignedParam(node.argument)) {
|
|
83
|
+
return;
|
|
84
|
+
}
|
|
85
|
+
context.report({
|
|
86
|
+
node: node.argument,
|
|
87
|
+
messageId: "forbiddenParamAssign"
|
|
88
|
+
});
|
|
89
|
+
}
|
|
90
|
+
})
|
|
91
|
+
});
|