Skip to content

chore: replace prettier with oxfmt - #694

Merged
jdx merged 1 commit into
mainfrom
chore/oxfmt
Oct 16, 2026
Merged

jdx merged 1 commit into
mainfrom
chore/oxfmt

Conversation

@jdx

@jdx jdx commented Oct 15, 2026 •

Copy link
Copy Markdown
Owner

https://entire.io/gh/jdx/mise-action/trails/54

Summary

Formatting now uses oxfmt instead of prettier. The prettier devDependency (and its aube-lock.yaml entries) is gone; oxfmt is a mise-managed tool.

What changes for contributors

  • oxfmt is a new tool in mise.toml and mise.lock; mise install provides it. Its pinned npm lock lives under .mise/locks/oxfmt/, which mise.lock references.
  • aubr format:write runs oxfmt "**/*.ts" and aubr format:check (used by aubr lint and CI) runs oxfmt --check "**/*.ts". The file scope is the same as before.
  • .prettierrc.json and .prettierignore are replaced by .oxfmtrc.json, produced with oxfmt --migrate=prettier: same options (single quotes, no semicolons, no trailing commas, arrowParens: avoid, 80 columns) and the same ignores (dist/, node_modules/, coverage/). sortPackageJson is turned off so package.json key order is left alone.
  • CLAUDE.md describes aubr format:write as oxfmt.

To format by hand: aubr format:write, or oxfmt "**/*.ts".

Formatting differences

None: oxfmt leaves every existing .ts file unchanged with this config, so there is no reformat commit and dist/ is identical (rebuilt with aubr package, no diff).

ESLint has no prettier plugin or eslint-config-prettier, so nothing there needed changing.

Verification

  • aubr all (format, ESLint, tests, rollup package) passes and leaves dist/ unchanged.
  • aubr typecheck passes.

AI-assisted — Tool: Claude Code; model: anthropic/claude-sonnet-5-5; version: claude-code_2-1-295_agent.

🤖 Generated with Claude Code


Note

Low Risk
Tooling-only swap with no application or runtime code changes; main risk is contributors missing mise-installed oxfmt or subtle formatter differences despite equivalent config.

Overview
Code formatting moves from Prettier to oxfmt: the prettier devDependency and its aube-lock.yaml entries are removed, and formatting is supplied by a mise-managed oxfmt tool (mise.toml, mise.lock, plus a pinned npm lock under .mise/locks/oxfmt/0.73.0/).

format:write and format:check (used by lint and the build pipeline) now run oxfmt on **/*.ts instead of Prettier. .prettierignore is dropped; ignores and migrated Prettier-style options live in .oxfmtrc.json (ignorePatterns for dist/, node_modules/, coverage/, and sortPackageJson: false). CLAUDE.md documents oxfmt for aubr format:write.

Contributors need mise install (or equivalent) to get oxfmt; aube install no longer installs a formatter.

Reviewed by Cursor Bugbot for commit 22a7f15. Bugbot is set up for automated code reviews on this repo. Configure here.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 15, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 10 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 2 included reviews currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Central YAML (base), Organization UI (inherited)
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: c468d7ac-a1d1-4362-988b-4d569057f2b9
📥 Commits

Reviewing files that changed from the base of the PR and between 04716d4 and 22a7f15.

⛔ Files ignored due to path filters: 1
  • mise.lock is excluded by !**/*.lock
📒 Files selected for processing: 8
  • .mise/locks/oxfmt/0.73.0/aube-lock.yaml
  • .mise/locks/oxfmt/0.73.0/package.json
  • .oxfmtrc.json
  • .prettierignore
  • CLAUDE.md
  • aube-lock.yaml
  • mise.toml
  • package.json
  • 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.

@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addednpm/​oxfmt@​0.73.0941008896100

View full report

@greptile-apps

greptile-apps Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[High impact] The PR appears safe to merge; no actionable issues were found.

Summary

Replaces Prettier with mise-managed oxfmt, pinned to version 0.73.0.

  • Project formatting commands now run mise-managed oxfmt.

Reviews (1) · Last reviewed commit: "chore: replace prettier with oxfmt" · Reviewed by Greptile

@jdx
jdx merged commit 93c62ad into main Oct 16, 2026
29 checks passed
@jdx
jdx deleted the chore/oxfmt branch October 16, 2026 00:01
@jdx jdx mentioned this pull request Oct 16, 2026
jdx added a commit that referenced this pull request Oct 16, 2026
<!-- entire-trail-link-start -->
https://entire.io/gh/jdx/mise-action/trails/46
<!-- entire-trail-link-end -->


---
## [5.2.0](https://github.com/jdx/mise-action/compare/v5.1.2..v5.2.0) -
2026-10-16

### 🚀 Features

- **(cache)** rebuild the cache every 30 days by default (#689) by
[@jdx](https://github.com/jdx) in
[#689](#689)
- **(cache)** retry throttled cache restores and saves (#692) by
[@jdx](https://github.com/jdx) in
[#692](#692)
- verify the mise binary against its signed packslip (#693) by
[@jdx](https://github.com/jdx) in
[#693](#693)

### 🐛 Bug Fixes

- export the PATH entries mise reports with mise env --paths (#685) by
[@jdx](https://github.com/jdx) in
[#685](#685)

### 🔍 Other Changes

- Use monorepo install when configured (#679) by
[@zeitlinger](https://github.com/zeitlinger) in
[#679](#679)
- Change minimum age for auto-merging release PRs by
[@jdx](https://github.com/jdx) in
[b511e8d](b511e8d)

### 📚 Documentation

- tell release workflows not to use the cache (#682) by
[@jdx](https://github.com/jdx) in
[#682](#682)
- explain caching mise downloads instead of installs (#686) by
[@zeitlinger](https://github.com/zeitlinger) in
[#686](#686)
- use resolved Windows downloads cache path (#690) by
[@zeitlinger](https://github.com/zeitlinger) in
[#690](#690)

### ⚙️ Miscellaneous Tasks

- stop restoring caches in release jobs (#683) by
[@jdx](https://github.com/jdx) in
[#683](#683)
- replace prettier with oxfmt (#694) by [@jdx](https://github.com/jdx)
in [#694](#694)

<!-- generated by git-cliff -->

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Only version and changelog updates; no runtime code changes in this
diff.
> 
> **Overview**
> **Release v5.2.0** — bumps `package.json` from **5.1.2** to **5.2.0**
and prepends a **5.2.0** `CHANGELOG.md` section (dated 2026-10-16)
generated from prior merged work.
> 
> The new changelog block documents what ships with this tag: cache
rebuild every 30 days, retries on throttled cache restore/save, signed
packslip verification for the mise binary, PATH export via `mise env
--paths`, monorepo install behavior, docs/CI cache tweaks, and replacing
Prettier with oxfmt. This diff also corrects the **5.1.2** entry date to
**2026-10-12**.
> 
> No application or `dist/` changes appear in this PR; it is version and
release-notes housekeeping only.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
2e65d8f. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Co-authored-by: mise-en-dev <123107610+mise-en-dev@users.noreply.github.com>
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