build(deps): unblock the two glitching Dependabot PRs (#1021, #1022)#1029
Conversation
Two open dependency-update PRs are failing required CI: - #1021 (github-actions group) bumps actions/checkout to v7.0.1 and anthropics/claude-code-action to v1.0.179, but both new commit SHAs are rejected by check:github-actions ("not a reviewed commit SHA"). Add both to the reviewed-pin allowlist so the batched action-bump PR passes once rebased. - #1022 bumps eslint 9 -> 10, which crashes `npm run lint` outright: eslint-config-next@16.2.10 bundles an eslint-plugin-react that still calls the removed context.getFilename() API. ESLint 10 is not yet supported by the lint toolchain, so hold the major in Dependabot, mirroring the existing TypeScript hold. Dependabot will close #1022. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FFME8u6CC4QRHPwcrD6TmG
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
|
Warning Review limit reached
Next review available in: 1 minute Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?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 reviews. How do review 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 refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
✨ 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. |
Summary
Two open Dependabot dependency-update PRs were failing the required CI aggregate. This unblocks both from
main.github-actionsgroup (actions/checkout→ v7.0.1,anthropics/claude-code-action→ v1.0.179). The bump was red becausecheck:github-actionsrejects any action SHA not in the reviewed-pin allowlist, and both new commit SHAs were unreviewed. This adds them to the allowlist inscripts/github-action-pins.mjs(keeping the prior SHAs, matching the existing multi-version pattern). Once Dependabot rebases chore(deps): bump the github-actions group across 1 directory with 2 updates #1021 onto this, its pin check passes.eslint9 → 10. ESLint 10 crashesnpm run lintoutright:eslint-config-next@16.2.10bundles aneslint-plugin-reactthat still calls the removedcontext.getFilename()API (TypeError: ... getFilename is not a function). The lint toolchain isn't ESLint-10-ready, so this holds the ESLint major in.github/dependabot.yml, mirroring the existing TypeScript-major hold. Dependabot will close chore(deps-dev): bump eslint from 9.39.5 to 10.7.0 #1022 on its next run.No application code changes; no
package.json/lockfile changes. This does not touch retrieval/ranking/ingestion/answer surfaces — RAG impact: no retrieval behaviour change — dependency/CI config only.Verification
npm run verify:cheap— all static gates pass (incl.check:github-actions,check:ci-scope,check:pr-policy), lint + typecheck clean, 3028/3032 unit tests pass.node scripts/check-github-action-pins.mjs→ "GitHub Actions pin check passed."tests/github-action-pins.test.ts→ 6/6 pass.One pre-existing unrelated failure:
tests/pdf-extraction-budget.test.ts(Python OCR subprocess — "python exploded"). Confirmed to fail identically on a clean tree with these changes stashed; it's the sandbox missing the Python OCR stack (perAGENTS.md), not a regression from this PR.Risk and rollout
Clinical Governance Preflight
Not applicable — no ingestion, answer generation, search/ranking, source rendering, document access, privacy, production env, or clinical output is touched.
Notes
@testing-library/jest-dom7.0.0) is already green and is intentionally left untouched.eslint-config-nextsupports ESLint 10.Generated by Claude Code