fix(core): support arrays as alias targets (#7453)

Author: arendjr

.changeset/aliased-alleys-align.md

@@ -0,0 +1,5 @@
+---
+"@biomejs/biome": patch
+---
+
+Fixed [#7242](https://github.com/biomejs/biome/issues/7242): Aliases specified in `package.json`'s `imports` section now support having multiple targets as part of an array.

crates/biome_resolver/src/lib.rs

@@ -235,7 +235,10 @@ fn resolve_import_alias(
     fs: &dyn ResolverFsProxy,
     options: &ResolveOptions,
 ) -> Result<Utf8PathBuf, ResolveError> {
-    let imports = package_json.imports.clone().ok_or(ResolveError::NotFound)?;
+    let imports = package_json
+        .imports
+        .as_ref()
+        .ok_or(ResolveError::NotFound)?;
     let imports = imports
         .as_object()
         .ok_or(ResolveError::InvalidMappingTarget)?;
@@ -438,6 +441,12 @@ fn resolve_target_value(
             }
             None => resolve_string(target.as_str()),
         },
+        JsonValue::Array(targets) => targets
+            .iter()
+            .find_map(|target| {
+                resolve_target_value(target, glob_replacement, package_path, fs, options).ok()
+            })
+            .ok_or(ResolveError::NotFound),
         _ => Err(ResolveError::InvalidMappingTarget),
     }
 }

crates/biome_resolver/tests/fixtures/resolver_cases_6/package.json

@@ -1,3 +1,11 @@
 {
-    "name": "resolver_cases_6"
+    "name": "resolver_cases_6",
+    "imports": {
+        "#*": [
+            "./src/*",
+            "./src/*.ts",
+            "./src/*.tsx",
+            "./src/*/index.ts"
+        ]
+    }
 }

crates/biome_resolver/tests/fixtures/resolver_cases_6/src/lib/a.tsx

@@ -533,3 +533,37 @@ fn test_resolve_type_definitions_without_type_specification() {
         )))
     );
 }
+
+#[test]
+fn test_resolve_alias_with_multiple_target_values() {
+    let base_dir = get_fixtures_path("resolver_cases_6");
+    let fs = OsFileSystem::new(base_dir.clone());
+
+    let options = ResolveOptions {
+        condition_names: &["types", "import", "default"],
+        default_files: &["index"],
+        extensions: &["ts", "js"],
+        ..Default::default()
+    };
+
+    assert_eq!(
+        resolve("#lib/a", &base_dir, &fs, &options),
+        Ok(Utf8PathBuf::from(format!("{base_dir}/src/lib/a.tsx")))
+    );
+    assert_eq!(
+        resolve("#lib/b", &base_dir, &fs, &options),
+        Ok(Utf8PathBuf::from(format!("{base_dir}/src/lib/b.ts")))
+    );
+    assert_eq!(
+        resolve("#lib/c", &base_dir, &fs, &options),
+        Ok(Utf8PathBuf::from(format!("{base_dir}/src/lib/c/index.ts")))
+    );
+    assert_eq!(
+        resolve("#lib/d", &base_dir, &fs, &options),
+        Err(ResolveError::NotFound)
+    );
+    assert_eq!(
+        resolve("#lib/d.js", &base_dir, &fs, &options),
+        Ok(Utf8PathBuf::from(format!("{base_dir}/src/lib/d.js")))
+    );
+}