@flamingo-stack/openframe-frontend-core 0.0.735 → 0.0.737

package/src/components/unified-pagination.tsx

@@ -33,6 +33,17 @@ export function UnifiedPagination({
   const searchParams = useSearchParams();
   const pathname = usePathname();
 
+  /**
+   * The address of a page of this list: the current URL with `page` set. Page 1
+   * is the list's own address (no `page`), so the first page has one URL.
+   */
+  const pageHref = (page: number) => {
+    const params = new URLSearchParams(searchParams.toString());
+    if (page <= 1) params.delete('page');
+    else params.set('page', page.toString());
+    return withQuery(pathname, params.toString());
+  };
+
   const handlePageChange = (page: number) => {
     // Preserve current scroll position
     const currentScrollY = window.scrollY;
@@ -42,13 +53,12 @@ export function UnifiedPagination({
       onPageChange(page);
     }
 
-    // Update URL for bookmarking without navigation — unless the caller owns
-    // the URL (`syncUrl={false}`), in which case a second writer here would
-    // race the caller's own param write.
+    // Update the address bar in place (a bookmark, a shared link), unless the
+    // caller owns the URL (`syncUrl={false}`): a second writer here would race
+    // the caller's own param write. The address is the one the page's link
+    // carries (`pageHref`), so a click and a new-tab open land on the same URL.
     if (syncUrl) {
-      const params = new URLSearchParams(searchParams.toString());
-      params.set('page', page.toString());
-      window.history.replaceState(null, '', withQuery(pathname, params.toString()));
+      window.history.replaceState(null, '', pageHref(page));
     }
 
     // Restore scroll position after a brief delay to allow content to render
@@ -79,7 +89,13 @@ export function UnifiedPagination({
 
   return (
     <div className={className}>
-      <Pagination currentPage={currentPage} totalPages={totalPages} onPageChange={handlePageChange} />
+      {/* Real links only while this component owns `?page=`: a caller that owns the URL names its own parameter. */}
+      <Pagination
+        currentPage={currentPage}
+        totalPages={totalPages}
+        onPageChange={handlePageChange}
+        getPageHref={syncUrl ? pageHref : undefined}
+      />
     </div>
   );
 }

package/src/contexts/assistant-runtime-context.tsx

@@ -56,11 +56,11 @@ export interface AssistantRuntime {
   source?: string;
   /**
    * GET endpoint the questions are read from:
-   *   `<url>?count=<n>&section=<topic>&exclude=<id,id>` → `{ prompts: AskPrompt[] }`
-   * `section` is ANY string: the host decides what topics exist (the hub: the
-   * `section` attribute of a quick action, set in chat config), so a new topic
-   * needs no change here. Hub: '/api/quick-actions/questions'. Absent: an "ask"
-   * surface shows its launcher with no questions.
+   *   `<url>?count=<n>&topic=<topic>&exclude=<id,id>` → `{ prompts: AskPrompt[] }`
+   * `topic` is ANY string: the host decides what topics exist (the hub: the
+   * topics of its FAQs, set in the FAQ admin), so a new topic needs no change
+   * here. Hub: '/api/faqs/questions'. Absent: an "ask" surface shows its
+   * launcher with no questions.
    */
   askPromptsUrl?: string;
   /**

package/src/types/faq.ts

@@ -1,8 +1,26 @@
+/**
+ * The two kinds of FAQ. An `answer` is a question with its written answer (the
+ * accordion, FAQPage JSON-LD). An `ask` is a question the assistant answers: it
+ * has a `prompt` (what is sent to the chat) and no written answer, and is what
+ * every "ask the assistant" surface offers.
+ */
+export const FAQ_KINDS = ['answer', 'ask'] as const;
+export type FaqKind = (typeof FAQ_KINDS)[number];
+
 export interface Faq {
   id: number;
   question: string;
   answer: string;
   section?: string | null;
+  /** Absent on a host that predates kinds: an `answer`. */
+  kind?: FaqKind;
+  /** An `ask` FAQ's prompt for the chat; null on an `answer`. */
+  prompt?: string | null;
+  /**
+   * The places the FAQ is written for, shared by both kinds: a page's topic
+   * (`pricing`, `careers`, `home`) or a section's. Empty: written for no place.
+   */
+  topics?: string[];
   display_order: number;
   is_active: boolean;
   created_at: string;