Repository navigation
chore: update oxlint and oxfmt - #17150
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: 📥 CommitsReviewing files that changed from the base of the PR and between 40d8dc72c13894fb493df9ded18e844476a28d84 and 0375841. ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (3)
Included review availability: Your plan provides up to 10 included reviews per hour; 7 remain after this review. 📝 WalkthroughWalkthroughThe pull request updates Merge Risk: ⚪ Minimal · up to This PR updates linting and formatting tools while preserving the existing policy and application behavior. The listed checks pass, and no actionable merge-blocking risk remains beyond normal review. 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
Full details: Description checkExplanation The description explains the dependency updates and lists validation results, but it does not use the repository template and omits required sections, including ELI5, What Changed, Why, Linked Issue, Visual Proof, Testing, Review, Agent skill upstream boundary, Notes, and Checklist. Resolution Update the description to use all template headings. Add a valid issue reference after "Fixes #", write "N/A" with a reason under Visual Proof, provide testing details and platforms, complete the required checkboxes, and address AI Disclosure and the remaining template sections. Full details: Docstring CoverageExplanation Docstring coverage is 14.81% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 27 functions across 49 files. (2 skipped: 2 unsupported.)
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 |
Greptile SummaryUpdates oxlint and oxfmt while preserving the existing lint policy.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| .oxlintrc.json | Explicitly disables the newly split React Compiler rules to preserve the existing lint policy. |
| package.json | Updates the root oxlint and oxfmt development dependencies. |
| mobile/package.json | Updates the mobile workspace’s oxlint and oxfmt development dependencies. |
| pnpm-lock.yaml | Regenerates root dependency resolutions for the updated tooling. |
| mobile/pnpm-lock.yaml | Regenerates mobile dependency resolutions for the updated tooling and existing React Native patch metadata. |
Reviews (2): Last reviewed commit: "chore: update oxlint and oxfmt" | Re-trigger Greptile
40d8dc7 to
0375841
Compare
Summary
Validation
pnpm exec oxlintpnpm exec oxfmt --check .pnpm run check:code-quality:changedpnpm tcpnpm test src/main/pi/agent-status-extension-source.test.tsAll passed.