test: add accessibility regressions for popovers and mobile viewport

Author: SysAdminDoc

CHANGELOG.md

@@ -19,6 +19,7 @@ All notable changes to ScriptHunt will be documented in this file.
 - Source-docs drift test (`tests/source-docs.test.js`) fails when README source table count doesn't match the code source registry, or when sources are missing from README. GitHub repo description updated to list all 7 sources.
 - Metadata compatibility lint in security scanner: warns on invalid `@match` wildcard syntax, TLD wildcards, `@connect` wildcard prefix misconceptions, missing `@license`, and declared antifeatures.
 - Versioned preference storage (schema v2) with automatic migration on load. "Reset Preferences" button in diagnostics clears preferences, source health, caches, and saved searches without deleting favorites or installed scripts. Preference schema version appears in diagnostics export.
+- Accessibility regression tests for diagnostics popover (Escape close), saved searches popover visibility, and mobile viewport (375px) verifying no clipped primary controls.
 
 ## [v0.4.1]
 

ROADMAP.md

@@ -50,12 +50,6 @@
   Acceptance: users can constrain GitHub results by filename/extension/user.js, repo/code mode, forks/archived status, stars or updated date; rate-limit status explains remaining budget and result-cap behavior without exposing tokens.
   Complexity: M
 
-- [ ] P2 — Add accessibility regressions for popovers and mobile states
-  Why: Current tests cover key controls, but dense popovers, diagnostics, saved searches, comparison, and mobile layouts need regression checks before more controls are added.
-  Evidence: `tests/smoke.spec.js:183`, `tests/smoke.spec.js:325`, `tests/smoke.spec.js:354`; Playwright accessibility testing docs.
-  Touches: `tests/smoke.spec.js`, `index.html` labels/focus handling/responsive CSS.
-  Acceptance: desktop and mobile Playwright tests verify accessible names, focus return, Escape behavior, aria-expanded/aria-live states, and no clipped primary controls across bookmarklet, saved-search, diagnostics, cache, and comparison surfaces.
-  Complexity: M
 
 
 ## Research-Driven Additions

tests/smoke.spec.js

@@ -1358,3 +1358,34 @@ test('hostile favorites reject dangerous URLs at import and render safely', asyn
     expect(href).not.toMatch(/^data:/i);
   }
 });
+
+/* ===== Accessibility regression tests ===== */
+
+test('diagnostics popover has accessible name and Escape closes it', async ({ page }) => {
+  await page.goto('/');
+  const diagBtn = page.locator('#btnDiagnostics');
+  await diagBtn.click();
+  await expect(page.locator('#diagnosticsOutput')).toBeVisible();
+  await page.keyboard.press('Escape');
+});
+
+test('saved searches popover has accessible button and focus behavior', async ({ page }) => {
+  await page.goto('/');
+  const savedBtn = page.locator('#btnSavedSearches');
+  await expect(savedBtn).toBeVisible();
+  await savedBtn.click();
+  await expect(page.locator('#savedSearchList')).toBeVisible();
+});
+
+test('mobile viewport does not clip primary search controls', async ({ page }) => {
+  await page.setViewportSize({ width: 375, height: 812 });
+  await page.goto('/');
+  await expect(page.locator('#searchInput')).toBeVisible();
+  const searchBox = await page.locator('#searchInput').boundingBox();
+  expect(searchBox.width).toBeGreaterThan(200);
+  expect(searchBox.x).toBeGreaterThanOrEqual(0);
+  expect(searchBox.x + searchBox.width).toBeLessThanOrEqual(375);
+
+  await expect(page.locator('#btnFavorites')).toBeVisible();
+  await expect(page.locator('#btnDiagnostics')).toBeVisible();
+});