diff options
author | Nathan Whitaker <17734409+nathanwhit@users.noreply.github.com> | 2024-09-21 16:10:38 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-09-21 16:10:38 -0700 |
commit | 9be8dce0c7f8deaeff5523735e0867194ec04c59 (patch) | |
tree | 36977fc586cff680bb0847bdb19e586e0e777955 /ext/node_resolver | |
parent | 4b022103a14916de1c3bc539d123273750138915 (diff) |
fix(node): Include "node" condition during CJS re-export analysis (#25785)
Fixes #25777.
We were missing the "node" condition, so we were resolving to the wrong
conditional export, causing our analysis to be incorrect.
Diffstat (limited to 'ext/node_resolver')
-rw-r--r-- | ext/node_resolver/analyze.rs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ext/node_resolver/analyze.rs b/ext/node_resolver/analyze.rs index aea33e29a..deb56d064 100644 --- a/ext/node_resolver/analyze.rs +++ b/ext/node_resolver/analyze.rs @@ -196,7 +196,7 @@ impl<TCjsCodeAnalyzer: CjsCodeAnalyzer, TNodeResolverEnv: NodeResolverEnv> &referrer, // FIXME(bartlomieju): check if these conditions are okay, probably // should be `deno-require`, because `deno` is already used in `esm_resolver.rs` - &["deno", "require", "default"], + &["deno", "node", "require", "default"], NodeResolutionMode::Execution, ); let reexport_specifier = match result { |