@iloveagents/foundry-agent 0.1.5 → 1.0.0

package/CHANGELOG.md

@@ -1,5 +1,37 @@
 # @iloveagents/foundry-agent
 
+## 1.0.0
+
+### Minor Changes
+
+- 9105483: shell: ChatConversationConfig API + sticky-runtime threadId
+
+  Lets a downstream module (e.g. `@ltwlf/spaces-web-ui`) attach a
+  `ChatConversationConfig` to its `ChatModule`. The shell then:
+  - Pre-mints a fresh thread UUID via `useRef` so the AG-UI runtime's
+    `threadId` is defined from first render — no `undefined → defined`
+    remount mid-conversation when the user sends the first message.
+  - Calls `config.useStickyConversationId()` (when declared) and
+    feeds the resulting id into the runtime so the chat stays "live"
+    while the user browses non-chat routes (Workspaces, Jobs, etc.).
+  - Per-config keyed child components (`StickyAwareRuntime` vs
+    `UrlOnlyRuntime`) so optional hooks satisfy Rules-of-Hooks across
+    multi-module config swaps.
+
+  UI: collapsible nav groups in `sidebar.tsx` with persisted state via
+  `useSyncExternalStore` + localStorage, plus stable group ordering via
+  new `priority` field in `nav-config.ts`.
+
+  `ag-ui-runtime-provider.tsx` accepts an external `threadId` and
+  `historyAdapterFactory` so the consuming app can wire its own
+  persistence (e.g. `ThreadHistoryAdapter` → REST `/api/conversations`).
+
+### Patch Changes
+
+- 2b10540: Fix `citationStore.clear()` so the registered handler survives per-conversation resets.
+
+  The previous implementation wiped both `results` AND `handler`. `useNewConversation` calls `clear()` on every "New Thread" click. Combined with the `registerOnce`-style handler registration in feature modules (e.g. SPACES' `registerSpacesCitationHandler`), the result was that all `[n]` citation markers in chat became inert non-clickable spans for the rest of the session after the first new-thread click. `clear()` now resets `results` only — the handler is module-level state and must persist.
+
 ## 0.1.5
 
 ## 0.1.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@iloveagents/foundry-agent",
-  "version": "0.1.5",
+  "version": "1.0.0",
   "license": "SEE LICENSE IN LICENSE",
   "type": "module",
   "types": "./src/index.ts",

package/src/__tests__/citation-store.test.ts

@@ -33,14 +33,20 @@ describe("citationStore", () => {
   });
 
   describe("clear", () => {
-    it("resets results and handler", () => {
+    it("resets results but preserves the registered handler", () => {
+      // The handler is registered once at app boot by a feature module
+      // (e.g. SPACES). It MUST survive "New Thread" / per-conversation
+      // resets — otherwise every [n] marker in chat becomes inert after
+      // the first new-thread click, since the registration guard
+      // suppresses re-binding for the rest of the session.
+      const handler: CitationHandler = { openCitation: () => {} };
       citationStore.getState().setResults([mockResult]);
-      citationStore.getState().setHandler({ openCitation: () => {} });
+      citationStore.getState().setHandler(handler);
 
       citationStore.getState().clear();
 
       expect(citationStore.getState().results).toEqual([]);
-      expect(citationStore.getState().handler).toBeNull();
+      expect(citationStore.getState().handler).toBe(handler);
     });
   });
 });

package/src/store/citation-store.ts

@@ -26,7 +26,14 @@ interface CitationState {
   setResults: (results: CitationResult[]) => void;
   /** Register a handler for citation click actions */
   setHandler: (handler: CitationHandler) => void;
-  /** Clear all citations (e.g. on new conversation) */
+  /**
+   * Reset per-conversation state. Clears ``results`` only — the handler
+   * is module-level state registered once at app boot by feature modules
+   * (e.g. ``registerSpacesCitationHandler``) and must persist across
+   * conversations. Wiping it on "New Thread" produced inert ``[n]``
+   * markers for the rest of the session, since the registration guard
+   * (``registerOnce`` in spaces ``use-spaces-init``) suppresses re-binding.
+   */
   clear: () => void;
 }
 
@@ -41,5 +48,5 @@ export const citationStore = createStore<CitationState>((set) => ({
   handler: null,
   setResults: (results) => set({ results }),
   setHandler: (handler) => set({ handler }),
-  clear: () => set({ results: [], handler: null }),
+  clear: () => set({ results: [] }),
 }));