@carlonicora/nextjs-jsonapi 5.4.0 → 5.6.0

package/src/core/registry/ModuleRegistry.ts

@@ -34,6 +34,9 @@ export interface FoundationModuleDefinitions {
   StripePrice: ModuleWithPermissions;
   StripeUsage: ModuleWithPermissions;
   StripePromotionCode: ModuleWithPermissions;
+  StripeCoupon: ModuleWithPermissions;
+  StripeCouponCode: ModuleWithPermissions;
+  StripeCouponOffer: ModuleWithPermissions;
   // AI connection administration
   AiConnection: ModuleWithPermissions;
   // OAuth modules

package/src/features/administration/components/AdminIndexContainer.tsx

@@ -8,14 +8,21 @@ import { Link } from "../../../shadcnui";
 import { AdministrationProvider } from "../contexts/AdministrationContext";
 import type { AdminGroup, AdminSection } from "../data/admin-section.types";
 
+/** Keys of the built-in platform sections, for `hiddenSections`. */
+export type PlatformSectionKey = "companies" | "users" | "token-usage" | "ai-connections" | "rbac" | "billing";
+
 /**
- * The sections every NJA project has. Hrefs are constants rather than props:
- * the path convention IS the contract that makes this component reusable, and
- * it already exists in the package (see TOKEN_USAGE_ADMIN_PAGE_URL in
- * features/tokenusage/contexts/TokenUsageAdminContext.tsx).
+ * The sections every NJA project has, by key. Hrefs are constants rather than
+ * props: the path convention IS the contract that makes this component
+ * reusable. Exported so an app that passes its own `groups` places these
+ * entries without retyping their keys, icons and copy.
+ *
+ * `billing` is the only conditional entry: the default platform group shows it
+ * only when Stripe is configured (see AdminIndexGrid). An app that places it in
+ * its own `groups` applies the same check itself.
  */
-const PLATFORM_SECTIONS: AdminSection[] = [
-  {
+export const ADMIN_PLATFORM_SECTIONS: Record<PlatformSectionKey, AdminSection> = {
+  companies: {
     key: "companies",
     icon: Building2Icon,
     labelKey: "entities.companies",
@@ -23,7 +30,7 @@ const PLATFORM_SECTIONS: AdminSection[] = [
     descriptionKey: "administration.companies.description",
     href: "/administration/companies",
   },
-  {
+  users: {
     key: "users",
     icon: UsersIcon,
     labelKey: "entities.users",
@@ -31,21 +38,21 @@ const PLATFORM_SECTIONS: AdminSection[] = [
     descriptionKey: "administration.users.description",
     href: "/administration/users",
   },
-  {
+  "token-usage": {
     key: "token-usage",
     icon: CoinsIcon,
     labelKey: "token_usage.admin.title",
     descriptionKey: "administration.token_usage.description",
     href: "/administration/token-usage",
   },
-  {
+  "ai-connections": {
     key: "ai-connections",
     icon: PlugZapIcon,
     labelKey: "ai_connections.admin.title",
     descriptionKey: "administration.ai_connections.description",
     href: "/administration/ai-connections",
   },
-  {
+  rbac: {
     key: "rbac",
     icon: ShieldCheckIcon,
     labelKey: "entities.rbac",
@@ -53,27 +60,33 @@ const PLATFORM_SECTIONS: AdminSection[] = [
     descriptionKey: "administration.rbac.description",
     href: "/administration/rbac",
   },
-];
-
-/**
- * The only conditional entry. getStripePublishableKey comes from /client rather
- * than isStripeConfigured() from /billing on purpose: /billing exists to keep
- * Stripe.js out of the global bundle, and isStripeConfigured lives in
- * StripeProvider.tsx, which imports @stripe/stripe-js at module scope. The two
- * are the same check — isStripeConfigured is a one-line wrapper over this call.
- */
-const BILLING_SECTION: AdminSection = {
-  key: "billing",
-  icon: CreditCardIcon,
-  labelKey: "billing.admin.products.title",
-  descriptionKey: "administration.billing.description",
-  href: "/administration/products",
+  billing: {
+    key: "billing",
+    icon: CreditCardIcon,
+    labelKey: "billing.admin.products.title",
+    descriptionKey: "administration.billing.description",
+    href: "/administration/products",
+  },
 };
 
-/** Keys of the built-in platform sections, for `hiddenSections`. */
-export type PlatformSectionKey = "companies" | "users" | "token-usage" | "ai-connections" | "rbac" | "billing";
+/** The unconditional platform entries, in the default group's order. */
+const PLATFORM_SECTIONS: AdminSection[] = [
+  ADMIN_PLATFORM_SECTIONS.companies,
+  ADMIN_PLATFORM_SECTIONS.users,
+  ADMIN_PLATFORM_SECTIONS["token-usage"],
+  ADMIN_PLATFORM_SECTIONS["ai-connections"],
+  ADMIN_PLATFORM_SECTIONS.rbac,
+];
 
 type AdminIndexProps = {
+  /**
+   * Replaces the default groups (the built-in platform group plus
+   * `additionalGroups`) with exactly these, in this order. For an app that
+   * arranges its administration by its own headings; it places the platform
+   * entries it wants from ADMIN_PLATFORM_SECTIONS. When set, `additionalGroups`
+   * is ignored and the Stripe check on `billing` is the app's to apply.
+   */
+  groups?: AdminGroup[];
   /** Project-specific groups, appended after the built-in platform group. */
   additionalGroups?: AdminGroup[];
   /**
@@ -90,10 +103,12 @@ type AdminIndexProps = {
  * The grid itself, with no page shell — this is what the specs render, because
  * RoundPageContainer pulls in the app Header and its four header contexts.
  */
-export function AdminIndexGrid({ additionalGroups = [], hiddenSections = [] }: AdminIndexProps) {
+export function AdminIndexGrid({ groups: customGroups, additionalGroups = [], hiddenSections = [] }: AdminIndexProps) {
   const t = useTranslations();
 
-  const available = getStripePublishableKey() ? [...PLATFORM_SECTIONS, BILLING_SECTION] : PLATFORM_SECTIONS;
+  const available = getStripePublishableKey()
+    ? [...PLATFORM_SECTIONS, ADMIN_PLATFORM_SECTIONS.billing]
+    : PLATFORM_SECTIONS;
 
   const platformGroup: AdminGroup = {
     key: "platform",
@@ -101,7 +116,16 @@ export function AdminIndexGrid({ additionalGroups = [], hiddenSections = [] }: A
     sections: available.filter((section) => !hiddenSections.includes(section.key as PlatformSectionKey)),
   };
 
-  const groups = [platformGroup, ...additionalGroups].filter((group) => group.sections.length > 0);
+  // Custom groups still honour hiddenSections, so a project that cannot serve
+  // a platform route drops it the same way in either mode.
+  const groups = (
+    customGroups
+      ? customGroups.map((group) => ({
+          ...group,
+          sections: group.sections.filter((section) => !hiddenSections.includes(section.key as PlatformSectionKey)),
+        }))
+      : [platformGroup, ...additionalGroups]
+  ).filter((group) => group.sections.length > 0);
 
   return (
     <div className="mx-auto flex w-full max-w-5xl flex-1 flex-col gap-8 py-8">
@@ -146,11 +170,15 @@ export function AdminIndexGrid({ additionalGroups = [], hiddenSections = [] }: A
  * and its own page chrome (AdministrationProvider), so it does not inherit the
  * title and breadcrumb of whatever page rendered before it.
  */
-export function AdminIndexContainer({ additionalGroups, hiddenSections }: AdminIndexProps) {
+export function AdminIndexContainer({ groups, additionalGroups, hiddenSections }: AdminIndexProps) {
+  const t = useTranslations();
+
+  // The index is the "Administration" page itself, so it names itself in the
+  // breadcrumb like every administration page does ("Home > Companies").
   return (
-    <AdministrationProvider>
+    <AdministrationProvider section={t("administration.title")}>
       <RoundPageContainer>
-        <AdminIndexGrid additionalGroups={additionalGroups} hiddenSections={hiddenSections} />
+        <AdminIndexGrid groups={groups} additionalGroups={additionalGroups} hiddenSections={hiddenSections} />
       </RoundPageContainer>
     </AdministrationProvider>
   );

package/src/features/administration/components/__tests__/AdminIndexContainer.spec.tsx

@@ -3,7 +3,7 @@ import { BookOpenIcon } from "lucide-react";
 import { beforeEach, describe, expect, it } from "vitest";
 import { configureJsonApi } from "../../../../client/config";
 import { configureI18n } from "../../../../i18n";
-import { AdminIndexGrid } from "../AdminIndexContainer";
+import { ADMIN_PLATFORM_SECTIONS, AdminIndexGrid } from "../AdminIndexContainer";
 import type { AdminGroup } from "../../data/admin-section.types";
 
 // next-intl is globally mocked in vitest.setup.ts with `t = (key) => key`, so
@@ -93,4 +93,64 @@ describe("AdminIndexGrid", () => {
     // RoundPageContainerTitle, so the grid must not repeat it.
     expect(screen.queryByText("administration.title")).not.toBeInTheDocument();
   });
+
+  describe("groups", () => {
+    const customGroups: AdminGroup[] = [
+      {
+        key: "customers",
+        labelKey: "administration.group.customers",
+        sections: [ADMIN_PLATFORM_SECTIONS.users, ADMIN_PLATFORM_SECTIONS.companies],
+      },
+      extraGroup,
+      {
+        key: "system",
+        labelKey: "administration.group.system",
+        sections: [ADMIN_PLATFORM_SECTIONS["ai-connections"], ADMIN_PLATFORM_SECTIONS.rbac],
+      },
+    ];
+
+    it("replaces the platform group and additionalGroups with exactly the given groups, in order", () => {
+      render(<AdminIndexGrid groups={customGroups} additionalGroups={[extraGroup]} />);
+
+      const headings = screen.getAllByRole("heading", { level: 3 }).map((h) => h.textContent);
+      expect(headings).toEqual([
+        "administration.group.customers",
+        "administration.group.documentation",
+        "administration.group.system",
+      ]);
+      expect(screen.queryByText("administration.group.platform")).not.toBeInTheDocument();
+      expect(screen.queryByTestId("admin-index-token-usage")).not.toBeInTheDocument();
+      expect(screen.getAllByTestId("admin-index-howtos")).toHaveLength(1);
+    });
+
+    it("renders sections in the order each group lists them", () => {
+      render(<AdminIndexGrid groups={customGroups} />);
+
+      const ids = screen.getAllByRole("link").map((link) => link.getAttribute("data-testid"));
+      expect(ids).toEqual([
+        "admin-index-users",
+        "admin-index-companies",
+        "admin-index-howtos",
+        "admin-index-ai-connections",
+        "admin-index-rbac",
+      ]);
+      expect(screen.getByTestId("admin-index-companies")).toHaveAttribute("href", "/administration/companies");
+    });
+
+    it("applies hiddenSections inside the given groups and skips a group left empty", () => {
+      render(
+        <AdminIndexGrid
+          groups={[
+            ...customGroups,
+            { key: "only-rbac", labelKey: "administration.group.only_rbac", sections: [ADMIN_PLATFORM_SECTIONS.rbac] },
+          ]}
+          hiddenSections={["rbac"]}
+        />,
+      );
+
+      expect(screen.queryByTestId("admin-index-rbac")).not.toBeInTheDocument();
+      expect(screen.getByTestId("admin-index-ai-connections")).toBeInTheDocument();
+      expect(screen.queryByText("administration.group.only_rbac")).not.toBeInTheDocument();
+    });
+  });
 });

package/src/features/administration/components/index.ts

@@ -1 +1,2 @@
-export { AdminIndexContainer, AdminIndexGrid } from "./AdminIndexContainer";
+export { ADMIN_PLATFORM_SECTIONS, AdminIndexContainer, AdminIndexGrid } from "./AdminIndexContainer";
+export type { PlatformSectionKey } from "./AdminIndexContainer";

package/src/features/administration/contexts/AdministrationContext.tsx

@@ -11,6 +11,14 @@ const AdministrationContext = createContext<AdministrationContextType | undefine
 
 type AdministrationProviderProps = {
   children: ReactNode;
+  /**
+   * The page's own name, already translated (the same label the sidebar and the
+   * administration index use for it). When given, the breadcrumb reads
+   * "Home > {section}" with no link (it is the current page) and the title bar
+   * shows it. A plain string so a Server Component page can pass an already translated label.
+   * Omitted, the breadcrumb is empty and the title is "Administration".
+   */
+  section?: string;
 };
 
 /**
@@ -26,13 +34,13 @@ type AdministrationProviderProps = {
  * Mirrors CommonProvider (contexts/CommonContext.tsx), which does the same job
  * for the ordinary application shell.
  */
-export const AdministrationProvider = ({ children }: AdministrationProviderProps) => {
+export const AdministrationProvider = ({ children, section }: AdministrationProviderProps) => {
   const t = useTranslations();
 
-  const breadcrumbs: BreadcrumbItemData[] = [];
+  const breadcrumbs: BreadcrumbItemData[] = section ? [{ name: section }] : [];
 
   return (
-    <SharedProvider value={{ breadcrumbs, title: { type: t(`administration.title`) } }}>
+    <SharedProvider value={{ breadcrumbs, title: { type: section ?? t(`administration.title`) } }}>
       <AdministrationContext.Provider value={{}}>{children}</AdministrationContext.Provider>
     </SharedProvider>
   );

package/src/features/administration/contexts/__tests__/AdministrationContext.spec.tsx

@@ -0,0 +1,40 @@
+import { render, screen } from "@testing-library/react";
+import { describe, expect, it } from "vitest";
+import { useSharedContext } from "../../../../contexts/SharedContext";
+import { AdministrationProvider } from "../AdministrationContext";
+
+// next-intl is globally mocked in vitest.setup.ts with `t = (key) => key`, so a
+// translated label renders as its own key.
+const SharedProbe = () => {
+  const { breadcrumbs, title } = useSharedContext();
+  return (
+    <>
+      <span data-testid="breadcrumbs">{JSON.stringify(breadcrumbs)}</span>
+      <span data-testid="title">{String(title.type)}</span>
+    </>
+  );
+};
+
+describe("AdministrationProvider", () => {
+  it("publishes an empty breadcrumb and the administration title when no section is given", () => {
+    render(
+      <AdministrationProvider>
+        <SharedProbe />
+      </AdministrationProvider>,
+    );
+
+    expect(screen.getByTestId("breadcrumbs")).toHaveTextContent("[]");
+    expect(screen.getByTestId("title")).toHaveTextContent("administration.title");
+  });
+
+  it("publishes the section as an unlinked breadcrumb and as the title", () => {
+    render(
+      <AdministrationProvider section="Calendars">
+        <SharedProbe />
+      </AdministrationProvider>,
+    );
+
+    expect(screen.getByTestId("breadcrumbs")).toHaveTextContent(JSON.stringify([{ name: "Calendars" }]));
+    expect(screen.getByTestId("title")).toHaveTextContent("Calendars");
+  });
+});

package/src/features/ai-connection/ai-connection.module.ts

@@ -1,3 +1,4 @@
+import { PlugZapIcon } from "lucide-react";
 import { ModuleFactory } from "../../permissions";
 import { AiConnection } from "./data";
 
@@ -11,4 +12,5 @@ export const AiConnectionModule = (factory: ModuleFactory) =>
     pageUrl: "/administration/ai-connections",
     model: AiConnection,
     moduleId: "7d3f2b8a-91c4-4e6d-a5f0-2c9b8e4d1a37",
+    icon: PlugZapIcon,
   });

package/src/features/ai-connection/contexts/AiConnectionsContext.tsx

@@ -3,18 +3,9 @@
 import { useTranslations } from "next-intl";
 import { createContext, ReactNode, useCallback, useContext, useEffect, useMemo, useState } from "react";
 import { SharedProvider } from "../../../contexts";
-import { usePageUrlGenerator } from "../../../hooks";
 import { BreadcrumbItemData } from "../../../interfaces";
 import { AiConnectionEnvDefaults, AiConnectionInterface, AiConnectionService, AiProviderRegistry } from "../data";
 
-/**
- * Route the breadcrumb links back to. Constant rather than a prop for the same
- * reason TOKEN_USAGE_ADMIN_PAGE_URL is
- * (features/tokenusage/contexts/TokenUsageAdminContext.tsx): the breadcrumb is
- * built here, not by the consuming app, and the path convention IS the contract.
- */
-export const AI_CONNECTIONS_ADMIN_PAGE_URL = "/administration/ai-connections";
-
 export interface AiConnectionsContextType {
   /** Every connection, both scopes — the cards filter per scope themselves. */
   connections: AiConnectionInterface[];
@@ -57,7 +48,6 @@ export type AiConnectionsProviderProps = {
  */
 export const AiConnectionsProvider = ({ children }: AiConnectionsProviderProps) => {
   const t = useTranslations();
-  const generateUrl = usePageUrlGenerator();
 
   const [connections, setConnections] = useState<AiConnectionInterface[]>([]);
   const [providerRegistry, setProviderRegistry] = useState<AiProviderRegistry>({});
@@ -94,12 +84,9 @@ export const AiConnectionsProvider = ({ children }: AiConnectionsProviderProps)
     [connections, providerRegistry, envDefaults, scopeCompanyId, refresh, isLoading],
   );
 
-  const breadcrumbs: BreadcrumbItemData[] = [
-    {
-      name: t("ai_connections.admin.title"),
-      href: generateUrl({ page: AI_CONNECTIONS_ADMIN_PAGE_URL }),
-    },
-  ];
+  // Unlinked: this provider only serves the AI-connections page, so the root is
+  // always the current page.
+  const breadcrumbs: BreadcrumbItemData[] = [{ name: t("ai_connections.admin.title") }];
 
   return (
     <SharedProvider value={{ breadcrumbs, title: { type: t("ai_connections.admin.title") } }}>

package/src/features/ai-connection/contexts/__tests__/AiConnectionsContext.spec.tsx

@@ -0,0 +1,33 @@
+import { render, screen, waitFor } from "@testing-library/react";
+import { describe, expect, it, vi } from "vitest";
+import { useSharedContext } from "../../../../contexts/SharedContext";
+import { AiConnectionService } from "../../data";
+import { AiConnectionsProvider } from "../AiConnectionsContext";
+
+vi.mock("../../data", () => ({
+  AiConnectionService: {
+    listConnections: vi.fn(async () => ({ connections: [], providerRegistry: {}, envDefaults: {} })),
+  },
+}));
+
+// next-intl is globally mocked in vitest.setup.ts with `t = (key) => key`, so a
+// translated label renders as its own key.
+function BreadcrumbProbe() {
+  const { breadcrumbs } = useSharedContext();
+  return <span data-testid="breadcrumbs">{JSON.stringify(breadcrumbs)}</span>;
+}
+
+describe("AiConnectionsContext", () => {
+  it("publishes the page as an unlinked breadcrumb, since it is the current page", async () => {
+    render(
+      <AiConnectionsProvider>
+        <BreadcrumbProbe />
+      </AiConnectionsProvider>,
+    );
+
+    await waitFor(() => expect(AiConnectionService.listConnections).toHaveBeenCalled());
+    expect(screen.getByTestId("breadcrumbs")).toHaveTextContent(
+      JSON.stringify([{ name: "ai_connections.admin.title" }]),
+    );
+  });
+});

package/src/features/analytics/__tests__/server-entry.spec.tsx

@@ -35,7 +35,8 @@ describe("analytics server entry", () => {
 
     expect(mocks.hasRole).toHaveBeenCalledWith("role-admin");
     expect(element.type).toBe(AnalyticsAdminPageContainer);
-    expect(element.props).toEqual({ pageUrl: "/admin/stats", sessionPageUrl: "/admin/stats/sessions" });
+    // `pageUrl` is deprecated and no longer forwarded: the dashboard breadcrumb is unlinked.
+    expect(element.props).toEqual({ sessionPageUrl: "/admin/stats/sessions" });
   });
 
   it("AnalyticsAdminPage redirects to `/` by default without the role", async () => {

package/src/features/analytics/analytics.modules.ts

@@ -1,3 +1,4 @@
+import { ChartNoAxesCombinedIcon } from "lucide-react";
 import { ModuleFactory, ModuleWithPermissions } from "../../permissions";
 import { AnalyticsBreakdown } from "./data/analytics-breakdown";
 import { AnalyticsEvent } from "./data/analytics-event";
@@ -22,9 +23,25 @@ import { AnalyticsTimeline } from "./data/analytics-timeline";
 export const analyticsModules = (factory: ModuleFactory) =>
   ({
     AnalyticsEvent: factory({ name: "analytics/events", model: AnalyticsEvent }),
-    AnalyticsSummary: factory({ name: "analytics/administration/summary", model: AnalyticsSummary }),
-    AnalyticsTimeline: factory({ name: "analytics/administration/timeline", model: AnalyticsTimeline }),
-    AnalyticsBreakdown: factory({ name: "analytics/administration/breakdown", model: AnalyticsBreakdown }),
-    AnalyticsSession: factory({ name: "analytics/administration/sessions", model: AnalyticsSession }),
-    AnalyticsPageView: factory({ name: "pageviews", model: AnalyticsPageView }),
+    AnalyticsSummary: factory({
+      name: "analytics/administration/summary",
+      model: AnalyticsSummary,
+      icon: ChartNoAxesCombinedIcon,
+    }),
+    AnalyticsTimeline: factory({
+      name: "analytics/administration/timeline",
+      model: AnalyticsTimeline,
+      icon: ChartNoAxesCombinedIcon,
+    }),
+    AnalyticsBreakdown: factory({
+      name: "analytics/administration/breakdown",
+      model: AnalyticsBreakdown,
+      icon: ChartNoAxesCombinedIcon,
+    }),
+    AnalyticsSession: factory({
+      name: "analytics/administration/sessions",
+      model: AnalyticsSession,
+      icon: ChartNoAxesCombinedIcon,
+    }),
+    AnalyticsPageView: factory({ name: "pageviews", model: AnalyticsPageView, icon: ChartNoAxesCombinedIcon }),
   }) satisfies Record<string, ModuleWithPermissions>;

package/src/features/analytics/components/AnalyticsAdminContainer.tsx

@@ -2,6 +2,7 @@
 
 import { useTranslations } from "next-intl";
 import { RoundPageContainer } from "../../../components";
+import { Modules } from "../../../core";
 import { Card, CardContent, CardHeader, CardTitle } from "../../../shadcnui";
 import { useAnalyticsAdmin } from "../contexts/AnalyticsAdminContext";
 import { AnalyticsAdminTiles } from "./AnalyticsAdminTiles";
@@ -33,7 +34,7 @@ export function AnalyticsAdminContainer() {
 
   if (error) {
     return (
-      <RoundPageContainer fullWidth forceHeader>
+      <RoundPageContainer module={Modules.AnalyticsSummary} fullWidth forceHeader>
         <div className="p-4">
           <Card>
             <CardContent>
@@ -47,10 +48,10 @@ export function AnalyticsAdminContainer() {
 
   // Loading renders nothing in the body: the title bar (with the filter bar) is
   // already mounted, so a spinner would only make the controls jump on arrival.
-  if (isLoading) return <RoundPageContainer fullWidth forceHeader />;
+  if (isLoading) return <RoundPageContainer module={Modules.AnalyticsSummary} fullWidth forceHeader />;
 
   return (
-    <RoundPageContainer fullWidth forceHeader>
+    <RoundPageContainer module={Modules.AnalyticsSummary} fullWidth forceHeader>
       <div className="flex w-full flex-col gap-4 p-4">
         <div className="flex flex-col gap-2">
           <AnalyticsAdminTiles summary={summary} />

package/src/features/analytics/components/AnalyticsAdminPageContainer.tsx

@@ -4,7 +4,10 @@ import { AnalyticsAdminProvider } from "../contexts/AnalyticsAdminContext";
 import { AnalyticsAdminContainer } from "./AnalyticsAdminContainer";
 
 type Props = {
-  /** Route the breadcrumb links back to. Defaults to the provider's default. */
+  /**
+   * @deprecated No longer read: the dashboard is the current page, so its
+   * breadcrumb is not linked. Kept so existing callers still compile.
+   */
   pageUrl?: string;
   /** Base route of the journey page each session row links to. */
   sessionPageUrl?: string;
@@ -16,9 +19,9 @@ type Props = {
  * the `RoundPageContainer`; the provider sits above it because it publishes the
  * filter bar into the title bar.
  */
-export function AnalyticsAdminPageContainer({ pageUrl, sessionPageUrl }: Props) {
+export function AnalyticsAdminPageContainer({ sessionPageUrl }: Props) {
   return (
-    <AnalyticsAdminProvider pageUrl={pageUrl} sessionPageUrl={sessionPageUrl}>
+    <AnalyticsAdminProvider sessionPageUrl={sessionPageUrl}>
       <AnalyticsAdminContainer />
     </AnalyticsAdminProvider>
   );

package/src/features/analytics/components/AnalyticsJourneyContainer.tsx

@@ -4,6 +4,7 @@ import { useLocale, useTranslations } from "next-intl";
 import { useEffect, useMemo, useState } from "react";
 import { DetailField, RoundPageContainer } from "../../../components";
 import { SharedProvider } from "../../../contexts";
+import { Modules } from "../../../core";
 import { usePageUrlGenerator } from "../../../hooks";
 import { BreadcrumbItemData } from "../../../interfaces";
 import {
@@ -125,7 +126,7 @@ export function AnalyticsJourneyContainer({ sessionId, pageUrl = ANALYTICS_ADMIN
   const renderBody = () => {
     if (error) {
       return (
-        <RoundPageContainer fullWidth forceHeader>
+        <RoundPageContainer module={Modules.AnalyticsSession} fullWidth forceHeader>
           <div className="p-4">
             <Card>
               <CardContent>
@@ -138,14 +139,14 @@ export function AnalyticsJourneyContainer({ sessionId, pageUrl = ANALYTICS_ADMIN
     }
 
     // Loading renders nothing in the body, as on the dashboard.
-    if (isLoading || !session) return <RoundPageContainer fullWidth forceHeader />;
+    if (isLoading || !session) return <RoundPageContainer module={Modules.AnalyticsSession} fullWidth forceHeader />;
 
     const source = [session.utmSource ?? session.referrerHost, session.utmMedium, session.utmCampaign]
       .filter(Boolean)
       .join(" / ");
 
     return (
-      <RoundPageContainer fullWidth forceHeader>
+      <RoundPageContainer module={Modules.AnalyticsSession} fullWidth forceHeader>
         <div className="flex w-full flex-col gap-4 p-4">
           <Card data-testid="analytics-journey-attribution">
             <CardContent className="flex flex-col gap-3">

package/src/features/analytics/components/__tests__/AnalyticsJourneyContainer.spec.tsx

@@ -22,6 +22,17 @@ vi.mock("../../../../components", async (importOriginal) => {
   };
 });
 
+// The container hands `Modules.AnalyticsSession` to the (mocked) page chrome for
+// its header icon; no app bootstrap registers modules here, so the registry
+// lookup would throw.
+vi.mock("../../../../core", async (importOriginal) => {
+  const actual = await importOriginal<typeof import("../../../../core")>();
+  return {
+    ...actual,
+    Modules: new Proxy({}, { get: () => undefined }),
+  };
+});
+
 beforeAll(() => {
   configureI18n({
     useRouter: () => ({

package/src/features/analytics/components/__tests__/AnalyticsPageContainers.spec.tsx

@@ -27,6 +27,17 @@ vi.mock("../../../../components", async (importOriginal) => {
   };
 });
 
+// The containers pass Modules.AnalyticsSummary / AnalyticsSession for the
+// header icon; no app bootstrap registers modules here, so the registry lookup
+// would throw.
+vi.mock("../../../../core", async (importOriginal) => {
+  const actual = await importOriginal<typeof import("../../../../core")>();
+  return {
+    ...actual,
+    Modules: new Proxy({}, { get: () => undefined }),
+  };
+});
+
 beforeAll(() => {
   configureI18n({
     useRouter: () => ({
@@ -81,7 +92,7 @@ describe("AnalyticsAdminPageContainer", () => {
   });
 
   it("loads the dashboard and links each session under the given sessionPageUrl", async () => {
-    render(<AnalyticsAdminPageContainer pageUrl="/admin/stats" sessionPageUrl="/admin/stats/sessions" />);
+    render(<AnalyticsAdminPageContainer sessionPageUrl="/admin/stats/sessions" />);
 
     expect(await screen.findByTestId("analytics-session-link-s1")).toHaveAttribute("href", "/admin/stats/sessions/s1");
     expect(AnalyticsAdminService.getSummary).toHaveBeenCalledTimes(1);

package/src/features/analytics/components/server/AnalyticsAdminPage.tsx

@@ -10,7 +10,10 @@ import { ServerSession } from "../../../../server";
 type Props = {
   /** Role id the viewer must hold. The package has no role constants, so the host app passes its own. */
   adminRoleId: string;
-  /** Route the breadcrumb links back to. */
+  /**
+   * @deprecated No longer read: the dashboard is the current page, so its
+   * breadcrumb is not linked. Kept so existing callers still compile.
+   */
   pageUrl?: string;
   /** Base route of the journey page each session row links to. */
   sessionPageUrl?: string;
@@ -23,10 +26,10 @@ type Props = {
  *
  * Reads the session: render inside a `<Suspense>` boundary (Cache Components).
  */
-export async function AnalyticsAdminPage({ adminRoleId, pageUrl, sessionPageUrl, deniedRedirect = "/" }: Props) {
+export async function AnalyticsAdminPage({ adminRoleId, sessionPageUrl, deniedRedirect = "/" }: Props) {
   if (!(await ServerSession.hasRole(adminRoleId))) {
     redirect(deniedRedirect);
   }
 
-  return <AnalyticsAdminPageContainer pageUrl={pageUrl} sessionPageUrl={sessionPageUrl} />;
+  return <AnalyticsAdminPageContainer sessionPageUrl={sessionPageUrl} />;
 }