@globalPrivate -> @linkTimeConstant

See https://github.com/WebKit/WebKit/pull/2553

I'm not 100% sure @linkTimeConstant is correct. Specifically, I don't know if it's a link time constant or just a global private thing.

6 files changed, 110 insertions, 111 deletions

diff --git a/src/bun.js/builtins/cpp/ImportMetaObjectBuiltins.cpp b/src/bun.js/builtins/cpp/ImportMetaObjectBuiltins.cpp
index acf81645f..eb3f0e1f9 100644
--- a/src/bun.js/builtins/cpp/ImportMetaObjectBuiltins.cpp
+++ b/src/bun.js/builtins/cpp/ImportMetaObjectBuiltins.cpp
@@ -48,74 +48,27 @@
 
 namespace WebCore {
 
-const JSC::ConstructAbility s_importMetaObjectRequireCodeConstructAbility = JSC::ConstructAbility::CannotConstruct;
-const JSC::ConstructorKind s_importMetaObjectRequireCodeConstructorKind = JSC::ConstructorKind::None;
-const JSC::ImplementationVisibility s_importMetaObjectRequireCodeImplementationVisibility = JSC::ImplementationVisibility::Public;
-const int s_importMetaObjectRequireCodeLength = 1162;
-static const JSC::Intrinsic s_importMetaObjectRequireCodeIntrinsic = JSC::NoIntrinsic;
-const char* const s_importMetaObjectRequireCode =
-    "(function (name) {\n" \
-    "  \"use strict\";\n" \
-    "  if (typeof name !== \"string\") {\n" \
-    "    @throwTypeError(\"require() expects a string as its argument\");\n" \
-    "  }\n" \
-    "  \n" \
-    "  const resolved = this.resolveSync(name, this.path);\n" \
-    "  var cached = @requireMap.@get(resolved);\n" \
-    "  const last5 = resolved.substring(resolved.length - 5);\n" \
-    "  if (cached) {\n" \
-    "    if (last5 === \".node\") {\n" \
-    "      return cached.exports;\n" \
-    "    }\n" \
-    "\n" \
-    "    return cached;\n" \
-    "  }\n" \
-    "\n" \
-    "  \n" \
-    "  //\n" \
-    "  if (last5 === \".json\") {\n" \
-    "    var fs = (globalThis[Symbol.for(\"_fs\")] ||= Bun.fs());\n" \
-    "    var exports = JSON.parse(fs.readFileSync(resolved, \"utf8\"));\n" \
-    "    @requireMap.@set(resolved, exports);\n" \
-    "    return exports;\n" \
-    "  } else if (last5 === \".node\") {\n" \
-    "    var module = { exports: {} };\n" \
-    "    globalThis.process.dlopen(module, resolved);\n" \
-    "    @requireMap.@set(resolved, module);\n" \
-    "    return module.exports;\n" \
-    "  } else if (last5 === \".toml\") {\n" \
-    "    var fs = (globalThis[Symbol.for(\"_fs\")] ||= Bun.fs());\n" \
-    "    var exports = Bun.TOML.parse(fs.readFileSync(resolved, \"utf8\"));\n" \
-    "    @requireMap.@set(resolved, exports);\n" \
-    "    return exports;\n" \
-    "  } else {\n" \
-    "    var exports = this.requireModule(this, resolved);\n" \
-    "    @requireMap.@set(resolved, exports);\n" \
-    "    return exports;\n" \
-    "  }\n" \
-    "})\n" \
-;
-
-const JSC::ConstructAbility s_importMetaObjectLoadModuleCodeConstructAbility = JSC::ConstructAbility::CannotConstruct;
-const JSC::ConstructorKind s_importMetaObjectLoadModuleCodeConstructorKind = JSC::ConstructorKind::None;
-const JSC::ImplementationVisibility s_importMetaObjectLoadModuleCodeImplementationVisibility = JSC::ImplementationVisibility::Public;
-const int s_importMetaObjectLoadModuleCodeLength = 2925;
-static const JSC::Intrinsic s_importMetaObjectLoadModuleCodeIntrinsic = JSC::NoIntrinsic;
-const char* const s_importMetaObjectLoadModuleCode =
-    "(function (meta, resolvedSpecifier) {\n" \
+const JSC::ConstructAbility s_importMetaObjectLoadCJS2ESMCodeConstructAbility = JSC::ConstructAbility::CannotConstruct;
+const JSC::ConstructorKind s_importMetaObjectLoadCJS2ESMCodeConstructorKind = JSC::ConstructorKind::None;
+const JSC::ImplementationVisibility s_importMetaObjectLoadCJS2ESMCodeImplementationVisibility = JSC::ImplementationVisibility::Public;
+const int s_importMetaObjectLoadCJS2ESMCodeLength = 2943;
+static const JSC::Intrinsic s_importMetaObjectLoadCJS2ESMCodeIntrinsic = JSC::NoIntrinsic;
+const char* const s_importMetaObjectLoadCJS2ESMCode =
+    "(function (resolvedSpecifier) {\n" \
     "  \"use strict\";\n" \
     "\n" \
+    "  var loader = @Loader;\n" \
     "  var queue = @createFIFO();\n" \
     "  var key = resolvedSpecifier;\n" \
     "  while (key) {\n" \
     "    //\n" \
     "    //\n" \
     "    //\n" \
-    "    var entry = Loader.registry.@get(key);\n" \
+    "    var entry = loader.registry.@get(key);\n" \
     "\n" \
     "    if (!entry || !entry.state || entry.state <= @ModuleFetch) {\n" \
     "      @fulfillModuleSync(key);\n" \
-    "      entry = Loader.registry.@get(key);\n" \
+    "      entry = loader.registry.@get(key);\n" \
     "    }\n" \
     "\n" \
     "\n" \
@@ -130,7 +83,7 @@ const char* const s_importMetaObjectLoadModuleCode =
     "    //\n" \
     "    //\n" \
     "    //\n" \
-    "    var moduleRecordPromise = Loader.parseModule(key, sourceCodeObject);\n" \
+    "    var moduleRecordPromise = loader.parseModule(key, sourceCodeObject);\n" \
     "    var module = entry.module;\n" \
     "    if (!module && moduleRecordPromise && @isPromise(moduleRecordPromise)) {\n" \
     "      var reactionsOrResult = @getPromiseInternalField(\n" \
@@ -165,7 +118,7 @@ const char* const s_importMetaObjectLoadModuleCode =
     "    //\n" \
     "    @setStateToMax(entry, @ModuleLink);\n" \
     "    var dependenciesMap = module.dependenciesMap;\n" \
-    "    var requestedModules = Loader.requestedModules(module);\n" \
+    "    var requestedModules = loader.requestedModules(module);\n" \
     "    var dependencies = @newArrayWithSize(requestedModules.length);\n" \
     "    for (var i = 0, length = requestedModules.length; i < length; ++i) {\n" \
     "      var depName = requestedModules[i];\n" \
@@ -174,8 +127,8 @@ const char* const s_importMetaObjectLoadModuleCode =
     "      var depKey =\n" \
     "        depName[0] === \"/\"\n" \
     "          ? depName\n" \
-    "          : Loader.resolve(depName, key, @undefined);\n" \
-    "      var depEntry = Loader.ensureRegistered(depKey);\n" \
+    "          : loader.resolve(depName, key, @undefined);\n" \
+    "      var depEntry = loader.ensureRegistered(depKey);\n" \
     "      if (depEntry.state < @ModuleLink) {\n" \
     "        queue.push(depKey);\n" \
     "      }\n" \
@@ -189,13 +142,13 @@ const char* const s_importMetaObjectLoadModuleCode =
     "    entry.instantiate = @Promise.resolve(entry)\n" \
     "    entry.satisfy = @Promise.resolve(entry);\n" \
     "    key = queue.shift();\n" \
-    "    while (key && (Loader.registry.@get(key)?.state ?? @ModuleFetch) >= @ModuleLink) {\n" \
+    "    while (key && (loader.registry.@get(key)?.state ?? @ModuleFetch) >= @ModuleLink) {\n" \
     "      key = queue.shift();\n" \
     "    }\n" \
     "\n" \
     "  }\n" \
     "\n" \
-    "  var linkAndEvaluateResult = Loader.linkAndEvaluateModule(\n" \
+    "  var linkAndEvaluateResult = loader.linkAndEvaluateModule(\n" \
     "    resolvedSpecifier,\n" \
     "    @undefined\n" \
     "  );\n" \
@@ -207,30 +160,29 @@ const char* const s_importMetaObjectLoadModuleCode =
     "    );\n" \
     "  }\n" \
     "\n" \
-    "  return Loader.registry.@get(resolvedSpecifier);\n" \
+    "  return loader.registry.@get(resolvedSpecifier);\n" \
     "\n" \
     "})\n" \
 ;
 
-const JSC::ConstructAbility s_importMetaObjectRequireModuleCodeConstructAbility = JSC::ConstructAbility::CannotConstruct;
-const JSC::ConstructorKind s_importMetaObjectRequireModuleCodeConstructorKind = JSC::ConstructorKind::None;
-const JSC::ImplementationVisibility s_importMetaObjectRequireModuleCodeImplementationVisibility = JSC::ImplementationVisibility::Public;
-const int s_importMetaObjectRequireModuleCodeLength = 606;
-static const JSC::Intrinsic s_importMetaObjectRequireModuleCodeIntrinsic = JSC::NoIntrinsic;
-const char* const s_importMetaObjectRequireModuleCode =
-    "(function (meta, resolved) {\n" \
+const JSC::ConstructAbility s_importMetaObjectRequireESMCodeConstructAbility = JSC::ConstructAbility::CannotConstruct;
+const JSC::ConstructorKind s_importMetaObjectRequireESMCodeConstructorKind = JSC::ConstructorKind::None;
+const JSC::ImplementationVisibility s_importMetaObjectRequireESMCodeImplementationVisibility = JSC::ImplementationVisibility::Public;
+const int s_importMetaObjectRequireESMCodeLength = 559;
+static const JSC::Intrinsic s_importMetaObjectRequireESMCodeIntrinsic = JSC::NoIntrinsic;
+const char* const s_importMetaObjectRequireESMCode =
+    "(function (resolved) {\n" \
     "  \"use strict\";\n" \
-    "  var Loader = globalThis.Loader;\n" \
-    "  var entry = Loader.registry.@get(resolved);\n" \
+    "  var entry = @Loader.registry.@get(resolved);\n" \
     "\n" \
     "  if (!entry || !entry.evaluated) {\n" \
-    "    entry = this.loadModule(meta, resolved); \n" \
+    "    entry = @loadCJS2ESM(resolved); \n" \
     "  }\n" \
     "\n" \
     "  if (!entry || !entry.evaluated || !entry.module) {\n" \
     "    @throwTypeError(`require() failed to evaluate module \\\"${resolved}\\\". This is an internal consistentency error.`);\n" \
     "  }\n" \
-    "  var exports = Loader.getModuleNamespaceObject(entry.module);\n" \
+    "  var exports = @Loader.getModuleNamespaceObject(entry.module);\n" \
     "  var commonJS = exports.default;\n" \
     "  if (commonJS && @isObject(commonJS) && @commonJSSymbol in commonJS) {\n" \
     "    return commonJS();\n" \
@@ -239,6 +191,53 @@ const char* const s_importMetaObjectRequireModuleCode =
     "})\n" \
 ;
 
+const JSC::ConstructAbility s_importMetaObjectRequireCodeConstructAbility = JSC::ConstructAbility::CannotConstruct;
+const JSC::ConstructorKind s_importMetaObjectRequireCodeConstructorKind = JSC::ConstructorKind::None;
+const JSC::ImplementationVisibility s_importMetaObjectRequireCodeImplementationVisibility = JSC::ImplementationVisibility::Public;
+const int s_importMetaObjectRequireCodeLength = 1133;
+static const JSC::Intrinsic s_importMetaObjectRequireCodeIntrinsic = JSC::NoIntrinsic;
+const char* const s_importMetaObjectRequireCode =
+    "(function (name) {\n" \
+    "  \"use strict\";\n" \
+    "  if (typeof name !== \"string\") {\n" \
+    "    @throwTypeError(\"require() expects a string as its argument\");\n" \
+    "  }\n" \
+    "  const resolved = @resolveSync(name, this.path);\n" \
+    "  var cached = @requireMap.@get(resolved);\n" \
+    "  const last5 = resolved.substring(resolved.length - 5);\n" \
+    "  if (cached) {\n" \
+    "    if (last5 === \".node\") {\n" \
+    "      return cached.exports;\n" \
+    "    }\n" \
+    "\n" \
+    "    return cached;\n" \
+    "  }\n" \
+    "\n" \
+    "  \n" \
+    "  //\n" \
+    "  if (last5 === \".json\") {\n" \
+    "    var fs = (globalThis[Symbol.for(\"_fs\")] ||= @Bun.fs());\n" \
+    "    var exports = JSON.parse(fs.readFileSync(resolved, \"utf8\"));\n" \
+    "    @requireMap.@set(resolved, exports);\n" \
+    "    return exports;\n" \
+    "  } else if (last5 === \".node\") {\n" \
+    "    var module = { exports: {} };\n" \
+    "    process.dlopen(module, resolved);\n" \
+    "    @requireMap.@set(resolved, module);\n" \
+    "    return module.exports;\n" \
+    "  } else if (last5 === \".toml\") {\n" \
+    "    var fs = (globalThis[Symbol.for(\"_fs\")] ||= @Bun.fs());\n" \
+    "    var exports = Bun.TOML.parse(fs.readFileSync(resolved, \"utf8\"));\n" \
+    "    @requireMap.@set(resolved, exports);\n" \
+    "    return exports;\n" \
+    "  } else {\n" \
+    "    var exports = @requireESM(resolved);\n" \
+    "    @requireMap.@set(resolved, exports);\n" \
+    "    return exports;\n" \
+    "  }\n" \
+    "})\n" \
+;
+
 
 #define DEFINE_BUILTIN_GENERATOR(codeName, functionName, overriddenName, argumentCount) \
 JSC::FunctionExecutable* codeName##Generator(JSC::VM& vm) \
diff --git a/src/bun.js/builtins/cpp/ImportMetaObjectBuiltins.h b/src/bun.js/builtins/cpp/ImportMetaObjectBuiltins.h
index 314079aa0..5cc424ec8 100644
--- a/src/bun.js/builtins/cpp/ImportMetaObjectBuiltins.h
+++ b/src/bun.js/builtins/cpp/ImportMetaObjectBuiltins.h
@@ -47,40 +47,40 @@ class FunctionExecutable;
 namespace WebCore {
 
 /* ImportMetaObject */
+extern const char* const s_importMetaObjectLoadCJS2ESMCode;
+extern const int s_importMetaObjectLoadCJS2ESMCodeLength;
+extern const JSC::ConstructAbility s_importMetaObjectLoadCJS2ESMCodeConstructAbility;
+extern const JSC::ConstructorKind s_importMetaObjectLoadCJS2ESMCodeConstructorKind;
+extern const JSC::ImplementationVisibility s_importMetaObjectLoadCJS2ESMCodeImplementationVisibility;
+extern const char* const s_importMetaObjectRequireESMCode;
+extern const int s_importMetaObjectRequireESMCodeLength;
+extern const JSC::ConstructAbility s_importMetaObjectRequireESMCodeConstructAbility;
+extern const JSC::ConstructorKind s_importMetaObjectRequireESMCodeConstructorKind;
+extern const JSC::ImplementationVisibility s_importMetaObjectRequireESMCodeImplementationVisibility;
 extern const char* const s_importMetaObjectRequireCode;
 extern const int s_importMetaObjectRequireCodeLength;
 extern const JSC::ConstructAbility s_importMetaObjectRequireCodeConstructAbility;
 extern const JSC::ConstructorKind s_importMetaObjectRequireCodeConstructorKind;
 extern const JSC::ImplementationVisibility s_importMetaObjectRequireCodeImplementationVisibility;
-extern const char* const s_importMetaObjectLoadModuleCode;
-extern const int s_importMetaObjectLoadModuleCodeLength;
-extern const JSC::ConstructAbility s_importMetaObjectLoadModuleCodeConstructAbility;
-extern const JSC::ConstructorKind s_importMetaObjectLoadModuleCodeConstructorKind;
-extern const JSC::ImplementationVisibility s_importMetaObjectLoadModuleCodeImplementationVisibility;
-extern const char* const s_importMetaObjectRequireModuleCode;
-extern const int s_importMetaObjectRequireModuleCodeLength;
-extern const JSC::ConstructAbility s_importMetaObjectRequireModuleCodeConstructAbility;
-extern const JSC::ConstructorKind s_importMetaObjectRequireModuleCodeConstructorKind;
-extern const JSC::ImplementationVisibility s_importMetaObjectRequireModuleCodeImplementationVisibility;
 
 #define WEBCORE_FOREACH_IMPORTMETAOBJECT_BUILTIN_DATA(macro) \
+    macro(loadCJS2ESM, importMetaObjectLoadCJS2ESM, 1) \
+    macro(requireESM, importMetaObjectRequireESM, 1) \
     macro(require, importMetaObjectRequire, 1) \
-    macro(loadModule, importMetaObjectLoadModule, 2) \
-    macro(requireModule, importMetaObjectRequireModule, 2) \
 
+#define WEBCORE_BUILTIN_IMPORTMETAOBJECT_LOADCJS2ESM 1
+#define WEBCORE_BUILTIN_IMPORTMETAOBJECT_REQUIREESM 1
 #define WEBCORE_BUILTIN_IMPORTMETAOBJECT_REQUIRE 1
-#define WEBCORE_BUILTIN_IMPORTMETAOBJECT_LOADMODULE 1
-#define WEBCORE_BUILTIN_IMPORTMETAOBJECT_REQUIREMODULE 1
 
 #define WEBCORE_FOREACH_IMPORTMETAOBJECT_BUILTIN_CODE(macro) \
+    macro(importMetaObjectLoadCJS2ESMCode, loadCJS2ESM, ASCIILiteral(), s_importMetaObjectLoadCJS2ESMCodeLength) \
+    macro(importMetaObjectRequireESMCode, requireESM, ASCIILiteral(), s_importMetaObjectRequireESMCodeLength) \
     macro(importMetaObjectRequireCode, require, ASCIILiteral(), s_importMetaObjectRequireCodeLength) \
-    macro(importMetaObjectLoadModuleCode, loadModule, ASCIILiteral(), s_importMetaObjectLoadModuleCodeLength) \
-    macro(importMetaObjectRequireModuleCode, requireModule, ASCIILiteral(), s_importMetaObjectRequireModuleCodeLength) \
 
 #define WEBCORE_FOREACH_IMPORTMETAOBJECT_BUILTIN_FUNCTION_NAME(macro) \
-    macro(loadModule) \
+    macro(loadCJS2ESM) \
     macro(require) \
-    macro(requireModule) \
+    macro(requireESM) \
 
 #define DECLARE_BUILTIN_GENERATOR(codeName, functionName, overriddenName, argumentCount) \
     JSC::FunctionExecutable* codeName##Generator(JSC::VM&);
diff --git a/src/bun.js/builtins/cpp/ReadableStreamBuiltins.cpp b/src/bun.js/builtins/cpp/ReadableStreamBuiltins.cpp
index 7db8c5f91..2b0797381 100644
--- a/src/bun.js/builtins/cpp/ReadableStreamBuiltins.cpp
+++ b/src/bun.js/builtins/cpp/ReadableStreamBuiltins.cpp
@@ -130,7 +130,7 @@ const char* const s_readableStreamInitializeReadableStreamCode =
 
 const JSC::ConstructAbility s_readableStreamReadableStreamToArrayCodeConstructAbility = JSC::ConstructAbility::CannotConstruct;
 const JSC::ConstructorKind s_readableStreamReadableStreamToArrayCodeConstructorKind = JSC::ConstructorKind::None;
-const JSC::ImplementationVisibility s_readableStreamReadableStreamToArrayCodeImplementationVisibility = JSC::ImplementationVisibility::Public;
+const JSC::ImplementationVisibility s_readableStreamReadableStreamToArrayCodeImplementationVisibility = JSC::ImplementationVisibility::Private;
 const int s_readableStreamReadableStreamToArrayCodeLength = 294;
 static const JSC::Intrinsic s_readableStreamReadableStreamToArrayCodeIntrinsic = JSC::NoIntrinsic;
 const char* const s_readableStreamReadableStreamToArrayCode =
@@ -149,7 +149,7 @@ const char* const s_readableStreamReadableStreamToArrayCode =
 
 const JSC::ConstructAbility s_readableStreamReadableStreamToTextCodeConstructAbility = JSC::ConstructAbility::CannotConstruct;
 const JSC::ConstructorKind s_readableStreamReadableStreamToTextCodeConstructorKind = JSC::ConstructorKind::None;
-const JSC::ImplementationVisibility s_readableStreamReadableStreamToTextCodeImplementationVisibility = JSC::ImplementationVisibility::Public;
+const JSC::ImplementationVisibility s_readableStreamReadableStreamToTextCodeImplementationVisibility = JSC::ImplementationVisibility::Private;
 const int s_readableStreamReadableStreamToTextCodeLength = 292;
 static const JSC::Intrinsic s_readableStreamReadableStreamToTextCodeIntrinsic = JSC::NoIntrinsic;
 const char* const s_readableStreamReadableStreamToTextCode =
@@ -168,7 +168,7 @@ const char* const s_readableStreamReadableStreamToTextCode =
 
 const JSC::ConstructAbility s_readableStreamReadableStreamToArrayBufferCodeConstructAbility = JSC::ConstructAbility::CannotConstruct;
 const JSC::ConstructorKind s_readableStreamReadableStreamToArrayBufferCodeConstructorKind = JSC::ConstructorKind::None;
-const JSC::ImplementationVisibility s_readableStreamReadableStreamToArrayBufferCodeImplementationVisibility = JSC::ImplementationVisibility::Public;
+const JSC::ImplementationVisibility s_readableStreamReadableStreamToArrayBufferCodeImplementationVisibility = JSC::ImplementationVisibility::Private;
 const int s_readableStreamReadableStreamToArrayBufferCodeLength = 334;
 static const JSC::Intrinsic s_readableStreamReadableStreamToArrayBufferCodeIntrinsic = JSC::NoIntrinsic;
 const char* const s_readableStreamReadableStreamToArrayBufferCode =
@@ -188,7 +188,7 @@ const char* const s_readableStreamReadableStreamToArrayBufferCode =
 
 const JSC::ConstructAbility s_readableStreamReadableStreamToJSONCodeConstructAbility = JSC::ConstructAbility::CannotConstruct;
 const JSC::ConstructorKind s_readableStreamReadableStreamToJSONCodeConstructorKind = JSC::ConstructorKind::None;
-const JSC::ImplementationVisibility s_readableStreamReadableStreamToJSONCodeImplementationVisibility = JSC::ImplementationVisibility::Public;
+const JSC::ImplementationVisibility s_readableStreamReadableStreamToJSONCodeImplementationVisibility = JSC::ImplementationVisibility::Private;
 const int s_readableStreamReadableStreamToJSONCodeLength = 118;
 static const JSC::Intrinsic s_readableStreamReadableStreamToJSONCodeIntrinsic = JSC::NoIntrinsic;
 const char* const s_readableStreamReadableStreamToJSONCode =
@@ -201,7 +201,7 @@ const char* const s_readableStreamReadableStreamToJSONCode =
 
 const JSC::ConstructAbility s_readableStreamReadableStreamToBlobCodeConstructAbility = JSC::ConstructAbility::CannotConstruct;
 const JSC::ConstructorKind s_readableStreamReadableStreamToBlobCodeConstructorKind = JSC::ConstructorKind::None;
-const JSC::ImplementationVisibility s_readableStreamReadableStreamToBlobCodeImplementationVisibility = JSC::ImplementationVisibility::Public;
+const JSC::ImplementationVisibility s_readableStreamReadableStreamToBlobCodeImplementationVisibility = JSC::ImplementationVisibility::Private;
 const int s_readableStreamReadableStreamToBlobCodeLength = 139;
 static const JSC::Intrinsic s_readableStreamReadableStreamToBlobCodeIntrinsic = JSC::NoIntrinsic;
 const char* const s_readableStreamReadableStreamToBlobCode =
@@ -213,7 +213,7 @@ const char* const s_readableStreamReadableStreamToBlobCode =
 
 const JSC::ConstructAbility s_readableStreamConsumeReadableStreamCodeConstructAbility = JSC::ConstructAbility::CannotConstruct;
 const JSC::ConstructorKind s_readableStreamConsumeReadableStreamCodeConstructorKind = JSC::ConstructorKind::None;
-const JSC::ImplementationVisibility s_readableStreamConsumeReadableStreamCodeImplementationVisibility = JSC::ImplementationVisibility::Public;
+const JSC::ImplementationVisibility s_readableStreamConsumeReadableStreamCodeImplementationVisibility = JSC::ImplementationVisibility::Private;
 const int s_readableStreamConsumeReadableStreamCodeLength = 3718;
 static const JSC::Intrinsic s_readableStreamConsumeReadableStreamCodeIntrinsic = JSC::NoIntrinsic;
 const char* const s_readableStreamConsumeReadableStreamCode =
@@ -336,7 +336,7 @@ const char* const s_readableStreamConsumeReadableStreamCode =
 
 const JSC::ConstructAbility s_readableStreamCreateEmptyReadableStreamCodeConstructAbility = JSC::ConstructAbility::CannotConstruct;
 const JSC::ConstructorKind s_readableStreamCreateEmptyReadableStreamCodeConstructorKind = JSC::ConstructorKind::None;
-const JSC::ImplementationVisibility s_readableStreamCreateEmptyReadableStreamCodeImplementationVisibility = JSC::ImplementationVisibility::Public;
+const JSC::ImplementationVisibility s_readableStreamCreateEmptyReadableStreamCodeImplementationVisibility = JSC::ImplementationVisibility::Private;
 const int s_readableStreamCreateEmptyReadableStreamCodeLength = 156;
 static const JSC::Intrinsic s_readableStreamCreateEmptyReadableStreamCodeIntrinsic = JSC::NoIntrinsic;
 const char* const s_readableStreamCreateEmptyReadableStreamCode =
@@ -353,7 +353,7 @@ const char* const s_readableStreamCreateEmptyReadableStreamCode =
 
 const JSC::ConstructAbility s_readableStreamCreateNativeReadableStreamCodeConstructAbility = JSC::ConstructAbility::CannotConstruct;
 const JSC::ConstructorKind s_readableStreamCreateNativeReadableStreamCodeConstructorKind = JSC::ConstructorKind::None;
-const JSC::ImplementationVisibility s_readableStreamCreateNativeReadableStreamCodeImplementationVisibility = JSC::ImplementationVisibility::Public;
+const JSC::ImplementationVisibility s_readableStreamCreateNativeReadableStreamCodeImplementationVisibility = JSC::ImplementationVisibility::Private;
 const int s_readableStreamCreateNativeReadableStreamCodeLength = 266;
 static const JSC::Intrinsic s_readableStreamCreateNativeReadableStreamCodeIntrinsic = JSC::NoIntrinsic;
 const char* const s_readableStreamCreateNativeReadableStreamCode =
diff --git a/src/bun.js/builtins/cpp/StreamInternalsBuiltins.cpp b/src/bun.js/builtins/cpp/StreamInternalsBuiltins.cpp
index 0a1a8143e..edaa6b89b 100644
--- a/src/bun.js/builtins/cpp/StreamInternalsBuiltins.cpp
+++ b/src/bun.js/builtins/cpp/StreamInternalsBuiltins.cpp
@@ -194,7 +194,7 @@ const char* const s_streamInternalsValidateAndNormalizeQueuingStrategyCode =
 
 const JSC::ConstructAbility s_streamInternalsCreateFIFOCodeConstructAbility = JSC::ConstructAbility::CannotConstruct;
 const JSC::ConstructorKind s_streamInternalsCreateFIFOCodeConstructorKind = JSC::ConstructorKind::None;
-const JSC::ImplementationVisibility s_streamInternalsCreateFIFOCodeImplementationVisibility = JSC::ImplementationVisibility::Public;
+const JSC::ImplementationVisibility s_streamInternalsCreateFIFOCodeImplementationVisibility = JSC::ImplementationVisibility::Private;
 const int s_streamInternalsCreateFIFOCodeLength = 2764;
 static const JSC::Intrinsic s_streamInternalsCreateFIFOCodeIntrinsic = JSC::NoIntrinsic;
 const char* const s_streamInternalsCreateFIFOCode =
diff --git a/src/bun.js/builtins/js/ReadableStream.js b/src/bun.js/builtins/js/ReadableStream.js
index a8aa5d8f3..8496b2d4b 100644
--- a/src/bun.js/builtins/js/ReadableStream.js
+++ b/src/bun.js/builtins/js/ReadableStream.js
@@ -98,7 +98,7 @@ function initializeReadableStream(underlyingSource, strategy)
 }
 
 
-@globalPrivate
+@linkTimeConstant
 function readableStreamToArray(stream) {
     "use strict";
 
@@ -111,7 +111,7 @@ function readableStreamToArray(stream) {
     return @readableStreamIntoArray(stream);
 }
 
-@globalPrivate
+@linkTimeConstant
 function readableStreamToText(stream) {
     "use strict";
 
@@ -124,7 +124,7 @@ function readableStreamToText(stream) {
     return @readableStreamIntoText(stream);
 }
 
-@globalPrivate
+@linkTimeConstant
 function readableStreamToArrayBuffer(stream) {
     "use strict";
 
@@ -138,20 +138,20 @@ function readableStreamToArrayBuffer(stream) {
     return @Bun.readableStreamToArray(stream).@then(@Bun.concatArrayBuffers);
 }
 
-@globalPrivate
+@linkTimeConstant
 function readableStreamToJSON(stream) {
     "use strict";
 
     return @Bun.readableStreamToText(stream).@then(globalThis.JSON.parse);
 }
 
-@globalPrivate
+@linkTimeConstant
 function readableStreamToBlob(stream) {
     "use strict";
     return @Promise.resolve(@Bun.readableStreamToArray(stream)).@then(array => new Blob(array));
 }
 
-@globalPrivate
+@linkTimeConstant
 function consumeReadableStream(nativePtr, nativeType, inputStream) {
     "use strict";
     const symbol = globalThis.Symbol.for("Bun.consumeReadableStreamPrototype");
@@ -268,7 +268,7 @@ function consumeReadableStream(nativePtr, nativeType, inputStream) {
    return new Prototype(inputStream.getReader(), nativePtr);
 }
 
-@globalPrivate
+@linkTimeConstant
 function createEmptyReadableStream() {
     "use strict";
 
@@ -279,7 +279,7 @@ function createEmptyReadableStream() {
     return stream;
 }
 
-@globalPrivate
+@linkTimeConstant
 function createNativeReadableStream(nativePtr, nativeType, autoAllocateChunkSize) {
     "use strict";
     return new @ReadableStream({
diff --git a/src/bun.js/builtins/js/ReadableStreamInternals.js b/src/bun.js/builtins/js/ReadableStreamInternals.js
index a6604a014..7be9410db 100644
--- a/src/bun.js/builtins/js/ReadableStreamInternals.js
+++ b/src/bun.js/builtins/js/ReadableStreamInternals.js
@@ -849,7 +849,7 @@ function readDirectStream(stream, sink, underlyingSource) {
 
 }
 
-@globalPrivate;
+@linkTimeConstant;
 function assignToStream(stream, sink) {
   "use strict";