Revert #2799 (#2826)

Author: fisker

rules/no-useless-undefined.js

@@ -216,31 +216,6 @@ const create = context => {
 		}
 	});
 
-	// `foo ?? undefined`
-	context.on('Identifier', node => {
-		if (
-			isUndefined(node)
-			&& node.parent.type === 'LogicalExpression'
-			&& node.parent.operator === '??'
-			&& node.parent.right === node
-		) {
-			return getProblem(
-				node,
-				fixer => {
-					const logicalExpression = node.parent;
-					const operatorToken = sourceCode.getTokenBefore(node, token => token.value === logicalExpression.operator);
-					const tokenBeforeOperatorToken = sourceCode.getTokenBefore(operatorToken);
-
-					return fixer.removeRange([
-						sourceCode.getRange(tokenBeforeOperatorToken)[1],
-						sourceCode.getRange(logicalExpression)[1],
-					]);
-				},
-				/* CheckFunctionReturnType */ false,
-			);
-		}
-	});
-
 	if (!options.checkArguments) {
 		return;
 	}

test/no-useless-undefined.js

@@ -497,19 +497,3 @@ test.snapshot({
 		},
 	],
 });
-
-// `foo ?? undefined`
-test.snapshot({
-	valid: [
-		'foo || undefined',
-		'foo && undefined',
-		'foo ?? null',
-		'undefined ?? foo',
-	],
-	invalid: [
-		'foo ?? undefined',
-		'(( (( foo )) ?? (( undefined )) ))',
-		'foo ?? null ?? undefined',
-		'call(foo ?? undefined,)',
-	],
-});

test/snapshots/no-useless-undefined.js.md

@@ -697,79 +697,3 @@ Generated by [AVA](https://avajs.dev).
     Output:␊
       1 | function a({foo}?) {}␊
     `
-
-## invalid(1): foo ?? undefined
-
-> Input
-
-    `␊
-      1 | foo ?? undefined␊
-    `
-
-> Error 1/1
-
-    `␊
-    Message:␊
-    > 1 | foo ?? undefined␊
-        |        ^^^^^^^^^ Do not use useless \`undefined\`.␊
-    ␊
-    Output:␊
-      1 | foo␊
-    `
-
-## invalid(2): (( (( foo )) ?? (( undefined )) ))
-
-> Input
-
-    `␊
-      1 | (( (( foo )) ?? (( undefined )) ))␊
-    `
-
-> Error 1/1
-
-    `␊
-    Message:␊
-    > 1 | (( (( foo )) ?? (( undefined )) ))␊
-        |                    ^^^^^^^^^ Do not use useless \`undefined\`.␊
-    ␊
-    Output:␊
-      1 | (( (( foo )) ))␊
-    `
-
-## invalid(3): foo ?? null ?? undefined
-
-> Input
-
-    `␊
-      1 | foo ?? null ?? undefined␊
-    `
-
-> Error 1/1
-
-    `␊
-    Message:␊
-    > 1 | foo ?? null ?? undefined␊
-        |                ^^^^^^^^^ Do not use useless \`undefined\`.␊
-    ␊
-    Output:␊
-      1 | foo ?? null␊
-    `
-
-## invalid(4): call(foo ?? undefined,)
-
-> Input
-
-    `␊
-      1 | call(foo ?? undefined,)␊
-    `
-
-> Error 1/1
-
-    `␊
-    Message:␊
-    > 1 | call(foo ?? undefined,)␊
-        |             ^^^^^^^^^ Do not use useless \`undefined\`.␊
-    ␊
-    Output:␊
-      1 | call(foo,)␊
-    `