fix(expo): preserve AppDelegate declaration whitespace

Author: zibs

packages/react-native-app-auth/plugin/src/__tests__/app-delegate.test.ts

@@ -55,6 +55,21 @@ describe('applyExpo53AppDelegatePatch', () => {
     );
   });
 
+  it('preserves whitespace before the class opening brace', () => {
+    const result = applyExpo53AppDelegatePatch(
+      expo56AppDelegate.replace(
+        'class AppDelegate: ExpoAppDelegate {',
+        `class AppDelegate: ExpoAppDelegate,
+  UIApplicationDelegate
+{`
+      )
+    );
+
+    expect(result).toContain(`class AppDelegate: ExpoAppDelegate,
+  UIApplicationDelegate, RNAppAuthAuthorizationFlowManager
+{`);
+  });
+
   it('does not duplicate an existing multiline AppAuth delegate property', () => {
     const appDelegateWithMultilineProperty = expo56AppDelegate.replace(
       '  var reactNativeFactory: RCTReactNativeFactory?',

packages/react-native-app-auth/plugin/src/ios/app-delegate.ts

@@ -18,12 +18,17 @@ export const applyExpo53AppDelegatePatch = (contents: string): string => {
   contents = contents.replace(
     /^(\s*(?:public\s+)?class\s+AppDelegate\s*:\s*ExpoAppDelegate)([^{]*)(\{)/m,
     (match, declaration, conformances, openingBrace) => {
-      const trimmedConformances = conformances.trim();
-      if (trimmedConformances.includes(APP_AUTH_PROTOCOL)) {
+      if (conformances.includes(APP_AUTH_PROTOCOL)) {
         return match;
       }
 
-      return `${declaration}${trimmedConformances}, ${APP_AUTH_PROTOCOL} ${openingBrace}`;
+      const trailingWhitespace = conformances.match(/\s*$/)?.[0] ?? '';
+      const existingConformances = conformances.slice(
+        0,
+        conformances.length - trailingWhitespace.length
+      );
+
+      return `${declaration}${existingConformances}, ${APP_AUTH_PROTOCOL}${trailingWhitespace}${openingBrace}`;
     }
   );