Address PR review feedback

Author: therobbiedavis

docs/intro.md

@@ -29,4 +29,4 @@ Listenarr is a self-hosted audiobook automation server built to streamline the p
 
 ## API
 
-Open the [API guide](pathname:///api/) to review the bundled OpenAPI snapshot and see how to use Swagger on your own Listenarr instance.
+Open the [API guide](/api/) to review the bundled OpenAPI snapshot and see how to use Swagger on your own Listenarr instance.

scripts/sync-listenarr.mjs

@@ -268,17 +268,30 @@ async function bundleSwaggerUi(port) {
   writeFileSync(
     path.join(apiDir, 'swagger-initializer.js'),
     `window.onload = async function () {
-  const response = await fetch('./openapi.json');
-  const spec = await response.json();
+  let specConfig = { url: './openapi.json' };
 
-  window.ui = SwaggerUIBundle({
-    spec: {
-      ...spec,
-      info: {
-        ...(spec.info || {}),
-        description: ''
+  try {
+    const response = await fetch('./openapi.json');
+    if (!response.ok) {
+      throw new Error(\`Unable to load ./openapi.json: \${response.status}\`);
+    }
+
+    const spec = await response.json();
+    specConfig = {
+      spec: {
+        ...spec,
+        info: {
+          ...(spec.info || {}),
+          description: ''
+        }
       }
-    },
+    };
+  } catch (error) {
+    console.error('Failed to preload Listenarr OpenAPI spec for Swagger UI.', error);
+  }
+
+  window.ui = SwaggerUIBundle({
+    ...specConfig,
     dom_id: '#swagger-ui',
     deepLinking: true,
     presets: [SwaggerUIBundle.presets.apis, SwaggerUIStandalonePreset],

src/pages/api/index.js

@@ -164,6 +164,10 @@ export default function ApiPage() {
                   View Listenarr source
                 </a>
               </div>
+              <p className={styles.metaNote}>
+                Bundled spec generated {generatedAt} from Listenarr {metadata.version} (
+                {metadata.commit}).
+              </p>
             </div>
 
             <div className={styles.grid}>

src/pages/api/index.module.css

@@ -45,6 +45,12 @@
   gap: 0.9rem;
 }
 
+.metaNote {
+  margin: 1rem 0 0;
+  color: var(--text-muted);
+  font-size: 0.9rem;
+}
+
 .grid {
   display: grid;
   grid-template-columns: minmax(0, 1fr);

src/theme/Navbar/MobileSidebar/SecondaryMenu/index.js

@@ -25,7 +25,12 @@ export default function NavbarMobileSidebarSecondaryMenu() {
   }
 
   return (
-    <ErrorCauseBoundary onError={() => new Error('The mobile secondary navbar menu failed to render.')}>
+    <ErrorCauseBoundary
+      onError={(error) =>
+        new Error('The mobile secondary navbar menu failed to render.', {
+          cause: error,
+        })
+      }>
       {!isPrimaryMenuEmpty && (
         <SecondaryMenuBackButton onClick={() => secondaryMenu.hide()} />
       )}

src/theme/Navbar/MobileSidebar/index.js

@@ -1,7 +1,9 @@
 import React from 'react';
 import {useLockBodyScroll, useNavbarMobileSidebar} from '@docusaurus/theme-common/internal';
+import NavbarMobileSidebarLayout from '@theme/Navbar/MobileSidebar/Layout';
 import NavbarMobileSidebarHeader from '@theme/Navbar/MobileSidebar/Header';
 import NavbarMobileSidebarPrimaryMenu from '@theme/Navbar/MobileSidebar/PrimaryMenu';
+import NavbarMobileSidebarSecondaryMenu from '@theme/Navbar/MobileSidebar/SecondaryMenu';
 
 export default function NavbarMobileSidebar() {
   const mobileSidebar = useNavbarMobileSidebar();
@@ -13,13 +15,10 @@ export default function NavbarMobileSidebar() {
   }
 
   return (
-    <div className="navbar-sidebar">
-      <NavbarMobileSidebarHeader />
-      <div className="navbar-sidebar__items">
-        <div className="navbar-sidebar__item menu">
-          <NavbarMobileSidebarPrimaryMenu />
-        </div>
-      </div>
-    </div>
+    <NavbarMobileSidebarLayout
+      header={<NavbarMobileSidebarHeader />}
+      primaryMenu={<NavbarMobileSidebarPrimaryMenu />}
+      secondaryMenu={<NavbarMobileSidebarSecondaryMenu />}
+    />
   );
 }

static/api-ui/swagger-initializer.js

@@ -1,15 +1,28 @@
 window.onload = async function () {
-  const response = await fetch('./openapi.json');
-  const spec = await response.json();
+  let specConfig = { url: './openapi.json' };
 
-  window.ui = SwaggerUIBundle({
-    spec: {
-      ...spec,
-      info: {
-        ...(spec.info || {}),
-        description: ''
+  try {
+    const response = await fetch('./openapi.json');
+    if (!response.ok) {
+      throw new Error(`Unable to load ./openapi.json: ${response.status}`);
+    }
+
+    const spec = await response.json();
+    specConfig = {
+      spec: {
+        ...spec,
+        info: {
+          ...(spec.info || {}),
+          description: ''
+        }
       }
-    },
+    };
+  } catch (error) {
+    console.error('Failed to preload Listenarr OpenAPI spec for Swagger UI.', error);
+  }
+
+  window.ui = SwaggerUIBundle({
+    ...specConfig,
     dom_id: '#swagger-ui',
     deepLinking: true,
     presets: [SwaggerUIBundle.presets.apis, SwaggerUIStandalonePreset],