cli/tsc/go.rs (1)

665-690: Early-return for non-JS/TS media types correctly avoids tsc parsing

The new match media_type cleanly separates JS/TS(+JSON) from everything else and returning Ok(None) for Css/Html/Sql/Wasm/SourceMap/Jsonc/Json5/Unknown is aligned with the goal of not letting tsc treat these as source files.

One subtle behavior change to be aware of: when load_inner returns None, getImpliedNodeFormatForFile now falls back to ResolutionMode::ESM, whereas previously CSS (and other non-JS/TS) went through get_resolution_mode and could end up with ResolutionMode::None. That’s probably fine if TypeScript never consults node format for files we skip as source, but it is a semantic change.

If you want to keep the old implied-format behavior while still skipping source creation, you could precompute and store module_kind for all media types, then early-return only for the non-JS/TS ones, for example:

-  let is_cjs = result.is_cjs;
-  let media_type = result.media_type;
-
-  match media_type {
+  let is_cjs = result.is_cjs;
+  let media_type = result.media_type;
+  let module_kind = get_resolution_mode(is_cjs, media_type);
+
+  match media_type {
     MediaType::JavaScript
@@
-    | MediaType::Unknown => return Ok(None),
+    | MediaType::Unknown => {
+      state
+        .module_kind_map
+        .insert(load_specifier.to_string(), module_kind);
+      return Ok(None);
+    }
   }
-
-  let module_kind = get_resolution_mode(is_cjs, media_type);
+  // JS/TS-like: fall through and return LoadResult as before

Not a blocker, but worth confirming this matches how tsgo expects getImpliedNodeFormatForFile to behave.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Learnt from: CR
Repo: denoland/deno PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-11-24T16:19:37.808Z
Learning: Applies to cli/module_loader.rs : Module loading and resolution is handled in `cli/module_loader.rs`