Nit: The size argument should be integer.

Is CommCoreModule the best place for this? I vaguely recall that we had been separating some stuff out into other files

I recall a conversation with @kamil where he shared similar concerns about over-populating CommCoreModule and planned to refactor the code if there iso enough time at the end of the month. The reason for not doing ot now is that it is not so straightforward since there are already some methods in CommCoreModule like getUserPublicKeys that utilize cryptoThread and are called from JS. So we either have to:

• move all methods accessing cryptoThread to a separate turbo module (CommCryptoModule?)
• Implement GlobalCryptoSingleton and split crypto methods between CommCoreModule and CommCryptoModule

I am afraid that this might not be the right way of checking for exception here. I have recently experienced it myself that exception thrown from purely Objective-C iOS code (@throw [NSException....]) is not caught within C++ try{}...catch{}. I am not sure about Java. Since generateRandomString uses native secureRandomBytes we should check that throwing an exception from Objective C and Java is detected here.

@kamil could you please check before landing if we are able to catch Objective-C or Java exception here by using try{} .. catch{}? If not then please create a follow up task and assign it to me.