fix(cache): do not attempt to emit non-emitable files (#15562)

cli/proc_state.rs

@@ -30,6 +30,7 @@ use crate::npm::NpmPackageResolver;
 use crate::resolver::ImportMapResolver;
 use crate::resolver::JsxResolver;
 
+use deno_ast::MediaType;
 use deno_core::anyhow::anyhow;
 use deno_core::anyhow::bail;
 use deno_core::anyhow::Context;
@@ -598,15 +599,25 @@ impl ProcState {
         code, media_type, ..
       } = entry
       {
-        emit_parsed_source(
-          &self.emit_cache,
-          &self.parsed_source_cache,
-          specifier,
-          *media_type,
-          code,
-          &self.emit_options,
-          self.emit_options_hash,
-        )?;
+        let is_emittable = matches!(
+          media_type,
+          MediaType::TypeScript
+            | MediaType::Mts
+            | MediaType::Cts
+            | MediaType::Jsx
+            | MediaType::Tsx
+        );
+        if is_emittable {
+          emit_parsed_source(
+            &self.emit_cache,
+            &self.parsed_source_cache,
+            specifier,
+            *media_type,
+            code,
+            &self.emit_options,
+            self.emit_options_hash,
+          )?;
+        }
       }
     }
     Ok(())

cli/tests/integration/cache_tests.rs

@@ -92,3 +92,8 @@ itest!(check_local_by_default2 {
   output: "cache/check_local_by_default2.out",
   http_server: true,
 });
+
+itest!(json_import {
+  // should not error
+  args: "cache --quiet cache/json_import/main.ts",
+});

cli/tests/testdata/cache/json_import/main.ts

@@ -0,0 +1,2 @@
+import asdf from "./test.json" assert { type: "json" };
+console.log(asdf);

cli/tests/testdata/cache/json_import/test.json

@@ -0,0 +1,5 @@
+{
+  "foo": {
+    "bar": 1
+  }
+}

test_util/src/lib.rs

@@ -1904,6 +1904,8 @@ impl<'a> CheckOutputIntegrationTest<'a> {
 
     let expected = if let Some(s) = self.output_str {
       s.to_owned()
+    } else if self.output.is_empty() {
+      String::new()
     } else {
       let output_path = testdata_dir.join(self.output);
       println!("output path {}", output_path.display());