Marc-André Bombeck
97f8cb4303
Remove the two per-document AI copy blocks in the document detail sheet: the external-egress notice and the "read this document" explainer. The settings-toggle honesty confirm already states the egress trade, so the per-document repeat is redundant. Drop the now-unused `egressExternal` prop and the `documents.ai.egressNotice` + `documents.ai.available` keys across all six locales. Keep the capability `egress` DTO field — the API still reports it. On /insights/recovery, the metric block rendered an outer heading plus an explainer above a tile whose own header repeated the same title. Move the explainer inside the tile as a `description` beneath its `TileHeader` and promote the tile title to a real `<h2>` (new `titleAs` affordance), so each metric shows one heading with the blurb under it and the h1->h2 outline stays nested. Applies to every metric the block renders.
e2e/
End-to-end browser tests. Playwright 1.60 drives a production build of the app against a real PostgreSQL, with @axe-core/playwright for accessibility assertions. This is the heaviest test tier; CI runs it on every PR, and locally it is worth running when a change touches a UI flow.
Running
pnpm e2e
The Playwright config (../playwright.config.ts) builds the app (next build) and brings up Postgres before the specs run; setup/global-setup.ts seeds the auth + fixture state. The suite is not part of the fast local gate (typecheck / lint / test) — it surfaces at PR CI, so expect responsive/wizard/nav reworks to flag here first.
Layout
setup/—global-setup.ts(auth + DB seeding),test-helpers.ts(shared login + navigation helpers).utils/— fixtures such asmock-dashboard-snapshot.ts.*.spec.ts— one spec per flow: auth/login/redirect, dashboard, charts, the medication-wizard cadences (medications-wizard-*), insights generation + scroll restoration, doctor report, settings, mobile-viewport consistency, locale switch, accessibility (a11y.spec.ts), and thev1427-*regression set.
Convention
Assert against stable data-* attributes (data-slot, data-display-step, root display attrs), not viewport-dependent visible text. Responsive sm:hidden UI changes break getByText on desktop while the underlying element is still present — anchoring on data attributes keeps the specs stable across breakpoints. See ../CLAUDE.md.