Skip to content

Convert type-length-value-js to oxlint and oxfmt - #200

Merged
joncinque merged 1 commit into
solana-program:mainfrom
clankmaxxing-clod:convert-tlv-js-to-oxlint-oxfmt
Aug 28, 2026
Merged

Convert type-length-value-js to oxlint and oxfmt#200
joncinque merged 1 commit into
solana-program:mainfrom
clankmaxxing-clod:convert-tlv-js-to-oxlint-oxfmt

Conversation

@clankmaxxing-clod

Copy link
Copy Markdown
Contributor

Replaces ESLint + Prettier with oxlint and oxfmt for the type-length-value-js package, mirroring solana-program/token-group#115 and solana-program/token-metadata#120.

Changes

  • package.json: dropped eslint, typescript-eslint, @eslint/js, @eslint/eslintrc, eslint-plugin-import-extensions, prettier, and @solana/prettier-config-solana; added oxlint and oxfmt. Scripts now run the Oxc tools:
    • lintoxlint --max-warnings 0
    • lint:fixoxlint --fix
    • formatoxfmt --check src test
    • format:fixoxfmt --write src test
  • .oxlintrc.json (new) mirrors the old flat ESLint config: correctness category as errors; consistent-type-imports enforced; ban-ts-comment / no-explicit-any / no-unused-vars / no-empty-object-type disabled; import/extensions (import plugin) replaces eslint-plugin-import-extensions and stays disabled for test/examples.
  • .oxfmtrc.json (new) migrated from the existing .prettierrc via oxfmt --migrate=prettier (printWidth 80, tabWidth 4, single quotes, semicolons, trailingComma: es5, arrowParens: always).
  • Removed eslint.config.mjs and .prettierrc.
  • Updated pnpm-lock.yaml (CI installs with --frozen-lockfile).

Notes

  • CI invokes pnpm lint / pnpm format through the repo Makefile, so no workflow files needed changing.
  • Existing source is already oxfmt-clean and oxlint-clean — no source reformatting in this PR.

Verification

pnpm install --frozen-lockfile, pnpm lint, pnpm format, pnpm build, and pnpm test (6 passing) all succeed.

🤖 Generated with Claude Code

Replaces ESLint + Prettier with oxlint and oxfmt for the
type-length-value-js package.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@joncinque
joncinque merged commit a442541 into solana-program:main Aug 28, 2026
53 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.

2 participants