Skip to content

Sweep site quality, accessibility, dependencies, and CI - #52

Merged
plx merged 1 commit into
mainfrom
plx/site-quality-sweep
Aug 30, 2026
Merged

Sweep site quality, accessibility, dependencies, and CI#52
plx merged 1 commit into
mainfrom
plx/site-quality-sweep

Conversation

@plx

@plx plx commented Aug 30, 2026

Copy link
Copy Markdown
Owner

Summary

  • fix duplicate/incorrect primary metadata, normalize Markdown out of social/RSS metadata, honor noOgImage, and make content dates timezone-stable
  • fix deterministic back navigation, reduced-motion scrolling, hidden focus, mobile inline-code overflow, heading levels, contrast, social-link names, and scrollable-code landmarks
  • restore real Astro accessibility linting, fail closed on malformed category metadata, factor shared social/OG logic, and remove unused React/icon/SEO/template code
  • refresh safe dependencies and the Node 24 patch, eliminate all npm audit findings, pin GitHub Actions, tighten workflow permissions, and add Dependabot
  • expand unit and Playwright coverage across metadata, RSS, dates, categories, themes, focus/navigation, axe scans, browser errors, and every sitemap route at narrow widths
  • run Chromium QA in CI with a deterministic static preview server, while also restoring Markdown linting to the required gate

The rendered body text remains unchanged across all 24 generated HTML pages. The only visual differences are targeted bug fixes: wrapping overflowing inline code and small contrast corrections.

Validation

  • npm ci — clean install; 0 vulnerabilities
  • npm run validate:all — 46 unit tests, 24-page Astro build with 0 diagnostics, spelling/prose/Markdown/link/feed validation, and 50 Chromium browser tests
  • Chromium/WebKit/Mobile Safari axe coverage in light and dark modes
  • all sitemap routes checked at 320px and 375px with zero document overflow
  • actionlint
  • zizmor .github/workflows --pedantic — no findings
  • just --fmt --check
  • rendered-body comparison against the pre-change build — unchanged on all 24 HTML pages

TypeScript 7, Tailwind 4, tailwind-merge 3, and Node 26 type definitions remain deliberately deferred as separate major migrations; the watch items are documented in planning/major-migration-plan.md.

Closes #42

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Aug 30, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-08-30T22:47:23.455856Z 9ab2fc0 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@plx
plx merged commit 132f4b3 into main Aug 30, 2026
2 checks passed
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.

Reintroduce accessibility linting (replacement for eslint-plugin-jsx-a11y)

1 participant