fix: remaining files

src/components/ShowContextMenuContext.ts

@@ -18,7 +18,7 @@ type ShowContextMenuContextProps = {
 };
 
 const ShowContextMenuContext = createContext<ShowContextMenuContextProps>({
-    anchor: undefined,
+    anchor: null,
     report: undefined,
     action: undefined,
     transactionThreadReport: undefined,

src/libs/DistanceRequestUtils.ts

@@ -43,7 +43,7 @@ const METERS_TO_MILES = 0.000621371; // There are approximately 0.000621371 mile
 function getMileageRates(policy: OnyxInputOrEntry<Policy>, includeDisabledRates = false): Record<string, MileageRate> {
     const mileageRates: Record<string, MileageRate> = {};
 
-    if (!policy?.customUnits) {
+    if (!policy || !policy?.customUnits) {
         return mileageRates;
     }
 

src/libs/PolicyUtils.ts

@@ -34,7 +34,7 @@ Onyx.connect({
  * Filter out the active policies, which will exclude policies with pending deletion
  * These are policies that we can use to create reports with in NewDot.
  */
-function getActivePolicies(policies: OnyxCollection<Policy>): Policy[] {
+function getActivePolicies(policies: OnyxCollection<Policy> | null): Policy[] {
     return Object.values(policies ?? {}).filter<Policy>(
         (policy): policy is Policy => !!policy && policy.pendingAction !== CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE && !!policy.name && !!policy.id,
     );
@@ -406,13 +406,13 @@ function getPolicy(policyID: string | undefined): Policy | EmptyObject {
 }
 
 /** Return active policies where current user is an admin */
-function getActiveAdminWorkspaces(policies: OnyxCollection<Policy>): Policy[] {
+function getActiveAdminWorkspaces(policies: OnyxCollection<Policy> | null): Policy[] {
     const activePolicies = getActivePolicies(policies);
     return activePolicies.filter((policy) => shouldShowPolicy(policy, NetworkStore.isOffline()) && isPolicyAdmin(policy));
 }
 
 /** Whether the user can send invoice */
-function canSendInvoice(policies: OnyxCollection<Policy>): boolean {
+function canSendInvoice(policies: OnyxCollection<Policy> | null): boolean {
     return getActiveAdminWorkspaces(policies).length > 0;
 }
 

src/pages/home/report/ContextMenu/PopoverReportActionContextMenu.tsx

@@ -34,7 +34,7 @@ function PopoverReportActionContextMenu(_props: unknown, ref: ForwardedRef<Repor
     const {translate} = useLocalize();
     const reportIDRef = useRef('0');
     const typeRef = useRef<ContextMenuType>();
-    const reportActionRef = useRef<OnyxEntry<ReportAction>>(undefined);
+    const reportActionRef = useRef<NonNullable<OnyxEntry<ReportAction>> | null>(null);
     const reportActionIDRef = useRef('0');
     const originalReportIDRef = useRef('0');
     const selectionRef = useRef('');
@@ -131,7 +131,7 @@ function PopoverReportActionContextMenu(_props: unknown, ref: ForwardedRef<Repor
 
     const clearActiveReportAction = () => {
         reportActionIDRef.current = '0';
-        reportActionRef.current = undefined;
+        reportActionRef.current = null;
     };
 
     /**
@@ -294,7 +294,7 @@ function PopoverReportActionContextMenu(_props: unknown, ref: ForwardedRef<Repor
         onComfirmDeleteModal.current = onConfirm;
 
         reportIDRef.current = reportID;
-        reportActionRef.current = reportAction;
+        reportActionRef.current = reportAction ?? null;
 
         setShouldSetModalVisibilityForDeleteConfirmation(shouldSetModalVisibility);
         setIsDeleteCommentConfirmModalVisible(true);

src/pages/home/report/withReportAndReportActionOrNotFound.tsx

@@ -32,7 +32,7 @@ type OnyxProps = {
     reportActions: OnyxEntry<OnyxTypes.ReportActions>;
 
     /** The report's parentReportAction */
-    parentReportAction: OnyxEntry<OnyxTypes.ReportAction>;
+    parentReportAction: NonNullable<OnyxEntry<OnyxTypes.ReportAction>> | null;
 
     /** The policies which the user has access to */
     policies: OnyxCollection<OnyxTypes.Policy>;
@@ -129,12 +129,12 @@ export default function <TProps extends WithReportAndReportActionOrNotFoundProps
             },
             parentReportAction: {
                 key: ({report}) => `${ONYXKEYS.COLLECTION.REPORT_ACTIONS}${report ? report.parentReportID : 0}`,
-                selector: (parentReportActions: OnyxEntry<OnyxTypes.ReportActions>, props?: WithOnyxState<OnyxProps>): OnyxEntry<OnyxTypes.ReportAction> => {
+                selector: (parentReportActions: OnyxEntry<OnyxTypes.ReportActions>, props?: WithOnyxState<OnyxProps>): NonNullable<OnyxEntry<OnyxTypes.ReportAction>> | null => {
                     const parentReportActionID = props?.report?.parentReportActionID;
                     if (!parentReportActionID) {
-                        return;
+                        return null;
                     }
-                    return parentReportActions?.[parentReportActionID];
+                    return parentReportActions?.[parentReportActionID] ?? null;
                 },
                 canEvict: false,
             },

src/pages/iou/request/step/IOURequestStepCategory.tsx

@@ -89,7 +89,7 @@ function IOURequestStepCategory({
     const transactionCategory = ReportUtils.getTransactionDetails(isEditingSplitBill && !lodashIsEmpty(splitDraftTransaction) ? splitDraftTransaction : transaction)?.category;
 
     // eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
-    const reportAction = reportActions?.[report?.parentReportActionID || reportActionID];
+    const reportAction = reportActions?.[report?.parentReportActionID || reportActionID] ?? null;
 
     const shouldShowCategory =
         (ReportUtils.isReportInGroupPolicy(report) || ReportUtils.isGroupPolicy(policy?.type ?? '')) &&

src/pages/iou/request/step/IOURequestStepDescription.tsx

@@ -138,7 +138,7 @@ function IOURequestStepDescription({
     };
 
     // eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing -- nullish coalescing doesn't achieve the same result in this case
-    const reportAction = reportActions?.[report?.parentReportActionID || reportActionID];
+    const reportAction = reportActions?.[report?.parentReportActionID || reportActionID] ?? null;
     const isEditing = action === CONST.IOU.ACTION.EDIT;
     const isSplitBill = iouType === CONST.IOU.TYPE.SPLIT;
     const canEditSplitBill = isSplitBill && reportAction && session?.accountID === reportAction.actorAccountID && TransactionUtils.areRequiredFieldsEmpty(transaction);

src/pages/workspace/AccessOrNotFoundWrapper.tsx

@@ -28,15 +28,18 @@ const ACCESS_VARIANTS = {
         policy: OnyxEntry<OnyxTypes.Policy>,
         login: string,
         report: OnyxEntry<OnyxTypes.Report>,
-        allPolicies: OnyxCollection<OnyxTypes.Policy>,
+        allPolicies: NonNullable<OnyxCollection<OnyxTypes.Policy>> | null,
         iouType?: IOUType,
     ) =>
         !!iouType &&
         IOUUtils.isValidMoneyRequestType(iouType) &&
         // Allow the user to submit the expense if we are submitting the expense in global menu or the report can create the expense
         (isEmptyObject(report?.reportID) || ReportUtils.canCreateRequest(report, policy, iouType)) &&
         (iouType !== CONST.IOU.TYPE.INVOICE || PolicyUtils.canSendInvoice(allPolicies)),
-} as const satisfies Record<string, (policy: OnyxTypes.Policy, login: string, report: OnyxTypes.Report, allPolicies: OnyxCollection<OnyxTypes.Policy>, iouType?: IOUType) => boolean>;
+} as const satisfies Record<
+    string,
+    (policy: OnyxTypes.Policy, login: string, report: OnyxTypes.Report, allPolicies: NonNullable<OnyxCollection<OnyxTypes.Policy>> | null, iouType?: IOUType) => boolean
+>;
 
 type AccessVariant = keyof typeof ACCESS_VARIANTS;
 type AccessOrNotFoundWrapperOnyxProps = {
@@ -122,7 +125,7 @@ function AccessOrNotFoundWrapper({accessVariants = [], fullPageNotFoundViewProps
 
     const isPageAccessible = accessVariants.reduce((acc, variant) => {
         const accessFunction = ACCESS_VARIANTS[variant];
-        return acc && accessFunction(policy, login, report, allPolicies, iouType);
+        return acc && accessFunction(policy, login, report, allPolicies ?? null, iouType);
     }, true);
 
     const isPolicyNotAccessible = isEmptyObject(policy) || (Object.keys(policy).length === 1 && !isEmptyObject(policy.errors)) || !policy?.id;