The bundler banners set g.bundler??=[], and then use that as a list of all the modules that got bundled. The Bun build plugin sets g.bundler=true. When it's an array of module names, we also have logic to ensure it stays unique.
It seems like it'd be easiest to make this a Set<string> instead of string[]. And, even if Bun doesn't actually need it to be a list of items, making it a Set would also be truthy, so it might be worth doing just to ensure it's always the same type, rather than having one bundler be a weird divergent case.
cc @JPeer264 @timfish Feel free to assign to me if you agree, or close if you don't ;)
The bundler banners set
g.bundler??=[], and then use that as a list of all the modules that got bundled. The Bun build plugin setsg.bundler=true. When it's an array of module names, we also have logic to ensure it stays unique.It seems like it'd be easiest to make this a
Set<string>instead ofstring[]. And, even if Bun doesn't actually need it to be a list of items, making it aSetwould also be truthy, so it might be worth doing just to ensure it's always the same type, rather than having one bundler be a weird divergent case.cc @JPeer264 @timfish Feel free to assign to me if you agree, or close if you don't ;)