no-unnecessary-await: Fix handling of experimental pipeline operator (#2658)

Co-authored-by: fisker <[email protected]>

Author: dword-design

rules/no-unnecessary-await.js

@@ -38,7 +38,11 @@ function notPromise(node) {
 /** @param {import('eslint').Rule.RuleContext} context */
 const create = context => ({
 	AwaitExpression(node) {
-		if (!notPromise(node.argument)) {
+		if (
+			// F#-style pipeline operator, `Promise.resolve() |> await`
+			!node.argument
+			|| !notPromise(node.argument)
+		) {
 			return;
 		}
 

test/no-unnecessary-await.js

@@ -113,3 +113,33 @@ test.snapshot({
 		'async function foo() {+await -1}',
 	],
 });
+
+test.babel({
+	valid: [
+		{
+			code: 'Promise.resolve() |> await',
+			languageOptions: {
+				parserOptions: {
+					babelOptions: {
+						parserOpts: {
+							plugins: [['pipelineOperator', {proposal: 'fsharp'}]],
+						},
+					},
+				},
+			},
+		},
+		{
+			code: 'Promise.resolve() |> await %',
+			languageOptions: {
+				parserOptions: {
+					babelOptions: {
+						parserOpts: {
+							plugins: [['pipelineOperator', {proposal: 'hack', topicToken: '%'}]],
+						},
+					},
+				},
+			},
+		},
+	],
+	invalid: [],
+});