Skip to content

fix: migrate plugin import to standalone package#43

Merged
itaymendel merged 2 commits intomainfrom
fix/migrate-plugin-imports
Feb 17, 2026
Merged

fix: migrate plugin import to standalone package#43
itaymendel merged 2 commits intomainfrom
fix/migrate-plugin-imports

Conversation

@itaymendel
Copy link
Owner

@itaymendel itaymendel commented Feb 17, 2026

resolves #39

Summary by CodeRabbit

  • Changed
    • Migrated plugin imports to the @oxlint/plugins package.
    • Bumped oxlint dependency to ^1.48.0 and added @oxlint/plugins ^1.48.0 as a dev dependency.
    • Raised peer dependency requirement for @oxlint/plugins to >=1.43.0.

@coderabbitai
Copy link

coderabbitai bot commented Feb 17, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

This PR migrates imports from oxlint/plugins to the standalone @oxlint/plugins, updates package dependency entries (oxlint → ^1.48.0 and adds @oxlint/plugins), and adds a changelog note documenting the migration for compatibility with oxlint >= 1.45.0. No runtime logic changes.

Changes

Cohort / File(s) Summary
Changelog
CHANGELOG.md
Added an Unreleased note documenting migration of plugin imports from oxlint/plugins to @oxlint/plugins.
Dependency config
package.json
Replaced/updated devDependency on oxlint to ^1.48.0 and added @oxlint/plugins ^1.48.0; peerDependencies updated to require @oxlint/plugins >=1.43.0.
Import updates — source files
src/index.ts, src/types.ts, src/extraction/variable-tracker.ts, src/rules/complexity.ts, src/rules/max-cognitive.ts, src/rules/shared.ts
Replaced imports from oxlint/plugins with @oxlint/plugins (e.g., definePlugin, defineRule, ESTree types and re-exports). No logic or API signature changes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I hopped through code with careful paws,

Swapped old plugin paths without a pause,
From oxlint's nest to a standalone den,
Now imports fit with the newest ken,
A tiny hop — compatibility again!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely summarizes the main change: migrating plugin imports from oxlint/plugins to the standalone @oxlint/plugins package.
Linked Issues check ✅ Passed The PR successfully addresses all coding objectives from issue #39: imports updated to @oxlint/plugins across all source files, and package.json updated with the new dependency.
Out of Scope Changes check ✅ Passed All changes are directly related to the migration objective; no unrelated modifications detected in the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/migrate-plugin-imports

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

@itaymendel itaymendel merged commit 65c3685 into main Feb 17, 2026
2 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.

Broken with oxlint >= 1.45.0: './plugins' subpath removed

1 participant