@availity/mui-spaces 1.0.30 → 1.0.31

package/CHANGELOG.md

@@ -2,6 +2,8 @@
 
 This file was generated using [@jscutlery/semver](https://github.com/jscutlery/semver).
 
+## [1.0.31](https://github.com/Availity/element/compare/@availity/mui-spaces@1.0.30...@availity/mui-spaces@1.0.31) (2025-09-22)
+
 ## [1.0.30](https://github.com/Availity/element/compare/@availity/mui-spaces@1.0.29...@availity/mui-spaces@1.0.30) (2025-08-21)
 
 

package/dist/index.js

@@ -616,14 +616,14 @@ var Spaces = ({
   }
   if (spaceIds && spaceIds.length > 0) {
     for (const id of spaceIds) {
-      if (!(spacesMap.has(id) || configIdsMap.has(id))) {
+      if (id && typeof id === "string" && id.trim() && !(spacesMap.has(id) || configIdsMap.has(id))) {
         spaceIdsToQuery.add(id);
       }
     }
   }
   if (payerIds && payerIds.length > 0) {
     for (const pid of payerIds) {
-      if (!payerIdsMap.has(pid)) {
+      if (pid && typeof pid === "string" && pid.trim() && !payerIdsMap.has(pid)) {
         payerIdsToQuery.add(pid);
       }
     }

package/dist/index.mjs

@@ -574,14 +574,14 @@ var Spaces = ({
   }
   if (spaceIds && spaceIds.length > 0) {
     for (const id of spaceIds) {
-      if (!(spacesMap.has(id) || configIdsMap.has(id))) {
+      if (id && typeof id === "string" && id.trim() && !(spacesMap.has(id) || configIdsMap.has(id))) {
         spaceIdsToQuery.add(id);
       }
     }
   }
   if (payerIds && payerIds.length > 0) {
     for (const pid of payerIds) {
-      if (!payerIdsMap.has(pid)) {
+      if (pid && typeof pid === "string" && pid.trim() && !payerIdsMap.has(pid)) {
         payerIdsToQuery.add(pid);
       }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@availity/mui-spaces",
-  "version": "1.0.30",
+  "version": "1.0.31",
   "description": "Availity MUI Spaces Component - part of the @availity/element design system",
   "keywords": [
     "react",

package/src/lib/Spaces.test.tsx

@@ -3,6 +3,7 @@ import { useState } from 'react';
 import { render, waitFor, fireEvent } from '@testing-library/react';
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query';
 import { Spaces, useSpaces, useSpacesContext } from '..';
+import { fetchAllSpaces } from './spaces-data';
 
 // eslint-disable-next-line @nx/enforce-module-boundaries
 import { server } from '../../../mock/src/lib/server';
@@ -258,3 +259,95 @@ it('returns first payer space with when no spaceId passed', async () => {
   expect(spc1).toBeDefined();
   expect(spc2).toBeDefined();
 });
+describe('ID validation', () => {
+  let mockFetchAllSpaces: jest.SpyInstance;
+
+  beforeEach(() => {
+    // eslint-disable-next-line @typescript-eslint/no-var-requires
+    mockFetchAllSpaces = jest.spyOn(require('./spaces-data'), 'fetchAllSpaces').mockResolvedValue([]);
+  });
+
+  afterEach(() => {
+    mockFetchAllSpaces.mockRestore();
+  });
+
+  it('filters invalid spaceIds before querying', async () => {
+    mockFetchAllSpaces.mockResolvedValue([]);
+    const queryClient = new QueryClient();
+
+    render(
+      <QueryClientProvider client={queryClient}>
+        <Spaces spaceIds={['valid-id', null, undefined, '', '  ', 123 as any] as any}>
+          <div>test</div>
+        </Spaces>
+      </QueryClientProvider>
+    );
+
+    await waitFor(() => {
+      expect(mockFetchAllSpaces).toHaveBeenCalledWith(
+        expect.objectContaining({
+          variables: expect.objectContaining({
+            ids: ['valid-id'],
+          }),
+        })
+      );
+    });
+  });
+
+  it('avoids querying if no valid spaceIds exist', async () => {
+    mockFetchAllSpaces.mockResolvedValue([]);
+    const queryClient = new QueryClient();
+
+    render(
+      <QueryClientProvider client={queryClient}>
+        <Spaces spaceIds={[null, undefined, '', '  ', 123 as any] as any}>
+          <div>test</div>
+        </Spaces>
+      </QueryClientProvider>
+    );
+
+    await waitFor(() => {
+      expect(mockFetchAllSpaces).not.toHaveBeenCalled();
+    });
+  });
+
+  it('filters invalid payerIds before querying', async () => {
+    mockFetchAllSpaces.mockResolvedValue([]);
+    const queryClient = new QueryClient();
+
+    render(
+      <QueryClientProvider client={queryClient}>
+        <Spaces payerIds={['valid-payer', null, undefined, '', '  ', 456 as any] as any}>
+          <div>test</div>
+        </Spaces>
+      </QueryClientProvider>
+    );
+
+    await waitFor(() => {
+      expect(mockFetchAllSpaces).toHaveBeenCalledWith(
+        expect.objectContaining({
+          variables: expect.objectContaining({
+            payerIDs: ['valid-payer'],
+          }),
+        })
+      );
+    });
+  });
+
+  it('avoids querying if no valid payerIds exist', async () => {
+    mockFetchAllSpaces.mockResolvedValue([]);
+    const queryClient = new QueryClient();
+
+    render(
+      <QueryClientProvider client={queryClient}>
+        <Spaces payerIds={[null, undefined, '', '  ', 456 as any] as any}>
+          <div>test</div>
+        </Spaces>
+      </QueryClientProvider>
+    );
+
+    await waitFor(() => {
+      expect(mockFetchAllSpaces).not.toHaveBeenCalled();
+    });
+  });
+});

package/src/lib/Spaces.tsx

@@ -57,7 +57,7 @@ export const Spaces = ({
 
   if (spaceIds && spaceIds.length > 0) {
     for (const id of spaceIds) {
-      if (!(spacesMap.has(id) || configIdsMap.has(id))) {
+      if (id && typeof id === 'string' && id.trim() && !(spacesMap.has(id) || configIdsMap.has(id))) {
         spaceIdsToQuery.add(id);
       }
     }
@@ -65,7 +65,7 @@ export const Spaces = ({
 
   if (payerIds && payerIds.length > 0) {
     for (const pid of payerIds) {
-      if (!payerIdsMap.has(pid)) {
+      if (pid && typeof pid === 'string' && pid.trim() && !payerIdsMap.has(pid)) {
         payerIdsToQuery.add(pid);
       }
     }