mirrors/bun.sh
1
0
Fork 0
mirror of https://github.com/oven-sh/bun synced 2026-02-14 12:51:54 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: use correct source index for imports_to_bind lookup

Browse Source 
Use the export's source_index instead of the entry point's source_index
when looking up imports_to_bind, matching the behavior in the walk
function. Also add null check for symbol lookup to avoid panics.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Claude Bot
2026-01-07 15:01:26 +00:00
parent 66dfc9172f
commit 1ac254f72c
1 changed files with 6 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/bundler/linker_context/computeCrossChunkDependencies.zig
View File

@@ -324,7 +324,6 @@ fn computeCrossChunkDependenciesWithChunkMetas(c: *LinkerContext, chunks: []Chun
const source_index = chunk.entry_point.source_index;
const entry_export_aliases = sorted_and_filtered_export_aliases[source_index];
const entry_resolved_exports = resolved_exports[source_index];
const entry_imports_to_bind = imports_to_bind[source_index];
if (entry_export_aliases.len > 0) {
entry_point_export_refs = .{};
@@ -333,16 +332,18 @@ fn computeCrossChunkDependenciesWithChunkMetas(c: *LinkerContext, chunks: []Chun
for (entry_export_aliases) |alias| {
if (entry_resolved_exports.get(alias)) |resolved_export| {
var ref = resolved_export.data.import_ref;
// Follow the import binding if necessary
if (entry_imports_to_bind.get(ref)) |import_data| {
// Follow the import binding if necessary (use the export's source index)
if (imports_to_bind[resolved_export.data.source_index.get()].get(ref)) |import_data| {
ref = import_data.data.import_ref;
}
// If this is an ES6 import from a CommonJS file, it will become a
// property access off the namespace symbol instead of a bare
// identifier. In that case we want to pull in the namespace symbol
// instead. The namespace symbol stores the result of "require()".
if (c.graph.symbols.getConst(ref).?.namespace_alias) |namespace_alias| {
ref = namespace_alias.namespace_ref;
if (c.graph.symbols.getConst(ref)) |symbol| {
if (symbol.namespace_alias) |namespace_alias| {
ref = namespace_alias.namespace_ref;
}
}
// Store the alias so we can use it for exports_to_other_chunks
entry_point_export_refs.?.putAssumeCapacity(ref, alias);