@akinon/eslint-plugin-projectzero 1.45.0-rc.0 → 1.45.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/CHANGELOG.md +1 -7
- package/configs/core.ts +0 -2
- package/dist/configs/core.js +0 -2
- package/dist/rules/index.js +1 -5
- package/package.json +1 -1
- package/rules/index.ts +1 -5
- package/dist/rules/case-warning.js +0 -32
- package/dist/rules/check-menu-depth.js +0 -43
- package/rules/case-warning.ts +0 -39
- package/rules/check-menu-depth.ts +0 -48
package/CHANGELOG.md
CHANGED
|
@@ -1,12 +1,6 @@
|
|
|
1
1
|
# @akinon/eslint-plugin-projectzero
|
|
2
2
|
|
|
3
|
-
## 1.45.0
|
|
4
|
-
|
|
5
|
-
### Minor Changes
|
|
6
|
-
|
|
7
|
-
- 5a4c607: ZERO-2764: Add case-warning rule to eslint-plugin-projectzero
|
|
8
|
-
- 64699d3: ZERO-2761: Fix invalid import for plugin module
|
|
9
|
-
- c474780: ZERO-2666: Add rule to check menu depth
|
|
3
|
+
## 1.45.0
|
|
10
4
|
|
|
11
5
|
## 1.44.0
|
|
12
6
|
|
package/configs/core.ts
CHANGED
|
@@ -9,8 +9,6 @@ export default {
|
|
|
9
9
|
'@akinon/projectzero/meta-data-import': 'error',
|
|
10
10
|
'@akinon/projectzero/urls-without-slash': 'error',
|
|
11
11
|
'@akinon/projectzero/invalid-imports': 'error',
|
|
12
|
-
'@akinon/projectzero/check-menu-depth': 'error',
|
|
13
|
-
'@akinon/projectzero/case-warning': 'warn',
|
|
14
12
|
'no-restricted-syntax': [
|
|
15
13
|
'error',
|
|
16
14
|
{
|
package/dist/configs/core.js
CHANGED
|
@@ -11,8 +11,6 @@ exports.default = {
|
|
|
11
11
|
'@akinon/projectzero/meta-data-import': 'error',
|
|
12
12
|
'@akinon/projectzero/urls-without-slash': 'error',
|
|
13
13
|
'@akinon/projectzero/invalid-imports': 'error',
|
|
14
|
-
'@akinon/projectzero/check-menu-depth': 'error',
|
|
15
|
-
'@akinon/projectzero/case-warning': 'warn',
|
|
16
14
|
'no-restricted-syntax': [
|
|
17
15
|
'error',
|
|
18
16
|
{
|
package/dist/rules/index.js
CHANGED
|
@@ -14,8 +14,6 @@ const meta_data_import_1 = __importDefault(require("./meta-data-import"));
|
|
|
14
14
|
const check_middleware_order_1 = __importDefault(require("./check-middleware-order"));
|
|
15
15
|
const urls_without_slash_1 = __importDefault(require("./urls-without-slash"));
|
|
16
16
|
const invalid_imports_1 = __importDefault(require("./invalid-imports"));
|
|
17
|
-
const check_menu_depth_1 = __importDefault(require("./check-menu-depth"));
|
|
18
|
-
const case_warning_1 = __importDefault(require("./case-warning"));
|
|
19
17
|
const rules = {
|
|
20
18
|
'client-url': client_url_1.default,
|
|
21
19
|
'image-import': image_import_1.default,
|
|
@@ -26,8 +24,6 @@ const rules = {
|
|
|
26
24
|
'meta-data-import': meta_data_import_1.default,
|
|
27
25
|
'check-middleware-order': check_middleware_order_1.default,
|
|
28
26
|
'urls-without-slash': urls_without_slash_1.default,
|
|
29
|
-
'invalid-imports': invalid_imports_1.default
|
|
30
|
-
'check-menu-depth': check_menu_depth_1.default,
|
|
31
|
-
'case-warning': case_warning_1.default
|
|
27
|
+
'invalid-imports': invalid_imports_1.default
|
|
32
28
|
};
|
|
33
29
|
exports.rules = rules;
|
package/package.json
CHANGED
package/rules/index.ts
CHANGED
|
@@ -8,8 +8,6 @@ import metaDataImport from './meta-data-import';
|
|
|
8
8
|
import checkMiddlewareOrder from './check-middleware-order';
|
|
9
9
|
import urlsWithoutSlash from './urls-without-slash';
|
|
10
10
|
import invalidImports from './invalid-imports';
|
|
11
|
-
import checkMenuDepth from './check-menu-depth';
|
|
12
|
-
import caseWarning from './case-warning';
|
|
13
11
|
|
|
14
12
|
const rules = {
|
|
15
13
|
'client-url': clientUrl,
|
|
@@ -21,9 +19,7 @@ const rules = {
|
|
|
21
19
|
'meta-data-import': metaDataImport,
|
|
22
20
|
'check-middleware-order': checkMiddlewareOrder,
|
|
23
21
|
'urls-without-slash': urlsWithoutSlash,
|
|
24
|
-
'invalid-imports': invalidImports
|
|
25
|
-
'check-menu-depth': checkMenuDepth,
|
|
26
|
-
'case-warning': caseWarning
|
|
22
|
+
'invalid-imports': invalidImports
|
|
27
23
|
};
|
|
28
24
|
|
|
29
25
|
export { rules };
|
|
@@ -1,32 +0,0 @@
|
|
|
1
|
-
"use strict";
|
|
2
|
-
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
-
const utils_1 = require("@typescript-eslint/utils");
|
|
4
|
-
exports.default = utils_1.ESLintUtils.RuleCreator.withoutDocs({
|
|
5
|
-
create(context) {
|
|
6
|
-
return {
|
|
7
|
-
CallExpression(node) {
|
|
8
|
-
if (node.callee.type === 'MemberExpression' &&
|
|
9
|
-
node.callee.property.type === 'Identifier' &&
|
|
10
|
-
(node.callee.property.name === 'toLocaleUpperCase' ||
|
|
11
|
-
node.callee.property.name === 'toLocaleLowerCase')) {
|
|
12
|
-
const args = node.arguments;
|
|
13
|
-
if (args.length === 0) {
|
|
14
|
-
context.report({
|
|
15
|
-
node,
|
|
16
|
-
messageId: 'noLocale'
|
|
17
|
-
});
|
|
18
|
-
}
|
|
19
|
-
}
|
|
20
|
-
}
|
|
21
|
-
};
|
|
22
|
-
},
|
|
23
|
-
meta: {
|
|
24
|
-
messages: {
|
|
25
|
-
noLocale: 'toLocaleUpperCase() or toLocaleLowerCase() should specify a locale for consistent behavior across different environments.'
|
|
26
|
-
},
|
|
27
|
-
type: 'problem',
|
|
28
|
-
fixable: 'code',
|
|
29
|
-
schema: []
|
|
30
|
-
},
|
|
31
|
-
defaultOptions: []
|
|
32
|
-
});
|
|
@@ -1,43 +0,0 @@
|
|
|
1
|
-
"use strict";
|
|
2
|
-
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
-
const utils_1 = require("@typescript-eslint/utils");
|
|
4
|
-
const types_1 = require("@typescript-eslint/types");
|
|
5
|
-
exports.default = utils_1.ESLintUtils.RuleCreator.withoutDocs({
|
|
6
|
-
create: function (context) {
|
|
7
|
-
return {
|
|
8
|
-
CallExpression(node) {
|
|
9
|
-
const { callee, arguments: args } = node;
|
|
10
|
-
if (callee.type !== types_1.AST_NODE_TYPES.Identifier ||
|
|
11
|
-
callee.name !== 'getMenu' ||
|
|
12
|
-
args.length === 0 ||
|
|
13
|
-
args[0].type !== types_1.AST_NODE_TYPES.ObjectExpression) {
|
|
14
|
-
return;
|
|
15
|
-
}
|
|
16
|
-
const depthProperty = args[0].properties.find((prop) => {
|
|
17
|
-
var _a;
|
|
18
|
-
return prop.type === types_1.AST_NODE_TYPES.Property &&
|
|
19
|
-
prop.key.type === types_1.AST_NODE_TYPES.Identifier &&
|
|
20
|
-
prop.key.name === 'depth' &&
|
|
21
|
-
((_a = prop.value) === null || _a === void 0 ? void 0 : _a.type) === types_1.AST_NODE_TYPES.Literal &&
|
|
22
|
-
typeof prop.value.value === 'number' &&
|
|
23
|
-
prop.value.value > 3;
|
|
24
|
-
});
|
|
25
|
-
if (depthProperty) {
|
|
26
|
-
context.report({
|
|
27
|
-
node: depthProperty,
|
|
28
|
-
messageId: 'depthMessage'
|
|
29
|
-
});
|
|
30
|
-
}
|
|
31
|
-
}
|
|
32
|
-
};
|
|
33
|
-
},
|
|
34
|
-
meta: {
|
|
35
|
-
type: 'problem',
|
|
36
|
-
fixable: 'code',
|
|
37
|
-
messages: {
|
|
38
|
-
depthMessage: 'A request for a menu deeper than 3 levels from the server can lead to unnecessary memory usage and unwanted outcomes. Therefore, sending a request for a 3-level menu on the client side and making additional requests on hover for deeper levels may be more efficient.'
|
|
39
|
-
},
|
|
40
|
-
schema: []
|
|
41
|
-
},
|
|
42
|
-
defaultOptions: []
|
|
43
|
-
});
|
package/rules/case-warning.ts
DELETED
|
@@ -1,39 +0,0 @@
|
|
|
1
|
-
import { ESLintUtils } from '@typescript-eslint/utils';
|
|
2
|
-
import { TSESTree } from '@typescript-eslint/types';
|
|
3
|
-
|
|
4
|
-
type MessageIds = 'noLocale';
|
|
5
|
-
type Options = readonly [];
|
|
6
|
-
|
|
7
|
-
export default ESLintUtils.RuleCreator.withoutDocs<Options, MessageIds>({
|
|
8
|
-
create(context) {
|
|
9
|
-
return {
|
|
10
|
-
CallExpression(node: TSESTree.CallExpression) {
|
|
11
|
-
if (
|
|
12
|
-
node.callee.type === 'MemberExpression' &&
|
|
13
|
-
node.callee.property.type === 'Identifier' &&
|
|
14
|
-
(node.callee.property.name === 'toLocaleUpperCase' ||
|
|
15
|
-
node.callee.property.name === 'toLocaleLowerCase')
|
|
16
|
-
) {
|
|
17
|
-
const args = node.arguments;
|
|
18
|
-
|
|
19
|
-
if (args.length === 0) {
|
|
20
|
-
context.report({
|
|
21
|
-
node,
|
|
22
|
-
messageId: 'noLocale'
|
|
23
|
-
});
|
|
24
|
-
}
|
|
25
|
-
}
|
|
26
|
-
}
|
|
27
|
-
};
|
|
28
|
-
},
|
|
29
|
-
meta: {
|
|
30
|
-
messages: {
|
|
31
|
-
noLocale:
|
|
32
|
-
'toLocaleUpperCase() or toLocaleLowerCase() should specify a locale for consistent behavior across different environments.'
|
|
33
|
-
},
|
|
34
|
-
type: 'problem',
|
|
35
|
-
fixable: 'code',
|
|
36
|
-
schema: []
|
|
37
|
-
},
|
|
38
|
-
defaultOptions: []
|
|
39
|
-
});
|
|
@@ -1,48 +0,0 @@
|
|
|
1
|
-
import { ESLintUtils } from '@typescript-eslint/utils';
|
|
2
|
-
import { AST_NODE_TYPES } from '@typescript-eslint/types';
|
|
3
|
-
|
|
4
|
-
export default ESLintUtils.RuleCreator.withoutDocs({
|
|
5
|
-
create: function (context) {
|
|
6
|
-
return {
|
|
7
|
-
CallExpression(node) {
|
|
8
|
-
const { callee, arguments: args } = node;
|
|
9
|
-
|
|
10
|
-
if (
|
|
11
|
-
callee.type !== AST_NODE_TYPES.Identifier ||
|
|
12
|
-
callee.name !== 'getMenu' ||
|
|
13
|
-
args.length === 0 ||
|
|
14
|
-
args[0].type !== AST_NODE_TYPES.ObjectExpression
|
|
15
|
-
) {
|
|
16
|
-
return;
|
|
17
|
-
}
|
|
18
|
-
|
|
19
|
-
const depthProperty = args[0].properties.find(
|
|
20
|
-
(prop) =>
|
|
21
|
-
prop.type === AST_NODE_TYPES.Property &&
|
|
22
|
-
prop.key.type === AST_NODE_TYPES.Identifier &&
|
|
23
|
-
prop.key.name === 'depth' &&
|
|
24
|
-
prop.value?.type === AST_NODE_TYPES.Literal &&
|
|
25
|
-
typeof prop.value.value === 'number' &&
|
|
26
|
-
prop.value.value > 3
|
|
27
|
-
);
|
|
28
|
-
|
|
29
|
-
if (depthProperty) {
|
|
30
|
-
context.report({
|
|
31
|
-
node: depthProperty,
|
|
32
|
-
messageId: 'depthMessage'
|
|
33
|
-
});
|
|
34
|
-
}
|
|
35
|
-
}
|
|
36
|
-
};
|
|
37
|
-
},
|
|
38
|
-
meta: {
|
|
39
|
-
type: 'problem',
|
|
40
|
-
fixable: 'code',
|
|
41
|
-
messages: {
|
|
42
|
-
depthMessage:
|
|
43
|
-
'A request for a menu deeper than 3 levels from the server can lead to unnecessary memory usage and unwanted outcomes. Therefore, sending a request for a 3-level menu on the client side and making additional requests on hover for deeper levels may be more efficient.'
|
|
44
|
-
},
|
|
45
|
-
schema: []
|
|
46
|
-
},
|
|
47
|
-
defaultOptions: []
|
|
48
|
-
});
|