feat: add Lighthouse CI for quality scoring - #221
Merged
Conversation
Audits 4 pages on every PR: performance, accessibility, best practices, SEO. Accessibility < 90 blocks merge; others warn at < 90. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The static server serves from dist/ directly, so URLs should not include the base path prefix. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
SonarCloud flagged temporary-public-storage as a security hotspot. Reports are still available as GitHub Actions artifacts. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replaces the GitHub App integration with a CI-based scan. Scans astro-site/src on every PR and push to main. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
v5 has a known security vulnerability per SonarSource advisory. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
SonarCloud needs node_modules to resolve tsconfig references. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Reads JSON reports from .lighthouseci/ and posts a table with per-page scores for performance, accessibility, best practices, and SEO. Updates existing comment on re-runs. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Use finalDisplayedUrl as primary, fall back to finalUrl, requestedUrl, then filename. Guard against non-http values. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Lighthouse CI may wrap reports under .lhr key. Skip files without categories to avoid errors on manifest or other metadata files. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Keep simple workflow — scores available via artifacts. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
CodeQL is GitHub-native, free for all repos (public and private), and aligns with the quality gates template in solid-ai-templates. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool. What Enabling Code Scanning Means:
For more information about GitHub Code Scanning, check out the documentation. |
Resolves CodeQL alert: workflows without explicit permissions run with broad defaults. Restrict to contents:read. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
GitHub's default CodeQL setup is already enabled and covers JavaScript/TypeScript and Actions. Custom workflow conflicts with the default setup. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Uh oh!
There was an error while loading. Please reload this page.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
🤖 Generated with Claude Code