Merge pull request #16831 from backstage/freben/client-nulls

catalog-client: getEntitiesByRefs should return undefined

.changeset/tidy-ties-guess.md

@@ -0,0 +1,5 @@
+---
+'@backstage/catalog-client': patch
+---
+
+Ensure that `getEntitiesByRefs` returns `undefined` instead of `null` for missing items

packages/catalog-client/src/CatalogClient.test.ts

@@ -249,7 +249,7 @@ describe('CatalogClient', () => {
         { token },
       );
 
-      expect(response).toEqual({ items: [entity, null] });
+      expect(response).toEqual({ items: [entity, undefined] });
     });
   });
 

packages/catalog-client/src/CatalogClient.ts

@@ -199,9 +199,11 @@ export class CatalogClient implements CatalogApi {
       throw await ResponseError.fromResponse(response);
     }
 
-    const { items } = await response.json();
+    const { items } = (await response.json()) as {
+      items: Array<Entity | null>;
+    };
 
-    return { items };
+    return { items: items.map(i => i ?? undefined) };
   }
 
   /**