Support for React Native new architecture and bridgeless about amplify-js

[cshfang]

@brentvatne - thanks for bringing this up. I have a couple of a clarifying questions:

1. Is the ask to validate that amazon-cognito-identity-js works as-is with RN 0.74? I.e. there is not an ask here to update the package itself and
2. Is this at all related to Expo Go?

[cshfang]

Gotcha. Will follow up with the team tomorrow morning and figure out a test plan.

Thanks!

[cwomack]

@brentvatne, we're marking this as a feature request and are reviewing this internally as @cshfang stated above. We'll let you know if there's any further questions. Thanks!

[brentvatne]

hi @cshfang! thanks for the quick response.

1. Is the ask to validate that amazon-cognito-identity-js works as-is with RN 0.74? I.e. there is not an ask here to update the package itself

correct! specifically, that it works with the latest [email protected] with the new architecture, with bridgeless enabled and without it enabled, using the information in this doc as guidance: https://gist.github.com/cipolleschi/82b7a9561b8861330efabbd3eb08c6f5

I expect that it will just work because 0.74-rc enables an interop layer by default for new arch, and this module is fairly simple, but it would be good to verify!

2. Is this at all related to Expo Go?

nope!

[cshfang]

@brentvatne

Sorry for not getting back to you sooner. Obviously, at this point, RN 0.74 has been out for weeks 🤦

We did actually run through most of the use cases for amazon-cognito-identity-js against RN 0.74 with and without bridgeless mode enabled on the new an old architecture and were able to confirm that everything worked as expected (as you suspected). We'll keep this ticket open for now to see if we need more in-depth testing but I appreciate you helping us to try to get ahead of this ❤️