fix: useInvoices, usePrefetchBranchDirEntry, BundleTrendQueryOpts, usePullComponents schemas (#3809)

Author: nsdeschenes

src/pages/PullRequestPage/PullCoverage/routes/ComponentsSelector/ComponentsSelector.tsx

@@ -25,7 +25,7 @@ function ComponentsSelector() {
   })
 
   const components = useMemo(() => {
-    return data?.pull?.compareWithBase.__typename === 'Comparison'
+    return data?.pull?.compareWithBase?.__typename === 'Comparison'
       ? data?.pull?.compareWithBase?.componentComparisons
       : []
   }, [data])

src/services/account/useInvoices.ts

@@ -61,7 +61,7 @@ const InvoicesSchema = z.array(InvoiceSchema)
 const OwnerInvoiceSchema = z.object({
   owner: z
     .object({
-      invoices: InvoicesSchema,
+      invoices: InvoicesSchema.nullable(),
     })
     .nullable(),
 })

src/services/bundleAnalysis/BundleTrendDataQueryOpts.tsx

@@ -51,10 +51,12 @@ const BranchSchema = z.object({
     .object({
       bundleAnalysis: z
         .object({
-          bundleAnalysisReport: z.discriminatedUnion('__typename', [
-            BundleReportSchema,
-            MissingHeadReportSchema,
-          ]),
+          bundleAnalysisReport: z
+            .discriminatedUnion('__typename', [
+              BundleReportSchema,
+              MissingHeadReportSchema,
+            ])
+            .nullable(),
         })
         .nullable(),
     })
@@ -228,7 +230,10 @@ export const BundleTrendDataQueryOpts = ({
           data.owner?.repository?.branch?.head?.bundleAnalysis
             ?.bundleAnalysisReport
 
-        if (bundleReport?.__typename === 'BundleAnalysisReport') {
+        if (
+          bundleReport &&
+          bundleReport?.__typename === 'BundleAnalysisReport'
+        ) {
           return bundleReport?.bundle?.measurements
         }
 

src/services/pathContents/branch/dir/usePrefetchBranchDirEntry.tsx

@@ -75,13 +75,15 @@ const PathContentsUnionSchema = z.discriminatedUnion('__typename', [
 const RepositorySchema = z.object({
   __typename: z.literal('Repository'),
   repositoryConfig: RepositoryConfigSchema,
-  branch: z.object({
-    head: z
-      .object({
-        deprecatedPathContents: PathContentsUnionSchema.nullish(),
-      })
-      .nullable(),
-  }),
+  branch: z
+    .object({
+      head: z
+        .object({
+          deprecatedPathContents: PathContentsUnionSchema.nullish(),
+        })
+        .nullable(),
+    })
+    .nullable(),
 })
 
 const BranchContentsSchema = z.object({

src/services/pull/usePullComponents.tsx

@@ -69,24 +69,26 @@ const RepositorySchema = z.object({
   __typename: z.literal('Repository'),
   pull: z
     .object({
-      compareWithBase: z.discriminatedUnion('__typename', [
-        z.object({
-          __typename: z.literal('Comparison'),
-          componentComparisons: z
-            .array(
-              z.object({
-                name: z.string(),
-              })
-            )
-            .nullable(),
-        }),
-        FirstPullRequestSchema,
-        MissingBaseCommitSchema,
-        MissingBaseReportSchema,
-        MissingComparisonSchema,
-        MissingHeadCommitSchema,
-        MissingHeadReportSchema,
-      ]),
+      compareWithBase: z
+        .discriminatedUnion('__typename', [
+          z.object({
+            __typename: z.literal('Comparison'),
+            componentComparisons: z
+              .array(
+                z.object({
+                  name: z.string(),
+                })
+              )
+              .nullable(),
+          }),
+          FirstPullRequestSchema,
+          MissingBaseCommitSchema,
+          MissingBaseReportSchema,
+          MissingComparisonSchema,
+          MissingHeadCommitSchema,
+          MissingHeadReportSchema,
+        ])
+        .nullable(),
     })
     .nullable(),
 })
@@ -193,9 +195,7 @@ export function usePullComponents({
           })
         }
 
-        return {
-          pull: data?.owner?.repository?.pull,
-        }
+        return { pull: data?.owner?.repository?.pull }
       }),
     ...options,
   })