From f5b37fa0b0544de11bde374deffda094fc3112ae Mon Sep 17 00:00:00 2001 From: Jarred Sumner <709451+Jarred-Sumner@users.noreply.github.com> Date: Sat, 16 Sep 2023 22:52:49 -0700 Subject: [PATCH] Use a better error label --- src/bundler/bundle_v2.zig | 4 ++-- src/import_record.zig | 17 +++++++++++++++++ 2 files changed, 19 insertions(+), 2 deletions(-) diff --git a/src/bundler/bundle_v2.zig b/src/bundler/bundle_v2.zig index 64aaf54d9e..359272e5f9 100644 --- a/src/bundler/bundle_v2.zig +++ b/src/bundler/bundle_v2.zig @@ -512,7 +512,7 @@ pub const BundleV2 = struct { import_record.range, this.graph.allocator, "Browser build cannot {s} Node.js module: \"{s}\". To use Node.js builtins, set target to 'node' or 'bun'", - .{ import_record.kind.label(), path_to_use }, + .{ import_record.kind.errorLabel(), path_to_use }, import_record.kind, ) catch unreachable; } else { @@ -1915,7 +1915,7 @@ pub const BundleV2 = struct { import_record.range, this.graph.allocator, "Browser build cannot {s} Node.js builtin: \"{s}\". To use Node.js builtins, set target to 'node' or 'bun'", - .{ import_record.kind.label(), import_record.path.text }, + .{ import_record.kind.errorLabel(), import_record.path.text }, import_record.kind, ) catch @panic("unexpected log error"); } else { diff --git a/src/import_record.zig b/src/import_record.zig index 3044c9b47e..d48cfb1b0c 100644 --- a/src/import_record.zig +++ b/src/import_record.zig @@ -51,10 +51,27 @@ pub const ImportKind = enum(u8) { break :brk labels; }; + pub const error_labels: Label = brk: { + var labels = Label.initFill(""); + labels.set(ImportKind.entry_point, "entry point"); + labels.set(ImportKind.stmt, "import"); + labels.set(ImportKind.require, "require()"); + labels.set(ImportKind.dynamic, "import()"); + labels.set(ImportKind.require_resolve, "require.resolve"); + labels.set(ImportKind.at, "@import"); + labels.set(ImportKind.url, "url()"); + labels.set(ImportKind.internal, ""); + break :brk labels; + }; + pub inline fn label(this: ImportKind) []const u8 { return all_labels.get(this); } + pub inline fn errorLabel(this: ImportKind) []const u8 { + return error_labels.get(this); + } + pub inline fn isCommonJS(this: ImportKind) bool { return switch (this) { .require, .require_resolve => true,