fix(lsp): import map expansion

cli/Cargo.toml

@@ -64,19 +64,19 @@ winres.workspace = true
 [dependencies]
 deno_ast = { workspace = true, features = ["bundler", "cjs", "codegen", "proposal", "react", "sourcemap", "transforms", "typescript", "view", "visit"] }
 deno_cache_dir = { workspace = true }
-deno_config = "=0.10.0"
+deno_config = "=0.11.0"
 deno_core = { workspace = true, features = ["include_js_files_for_snapshotting"] }
-deno_doc = { version = "=0.110.1", features = ["html"] }
-deno_emit = "=0.38.1"
-deno_graph = { version = "=0.69.0", features = ["tokio_executor"] }
+deno_doc = { version = "=0.113.1", features = ["html"] }
+deno_emit = "=0.38.2"
+deno_graph = { version = "=0.69.4", features = ["tokio_executor"] }
 deno_lint = { version = "=0.57.1", features = ["docs"] }
 deno_lockfile.workspace = true
 deno_npm = "=0.17.0"
 deno_runtime = { workspace = true, features = ["include_js_files_for_snapshotting"] }
 deno_semver = "=0.5.4"
 deno_task_shell = "=0.14.3"
 deno_terminal.workspace = true
-eszip = "=0.64.1"
+eszip = "=0.64.2"
 napi_sym.workspace = true
 
 async-trait.workspace = true
@@ -107,7 +107,7 @@ fs3.workspace = true
 glob = "0.3.1"
 hex.workspace = true
 ignore = "0.4"
-import_map = { version = "=0.18.3", features = ["ext"] }
+import_map = { version = "=0.19.0", features = ["ext"] }
 indexmap.workspace = true
 jsonc-parser = { version = "=0.23.0", features = ["serde"] }
 lazy-regex.workspace = true

cli/args/import_map.rs

@@ -1,5 +1,6 @@
 // Copyright 2018-2024 the Deno authors. All rights reserved. MIT license.
 
+use deno_core::anyhow::Context;
 use deno_core::error::AnyError;
 use deno_core::serde_json;
 use deno_core::url::Url;
@@ -11,35 +12,66 @@ use log::warn;
 use super::ConfigFile;
 use crate::file_fetcher::FileFetcher;
 
-pub async fn resolve_import_map_from_specifier(
-  specifier: &Url,
+pub async fn resolve_import_map(
+  specified_specifier: Option<&Url>,
   maybe_config_file: Option<&ConfigFile>,
   file_fetcher: &FileFetcher,
+) -> Result<Option<ImportMap>, AnyError> {
+  if let Some(specifier) = specified_specifier {
+    resolve_import_map_from_specifier(specifier.clone(), file_fetcher)
+      .await
+      .with_context(|| format!("Unable to load '{}' import map", specifier))
+      .map(Some)
+  } else if let Some(config_file) = maybe_config_file {
+    let maybe_url_and_value = config_file
+      .to_import_map_value(|specifier| {
+        let specifier = specifier.clone();
+        async move {
+          let file = file_fetcher
+            .fetch(&specifier, PermissionsContainer::allow_all())
+            .await?
+            .into_text_decoded()?;
+          Ok(file.source.to_string())
+        }
+      })
+      .await
+      .with_context(|| {
+        format!(
+          "Unable to resolve import map in '{}'",
+          config_file.specifier
+        )
+      })?;
+    match maybe_url_and_value {
+      Some((url, value)) => {
+        import_map_from_value(url.into_owned(), value).map(Some)
+      }
+      None => Ok(None),
+    }
+  } else {
+    Ok(None)
+  }
+}
+
+async fn resolve_import_map_from_specifier(
+  specifier: Url,
+  file_fetcher: &FileFetcher,
 ) -> Result<ImportMap, AnyError> {
   let value: serde_json::Value = if specifier.scheme() == "data" {
     let data_url_text =
-      deno_graph::source::RawDataUrl::parse(specifier)?.decode()?;
+      deno_graph::source::RawDataUrl::parse(&specifier)?.decode()?;
     serde_json::from_str(&data_url_text)?
   } else {
-    let import_map_config = maybe_config_file
-      .as_ref()
-      .filter(|c| c.specifier == *specifier);
-    match import_map_config {
-      Some(config) => config.to_import_map_value(),
-      None => {
-        let file = file_fetcher
-          .fetch(specifier, PermissionsContainer::allow_all())
-          .await?
-          .into_text_decoded()?;
-        serde_json::from_str(&file.source)?
-      }
-    }
+    let file = file_fetcher
+      .fetch(&specifier, PermissionsContainer::allow_all())
+      .await?
+      .into_text_decoded()?;
+    serde_json::from_str(&file.source)?
   };
   import_map_from_value(specifier, value)
 }
 
 pub fn import_map_from_value(
-  specifier: &Url,
+  specifier: Url,
   json_value: serde_json::Value,
 ) -> Result<ImportMap, AnyError> {
   debug_assert!(