chore: add lint:fix script for ESLint auto-fixes - #1510
Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
WalkthroughThe change adds a root ChangesLint auto-fix scripts
Estimated code review effort: 1 (Trivial) | ~2 minutes 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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 `@CHANGELOG.md`:
- Around line 10-11: Update the changelog entry for the lint:fix script to
replace the placeholder [`#XXXX`] reference and URL with the actual pull request
number and corresponding GitHub link once the PR is created.
🪄 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: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 60313d5c-9208-4cbc-bd69-580d74077215
📒 Files selected for processing (3)
CHANGELOG.mdpackage.jsonpackages/web/package.json
Closes #1466
Added a
lint:fixscript that mirrors the existinglintscript setup — root script fans out viayarn workspaces foreach, and added the actualeslint . --fixcommand in packages/web (currently the only workspace with a lint script).Tested by running
yarn lint:fix— it auto-fixed what ESLint could fix and correctly left 4 warnings that need manual review (unused vars, missing hook deps), same as expected behavior with --fix.Also added a CHANGELOG entry under Unreleased.
Note
Low Risk
Tooling-only script additions with no runtime, auth, or data-path changes.
Overview
Adds
yarn lint:fixat the repo root, using the sameyarn workspaces foreachpattern aslintso workspaces can run an auto-fix pass in one command.In
@sourcebot/web, addslint:fixaseslint . --fixwith the sameSKIP_ENV_VALIDATIONwrapper as the existinglintscript. That package is currently the only workspace defininglint, so this is where fix behavior lands today.Reviewed by Cursor Bugbot for commit 1f6a33f. Bugbot is set up for automated code reviews on this repo. Configure here.
Summary by CodeRabbit