From a97204f12cde0d5d62e6066f6dc9120aa417a4d3 Mon Sep 17 00:00:00 2001
From: Waleed Latif <walif6@gmail.com>
Date: Fri, 30 Oct 2026 18:57:47 -0700
Subject: [PATCH 1/2] fix(docs): simplify selection and verify media
 interactions

---
 apps/docs/components/docs/navigation.tsx      |  3 +--
 apps/docs/components/docs/tokens.css          |  2 --
 apps/docs/migration/design-system.md          |  2 +-
 apps/docs/scripts/test-navigation-e2e.ts      | 14 +++++++++++++-
 apps/docs/scripts/test-search-postgres-e2e.ts | 13 ++++++++-----
 5 files changed, 23 insertions(+), 11 deletions(-)

diff --git a/apps/docs/components/docs/navigation.tsx b/apps/docs/components/docs/navigation.tsx
index 5e591564cbe..73d5eb4f693 100644
--- a/apps/docs/components/docs/navigation.tsx
+++ b/apps/docs/components/docs/navigation.tsx
@@ -68,8 +68,7 @@ function NavigationItem({ node, pathname, onNavigate }: NavigationItemProps) {
         <div
           className={cn(
             'group/branch flex min-h-[30px] items-center rounded-lg transition-colors hover:bg-[var(--docs-hover)] active:bg-[var(--docs-pressed)]',
-            isCurrent &&
-              'bg-[var(--docs-selected)] shadow-[var(--docs-current-edge)] hover:bg-[var(--docs-selected-hover)]'
+            isCurrent && 'bg-[var(--docs-selected)] hover:bg-[var(--docs-selected-hover)]'
           )}
         >
           <Link
diff --git a/apps/docs/components/docs/tokens.css b/apps/docs/components/docs/tokens.css
index caa7ec3f706..c20e86ffcbe 100644
--- a/apps/docs/components/docs/tokens.css
+++ b/apps/docs/components/docs/tokens.css
@@ -45,7 +45,6 @@
   --docs-hover: rgb(10 10 10 / 6%);
   --docs-selected: rgb(0 0 0 / 8%);
   --docs-selected-hover: var(--docs-selected);
-  --docs-current-edge: none;
   --docs-pressed: rgb(0 0 0 / 12%);
   --docs-link: #0066cc;
   --docs-prose-link: var(--docs-link);
@@ -134,7 +133,6 @@
   --docs-hover: #2b2b2b;
   --docs-selected: #333;
   --docs-selected-hover: var(--docs-pressed);
-  --docs-current-edge: inset 2px 0 0 var(--docs-boundary);
   --docs-pressed: #3a3a3a;
   --docs-link: #8abeff;
   --docs-prose-link: var(--docs-heading);
diff --git a/apps/docs/migration/design-system.md b/apps/docs/migration/design-system.md
index 31e08684519..bab96073e1d 100644
--- a/apps/docs/migration/design-system.md
+++ b/apps/docs/migration/design-system.md
@@ -22,7 +22,7 @@ Three-column reading begins at 1280px. At 1024–1279px the TOC becomes a compac
 | Code | Geist Mono | 13 / 22px | 400 | normal |
 | Keyboard | Geist | 11 / 14px | 400 | normal |
 
-Marketing calls to action use variable weight 450, matching the landing page. Heading weight remains 500; selection in the sidebar uses a neutral surface and, in dark mode, a 2px inset left edge without changing text color or weight.
+Marketing calls to action use variable weight 450, matching the landing page. Heading weight remains 500; selection in the sidebar uses a neutral surface in both themes without changing text color or weight. Keyboard focus retains its separate outline.
 
 The header keeps weight 400 in every navigation state at the user's request, matching the landing navigation and preventing label widths from shifting. Current-section ink changes without changing geometry. The shared wordmark uses the landing's visible SVG overflow and optical offset; header and footer links do not initiate native drag operations. Root vertical overscroll is disabled while normal page and contained reference scrolling remain available.
 
diff --git a/apps/docs/scripts/test-navigation-e2e.ts b/apps/docs/scripts/test-navigation-e2e.ts
index ee246b6b67d..c488242d522 100644
--- a/apps/docs/scripts/test-navigation-e2e.ts
+++ b/apps/docs/scripts/test-navigation-e2e.ts
@@ -543,7 +543,8 @@ try {
               path: join(directory, `image-${scene.variant}-${width}.png`),
               animations: 'disabled',
             })
-            await figure.getByRole('button').click()
+            const trigger = figure.getByRole('button')
+            await trigger.click()
             const viewer = page.getByRole('dialog')
             await expect(viewer.getByRole('img')).toBeVisible()
             await expect
@@ -589,6 +590,17 @@ try {
               })
             await page.keyboard.press('Escape')
             await expect(viewer).toHaveCount(0)
+            await expect(trigger).toBeFocused()
+            await trigger.press('Enter')
+            await expect(viewer).toBeVisible()
+            await page.mouse.click(1, 1)
+            await expect(viewer).toHaveCount(0)
+            await expect(trigger).toBeFocused()
+            await trigger.press('Enter')
+            await expect(viewer).toBeVisible()
+            await viewer.getByRole('button', { name: 'Close media viewer', exact: true }).click()
+            await expect(viewer).toHaveCount(0)
+            await expect(trigger).toBeFocused()
           } finally {
             await page.close()
           }
diff --git a/apps/docs/scripts/test-search-postgres-e2e.ts b/apps/docs/scripts/test-search-postgres-e2e.ts
index e0bf18fbb76..f1a5c86f2ef 100644
--- a/apps/docs/scripts/test-search-postgres-e2e.ts
+++ b/apps/docs/scripts/test-search-postgres-e2e.ts
@@ -400,16 +400,19 @@ async function verifyDatabase() {
   await check(
     'Citation-only changes are stale and repair without replacing embeddings',
     async () => {
+      const citations = await sql<
+        Array<{ chunk_id: string; source_link: string; embedding: string }>
+      >`SELECT chunk_id, source_link, embedding::text FROM docs_embeddings WHERE source_document = ${page.sourcePath} ORDER BY chunk_id`
+      assert(citations.length > 0)
       await sql`UPDATE docs_embeddings SET source_link = 'https://old.example/introduction' WHERE source_document = ${page.sourcePath}`
       assert.equal((await index('check', 'citation-stale', ['/introduction'])).exitCode, 1)
       const repaired = await index('publish', 'citation-repaired', ['/introduction'])
       assert.equal(repaired.exitCode, 0)
       assert.deepEqual(await fingerprint(), fresh)
-      const [row] = await sql<
-        Array<{ source_link: string; embedding: string }>
-      >`SELECT source_link, embedding::text FROM docs_embeddings WHERE source_document = ${page.sourcePath}`
-      assert.equal(row?.source_link, 'https://docs.sim.ai/introduction')
-      assert.equal(row?.embedding, vector)
+      const repairedCitations = await sql<
+        Array<{ chunk_id: string; source_link: string; embedding: string }>
+      >`SELECT chunk_id, source_link, embedding::text FROM docs_embeddings WHERE source_document = ${page.sourcePath} ORDER BY chunk_id`
+      assert.deepEqual(repairedCitations, citations)
       assert.equal((await index('check', 'citation-fresh', ['/introduction'])).exitCode, 0)
     }
   )

From 66fb724d01de997d7314796788eab38b6689abe8 Mon Sep 17 00:00:00 2001
From: Waleed Latif <walif6@gmail.com>
Date: Fri, 30 Oct 2026 19:13:01 -0700
Subject: [PATCH 2/2] fix(docs): wait for actionable lightbox backdrop in e2e

---
 apps/docs/scripts/test-navigation-e2e.ts | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/apps/docs/scripts/test-navigation-e2e.ts b/apps/docs/scripts/test-navigation-e2e.ts
index c488242d522..005ee37baf3 100644
--- a/apps/docs/scripts/test-navigation-e2e.ts
+++ b/apps/docs/scripts/test-navigation-e2e.ts
@@ -593,7 +593,9 @@ try {
             await expect(trigger).toBeFocused()
             await trigger.press('Enter')
             await expect(viewer).toBeVisible()
-            await page.mouse.click(1, 1)
+            await page
+              .locator('.docs-portal[data-state="open"][aria-hidden="true"]')
+              .click({ position: { x: 1, y: 1 } })
             await expect(viewer).toHaveCount(0)
             await expect(trigger).toBeFocused()
             await trigger.press('Enter')