fix(migrate-eslint): correctly resolve shared config @scope/eslint-co…

…nfig (#2713)
biomejs · May 4, 2024 · 24f06ed · 24f06ed
1 parent cb0182e
commit 24f06ed
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -23,6 +23,8 @@ our [guidelines for writing a good changelog entry](https://github.com/biomejs/b
 
   Contributed by @tasshi-me
 
+- `biome migrate eslint` now correctly resolve `@scope/eslint-config` ([#2705](https://github.com/biomejs/biome/issues/2705)). Contributed by @Conaclos
+
 ### Configuration
 
 ### Editors

diff --git a/crates/biome_cli/src/execute/migrate/eslint.rs b/crates/biome_cli/src/execute/migrate/eslint.rs
@@ -370,11 +370,10 @@ impl EslintPackage {
             EslintPackage::Config => "eslint-config-",
             EslintPackage::Plugin => "eslint-plugin-",
         };
-        debug_assert!(matches!(artifact, "eslint-plugin-" | "eslint-config-"));
         if name.starts_with('@') {
             // handle scoped module
             if let Some((scope, scoped)) = name.split_once('/') {
-                if scoped.starts_with(artifact) {
+                if scoped.starts_with(artifact) || scoped == artifact.trim_end_matches('-') {
                     Cow::Borrowed(name)
                 } else {
                     Cow::Owned(format!("{scope}/{artifact}{scoped}"))