@checkdigit/eslint-plugin 7.8.0-PR.97-a3ad → 7.8.0-PR.97-bffd
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.
|
@@ -1,28 +1,11 @@
|
|
|
1
1
|
// src/no-status-code-assert.ts
|
|
2
|
-
import { StatusCodes } from "http-status-codes";
|
|
3
2
|
import { AST_NODE_TYPES, ESLintUtils } from "@typescript-eslint/utils";
|
|
4
3
|
import getDocumentationUrl from "./get-documentation-url.mjs";
|
|
5
4
|
var ruleId = "no-status-code-assert";
|
|
6
5
|
var NO_STATUS_CODE_ASSERT = "NO_STATUS_CODE_ASSERT";
|
|
7
|
-
var statusCodes = [
|
|
8
|
-
StatusCodes.OK,
|
|
9
|
-
StatusCodes.CREATED,
|
|
10
|
-
StatusCodes.BAD_REQUEST,
|
|
11
|
-
StatusCodes.UNAUTHORIZED,
|
|
12
|
-
StatusCodes.FORBIDDEN,
|
|
13
|
-
StatusCodes.NOT_FOUND,
|
|
14
|
-
StatusCodes.CONFLICT,
|
|
15
|
-
StatusCodes.INTERNAL_SERVER_ERROR
|
|
16
|
-
];
|
|
17
6
|
var createRule = ESLintUtils.RuleCreator((name) => getDocumentationUrl(name));
|
|
18
7
|
var hasStatusCodeOrValue = (arg) => {
|
|
19
|
-
const isStatusCodeLiteral = (value) => statusCodes.includes(value);
|
|
20
8
|
switch (arg.type) {
|
|
21
|
-
case AST_NODE_TYPES.Literal:
|
|
22
|
-
if (typeof arg.value === "number") {
|
|
23
|
-
return isStatusCodeLiteral(arg.value);
|
|
24
|
-
}
|
|
25
|
-
break;
|
|
26
9
|
case AST_NODE_TYPES.MemberExpression: {
|
|
27
10
|
const object = arg.object;
|
|
28
11
|
const property = arg.property;
|
|
@@ -38,7 +21,7 @@ var hasStatusCodeOrValue = (arg) => {
|
|
|
38
21
|
};
|
|
39
22
|
var isAssertIdentifier = (node) => node.type === AST_NODE_TYPES.Identifier && node.name === "assert";
|
|
40
23
|
var isAssertMemberExpression = (node) => node.type === AST_NODE_TYPES.MemberExpression && node.object.type === AST_NODE_TYPES.Identifier && node.object.name === "assert" && node.property.type === AST_NODE_TYPES.Identifier;
|
|
41
|
-
var isAssertCallWithStatusCode = (callee, args) => (isAssertIdentifier(callee) || isAssertMemberExpression(callee)) && args.some((arg) => hasStatusCodeOrValue(arg)
|
|
24
|
+
var isAssertCallWithStatusCode = (callee, args) => (isAssertIdentifier(callee) || isAssertMemberExpression(callee)) && args.some((arg) => hasStatusCodeOrValue(arg));
|
|
42
25
|
var rule = createRule({
|
|
43
26
|
name: ruleId,
|
|
44
27
|
meta: {
|
|
@@ -71,4 +54,4 @@ export {
|
|
|
71
54
|
no_status_code_assert_default as default,
|
|
72
55
|
ruleId
|
|
73
56
|
};
|
|
74
|
-
//# sourceMappingURL=data:application/json;base64,
|
|
57
|
+
//# sourceMappingURL=data:application/json;base64,ewogICJ2ZXJzaW9uIjogMywKICAic291cmNlcyI6IFsiLi4vc3JjL25vLXN0YXR1cy1jb2RlLWFzc2VydC50cyJdLAogICJtYXBwaW5ncyI6ICI7QUFRQSxTQUFTLGdCQUFnQixtQkFBdUM7QUFDaEUsT0FBTyx5QkFBeUI7QUFFekIsSUFBTSxTQUFTO0FBQ3RCLElBQU0sd0JBQXdCO0FBRTlCLElBQU0sYUFBYSxZQUFZLFlBQVksQ0FBQyxTQUFTLG9CQUFvQixJQUFJLENBQUM7QUFHOUUsSUFBTSx1QkFBdUIsQ0FBQyxRQUFnQztBQUM1RCxVQUFRLElBQUksTUFBTTtBQUFBLElBQ2hCLEtBQUssZUFBZSxrQkFBa0I7QUFDcEMsWUFBTSxTQUFTLElBQUk7QUFDbkIsWUFBTSxXQUFXLElBQUk7QUFDckIsVUFDRSxPQUFPLFNBQVMsZUFBZSxjQUMvQixPQUFPLFNBQVMsaUJBQ2hCLFNBQVMsU0FBUyxlQUFlLFlBQ2pDO0FBQ0EsZUFBTztBQUFBLE1BQ1Q7QUFDQTtBQUFBLElBQ0Y7QUFBQSxJQUNBLEtBQUssZUFBZTtBQUNsQixhQUFPLHFCQUFxQixJQUFJLElBQUksS0FBSyxxQkFBcUIsSUFBSSxLQUFLO0FBQUEsRUFDM0U7QUFDQSxTQUFPO0FBQ1Q7QUFHQSxJQUFNLHFCQUFxQixDQUFDLFNBQzFCLEtBQUssU0FBUyxlQUFlLGNBQWMsS0FBSyxTQUFTO0FBRzNELElBQU0sMkJBQTJCLENBQUMsU0FDaEMsS0FBSyxTQUFTLGVBQWUsb0JBQzdCLEtBQUssT0FBTyxTQUFTLGVBQWUsY0FDcEMsS0FBSyxPQUFPLFNBQVMsWUFDckIsS0FBSyxTQUFTLFNBQVMsZUFBZTtBQUV4QyxJQUFNLDZCQUE2QixDQUFDLFFBQXVCLFVBQ3hELG1CQUFtQixNQUFNLEtBQUsseUJBQXlCLE1BQU0sTUFBTSxLQUFLLEtBQUssQ0FBQyxRQUFRLHFCQUFxQixHQUFHLENBQUM7QUFFbEgsSUFBTSxPQUEwRCxXQUFXO0FBQUEsRUFDekUsTUFBTTtBQUFBLEVBQ04sTUFBTTtBQUFBLElBQ0osTUFBTTtBQUFBLElBQ04sTUFBTTtBQUFBLE1BQ0osYUFBYTtBQUFBLElBQ2Y7QUFBQSxJQUNBLFFBQVEsQ0FBQztBQUFBLElBQ1QsVUFBVTtBQUFBLE1BQ1IsQ0FBQyxxQkFBcUIsR0FBRztBQUFBLElBQzNCO0FBQUEsRUFDRjtBQUFBLEVBQ0EsZ0JBQWdCLENBQUM7QUFBQSxFQUNqQixPQUFPLFNBQVM7QUFDZCxXQUFPO0FBQUEsTUFDTCxlQUFlLE1BQStCO0FBQzVDLGNBQU0sU0FBUyxLQUFLO0FBQ3BCLFlBQUksMkJBQTJCLFFBQVEsS0FBSyxTQUFTLEdBQUc7QUFDdEQsa0JBQVEsT0FBTztBQUFBLFlBQ2I7QUFBQSxZQUNBLFdBQVc7QUFBQSxVQUNiLENBQUM7QUFBQSxRQUNIO0FBQUEsTUFDRjtBQUFBLElBQ0Y7QUFBQSxFQUNGO0FBQ0YsQ0FBQztBQUVELElBQU8sZ0NBQVE7IiwKICAibmFtZXMiOiBbXQp9Cg==
|
package/package.json
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"name":"@checkdigit/eslint-plugin","version":"7.8.0-PR.97-
|
|
1
|
+
{"name":"@checkdigit/eslint-plugin","version":"7.8.0-PR.97-bffd","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.18.1","@typescript-eslint/utils":"^8.18.1","ts-api-utils":"^2.0.0"},"devDependencies":{"@checkdigit/jest-config":"^6.0.2","@checkdigit/prettier-config":"^6.0.0","@checkdigit/typescript-config":"^9.0.0","@eslint/js":"^9.17.0","@types/eslint":"^9.6.1","@types/eslint-config-prettier":"^6.11.3","@typescript-eslint/parser":"^8.18.1","@typescript-eslint/rule-tester":"^8.18.1","eslint":"^9.17.0","eslint-config-prettier":"^9.1.0","eslint-import-resolver-typescript":"^3.7.0","eslint-plugin-eslint-plugin":"^6.3.2","eslint-plugin-import":"^2.31.0","eslint-plugin-no-only-tests":"^3.3.0","eslint-plugin-no-secrets":"^2.1.1","eslint-plugin-node":"^11.1.0","eslint-plugin-sonarjs":"1.0.4","http-status-codes":"^2.3.0","rimraf":"^6.0.1","typescript-eslint":"^8.18.1"},"peerDependencies":{"eslint":">=9 <10"},"engines":{"node":">=20.17"}}
|
|
@@ -6,37 +6,17 @@
|
|
|
6
6
|
* This code is licensed under the MIT license (see LICENSE.txt for details).
|
|
7
7
|
*/
|
|
8
8
|
|
|
9
|
-
import { StatusCodes } from 'http-status-codes';
|
|
10
|
-
|
|
11
9
|
import { AST_NODE_TYPES, ESLintUtils, TSESLint, TSESTree } from '@typescript-eslint/utils';
|
|
12
10
|
import getDocumentationUrl from './get-documentation-url';
|
|
13
11
|
|
|
14
12
|
export const ruleId = 'no-status-code-assert';
|
|
15
13
|
const NO_STATUS_CODE_ASSERT = 'NO_STATUS_CODE_ASSERT';
|
|
16
14
|
|
|
17
|
-
const statusCodes = [
|
|
18
|
-
StatusCodes.OK,
|
|
19
|
-
StatusCodes.CREATED,
|
|
20
|
-
StatusCodes.BAD_REQUEST,
|
|
21
|
-
StatusCodes.UNAUTHORIZED,
|
|
22
|
-
StatusCodes.FORBIDDEN,
|
|
23
|
-
StatusCodes.NOT_FOUND,
|
|
24
|
-
StatusCodes.CONFLICT,
|
|
25
|
-
StatusCodes.INTERNAL_SERVER_ERROR,
|
|
26
|
-
];
|
|
27
|
-
|
|
28
15
|
const createRule = ESLintUtils.RuleCreator((name) => getDocumentationUrl(name));
|
|
29
16
|
|
|
30
17
|
// Checks if a given AST node contains any status code value.
|
|
31
18
|
const hasStatusCodeOrValue = (arg: TSESTree.Node): boolean => {
|
|
32
|
-
const isStatusCodeLiteral = (value: number): boolean => statusCodes.includes(value);
|
|
33
|
-
|
|
34
19
|
switch (arg.type) {
|
|
35
|
-
case AST_NODE_TYPES.Literal:
|
|
36
|
-
if (typeof arg.value === 'number') {
|
|
37
|
-
return isStatusCodeLiteral(arg.value);
|
|
38
|
-
}
|
|
39
|
-
break;
|
|
40
20
|
case AST_NODE_TYPES.MemberExpression: {
|
|
41
21
|
const object = arg.object;
|
|
42
22
|
const property = arg.property;
|
|
@@ -66,11 +46,8 @@ const isAssertMemberExpression = (node: TSESTree.Node): boolean =>
|
|
|
66
46
|
node.object.name === 'assert' &&
|
|
67
47
|
node.property.type === AST_NODE_TYPES.Identifier;
|
|
68
48
|
|
|
69
|
-
//Checks if a given AST node is an identifier with the name 'assert'.
|
|
70
|
-
|
|
71
49
|
const isAssertCallWithStatusCode = (callee: TSESTree.Node, args: TSESTree.Node[]): boolean =>
|
|
72
|
-
(isAssertIdentifier(callee) || isAssertMemberExpression(callee)) &&
|
|
73
|
-
args.some((arg) => hasStatusCodeOrValue(arg) && arg.type !== AST_NODE_TYPES.Identifier);
|
|
50
|
+
(isAssertIdentifier(callee) || isAssertMemberExpression(callee)) && args.some((arg) => hasStatusCodeOrValue(arg));
|
|
74
51
|
|
|
75
52
|
const rule: TSESLint.RuleModule<typeof NO_STATUS_CODE_ASSERT> = createRule({
|
|
76
53
|
name: ruleId,
|