@gsa-tts/graymatter-ui 0.3.18 → 0.3.20

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gsa-tts/graymatter-ui",
-  "version": "0.3.18",
+  "version": "0.3.20",
   "type": "module",
   "publishConfig": {
     "access": "public"

package/src/components/DesktopSideNav.svelte

@@ -19,7 +19,6 @@
   import ApiIcon from './icons/ApiIcon.svelte';
   import MenuIcon from './icons/MenuIcon.svelte';
   import ProfileMenu from './ProfileMenu.svelte';
-  import { getAppUrls } from '../utils/getAppUrls';
   // import HelpIcon from './icons/HelpIcon.svelte';
 
   const {
@@ -35,12 +34,6 @@
     discoverUrl,
   } = $props();
 
-  // Only call getAppUrls if not all URLs are provided via props
-  const urls =
-    apiUrl && chatUrl && consoleUrl && discoverUrl
-      ? { api: '#', chat: '#', console: '#', discover: '#' }
-      : getAppUrls();
-
   let hydrated = false;
   let navContainer: HTMLDivElement;
   let hoveredItem = $state<string | null>(null);
@@ -332,7 +325,7 @@
         <div class="nav-items" part="nav-items">
           {#if !hideDiscover}
             <a
-              href={discoverUrl || urls.discover}
+              href={discoverUrl || '#'}
               class="nav-item discover-item"
               class:selected={isNavSelected('discover')}
               class:hovered={isNavHovered('discover')}
@@ -356,7 +349,7 @@
             </a>
           {/if}
           <a
-            href={chatUrl || urls.chat}
+            href={chatUrl || '#'}
             class="nav-item chat-item"
             class:selected={isNavSelected('chat')}
             class:hovered={isNavHovered('chat')}
@@ -380,7 +373,7 @@
           </a>
 
           <a
-            href={consoleUrl || urls.console}
+            href={consoleUrl || '#'}
             class="nav-item console-item"
             class:selected={isNavSelected('console')}
             class:hovered={isNavHovered('console')}
@@ -404,7 +397,7 @@
           </a>
 
           <a
-            href={apiUrl || urls.api}
+            href={apiUrl || '#'}
             class="nav-item api-item"
             class:selected={isNavSelected('api')}
             class:hovered={isNavHovered('api')}

package/src/components/MobileBottomNav.svelte

@@ -5,7 +5,6 @@
   import ChatIcon from './icons/ChatIcon.svelte';
   import ConsoleIcon from './icons/ConsoleIcon.svelte';
   import ApiIcon from './icons/ApiIcon.svelte';
-  import { getAppUrls } from '../utils/getAppUrls';
   import slugify from 'slugify';
 
   let {
@@ -29,27 +28,21 @@
     discoverUrl?: string;
   } = $props();
 
-  // Only call getAppUrls if not all URLs are provided via props
-  const urls =
-    apiUrl && chatUrl && consoleUrl && discoverUrl
-      ? { api: '#', chat: '#', console: '#', discover: '#' }
-      : getAppUrls();
-
   const allNavItems = [
     {
       id: 'discover',
       label: 'Discover',
       icon: DiscoverIcon,
-      href: discoverUrl || urls.discover,
+      href: discoverUrl || '#',
     },
-    { id: 'chat', label: 'Chat', icon: ChatIcon, href: chatUrl || urls.chat },
+    { id: 'chat', label: 'Chat', icon: ChatIcon, href: chatUrl || '#' },
     {
       id: 'console',
       label: 'Console',
       icon: ConsoleIcon,
-      href: consoleUrl || urls.console,
+      href: consoleUrl || '#',
     },
-    { id: 'api', label: 'API', icon: ApiIcon, href: apiUrl || urls.api },
+    { id: 'api', label: 'API', icon: ApiIcon, href: apiUrl || '#' },
   ] as const;
 
   // Filter out discover item if hideDiscover is true

package/src/layouts/DocsLayout.astro

@@ -15,6 +15,7 @@ const {
   disableCodeCopyButtons = false,
   postItems = null,
   profileMenuData = null,
+  hideDiscover = true,
 } = Astro.props;
 ---
 
@@ -28,6 +29,7 @@ const {
   title={title}
   {description}
   {openGraphImage}
+  {hideDiscover}
 >
   <!-- Initialize navigation state and auto-expand submenu for section pages -->
   <NavigationInitializer

package/src/layouts/GlobalAppLayout.astro

@@ -8,6 +8,7 @@ import MobileBottomNav from '../components/MobileBottomNav.svelte';
 import MobileSideMenu from '../components/MobileSideMenu.svelte';
 import Logo from '../components/Logo.svelte';
 import { siteName } from '../constants';
+import { getAppUrls } from '../utils/getAppUrls';
 
 const {
   componentOptions = {},
@@ -20,7 +21,10 @@ const {
   title: pageTitle,
   description,
   openGraphImage,
+  hideDiscover = true,
 } = Astro.props;
+
+const { api, chat, console, discover } = getAppUrls();
 ---
 
 <BaseLayout title={pageTitle} {gtmID} {description} {openGraphImage}>
@@ -31,8 +35,13 @@ const {
         <DesktopSideNav
           client:load
           ssrSelectedItem={ssrSelectedItem}
-          {showAppIcons}
-          {profileMenuData}
+          showAppIcons={showAppIcons}
+          hideDiscover={hideDiscover}
+          profileMenuData={profileMenuData}
+          apiUrl={api}
+          chatUrl={chat}
+          consoleUrl={console}
+          discoverUrl={discover}
         >
           <slot name="sub-nav" />
         </DesktopSideNav>
@@ -51,14 +60,26 @@ const {
     <nav aria-label="Mobile navigation">
       <MobileTopNav
         client:load
-        {profileMenuData}
-        {logoLinkUrl}
-        {logoLinkLabel}
+        profileMenuData={profileMenuData}
+        logoLinkUrl={logoLinkUrl}
+        logoLinkLabel={logoLinkLabel}
+        apiUrl={api}
+        chatUrl={chat}
+        consoleUrl={console}
+        discoverUrl={discover}
       />
       <MobileSideMenu client:load {logoLinkUrl} {logoLinkLabel}>
         <slot name="sub-nav" />
       </MobileSideMenu>
-      <MobileBottomNav client:load {showAppIcons} />
+      <MobileBottomNav
+        client:load
+        showAppIcons={showAppIcons}
+        hideDiscover={hideDiscover}
+        apiUrl={api}
+        chatUrl={chat}
+        consoleUrl={console}
+        discoverUrl={discover}
+      />
     </nav>
 
     <!-- Fixed Logo (Desktop only) -->

package/src/utils/getAppUrls.ts

@@ -1,11 +1,23 @@
 import { getUrlFromBase } from '@gsa-tts/graymatter-ui/helpers/url.js';
 
+/**
+ * Logs a warning message when an environment variable is missing.
+ *
+ * @param name - The name of the missing environment variable
+ * @param value - The value of the environment variable (should be undefined)
+ */
 function warnIfMissing(name: string, value: string | undefined) {
   if (!value) {
     console.warn(`[getAppUrls] Environment variable for ${name} is missing!`);
   }
 }
 
+/**
+ * Checks if a URL is absolute (starts with protocol like http:// or https://).
+ *
+ * @param url - The URL string to check
+ * @returns True if the URL is absolute, false otherwise
+ */
 function isAbsoluteUrl(url: string): boolean {
   try {
     new URL(url);
@@ -15,6 +27,24 @@ function isAbsoluteUrl(url: string): boolean {
   }
 }
 
+/**
+ * Processes a URL string, handling both absolute and relative URLs.
+ *
+ * If the URL is undefined, returns the fallback. If it's absolute (starts with protocol),
+ * returns it as-is. If it's relative, processes it through `getUrlFromBase()` to create
+ * a full URL.
+ *
+ * @param url - The URL string to process (can be undefined)
+ * @param fallback - The fallback URL to use if the input URL is undefined
+ * @returns The processed URL string (can be absolute URL or relative path)
+ *
+ * @example
+ * ```typescript
+ * processUrl('https://example.com', '/fallback'); // "https://example.com"
+ * processUrl('/api/docs', '/fallback'); // "/api/docs" (processed by getUrlFromBase)
+ * processUrl(undefined, '/fallback'); // "/fallback"
+ * ```
+ */
 function processUrl(url: string | undefined, fallback: string): string {
   if (!url) {
     return fallback;
@@ -27,6 +57,60 @@ function processUrl(url: string | undefined, fallback: string): string {
   return getUrlFromBase(url);
 }
 
+/**
+ * Retrieves application URLs/paths from environment variables with fallback support.
+ *
+ * This utility function processes environment variables to provide URLs or paths for different
+ * application sections (chat, console, API, discover). It supports both absolute URLs
+ * and relative paths, with intelligent fallback handling.
+ *
+ * @param envArg - Optional environment object to use instead of `import.meta.env`.
+ *                 Useful for testing or when you want to override environment variables.
+ *                 If not provided, uses Vite's `import.meta.env`.
+ *
+ * @returns An object containing processed URLs/paths for each application section:
+ *   - `chat`: Chat application URL or path
+ *   - `console`: Console application URL or path
+ *   - `api`: API documentation URL or path
+ *   - `discover`: Discover page URL or path
+ *
+ * @example
+ * ```typescript
+ * // Using default environment variables
+ * const urls = getAppUrls();
+ * console.log(urls.chat); // "#" (fallback) or "https://chat.example.com" (URL)
+ *
+ * // Using custom environment object (useful for testing)
+ * const urls = getAppUrls({
+ *   PUBLIC_CHAT_URL: 'https://chat.dev/',
+ *   PUBLIC_CONSOLE_URL: 'https://console.dev/',
+ *   PUBLIC_API_URL: 'https://api.dev/docs',
+ *   PUBLIC_DISCOVER_URL: 'https://discover.dev/'
+ * });
+ * ```
+ *
+ * @example
+ * ```typescript
+ * // Environment variables (.env file)
+ * PUBLIC_CHAT_URL=https://chat.example.com
+ * PUBLIC_CONSOLE_URL=https://console.example.com
+ * PUBLIC_API_URL=/api/documentation
+ * PUBLIC_DISCOVER_URL=https://discover.example.com
+ *
+ * // Usage in code
+ * const { chat, console, api, discover } = getAppUrls();
+ * ```
+ *
+ * @remarks
+ * - URLs are processed through `processUrl()` which handles both absolute and relative URLs
+ * - Relative URLs are processed through `getUrlFromBase()` to create full URLs
+ * - Missing environment variables result in fallback values (`#` for most, `/api/documentation` for API)
+ * - Warnings are logged in development when environment variables are missing
+ * - The function is designed to work with Vite's `import.meta.env` system
+ *
+ * @see {@link processUrl} - URL processing logic
+ * @see {@link getUrlFromBase} - Base URL resolution
+ */
 export function getAppUrls(envArg?: Record<string, string | undefined>) {
   const env = envArg || import.meta.env;
   const fallback = '#';