name: qa description: Use when completing any task (final validation step), running audits, preparing for deployment, or when ESLint/TypeScript/build errors occur.

QA Skill

Canonical Sources

This skill references the following globals — read them BEFORE starting work:

• system/globals/responsiveness.md — Breakpoints, test order, anti-patterns

Domain

Testing, Validation, Code Quality, Launch Readiness

KPIs

Rules

Build Pipeline

# Full QA pipeline
npm run lint          # ESLint errors/warnings
npm run format:check  # Prettier formatting
npx tsc --noEmit      # TypeScript errors
npm run build         # Astro build (0 errors, 0 warnings)

Pre-Launch Checklist

Content

• No placeholder content ("Lorem ipsum", "TODO", "FIXME")
• All links work (no # or javascript:void(0))
• Images have descriptive alt text
• Text is proofread

Branding

• Logo replaced (no longer AstroDeck default)
• Favicon set (public/favicon.svg)
• OG image created (public/og-image.png, 1200x630px)
• Site title and description in astro.config.mjs

Legal

• Imprint present (required in DACH region)
• Privacy policy present
• Cookie banner configured (if tracking active)

Analytics & Forms

• Analytics code integrated (if desired)
• Forms submit to correct endpoints
• Forms have honeypot/CAPTCHA

Performance

• Lighthouse Performance >90
• Images optimized (WebP/AVIF, responsive sizes)
• No unnecessary client:load directives
• External scripts with SRI

Lighthouse Targets

Dark Mode Testing

• All pages checked in dark mode
• No white surfaces or unreadable text
• Images/icons visible (no black logo on black background)
• Toggle works and persists selection

Cross-Browser

• Chrome/Edge (Chromium)
• Firefox
• Safari (WebKit)
• Mobile Safari (iOS)
• Chrome Mobile (Android)

Non-Negotiable

These rules always apply — even under time pressure, even for "just a small update":

• No commit without npm run build. "I only changed text" — content changes can also break the build (missing frontmatter fields, broken links).
• No ESLint ignore without comment. // eslint-disable-next-line is allowed, but only with an explanation of why the rule doesn't apply here.
• No skipping the QA pipeline. "It's urgent" is not an exception. The pipeline (lint → format → tsc → build) takes under 30 seconds.
• Lighthouse always all 4 categories. Never measure only performance and ignore the rest. A score drop in any category is a blocker.
• Responsive test is part of QA. "Looks good on my monitor" doesn't count. At minimum, 375px and 1280px must be checked.

Before Applying

Read LEARNINGS.md in this directory to avoid known anti-patterns.