No exports for createLogger and SovranStorage

[stereodenis]

Hi! 👋

Firstly, thanks for your work on this project! 🙂

Today I used patch-package to patch @segment/analytics-react-native@2.20.3 for the project I'm working on.

Here is the diff that solved my problem:

diff --git a/node_modules/@segment/analytics-react-native/src/index.ts b/node_modules/@segment/analytics-react-native/src/index.ts
index 8263f16..439dde9 100644
--- a/node_modules/@segment/analytics-react-native/src/index.ts
+++ b/node_modules/@segment/analytics-react-native/src/index.ts
@@ -2,6 +2,8 @@ export { defaultConfig } from './constants';
 export * from './client';
 export * from './plugin';
 export * from './types';
+export * from './logger';
+export * from './storage';
 export * from './mapTransform';
 export {
   getNativeModule,

This issue body was partially generated by patch-package.

[sunitaprajapati89]

@stereodenis : Thanks for reaching out to us !!

Would you like to tell us the use case for which you need these exports?

[stereodenis]

@sunitaprajapati89
I'm using SovranStrage and createLogger:

export const createSegmentClient = (config: Config) => {
  const logger = config?.logger ?? createLogger();
  if (typeof config?.debug === "boolean") {
    if (config.debug) {
      logger.enable();
    } else {
      logger.disable();
    }
  }
  const clientConfig = { ...defaultConfig, ...config };

  segmentStore = new SovranStorage({
    storeId: config.writeKey,
    storePersistor: config.storePersistor,
  });

  // We don't await the client to be initialized to let the user start attaching plugins and queueing events
  // The client will handle initialization in the background and send events when it is ready
  // To subscribe to the client being fully initialized use client.isReady.onChange()
  //void client.init();

  return new SegmentClientOverride({
    config: clientConfig,
    logger,
    store: segmentStore,
  });
};

[sunitaprajapati89]

@stereodenis : Thanks !! I am interesting in knowing, why you want to patch @segment/analytics-react-native@2.20.3? What's the use case that is missing in our package?

[stereodenis]

@stereodenis I already answered above. We don't have enough exports

[sunitaprajapati89]

@stereodenis : Please provide more context of your use case. Not providing exports for createLogger and SovranStorage is a design decision. We can't simply expose them without understanding the proper use case. Feel free to reopen this issue if required.