@checkdigit/eslint-plugin 7.13.0-PR.119-f22e → 7.13.0-PR.119-b412
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.
|
@@ -29,7 +29,7 @@ var rule = createRule({
|
|
|
29
29
|
},
|
|
30
30
|
schema: [],
|
|
31
31
|
messages: {
|
|
32
|
-
[NO_RANDOM_V4_UUID]: "Avoid using `
|
|
32
|
+
[NO_RANDOM_V4_UUID]: "Avoid using `crypto.randomUUID` for generating random v4 UUIDs.",
|
|
33
33
|
[NO_UUID_MODULE_FOR_V4]: "Avoid using the `uuid` module for v4 UUID generation. Use `crypto.randomUUID` instead."
|
|
34
34
|
}
|
|
35
35
|
},
|
|
@@ -37,6 +37,7 @@ var rule = createRule({
|
|
|
37
37
|
create(context) {
|
|
38
38
|
const aliases = {};
|
|
39
39
|
let uuidImportNode = null;
|
|
40
|
+
let hasReportedCallExpression = false;
|
|
40
41
|
return {
|
|
41
42
|
ImportDeclaration(node) {
|
|
42
43
|
processImportDeclaration(node, aliases);
|
|
@@ -45,15 +46,22 @@ var rule = createRule({
|
|
|
45
46
|
}
|
|
46
47
|
},
|
|
47
48
|
CallExpression(node) {
|
|
48
|
-
if (isUuid4Call(node, aliases)
|
|
49
|
+
if (isUuid4Call(node, aliases)) {
|
|
50
|
+
context.report({
|
|
51
|
+
node,
|
|
52
|
+
messageId: NO_UUID_MODULE_FOR_V4
|
|
53
|
+
});
|
|
54
|
+
hasReportedCallExpression = true;
|
|
55
|
+
} else if (isCryptoRandomUUIDCall(node, aliases.cryptoRandomUUIDAlias)) {
|
|
49
56
|
context.report({
|
|
50
57
|
node,
|
|
51
58
|
messageId: NO_RANDOM_V4_UUID
|
|
52
59
|
});
|
|
60
|
+
hasReportedCallExpression = true;
|
|
53
61
|
}
|
|
54
62
|
},
|
|
55
63
|
"Program:exit"() {
|
|
56
|
-
if (uuidImportNode && aliases.uuid4Alias !== void 0 && aliases.uuid4Alias !== "" && (aliases.uuidDefaultAlias === void 0 || aliases.uuidDefaultAlias === "")) {
|
|
64
|
+
if (uuidImportNode && !hasReportedCallExpression && aliases.uuid4Alias !== void 0 && aliases.uuid4Alias !== "" && (aliases.uuidDefaultAlias === void 0 || aliases.uuidDefaultAlias === "")) {
|
|
57
65
|
context.report({
|
|
58
66
|
node: uuidImportNode,
|
|
59
67
|
messageId: NO_UUID_MODULE_FOR_V4
|
|
@@ -68,4 +76,4 @@ export {
|
|
|
68
76
|
no_random_v4_uuid_default as default,
|
|
69
77
|
ruleId
|
|
70
78
|
};
|
|
71
|
-
//# sourceMappingURL=data:application/json;base64,
|
|
79
|
+
//# sourceMappingURL=data:application/json;base64,ewogICJ2ZXJzaW9uIjogMywKICAic291cmNlcyI6IFsiLi4vc3JjL25vLXJhbmRvbS12NC11dWlkLnRzIl0sCiAgIm1hcHBpbmdzIjogIjtBQVFBLFNBQVMsZ0JBQWdCLG1CQUF1QztBQUNoRSxPQUFPLHlCQUF5QjtBQUV6QixJQUFNLFNBQVM7QUFDdEIsSUFBTSxvQkFBb0I7QUFDMUIsSUFBTSx3QkFBd0I7QUFFOUIsSUFBTSxhQUFhLFlBQVksWUFBWSxDQUFDLFNBQVMsb0JBQW9CLElBQUksQ0FBQztBQUU5RSxJQUFNLDJCQUEyQixDQUMvQixNQUNBLFlBQ0c7QUFDSCxPQUFLLFdBQVcsUUFBUSxDQUFDLGNBQWM7QUFDckMsUUFBSSxVQUFVLFNBQVMsZUFBZSxpQkFBaUI7QUFDckQsVUFDRSxLQUFLLE9BQU8sVUFBVSxVQUN0QixVQUFVLFNBQVMsU0FBUyxlQUFlLGNBQzNDLFVBQVUsU0FBUyxTQUFTLE1BQzVCO0FBQ0EsZ0JBQVEsYUFBYSxVQUFVLE1BQU07QUFBQSxNQUN2QyxXQUNFLEtBQUssT0FBTyxVQUFVLGlCQUN0QixVQUFVLFNBQVMsU0FBUyxlQUFlLGNBQzNDLFVBQVUsU0FBUyxTQUFTLGNBQzVCO0FBQ0EsZ0JBQVEsd0JBQXdCLFVBQVUsTUFBTTtBQUFBLE1BQ2xEO0FBQUEsSUFDRixXQUFXLFVBQVUsU0FBUyxlQUFlLDBCQUEwQixLQUFLLE9BQU8sVUFBVSxRQUFRO0FBQ25HLGNBQVEsbUJBQW1CLFVBQVUsTUFBTTtBQUFBLElBQzdDO0FBQUEsRUFDRixDQUFDO0FBQ0g7QUFFQSxJQUFNLGNBQWMsQ0FDbEIsTUFDQSxZQUVDLEtBQUssT0FBTyxTQUFTLGVBQWUsY0FBYyxLQUFLLE9BQU8sU0FBUyxRQUFRLGNBQy9FLEtBQUssT0FBTyxTQUFTLGVBQWUsb0JBQ25DLEtBQUssT0FBTyxPQUFPLFNBQVMsZUFBZSxjQUMzQyxLQUFLLE9BQU8sT0FBTyxTQUFTLFFBQVEsb0JBQ3BDLEtBQUssT0FBTyxTQUFTLFNBQVMsZUFBZSxjQUM3QyxLQUFLLE9BQU8sU0FBUyxTQUFTO0FBRWxDLElBQU0seUJBQXlCLENBQUMsTUFBK0IsVUFDNUQsS0FBSyxPQUFPLFNBQVMsZUFBZSxjQUFjLEtBQUssT0FBTyxTQUFTLFNBQ3ZFLEtBQUssT0FBTyxTQUFTLGVBQWUsb0JBQ25DLEtBQUssT0FBTyxPQUFPLFNBQVMsZUFBZSxjQUMzQyxLQUFLLE9BQU8sT0FBTyxTQUFTLFlBQzVCLEtBQUssT0FBTyxTQUFTLFNBQVMsZUFBZSxjQUM3QyxLQUFLLE9BQU8sU0FBUyxTQUFTO0FBRWxDLElBQU0sT0FBcUYsV0FBVztBQUFBLEVBQ3BHLE1BQU07QUFBQSxFQUNOLE1BQU07QUFBQSxJQUNKLE1BQU07QUFBQSxJQUNOLE1BQU07QUFBQSxNQUNKLGFBQ0U7QUFBQSxJQUNKO0FBQUEsSUFDQSxRQUFRLENBQUM7QUFBQSxJQUNULFVBQVU7QUFBQSxNQUNSLENBQUMsaUJBQWlCLEdBQUc7QUFBQSxNQUNyQixDQUFDLHFCQUFxQixHQUFHO0FBQUEsSUFDM0I7QUFBQSxFQUNGO0FBQUEsRUFDQSxnQkFBZ0IsQ0FBQztBQUFBLEVBQ2pCLE9BQU8sU0FBUztBQUNkLFVBQU0sVUFBOEYsQ0FBQztBQUNyRyxRQUFJLGlCQUFvRDtBQUN4RCxRQUFJLDRCQUE0QjtBQUVoQyxXQUFPO0FBQUEsTUFDTCxrQkFBa0IsTUFBa0M7QUFDbEQsaUNBQXlCLE1BQU0sT0FBTztBQUN0QyxZQUFJLEtBQUssT0FBTyxVQUFVLFFBQVE7QUFDaEMsMkJBQWlCO0FBQUEsUUFDbkI7QUFBQSxNQUNGO0FBQUEsTUFDQSxlQUFlLE1BQStCO0FBQzVDLFlBQUksWUFBWSxNQUFNLE9BQU8sR0FBRztBQUM5QixrQkFBUSxPQUFPO0FBQUEsWUFDYjtBQUFBLFlBQ0EsV0FBVztBQUFBLFVBQ2IsQ0FBQztBQUNELHNDQUE0QjtBQUFBLFFBQzlCLFdBQVcsdUJBQXVCLE1BQU0sUUFBUSxxQkFBcUIsR0FBRztBQUN0RSxrQkFBUSxPQUFPO0FBQUEsWUFDYjtBQUFBLFlBQ0EsV0FBVztBQUFBLFVBQ2IsQ0FBQztBQUNELHNDQUE0QjtBQUFBLFFBQzlCO0FBQUEsTUFDRjtBQUFBLE1BQ0EsaUJBQWlCO0FBQ2YsWUFDRSxrQkFDQSxDQUFDLDZCQUNELFFBQVEsZUFBZSxVQUN2QixRQUFRLGVBQWUsT0FDdEIsUUFBUSxxQkFBcUIsVUFBYSxRQUFRLHFCQUFxQixLQUN4RTtBQUNBLGtCQUFRLE9BQU87QUFBQSxZQUNiLE1BQU07QUFBQSxZQUNOLFdBQVc7QUFBQSxVQUNiLENBQUM7QUFBQSxRQUNIO0FBQUEsTUFDRjtBQUFBLElBQ0Y7QUFBQSxFQUNGO0FBQ0YsQ0FBQztBQUVELElBQU8sNEJBQVE7IiwKICAibmFtZXMiOiBbXQp9Cg==
|
package/package.json
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"name":"@checkdigit/eslint-plugin","version":"7.13.0-PR.119-
|
|
1
|
+
{"name":"@checkdigit/eslint-plugin","version":"7.13.0-PR.119-b412","description":"Check Digit eslint plugins","keywords":["eslint","eslintplugin"],"homepage":"https://github.com/checkdigit/eslint-plugin#readme","bugs":{"url":"https://github.com/checkdigit/eslint-plugin/issues"},"repository":{"type":"git","url":"https://github.com/checkdigit/eslint-plugin"},"license":"MIT","author":"Check Digit, LLC","sideEffects":false,"type":"module","exports":{".":{"types":"./dist-types/index.d.ts","import":"./dist-mjs/index.mjs","default":"./dist-mjs/index.mjs"}},"files":["src","dist-types","dist-mjs","!src/**/test/**","!src/**/*.test.ts","!src/**/*.spec.ts","!dist-types/**/test/**","!dist-types/**/*.test.d.ts","!dist-types/**/*.spec.d.ts","!dist-mjs/**/test/**","!dist-mjs/**/*.test.mjs","!dist-mjs/**/*.spec.mjs","SECURITY.md"],"scripts":{"build:dist-mjs":"rimraf dist-mjs && npx builder --type=module --sourceMap --outDir=dist-mjs && node dist-mjs/index.mjs","build:dist-types":"rimraf dist-types && npx builder --type=types --outDir=dist-types","ci:compile":"tsc --noEmit","ci:coverage":"NODE_OPTIONS=\"--disable-warning ExperimentalWarning --experimental-vm-modules\" jest --coverage=true","ci:lint":"npm run lint","ci:style":"npm run prettier","ci:test":"NODE_OPTIONS=\"--disable-warning ExperimentalWarning --experimental-vm-modules\" jest --coverage=false","lint":"eslint --max-warnings 0 .","lint:fix":"eslint --max-warnings 0 --fix .","prepare":"","prepublishOnly":"npm run build:dist-types && npm run build:dist-mjs","prettier":"prettier --ignore-path .gitignore --list-different .","prettier:fix":"prettier --ignore-path .gitignore --write .","test":"npm run ci:compile && npm run ci:test && npm run ci:lint && npm run ci:style"},"prettier":"@checkdigit/prettier-config","jest":{"preset":"@checkdigit/jest-config"},"dependencies":{"@typescript-eslint/type-utils":"^8.23.0","@typescript-eslint/utils":"^8.23.0","http-status-codes":"^2.3.0","ts-api-utils":"^2.0.1"},"devDependencies":{"@checkdigit/jest-config":"^6.0.2","@checkdigit/prettier-config":"^6.1.0","@checkdigit/typescript-config":"^9.0.0","@eslint/js":"^9.19.0","@types/eslint":"^9.6.1","@types/eslint-config-prettier":"^6.11.3","@typescript-eslint/parser":"^8.23.0","@typescript-eslint/rule-tester":"^8.23.0","eslint":"^9.19.0","eslint-config-prettier":"^10.0.1","eslint-import-resolver-typescript":"^3.7.0","eslint-plugin-eslint-plugin":"^6.4.0","eslint-plugin-import":"^2.31.0","eslint-plugin-no-only-tests":"^3.3.0","eslint-plugin-no-secrets":"^2.2.1","eslint-plugin-node":"^11.1.0","eslint-plugin-sonarjs":"1.0.4","rimraf":"^6.0.1","typescript-eslint":"^8.23.0"},"peerDependencies":{"eslint":">=9 <10"},"engines":{"node":">=20.17"}}
|
package/src/no-random-v4-uuid.ts
CHANGED
|
@@ -69,7 +69,7 @@ const rule: TSESLint.RuleModule<typeof NO_RANDOM_V4_UUID | typeof NO_UUID_MODULE
|
|
|
69
69
|
},
|
|
70
70
|
schema: [],
|
|
71
71
|
messages: {
|
|
72
|
-
[NO_RANDOM_V4_UUID]: 'Avoid using `
|
|
72
|
+
[NO_RANDOM_V4_UUID]: 'Avoid using `crypto.randomUUID` for generating random v4 UUIDs.',
|
|
73
73
|
[NO_UUID_MODULE_FOR_V4]: 'Avoid using the `uuid` module for v4 UUID generation. Use `crypto.randomUUID` instead.',
|
|
74
74
|
},
|
|
75
75
|
},
|
|
@@ -77,6 +77,7 @@ const rule: TSESLint.RuleModule<typeof NO_RANDOM_V4_UUID | typeof NO_UUID_MODULE
|
|
|
77
77
|
create(context) {
|
|
78
78
|
const aliases: { uuid4Alias?: string; uuidDefaultAlias?: string; cryptoRandomUUIDAlias?: string } = {};
|
|
79
79
|
let uuidImportNode: TSESTree.ImportDeclaration | null = null;
|
|
80
|
+
let hasReportedCallExpression = false;
|
|
80
81
|
|
|
81
82
|
return {
|
|
82
83
|
ImportDeclaration(node: TSESTree.ImportDeclaration) {
|
|
@@ -86,16 +87,24 @@ const rule: TSESLint.RuleModule<typeof NO_RANDOM_V4_UUID | typeof NO_UUID_MODULE
|
|
|
86
87
|
}
|
|
87
88
|
},
|
|
88
89
|
CallExpression(node: TSESTree.CallExpression) {
|
|
89
|
-
if (isUuid4Call(node, aliases)
|
|
90
|
+
if (isUuid4Call(node, aliases)) {
|
|
91
|
+
context.report({
|
|
92
|
+
node,
|
|
93
|
+
messageId: NO_UUID_MODULE_FOR_V4,
|
|
94
|
+
});
|
|
95
|
+
hasReportedCallExpression = true;
|
|
96
|
+
} else if (isCryptoRandomUUIDCall(node, aliases.cryptoRandomUUIDAlias)) {
|
|
90
97
|
context.report({
|
|
91
98
|
node,
|
|
92
99
|
messageId: NO_RANDOM_V4_UUID,
|
|
93
100
|
});
|
|
101
|
+
hasReportedCallExpression = true;
|
|
94
102
|
}
|
|
95
103
|
},
|
|
96
104
|
'Program:exit'() {
|
|
97
105
|
if (
|
|
98
106
|
uuidImportNode &&
|
|
107
|
+
!hasReportedCallExpression &&
|
|
99
108
|
aliases.uuid4Alias !== undefined &&
|
|
100
109
|
aliases.uuid4Alias !== '' &&
|
|
101
110
|
(aliases.uuidDefaultAlias === undefined || aliases.uuidDefaultAlias === '')
|