fix: support virtual: and cloudflare: schemes (#1035)

Closes #986
Closes #1034

@zephraph this should address your problem, alongside
denoland/deno#28922

Author: crowlKats

api/src/analysis.rs

@@ -477,11 +477,10 @@ impl SyncLoader<'_> {
           maybe_headers: None,
         }))
       }
-      "http" | "https" | "node" | "npm" | "jsr" | "bun" => {
-        Ok(Some(deno_graph::source::LoadResponse::External {
-          specifier: specifier.clone(),
-        }))
-      }
+      "http" | "https" | "node" | "npm" | "jsr" | "bun" | "virtual"
+      | "cloudflare" => Ok(Some(deno_graph::source::LoadResponse::External {
+        specifier: specifier.clone(),
+      })),
       "data" => load_data_url(specifier)
         .map_err(|e| LoadError::Other(Arc::new(JsErrorBox::from_err(e)))),
       _ => Ok(None),
@@ -796,7 +795,7 @@ fn collect_dependencies(
           }
         }
       }
-      "file" | "data" | "node" | "bun" => {}
+      "file" | "data" | "node" | "bun" | "virtual" | "cloudflare" => {}
       "http" | "https" => {
         return Err(PublishError::InvalidExternalImport {
           specifier: module.specifier().to_string(),

api/src/api/package.rs

@@ -1828,7 +1828,7 @@ impl DepTreeLoader {
       }
       "jsr" => unreachable!("{specifier}"),
       // TODO(@crowlKats): handle npm specifiers
-      "npm" | "node" | "bun" => async move {
+      "npm" | "node" | "bun" | "virtual" | "cloudflare" => async move {
         Ok(Some(deno_graph::source::LoadResponse::External {
           specifier: specifier.clone(),
         }))

api/src/docs.rs

@@ -1098,7 +1098,7 @@ impl HrefResolver for DocResolver {
     if let Ok(url) = Url::parse(src) {
       match url.scheme() {
         "node" => Some(format!("https://nodejs.org/api/{}.html", url.path())),
-        "bun" => None,
+        "bun" | "virtual" | "cloudflare" => None,
         "npm" => {
           let npm_package_req =
             deno_semver::npm::NpmPackageReqReference::from_str(src).ok()?;

api/src/publish.rs

@@ -1161,6 +1161,24 @@ pub mod tests {
     assert!(!uses_npm(&t, &task).await);
   }
 
+  #[tokio::test]
+  async fn virtual_import() {
+    let t = TestSetup::new().await;
+    let bytes = create_mock_tarball("virtual_import");
+    let task = process_tarball_setup(&t, bytes).await;
+    assert_eq!(task.status, PublishingTaskStatus::Success, "{task:#?}");
+    assert!(!uses_npm(&t, &task).await);
+  }
+
+  #[tokio::test]
+  async fn cloudflare_import() {
+    let t = TestSetup::new().await;
+    let bytes = create_mock_tarball("cloudflare_import");
+    let task = process_tarball_setup(&t, bytes).await;
+    assert_eq!(task.status, PublishingTaskStatus::Success, "{task:#?}");
+    assert!(!uses_npm(&t, &task).await);
+  }
+
   #[tokio::test]
   async fn jsr_import() {
     let t = TestSetup::new().await;

api/testdata/tarballs/cloudflare_import/jsr.json

@@ -0,0 +1,5 @@
+{
+  "name": "@scope/foo",
+  "version": "1.2.3",
+  "exports": "./mod.ts"
+}

api/testdata/tarballs/cloudflare_import/mod.ts

@@ -0,0 +1 @@
+export * from "cloudflare:sockets";

api/testdata/tarballs/virtual_import/jsr.json

@@ -0,0 +1,5 @@
+{
+  "name": "@scope/foo",
+  "version": "1.2.3",
+  "exports": "./mod.ts"
+}

api/testdata/tarballs/virtual_import/mod.ts

@@ -0,0 +1 @@
+export * from "virtual:test";