fix(eslint-plugin): [prefer-optional-chain] only look at left operand for requireNullish

packages/eslint-plugin/src/rules/prefer-optional-chain-utils/gatherLogicalOperands.ts

@@ -61,14 +61,15 @@ type Operand = ValidOperand | InvalidOperand;
 const NULLISH_FLAGS = ts.TypeFlags.Null | ts.TypeFlags.Undefined;
 function isValidFalseBooleanCheckType(
   node: TSESTree.Node,
-  operator: TSESTree.LogicalExpression['operator'],
+  operatorNode: TSESTree.LogicalExpression,
   checkType: 'true' | 'false',
   parserServices: ParserServicesWithTypeInformation,
   options: PreferOptionalChainOptions,
 ): boolean {
   const type = parserServices.getTypeAtLocation(node);
   const types = unionTypeParts(type);
 
+  const { operator } = operatorNode;
   const disallowFalseyLiteral =
     (operator === '||' && checkType === 'false') ||
     (operator === '&&' && checkType === 'true');
@@ -94,7 +95,7 @@ function isValidFalseBooleanCheckType(
     }
   }
 
-  if (options.requireNullish === true) {
+  if (options.requireNullish === true && operatorNode.left === node) {
     return types.some(t => isTypeFlagSet(t, NULLISH_FLAGS));
   }
 
@@ -258,7 +259,7 @@ export function gatherLogicalOperands(
           operand.operator === '!' &&
           isValidFalseBooleanCheckType(
             operand.argument,
-            node.operator,
+            node,
             'false',
             parserServices,
             options,
@@ -285,7 +286,7 @@ export function gatherLogicalOperands(
         if (
           isValidFalseBooleanCheckType(
             operand,
-            node.operator,
+            node,
             'true',
             parserServices,
             options,

packages/eslint-plugin/tests/rules/prefer-optional-chain/prefer-optional-chain.test.ts

@@ -1866,6 +1866,16 @@ describe('hand-crafted cases', () => {
         ],
       },
 
+      // requireNullish
+      {
+        code: `
+          declare const thing1: string | null;
+          thing1 && thing1.toString();
+        `,
+        options: [{ requireNullish: false }],
+        errors: [{ messageId: `preferOptionalChain` }],
+      },
+
       // allowPotentiallyUnsafeFixesThatModifyTheReturnTypeIKnowWhatImDoing
       {
         code: `