@promptbook/cli 0.114.0-34 → 0.114.0-38

package/apps/agents-server/package.json

@@ -8,7 +8,7 @@
         "test": "npm run test-e2e",
         "test-e2e": "node ./scripts/run-e2e-tests.js",
         "pretest-build": "node ./scripts/kill-port.js 4021 4440",
-        "test-build": "npm run build",
+        "test-build": "npm run build -- --no-sentry-upload",
         "pretest-e2e": "node ./scripts/kill-port.js 4021 4440",
         "build": "node ./scripts/build-agents-server.js",
         "build-e2e": "node ./scripts/build-e2e.js",

package/apps/agents-server/scripts/build-agents-server.js

@@ -25,6 +25,8 @@ const AGENTS_SERVER_BUILD_MAX_OLD_SPACE_MIB = 8192;
  * lint command has already completed successfully. The optional `--no-homepage-prerender`
  * argument is used by builds whose caller starts the very same production server right
  * afterwards, which makes the extra prerender server a repetition of that work.
+ * Test builds pass `--no-sentry-upload` to avoid publishing source maps and creating
+ * temporary upload bundles with credentials from the local `.env` file.
  */
 function buildAgentsServer() {
     const nextBuildArguments = [
@@ -44,6 +46,8 @@ function buildAgentsServer() {
         //       the build process which spawned it. The same merge is done for the CLI-owned
         //       production build in `src/cli/cli-commands/agents-server/buildAgentsServer/runNextBuild.ts`.
         NODE_OPTIONS: mergeNodeOptionsWithHeapSize(process.env.NODE_OPTIONS, AGENTS_SERVER_BUILD_MAX_OLD_SPACE_MIB),
+        // An empty value also prevents Next.js from loading an upload token from `.env`.
+        ...(process.argv.includes('--no-sentry-upload') ? { SENTRY_AUTH_TOKEN: '' } : {}),
     });
 
     if (!process.argv.includes('--no-homepage-prerender')) {

package/apps/agents-server/scripts/build-e2e.js

@@ -17,7 +17,7 @@ const APP_ROOT = path.resolve(__dirname, '..');
  * is compiled, and the tests open the homepage against that very same server, so the snapshot would
  * only repeat work the tests already do.
  */
-const E2E_BUILD_ARGUMENTS = ['--no-homepage-prerender'];
+const E2E_BUILD_ARGUMENTS = ['--no-homepage-prerender', '--no-sentry-upload'];
 
 /**
  * Builds the production bundle used by the Agents Server E2E tests.

package/apps/agents-server/src/app/swagger/SwaggerPageClient.tsx

@@ -1,7 +1,7 @@
 'use client';
 
-import { type ComponentType, useEffect, useState } from 'react';
-import 'swagger-ui-react/swagger-ui.css';
+import { useEffect, useRef, useState } from 'react';
+import 'swagger-ui-dist/swagger-ui.css';
 import { SwaggerApiKeysPanel } from './SwaggerApiKeysPanel';
 
 /**
@@ -18,26 +18,43 @@ type SwaggerPageClientProps = {
  * Renders the runtime-generated Swagger UI and the compact API-key helper panel.
  */
 export function SwaggerPageClient({ isAdmin }: SwaggerPageClientProps) {
-    const [SwaggerUI, setSwaggerUI] = useState<ComponentType<{
-        url: string;
-        persistAuthorization?: boolean;
-        defaultModelsExpandDepth?: number;
-        docExpansion?: 'list' | 'full' | 'none';
-    }> | null>(null);
+    const containerRef = useRef<HTMLDivElement>(null);
+    const [isLoading, setIsLoading] = useState(true);
+    const [hasLoadError, setHasLoadError] = useState(false);
 
     useEffect(() => {
         let isMounted = true;
+        const container = containerRef.current;
 
         void (async () => {
-            const swaggerUiModule = await import('swagger-ui-react');
+            try {
+                // Import the browser bundle directly; the package entry point also imports Node.js helpers.
+                const { default: SwaggerUIBundle } = await import('swagger-ui-dist/swagger-ui-bundle.js');
 
-            if (isMounted) {
-                setSwaggerUI(() => swaggerUiModule.default);
+                if (!isMounted || !container) {
+                    return;
+                }
+
+                SwaggerUIBundle({
+                    domNode: container,
+                    url: '/openapi.json',
+                    persistAuthorization: true,
+                    defaultModelsExpandDepth: -1,
+                    docExpansion: 'list',
+                });
+                setIsLoading(false);
+            } catch (error) {
+                if (isMounted) {
+                    console.error('Failed to load Swagger UI:', error);
+                    setHasLoadError(true);
+                    setIsLoading(false);
+                }
             }
         })();
 
         return () => {
             isMounted = false;
+            container?.replaceChildren();
         };
     }, []);
 
@@ -47,16 +64,15 @@ export function SwaggerPageClient({ isAdmin }: SwaggerPageClientProps) {
                 <SwaggerApiKeysPanel isAdmin={isAdmin} />
             </div>
             <div className="swagger-container bg-white min-h-screen">
-                {SwaggerUI ? (
-                    <SwaggerUI
-                        url="/openapi.json"
-                        persistAuthorization
-                        defaultModelsExpandDepth={-1}
-                        docExpansion="list"
-                    />
-                ) : (
+                {isLoading && (
                     <div className="mx-auto max-w-7xl px-4 py-10 text-sm text-slate-500">Loading Swagger UI...</div>
                 )}
+                {hasLoadError && (
+                    <div role="alert" className="mx-auto max-w-7xl px-4 py-10 text-sm text-red-600">
+                        Failed to load Swagger UI. Please reload the page.
+                    </div>
+                )}
+                <div ref={containerRef} />
             </div>
         </div>
     );

package/apps/agents-server/src/types/swagger-ui-dist.d.ts

@@ -0,0 +1,5 @@
+declare module 'swagger-ui-dist/swagger-ui-bundle.js' {
+    import { SwaggerUIBundle } from 'swagger-ui-dist';
+
+    export default SwaggerUIBundle;
+}