@availity/mui-spaces 1.0.29 → 1.0.31

package/CHANGELOG.md

@@ -2,6 +2,15 @@
 
 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)
+
+
+### Bug Fixes
+
+* spaces link full url only if starts with web ([5db57d4](https://github.com/Availity/element/commit/5db57d4c0e58f1b1c98b95186b6600ebb9d44493))
+
 ## [1.0.29](https://github.com/Availity/element/compare/@availity/mui-spaces@1.0.28...@availity/mui-spaces@1.0.29) (2025-07-02)
 
 

package/dist/index.js

@@ -118,7 +118,7 @@ var spacesReducer = (state, action) => {
   const { type } = action;
   return actions[type](state, action);
 };
-var fetchSpaces = (_0) => __async(void 0, [_0], function* ({ query, clientId, variables }) {
+var fetchSpaces = (_0) => __async(null, [_0], function* ({ query, clientId, variables }) {
   const headers = {};
   if (clientId) {
     headers["X-Client-ID"] = clientId;
@@ -144,7 +144,7 @@ var fetchSpaces = (_0) => __async(void 0, [_0], function* ({ query, clientId, va
     hasNextPage
   };
 });
-var fetchAllSpaces = (_0) => __async(void 0, [_0], function* ({
+var fetchAllSpaces = (_0) => __async(null, [_0], function* ({
   query,
   clientId,
   variables,
@@ -330,7 +330,7 @@ var getLocalStorageTopApps = (akaname) => {
   const topAppsValues = getItemLocalStorage(`${TOP_APPS.KEYS.VALUES}-${akaname}`);
   return topAppsValues;
 };
-var updateTopApps = (space, akaname) => __async(void 0, null, function* () {
+var updateTopApps = (space, akaname) => __async(null, null, function* () {
   if (!space.configurationId || !space.type) return;
   if (canTrackSpace(space.configurationId, space.type)) {
     const today = (0, import_dayjs.default)();
@@ -363,7 +363,7 @@ var disclaimerQuery = `query disclaimerFindOne($id: ID!) {
 var DisclaimerModal = ({ disclaimerId }) => {
   const [disclaimer, setDisclaimer] = (0, import_react.useState)("");
   (0, import_react.useEffect)(() => {
-    const fetchDisclaimer = () => __async(void 0, null, function* () {
+    const fetchDisclaimer = () => __async(null, null, function* () {
       if (disclaimerId) {
         const result = yield import_api_axios2.avWebQLApi.create({ query: disclaimerQuery, variables: { id: disclaimerId } });
         setDisclaimer(result.data.data.configurationFindOne.description);
@@ -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);
       }
     }
@@ -780,7 +780,7 @@ var import_hooks = require("@availity/hooks");
 // src/lib/SpacesLink/linkHandlers.tsx
 var import_native_form = __toESM(require("@availity/native-form"));
 var import_resolve_url2 = require("@availity/resolve-url");
-var openLink = (space, params) => __async(void 0, null, function* () {
+var openLink = (space, params) => __async(null, null, function* () {
   var _a, _b, _c;
   if (!((_a = space == null ? void 0 : space.link) == null ? void 0 : _a.url)) {
     return;
@@ -791,14 +791,14 @@ var openLink = (space, params) => __async(void 0, null, function* () {
   const needsSpaceId = !(0, import_resolve_url2.isAbsoluteUrl)(space.link.url) || essentialsDomainRegex.test(space.link.url);
   const urlWithParams = needsSpaceId ? updateUrl(space.link.url, "spaceId", spaceId) : space.link.url;
   let url = !(0, import_resolve_url2.isAbsoluteUrl)(space.link.url) ? getUrl(urlWithParams, false, false) : urlWithParams;
-  if (!(0, import_resolve_url2.isAbsoluteUrl)(url) && !essentialsDomainRegex.test(document.referrer) && /\/web\/|\/public\/(apps|spaces)/.test(url)) {
+  if (!(0, import_resolve_url2.isAbsoluteUrl)(url) && !essentialsDomainRegex.test(document.referrer) && /^\/web\/|\/public\/(apps|spaces)/.test(url)) {
     const referrer = new URL(document.referrer);
     url = `${referrer.origin}${url}`;
   }
   const target = getTarget(space.link.target);
   window.open(url, target);
 });
-var openLinkWithSso = (_0, _1) => __async(void 0, [_0, _1], function* (space, { akaname, clientId, payerSpaceId, ssoParams }) {
+var openLinkWithSso = (_0, _1) => __async(null, [_0, _1], function* (space, { akaname, clientId, payerSpaceId, ssoParams }) {
   var _a, _b;
   if (space.meta && space.meta.ssoId) {
     let options;

package/dist/index.mjs

@@ -76,7 +76,7 @@ var spacesReducer = (state, action) => {
   const { type } = action;
   return actions[type](state, action);
 };
-var fetchSpaces = (_0) => __async(void 0, [_0], function* ({ query, clientId, variables }) {
+var fetchSpaces = (_0) => __async(null, [_0], function* ({ query, clientId, variables }) {
   const headers = {};
   if (clientId) {
     headers["X-Client-ID"] = clientId;
@@ -102,7 +102,7 @@ var fetchSpaces = (_0) => __async(void 0, [_0], function* ({ query, clientId, va
     hasNextPage
   };
 });
-var fetchAllSpaces = (_0) => __async(void 0, [_0], function* ({
+var fetchAllSpaces = (_0) => __async(null, [_0], function* ({
   query,
   clientId,
   variables,
@@ -288,7 +288,7 @@ var getLocalStorageTopApps = (akaname) => {
   const topAppsValues = getItemLocalStorage(`${TOP_APPS.KEYS.VALUES}-${akaname}`);
   return topAppsValues;
 };
-var updateTopApps = (space, akaname) => __async(void 0, null, function* () {
+var updateTopApps = (space, akaname) => __async(null, null, function* () {
   if (!space.configurationId || !space.type) return;
   if (canTrackSpace(space.configurationId, space.type)) {
     const today = dayjs();
@@ -321,7 +321,7 @@ var disclaimerQuery = `query disclaimerFindOne($id: ID!) {
 var DisclaimerModal = ({ disclaimerId }) => {
   const [disclaimer, setDisclaimer] = useState("");
   useEffect(() => {
-    const fetchDisclaimer = () => __async(void 0, null, function* () {
+    const fetchDisclaimer = () => __async(null, null, function* () {
       if (disclaimerId) {
         const result = yield avWebQLApi2.create({ query: disclaimerQuery, variables: { id: disclaimerId } });
         setDisclaimer(result.data.data.configurationFindOne.description);
@@ -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);
       }
     }
@@ -738,7 +738,7 @@ import { useCurrentUser } from "@availity/hooks";
 // src/lib/SpacesLink/linkHandlers.tsx
 import nativeForm from "@availity/native-form";
 import { isAbsoluteUrl as isAbsoluteUrl2 } from "@availity/resolve-url";
-var openLink = (space, params) => __async(void 0, null, function* () {
+var openLink = (space, params) => __async(null, null, function* () {
   var _a, _b, _c;
   if (!((_a = space == null ? void 0 : space.link) == null ? void 0 : _a.url)) {
     return;
@@ -749,14 +749,14 @@ var openLink = (space, params) => __async(void 0, null, function* () {
   const needsSpaceId = !isAbsoluteUrl2(space.link.url) || essentialsDomainRegex.test(space.link.url);
   const urlWithParams = needsSpaceId ? updateUrl(space.link.url, "spaceId", spaceId) : space.link.url;
   let url = !isAbsoluteUrl2(space.link.url) ? getUrl(urlWithParams, false, false) : urlWithParams;
-  if (!isAbsoluteUrl2(url) && !essentialsDomainRegex.test(document.referrer) && /\/web\/|\/public\/(apps|spaces)/.test(url)) {
+  if (!isAbsoluteUrl2(url) && !essentialsDomainRegex.test(document.referrer) && /^\/web\/|\/public\/(apps|spaces)/.test(url)) {
     const referrer = new URL(document.referrer);
     url = `${referrer.origin}${url}`;
   }
   const target = getTarget(space.link.target);
   window.open(url, target);
 });
-var openLinkWithSso = (_0, _1) => __async(void 0, [_0, _1], function* (space, { akaname, clientId, payerSpaceId, ssoParams }) {
+var openLinkWithSso = (_0, _1) => __async(null, [_0, _1], function* (space, { akaname, clientId, payerSpaceId, ssoParams }) {
   var _a, _b;
   if (space.meta && space.meta.ssoId) {
     let options;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@availity/mui-spaces",
-  "version": "1.0.29",
+  "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);
       }
     }

package/src/lib/SpacesLink/linkHandlers.tsx

@@ -20,7 +20,7 @@ export const openLink: OpenLink = async (space, params) => {
 
   let url = !isAbsoluteUrl(space.link.url) ? getUrl(urlWithParams, false, false) : urlWithParams;
 
-  if (!isAbsoluteUrl(url) && !essentialsDomainRegex.test(document.referrer) && /\/web\/|\/public\/(apps|spaces)/.test(url)) {
+  if (!isAbsoluteUrl(url) && !essentialsDomainRegex.test(document.referrer) && /^\/web\/|\/public\/(apps|spaces)/.test(url)) {
     const referrer = new URL(document.referrer);
     url = `${referrer.origin}${url}`
   }