refactor(cloudflare): make node:util/types a nodejs_compat module

src/presets/cloudflare.ts

@@ -19,6 +19,7 @@ const cloudflareNodeCompatModules = [
   "stream/promises",
   "stream/web",
   "string_decoder",
+  "util/types",
   "zlib",
 ];
 
@@ -33,7 +34,6 @@ const hybridNodeCompatModules = [
   "process",
   "timers",
   "util",
-  "util/types",
 ];
 
 const cloudflarePreset: Preset = {

src/runtime/node/util/$cloudflare.ts

@@ -49,7 +49,6 @@ import {
   isUndefined,
   parseEnv,
   styleText,
-  types as unenvUtilTypes,
 } from "./index";
 
 const workerdUtil = process.getBuiltinModule("node:util");
@@ -81,12 +80,7 @@ export const {
   transferableAbortSignal,
 } = workerdUtil;
 
-// TODO(cloudflare): we should just implement this in workerd and drop this special case.
-export const types = {
-  ...workerdUtil.types,
-  isExternal: unenvUtilTypes.isExternal,
-  isAnyArrayBuffer: workerdUtil.types.isAnyArrayBuffer,
-} satisfies typeof nodeUtil.types;
+export const types = workerdUtil.types;
 
 export default {
   /**

src/runtime/node/util/internal/types.ts

@@ -1,9 +1,7 @@
 import type utilTypes from "node:util/types";
 import { notImplemented } from "../../../_internal/utils";
 
-export const isExternal = notImplemented<typeof utilTypes.isExternal>(
-  "util.types.isExternal",
-);
+export const isExternal: typeof utilTypes.isExternal = (_obj) => false;
 
 export const isDate: typeof utilTypes.isDate = (val): val is Date =>
   val instanceof Date;

test/workerd/tests.mjs

@@ -58,3 +58,13 @@ export const buffer_implements = {
     assert.ok(new Buffer.Blob([]));
   },
 };
+
+// --- node:util
+
+export const util_implements = {
+  async test() {
+    const { types } = await import("unenv/runtime/node/util");
+    assert.strictEqual(types.isExternal("hello world"), false);
+    assert.strictEqual(types.isAnyArrayBuffer(new ArrayBuffer(0)), true);
+  },
+};