Revert "Revert "Revert "NodeJS Compat: process features (#4271)"""

This reverts commit df63af9.

Author: danlapid

build/wpt_test.bzl

@@ -51,7 +51,7 @@ def wpt_test(name, wpt_directory, config, compat_date = "", compat_flags = [], a
         compat_date_path = compat_date_path,
         autogates = autogates,
         wpt_cacert = wpt_cacert,
-        compat_flags = compat_flags + ["experimental", "nodejs_compat", "unsupported_process_real_platform"],
+        compat_flags = compat_flags + ["experimental", "nodejs_compat"],
     )
 
     data = [

src/node/events.ts

@@ -3,20 +3,5 @@
 //     https://opensource.org/licenses/Apache-2.0
 //
 
-// Explicit list used to hide _initialized private export
-export {
-  EventEmitter,
-  EventEmitterAsyncResource,
-  addAbortListener,
-  captureRejectionSymbol,
-  errorMonitor,
-  getMaxListeners,
-  usingDomains,
-  defaultMaxListeners,
-  setMaxListeners,
-  listenerCount,
-  getEventListeners,
-  once,
-  on,
-} from 'node-internal:events';
+export * from 'node-internal:events';
 export { default } from 'node-internal:events';

src/node/internal/events.ts

@@ -42,7 +42,7 @@ import {
   validateObject,
 } from 'node-internal:validators';
 
-import * as process from 'node-internal:internal_process';
+import * as process from 'node-internal:process';
 
 import { spliceOne } from 'node-internal:internal_utils';
 
@@ -163,8 +163,6 @@ export function addAbortListener(signal: AbortSignal, listener: any) {
 }
 
 export default EventEmitter;
-
-// NOTE: all properties of event emitter must also be re-exported by the events.ts wrapper
 EventEmitter.on = on;
 EventEmitter.once = once;
 EventEmitter.getEventListeners = getEventListeners;
@@ -497,34 +495,26 @@ function _addListener(
           `added to an EventEmitter. Use ` +
           'emitter.setMaxListeners() to increase limit'
       );
-      const w: EventEmitterError = Object.assign(
-        new Error(
-          'Possible EventEmitter memory leak detected. ' +
-            `${existing.length} ${String(type)} listeners ` +
-            `added to ${inspect(target, { depth: -1 })}. Use ` +
-            'emitter.setMaxListeners() to increase limit'
-        ),
-        {
-          name: 'MaxListenersExceededWarning',
-          emitter: target,
-          type: type,
-          count: existing.length,
-        }
-      );
-      process.emitWarning(w);
+      // TODO(soon): Implement process.emitWarning and inspect
+      // // No error code for this since it is a Warning
+      // // eslint-disable-next-line no-restricted-syntax
+      // const w = new Error(
+      //   "Possible EventEmitter memory leak detected. " +
+      //     `${existing.length} ${String(type)} listeners ` +
+      //     `added to ${inspect(target, { depth: -1 })}. Use ` +
+      //     "emitter.setMaxListeners() to increase limit",
+      // );
+      // w.name = "MaxListenersExceededWarning";
+      // w.emitter = target;
+      // w.type = type;
+      // w.count = existing.length;
+      // process.emitWarning(w);
     }
   }
 
   return target;
 }
 
-interface EventEmitterError extends Error {
-  name: string;
-  emitter: unknown;
-  type: string | symbol;
-  count: number;
-}
-
 EventEmitter.prototype.addListener = function addListener(
   type: string | symbol,
   listener: unknown
@@ -1030,8 +1020,3 @@ export function on(
     iterator.return();
   }
 }
-
-// Initialize the process global as an event emitter, if process executed first
-if (process._initialized) process._initProcess();
-
-export var _initialized = true;

src/node/internal/internal_errors.ts

@@ -862,13 +862,6 @@ export class ERR_EEXIST extends NodeSyscallError {
   }
 }
 
-export class ERR_EPERM extends NodeSyscallError {
-  constructor(options: { syscall: string; errno?: number }) {
-    super('EPERM', 'Operation not permitted', options.syscall);
-    this.errno = options.errno || 1; // EPERM
-  }
-}
-
 export class ERR_DIR_CLOSED extends NodeError {
   constructor() {
     super('ERR_DIR_CLOSED', 'Directory is closed');