From 55b223d4ed27e66a76fea3b33c826c18bae09d63 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marc-Andr=C3=A9=20Bombeck?= <marc@bombeck.io>
Date: Thu, 12 Nov 2026 12:53:12 +0100
Subject: [PATCH] fix(onboarding): show the closing AI line as body copy

The done screen's "AI is optional" line is a sentence the person reads,
set in muted text, which the UI standards keep for meta. It is body
text now. Its test asserted the absence of strings deleted from every
bundle in the same change, so it could not fail; it now checks that the
line is not muted, that the screen links to the AI settings once, and
that no button sits beside the link.
---
 .../onboarding/__tests__/done-screen.test.tsx          | 10 +++++-----
 src/components/onboarding/done-screen.tsx              |  5 +----
 2 files changed, 6 insertions(+), 9 deletions(-)

diff --git a/src/components/onboarding/__tests__/done-screen.test.tsx b/src/components/onboarding/__tests__/done-screen.test.tsx
index f9113451e..946136252 100644
--- a/src/components/onboarding/__tests__/done-screen.test.tsx
+++ b/src/components/onboarding/__tests__/done-screen.test.tsx
@@ -117,11 +117,11 @@ describe("<DoneScreen> AI line", () => {
     );
     expect(line).not.toBeNull();
     expect(line![1]).toContain("AI is optional");
-    expect(line![1]).toContain('href="/settings/ai"');
-    // No sample briefing, no provider ladder, no one-tap offer.
-    expect(html).not.toContain("Example");
-    expect(html).not.toContain("Three ways to turn it on");
-    expect(html).not.toMatch(/<button[^>]*>[^<]*Turn on/);
+    // Body copy, not meta: the line is read, so it is not muted.
+    expect(line![0]).not.toContain("text-muted-foreground");
+    // One way on, the link, and no one-tap offer beside it.
+    expect(html.split('href="/settings/ai"')).toHaveLength(2);
+    expect(line![1]).not.toContain("<button");
   });
 
   it("is left out when AI cannot be set up here", () => {
diff --git a/src/components/onboarding/done-screen.tsx b/src/components/onboarding/done-screen.tsx
index c95709db9..ac15f61f3 100644
--- a/src/components/onboarding/done-screen.tsx
+++ b/src/components/onboarding/done-screen.tsx
@@ -123,10 +123,7 @@ export function DoneScreen({ state }: { state: OnboardingStateDto }) {
       </div>
 
       {showAiLine ? (
-        <p
-          data-slot="onboarding-ai-line"
-          className="text-muted-foreground max-w-md text-sm"
-        >
+        <p data-slot="onboarding-ai-line" className="max-w-md text-sm">
           {t("onboarding.done.aiOptional")}{" "}
           <InlineAction asChild size="sm" className="underline">
             <Link href="/settings/ai">{t("onboarding.done.aiSetupCta")}</Link>