@iloveagents/foundry-web-shell 0.11.0 → 0.12.0

package/dist/shell-app.d.ts

@@ -8,11 +8,5 @@ interface ShellAppProps {
     /** Test seam — overrides defaultAgentFetch. */
     agentFetch?: typeof fetch;
 }
-/**
- * Inner shell component — pure React (no `createRoot`). Imported by
- * `bootstrap-shell.tsx` for production and by `__tests__/` for assertions.
- *
- * Wrapper ordering is contractual; see `wrapper-order.test.tsx`.
- */
 export declare function ShellApp({ modules, pages, baseThemeLayers, authProvider, agentFetch, }: ShellAppProps): import("react/jsx-runtime").JSX.Element;
 export {};

package/dist/shell-app.js

@@ -12,17 +12,29 @@ import { _setChatConfigFetch, loadChatConfig } from "./load-chat-config.js";
  *
  * Wrapper ordering is contractual; see `wrapper-order.test.tsx`.
  */
+/**
+ * Loads the chat config once, from INSIDE the authenticated subtree.
+ *
+ * This must not run in `ShellApp`'s own body: `ShellAuth` is its CHILD, so
+ * an effect there fires before sign-in completes. The agent fetch would
+ * then 401, exhaust its refresh retry, and call
+ * `recoverFromHardAuthFailure` -> `loginRedirect` -> remount -> repeat: an
+ * infinite login loop on any session that isn't already cached. Rendering
+ * the load here means there is a token by the time it runs.
+ */
+function ChatConfigBoot({ fetchImpl }) {
+    useEffect(() => {
+        void loadChatConfig(fetchImpl);
+    }, [fetchImpl]);
+    return null;
+}
 export function ShellApp({ modules, pages, baseThemeLayers, authProvider, agentFetch, }) {
-    // Ask the backend what `Auto` currently resolves to, so the composer's
-    // effort picker can name the level instead of leaving the user guessing.
-    // Uses the same agentFetch seam as everything else (auth, base URL, test
-    // override); failure resets the level to unknown and the picker says so.
     const resolvedAgentFetch = agentFetch ?? defaultAgentFetch;
+    // Bind the refresh seam here (cheap, no I/O) so a host-app context
+    // refresh that races boot uses the same fetch client. The actual load
+    // happens in <ChatConfigBoot> INSIDE ShellAuth — see the note there.
     useEffect(() => {
-        // Bind the refresh seam first so a host-app context refresh that races
-        // shell boot still uses the same fetch client.
         _setChatConfigFetch(resolvedAgentFetch);
-        void loadChatConfig(resolvedAgentFetch);
     }, [resolvedAgentFetch]);
     // Merge module pages with customer pages — customer wins on path collision
     // (last-wins). Stable order: customer pages first (they appear in Routes
@@ -60,7 +72,7 @@ export function ShellApp({ modules, pages, baseThemeLayers, authProvider, agentF
     // Aggregate chatConversation configs from all modules. First-match
     // wins at URL-match time (rare to have more than one anyway).
     const chatConversationConfigs = useMemo(() => modules.flatMap((m) => (m.chatConversation ? [m.chatConversation] : [])), [modules]);
-    return (_jsx(BrowserRouter, { children: _jsx(ShellAuth, { adapter: authProvider, children: _jsx(ComposedWrappers, { wrappers: wrappers, children: _jsx(ChatConversationAwareRuntime, { fetchFn: fetchFn, configs: chatConversationConfigs, toolUIs: toolUIs, children: _jsx(Suspense, { fallback: null, children: _jsx(Routes, { children: _jsx(Route, { element: _jsx(ShellLayout, { modules: modules, baseThemeLayers: baseThemeLayers }), children: mergedPages.map((p) => (_jsx(Route, { path: p.path === "/" ? undefined : p.path, index: p.path === "/", element: p.element }, p.path))) }) }) }) }) }) }) }));
+    return (_jsx(BrowserRouter, { children: _jsxs(ShellAuth, { adapter: authProvider, children: [_jsx(ChatConfigBoot, { fetchImpl: resolvedAgentFetch }), _jsx(ComposedWrappers, { wrappers: wrappers, children: _jsx(ChatConversationAwareRuntime, { fetchFn: fetchFn, configs: chatConversationConfigs, toolUIs: toolUIs, children: _jsx(Suspense, { fallback: null, children: _jsx(Routes, { children: _jsx(Route, { element: _jsx(ShellLayout, { modules: modules, baseThemeLayers: baseThemeLayers }), children: mergedPages.map((p) => (_jsx(Route, { path: p.path === "/" ? undefined : p.path, index: p.path === "/", element: p.element }, p.path))) }) }) }) }) })] }) }));
 }
 /**
  * Reads the current pathname (only meaningful inside ``<BrowserRouter>``),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@iloveagents/foundry-web-shell",
-  "version": "0.11.0",
+  "version": "0.12.0",
   "license": "MIT",
   "description": "Browser bootstrap and layout shell for Foundry UI — bootstrapShell({ modules, pages, theme, authProvider }) mounts the SPA around foundry-web-ui. Module-agnostic.",
   "keywords": [
@@ -45,9 +45,9 @@
     "zustand": "^5.0.0"
   },
   "dependencies": {
-    "@iloveagents/foundry-agent": "^0.11.0",
-    "@iloveagents/foundry-web-primitives": "^0.11.0",
-    "@iloveagents/foundry-web-ui": "^0.11.0"
+    "@iloveagents/foundry-agent": "^0.12.0",
+    "@iloveagents/foundry-web-primitives": "^0.12.0",
+    "@iloveagents/foundry-web-ui": "^0.12.0"
   },
   "devDependencies": {
     "@assistant-ui/react": "^0.15.1",