add experimental logger module and update related tests

Author: mertcanaltin

doc/api/cli.md

@@ -1205,6 +1205,17 @@ Specify the `module` containing exported [asynchronous module customization hook
 
 This feature requires `--allow-worker` if used with the [Permission Model][].
 
+### `--experimental-logger`
+
+<!-- YAML
+added: REPLACEME
+-->
+
+> Stability: 1 - Experimental
+
+Enable the experimental `node:logger` module for structured logging.
+See the [Logger][] documentation for more details.
+
 ### `--experimental-network-inspection`
 
 <!-- YAML
@@ -3594,6 +3605,7 @@ one is included in the list below.
 * `--experimental-import-meta-resolve`
 * `--experimental-json-modules`
 * `--experimental-loader`
+* `--experimental-logger`
 * `--experimental-modules`
 * `--experimental-print-required-tla`
 * `--experimental-quic`
@@ -4177,6 +4189,7 @@ node --stack-trace-limit=12 -p -e "Error.stackTraceLimit" # prints 12
 [ExperimentalWarning: `vm.measureMemory` is an experimental feature]: vm.md#vmmeasurememoryoptions
 [File System Permissions]: permissions.md#file-system-permissions
 [Loading ECMAScript modules using `require()`]: modules.md#loading-ecmascript-modules-using-require
+[Logger]: logger.md
 [Module resolution and loading]: packages.md#module-resolution-and-loading
 [Navigator API]: globals.md#navigator
 [Node.js issue tracker]: https://github.com/nodejs/node/issues

test/fixtures/es-modules/transpiled-cjs-require-module/dist/import-both.cjs

@@ -24,6 +24,4 @@ var __importStar = (this && this.__importStar) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 var logger_1 = __importStar(require("logger"));
-var logger = logger_1.createLogger();
-new logger_1.JSONConsumer().attach();
-logger.info('import both');
+(0, logger_1.log)(new logger_1.default(), 'import both');

test/fixtures/es-modules/transpiled-cjs-require-module/dist/import-default.cjs

@@ -4,6 +4,4 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 var logger_1 = __importDefault(require("logger"));
-new logger_1.default.JSONConsumer().attach();
-var logger = logger_1.default.createLogger();
-logger.info('import default');
+new logger_1.default().log('import default');

test/fixtures/es-modules/transpiled-cjs-require-module/dist/import-named.cjs

@@ -1,6 +1,4 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 var logger_1 = require("logger");
-new logger_1.JSONConsumer().attach();
-var logger = logger_1.createLogger();
-logger.info('import named');
+(0, logger_1.log)(console, 'import named');

test/parallel/test-code-cache.js

@@ -17,6 +17,8 @@ const {
 } = internalBinding('builtins');
 
 for (const key of canBeRequired) {
+  // Skip experimental modules that require a flag
+  if (key === 'logger') continue;
   require(`node:${key}`);
 }
 

test/parallel/test-process-get-builtin.mjs

@@ -38,6 +38,8 @@ if (!hasIntl) {
 }
 // TODO(@jasnell): Remove this once node:quic graduates from unflagged.
 publicBuiltins.delete('node:quic');
+// node:logger requires --experimental-logger flag.
+publicBuiltins.delete('node:logger');
 
 if (!hasInspector) {
   publicBuiltins.delete('inspector');

test/parallel/test-require-resolve.js

@@ -62,6 +62,8 @@ require(fixtures.path('resolve-paths', 'default', 'verify-paths.js'));
   builtinModules.forEach((mod) => {
     // TODO(@jasnell): Remove once node:quic is no longer flagged
     if (mod === 'node:quic') return;
+    // node:logger requires --experimental-logger flag
+    if (mod === 'node:logger') return;
     assert.strictEqual(require.resolve.paths(mod), null);
     if (!mod.startsWith('node:')) {
       assert.strictEqual(require.resolve.paths(`node:${mod}`), null);