fix(ci): align vibecoder workflows with npm and current trivy action#2
fix(ci): align vibecoder workflows with npm and current trivy action#2
Conversation
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 13 minutes and 37 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
965c8a1 to
7615e40
Compare
There was a problem hiding this comment.
Pull request overview
Updates CI/security workflows to match the repository’s npm + package-lock.json setup and to harden Trivy action usage.
Changes:
- Switch CI install/lint/typecheck steps from pnpm to npm (
npm ci,npm run ...). - Pin
aquasecurity/trivy-actionto an immutable commit SHA (annotated asv0.35.0) across workflows. - In Trivy auto-fix workflow, regenerate
package-lock.jsonafter applyingpackage.jsonoverrides.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| .github/workflows/trivy-autofix.yml | Pins Trivy action; removes pnpm enablement; adds npm install --package-lock-only after override patching. |
| .github/workflows/security.yml | Pins Trivy action to the same immutable SHA. |
| .github/workflows/ci.yml | Moves CI to npm commands and simplifies CI-gate dependencies; removes prior Node PATH setup. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| - name: Add Node.js (fnm) to PATH | ||
| run: echo "$HOME/.local/share/fnm/node-versions/v24.14.0/installation/bin" >> $GITHUB_PATH | ||
|
|
||
| - name: Enable pnpm via corepack | ||
| run: corepack enable pnpm | ||
|
|
||
|
|
||
| - name: Install dependencies | ||
| run: pnpm install --frozen-lockfile | ||
| run: npm ci | ||
|
|
There was a problem hiding this comment.
lint-typecheck runs npm ci/npm run ... on a self-hosted runner but the workflow no longer ensures Node.js is available (no actions/setup-node and no fnm PATH injection). Other workflows in this repo still add the fnm-managed Node v24.14.0 to $GITHUB_PATH, so this job may fail if Node isn’t already on PATH or if a different Node version is picked up. Consider adding back a Node setup step (either actions/setup-node pinned to the expected version or the same fnm PATH export used elsewhere) before npm ci.
Fixes current-head CI/security failures by switching the workflows to the repo's actual npm lockfile model and pinning the Trivy action to the current safe revision.