Skip to content

Add Playwright coverage and CI quality gates - #53

Open
iatopilskii wants to merge 4 commits into
mainfrom
test/critical-user-flows
Open

Add Playwright coverage and CI quality gates#53
iatopilskii wants to merge 4 commits into
mainfrom
test/critical-user-flows

Conversation

@iatopilskii

Copy link
Copy Markdown
Collaborator

Summary

  • add a Playwright configuration that runs critical flows in desktop and mobile Chromium
  • cover critical route rendering, conversion links, responsive navigation, application form validation, pricing interactions, and blog/glossary search
  • retain screenshots, traces, and videos when a browser test fails
  • add a GitHub Actions workflow for pull requests, pushes to main, and manual runs
  • add repository-managed pre-commit and pre-push hooks with simple-git-hooks
  • use a dedicated test server on port 3100 and refuse to reuse unrelated local servers
  • align local, GitHub Actions, and Vercel builds on Node.js 22 for pnpm 11 compatibility
  • explicitly allow only the simple-git-hooks dependency install script while preserving the existing sharp denial

Quality gates

Pre-commit

  • format staged JavaScript, TypeScript, HTML, CSS, Markdown, and MDX files
  • lint staged JavaScript and TypeScript files
  • run TypeScript validation

Pre-push and CI

Both run the shared pnpm check command:

  1. ESLint
  2. TypeScript validation
  3. Production build
  4. 32 Playwright tests across desktop and mobile Chromium

Why

The repository did not have automated browser coverage or a shared quality gate for the user journeys most likely to affect conversion and content discovery. This change keeps local Git hooks and CI aligned while avoiding changes to production behavior.

Developer impact

Git hooks are installed during pnpm install through the prepare script. Developers can also refresh them with pnpm update-git-hooks and run the complete gate manually with pnpm check.

Validation

  • GitHub Actions quality workflow passed
  • Vercel preview deployment passed
  • pre-commit hook executed successfully during commit
  • pre-push hook executed successfully during push
  • pnpm check passed
  • 32 Playwright tests passed across desktop and mobile Chromium
  • production build passed
  • TypeScript validation passed
  • ESLint passed with 4 pre-existing warnings
  • targeted Prettier check passed
  • git diff --check passed

@vercel

vercel Bot commented Aug 7, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
marketing-site Ready Ready Preview Aug 7, 2026 4:25pm

Request Review

@iatopilskii
iatopilskii marked this pull request as ready for review August 7, 2026 16:35
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.

1 participant