feat(lighthouse): add weekly lighthouse audit scan - #286
Conversation
|
Warning Review limit reached
More reviews will be available in 27 minutes. Learn how PR review limits work. To continue reviewing without waiting, enable usage-based billing in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits. 🚦 How do rate limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
📝 WalkthroughWalkthroughAdds a GitHub Actions workflow that runs weekly or manually and invokes a reusable lighthouse scan workflow against ChangesWeekly Lighthouse scan workflow
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
View your CI Pipeline Execution ↗ for commit 751ae0c
💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗ ☁️ Nx Cloud last updated this comment at |
|
View your CI Pipeline Execution ↗ for commit ec5726f
💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗ ☁️ Nx Cloud last updated this comment at |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/workflows/lighthouse-scan.yml:
- Line 11: The reusable workflow reference in the lighthouse scan job is pinned
to a mutable branch, so update the uses reference to a fixed, reviewable ref
such as a tag or commit SHA. Locate the workflow call in the lighthouse scan job
and replace the current `@main` target with an immutable version so the dependency
cannot change unexpectedly.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 48102aa9-b535-4241-9d30-b1d2ae89421b
📒 Files selected for processing (1)
.github/workflows/lighthouse-scan.yml
| jobs: | ||
| lighthouse: | ||
| name: external workflow | ||
| uses: tehw0lf/workflows/.github/workflows/lighthouse-scan.yml@main |
There was a problem hiding this comment.
🔒 Security & Privacy | 🟠 Major
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
printf '\n== file listing ==\n'
git ls-files .github/workflows/lighthouse-scan.yml
printf '\n== workflow file with line numbers ==\n'
cat -n .github/workflows/lighthouse-scan.yml
printf '\n== any local references to the external workflow repo/ref ==\n'
rg -n "tehw0lf/workflows|lighthouse-scan.yml@|workflow_call" .github/workflows . -g '!**/node_modules/**' || trueRepository: tehw0lf/tehwol.fi
Length of output: 1312
Pin the reusable workflow ref.
uses: tehw0lf/workflows/.github/workflows/lighthouse-scan.yml@main tracks a mutable branch, so this external dependency can change without review and weakens reproducibility.
🔧 Suggested fix
- uses: tehw0lf/workflows/.github/workflows/lighthouse-scan.yml@main
+ uses: tehw0lf/workflows/.github/workflows/lighthouse-scan.yml@<commit-sha>📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| uses: tehw0lf/workflows/.github/workflows/lighthouse-scan.yml@main | |
| uses: tehw0lf/workflows/.github/workflows/lighthouse-scan.yml@<commit-sha> |
🧰 Tools
🪛 zizmor (1.26.1)
[error] 11-11: unpinned action reference (unpinned-uses): action is not pinned to a hash (required by blanket policy)
(unpinned-uses)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In @.github/workflows/lighthouse-scan.yml at line 11, The reusable workflow
reference in the lighthouse scan job is pinned to a mutable branch, so update
the uses reference to a fixed, reviewable ref such as a tag or commit SHA.
Locate the workflow call in the lighthouse scan job and replace the current
`@main` target with an immutable version so the dependency cannot change
unexpectedly.
Source: Linters/SAST tools
751ae0c to
3f92634
Compare
Summary
tehw0lf/workflows— requires workflows PR Revert "Fix master pipeline" #89 to be merged firstTest plan
Summary by CodeRabbit