@kununu/eslint-config 5.2.3-beta-5 → 5.2.3-beta-6
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/index.js +4 -66
- package/package.json +1 -1
package/index.js
CHANGED
|
@@ -1,67 +1,3 @@
|
|
|
1
|
-
const preferEarlyReturnRule = {
|
|
2
|
-
meta: {
|
|
3
|
-
type: 'suggestion',
|
|
4
|
-
docs: {
|
|
5
|
-
description: 'Prefer early returns over logical expressions in return statements',
|
|
6
|
-
category: 'Best Practices',
|
|
7
|
-
recommended: true,
|
|
8
|
-
},
|
|
9
|
-
fixable: 'code',
|
|
10
|
-
},
|
|
11
|
-
create(context) {
|
|
12
|
-
return {
|
|
13
|
-
ReturnStatement(node) {
|
|
14
|
-
const argument = node.argument?.type === 'ParenthesizedExpression'
|
|
15
|
-
? node.argument.expression
|
|
16
|
-
: node.argument;
|
|
17
|
-
|
|
18
|
-
// Only handle && operators
|
|
19
|
-
if (argument?.type === 'LogicalExpression' && argument.operator === '&&') {
|
|
20
|
-
context.report({
|
|
21
|
-
node,
|
|
22
|
-
message: 'Prefer early returns over logical expressions in return statements',
|
|
23
|
-
fix(fixer) {
|
|
24
|
-
const sourceCode = context.getSourceCode();
|
|
25
|
-
const condition = sourceCode.getText(argument.left);
|
|
26
|
-
const returnValue = sourceCode.getText(argument.right);
|
|
27
|
-
|
|
28
|
-
return fixer.replaceText(
|
|
29
|
-
node,
|
|
30
|
-
`if (${condition}) return ${returnValue};`
|
|
31
|
-
);
|
|
32
|
-
}
|
|
33
|
-
});
|
|
34
|
-
}
|
|
35
|
-
}
|
|
36
|
-
};
|
|
37
|
-
}
|
|
38
|
-
};
|
|
39
|
-
|
|
40
|
-
// Create the plugin with our rule
|
|
41
|
-
const customPlugin = {
|
|
42
|
-
rules: {
|
|
43
|
-
'prefer-early-return': preferEarlyReturnRule
|
|
44
|
-
}
|
|
45
|
-
};
|
|
46
|
-
|
|
47
|
-
// Register the custom plugin
|
|
48
|
-
try {
|
|
49
|
-
// This is the path in ESLint v8+
|
|
50
|
-
require('eslint/use-at-your-own-risk').builtinPlugins.define('kununu-custom', customPlugin);
|
|
51
|
-
} catch (e) {
|
|
52
|
-
try {
|
|
53
|
-
// This is the path in ESLint v7
|
|
54
|
-
require('eslint/lib/cli-engine/plugins').define('kununu-custom', customPlugin);
|
|
55
|
-
} catch (e) {
|
|
56
|
-
// Fallback for ESLint v6
|
|
57
|
-
try {
|
|
58
|
-
require('eslint/lib/util/plugins').define('kununu-custom', customPlugin);
|
|
59
|
-
} catch (e) {
|
|
60
|
-
console.warn('Failed to register custom ESLint plugin:', e.message);
|
|
61
|
-
}
|
|
62
|
-
}
|
|
63
|
-
}
|
|
64
|
-
|
|
65
1
|
// we need to have the baseRules to have the same on the typescript override
|
|
66
2
|
// because it has extends the eslint does not take into consideration the ones from the base
|
|
67
3
|
const baseRules = {
|
|
@@ -268,7 +204,10 @@ const baseRules = {
|
|
|
268
204
|
// https://react.dev/blog/2024/04/25/react-19-upgrade-guide#removed-proptypes-and-defaultprops
|
|
269
205
|
'react/require-default-props': 0,
|
|
270
206
|
|
|
271
|
-
|
|
207
|
+
'no-restricted-syntax': ['error', {
|
|
208
|
+
selector: 'ReturnStatement > LogicalExpression[operator="&&"]',
|
|
209
|
+
message: 'Prefer early returns over logical expressions in return statements. Use if (condition) return value; instead.'
|
|
210
|
+
}],
|
|
272
211
|
};
|
|
273
212
|
|
|
274
213
|
module.exports = {
|
|
@@ -288,7 +227,6 @@ module.exports = {
|
|
|
288
227
|
'@babel',
|
|
289
228
|
'lodash',
|
|
290
229
|
'sort-destructure-keys',
|
|
291
|
-
'kununu-custom'
|
|
292
230
|
],
|
|
293
231
|
|
|
294
232
|
env: {
|