eslint-plugin-jsdoc 52.0.1 → 52.0.3
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/dist/WarnSettings.cjs +18 -35
- package/dist/WarnSettings.cjs.map +1 -1
- package/dist/WarnSettings.js +20 -0
- package/dist/WarnSettings.js.map +1 -0
- package/dist/_virtual/rolldown_runtime.cjs +32 -0
- package/dist/_virtual/rolldown_runtime.js +11 -0
- package/dist/alignTransform.cjs +224 -305
- package/dist/alignTransform.cjs.map +1 -1
- package/dist/alignTransform.js +241 -0
- package/dist/alignTransform.js.map +1 -0
- package/dist/defaultTagOrder.cjs +132 -43
- package/dist/defaultTagOrder.cjs.map +1 -1
- package/dist/defaultTagOrder.js +134 -0
- package/dist/defaultTagOrder.js.map +1 -0
- package/dist/exportParser.cjs +478 -696
- package/dist/exportParser.cjs.map +1 -1
- package/dist/exportParser.js +518 -0
- package/dist/exportParser.js.map +1 -0
- package/dist/getDefaultTagStructureForMode.cjs +184 -288
- package/dist/getDefaultTagStructureForMode.cjs.map +1 -1
- package/dist/getDefaultTagStructureForMode.js +188 -0
- package/dist/getDefaultTagStructureForMode.js.map +1 -0
- package/dist/getJsdocProcessorPlugin.cjs +365 -532
- package/dist/getJsdocProcessorPlugin.cjs.map +1 -1
- package/dist/getJsdocProcessorPlugin.d.cts +70 -0
- package/dist/getJsdocProcessorPlugin.d.cts.map +1 -0
- package/dist/getJsdocProcessorPlugin.d.ts +68 -90
- package/dist/getJsdocProcessorPlugin.d.ts.map +1 -1
- package/dist/getJsdocProcessorPlugin.js +383 -0
- package/dist/getJsdocProcessorPlugin.js.map +1 -0
- package/dist/index.cjs +398 -383
- package/dist/index.cjs.map +1 -1
- package/dist/index.d.cts +22 -0
- package/dist/index.d.cts.map +1 -0
- package/dist/index.d.ts +11 -6
- package/dist/index.d.ts.map +1 -1
- package/dist/index.js +425 -0
- package/dist/index.js.map +1 -0
- package/dist/iterateJsdoc.cjs +1528 -1988
- package/dist/iterateJsdoc.cjs.map +1 -1
- package/dist/iterateJsdoc.d.cts +471 -0
- package/dist/iterateJsdoc.d.cts.map +1 -0
- package/dist/iterateJsdoc.d.ts +358 -349
- package/dist/iterateJsdoc.d.ts.map +1 -1
- package/dist/iterateJsdoc.js +1617 -0
- package/dist/iterateJsdoc.js.map +1 -0
- package/dist/jsdocUtils.cjs +1009 -1376
- package/dist/jsdocUtils.cjs.map +1 -1
- package/dist/jsdocUtils.js +1123 -0
- package/dist/jsdocUtils.js.map +1 -0
- package/dist/rules/checkAccess.cjs +29 -36
- package/dist/rules/checkAccess.cjs.map +1 -1
- package/dist/rules/checkAccess.js +33 -0
- package/dist/rules/checkAccess.js.map +1 -0
- package/dist/rules/checkAlignment.cjs +41 -54
- package/dist/rules/checkAlignment.cjs.map +1 -1
- package/dist/rules/checkAlignment.js +47 -0
- package/dist/rules/checkAlignment.js.map +1 -0
- package/dist/rules/checkExamples.cjs +327 -484
- package/dist/rules/checkExamples.cjs.map +1 -1
- package/dist/rules/checkExamples.js +348 -0
- package/dist/rules/checkExamples.js.map +1 -0
- package/dist/rules/checkIndentation.cjs +50 -65
- package/dist/rules/checkIndentation.cjs.map +1 -1
- package/dist/rules/checkIndentation.js +59 -0
- package/dist/rules/checkIndentation.js.map +1 -0
- package/dist/rules/checkLineAlignment.cjs +220 -311
- package/dist/rules/checkLineAlignment.cjs.map +1 -1
- package/dist/rules/checkLineAlignment.js +229 -0
- package/dist/rules/checkLineAlignment.js.map +1 -0
- package/dist/rules/checkParamNames.cjs +227 -335
- package/dist/rules/checkParamNames.cjs.map +1 -1
- package/dist/rules/checkParamNames.js +237 -0
- package/dist/rules/checkParamNames.js.map +1 -0
- package/dist/rules/checkPropertyNames.cjs +78 -106
- package/dist/rules/checkPropertyNames.cjs.map +1 -1
- package/dist/rules/checkPropertyNames.js +88 -0
- package/dist/rules/checkPropertyNames.js.map +1 -0
- package/dist/rules/checkSyntax.cjs +21 -34
- package/dist/rules/checkSyntax.cjs.map +1 -1
- package/dist/rules/checkSyntax.js +25 -0
- package/dist/rules/checkSyntax.js.map +1 -0
- package/dist/rules/checkTagNames.cjs +188 -210
- package/dist/rules/checkTagNames.cjs.map +1 -1
- package/dist/rules/checkTagNames.js +191 -0
- package/dist/rules/checkTagNames.js.map +1 -0
- package/dist/rules/checkTemplateNames.cjs +121 -178
- package/dist/rules/checkTemplateNames.cjs.map +1 -1
- package/dist/rules/checkTemplateNames.js +124 -0
- package/dist/rules/checkTemplateNames.js.map +1 -0
- package/dist/rules/checkTypes.cjs +291 -385
- package/dist/rules/checkTypes.cjs.map +1 -1
- package/dist/rules/checkTypes.js +299 -0
- package/dist/rules/checkTypes.js.map +1 -0
- package/dist/rules/checkValues.cjs +100 -146
- package/dist/rules/checkValues.cjs.map +1 -1
- package/dist/rules/checkValues.js +103 -0
- package/dist/rules/checkValues.js.map +1 -0
- package/dist/rules/convertToJsdocComments.cjs +228 -306
- package/dist/rules/convertToJsdocComments.cjs.map +1 -1
- package/dist/rules/convertToJsdocComments.js +231 -0
- package/dist/rules/convertToJsdocComments.js.map +1 -0
- package/dist/rules/emptyTags.cjs +62 -72
- package/dist/rules/emptyTags.cjs.map +1 -1
- package/dist/rules/emptyTags.js +67 -0
- package/dist/rules/emptyTags.js.map +1 -0
- package/dist/rules/implementsOnClasses.cjs +36 -56
- package/dist/rules/implementsOnClasses.cjs.map +1 -1
- package/dist/rules/implementsOnClasses.js +40 -0
- package/dist/rules/implementsOnClasses.js.map +1 -0
- package/dist/rules/importsAsDependencies.cjs +62 -99
- package/dist/rules/importsAsDependencies.cjs.map +1 -1
- package/dist/rules/importsAsDependencies.js +68 -0
- package/dist/rules/importsAsDependencies.js.map +1 -0
- package/dist/rules/informativeDocs.cjs +105 -142
- package/dist/rules/informativeDocs.cjs.map +1 -1
- package/dist/rules/informativeDocs.js +110 -0
- package/dist/rules/informativeDocs.js.map +1 -0
- package/dist/rules/linesBeforeBlock.cjs +70 -105
- package/dist/rules/linesBeforeBlock.cjs.map +1 -1
- package/dist/rules/linesBeforeBlock.js +75 -0
- package/dist/rules/linesBeforeBlock.js.map +1 -0
- package/dist/rules/matchDescription.cjs +160 -222
- package/dist/rules/matchDescription.cjs.map +1 -1
- package/dist/rules/matchDescription.js +167 -0
- package/dist/rules/matchDescription.js.map +1 -0
- package/dist/rules/matchName.cjs +73 -128
- package/dist/rules/matchName.cjs.map +1 -1
- package/dist/rules/matchName.js +77 -0
- package/dist/rules/matchName.js.map +1 -0
- package/dist/rules/multilineBlocks.cjs +235 -352
- package/dist/rules/multilineBlocks.cjs.map +1 -1
- package/dist/rules/multilineBlocks.js +245 -0
- package/dist/rules/multilineBlocks.js.map +1 -0
- package/dist/rules/noBadBlocks.cjs +63 -86
- package/dist/rules/noBadBlocks.cjs.map +1 -1
- package/dist/rules/noBadBlocks.js +68 -0
- package/dist/rules/noBadBlocks.js.map +1 -0
- package/dist/rules/noBlankBlockDescriptions.cjs +35 -57
- package/dist/rules/noBlankBlockDescriptions.cjs.map +1 -1
- package/dist/rules/noBlankBlockDescriptions.js +41 -0
- package/dist/rules/noBlankBlockDescriptions.js.map +1 -0
- package/dist/rules/noBlankBlocks.cjs +26 -48
- package/dist/rules/noBlankBlocks.cjs.map +1 -1
- package/dist/rules/noBlankBlocks.js +30 -0
- package/dist/rules/noBlankBlocks.js.map +1 -0
- package/dist/rules/noDefaults.cjs +52 -79
- package/dist/rules/noDefaults.cjs.map +1 -1
- package/dist/rules/noDefaults.js +56 -0
- package/dist/rules/noDefaults.js.map +1 -0
- package/dist/rules/noMissingSyntax.cjs +115 -165
- package/dist/rules/noMissingSyntax.cjs.map +1 -1
- package/dist/rules/noMissingSyntax.js +126 -0
- package/dist/rules/noMissingSyntax.js.map +1 -0
- package/dist/rules/noMultiAsterisks.cjs +48 -89
- package/dist/rules/noMultiAsterisks.cjs.map +1 -1
- package/dist/rules/noMultiAsterisks.js +58 -0
- package/dist/rules/noMultiAsterisks.js.map +1 -0
- package/dist/rules/noRestrictedSyntax.cjs +45 -79
- package/dist/rules/noRestrictedSyntax.cjs.map +1 -1
- package/dist/rules/noRestrictedSyntax.js +49 -0
- package/dist/rules/noRestrictedSyntax.js.map +1 -0
- package/dist/rules/noTypes.cjs +59 -80
- package/dist/rules/noTypes.cjs.map +1 -1
- package/dist/rules/noTypes.js +65 -0
- package/dist/rules/noTypes.js.map +1 -0
- package/dist/rules/noUndefinedTypes.cjs +297 -388
- package/dist/rules/noUndefinedTypes.cjs.map +1 -1
- package/dist/rules/noUndefinedTypes.js +303 -0
- package/dist/rules/noUndefinedTypes.js.map +1 -0
- package/dist/rules/requireAsteriskPrefix.cjs +108 -159
- package/dist/rules/requireAsteriskPrefix.cjs.map +1 -1
- package/dist/rules/requireAsteriskPrefix.js +112 -0
- package/dist/rules/requireAsteriskPrefix.js.map +1 -0
- package/dist/rules/requireDescription.cjs +89 -129
- package/dist/rules/requireDescription.cjs.map +1 -1
- package/dist/rules/requireDescription.js +95 -0
- package/dist/rules/requireDescription.js.map +1 -0
- package/dist/rules/requireDescriptionCompleteSentence.cjs +201 -262
- package/dist/rules/requireDescriptionCompleteSentence.cjs.map +1 -1
- package/dist/rules/requireDescriptionCompleteSentence.js +220 -0
- package/dist/rules/requireDescriptionCompleteSentence.js.map +1 -0
- package/dist/rules/requireExample.cjs +73 -104
- package/dist/rules/requireExample.cjs.map +1 -1
- package/dist/rules/requireExample.js +77 -0
- package/dist/rules/requireExample.js.map +1 -0
- package/dist/rules/requireFileOverview.cjs +75 -129
- package/dist/rules/requireFileOverview.cjs.map +1 -1
- package/dist/rules/requireFileOverview.js +81 -0
- package/dist/rules/requireFileOverview.js.map +1 -0
- package/dist/rules/requireHyphenBeforeParamDescription.cjs +85 -133
- package/dist/rules/requireHyphenBeforeParamDescription.cjs.map +1 -1
- package/dist/rules/requireHyphenBeforeParamDescription.js +89 -0
- package/dist/rules/requireHyphenBeforeParamDescription.js.map +1 -0
- package/dist/rules/requireJsdoc.cjs +384 -557
- package/dist/rules/requireJsdoc.cjs.map +1 -1
- package/dist/rules/requireJsdoc.js +404 -0
- package/dist/rules/requireJsdoc.js.map +1 -0
- package/dist/rules/requireParam.cjs +336 -526
- package/dist/rules/requireParam.cjs.map +1 -1
- package/dist/rules/requireParam.js +344 -0
- package/dist/rules/requireParam.js.map +1 -0
- package/dist/rules/requireParamDescription.cjs +55 -80
- package/dist/rules/requireParamDescription.cjs.map +1 -1
- package/dist/rules/requireParamDescription.js +59 -0
- package/dist/rules/requireParamDescription.js.map +1 -0
- package/dist/rules/requireParamName.cjs +32 -50
- package/dist/rules/requireParamName.cjs.map +1 -1
- package/dist/rules/requireParamName.js +36 -0
- package/dist/rules/requireParamName.js.map +1 -0
- package/dist/rules/requireParamType.cjs +55 -80
- package/dist/rules/requireParamType.cjs.map +1 -1
- package/dist/rules/requireParamType.js +59 -0
- package/dist/rules/requireParamType.js.map +1 -0
- package/dist/rules/requireProperty.cjs +31 -42
- package/dist/rules/requireProperty.cjs.map +1 -1
- package/dist/rules/requireProperty.js +35 -0
- package/dist/rules/requireProperty.js.map +1 -0
- package/dist/rules/requirePropertyDescription.cjs +17 -25
- package/dist/rules/requirePropertyDescription.cjs.map +1 -1
- package/dist/rules/requirePropertyDescription.js +21 -0
- package/dist/rules/requirePropertyDescription.js.map +1 -0
- package/dist/rules/requirePropertyName.cjs +17 -25
- package/dist/rules/requirePropertyName.cjs.map +1 -1
- package/dist/rules/requirePropertyName.js +21 -0
- package/dist/rules/requirePropertyName.js.map +1 -0
- package/dist/rules/requirePropertyType.cjs +17 -25
- package/dist/rules/requirePropertyType.cjs.map +1 -1
- package/dist/rules/requirePropertyType.js +21 -0
- package/dist/rules/requirePropertyType.js.map +1 -0
- package/dist/rules/requireReturns.cjs +125 -203
- package/dist/rules/requireReturns.cjs.map +1 -1
- package/dist/rules/requireReturns.js +131 -0
- package/dist/rules/requireReturns.js.map +1 -0
- package/dist/rules/requireReturnsCheck.cjs +60 -103
- package/dist/rules/requireReturnsCheck.cjs.map +1 -1
- package/dist/rules/requireReturnsCheck.js +66 -0
- package/dist/rules/requireReturnsCheck.js.map +1 -0
- package/dist/rules/requireReturnsDescription.cjs +39 -54
- package/dist/rules/requireReturnsDescription.cjs.map +1 -1
- package/dist/rules/requireReturnsDescription.js +43 -0
- package/dist/rules/requireReturnsDescription.js.map +1 -0
- package/dist/rules/requireReturnsType.cjs +32 -50
- package/dist/rules/requireReturnsType.cjs.map +1 -1
- package/dist/rules/requireReturnsType.js +36 -0
- package/dist/rules/requireReturnsType.js.map +1 -0
- package/dist/rules/requireTemplate.cjs +119 -178
- package/dist/rules/requireTemplate.cjs.map +1 -1
- package/dist/rules/requireTemplate.js +122 -0
- package/dist/rules/requireTemplate.js.map +1 -0
- package/dist/rules/requireThrows.cjs +61 -95
- package/dist/rules/requireThrows.cjs.map +1 -1
- package/dist/rules/requireThrows.js +67 -0
- package/dist/rules/requireThrows.js.map +1 -0
- package/dist/rules/requireYields.cjs +106 -166
- package/dist/rules/requireYields.cjs.map +1 -1
- package/dist/rules/requireYields.js +115 -0
- package/dist/rules/requireYields.js.map +1 -0
- package/dist/rules/requireYieldsCheck.cjs +96 -152
- package/dist/rules/requireYieldsCheck.cjs.map +1 -1
- package/dist/rules/requireYieldsCheck.js +105 -0
- package/dist/rules/requireYieldsCheck.js.map +1 -0
- package/dist/rules/sortTags.cjs +258 -444
- package/dist/rules/sortTags.cjs.map +1 -1
- package/dist/rules/sortTags.js +262 -0
- package/dist/rules/sortTags.js.map +1 -0
- package/dist/rules/tagLines.cjs +179 -266
- package/dist/rules/tagLines.cjs.map +1 -1
- package/dist/rules/tagLines.js +183 -0
- package/dist/rules/tagLines.js.map +1 -0
- package/dist/rules/textEscaping.cjs +92 -127
- package/dist/rules/textEscaping.cjs.map +1 -1
- package/dist/rules/textEscaping.js +102 -0
- package/dist/rules/textEscaping.js.map +1 -0
- package/dist/rules/validTypes.cjs +252 -265
- package/dist/rules/validTypes.cjs.map +1 -1
- package/dist/rules/validTypes.js +259 -0
- package/dist/rules/validTypes.js.map +1 -0
- package/dist/tagNames.cjs +134 -170
- package/dist/tagNames.cjs.map +1 -1
- package/dist/tagNames.js +144 -0
- package/dist/tagNames.js.map +1 -0
- package/dist/utils/hasReturnValue.cjs +246 -474
- package/dist/utils/hasReturnValue.cjs.map +1 -1
- package/dist/utils/hasReturnValue.js +265 -0
- package/dist/utils/hasReturnValue.js.map +1 -0
- package/package.json +48 -38
- package/dist/generateRule.cjs +0 -242
- package/dist/generateRule.cjs.map +0 -1
- package/src/WarnSettings.js +0 -34
- package/src/alignTransform.js +0 -358
- package/src/defaultTagOrder.js +0 -169
- package/src/exportParser.js +0 -978
- package/src/getDefaultTagStructureForMode.js +0 -969
- package/src/getJsdocProcessorPlugin.js +0 -652
- package/src/index.js +0 -530
- package/src/iterateJsdoc.js +0 -2518
- package/src/jsdocUtils.js +0 -1896
- package/src/rules/checkAccess.js +0 -45
- package/src/rules/checkAlignment.js +0 -63
- package/src/rules/checkExamples.js +0 -589
- package/src/rules/checkIndentation.js +0 -75
- package/src/rules/checkLineAlignment.js +0 -372
- package/src/rules/checkParamNames.js +0 -474
- package/src/rules/checkPropertyNames.js +0 -152
- package/src/rules/checkSyntax.js +0 -30
- package/src/rules/checkTagNames.js +0 -314
- package/src/rules/checkTemplateNames.js +0 -204
- package/src/rules/checkTypes.js +0 -535
- package/src/rules/checkValues.js +0 -248
- package/src/rules/convertToJsdocComments.js +0 -398
- package/src/rules/emptyTags.js +0 -98
- package/src/rules/implementsOnClasses.js +0 -64
- package/src/rules/importsAsDependencies.js +0 -133
- package/src/rules/informativeDocs.js +0 -189
- package/src/rules/linesBeforeBlock.js +0 -134
- package/src/rules/matchDescription.js +0 -286
- package/src/rules/matchName.js +0 -151
- package/src/rules/multilineBlocks.js +0 -493
- package/src/rules/noBadBlocks.js +0 -119
- package/src/rules/noBlankBlockDescriptions.js +0 -69
- package/src/rules/noBlankBlocks.js +0 -53
- package/src/rules/noDefaults.js +0 -85
- package/src/rules/noMissingSyntax.js +0 -195
- package/src/rules/noMultiAsterisks.js +0 -134
- package/src/rules/noRestrictedSyntax.js +0 -91
- package/src/rules/noTypes.js +0 -93
- package/src/rules/noUndefinedTypes.js +0 -543
- package/src/rules/requireAsteriskPrefix.js +0 -190
- package/src/rules/requireDescription.js +0 -161
- package/src/rules/requireDescriptionCompleteSentence.js +0 -335
- package/src/rules/requireExample.js +0 -118
- package/src/rules/requireFileOverview.js +0 -154
- package/src/rules/requireHyphenBeforeParamDescription.js +0 -176
- package/src/rules/requireJsdoc.js +0 -743
- package/src/rules/requireParam.js +0 -602
- package/src/rules/requireParamDescription.js +0 -89
- package/src/rules/requireParamName.js +0 -55
- package/src/rules/requireParamType.js +0 -89
- package/src/rules/requireProperty.js +0 -48
- package/src/rules/requirePropertyDescription.js +0 -25
- package/src/rules/requirePropertyName.js +0 -25
- package/src/rules/requirePropertyType.js +0 -25
- package/src/rules/requireReturns.js +0 -238
- package/src/rules/requireReturnsCheck.js +0 -145
- package/src/rules/requireReturnsDescription.js +0 -59
- package/src/rules/requireReturnsType.js +0 -51
- package/src/rules/requireTemplate.js +0 -201
- package/src/rules/requireThrows.js +0 -111
- package/src/rules/requireYields.js +0 -216
- package/src/rules/requireYieldsCheck.js +0 -208
- package/src/rules/sortTags.js +0 -558
- package/src/rules/tagLines.js +0 -359
- package/src/rules/textEscaping.js +0 -154
- package/src/rules/validTypes.js +0 -401
- package/src/tagNames.js +0 -238
- package/src/utils/hasReturnValue.js +0 -572
|
@@ -1,495 +1,267 @@
|
|
|
1
|
-
"use strict";
|
|
2
1
|
|
|
3
|
-
|
|
4
|
-
value: true
|
|
5
|
-
});
|
|
6
|
-
exports.hasValueOrExecutorHasNonEmptyResolveValue = exports.hasReturnValue = void 0;
|
|
2
|
+
//#region src/utils/hasReturnValue.js
|
|
7
3
|
/**
|
|
8
|
-
|
|
9
|
-
|
|
10
|
-
|
|
11
|
-
|
|
4
|
+
* @typedef {import('estree').Node|
|
|
5
|
+
* import('@typescript-eslint/types').TSESTree.Node} ESTreeOrTypeScriptNode
|
|
6
|
+
*/
|
|
12
7
|
/**
|
|
13
|
-
|
|
14
|
-
|
|
15
|
-
|
|
16
|
-
|
|
17
|
-
|
|
18
|
-
const isNewPromiseExpression = node => {
|
|
19
|
-
|
|
8
|
+
* Checks if a node is a promise but has no resolve value or an empty value.
|
|
9
|
+
* An `undefined` resolve does not count.
|
|
10
|
+
* @param {ESTreeOrTypeScriptNode|undefined|null} node
|
|
11
|
+
* @returns {boolean|undefined|null}
|
|
12
|
+
*/
|
|
13
|
+
const isNewPromiseExpression = (node) => {
|
|
14
|
+
return node && node.type === "NewExpression" && node.callee.type === "Identifier" && node.callee.name === "Promise";
|
|
20
15
|
};
|
|
21
|
-
|
|
22
16
|
/**
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
const isVoidPromise = node => {
|
|
27
|
-
|
|
28
|
-
/* c8 ignore next 5 */
|
|
29
|
-
// eslint-disable-next-line @stylistic/operator-linebreak -- c8
|
|
30
|
-
|| /** @type {import('@typescript-eslint/types').TSESTree.TSTypeReference} */node
|
|
31
|
-
// @ts-expect-error Ok
|
|
32
|
-
?.typeParameters?.params?.[0]?.type === 'TSVoidKeyword';
|
|
17
|
+
* @param {ESTreeOrTypeScriptNode|null|undefined} node
|
|
18
|
+
* @returns {boolean}
|
|
19
|
+
*/
|
|
20
|
+
const isVoidPromise = (node) => {
|
|
21
|
+
return node?.typeArguments?.params?.[0]?.type === "TSVoidKeyword" || node?.typeParameters?.params?.[0]?.type === "TSVoidKeyword";
|
|
33
22
|
};
|
|
34
|
-
const undefinedKeywords = new Set([
|
|
35
|
-
|
|
23
|
+
const undefinedKeywords = new Set([
|
|
24
|
+
"TSNeverKeyword",
|
|
25
|
+
"TSUndefinedKeyword",
|
|
26
|
+
"TSVoidKeyword"
|
|
27
|
+
]);
|
|
36
28
|
/**
|
|
37
|
-
|
|
38
|
-
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
|
|
42
|
-
|
|
43
|
-
// eslint-disable-next-line complexity
|
|
29
|
+
* Checks if a node has a return statement. Void return does not count.
|
|
30
|
+
* @param {ESTreeOrTypeScriptNode|undefined|null} node
|
|
31
|
+
* @param {boolean} [throwOnNullReturn]
|
|
32
|
+
* @param {PromiseFilter} [promFilter]
|
|
33
|
+
* @returns {boolean|undefined}
|
|
34
|
+
*/
|
|
44
35
|
const hasReturnValue = (node, throwOnNullReturn, promFilter) => {
|
|
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
|
-
|
|
73
|
-
|
|
74
|
-
|
|
75
|
-
|
|
76
|
-
|
|
77
|
-
|
|
78
|
-
|
|
79
|
-
|
|
80
|
-
|
|
81
|
-
|
|
82
|
-
|
|
83
|
-
}
|
|
84
|
-
return false;
|
|
85
|
-
}
|
|
86
|
-
if (promFilter && isNewPromiseExpression(node.argument)) {
|
|
87
|
-
// Let caller decide how to filter, but this is, at the least,
|
|
88
|
-
// a return of sorts and truthy
|
|
89
|
-
return promFilter(node.argument);
|
|
90
|
-
}
|
|
91
|
-
return true;
|
|
92
|
-
}
|
|
93
|
-
case 'SwitchStatement':
|
|
94
|
-
{
|
|
95
|
-
return node.cases.some(someCase => {
|
|
96
|
-
return someCase.consequent.some(nde => {
|
|
97
|
-
return hasReturnValue(nde, throwOnNullReturn, promFilter);
|
|
98
|
-
});
|
|
99
|
-
});
|
|
100
|
-
}
|
|
101
|
-
case 'TryStatement':
|
|
102
|
-
{
|
|
103
|
-
return hasReturnValue(node.block, throwOnNullReturn, promFilter) || hasReturnValue(node.handler && node.handler.body, throwOnNullReturn, promFilter) || hasReturnValue(node.finalizer, throwOnNullReturn, promFilter);
|
|
104
|
-
}
|
|
105
|
-
case 'TSDeclareFunction':
|
|
106
|
-
case 'TSFunctionType':
|
|
107
|
-
case 'TSMethodSignature':
|
|
108
|
-
{
|
|
109
|
-
const type = node?.returnType?.typeAnnotation?.type;
|
|
110
|
-
return type && !undefinedKeywords.has(type);
|
|
111
|
-
}
|
|
112
|
-
default:
|
|
113
|
-
{
|
|
114
|
-
return false;
|
|
115
|
-
}
|
|
116
|
-
}
|
|
36
|
+
if (!node) return false;
|
|
37
|
+
switch (node.type) {
|
|
38
|
+
case "ArrowFunctionExpression":
|
|
39
|
+
case "FunctionDeclaration":
|
|
40
|
+
case "FunctionExpression": return "expression" in node && node.expression && (!isNewPromiseExpression(node.body) || !isVoidPromise(node.body)) || hasReturnValue(node.body, throwOnNullReturn, promFilter);
|
|
41
|
+
case "BlockStatement": return node.body.some((bodyNode) => {
|
|
42
|
+
return bodyNode.type !== "FunctionDeclaration" && hasReturnValue(bodyNode, throwOnNullReturn, promFilter);
|
|
43
|
+
});
|
|
44
|
+
case "DoWhileStatement":
|
|
45
|
+
case "ForInStatement":
|
|
46
|
+
case "ForOfStatement":
|
|
47
|
+
case "ForStatement":
|
|
48
|
+
case "LabeledStatement":
|
|
49
|
+
case "WhileStatement":
|
|
50
|
+
case "WithStatement": return hasReturnValue(node.body, throwOnNullReturn, promFilter);
|
|
51
|
+
case "IfStatement": return hasReturnValue(node.consequent, throwOnNullReturn, promFilter) || hasReturnValue(node.alternate, throwOnNullReturn, promFilter);
|
|
52
|
+
case "MethodDefinition": return hasReturnValue(node.value, throwOnNullReturn, promFilter);
|
|
53
|
+
case "ReturnStatement":
|
|
54
|
+
if (node.argument === null) {
|
|
55
|
+
if (throwOnNullReturn) throw new Error("Null return");
|
|
56
|
+
return false;
|
|
57
|
+
}
|
|
58
|
+
if (promFilter && isNewPromiseExpression(node.argument)) return promFilter(node.argument);
|
|
59
|
+
return true;
|
|
60
|
+
case "SwitchStatement": return node.cases.some((someCase) => {
|
|
61
|
+
return someCase.consequent.some((nde) => {
|
|
62
|
+
return hasReturnValue(nde, throwOnNullReturn, promFilter);
|
|
63
|
+
});
|
|
64
|
+
});
|
|
65
|
+
case "TryStatement": return hasReturnValue(node.block, throwOnNullReturn, promFilter) || hasReturnValue(node.handler && node.handler.body, throwOnNullReturn, promFilter) || hasReturnValue(node.finalizer, throwOnNullReturn, promFilter);
|
|
66
|
+
case "TSDeclareFunction":
|
|
67
|
+
case "TSFunctionType":
|
|
68
|
+
case "TSMethodSignature": {
|
|
69
|
+
const type = node?.returnType?.typeAnnotation?.type;
|
|
70
|
+
return type && !undefinedKeywords.has(type);
|
|
71
|
+
}
|
|
72
|
+
default: return false;
|
|
73
|
+
}
|
|
117
74
|
};
|
|
118
|
-
|
|
119
75
|
/**
|
|
120
|
-
|
|
121
|
-
|
|
122
|
-
|
|
123
|
-
|
|
124
|
-
|
|
125
|
-
// eslint-disable-next-line complexity
|
|
126
|
-
exports.hasReturnValue = hasReturnValue;
|
|
76
|
+
* Checks if a node has a return statement. Void return does not count.
|
|
77
|
+
* @param {ESTreeOrTypeScriptNode|null|undefined} node
|
|
78
|
+
* @param {PromiseFilter} promFilter
|
|
79
|
+
* @returns {undefined|boolean|ESTreeOrTypeScriptNode}
|
|
80
|
+
*/
|
|
127
81
|
const allBrancheshaveReturnValues = (node, promFilter) => {
|
|
128
|
-
|
|
129
|
-
|
|
130
|
-
|
|
131
|
-
|
|
132
|
-
|
|
133
|
-
|
|
134
|
-
|
|
135
|
-
|
|
136
|
-
|
|
137
|
-
|
|
138
|
-
|
|
139
|
-
|
|
140
|
-
|
|
141
|
-
|
|
142
|
-
|
|
143
|
-
|
|
144
|
-
|
|
145
|
-
|
|
146
|
-
|
|
147
|
-
|
|
148
|
-
|
|
149
|
-
|
|
150
|
-
|
|
151
|
-
|
|
152
|
-
|
|
153
|
-
|
|
154
|
-
|
|
155
|
-
|
|
156
|
-
|
|
157
|
-
|
|
158
|
-
|
|
159
|
-
|
|
160
|
-
|
|
161
|
-
|
|
162
|
-
|
|
163
|
-
|
|
164
|
-
|
|
165
|
-
|
|
166
|
-
|
|
167
|
-
|
|
168
|
-
|
|
169
|
-
|
|
170
|
-
|
|
171
|
-
|
|
172
|
-
|
|
173
|
-
|
|
174
|
-
|
|
175
|
-
{
|
|
176
|
-
return allBrancheshaveReturnValues(node.consequent, promFilter) && allBrancheshaveReturnValues(node.alternate, promFilter);
|
|
177
|
-
}
|
|
178
|
-
case 'ReturnStatement':
|
|
179
|
-
{
|
|
180
|
-
// void return does not count.
|
|
181
|
-
if (node.argument === null) {
|
|
182
|
-
return false;
|
|
183
|
-
}
|
|
184
|
-
if (promFilter && isNewPromiseExpression(node.argument)) {
|
|
185
|
-
// Let caller decide how to filter, but this is, at the least,
|
|
186
|
-
// a return of sorts and truthy
|
|
187
|
-
return promFilter(node.argument);
|
|
188
|
-
}
|
|
189
|
-
return true;
|
|
190
|
-
}
|
|
191
|
-
case 'SwitchStatement':
|
|
192
|
-
{
|
|
193
|
-
return /** @type {import('@typescript-eslint/types').TSESTree.SwitchStatement} */node.cases.every(someCase => {
|
|
194
|
-
return !someCase.consequent.some(consNode => {
|
|
195
|
-
return consNode.type === 'BreakStatement' || consNode.type === 'ReturnStatement' && consNode.argument === null;
|
|
196
|
-
});
|
|
197
|
-
});
|
|
198
|
-
}
|
|
199
|
-
case 'ThrowStatement':
|
|
200
|
-
{
|
|
201
|
-
return true;
|
|
202
|
-
}
|
|
203
|
-
case 'TryStatement':
|
|
204
|
-
{
|
|
205
|
-
// If `finally` returns, all return
|
|
206
|
-
return node.finalizer && allBrancheshaveReturnValues(node.finalizer, promFilter) ||
|
|
207
|
-
// Return in `try`/`catch` may still occur despite `finally`
|
|
208
|
-
allBrancheshaveReturnValues(node.block, promFilter) && (!node.handler || allBrancheshaveReturnValues(node.handler && node.handler.body, promFilter)) && (!node.finalizer || (() => {
|
|
209
|
-
try {
|
|
210
|
-
hasReturnValue(node.finalizer, true, promFilter);
|
|
211
|
-
} catch (error) {
|
|
212
|
-
if (/** @type {Error} */error.message === 'Null return') {
|
|
213
|
-
return false;
|
|
214
|
-
}
|
|
215
|
-
/* c8 ignore next 3 */
|
|
216
|
-
// eslint-disable-next-line @stylistic/padding-line-between-statements -- c8
|
|
217
|
-
throw error;
|
|
218
|
-
}
|
|
219
|
-
|
|
220
|
-
// As long as not an explicit empty return, then return true
|
|
221
|
-
return true;
|
|
222
|
-
})());
|
|
223
|
-
}
|
|
224
|
-
case 'TSDeclareFunction':
|
|
225
|
-
case 'TSFunctionType':
|
|
226
|
-
case 'TSMethodSignature':
|
|
227
|
-
{
|
|
228
|
-
const type = node?.returnType?.typeAnnotation?.type;
|
|
229
|
-
return type && !undefinedKeywords.has(type);
|
|
230
|
-
}
|
|
231
|
-
default:
|
|
232
|
-
{
|
|
233
|
-
return false;
|
|
234
|
-
}
|
|
235
|
-
}
|
|
82
|
+
if (!node) return false;
|
|
83
|
+
switch (node.type) {
|
|
84
|
+
case "ArrowFunctionExpression":
|
|
85
|
+
case "FunctionDeclaration":
|
|
86
|
+
case "FunctionExpression": return "expression" in node && node.expression && (!isNewPromiseExpression(node.body) || !isVoidPromise(node.body)) || allBrancheshaveReturnValues(node.body, promFilter) || node.body.body.some((nde) => {
|
|
87
|
+
return nde.type === "ReturnStatement";
|
|
88
|
+
});
|
|
89
|
+
case "BlockStatement": {
|
|
90
|
+
const lastBodyNode = node.body.slice(-1)[0];
|
|
91
|
+
return allBrancheshaveReturnValues(lastBodyNode, promFilter);
|
|
92
|
+
}
|
|
93
|
+
case "DoWhileStatement":
|
|
94
|
+
case "WhileStatement": if (node.test.value === true) return hasReturnValue(node.body, false, promFilter);
|
|
95
|
+
case "ForStatement": if (node.test === null) return hasReturnValue(node.body, false, promFilter);
|
|
96
|
+
case "ForInStatement":
|
|
97
|
+
case "ForOfStatement":
|
|
98
|
+
case "LabeledStatement":
|
|
99
|
+
case "WithStatement": return allBrancheshaveReturnValues(node.body, promFilter);
|
|
100
|
+
case "IfStatement": return allBrancheshaveReturnValues(node.consequent, promFilter) && allBrancheshaveReturnValues(node.alternate, promFilter);
|
|
101
|
+
case "ReturnStatement":
|
|
102
|
+
if (node.argument === null) return false;
|
|
103
|
+
if (promFilter && isNewPromiseExpression(node.argument)) return promFilter(node.argument);
|
|
104
|
+
return true;
|
|
105
|
+
case "SwitchStatement": return node.cases.every((someCase) => {
|
|
106
|
+
return !someCase.consequent.some((consNode) => {
|
|
107
|
+
return consNode.type === "BreakStatement" || consNode.type === "ReturnStatement" && consNode.argument === null;
|
|
108
|
+
});
|
|
109
|
+
});
|
|
110
|
+
case "ThrowStatement": return true;
|
|
111
|
+
case "TryStatement": return node.finalizer && allBrancheshaveReturnValues(node.finalizer, promFilter) || allBrancheshaveReturnValues(node.block, promFilter) && (!node.handler || allBrancheshaveReturnValues(node.handler && node.handler.body, promFilter)) && (!node.finalizer || (() => {
|
|
112
|
+
try {
|
|
113
|
+
hasReturnValue(node.finalizer, true, promFilter);
|
|
114
|
+
} catch (error) {
|
|
115
|
+
if (error.message === "Null return") return false;
|
|
116
|
+
/* c8 ignore next 3 */
|
|
117
|
+
throw error;
|
|
118
|
+
}
|
|
119
|
+
return true;
|
|
120
|
+
})());
|
|
121
|
+
case "TSDeclareFunction":
|
|
122
|
+
case "TSFunctionType":
|
|
123
|
+
case "TSMethodSignature": {
|
|
124
|
+
const type = node?.returnType?.typeAnnotation?.type;
|
|
125
|
+
return type && !undefinedKeywords.has(type);
|
|
126
|
+
}
|
|
127
|
+
default: return false;
|
|
128
|
+
}
|
|
236
129
|
};
|
|
237
|
-
|
|
238
130
|
/**
|
|
239
|
-
|
|
240
|
-
|
|
241
|
-
|
|
242
|
-
|
|
243
|
-
|
|
131
|
+
* @callback PromiseFilter
|
|
132
|
+
* @param {ESTreeOrTypeScriptNode|undefined} node
|
|
133
|
+
* @returns {boolean}
|
|
134
|
+
*/
|
|
244
135
|
/**
|
|
245
|
-
|
|
246
|
-
|
|
247
|
-
|
|
248
|
-
|
|
249
|
-
|
|
250
|
-
|
|
251
|
-
|
|
252
|
-
|
|
253
|
-
|
|
254
|
-
|
|
255
|
-
|
|
256
|
-
// eslint-disable-next-line complexity
|
|
136
|
+
* Avoids further checking child nodes if a nested function shadows the
|
|
137
|
+
* resolver, but otherwise, if name is used (by call or passed in as an
|
|
138
|
+
* argument to another function), will be considered as non-empty.
|
|
139
|
+
*
|
|
140
|
+
* This could check for redeclaration of the resolver, but as such is
|
|
141
|
+
* unlikely, we avoid the performance cost of checking everywhere for
|
|
142
|
+
* (re)declarations or assignments.
|
|
143
|
+
* @param {import('@typescript-eslint/types').TSESTree.Node|null|undefined} node
|
|
144
|
+
* @param {string} resolverName
|
|
145
|
+
* @returns {boolean}
|
|
146
|
+
*/
|
|
257
147
|
const hasNonEmptyResolverCall = (node, resolverName) => {
|
|
258
|
-
|
|
259
|
-
|
|
260
|
-
|
|
261
|
-
|
|
262
|
-
|
|
263
|
-
|
|
264
|
-
|
|
265
|
-
|
|
266
|
-
|
|
267
|
-
|
|
268
|
-
|
|
269
|
-
|
|
270
|
-
|
|
271
|
-
|
|
272
|
-
|
|
273
|
-
|
|
274
|
-
|
|
275
|
-
|
|
276
|
-
|
|
277
|
-
|
|
278
|
-
|
|
279
|
-
|
|
280
|
-
|
|
281
|
-
|
|
282
|
-
|
|
283
|
-
|
|
284
|
-
|
|
285
|
-
|
|
286
|
-
|
|
287
|
-
|
|
288
|
-
|
|
289
|
-
|
|
290
|
-
|
|
291
|
-
|
|
292
|
-
|
|
293
|
-
|
|
294
|
-
|
|
295
|
-
|
|
296
|
-
|
|
297
|
-
|
|
298
|
-
|
|
299
|
-
|
|
300
|
-
|
|
301
|
-
|
|
302
|
-
|
|
303
|
-
|
|
304
|
-
|
|
305
|
-
|
|
306
|
-
|
|
307
|
-
|
|
308
|
-
|
|
309
|
-
|
|
310
|
-
|
|
311
|
-
|
|
312
|
-
|
|
313
|
-
|
|
314
|
-
|
|
315
|
-
|
|
316
|
-
|
|
317
|
-
|
|
318
|
-
|
|
319
|
-
|
|
320
|
-
|
|
321
|
-
|
|
322
|
-
|
|
323
|
-
|
|
324
|
-
|
|
325
|
-
|
|
326
|
-
|
|
327
|
-
|
|
328
|
-
|
|
329
|
-
|
|
330
|
-
|
|
331
|
-
|
|
332
|
-
|
|
333
|
-
|
|
334
|
-
|
|
335
|
-
|
|
336
|
-
|
|
337
|
-
|
|
338
|
-
|
|
339
|
-
|
|
340
|
-
case 'ForOfStatement':
|
|
341
|
-
case 'ForStatement':
|
|
342
|
-
case 'LabeledStatement':
|
|
343
|
-
case 'WhileStatement':
|
|
344
|
-
case 'WithStatement':
|
|
345
|
-
{
|
|
346
|
-
return hasNonEmptyResolverCall(node.body, resolverName);
|
|
347
|
-
}
|
|
348
|
-
|
|
349
|
-
/* c8 ignore next 2 -- In Babel? */
|
|
350
|
-
// @ts-expect-error Babel?
|
|
351
|
-
case 'Import':
|
|
352
|
-
case 'ImportExpression':
|
|
353
|
-
return hasNonEmptyResolverCall(node.source, resolverName);
|
|
354
|
-
// ?.
|
|
355
|
-
/* c8 ignore next 2 -- In Babel? */
|
|
356
|
-
case 'MemberExpression':
|
|
357
|
-
|
|
358
|
-
// @ts-expect-error Babel?
|
|
359
|
-
case 'OptionalMemberExpression':
|
|
360
|
-
return hasNonEmptyResolverCall(node.object, resolverName) || hasNonEmptyResolverCall(node.property, resolverName);
|
|
361
|
-
case 'ObjectExpression':
|
|
362
|
-
case 'ObjectPattern':
|
|
363
|
-
return node.properties.some(property => {
|
|
364
|
-
return hasNonEmptyResolverCall(property, resolverName);
|
|
365
|
-
});
|
|
366
|
-
/* c8 ignore next 2 -- In Babel? */
|
|
367
|
-
// @ts-expect-error Babel?
|
|
368
|
-
case 'ObjectMethod':
|
|
369
|
-
/* c8 ignore next 6 -- In Babel? */
|
|
370
|
-
// @ts-expect-error
|
|
371
|
-
return node.computed && hasNonEmptyResolverCall(node.key, resolverName) ||
|
|
372
|
-
// @ts-expect-error
|
|
373
|
-
node.arguments.some(nde => {
|
|
374
|
-
return hasNonEmptyResolverCall(nde, resolverName);
|
|
375
|
-
});
|
|
376
|
-
case 'ReturnStatement':
|
|
377
|
-
{
|
|
378
|
-
if (node.argument === null) {
|
|
379
|
-
return false;
|
|
380
|
-
}
|
|
381
|
-
return hasNonEmptyResolverCall(node.argument, resolverName);
|
|
382
|
-
}
|
|
383
|
-
|
|
384
|
-
// Comma
|
|
385
|
-
case 'SequenceExpression':
|
|
386
|
-
case 'TemplateLiteral':
|
|
387
|
-
return node.expressions.some(subExpression => {
|
|
388
|
-
return hasNonEmptyResolverCall(subExpression, resolverName);
|
|
389
|
-
});
|
|
390
|
-
case 'SwitchStatement':
|
|
391
|
-
{
|
|
392
|
-
return node.cases.some(someCase => {
|
|
393
|
-
return someCase.consequent.some(nde => {
|
|
394
|
-
return hasNonEmptyResolverCall(nde, resolverName);
|
|
395
|
-
});
|
|
396
|
-
});
|
|
397
|
-
}
|
|
398
|
-
case 'TaggedTemplateExpression':
|
|
399
|
-
return hasNonEmptyResolverCall(node.quasi, resolverName);
|
|
400
|
-
case 'TryStatement':
|
|
401
|
-
{
|
|
402
|
-
return hasNonEmptyResolverCall(node.block, resolverName) || hasNonEmptyResolverCall(node.handler && node.handler.body, resolverName) || hasNonEmptyResolverCall(node.finalizer, resolverName);
|
|
403
|
-
}
|
|
404
|
-
case 'VariableDeclaration':
|
|
405
|
-
{
|
|
406
|
-
return node.declarations.some(nde => {
|
|
407
|
-
return hasNonEmptyResolverCall(nde, resolverName);
|
|
408
|
-
});
|
|
409
|
-
}
|
|
410
|
-
case 'VariableDeclarator':
|
|
411
|
-
{
|
|
412
|
-
return hasNonEmptyResolverCall(node.id, resolverName) || hasNonEmptyResolverCall(node.init, resolverName);
|
|
413
|
-
}
|
|
414
|
-
|
|
415
|
-
/*
|
|
416
|
-
// Shouldn't need to parse literals/literal components, etc.
|
|
417
|
-
case 'Identifier':
|
|
418
|
-
case 'TemplateElement':
|
|
419
|
-
case 'Super':
|
|
420
|
-
// Exports not relevant in this context
|
|
421
|
-
*/
|
|
422
|
-
default:
|
|
423
|
-
return false;
|
|
424
|
-
}
|
|
148
|
+
if (!node) return false;
|
|
149
|
+
switch (node.type) {
|
|
150
|
+
case "ArrayExpression":
|
|
151
|
+
case "ArrayPattern": return node.elements.some((element) => {
|
|
152
|
+
return hasNonEmptyResolverCall(element, resolverName);
|
|
153
|
+
});
|
|
154
|
+
case "ArrowFunctionExpression":
|
|
155
|
+
case "FunctionDeclaration":
|
|
156
|
+
case "FunctionExpression":
|
|
157
|
+
if (node.params[0]?.name === resolverName) return false;
|
|
158
|
+
return hasNonEmptyResolverCall(node.body, resolverName);
|
|
159
|
+
case "AssignmentExpression":
|
|
160
|
+
case "BinaryExpression":
|
|
161
|
+
case "LogicalExpression": return hasNonEmptyResolverCall(node.left, resolverName) || hasNonEmptyResolverCall(node.right, resolverName);
|
|
162
|
+
case "AssignmentPattern": return hasNonEmptyResolverCall(node.right, resolverName);
|
|
163
|
+
case "AwaitExpression":
|
|
164
|
+
case "SpreadElement":
|
|
165
|
+
case "UnaryExpression":
|
|
166
|
+
case "YieldExpression": return hasNonEmptyResolverCall(node.argument, resolverName);
|
|
167
|
+
case "BlockStatement":
|
|
168
|
+
case "ClassBody": return node.body.some((bodyNode) => {
|
|
169
|
+
return hasNonEmptyResolverCall(bodyNode, resolverName);
|
|
170
|
+
});
|
|
171
|
+
case "CallExpression":
|
|
172
|
+
case "OptionalCallExpression": return node.callee.name === resolverName && (node.arguments.length > 1 || node.arguments[0] !== void 0) || node.arguments.some((nde) => {
|
|
173
|
+
return nde.type === "Identifier" && nde.name === resolverName || hasNonEmptyResolverCall(nde, resolverName);
|
|
174
|
+
});
|
|
175
|
+
case "ChainExpression":
|
|
176
|
+
case "Decorator":
|
|
177
|
+
case "ExpressionStatement": return hasNonEmptyResolverCall(node.expression, resolverName);
|
|
178
|
+
case "ClassDeclaration":
|
|
179
|
+
case "ClassExpression": return hasNonEmptyResolverCall(node.body, resolverName);
|
|
180
|
+
case "ClassMethod":
|
|
181
|
+
case "MethodDefinition": return node.decorators && node.decorators.some((decorator) => {
|
|
182
|
+
return hasNonEmptyResolverCall(decorator, resolverName);
|
|
183
|
+
}) || node.computed && hasNonEmptyResolverCall(node.key, resolverName) || hasNonEmptyResolverCall(node.value, resolverName);
|
|
184
|
+
case "ClassProperty":
|
|
185
|
+
case "ObjectProperty":
|
|
186
|
+
case "Property":
|
|
187
|
+
case "PropertyDefinition": return node.computed && hasNonEmptyResolverCall(node.key, resolverName) || hasNonEmptyResolverCall(node.value, resolverName);
|
|
188
|
+
case "ConditionalExpression":
|
|
189
|
+
case "IfStatement": return hasNonEmptyResolverCall(node.test, resolverName) || hasNonEmptyResolverCall(node.consequent, resolverName) || hasNonEmptyResolverCall(node.alternate, resolverName);
|
|
190
|
+
case "DoWhileStatement":
|
|
191
|
+
case "ForInStatement":
|
|
192
|
+
case "ForOfStatement":
|
|
193
|
+
case "ForStatement":
|
|
194
|
+
case "LabeledStatement":
|
|
195
|
+
case "WhileStatement":
|
|
196
|
+
case "WithStatement": return hasNonEmptyResolverCall(node.body, resolverName);
|
|
197
|
+
case "Import":
|
|
198
|
+
case "ImportExpression": return hasNonEmptyResolverCall(node.source, resolverName);
|
|
199
|
+
case "MemberExpression":
|
|
200
|
+
case "OptionalMemberExpression": return hasNonEmptyResolverCall(node.object, resolverName) || hasNonEmptyResolverCall(node.property, resolverName);
|
|
201
|
+
case "ObjectExpression":
|
|
202
|
+
case "ObjectPattern": return node.properties.some((property) => {
|
|
203
|
+
return hasNonEmptyResolverCall(property, resolverName);
|
|
204
|
+
});
|
|
205
|
+
case "ObjectMethod":
|
|
206
|
+
/* c8 ignore next 6 -- In Babel? */
|
|
207
|
+
return node.computed && hasNonEmptyResolverCall(node.key, resolverName) || node.arguments.some((nde) => {
|
|
208
|
+
return hasNonEmptyResolverCall(nde, resolverName);
|
|
209
|
+
});
|
|
210
|
+
case "ReturnStatement":
|
|
211
|
+
if (node.argument === null) return false;
|
|
212
|
+
return hasNonEmptyResolverCall(node.argument, resolverName);
|
|
213
|
+
case "SequenceExpression":
|
|
214
|
+
case "TemplateLiteral": return node.expressions.some((subExpression) => {
|
|
215
|
+
return hasNonEmptyResolverCall(subExpression, resolverName);
|
|
216
|
+
});
|
|
217
|
+
case "SwitchStatement": return node.cases.some((someCase) => {
|
|
218
|
+
return someCase.consequent.some((nde) => {
|
|
219
|
+
return hasNonEmptyResolverCall(nde, resolverName);
|
|
220
|
+
});
|
|
221
|
+
});
|
|
222
|
+
case "TaggedTemplateExpression": return hasNonEmptyResolverCall(node.quasi, resolverName);
|
|
223
|
+
case "TryStatement": return hasNonEmptyResolverCall(node.block, resolverName) || hasNonEmptyResolverCall(node.handler && node.handler.body, resolverName) || hasNonEmptyResolverCall(node.finalizer, resolverName);
|
|
224
|
+
case "VariableDeclaration": return node.declarations.some((nde) => {
|
|
225
|
+
return hasNonEmptyResolverCall(nde, resolverName);
|
|
226
|
+
});
|
|
227
|
+
case "VariableDeclarator": return hasNonEmptyResolverCall(node.id, resolverName) || hasNonEmptyResolverCall(node.init, resolverName);
|
|
228
|
+
default: return false;
|
|
229
|
+
}
|
|
425
230
|
};
|
|
426
|
-
|
|
427
231
|
/**
|
|
428
|
-
|
|
429
|
-
|
|
430
|
-
|
|
431
|
-
|
|
432
|
-
|
|
433
|
-
|
|
434
|
-
|
|
232
|
+
* Checks if a Promise executor has no resolve value or an empty value.
|
|
233
|
+
* An `undefined` resolve does not count.
|
|
234
|
+
* @param {ESTreeOrTypeScriptNode} node
|
|
235
|
+
* @param {boolean} anyPromiseAsReturn
|
|
236
|
+
* @param {boolean} [allBranches]
|
|
237
|
+
* @returns {boolean}
|
|
238
|
+
*/
|
|
435
239
|
const hasValueOrExecutorHasNonEmptyResolveValue = (node, anyPromiseAsReturn, allBranches) => {
|
|
436
|
-
|
|
437
|
-
|
|
438
|
-
|
|
439
|
-
|
|
440
|
-
|
|
441
|
-
|
|
442
|
-
|
|
443
|
-
|
|
444
|
-
|
|
445
|
-
|
|
446
|
-
|
|
447
|
-
|
|
448
|
-
|
|
449
|
-
|
|
450
|
-
|
|
451
|
-
|
|
452
|
-
|
|
453
|
-
|
|
454
|
-
|
|
455
|
-
|
|
456
|
-
|
|
457
|
-
|
|
458
|
-
return Boolean(hasReturn && allBrancheshaveReturnValues(nde, promiseFilter));
|
|
459
|
-
} :
|
|
460
|
-
/**
|
|
461
|
-
* @param {ESTreeOrTypeScriptNode} nde
|
|
462
|
-
* @param {PromiseFilter} promiseFilter
|
|
463
|
-
* @returns {boolean}
|
|
464
|
-
*/
|
|
465
|
-
(nde, promiseFilter) => {
|
|
466
|
-
return Boolean(hasReturnValue(nde, false, promiseFilter));
|
|
467
|
-
};
|
|
468
|
-
return hasReturnMethod(node, prom => {
|
|
469
|
-
if (anyPromiseAsReturn) {
|
|
470
|
-
return true;
|
|
471
|
-
}
|
|
472
|
-
if (isVoidPromise(prom)) {
|
|
473
|
-
return false;
|
|
474
|
-
}
|
|
475
|
-
const {
|
|
476
|
-
body,
|
|
477
|
-
params
|
|
478
|
-
} =
|
|
479
|
-
/**
|
|
480
|
-
* @type {import('@typescript-eslint/types').TSESTree.FunctionExpression|
|
|
481
|
-
* import('@typescript-eslint/types').TSESTree.ArrowFunctionExpression}
|
|
482
|
-
*/
|
|
483
|
-
(/** @type {import('@typescript-eslint/types').TSESTree.NewExpression} */prom.arguments[0]) || {};
|
|
484
|
-
if (!params?.length) {
|
|
485
|
-
return false;
|
|
486
|
-
}
|
|
487
|
-
const {
|
|
488
|
-
name: resolverName
|
|
489
|
-
} = /** @type {import('@typescript-eslint/types').TSESTree.Identifier} */
|
|
490
|
-
params[0];
|
|
491
|
-
return hasNonEmptyResolverCall(body, resolverName);
|
|
492
|
-
});
|
|
240
|
+
const hasReturnMethod = allBranches ? (nde, promiseFilter) => {
|
|
241
|
+
let hasReturn;
|
|
242
|
+
try {
|
|
243
|
+
hasReturn = hasReturnValue(nde, true, promiseFilter);
|
|
244
|
+
} catch (error) {
|
|
245
|
+
// c8 ignore else
|
|
246
|
+
if (error.message === "Null return") return false;
|
|
247
|
+
/* c8 ignore next 3 */
|
|
248
|
+
throw error;
|
|
249
|
+
}
|
|
250
|
+
return Boolean(hasReturn && allBrancheshaveReturnValues(nde, promiseFilter));
|
|
251
|
+
} : (nde, promiseFilter) => {
|
|
252
|
+
return Boolean(hasReturnValue(nde, false, promiseFilter));
|
|
253
|
+
};
|
|
254
|
+
return hasReturnMethod(node, (prom) => {
|
|
255
|
+
if (anyPromiseAsReturn) return true;
|
|
256
|
+
if (isVoidPromise(prom)) return false;
|
|
257
|
+
const { body, params } = prom.arguments[0] || {};
|
|
258
|
+
if (!params?.length) return false;
|
|
259
|
+
const { name: resolverName } = params[0];
|
|
260
|
+
return hasNonEmptyResolverCall(body, resolverName);
|
|
261
|
+
});
|
|
493
262
|
};
|
|
263
|
+
|
|
264
|
+
//#endregion
|
|
265
|
+
exports.hasReturnValue = hasReturnValue;
|
|
494
266
|
exports.hasValueOrExecutorHasNonEmptyResolveValue = hasValueOrExecutorHasNonEmptyResolveValue;
|
|
495
267
|
//# sourceMappingURL=hasReturnValue.cjs.map
|