Skip to content

refactor(linter/plugins): rename oxlint/rule-tester to oxlint/plugins-dev#18793

Merged
graphite-app[bot] merged 1 commit intomainfrom
om/02-01-refactor_linter_plugins_rename_oxlint_rule-tester_to_oxlint_plugins-dev_
Feb 1, 2026
Merged

refactor(linter/plugins): rename oxlint/rule-tester to oxlint/plugins-dev#18793
graphite-app[bot] merged 1 commit intomainfrom
om/02-01-refactor_linter_plugins_rename_oxlint_rule-tester_to_oxlint_plugins-dev_

Conversation

@overlookmotel
Copy link
Member

@overlookmotel overlookmotel commented Jan 31, 2026

Part of #18610.

Our intent is to move RuleTester to a separate package @oxlint/plugins-dev.

So rename the current entry point which exports RuleTester from oxlint/rule-tester to oxlint/plugins-dev.

@github-actions github-actions bot added A-linter Area - Linter A-cli Area - CLI A-linter-plugins Area - Linter JS plugins C-cleanup Category - technical debt or refactoring. Solution not expected to change behavior labels Jan 31, 2026
Copy link
Member Author

overlookmotel commented Jan 31, 2026


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • 0-merge - adds this PR to the back of the merge queue
  • hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@overlookmotel overlookmotel marked this pull request as ready for review January 31, 2026 23:55
Copilot AI review requested due to automatic review settings January 31, 2026 23:55
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR renames the oxlint/rule-tester export to oxlint/plugins-dev as part of a larger refactoring effort to separate plugin authoring tools into dedicated packages. This is a preparatory step for eventually moving RuleTester into a standalone @oxlint/plugins-dev package.

Changes:

  • Renamed the entry point from oxlint/rule-tester to oxlint/plugins-dev
  • Updated deprecation notice to point to the new import path
  • Updated all internal references to use the new path

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
npm/oxlint/package.json Updated exports to expose ./plugins-dev instead of ./rule-tester
apps/oxlint/package.json Updated exports and internal imports mapping to use plugins-dev
apps/oxlint/tsdown.config.ts Updated build entry point to include plugins-dev.ts instead of rule-tester.ts
apps/oxlint/src-js/plugins-dev.ts New entry point file that exports RuleTester
apps/oxlint/src-js/index.ts Updated deprecation notice to reference new import path
apps/oxlint/conformance/src/rule_tester.ts Updated import to use #oxlint/plugins-dev
apps/oxlint/conformance/src/groups/stylistic.ts Updated type import to use #oxlint/plugins-dev

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@camc314 camc314 added the 0-merge Merge with Graphite Merge Queue label Feb 1, 2026
Copy link
Contributor

camc314 commented Feb 1, 2026

Merge activity

@graphite-app graphite-app bot removed the 0-merge Merge with Graphite Merge Queue label Feb 1, 2026
@camc314 camc314 force-pushed the om/02-01-refactor_linter_plugins_rename_oxlint_plugin_to_oxlint_plugins_ branch from 95414cb to 0c382c4 Compare February 1, 2026 13:16
@camc314 camc314 force-pushed the om/02-01-refactor_linter_plugins_rename_oxlint_rule-tester_to_oxlint_plugins-dev_ branch from 9fae30e to ceae6b5 Compare February 1, 2026 13:16
@camc314 camc314 added the 0-merge Merge with Graphite Merge Queue label Feb 1, 2026
…ins-dev` (#18793)

Part of #18610.

Our intent is to move `RuleTester` to a separate package `@oxlint/plugins-dev`.

So rename the current entry point which exports `RuleTester` from `oxlint/rule-tester` to `oxlint/plugins-dev`.
@graphite-app graphite-app bot force-pushed the om/02-01-refactor_linter_plugins_rename_oxlint_plugin_to_oxlint_plugins_ branch from 0c382c4 to 9de9385 Compare February 1, 2026 13:23
@graphite-app graphite-app bot force-pushed the om/02-01-refactor_linter_plugins_rename_oxlint_rule-tester_to_oxlint_plugins-dev_ branch from ceae6b5 to ab4fddd Compare February 1, 2026 13:23
Base automatically changed from om/02-01-refactor_linter_plugins_rename_oxlint_plugin_to_oxlint_plugins_ to main February 1, 2026 13:29
@graphite-app graphite-app bot merged commit ab4fddd into main Feb 1, 2026
21 checks passed
@graphite-app graphite-app bot deleted the om/02-01-refactor_linter_plugins_rename_oxlint_rule-tester_to_oxlint_plugins-dev_ branch February 1, 2026 13:30
@graphite-app graphite-app bot removed the 0-merge Merge with Graphite Merge Queue label Feb 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-cli Area - CLI A-linter Area - Linter A-linter-plugins Area - Linter JS plugins C-cleanup Category - technical debt or refactoring. Solution not expected to change behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments