Skip to content

feat: P0 fixes, a11y, ⌘K, Copy for LLM, contrast, ZIP, crawl Full render - #4

Merged
vgtray merged 5 commits into
mainfrom
feat/markdown-converter
Jul 1, 2026
Merged

feat: P0 fixes, a11y, ⌘K, Copy for LLM, contrast, ZIP, crawl Full render#4
vgtray merged 5 commits into
mainfrom
feat/markdown-converter

Conversation

@vgtray

@vgtray vgtray commented Jul 1, 2026

Copy link
Copy Markdown
Owner

Résumé

Suite de la PR #3 (mergée prématurément à l'état "crawl + settings + MV3 fixes"). Cette PR amène dans main les 5 commits restants : correctness P0, accessibilité, features (⌘K, Copy for LLM, contrast, ZIP) et le crawl Full pour les sites JS.

Commits

  • fix P0 : couleurs CSS modernes (oklch/display-p3/color-mix) → hex valide ; scans dupliqués ; spinner scan infini ; crawler timeout/Stop ; body/html + sous-arbres scannés
  • refactor : suppression code mort (grep-confirmé) + a11y (focus/aria/reduced-motion/AA) + logo unifié + raccourcis réels
  • feat : command palette ⌘K + Copy for LLM (page + design tokens + méta)
  • feat : contrast checker WCAG + export crawl en ZIP (fflate)
  • feat/fix : crawl mode Full (rendu réel en onglet pour SPA) + skip reasons visibles + robots toggle + same-site + fix scan périmé cross-site

Tests

  • 245 tests unitaires, tsc/eslint/build verts, npm audit 0 vuln
  • Smoke test Chrome (crawl Full sur SPA, ⌘K, contrast, persistance)

🤖 Generated with Claude Code

vgtray added 5 commits July 1, 2026 03:17
Cleanup (grep-confirmed 0-usage, no behavior change):
- Remove parseShadow, detectTypeScaleRatio, parseComputedStyles,
  getElementPath, ColorToken.rgb/hsl, OPEN_SIDE_PANEL enum, dead
  GET/SET_PREFERENCES service-worker handlers; dedupe isPixelLensElement.

Accessibility & polish:
- focus-visible + aria-current on nav, aria-label on all icon-only buttons
- prefers-reduced-motion (CSS media + GSAP guard helper)
- text-dim contrast raised to AA (4.81:1)
- unified logo (PixelLensLogo), real keyboard shortcuts (manifest suggested_key + mac)
- brand accent left unchanged (user decision)
- Command palette (Cmd+K / Ctrl+K): fuzzy-search all destinations and
  actions, full keyboard nav, a11y dialog/combobox, reduced-motion aware,
  extensible command registry.
- Copy for LLM: one-click bundle combining the page Markdown + extracted
  design system (colors/typography/spacing/shadows) + metadata, formatted
  for an AI context. Button in the Markdown view + popup quick action.
- Unit tests 176 -> 193.
- Contrast checker: new view scoring the extracted palette's text/background
  pairs against WCAG AA/AAA (text labels, not color-only), reachable via nav
  and the Cmd+K palette.
- Crawl export as ZIP (fflate 0.8.3): one .md per page + index.md, alongside
  the existing single-file mode. Toggle in the Crawl view.
- Unit tests 193 -> 209.
- Each skipped URL now carries a reason (http-<status>, timeout, non-text,
  cross-origin, network, robots). CrawlView shows a breakdown, so 'everything
  skipped' is no longer silent.
- fetchTextWithTimeout returns a discriminated result capturing the real HTTP
  status (http-403 = anti-bot, etc).
- 'Respect robots.txt' toggle (default on): the user is a human on their own
  session; turn off to include disallowed pages. Hint shown when pages are blocked.
- same-site matching (apex <-> www) instead of strict same-origin, so
  www-hosted sitemaps are no longer excluded.
- Unit tests 209 -> 218.
…site display

- Crawl 'Full' mode renders each page in a background tab so the site's JS
  runs, captures the rendered DOM via the content script's EXTRACT_MARKDOWN,
  then closes the tab. Fixes SPA sites that returned empty pages in fetch mode.
  Sequential, guaranteed tab cleanup, 20s per-page timeout, never steals focus.
  Fast/Full toggle + 'empty -> try Full' hint. Adds the 'tabs' permission.
- Fix stale scan: the panel ties the shown scan/markdown to the active tab's
  URL; a non-destructive banner appears when showing a scan from another host
  (updates on tab switch thanks to the new tabs permission).
- Unit tests 218 -> 245.
Copilot AI review requested due to automatic review settings July 1, 2026 02:49

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The job was not started because the account is locked due to a billing issue.

@vgtray
vgtray merged commit 7070596 into main Jul 1, 2026
1 of 2 checks passed
@vgtray
vgtray deleted the feat/markdown-converter branch July 1, 2026 02:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants