Skip to content

build(oxfmt): migrate prettier to oxfmt#1203

Merged
chimurai merged 1 commit intomasterfrom
oxfmt
Apr 11, 2026
Merged

build(oxfmt): migrate prettier to oxfmt#1203
chimurai merged 1 commit intomasterfrom
oxfmt

Conversation

@chimurai
Copy link
Copy Markdown
Owner

@chimurai chimurai commented Apr 11, 2026

Summary by CodeRabbit

  • Chores
    • Migrated code formatter from Prettier to oxfmt for improved performance and consistency.
    • Updated VS Code configuration and extension recommendations to reflect new formatter.
    • Updated development dependencies and build scripts to use the new formatting tool.
    • Applied consistent code formatting across the codebase using updated style settings.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 11, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: abedac14-4220-4c82-9057-7849cbc35899

📥 Commits

Reviewing files that changed from the base of the PR and between e441d61 and bba58da.

⛔ Files ignored due to path filters (1)
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (12)
  • .devcontainer/devcontainer.json
  • .lintstagedrc
  • .oxfmtrc.json
  • .prettierignore
  • .prettierrc
  • .vscode/extensions.json
  • .vscode/settings.json
  • cspell.json
  • examples/next-app/app/layout.tsx
  • examples/next-app/app/page.tsx
  • examples/package.json
  • package.json
💤 Files with no reviewable changes (2)
  • .prettierignore
  • .prettierrc

📝 Walkthrough

Walkthrough

This pull request migrates the code formatter from Prettier to oxfmt across the project. Configuration files are updated to use oxfmt with custom formatting rules, development dependencies are replaced, and code examples are reformatted to match oxfmt's style defaults. VS Code extensions and settings are updated accordingly.

Changes

Cohort / File(s) Summary
Formatter Configuration
.oxfmtrc.json, .prettierrc, .prettierignore
New oxfmt configuration added with formatting rules (printWidth: 100, tabWidth: 2, semi: true, singleQuote: true). Prettier configuration file removed and ignore patterns updated to reflect oxfmt's formatter scope.
Development Environment
.devcontainer/devcontainer.json, .vscode/extensions.json, .vscode/settings.json, cspell.json
VS Code extensions and settings updated to use oxc.oxc-vscode instead of esbenp.prettier-vscode. Dictionary expanded to include oxfmt as valid spelling.
Lint Configuration
.lintstagedrc
Lint-staged formatting command updated from prettier --write to oxfmt --write for multi-file types.
Package Configuration
package.json, examples/package.json
Package.json files reordered for consistency. Root package.json updates: format-related scripts renamed from prettier to oxfmt, @trivago/prettier-plugin-sort-imports dev dependency removed, prettier replaced with oxfmt in devDependencies.
Code Examples
examples/next-app/app/layout.tsx, examples/next-app/app/page.tsx
Import and string literals reformatted to use single quotes matching oxfmt style defaults. JSX whitespace handling adjusted for consistency.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

Poem

A formatter hops into view so bright,
With oxfmt's touch, the code's formatted right,
Single quotes sparkle where double ones were,
Prettier's gone, but the style won't blur! ✨🐰

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'build(oxfmt): migrate prettier to oxfmt' clearly and concisely summarizes the main change - replacing Prettier with oxfmt as the code formatter across the project.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch oxfmt

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

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 11, 2026

npm i https://pkg.pr.new/http-proxy-middleware@1203

commit: bba58da

@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 93.103%. remained the same — oxfmt into master

@chimurai chimurai merged commit a4f0821 into master Apr 11, 2026
21 checks passed
@chimurai chimurai deleted the oxfmt branch April 11, 2026 20:55
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.

2 participants