build: add oxlint and oxfmt tooling#65
Conversation
Add oxlint (linter) and oxfmt (formatter) as root dev dependencies with config files and npm scripts. Commit .vscode folder with the oxc extension recommendation and fix/format-on-save settings so contributors get a consistent editor setup out of the box.
|
@azizbecha is attempting to deploy a commit to the Chonkie Inc Team on Vercel. A member of the Team first needs to authorize it. |
|
No actionable comments were generated in the recent review. π βΉοΈ Recent review infoβοΈ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: β Files ignored due to path filters (1)
π Files selected for processing (6)
π€ Files with no reviewable changes (1)
π WalkthroughWalkthroughThis PR introduces oxc-based tooling (oxlint, oxfmt) to the repository, adding configuration files, VS Code editor integration, and root package.json scripts and devDependencies, while removing the ChangesOxc Tooling Adoption
Estimated code review effort: 1 (Trivial) | ~5 minutes Poem
π₯ Pre-merge checks | β 5β Passed checks (5 passed)
β¨ Finishing Touchesπ§ͺ 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 |
Summary
Adds oxlint (linter) and oxfmt (formatter) to the monorepo, plus a committed
.vscodefolder so contributors get a consistent editor setup out of the box.Changes
oxlintandoxfmtas root dev dependencies.oxlintrc.jsonβ correctness rules as warnings; ignoresdist,legacy, and docs build output.oxfmtrc.jsonβ formatter config with the same ignores.vscode/extensions.jsonβ recommends the officialoxc.oxc-vscodeextension.vscode/settings.jsonβ enables fix-on-save (source.fixAll.oxc), sets oxc as default formatter with format-on-savelint,lint:fix,format,format:check.vscodefrom.gitignore(it was ignored, so the folder could never be committed)Notes
oxlint-tsgolintdependency) β can be a follow-uppnpm lintcurrently reports warnings on existing code (unused vars in handshakes/tests); nothing blockingpnpm format:checkreports ~109 unformatted files β the repo-wide reformat is intentionally excluded to keep this diff reviewable, and can land as a separate PRTest plan
pnpm lintruns and respects the configpnpm format:checkruns and respects the configSummary by CodeRabbit
New Features
Chores