fix(interactions): use pako instead of fflate in RN

[ashwinkumar6]

Description of changes

fflat lib is used in interactions to gzip data.

This lib uses .cjs files which would throw an error during build in RN apps. RN apps would need to resolve this by adding the following in metro config file

resolver: {
  sourceExts: [...sourceExts, "cjs"]
}

To avoid this, use an alternative lib pako instead of fflate in RN. For web continue using fflate since it's bundle size is much smaller than pako

Description of how you validated changes

Checklist

• PR description included
• yarn test passes
• Tests are changed or added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[elorzafe]

Looks good to me!

Thanks @ashwinkumar6 🎉

[Samaritan1011001]

Nice! Thank you! 🎉

[cshfang]

Still have a minor question around the typing of InteractionsMessage. I don't think it's blocking but some comments describing why the guard is needed could be helpful as it's not intuitive from looking at typing alone.

[cshfang]

Is this check in place because, at runtime, it is possible for the message to not be string | InteractionsMessage? From looking at typing, it does not seem possible for a message with messageType of voice to have a content that isn't object.

[cshfang]

Chatted offline about this. The issue is in the typing as I understand it. We should probably update the typing to be able to infer voice or text message (and therefore the content type) but we can do so in a separate PR

[ashwinkumar6]

As discussed it's inferring to union type because of the way the type is defined.
So a guard clause is required

export type InteractionsTextMessage = {
	content: string;
	options: {
		messageType: 'text';
	};
};

export type InteractionsVoiceMessage = {
	content: object;
	options: {
		messageType: 'voice';
	};
};

export type InteractionsMessage =
	| InteractionsTextMessage
	| InteractionsVoiceMessage;

[codecov-commenter]

Codecov Report

Merging #10278 (b8e1a95) into main (840086d) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main   #10278      +/-   ##
==========================================
+ Coverage   84.40%   84.42%   +0.01%     
==========================================
  Files         258      258              
  Lines       18691    18694       +3     
  Branches     4017     4019       +2     
==========================================
+ Hits        15777    15782       +5     
+ Misses       2824     2822       -2     
  Partials       90       90              

Impacted Files	Coverage Δ
...kages/interactions/src/Providers/AWSLexProvider.ts	96.15% <100.00%> (+1.41%)	⬆️
.../src/Providers/AWSLexProviderHelper/commonUtils.ts	85.71% <100.00%> (-3.18%)	⬇️
...ctions/src/Providers/AWSLexProviderHelper/utils.ts	92.85% <100.00%> (+0.54%)	⬆️
...ges/interactions/src/Providers/AWSLexV2Provider.ts	96.96% <100.00%> (+1.09%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more