Skip to content

chore: update oxfmt and oxlint - #416

Closed
bompus wants to merge 1 commit into
mainfrom
chore/oxc-tooling-20261020
Closed

bompus wants to merge 1 commit into
mainfrom
chore/oxc-tooling-20261020

Conversation

@bompus

@bompus bompus commented Oct 20, 2026 •

Copy link
Copy Markdown
Owner

Update the development formatter to oxfmt 0.74.0 and linter to oxlint 1.89.0, including their matching platform bindings. Existing configuration and all other dependency pins stay unchanged.

The adjacent release notes require no migration for this repository's JSON configuration. The existing oxlint-tsgolint version meets the new peer requirement. Formatting and lint checks needed no source corrections.

Validation: the README development checks pass locally, covering static checks, Node and Bun tests, skill scripts, and the npm tarball. The lockfile comparison confirms all unrelated entries are unchanged. Linux and Windows CI remain required before landing.


Summary by cubic

Updates the development formatter to oxfmt 0.74.0 and the linter to oxlint 1.89.0, including their matching platform bindings in bun.lock.

No migration is needed: the existing configuration stays valid, the current oxlint-tsgolint pin satisfies the new peer requirement, and no source corrections were required. The README development checks pass locally; Linux and Windows CI are still required before landing.

Written for commit 47bd8b0. Summary will update on new commits.

View guided diff Turn on auto-fix

Summary by CodeRabbit

  • Chores
    • Updated the development formatter to oxfmt 0.74.0.
    • Updated the development linter to oxlint 1.89.0.
  • Documentation
    • Recorded the formatter and linter updates in the unreleased changelog.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bompus has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@bompus

bompus commented Oct 20, 2026

Copy link
Copy Markdown
Owner Author

@coderabbitai full review

@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository YAML (base), Organization UI (inherited)
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: b3a5e0c8-35c3-4895-8c50-6de29e9acc6d
📥 Commits

Reviewing files that changed from the base of the PR and between fcdf5d1 and 47bd8b0.

⛔ Files ignored due to path filters: 1
  • bun.lock is excluded by !**/*.lock
📒 Files selected for processing: 2
  • CHANGELOG.md
  • package.json

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

📝 Walkthrough

The development dependency versions for oxfmt and oxlint changed. The Unreleased changelog records the updated versions.

Changes: Development tool updates

Layer / File(s) Summary
Update development tool versions
package.json, CHANGELOG.md
package.json updates oxfmt from 0.73.0 to 0.74.0 and oxlint from 1.88.0 to 1.89.0. The changelog records both updates.

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 47bd8

The development tool upgrades are consistently locked and introduce no established merge-blocking risk.

Architecture risk: 🔵 Low · up to 47bd8

The change affects 2 systems.

Changed systems: CHANGELOG.md, package.json

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — CHANGELOG.md (service) was modified; 1 changed file maps to changed impact.
  • observed — package.json (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in CHANGELOG.md: Added an Unreleased changelog entry recording upgrades to oxfmt 0.74.0 and oxlint 1.89.0.
  • observed — Modified behavior in package.json: Development dependency versions changed: oxfmt 0.73.0 → 0.74.0 and oxlint 1.88.0 → 1.89.0.
Pre-merge checks: Passed 6
Check name Status Explanation
Title check Passed The title clearly and concisely identifies the main change: updating the oxfmt and oxlint development dependencies.
Description check Passed The description explains the dependency updates, migration impact, validation results, changelog coverage, and unchanged dependency scope. It addresses the required checklist items, although it does n…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Suppressions Explained Passed The pull request changes only dependency versions, matching lockfile entries, and a changelog entry. No changed line adds or broadens a lint, type-check, or formatter suppression directive or ignore c…
Rule Changes Documented Passed The pull request changes only CHANGELOG.md, bun.lock, and package.json. It does not change rules/core.md, any file under rules/modifiers/, or any skills/*/SKILL.md. It also does not add, remove, or re…
Full details: Description check

Explanation

The description explains the dependency updates, migration impact, validation results, changelog coverage, and unchanged dependency scope. It addresses the required checklist items, although it does not include the template checkboxes explicitly.

Full details: Suppressions Explained

Explanation

The pull request changes only dependency versions, matching lockfile entries, and a changelog entry. No changed line adds or broadens a lint, type-check, or formatter suppression directive or ignore configuration.

Full details: Rule Changes Documented

Explanation

The pull request changes only CHANGELOG.md, bun.lock, and package.json. It does not change rules/core.md, any file under rules/modifiers/, or any skills/*/SKILL.md. It also does not add, remove, or rename a modifier or skill. Therefore, the documentation conditions are not triggered.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown
✅ Action performed

Full review finished.

@bompus

bompus commented Oct 20, 2026

Copy link
Copy Markdown
Owner Author

CodeRabbit completed its review without actionable findings. Cubic skipped this head because the monthly review line limit was reached. Greptile skipped because the trial credit limit was reached. These capacity skips follow the standing review policy; the remaining CI checks must pass before landing.

@bompus bompus closed this Oct 20, 2026
@bompus
bompus deleted the chore/oxc-tooling-20261020 branch October 20, 2026 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant