Address PR review feedback

Author: mattinannt

package.json

@@ -18,7 +18,7 @@
   },
   "packageManager": "pnpm@10.32.1",
   "engines": {
-    "node": ">=20"
+    "node": ">=20.19.0"
   },
   "pnpm": {
     "overrides": {

packages/react-native/src/index.ts

@@ -39,3 +39,5 @@ export const logout = async (): Promise<void> => {
 };
 
 export { Formbricks } from "@/components/formbricks";
+// eslint-disable-next-line import/no-default-export -- preserve the public SDK default export
+export { Formbricks as default } from "@/components/formbricks";

packages/react-native/src/lib/common/setup.ts

@@ -37,31 +37,6 @@ export const setIsSetup = (state: boolean): void => {
   isSetup = state;
 };
 
-export const migrateUserStateAddContactId = async (): Promise<{
-  changed: boolean;
-}> => {
-  const existingConfigString = await AsyncStorage.getItem(RN_ASYNC_STORAGE_KEY);
-
-  if (existingConfigString) {
-    const existingConfig = JSON.parse(existingConfigString) as Partial<TConfig>;
-
-    // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition -- data could be undefined
-    if (existingConfig.user?.data?.contactId) {
-      return { changed: false };
-    }
-
-     
-    if (
-      !existingConfig.user?.data.contactId &&
-      existingConfig.user?.data.userId
-    ) {
-      return { changed: true };
-    }
-  }
-
-  return { changed: false };
-};
-
 // Helper: Handle missing field error
 function handleMissingField(
   field: string
@@ -294,15 +269,9 @@ export const setup = async (
 ): Promise<
   Result<void, MissingFieldError | NetworkError | MissingPersonError>
 > => {
-  let appConfig = await RNConfig.getInstance();
+  const appConfig = await RNConfig.getInstance();
 
   const logger = Logger.getInstance();
-  const { changed } = await migrateUserStateAddContactId();
-
-  if (changed) {
-    await appConfig.resetConfig();
-    appConfig = await RNConfig.getInstance();
-  }
 
   if (isSetup) {
     logger.debug("Already set up, skipping setup.");

packages/react-native/src/lib/common/tests/setup.test.ts

@@ -274,61 +274,6 @@ describe("setup.ts", () => {
       );
     });
 
-    test("reloads config after migrating legacy user state", async () => {
-      const initialConfig = {
-        resetConfig: vi.fn(),
-      };
-      const migratedConfig = {
-        get: vi.fn().mockReturnValue({
-          environmentId: "env_123",
-          appUrl: "https://my.url",
-          environment: {
-            data: { surveys: [] },
-            expiresAt: new Date(Date.now() + 60_000),
-          },
-          user: {
-            data: {
-              userId: "user_abc",
-              contactId: null,
-              segments: [],
-              displays: [],
-              responses: [],
-              lastDisplayAt: null,
-            },
-            expiresAt: null,
-          },
-          filteredSurveys: [],
-          status: { value: "success", expiresAt: null },
-        }),
-        update: vi.fn(),
-      };
-
-      (AsyncStorage.getItem as Mock).mockResolvedValueOnce(
-        JSON.stringify({
-          user: {
-            data: {
-              userId: "user_abc",
-              contactId: null,
-            },
-          },
-        })
-      );
-      getInstanceConfigMock
-        .mockReturnValueOnce(initialConfig as unknown as Promise<RNConfig>)
-        .mockReturnValueOnce(migratedConfig as unknown as Promise<RNConfig>);
-      (filterSurveys as unknown as Mock).mockReturnValueOnce([]);
-
-      const result = await setup({
-        environmentId: "env_123",
-        appUrl: "https://my.url",
-      });
-
-      expect(result.ok).toBe(true);
-      expect(initialConfig.resetConfig).toHaveBeenCalledTimes(1);
-      expect(getInstanceConfigMock).toHaveBeenCalledTimes(2);
-      expect(migratedConfig.update).toHaveBeenCalled();
-    });
-
     test("returns an error when environment sync fails", async () => {
       const mockConfig = {
         get: vi.fn().mockReturnValue({

scripts/check-exact-deps.mjs

@@ -23,7 +23,16 @@ collectPackageJsonPaths(rootDir, manifestPaths);
 
 const violations = [];
 for (const manifestPath of manifestPaths) {
-  const manifest = JSON.parse(fs.readFileSync(manifestPath, "utf8"));
+  let manifest;
+  try {
+    manifest = JSON.parse(fs.readFileSync(manifestPath, "utf8"));
+  } catch (error) {
+    const message = error instanceof Error ? error.message : String(error);
+    console.error(
+      `Failed to parse ${relativePath(manifestPath)}: ${message}`,
+    );
+    process.exit(1);
+  }
 
   for (const field of dependencyFields) {
     const dependencies = manifest[field] ?? {};
@@ -54,7 +63,11 @@ if (violations.length > 0) {
 }
 
 function collectPackageJsonPaths(dirPath, results) {
-  for (const entry of fs.readdirSync(dirPath, { withFileTypes: true })) {
+  const entries = fs
+    .readdirSync(dirPath, { withFileTypes: true })
+    .sort((a, b) => a.name.localeCompare(b.name));
+
+  for (const entry of entries) {
     if (ignoredDirs.has(entry.name)) {
       continue;
     }