forked from e18e/eslint-plugin
-
Notifications
You must be signed in to change notification settings - Fork 0
Expand file tree
/
Copy pathprefer-array-to-reversed.ts
More file actions
72 lines (65 loc) · 1.88 KB
/
Copy pathprefer-array-to-reversed.ts
File metadata and controls
72 lines (65 loc) · 1.88 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
import type {TSESLint, TSESTree} from '@typescript-eslint/utils';
import {
getArrayFromCopyPattern,
needsParensForPropertyAccess,
isCopyPatternOptional
} from '../utils/ast.js';
import {isArrayType} from '../utils/typescript.js';
type MessageIds = 'preferToReversed';
export const preferArrayToReversed: TSESLint.RuleModule<MessageIds, []> = {
meta: {
type: 'suggestion',
docs: {
description:
'Prefer Array.prototype.toReversed() over copying and reversing arrays'
},
fixable: 'code',
schema: [],
messages: {
preferToReversed:
'Use {{array}}.toReversed() instead of copying and reversing'
}
},
defaultOptions: [],
create(context) {
const sourceCode = context.sourceCode;
return {
CallExpression(node: TSESTree.CallExpression) {
if (
node.callee.type !== 'MemberExpression' ||
node.callee.property.type !== 'Identifier' ||
node.callee.property.name !== 'reverse'
) {
return;
}
const reverseCallee = node.callee.object;
const arrayNode = getArrayFromCopyPattern(reverseCallee);
if (arrayNode) {
if (isArrayType(arrayNode, context) === false) {
return;
}
const rawText = sourceCode.getText(arrayNode);
const arrayText = needsParensForPropertyAccess(arrayNode)
? `(${rawText})`
: rawText;
const optionalChain = isCopyPatternOptional(reverseCallee)
? '?.'
: '.';
context.report({
node,
messageId: 'preferToReversed',
data: {
array: rawText
},
fix(fixer) {
return fixer.replaceText(
node,
`${arrayText}${optionalChain}toReversed()`
);
}
});
}
}
};
}
};