create-ailk 0.18.0 → 0.19.1

package/templates/apps/web/app/[locale]/(authed)/account/account-frame.tsx

@@ -1,4 +1,4 @@
-"use client"; // the shell owns collapse state; sign-out is a client-side Better Auth call + router navigation
+"use client"; // the shell owns collapse state; sign-out is a client-side hub-session POST or Better Auth call + router navigation
 
 /**
  * AccountFrame — the account area's SignedInShell composition (#7430, spec
@@ -7,8 +7,28 @@
  * what only the client has: the current path and the sign-out call.
  *
  * `createAuthClient()` at module scope is the `sign-in-gate.tsx` idiom. Log
- * out ends the session, then navigates home and refreshes so every RSC under
- * the `(authed)` gate re-reads the (now absent) session.
+ * out ends the session, then navigates and refreshes so every RSC under the
+ * `(authed)` gate re-reads the (now absent) session.
+ *
+ * #8465 — a spoke that signs in through its hub (#7894) holds its session in
+ * a sealed cookie only `POST /api/hub-session/sign-out` clears (it also
+ * revokes the credential at the hub); Better Auth's `signOut()` hits a route
+ * that spoke does not serve. The server layout reads the generated seam's
+ * `SIGN_OUT_ENDPOINT` and hands it down as `signOutEndpoint` — the signed-in
+ * Nav's own pattern, because a client component cannot import the seam. With
+ * an endpoint, Log out POSTs it and navigates to the sign-in page only when
+ * the answer was 2xx; a refused or failed sign-out (a 403 on an Origin
+ * mismatch, a 5xx, a network error) leaves the cookie valid, so the frame
+ * stays on the page and says so, never routing away with a live session.
+ * Without an endpoint (a hub or a single site) nothing changes: Better Auth,
+ * then home.
+ *
+ * #8231 D1, as amended by #8334 — the rail reads Inbox, a Library group
+ * holding Artifacts, and the footer's ?. Settings is not a rail row: it lives
+ * in the avatar menu, as on the Admin (the operator's 2026-10-11 Phase 6
+ * read), so no row is selected on a Settings page. A row is selected on its
+ * own path and the paths under it; the Inbox also on an item's page
+ * (/account/inbox/…).
  *
  * #7469 (DESIGN.md §2b as amended 2026-10-01, matching the Admin's §5.15) —
  * below `md` the bar reads avatar · Inbox · For you · Help, with no count
@@ -16,6 +36,27 @@
  * frame reads the URL and hands the tab to the page (`PhoneTabProvider`),
  * which shows one panel per tab. At `md`+ nothing changes — For you stays
  * the aside.
+ *
+ * #8233 D4 — on the Inbox route itself the phone top bar's trailing cell is
+ * an empty node handed to the page (`NewMenuSlotProvider`), which portals
+ * its compact New into it; every other path has no trailing cell.
+ *
+ * #8337 D4 — the `data-ailk-shell="account"` marker on a `display: contents`
+ * wrapper around `SignedInShell`: the layout's `shell-vars` reset
+ * (`:root:has([data-ailk-shell])`) matches only while this frame is in the
+ * document, so the Admin's colour tokens reach the shell and everything it
+ * portals to `document.body` (the avatar/help menus, For-you, New, the phone
+ * sheets) without the wrapper adding a box — `contents` keeps the shell's own
+ * layout (`min-h-screen`) unchanged.
+ *
+ * #8430 — the User settings routes take the screen as the Admin's do: the
+ * frame renders the page alone, inside the marker only — no rail, no Inbox,
+ * no AppShell — and the page draws the shared `AccountSettingsFrame` in its
+ * standalone shape, its own column flush and full height with Back to the
+ * Inbox at its top. The marker stays, so the Admin's tokens still reach the
+ * settings in both modes. Which routes those are is `isSettingsPath` — an
+ * exact match on `SETTINGS_ROUTES`, the list the settings tabs read too
+ * (#8441), so a sub-route stays inside the shell.
  */
 
 import {
@@ -31,13 +72,15 @@ import {
   type SignedInShellProps,
 } from "@working-theory/templates/shells";
 import { localeHref } from "@working-theory/ui";
-import { Link } from "@working-theory/ui/primitives";
+import { InlineMessage, Link } from "@working-theory/ui/primitives";
 import { createAuthClient } from "better-auth/react";
-import { Inbox, Settings, Sparkles } from "lucide-react";
+import { Inbox, Layers, Settings, Sparkles } from "lucide-react";
 import { usePathname, useRouter, useSearchParams } from "next/navigation";
-import type { ReactElement, ReactNode } from "react";
+import { useState, type ReactElement, type ReactNode } from "react";
 
 import { ACCOUNT_PERSIST_KEY } from "./for-you-labels";
+import { NewMenuSlotProvider } from "./new-menu";
+import { isSettingsPath } from "./settings-frame";
 
 const authClient = createAuthClient();
 
@@ -51,6 +94,11 @@ export interface AccountFrameLabels extends SignedInShellLabels {
   /** The phone's For you tab label (#7469). */
   forYou: string;
   settings: string;
+  /** The rail's Library group heading (#8231 D1). */
+  library: string;
+  artifacts: string;
+  /** Shown when a hub sign-out is refused or fails (#8465). */
+  signOutFailed: string;
 }
 
 export interface AccountFrameProps {
@@ -61,6 +109,14 @@ export interface AccountFrameProps {
   helpItems: NonNullable<SignedInShellProps["helpItems"]>;
   forYou: { items: readonly ForYouItem[]; now: string; labels: ForYouLabels };
   labels: AccountFrameLabels;
+  /**
+   * Where Log out POSTs on a spoke that signs in through its hub (the seam's
+   * `SIGN_OUT_ENDPOINT`, read by the server layout). Absent or `null`: Better
+   * Auth's client signs out (#8465).
+   */
+  signOutEndpoint?: string | null;
+  /** Where Log out lands after a hub sign-out; defaults to the site home. */
+  signInHref?: string;
   children: ReactNode;
 }
 
@@ -71,23 +127,61 @@ export function AccountFrame({
   helpItems,
   forYou,
   labels,
+  signOutEndpoint,
+  signInHref,
   children,
 }: AccountFrameProps): ReactElement {
   const router = useRouter();
   const pathname = usePathname();
   const searchParams = useSearchParams();
   const [dismissed] = useForYouDismissed(ACCOUNT_PERSIST_KEY);
+  const [topBarActions, setTopBarActions] = useState<HTMLElement | null>(null);
+  const [signOutFailed, setSignOutFailed] = useState(false);
 
   const homeHref = localeHref(locale, "/");
   const inboxHref = localeHref(locale, "/account");
+  const artifactsHref = localeHref(locale, "/account/artifacts");
+  const settingsHref = localeHref(locale, "/account/general");
+  const within = (href: string): boolean => pathname === href || pathname.startsWith(`${href}/`);
+
+  // #8430 — a Settings page draws its own standalone frame: the marker only.
+  if (isSettingsPath(pathname, locale)) {
+    return (
+      <div className="contents" data-ailk-shell="account" data-testid="account-shell">
+        {children}
+      </div>
+    );
+  }
+
   // The Inbox row stays selected on an item's own page (/account/inbox/…).
   const onInbox = pathname === inboxHref || pathname.startsWith(`${inboxHref}/inbox/`);
+  const activePath = onInbox ? inboxHref : within(artifactsHref) ? artifactsHref : undefined;
   // #7469 — the phone tab: For you only on the Inbox route itself.
   const phoneTab =
     pathname === inboxHref && searchParams?.get("tab") === FOR_YOU_TAB ? FOR_YOU_TAB : "inbox";
+  // #8233 D4 — the Inbox route's New lives in the top bar; nowhere else.
+  const onInboxRoute = pathname === inboxHref;
 
   const onSignOut = (): void => {
     void (async () => {
+      setSignOutFailed(false);
+      if (signOutEndpoint) {
+        // A refused or failed sign-out leaves the session cookie valid, so the
+        // page must not pretend otherwise by navigating away (#8465).
+        try {
+          const res = await fetch(signOutEndpoint, { method: "POST", credentials: "same-origin" });
+          if (!res.ok) {
+            setSignOutFailed(true);
+            return;
+          }
+        } catch {
+          setSignOutFailed(true);
+          return;
+        }
+        router.push(signInHref ?? homeHref);
+        router.refresh();
+        return;
+      }
       try {
         await authClient.signOut();
       } catch {
@@ -99,7 +193,16 @@ export function AccountFrame({
     })();
   };
 
-  const { home, inbox, forYou: forYouLabel, settings, ...shellLabels } = labels;
+  const {
+    home,
+    inbox,
+    forYou: forYouLabel,
+    settings,
+    library,
+    artifacts,
+    signOutFailed: signOutFailedLabel,
+    ...shellLabels
+  } = labels;
 
   // D13 — the logo only: no box, no name, no popover; it links home. The
   // same link is the phone top bar's leading cell.
@@ -118,52 +221,83 @@ export function AccountFrame({
   const forYouLeft = forYou.items.filter((item) => !dismissed.has(item.id));
 
   return (
-    <SignedInShell
-      persistKey={ACCOUNT_PERSIST_KEY}
-      locale={locale}
-      navItems={[{ labelKey: "inbox", label: inbox, path: "/account", icon: Inbox }]}
-      tabItems={[
-        { key: "inbox", label: inbox, href: inboxHref, icon: Inbox, isActive: onInbox && phoneTab === "inbox" },
-        {
-          key: FOR_YOU_TAB,
-          label: forYouLabel,
-          href: `${inboxHref}?tab=${FOR_YOU_TAB}`,
-          icon: Sparkles,
-          isActive: phoneTab === FOR_YOU_TAB,
-        },
-      ]}
-      {...(onInbox ? { activePath: inboxHref } : {})}
-      user={user}
-      userMenuItems={[
-        {
-          id: "settings",
-          label: settings,
-          icon: Settings,
-          href: localeHref(locale, "/account/settings"),
-        },
-      ]}
-      onSignOut={onSignOut}
-      userDisplay="avatar"
-      accountMenuLayout="card"
-      helpItems={helpItems}
-      railHeader={logoLink}
-      topBarStart={logoLink}
-      {...(forYouLeft.length > 0
-        ? {
-            aside: (
-              <ForYouPanel
-                items={forYouLeft}
-                persistKey={ACCOUNT_PERSIST_KEY}
-                labels={forYou.labels}
-                locale={locale}
-                now={forYou.now}
-              />
-            ),
-          }
-        : {})}
-      labels={shellLabels}
-    >
-      <PhoneTabProvider tab={phoneTab}>{children}</PhoneTabProvider>
-    </SignedInShell>
+    <div className="contents" data-ailk-shell="account" data-testid="account-shell">
+      <SignedInShell
+        persistKey={ACCOUNT_PERSIST_KEY}
+        locale={locale}
+        navItems={[{ labelKey: "inbox", label: inbox, path: "/account", icon: Inbox }]}
+        railGroups={[
+          {
+            key: "library",
+            heading: library,
+            items: [
+              { labelKey: "artifacts", label: artifacts, path: "/account/artifacts", icon: Layers },
+            ],
+          },
+        ]}
+        tabItems={[
+          { key: "inbox", label: inbox, href: inboxHref, icon: Inbox, isActive: onInbox && phoneTab === "inbox" },
+          {
+            key: FOR_YOU_TAB,
+            label: forYouLabel,
+            href: `${inboxHref}?tab=${FOR_YOU_TAB}`,
+            icon: Sparkles,
+            isActive: phoneTab === FOR_YOU_TAB,
+          },
+        ]}
+        {...(activePath ? { activePath } : {})}
+        user={user}
+        userMenuItems={[
+          {
+            id: "settings",
+            label: settings,
+            icon: Settings,
+            href: settingsHref,
+          },
+        ]}
+        onSignOut={onSignOut}
+        userDisplay="avatar"
+        accountMenuLayout="card"
+        helpItems={helpItems}
+        railHeader={logoLink}
+        topBarStart={logoLink}
+        {...(onInboxRoute
+          ? {
+              topBarEnd: (
+                <span
+                  ref={setTopBarActions}
+                  className="flex items-center"
+                  data-testid="account-top-bar-actions"
+                />
+              ),
+            }
+          : {})}
+        {...(forYouLeft.length > 0
+          ? {
+              aside: (
+                <ForYouPanel
+                  items={forYouLeft}
+                  persistKey={ACCOUNT_PERSIST_KEY}
+                  labels={forYou.labels}
+                  locale={locale}
+                  now={forYou.now}
+                />
+              ),
+            }
+          : {})}
+        labels={shellLabels}
+      >
+        {signOutFailed ? (
+          <InlineMessage variant="error" className="px-4 pt-3" data-testid="account-sign-out-error">
+            {signOutFailedLabel}
+          </InlineMessage>
+        ) : null}
+        <PhoneTabProvider tab={phoneTab}>
+          <NewMenuSlotProvider node={onInboxRoute ? topBarActions : null}>
+            {children}
+          </NewMenuSlotProvider>
+        </PhoneTabProvider>
+      </SignedInShell>
+    </div>
   );
 }

package/templates/apps/web/app/[locale]/(authed)/account/actions.ts

@@ -13,62 +13,41 @@
  * navigation telemetry for a Server Action, which is inherent to any hosted-
  * portal handoff, not something this design avoids).
  *
- * CSRF protection depends on the render (`forwardedCookieInit()` branches on
- * `SIGN_IN_PROVIDER`, `lib/server-api.ts`):
+ * Where the portal comes from is `billing-source.ts`'s `billingPortalUrl()`
+ * (#8217, spec docs/specs/8217-spoke-billing-spoke-checkout-reads-spec.md
+ * D10, D11), the account tree's billing seam — this file tests no topology:
  *
- * - Hub render (`SIGN_IN_PROVIDER === "site"`): `forwardedCookieInit()`
- *   forwards the incoming request's real `Origin` alongside the Better Auth
- *   cookie, and `apps/api`'s `authPreHandler` requires that `Origin` to be one
- *   of its trusted origins on this cookie-authenticated mutating request
- *   (`isOriginAllowed`, `middleware/auth.ts`). The portal-session route also
- *   accepts only a cookie session (`request.sessionTenant`), never a bearer
- *   (D3.1/D5).
- * - Spoke render, sign-in through the hub (`SIGN_IN_PROVIDER === "hub"`):
- *   there is no cookie and no `Origin` forward. `hubSessionInit()`
- *   (`lib/hub-session/session.ts`) sends the sealed session's credential as
- *   `Authorization: Bearer`, and `authPreHandler` skips its origin check for a
- *   bearer. This action does not complete on a spoke: the hub admits a hub
- *   credential on four read routes only (`HUB_SESSION_ROUTES`), so
- *   `POST /v1/subscription/portal-session` is a 404 there, and Settings hides
- *   the Billing button because `subscription.me()` is a 404 too.
+ * - A single site: the hub's `POST /v1/subscription/portal-session`, with
+ *   the caller's cookie and real `Origin` forwarded, which `apps/api`'s
+ *   `authPreHandler` checks against its trusted origins on this
+ *   cookie-authenticated mutating request; the route accepts only a cookie
+ *   session, never a bearer (D3.1/D5).
+ * - A spoke that signs in through its hub and carries commerce: the spoke's
+ *   own Stripe key, for the customer its hub vouches for — read server-side
+ *   by the rendered billing action, never from this action's input.
+ * - A spoke without that action: no portal, so nothing to redirect to.
  *
- * `redirect()` throws `NEXT_REDIRECT` by design — called OUTSIDE the
- * try/catch (mirrors `checkout.actions.ts`'s idiom) so the catch never
+ * It takes no argument, so nothing a browser sends can name a customer.
+ * Next's Server Action origin check applies as before.
+ *
+ * `redirect()` throws `NEXT_REDIRECT` by design — called OUTSIDE any
+ * try/catch (mirrors `checkout.actions.ts`'s idiom) so a catch never
  * swallows the navigation this action exists to perform.
  */
 
-import { subscription } from "@working-theory/api-client";
 import { redirect } from "next/navigation";
 
-import { forwardedCookieInit } from "../../../../lib/server-api";
-
-/** PII-safe error identity for logs (the `checkout.actions.ts` idiom: `err.name` only). */
-function errorName(err: unknown): string {
-  return err instanceof Error ? err.name : "unknown";
-}
+import { billingPortalUrl } from "./billing-source";
 
 /**
  * Opens the caller's own Stripe billing-portal session and redirects there.
- * Never returns on success. On a non-2xx or thrown error, logs a PII-safe
- * status/name and redirects back to `/account` unchanged (the banner's
- * action-unavailable posture — there is no error UI slot on this button).
+ * Never returns on success. With no portal (a failed mint, logged PII-safe by
+ * `billingPortalUrl`, or none to open), it returns and the page stays as it
+ * is — there is no error UI slot on this button.
  */
 export async function openBillingPortal(): Promise<void> {
-  const init = await forwardedCookieInit();
-
-  let url: string;
-  try {
-    const res = await subscription.portalSession(init);
-    if (!res.ok) {
-      console.error(`[openBillingPortal] status=${res.status}`);
-      return;
-    }
-    url = res.body.url;
-  } catch (err) {
-    console.error(`[openBillingPortal] errName=${errorName(err)}`);
-    return;
-  }
-
-  // Outside the try/catch on purpose — see the file header.
+  const url = await billingPortalUrl();
+  if (url === null) return;
+  // Outside any try/catch on purpose — see the file header.
   redirect(url);
 }

package/templates/apps/web/app/[locale]/(authed)/account/artifacts/[pieceId]/page.tsx

@@ -0,0 +1,253 @@
+/**
+ * /[locale]/account/artifacts/[pieceId] — one artifact (#8234, spec
+ * docs/specs/8234-member-artifacts-account-artifacts-and-spec.md D4).
+ *
+ * RSC, inside the account layout's SignedInShell, through the shared
+ * `InboxItemFrame`: back to Artifacts, the title, the review state, and —
+ * once approved — "Approved <date>". The body lists every served version in
+ * the read's order, newest first: its number, its delivery date, its note,
+ * the member's own review when there is one, and one action — Download for a
+ * file (re-served by `versions/[version]/file/route.ts`, never shown inline
+ * on this origin) or View for an https link, opened in a new tab. A link
+ * that is not https renders no action. The read serves no deliverer (8169
+ * D6), so none is shown.
+ *
+ * Inside the latest version only: the review form while it is reviewable
+ * (not approved, no review yet — `isReviewable`), else the waiting line once
+ * the member asked for changes; nothing once approved. Nothing limits how
+ * often: every new latest version is reviewable again (#8140 disposition 3).
+ *
+ * The artifact is found the way the Inbox item page finds its item: the
+ * artifacts read runs (deduped per request) and `[pieceId]` is matched raw or
+ * decoded. No session, no match or a 404 read is a not-found; any other
+ * failed read renders the error row with Retry (spec 7423 D12). Titles,
+ * notes and comments render as text only.
+ */
+
+import { InboxItemFrame } from "@working-theory/templates/account";
+import { localeHref } from "@working-theory/ui";
+import { Button, Heading, Link, Text } from "@working-theory/ui/primitives";
+import {
+  isHttpsLink,
+  type MemberArtifact,
+  type MemberPieceVersion,
+} from "@working-theory/validation";
+import type { Metadata } from "next";
+import { notFound } from "next/navigation";
+import { getTranslations } from "next-intl/server";
+import type { ReactElement } from "react";
+
+import { accountSession, logReadFailure } from "../../account-data";
+import { ErrorRow } from "../../error-row";
+import {
+  artifactState,
+  isReviewable,
+  loadArtifacts,
+  STATE_KEYS,
+  versionFileHref,
+} from "../artifact-data";
+
+import { ReviewForm, type ReviewFormLabels } from "./review-form";
+
+interface Props {
+  params: Promise<{ locale: string; pieceId: string }>;
+}
+
+/**
+ * A segment, percent-decoded when it decodes — matched in both forms, so an
+ * id is found whether or not the framework already decoded it.
+ */
+function decodeSegment(segment: string): string {
+  try {
+    return decodeURIComponent(segment);
+  } catch {
+    return segment;
+  }
+}
+
+export async function generateMetadata({ params }: Props): Promise<Metadata> {
+  const { locale } = await params;
+  const t = await getTranslations({ locale, namespace: "account" });
+  return { title: t("artifacts.title") };
+}
+
+export default async function AccountArtifactPage({
+  params,
+}: Props): Promise<ReactElement> {
+  const { locale, pieceId: segment } = await params;
+  const t = await getTranslations({ locale, namespace: "account" });
+  const session = await accountSession(locale);
+  if (!session) notFound();
+
+  let artifacts: MemberArtifact[];
+  try {
+    artifacts = await loadArtifacts(session.ctx);
+  } catch (err) {
+    logReadFailure("artifacts", err);
+    return (
+      <div className="max-w-3xl py-8" data-testid="artifact">
+        <ErrorRow
+          message={t("artifacts.itemError")}
+          retryLabel={t("artifacts.retry")}
+        />
+      </div>
+    );
+  }
+
+  const artifact = artifacts.find(
+    (candidate) =>
+      candidate.id === segment || candidate.id === decodeSegment(segment),
+  );
+  if (!artifact) notFound();
+
+  const date = (iso: string): string =>
+    new Intl.DateTimeFormat(locale, {
+      dateStyle: "medium",
+      timeZone: "UTC",
+    }).format(new Date(iso));
+  const state = artifactState(artifact);
+  const reviewable = isReviewable(artifact);
+  const latest = artifact.versions[0]!.version;
+  const labels: ReviewFormLabels = {
+    heading: t("artifacts.review.heading", { version: latest }),
+    comment: t("artifacts.review.comment"),
+    commentHint: t("artifacts.review.commentHint"),
+    approve: t("artifacts.review.approve"),
+    requestChanges: t("artifacts.review.requestChanges"),
+    submitting: t("artifacts.review.submitting"),
+    sent: t("artifacts.review.sent"),
+    errors: {
+      comment_required: t("artifacts.review.errors.commentRequired"),
+      closed: t("artifacts.review.errors.closed"),
+      not_latest: t("artifacts.review.errors.notLatest"),
+      rate_limited: t("artifacts.review.errors.rateLimited"),
+      failed: t("artifacts.review.errors.failed"),
+    },
+  };
+
+  // The one action of a version: Download a file, View an https link, else none.
+  const versionAction = (v: MemberPieceVersion): ReactElement | null => {
+    if (v.file !== null) {
+      return (
+        <Button asChild variant="outline" className="max-md:min-h-11">
+          <Link
+            href={versionFileHref(locale, artifact.id, v.version)}
+            variant="bare"
+            data-testid={`artifact-version-${v.version}-download`}
+          >
+            {t("artifacts.download", { filename: v.file.filename })}
+          </Link>
+        </Button>
+      );
+    }
+    if (isHttpsLink(v.link)) {
+      return (
+        <Button asChild variant="outline" className="max-md:min-h-11">
+          <Link
+            href={v.link}
+            external
+            variant="bare"
+            data-testid={`artifact-version-${v.version}-view`}
+          >
+            {t("artifacts.view")}
+          </Link>
+        </Button>
+      );
+    }
+    return null;
+  };
+
+  return (
+    <InboxItemFrame
+      back={{
+        href: localeHref(locale, "/account/artifacts"),
+        label: t("artifacts.back"),
+      }}
+      caption={t("artifacts.caption")}
+      title={artifact.title}
+      status={t(STATE_KEYS[state])}
+      lead={
+        artifact.approvedAt !== null
+          ? t("artifacts.approvedOn", { date: date(artifact.approvedAt) })
+          : null
+      }
+      testIdPrefix="artifact"
+    >
+      <section className="mt-block-loose">
+        <Heading level={2} visual={4} className="!text-body font-semibold">
+          {t("artifacts.versions")}
+        </Heading>
+        <div className="mt-block-tight flex flex-col gap-4">
+          {artifact.versions.map((v) => {
+            const action = versionAction(v);
+            return (
+              <div
+                key={v.version}
+                className="rounded-md border-hairline border-(--color-border) p-card"
+                data-testid={`artifact-version-${v.version}`}
+              >
+                <Text as="p" variant="body" className="font-semibold">
+                  {t("artifacts.version", { version: v.version })}
+                </Text>
+                <Text as="p" variant="body-sm" tone="muted">
+                  {t("artifacts.deliveredOn", { date: date(v.deliveredAt) })}
+                </Text>
+                {v.note !== null ? (
+                  <Text
+                    as="p"
+                    variant="body-sm"
+                    className="mt-2 whitespace-pre-line"
+                  >
+                    {v.note}
+                  </Text>
+                ) : null}
+                {v.review !== null && v.reviewedAt !== null ? (
+                  <div className="mt-2">
+                    <Text as="p" variant="body-sm" tone="muted">
+                      {v.review === "approved"
+                        ? t("artifacts.reviewApproved", {
+                            date: date(v.reviewedAt),
+                          })
+                        : t("artifacts.reviewChangesRequested", {
+                            date: date(v.reviewedAt),
+                          })}
+                    </Text>
+                    {v.reviewComment !== null ? (
+                      <Text
+                        as="p"
+                        variant="body-sm"
+                        className="whitespace-pre-line"
+                      >
+                        {v.reviewComment}
+                      </Text>
+                    ) : null}
+                  </div>
+                ) : null}
+                {action !== null ? (
+                  <div className="mt-block-tight">{action}</div>
+                ) : null}
+                {v.version === latest && reviewable ? (
+                  <ReviewForm
+                    pieceId={artifact.id}
+                    version={v.version}
+                    labels={labels}
+                  />
+                ) : null}
+                {v.version === latest && state === "changes-requested" ? (
+                  <div
+                    className="mt-block-tight"
+                    data-testid="artifact-review-waiting"
+                  >
+                    <Text as="p" variant="body-sm">
+                      {t("artifacts.review.waiting")}
+                    </Text>
+                  </div>
+                ) : null}
+              </div>
+            );
+          })}
+        </div>
+      </section>
+    </InboxItemFrame>
+  );
+}