chore: disable prefer-optional-chain ESLint rule

The rule's --fix introduces logic bugs by converting falsy guards
to optional chains, removing protection against empty strings and
nullish values. Workarounds (eslint-disable comments, magic-string
hacks) are worse than the original code. Disabling the rule and
removing all related eslint-disable comments.

Author: dariusz-biela

eslint.config.mjs

@@ -446,6 +446,7 @@ const config = defineConfig([
             ],
             '@typescript-eslint/prefer-find': 'off',
             '@typescript-eslint/prefer-includes': 'off',
+            '@typescript-eslint/prefer-optional-chain': 'off',
             '@typescript-eslint/prefer-nullish-coalescing': [
                 'error',
                 {

src/components/Modal/ReanimatedModal/Container/GestureHandler.tsx

@@ -16,7 +16,6 @@ function hasSwipeEnded(
 ) {
     'worklet';
 
-    // eslint-disable-next-line @typescript-eslint/prefer-optional-chain
     if (!swipeDirection || !swipeDirection?.length || !onSwipeComplete) {
         return;
     }
@@ -66,7 +65,6 @@ function GestureHandler({swipeDirection, onSwipeComplete, swipeThreshold = 100,
         [initialTranslationX, initialTranslationY, onSwipeComplete, swipeDirection, swipeThreshold],
     );
 
-    // eslint-disable-next-line @typescript-eslint/prefer-optional-chain
     if (!swipeDirection || !swipeDirection?.length || !onSwipeComplete) {
         return children;
     }

src/libs/PolicyUtils.ts

@@ -1299,8 +1299,6 @@ function hasIndependentTags(policy: OnyxEntry<Policy>, policyTagList: OnyxEntry<
 
 /** Get the Xero organizations connected to the policy */
 function getXeroTenants(policy: Policy | undefined): Tenant[] {
-    // Due to the way optional chain is being handled in this useMemo we are forced to use this approach to properly handle undefined values
-    // eslint-disable-next-line @typescript-eslint/prefer-optional-chain
     if (!policy || !policy.connections || !policy.connections.xero || !policy.connections.xero.data) {
         return [];
     }

src/pages/OnboardingWorkEmail/BaseOnboardingWorkEmail.tsx

@@ -112,7 +112,6 @@ function BaseOnboardingWorkEmail({shouldUseNativeStyles}: BaseOnboardingWorkEmai
         const emailParts = userEmail.split('@');
         const domain = emailParts.at(1) ?? '';
 
-        // eslint-disable-next-line @typescript-eslint/prefer-optional-chain -- optional chain removes the falsy guard for empty strings, causing a false positive when both emails are empty
         if (session?.email && userEmail.toLowerCase() === session.email.toLowerCase() && !isOffline) {
             addErrorMessage(errors, INPUT_IDS.ONBOARDING_WORK_EMAIL, translate('onboarding.workEmailValidationError.sameAsSignupEmail'));
         } else if ((!Str.isValidEmail(userEmail) || PUBLIC_DOMAINS_SET.has(domain.toLowerCase())) && !isOffline) {

src/pages/workspace/companyCards/WorkspaceCompanyCardAddWorkEmailPage.tsx

@@ -95,7 +95,6 @@ function WorkspaceCompanyCardAddWorkEmailPage({route}: WorkspaceCompanyCardAddWo
     };
 
     useEffect(() => {
-        // eslint-disable-next-line @typescript-eslint/prefer-optional-chain -- optional chain changes logic: when both email and primaryContactMethod are falsy, it skips early return and navigates incorrectly
         if (!email || !primaryContactMethod || primaryContactMethod.toLowerCase() !== email.toLowerCase() || isWorkEmailValidated) {
             return;
         }

src/pages/workspace/expensifyCard/WorkspaceExpensifyCardAddWorkEmailPage.tsx

@@ -86,7 +86,6 @@ function WorkspaceExpensifyCardAddWorkEmailPage({route}: WorkspaceExpensifyCardA
     };
 
     useEffect(() => {
-        // eslint-disable-next-line @typescript-eslint/prefer-optional-chain -- optional chain changes logic: when both email and primaryContactMethod are falsy, it skips early return and navigates incorrectly
         if (!email || !primaryContactMethod || primaryContactMethod.toLowerCase() !== email.toLowerCase() || isWorkEmailValidated) {
             return;
         }