@darraghor/eslint-plugin-nestjs-typed 4.2.1 → 4.3.0
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/README.md
CHANGED
|
@@ -39,6 +39,7 @@ Awsome! [Click here](https://github.com/darraghoriordan/eslint-plugin-nestjs-typ
|
|
|
39
39
|
| | [`validated-non-primitive-property-needs-type-decorator`](./src/docs/rules/validated-non-primitive-property-needs-type-decorator.md) | Y |
|
|
40
40
|
| | [`all-properties-are-whitelisted`](./src/docs/rules/all-properties-are-whitelisted.md) | Y |
|
|
41
41
|
| | [`all-properties-have-explicit-defined`](./src/docs/rules/all-properties-have-explicit-defined.md) | Y |
|
|
42
|
+
| | [`no-duplicate-decorators`](./src/docs/rules/no-duplicate-decorators.md) | Y |
|
|
42
43
|
| | | |
|
|
43
44
|
| Security | [`validation-pipe-should-forbid-unknown`](./src/docs/rules/validation-pipe-should-use-forbid-unknown.md) | Y |
|
|
44
45
|
| | [`api-methods-should-be-guarded`](./src/docs/rules/api-methods-should-be-guarded.md) | N |
|
|
@@ -25,6 +25,7 @@ module.exports = {
|
|
|
25
25
|
"@darraghor/nestjs-typed/api-methods-should-be-guarded": "off",
|
|
26
26
|
"@darraghor/nestjs-typed/api-method-should-specify-api-operation": "off",
|
|
27
27
|
"@darraghor/nestjs-typed/sort-module-metadata-arrays": "off",
|
|
28
|
+
"@darraghor/nestjs-typed/no-duplicate-decorators": "error",
|
|
28
29
|
},
|
|
29
30
|
};
|
|
30
|
-
//# sourceMappingURL=data:application/json;base64,
|
|
31
|
+
//# sourceMappingURL=data:application/json;base64,eyJ2ZXJzaW9uIjozLCJmaWxlIjoicmVjb21tZW5kZWQuanMiLCJzb3VyY2VSb290IjoiIiwic291cmNlcyI6WyIuLi8uLi9zcmMvY29uZmlncy9yZWNvbW1lbmRlZC50cyJdLCJuYW1lcyI6W10sIm1hcHBpbmdzIjoiO0FBQUEsaUJBQVM7SUFDTCxNQUFNLEVBQUUsMkJBQTJCO0lBQ25DLGFBQWEsRUFBRSxFQUFDLFVBQVUsRUFBRSxRQUFRLEVBQUM7SUFDckMsS0FBSyxFQUFFO1FBQ0gsMkVBQTJFLEVBQ3ZFLE9BQU87UUFDWCx1REFBdUQsRUFBRTtZQUNyRCxPQUFPO1lBQ1A7Z0JBQ0ksR0FBRyxFQUFFLENBQUMsYUFBYSxDQUFDO2dCQUNwQixlQUFlLEVBQUUsQ0FBQyxjQUFjLEVBQUUsUUFBUSxFQUFFLFFBQVEsQ0FBQzthQUN4RDtTQUNKO1FBQ0QsbUVBQW1FLEVBQy9ELE9BQU87UUFDWCxnRUFBZ0UsRUFDNUQsT0FBTztRQUNYLDREQUE0RCxFQUFFLE9BQU87UUFDckUsMERBQTBELEVBQUUsT0FBTztRQUNuRSx1RUFBdUUsRUFDbkUsT0FBTztRQUNYLDhEQUE4RCxFQUFFLE9BQU87UUFDdkUsa0VBQWtFLEVBQzlELE9BQU87UUFDWCwrRUFBK0UsRUFDM0UsT0FBTztRQUNYLGtFQUFrRSxFQUM5RCxPQUFPO1FBQ1gsd0RBQXdELEVBQUUsT0FBTztRQUNqRSw4REFBOEQsRUFBRSxPQUFPO1FBQ3ZFLHVEQUF1RCxFQUFFLEtBQUs7UUFDOUQsaUVBQWlFLEVBQzdELEtBQUs7UUFDVCxxREFBcUQsRUFBRSxLQUFLO1FBQzVELGlEQUFpRCxFQUFFLE9BQU87S0FDN0Q7Q0FDSixDQUFDIn0=
|
package/dist/rules/index.js
CHANGED
|
@@ -19,10 +19,12 @@ const allPropertiesHaveExplicitDefined_1 = __importDefault(require("./allPropert
|
|
|
19
19
|
const apiMethodsShouldBeGuarded_1 = __importDefault(require("./apiMethodsShouldBeGuarded/apiMethodsShouldBeGuarded"));
|
|
20
20
|
const apiMethodsShouldSpecifyApiOperation_1 = __importDefault(require("./apiMethodsShouldSpecifyApiOperation/apiMethodsShouldSpecifyApiOperation"));
|
|
21
21
|
const sortModuleMetadataArrays_1 = __importDefault(require("./sortModuleMetadataArrays/sortModuleMetadataArrays"));
|
|
22
|
+
const noDuplicateDecorators_1 = __importDefault(require("./noDuplicateDecorators/noDuplicateDecorators"));
|
|
22
23
|
const allRules = {
|
|
23
24
|
"all-properties-have-explicit-defined": allPropertiesHaveExplicitDefined_1.default,
|
|
24
25
|
"api-property-matches-property-optionality": apiPropertyMatchesPropertyOptionality_1.default,
|
|
25
26
|
"injectable-should-be-provided": injectableShouldBeProvided_1.default,
|
|
27
|
+
"no-duplicate-decorators": noDuplicateDecorators_1.default,
|
|
26
28
|
"provided-injected-should-match-factory-parameters": ProviderInjectedShouldMatchFactory_1.default,
|
|
27
29
|
"controllers-should-supply-api-tags": controllerDecoratedHasApiTags_1.default,
|
|
28
30
|
"api-method-should-specify-api-response": apiMethodsShouldSpecifyApiResponse_1.default,
|
|
@@ -38,4 +40,4 @@ const allRules = {
|
|
|
38
40
|
"sort-module-metadata-arrays": sortModuleMetadataArrays_1.default,
|
|
39
41
|
};
|
|
40
42
|
exports.default = allRules;
|
|
41
|
-
//# sourceMappingURL=data:application/json;base64,
|
|
43
|
+
//# sourceMappingURL=data:application/json;base64,eyJ2ZXJzaW9uIjozLCJmaWxlIjoiaW5kZXguanMiLCJzb3VyY2VSb290IjoiIiwic291cmNlcyI6WyIuLi8uLi9zcmMvcnVsZXMvaW5kZXgudHMiXSwibmFtZXMiOltdLCJtYXBwaW5ncyI6Ijs7Ozs7QUFBQSwwSEFBa0c7QUFDbEcsaUpBQW1JO0FBQ25JLDBKQUFrSTtBQUNsSSxrSUFBMEc7QUFDMUcsaUpBQXlIO0FBQ3pILCtIQUF1RztBQUN2RyxnS0FBd0k7QUFDeEksbUpBQXVIO0FBQ3ZILG9KQUFnSTtBQUNoSSx1SkFBK0g7QUFDL0gsaUpBQXlIO0FBQ3pILDRIQUFvRztBQUNwRywySUFBbUg7QUFDbkgsc0hBQThGO0FBQzlGLG9KQUE0SDtBQUM1SCxtSEFBMkY7QUFDM0YsMEdBQWtGO0FBQ2xGLE1BQU0sUUFBUSxHQUFHO0lBQ2Isc0NBQXNDLEVBQUUsMENBQWdDO0lBQ3hFLDJDQUEyQyxFQUN2QywrQ0FBcUM7SUFDekMsK0JBQStCLEVBQUUsb0NBQTBCO0lBQzNELHlCQUF5QixFQUFFLCtCQUFxQjtJQUNoRCxtREFBbUQsRUFDL0MsNENBQTRDO0lBQ2hELG9DQUFvQyxFQUFFLHVDQUE2QjtJQUNuRSx3Q0FBd0MsRUFDcEMsNENBQWtDO0lBQ3RDLHlDQUF5QyxFQUNyQyw2Q0FBbUM7SUFDdkMsa0NBQWtDLEVBQUUsc0NBQTRCO0lBQ2hFLCtDQUErQyxFQUMzQyxpREFBdUM7SUFDM0Msc0NBQXNDLEVBQUUsOENBQWdDO0lBQ3hFLDBDQUEwQyxFQUN0Qyw2Q0FBdUM7SUFDM0MsdURBQXVELEVBQ25ELDZDQUFtQztJQUN2QywwQ0FBMEMsRUFDdEMsNENBQWtDO0lBQ3RDLGdDQUFnQyxFQUFFLHFDQUEyQjtJQUM3RCwrQkFBK0IsRUFBRSxtQ0FBeUI7SUFDMUQsNkJBQTZCLEVBQUUsa0NBQXdCO0NBQzFELENBQUM7QUFFRixrQkFBZSxRQUFRLENBQUMifQ==
|
|
@@ -0,0 +1,100 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.standardDecoratorsToValidate = exports.shouldTrigger = void 0;
|
|
4
|
+
/* eslint-disable @typescript-eslint/no-unsafe-member-access */
|
|
5
|
+
/* eslint-disable unicorn/prevent-abbreviations */
|
|
6
|
+
const createRule_1 = require("../../utils/createRule");
|
|
7
|
+
const utils_1 = require("@typescript-eslint/utils");
|
|
8
|
+
const shouldTrigger = () => {
|
|
9
|
+
return true;
|
|
10
|
+
};
|
|
11
|
+
exports.shouldTrigger = shouldTrigger;
|
|
12
|
+
exports.standardDecoratorsToValidate = ["DiscoverDecorator"];
|
|
13
|
+
const rule = (0, createRule_1.createRule)({
|
|
14
|
+
name: "no-duplicate-decorators",
|
|
15
|
+
meta: {
|
|
16
|
+
docs: {
|
|
17
|
+
description: "Some decorators should only be used once on a property or class. This rule enforces that.",
|
|
18
|
+
requiresTypeChecking: false,
|
|
19
|
+
},
|
|
20
|
+
messages: {
|
|
21
|
+
noDuplicateDecorators: "You have listed the same decorator more than once. Was this intentional?",
|
|
22
|
+
},
|
|
23
|
+
schema: [
|
|
24
|
+
{
|
|
25
|
+
type: "object",
|
|
26
|
+
properties: {
|
|
27
|
+
customList: {
|
|
28
|
+
description: "A list of custom decorators that this rule will validate for duplicates",
|
|
29
|
+
type: "array",
|
|
30
|
+
minItems: 0,
|
|
31
|
+
items: {
|
|
32
|
+
type: "string",
|
|
33
|
+
minLength: 1,
|
|
34
|
+
},
|
|
35
|
+
},
|
|
36
|
+
},
|
|
37
|
+
},
|
|
38
|
+
],
|
|
39
|
+
hasSuggestions: true,
|
|
40
|
+
type: "suggestion",
|
|
41
|
+
},
|
|
42
|
+
defaultOptions: [
|
|
43
|
+
{
|
|
44
|
+
customList: new Array(),
|
|
45
|
+
},
|
|
46
|
+
],
|
|
47
|
+
create(context) {
|
|
48
|
+
const customListArrayItem = context.options?.[0];
|
|
49
|
+
let decoratorsToValidate = customListArrayItem?.customList;
|
|
50
|
+
if (!customListArrayItem?.customList ||
|
|
51
|
+
customListArrayItem?.customList.length === 0) {
|
|
52
|
+
decoratorsToValidate = exports.standardDecoratorsToValidate;
|
|
53
|
+
}
|
|
54
|
+
return {
|
|
55
|
+
["PropertyDefinition,ClassDeclaration"](node) {
|
|
56
|
+
// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-explicit-any
|
|
57
|
+
const allDecorators = node?.decorators;
|
|
58
|
+
if (allDecorators && allDecorators.length > 1) {
|
|
59
|
+
const decoratorNames = allDecorators.map((decorator) => {
|
|
60
|
+
if (decorator.expression.type ===
|
|
61
|
+
utils_1.TSESTree.AST_NODE_TYPES.CallExpression &&
|
|
62
|
+
decorator.expression.callee &&
|
|
63
|
+
decorator.expression.callee.type ===
|
|
64
|
+
utils_1.TSESTree.AST_NODE_TYPES.Identifier) {
|
|
65
|
+
// eslint-disable-next-line @typescript-eslint/no-unsafe-return
|
|
66
|
+
return decorator.expression.callee.name;
|
|
67
|
+
}
|
|
68
|
+
return "";
|
|
69
|
+
});
|
|
70
|
+
const duplicateDecorators = decoratorNames.filter((decoratorName, index) => {
|
|
71
|
+
return (decoratorName &&
|
|
72
|
+
decoratorNames.indexOf(decoratorName) !== index);
|
|
73
|
+
});
|
|
74
|
+
if (duplicateDecorators.length > 0 &&
|
|
75
|
+
decoratorsToValidate.some((decoratorToValidate) => duplicateDecorators.includes(decoratorToValidate))) {
|
|
76
|
+
context.report({
|
|
77
|
+
node: node,
|
|
78
|
+
messageId: "noDuplicateDecorators",
|
|
79
|
+
suggest: [
|
|
80
|
+
{
|
|
81
|
+
messageId: "noDuplicateDecorators",
|
|
82
|
+
fix: (fixer) => {
|
|
83
|
+
const decoratorsToKeep = decoratorNames.filter((decoratorName, index) => {
|
|
84
|
+
return (decoratorName &&
|
|
85
|
+
decoratorNames.indexOf(decoratorName) === index);
|
|
86
|
+
});
|
|
87
|
+
const decoratorsToKeepText = decoratorsToKeep.join(", ");
|
|
88
|
+
return fixer.replaceText(node, decoratorsToKeepText);
|
|
89
|
+
},
|
|
90
|
+
},
|
|
91
|
+
],
|
|
92
|
+
});
|
|
93
|
+
}
|
|
94
|
+
}
|
|
95
|
+
},
|
|
96
|
+
};
|
|
97
|
+
},
|
|
98
|
+
});
|
|
99
|
+
exports.default = rule;
|
|
100
|
+
//# sourceMappingURL=data:application/json;base64,eyJ2ZXJzaW9uIjozLCJmaWxlIjoibm9EdXBsaWNhdGVEZWNvcmF0b3JzLmpzIiwic291cmNlUm9vdCI6IiIsInNvdXJjZXMiOlsiLi4vLi4vLi4vc3JjL3J1bGVzL25vRHVwbGljYXRlRGVjb3JhdG9ycy9ub0R1cGxpY2F0ZURlY29yYXRvcnMudHMiXSwibmFtZXMiOltdLCJtYXBwaW5ncyI6Ijs7O0FBQUEsK0RBQStEO0FBQy9ELGtEQUFrRDtBQUNsRCx1REFBa0Q7QUFFbEQsb0RBQWtEO0FBTzNDLE1BQU0sYUFBYSxHQUFHLEdBQVksRUFBRTtJQUN2QyxPQUFPLElBQUksQ0FBQztBQUNoQixDQUFDLENBQUM7QUFGVyxRQUFBLGFBQWEsaUJBRXhCO0FBQ1csUUFBQSw0QkFBNEIsR0FBRyxDQUFDLG1CQUFtQixDQUFDLENBQUM7QUFDbEUsTUFBTSxJQUFJLEdBQUcsSUFBQSx1QkFBVSxFQUF3RDtJQUMzRSxJQUFJLEVBQUUseUJBQXlCO0lBQy9CLElBQUksRUFBRTtRQUNGLElBQUksRUFBRTtZQUNGLFdBQVcsRUFDUCwyRkFBMkY7WUFDL0Ysb0JBQW9CLEVBQUUsS0FBSztTQUM5QjtRQUNELFFBQVEsRUFBRTtZQUNOLHFCQUFxQixFQUNqQiwwRUFBMEU7U0FDakY7UUFDRCxNQUFNLEVBQUU7WUFDSjtnQkFDSSxJQUFJLEVBQUUsUUFBK0I7Z0JBQ3JDLFVBQVUsRUFBRTtvQkFDUixVQUFVLEVBQUU7d0JBQ1IsV0FBVyxFQUNQLHlFQUF5RTt3QkFDN0UsSUFBSSxFQUFFLE9BQThCO3dCQUNwQyxRQUFRLEVBQUUsQ0FBQzt3QkFDWCxLQUFLLEVBQUU7NEJBQ0gsSUFBSSxFQUFFLFFBQStCOzRCQUNyQyxTQUFTLEVBQUUsQ0FBQzt5QkFDZjtxQkFDSjtpQkFDSjthQUNKO1NBQ0o7UUFDRCxjQUFjLEVBQUUsSUFBSTtRQUNwQixJQUFJLEVBQUUsWUFBWTtLQUNyQjtJQUNELGNBQWMsRUFBRTtRQUNaO1lBQ0ksVUFBVSxFQUFFLElBQUksS0FBSyxFQUFVO1NBQ2xDO0tBQ0o7SUFFRCxNQUFNLENBQUMsT0FBTztRQUNWLE1BQU0sbUJBQW1CLEdBQUcsT0FBTyxDQUFDLE9BQU8sRUFBRSxDQUFDLENBQUMsQ0FBQyxDQUFDO1FBQ2pELElBQUksb0JBQW9CLEdBQWEsbUJBQW1CLEVBQUUsVUFBVSxDQUFDO1FBQ3JFLElBQ0ksQ0FBQyxtQkFBbUIsRUFBRSxVQUFVO1lBQ2hDLG1CQUFtQixFQUFFLFVBQVUsQ0FBQyxNQUFNLEtBQUssQ0FBQyxFQUM5QztZQUNFLG9CQUFvQixHQUFHLG9DQUE0QixDQUFDO1NBQ3ZEO1FBQ0QsT0FBTztZQUNILENBQUMscUNBQXFDLENBQUMsQ0FBQyxJQUFJO2dCQUN4QywwR0FBMEc7Z0JBQzFHLE1BQU0sYUFBYSxHQUNmLElBR0gsRUFBRSxVQUFVLENBQUM7Z0JBQ2QsSUFBSSxhQUFhLElBQUksYUFBYSxDQUFDLE1BQU0sR0FBRyxDQUFDLEVBQUU7b0JBQzNDLE1BQU0sY0FBYyxHQUFHLGFBQWEsQ0FBQyxHQUFHLENBQ3BDLENBQUMsU0FBUyxFQUFVLEVBQUU7d0JBQ2xCLElBQ0ksU0FBUyxDQUFDLFVBQVUsQ0FBQyxJQUFJOzRCQUNyQixnQkFBUSxDQUFDLGNBQWMsQ0FBQyxjQUFjOzRCQUMxQyxTQUFTLENBQUMsVUFBVSxDQUFDLE1BQU07NEJBQzNCLFNBQVMsQ0FBQyxVQUFVLENBQUMsTUFBTSxDQUFDLElBQUk7Z0NBQzVCLGdCQUFRLENBQUMsY0FBYyxDQUFDLFVBQVUsRUFDeEM7NEJBQ0UsK0RBQStEOzRCQUMvRCxPQUFPLFNBQVMsQ0FBQyxVQUFVLENBQUMsTUFBTSxDQUFDLElBQUksQ0FBQzt5QkFDM0M7d0JBQ0QsT0FBTyxFQUFFLENBQUM7b0JBQ2QsQ0FBQyxDQUNKLENBQUM7b0JBQ0YsTUFBTSxtQkFBbUIsR0FBRyxjQUFjLENBQUMsTUFBTSxDQUM3QyxDQUFDLGFBQWEsRUFBRSxLQUFLLEVBQUUsRUFBRTt3QkFDckIsT0FBTyxDQUNILGFBQWE7NEJBQ2IsY0FBYyxDQUFDLE9BQU8sQ0FBQyxhQUFhLENBQUMsS0FBSyxLQUFLLENBQ2xELENBQUM7b0JBQ04sQ0FBQyxDQUNKLENBQUM7b0JBQ0YsSUFDSSxtQkFBbUIsQ0FBQyxNQUFNLEdBQUcsQ0FBQzt3QkFDOUIsb0JBQW9CLENBQUMsSUFBSSxDQUFDLENBQUMsbUJBQW1CLEVBQUUsRUFBRSxDQUM5QyxtQkFBbUIsQ0FBQyxRQUFRLENBQUMsbUJBQW1CLENBQUMsQ0FDcEQsRUFDSDt3QkFDRSxPQUFPLENBQUMsTUFBTSxDQUFDOzRCQUNYLElBQUksRUFBRSxJQUFJOzRCQUNWLFNBQVMsRUFBRSx1QkFBdUI7NEJBQ2xDLE9BQU8sRUFBRTtnQ0FDTDtvQ0FDSSxTQUFTLEVBQUUsdUJBQXVCO29DQUNsQyxHQUFHLEVBQUUsQ0FBQyxLQUFLLEVBQUUsRUFBRTt3Q0FDWCxNQUFNLGdCQUFnQixHQUNsQixjQUFjLENBQUMsTUFBTSxDQUNqQixDQUFDLGFBQWEsRUFBRSxLQUFLLEVBQUUsRUFBRTs0Q0FDckIsT0FBTyxDQUNILGFBQWE7Z0RBQ2IsY0FBYyxDQUFDLE9BQU8sQ0FDbEIsYUFBYSxDQUNoQixLQUFLLEtBQUssQ0FDZCxDQUFDO3dDQUNOLENBQUMsQ0FDSixDQUFDO3dDQUNOLE1BQU0sb0JBQW9CLEdBQ3RCLGdCQUFnQixDQUFDLElBQUksQ0FBQyxJQUFJLENBQUMsQ0FBQzt3Q0FDaEMsT0FBTyxLQUFLLENBQUMsV0FBVyxDQUNwQixJQUFJLEVBQ0osb0JBQW9CLENBQ3ZCLENBQUM7b0NBQ04sQ0FBQztpQ0FDSjs2QkFDSjt5QkFDSixDQUFDLENBQUM7cUJBQ047aUJBQ0o7WUFDTCxDQUFDO1NBQ0osQ0FBQztJQUNOLENBQUM7Q0FDSixDQUFDLENBQUM7QUFFSCxrQkFBZSxJQUFJLENBQUMifQ==
|