@bigbinary/neeto-commons-rn 2.0.3 → 2.2.0

package/src/screens/team_members/ProductSelect.tsx

@@ -2,7 +2,7 @@ import React, { useState, type ComponentType } from "react";
 import { Pressable, StyleSheet } from "react-native";
 
 import { findIndexBy } from "@bigbinary/neeto-cist";
-import { Down } from "@bigbinary/neeto-icons-rn";
+import Down from "@bigbinary/neeto-icons-rn/icons/Down";
 import {
   BottomSheet as UIBottomSheet,
   Container,
@@ -13,7 +13,7 @@ import {
   type BottomSheetProps,
 } from "@bigbinary/neetoui-rn";
 import { type FormikProps } from "formik";
-import { mergeRight } from "ramda";
+import mergeRight from "ramda/src/mergeRight";
 
 import Checkbox from "./Checkbox";
 import { APP_NAME } from "./constants";
@@ -32,11 +32,10 @@ interface RoleSheetItem {
   value: string;
 }
 
-// `isVisible` starts out null here, which v3 narrowed to `boolean`.
+// Pins the sheet's row type — `data` can be undefined mid-close, which
+// blocks inference at the call site.
 const BottomSheet = UIBottomSheet as ComponentType<
-  Omit<BottomSheetProps<RoleSheetItem>, "isVisible"> & {
-    isVisible: boolean | null;
-  }
+  BottomSheetProps<RoleSheetItem>
 >;
 
 interface ProductSelectProps {
@@ -52,7 +51,7 @@ const ProductSelect = ({
   setFieldValue,
 }: ProductSelectProps) => {
   const otherProducts = values?.otherProducts ?? true;
-  const [openDropdown, setOpenDropdown] = useState<boolean | null>(null);
+  const [openDropdown, setOpenDropdown] = useState(false);
   const [selectedAppName, setSelectedAppName] = useState<string | null>(null);
 
   const updateProduct = (appName: string, object: Partial<ProductRole>) => {

package/src/screens/team_members/index.tsx

@@ -1,7 +1,8 @@
 import React, { useCallback, useState } from "react";
 import { StyleSheet } from "react-native";
 
-import { Edit, Delete } from "@bigbinary/neeto-icons-rn";
+import Delete from "@bigbinary/neeto-icons-rn/icons/Delete";
+import Edit from "@bigbinary/neeto-icons-rn/icons/Edit";
 import {
   Container,
   Divider,

package/src/screens/team_members/utils.ts

@@ -1,5 +1,7 @@
 import { _findBy, notEquals } from "@bigbinary/neeto-cist";
-import { equals, includes, move } from "ramda";
+import equals from "ramda/src/equals";
+import includes from "ramda/src/includes";
+import move from "ramda/src/move";
 
 import { ADMIN_ROLE_NAME, ADMIN_ROLES, APP_NAME } from "./constants";
 

package/src/store/index.ts

@@ -1,5 +1,9 @@
 import AsyncStorage from "@react-native-async-storage/async-storage";
-import { configureStore } from "@reduxjs/toolkit";
+import {
+  configureStore,
+  type StoreEnhancer,
+  type UnknownAction,
+} from "@reduxjs/toolkit";
 import {
   persistStore,
   persistReducer,
@@ -27,6 +31,67 @@ const persistConfig: PersistConfig<CommonsState> = {
 
 const persistedReducer = persistReducer(persistConfig, rootReducer);
 
+// The enhancer must exist when configureStore runs — at module-import time —
+// so the store wires it itself; host apps could never inject it early enough.
+// The guarded require keeps hosts without the plugin working (it is a dev
+// tool the host may not install), and metro strips the whole branch from
+// release bundles because __DEV__ is inlined before dependency collection.
+const devToolsEnhancers = () => {
+  if (__DEV__) {
+    try {
+      // The require is untyped on purpose (consistent-type-imports forbids
+      // import() annotations); cast just the surface used here.
+      const { rozeniteDevToolsEnhancer } =
+        require("@rozenite/redux-devtools-plugin") as {
+          rozeniteDevToolsEnhancer: (options: {
+            name?: string;
+            maxAge?: number;
+            actionSanitizer?: (action: UnknownAction, id: number) => unknown;
+            stateSanitizer?: (state: unknown, index: number) => unknown;
+          }) => StoreEnhancer;
+        };
+
+      // The workspace switcher keeps a per-organization auth token in the
+      // store (every other credential lives in the keychain). The DevTools
+      // bridge rides metro's websocket, so those tokens must not cross it.
+      const redactOrganizationTokens = (state: unknown) => {
+        const snapshot = state as CommonsState | undefined;
+        if (!snapshot?.auth?.organizations?.length) return state;
+
+        return {
+          ...snapshot,
+          auth: {
+            ...snapshot.auth,
+            organizations: snapshot.auth.organizations.map(organization => ({
+              ...organization,
+              auth_token: "<<redacted>>",
+            })),
+          },
+        };
+      };
+
+      return [
+        rozeniteDevToolsEnhancer({
+          name: "neeto-commons-rn store",
+          // The whole persisted tree rides through REHYDRATE; big snapshots
+          // can OOM the Android DevTools panel, so cap history and elide the
+          // payload.
+          maxAge: 25,
+          actionSanitizer: action =>
+            action.type === REHYDRATE
+              ? { ...action, payload: "<<elided: persisted state>>" }
+              : action,
+          stateSanitizer: redactOrganizationTokens,
+        }),
+      ];
+    } catch {
+      return [];
+    }
+  }
+
+  return [];
+};
+
 const baseStore = configureStore({
   reducer: persistedReducer,
   middleware: getDefaultMiddleware =>
@@ -35,6 +100,8 @@ const baseStore = configureStore({
         ignoredActions: [FLUSH, REHYDRATE, PAUSE, PERSIST, PURGE, REGISTER],
       },
     }),
+  enhancers: getDefaultEnhancers =>
+    getDefaultEnhancers().concat(devToolsEnhancers()),
 });
 const Persistor = persistStore(baseStore);
 

package/src/utils/apiHelpers.ts

@@ -1,7 +1,10 @@
+// `appName` doubles as user-facing copy ("Update neetoDesk"), so it keeps its
+// camel case. Hosts are case-insensitive, but the mixed-case spelling shows up
+// verbatim in request logs and Bugsnag error groups, so lowercase it here.
 export const appVersionsUrl = ({
   appName,
   environment,
 }: {
   appName: string;
   environment: string;
-}) => `https://api.${appName}.${environment}/app_versions`;
+}) => `https://api.${appName.toLowerCase()}.${environment}/app_versions`;