Format code with improved line breaks and add oxfmt config#77
Merged
bartveneman merged 4 commits intomainfrom Mar 12, 2026
Merged
Format code with improved line breaks and add oxfmt config#77bartveneman merged 4 commits intomainfrom
bartveneman merged 4 commits intomainfrom
Conversation
- Add oxfmt as dev dependency and run --init to create .oxfmtrc.json - Configure oxfmt with tabs, single quotes, and no semicolons - Add oxfmt --check to the lint script alongside oxlint - Add oxc dependency group in dependabot.yml to update oxlint and oxfmt together - Format all files with oxfmt https://claude.ai/code/session_018pu9kBHwmA9L7YnAk3SjXg
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #77 +/- ##
==========================================
- Coverage 96.93% 96.49% -0.45%
==========================================
Files 14 14
Lines 947 1026 +79
Branches 117 118 +1
==========================================
+ Hits 918 990 +72
- Misses 27 34 +7
Partials 2 2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Bundle ReportBundle size has no change ✅ |
Fixtures are test snapshots that should not be reformatted. Add src/lib/test/fixtures/** to oxfmt ignorePatterns. https://claude.ai/code/session_018pu9kBHwmA9L7YnAk3SjXg
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR improves code formatting consistency across the codebase and adds configuration for the oxfmt formatter.
Summary
Applied formatting improvements to enhance code readability by breaking long lines into multiple lines, particularly in function calls and array operations. Added oxfmt configuration file to standardize formatting rules.
Key Changes
Code formatting: Reformatted several files to break long lines into multiple lines for better readability:
src/cli/reporters/pretty.ts: Split long array joins and function calls across multiple linessrc/lib/index.ts: Reformattedreduce()calls with proper line breakssrc/lib/index.test.ts: Improved formatting of test expectationssrc/cli/reporters/json.ts: Enhanced readability of filter logicpackage.json: Reorganized keywords in alphabetical orderConfiguration: Added
.oxfmtrc.jsonwith formatting preferences:File endings: Fixed missing newlines in configuration files (
.oxlintrc.json,.prettierrc,.vscode/settings.json)Implementation Details
The formatting changes maintain functional equivalence while improving code readability through strategic line breaks in complex expressions. The oxfmt configuration ensures consistent formatting standards across the project going forward.
https://claude.ai/code/session_018pu9kBHwmA9L7YnAk3SjXg