fixes

Author: dsherret

.github/workflows/ci.yml

@@ -26,7 +26,6 @@ jobs:
       - uses: actions/setup-node@v7
         with:
           node-version: "20"
-          registry-url: "https://registry.npmjs.org"
 
       - name: Format
         if: matrix.os == 'ubuntu-latest'

rs-lib/src/mappings.rs

@@ -125,7 +125,8 @@ impl Mappings {
       if !mappings.contains_key(key) {
         if let Some(path) = mappings.get(value).map(ToOwned::to_owned) {
           mappings.insert(key.clone(), path);
-        } else {
+        } else if !specifiers.has_mapped(value) {
+          // specifiers mapped to a package have no file in the output
           panic!("dnt bug - Could not find the mapping for {}", value);
         }
       }

rs-lib/tests/integration/in_memory_loader.rs

@@ -24,6 +24,7 @@ type RemoteFileResult = Result<(RemoteFileText, RemoteFileHeaders), String>;
 pub struct InMemoryLoader {
   pub sys: InMemorySys,
   remote_files: HashMap<ModuleSpecifier, RemoteFileResult>,
+  remote_redirects: HashMap<ModuleSpecifier, String>,
 }
 
 impl InMemoryLoader {
@@ -38,6 +39,7 @@ impl InMemoryLoader {
     Self {
       sys,
       remote_files: HashMap::new(),
+      remote_redirects: HashMap::new(),
     }
   }
 
@@ -87,6 +89,18 @@ impl InMemoryLoader {
     self
   }
 
+  pub fn add_remote_redirect(
+    &mut self,
+    specifier: impl AsRef<str>,
+    location: impl AsRef<str>,
+  ) -> &mut Self {
+    self.remote_redirects.insert(
+      ModuleSpecifier::parse(specifier.as_ref()).unwrap(),
+      location.as_ref().to_string(),
+    );
+    self
+  }
+
   pub fn add_remote_file_with_error(
     &mut self,
     specifier: impl AsRef<str>,
@@ -118,6 +132,12 @@ impl deno_cache_dir::file_fetcher::HttpClient for InMemoryLoader {
     specifier: &ModuleSpecifier,
     _headers: HeaderMap,
   ) -> Result<SendResponse, SendError> {
+    if let Some(location) = self.remote_redirects.get(specifier) {
+      return Ok(SendResponse::Redirect(to_headers(HashMap::from([(
+        "location".to_string(),
+        location.clone(),
+      )]))));
+    }
     let result = self
       .remote_files
       .get(&specifier)

rs-lib/tests/integration_test.rs

@@ -1168,6 +1168,45 @@ async fn transform_specifier_mappings() {
   );
 }
 
+#[tokio::test]
+async fn transform_specifier_mapping_of_redirected_specifier() {
+  let result = TestBuilder::new()
+    .with_loader(|loader| {
+      loader
+        .add_local_file(
+          "/mod.ts",
+          "import * as remote from 'http://localhost/mod.ts';",
+        )
+        .add_remote_redirect(
+          "http://localhost/mod.ts",
+          "http://localhost/redirected/mod.ts",
+        )
+        .add_remote_file("http://localhost/redirected/mod.ts", "");
+    })
+    .add_package_specifier_mapping(
+      "http://localhost/redirected/mod.ts",
+      "remote-module",
+      Some("1.0.0"),
+      None,
+    )
+    .transform()
+    .await
+    .unwrap();
+
+  assert_files!(
+    result.main.files,
+    &[("mod.ts", "import * as remote from 'remote-module';")]
+  );
+  assert_eq!(
+    result.main.dependencies,
+    &[Dependency {
+      name: "remote-module".to_string(),
+      version: "1.0.0".to_string(),
+      peer_dependency: false,
+    }]
+  );
+}
+
 #[tokio::test]
 async fn transform_not_found_mappings() {
   let error_message = TestBuilder::new()