feat: Add direct report links alongside comparison reports #100
Conversation
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
github-actions
bot
added
ci/cd
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds functionality to extract and display direct Lighthouse audit report URLs alongside comparison URLs. Previously, only comparison URLs were shown; now users can access individual audit results without the comparison view.
Changes:
- Parse
compareReportURL parameter from Lighthouse CI upload logs to extract direct report URLs - Update PR comments and job summaries in both mobile and desktop workflows to display both direct and comparison report links
- Refactor URL link rendering to show multiple report types with distinct emoji indicators
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| scripts/lighthouse-metadata.cjs | Extracts direct report URL from URL-encoded compareReport parameter and adds BOM removal for file encoding compatibility |
| scripts/lighthouse-report-markdown.cjs | Updates markdown generation to include direct report link with updated labels for comparison reports |
| .github/workflows/ci-lighthouse-mobile.yml | Modifies PR comment to display both direct and comparison report links |
| .github/workflows/ci-lighthouse-desktop.yml | Modifies PR comment to display both direct and comparison report links |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
garybrowndev
force-pushed
the
feature/lighthouse-direct-report-links
branch
from
1b69993 to
d02be26
Compare
…rove metadata debugging - Added PR comment steps for mobile and desktop Lighthouse results - Each result shows both Direct Report and Comparison Report links - Added debug warnings in metadata extraction for better troubleshooting - Ensures Lighthouse audits on Surge preview are visible in PR
garybrowndev
force-pushed
the
feature/lighthouse-direct-report-links
branch
from
d02be26 to
0411cf0
Compare
…hthouse metadata extraction
Contributor
🤖 Automated PR Report for 5ab3ec2Pull Request Labeler (automation-label-pull-request.yml)PR: feat: Add direct report links alongside comparison reports (#100) Current Labels
Dependency Review (security-dependency-review.yml)✅ No vulnerabilities or denied packages found. Trivy Security Scan (security-trivy-scan.yml)Trivy scan completed. Results have been uploaded to the GitHub Security tab. Scanned Categories
OWASP Dependency Check (security-owasp-dependency-check.yml)Total Vulnerabilities: 0 Severity Breakdown
✅ No vulnerabilities found. Standalone Build Results (ci-standalone-build.yml)
npm Audit Results (security-npm-audit.yml)✅ No vulnerabilities found. Bundle Size Report (ci-bundle-analysis.yml)
Recommendations
Bundle analysis completed at 2026-01-18T17:40:00.208Z CodeQL Security Analysis (security-codeql-security-analysis.yml)Total Alerts: 1 Severity Breakdown
Detailed Findings
Code Quality Results (ci-code-quality.yml)
📊 Code Statistics
Lighthouse Audit - Desktop (ci-lighthouse-desktop.yml)📊 Desktop Lighthouse ResultsTested URL: http://localhost:9222/pinball-trainer-standalone.html
Reports: WebKit Browser Test Results (ci-webkit-browser-tests.yml)
Lighthouse Audit - Mobile (ci-lighthouse-mobile.yml)📊 Mobile Lighthouse ResultsTested URL: http://localhost:9222/pinball-trainer-standalone.html
Reports: Chromium Browser Test Results (ci-chromium-browser-tests.yml)
Firefox Browser Test Results (ci-firefox-browser-tests.yml)
Visual Regression Results (ci-visual-regression-tests.yml)
Unit Test Results (ci-unit-tests.yml)
Code Coverage Results (ci-coverage-reporting.yml)
Lighthouse Audit - Surge Mobile (cd-deploy-pr-preview.yml)📱 Mobile Lighthouse ResultsTested URL: https://pinball-trainer-pr-100.surge.sh/
Reports: Lighthouse Audit - Surge Desktop (cd-deploy-pr-preview.yml)🖥️ Desktop Lighthouse ResultsTested URL: https://pinball-trainer-pr-100.surge.sh/
Reports: |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
…QL security check
Contributor
🤖 Automated PR Report for b757eefPull Request Labeler (automation-label-pull-request.yml)PR: feat: Add direct report links alongside comparison reports (#100) Current Labels
Trivy Security Scan (security-trivy-scan.yml)Trivy scan completed. Results have been uploaded to the GitHub Security tab. Scanned Categories
OWASP Dependency Check (security-owasp-dependency-check.yml)Total Vulnerabilities: 0 Severity Breakdown
✅ No vulnerabilities found. npm Audit Results (security-npm-audit.yml)✅ No vulnerabilities found. Bundle Size Report (ci-bundle-analysis.yml)
Recommendations
Bundle analysis completed at 2026-01-18T17:43:30.049Z CodeQL Security Analysis (security-codeql-security-analysis.yml)Total Alerts: 1 Severity Breakdown
Detailed Findings
Standalone Build Results (ci-standalone-build.yml)
Code Quality Results (ci-code-quality.yml)
📊 Code Statistics
Chromium Browser Test Results (ci-chromium-browser-tests.yml)
Unit Test Results (ci-unit-tests.yml)
Lighthouse Audit - Mobile (ci-lighthouse-mobile.yml)📊 Mobile Lighthouse ResultsTested URL: http://localhost:9222/pinball-trainer-standalone.html
Reports: WebKit Browser Test Results (ci-webkit-browser-tests.yml)
Visual Regression Results (ci-visual-regression-tests.yml)
Firefox Browser Test Results (ci-firefox-browser-tests.yml)
Code Coverage Results (ci-coverage-reporting.yml)
Lighthouse Audit - Desktop (ci-lighthouse-desktop.yml)📊 Desktop Lighthouse ResultsTested URL: http://localhost:9222/pinball-trainer-standalone.html
Reports: Lighthouse Audit - Surge Mobile (cd-deploy-pr-preview.yml)📱 Mobile Lighthouse ResultsTested URL: https://pinball-trainer-pr-100.surge.sh/
Reports: Lighthouse Audit - Surge Desktop (cd-deploy-pr-preview.yml)🖥️ Desktop Lighthouse ResultsTested URL: https://pinball-trainer-pr-100.surge.sh/
Reports: Preview Deployment (cd-deploy-pr-preview.yml)Your preview deployment is ready! 🔗 Preview URL: https://pinball-trainer-pr-100.surge.sh What's included:
Testing checklist:
Preview will be updated on each new commit to this PR. Deployed at 2026-01-18T17:46:27.130Z |
Contributor
🤖 Automated PR Report for b757eefDependency Review (security-dependency-review.yml)✅ No vulnerabilities or denied packages found. |
…QL security requirement
Contributor
🤖 Automated PR Report for 02b01d3Pull Request Labeler (automation-label-pull-request.yml)PR: feat: Add direct report links alongside comparison reports (#100) Current Labels
Trivy Security Scan (security-trivy-scan.yml)Trivy scan completed. Results have been uploaded to the GitHub Security tab. Scanned Categories
OWASP Dependency Check (security-owasp-dependency-check.yml)Total Vulnerabilities: 0 Severity Breakdown
✅ No vulnerabilities found. npm Audit Results (security-npm-audit.yml)✅ No vulnerabilities found. Standalone Build Results (ci-standalone-build.yml)
Code Quality Results (ci-code-quality.yml)
📊 Code Statistics
Bundle Size Report (ci-bundle-analysis.yml)
Recommendations
Bundle analysis completed at 2026-01-18T17:49:45.784Z Firefox Browser Test Results (ci-firefox-browser-tests.yml)
Lighthouse Audit - Mobile (ci-lighthouse-mobile.yml)📊 Mobile Lighthouse ResultsTested URL: http://localhost:9222/pinball-trainer-standalone.html
Reports: WebKit Browser Test Results (ci-webkit-browser-tests.yml)
Chromium Browser Test Results (ci-chromium-browser-tests.yml)
Lighthouse Audit - Desktop (ci-lighthouse-desktop.yml)📊 Desktop Lighthouse ResultsTested URL: http://localhost:9222/pinball-trainer-standalone.html
Reports: Code Coverage Results (ci-coverage-reporting.yml)
Lighthouse Audit - Surge Mobile (cd-deploy-pr-preview.yml)📱 Mobile Lighthouse ResultsTested URL: https://pinball-trainer-pr-100.surge.sh/
Reports: Lighthouse Audit - Surge Desktop (cd-deploy-pr-preview.yml)🖥️ Desktop Lighthouse ResultsTested URL: https://pinball-trainer-pr-100.surge.sh/
Reports: Preview Deployment (cd-deploy-pr-preview.yml)Your preview deployment is ready! 🔗 Preview URL: https://pinball-trainer-pr-100.surge.sh What's included:
Testing checklist:
Preview will be updated on each new commit to this PR. Deployed at 2026-01-18T17:52:32.629Z |
Contributor
🤖 Automated PR Report for 02b01d3Dependency Review (security-dependency-review.yml)✅ No vulnerabilities or denied packages found. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Changes: