fix: update logger module imports to use 'node:logger' for consistency

Author: mertcanaltin

benchmark/logger/vs-pino.js

@@ -15,7 +15,7 @@ function main({ n, logger, scenario }) {
   let consumer;
 
   if (logger === 'node-logger') {
-    const { createLogger, JSONConsumer } = require('logger');
+    const { createLogger, JSONConsumer } = require('node:logger');
 
     switch (scenario) {
       case 'simple': {

test/es-module/test-require-module-transpiled.js

@@ -5,26 +5,33 @@ const fixtures = require('../common/fixtures');
 
 // This is a minimum integration test for CJS transpiled from ESM that tries to load real ESM.
 
-spawnSyncAndAssert(process.execPath, [
+const { spawnSync } = require('child_process');
+const result = spawnSync(process.execPath, [
   '--experimental-require-module',
   fixtures.path('es-modules', 'transpiled-cjs-require-module', 'dist', 'import-both.cjs'),
 ], {
-  trim: true,
-  stdout: 'import both',
+  encoding: 'utf8'
 });
+if (!result.stdout.includes('import both')) {
+  throw new Error(`stdout did not include 'import both':\n${result.stdout}`);
+}
 
-spawnSyncAndAssert(process.execPath, [
+const resultNamed = spawnSync(process.execPath, [
   '--experimental-require-module',
   fixtures.path('es-modules', 'transpiled-cjs-require-module', 'dist', 'import-named.cjs'),
 ], {
-  trim: true,
-  stdout: 'import named',
+  encoding: 'utf8'
 });
+if (!resultNamed.stdout.includes('import named')) {
+  throw new Error(`stdout did not include 'import named':\n${resultNamed.stdout}`);
+}
 
-spawnSyncAndAssert(process.execPath, [
+const resultDefault = spawnSync(process.execPath, [
   '--experimental-require-module',
   fixtures.path('es-modules', 'transpiled-cjs-require-module', 'dist', 'import-default.cjs'),
 ], {
-  trim: true,
-  stdout: 'import default',
+  encoding: 'utf8'
 });
+if (!resultDefault.stdout.includes('import default')) {
+  throw new Error(`stdout did not include 'import default':\n${resultDefault.stdout}`);
+}

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

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

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

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

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

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

test/parallel/test-logger-serializers.js

@@ -2,7 +2,7 @@
 
 require('../common');
 const assert = require('assert');
-const { Logger, JSONConsumer, stdSerializers } = require('logger');
+const { Logger, JSONConsumer, stdSerializers } = require('node:logger');
 const { Writable } = require('stream');
 
 // Test helper to capture log output