Skip to content

Migrate JS lint and format tooling to Oxlint and Oxfmt#77

Merged
lorisleiva merged 1 commit into
mainfrom
loris/oxc-lint-format
Jun 15, 2026
Merged

Migrate JS lint and format tooling to Oxlint and Oxfmt#77
lorisleiva merged 1 commit into
mainfrom
loris/oxc-lint-format

Conversation

@lorisleiva

Copy link
Copy Markdown
Member

This PR migrates the feature-gate JS client from ESLint and Prettier to Oxlint and Oxfmt via the shared @solana-config/oxc package, to align with the other repos in the solana-program org. It adds oxlint.config.ts and oxfmt.config.ts, updates the lint and format scripts and dependencies, drops the now-unused ESLint and Prettier configuration, and pins a Node 24 floor (which the Oxc TypeScript configs require). Codama's JS renderer is given explicit prettier options so that regeneration still produces a zero diff. The remaining source changes are a one-time reformat applying the shared Oxfmt style. It also drops the .prettierrc.json that previously fed Codama, so the regenerated client adopts the org-standard 4-space formatting in this one-time reformat.

This PR migrates the feature-gate JS client from ESLint and Prettier to Oxlint and Oxfmt via the shared @solana-config/oxc package, to align with the other repos in the solana-program org. It adds oxlint.config.ts and oxfmt.config.ts, updates the lint and format scripts and dependencies, drops the now-unused ESLint and Prettier configuration (including the .prettierrc.json that previously fed Codama), and pins a Node 24 floor (which the Oxc TypeScript configs require). Codama's JS renderer now uses the shared prettier options inline, so the regenerated client adopts the org-standard formatting. The remaining changes are a one-time reformat applying the shared Oxfmt style.
@lorisleiva lorisleiva marked this pull request as ready for review June 15, 2026 16:49
@lorisleiva lorisleiva merged commit fd819c1 into main Jun 15, 2026
22 checks passed
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