From 917e7441431f84527623e2de851b1d7e9556e7b9 Mon Sep 17 00:00:00 2001
From: Jordan Harband <ljharb@gmail.com>
Date: Wed, 24 Apr 2024 20:57:29 -0700
Subject: [PATCH] lib: make navigator not runtime-lookup
 process.version/arch/platform

Preserves most of #53649, except for `#language` due to the lack of Intl primordials.
---
 lib/internal/bootstrap/node.js   |  4 +++
 lib/internal/navigator.js        | 49 ++++++++++++++++----------------
 lib/internal/process/arch.js     |  1 +
 lib/internal/process/platform.js |  1 +
 lib/internal/process/version.js  |  1 +
 test/parallel/test-navigator.js  | 22 +++++++++++++-
 6 files changed, 53 insertions(+), 25 deletions(-)
 create mode 100644 lib/internal/process/arch.js
 create mode 100644 lib/internal/process/platform.js
 create mode 100644 lib/internal/process/version.js

diff --git a/lib/internal/bootstrap/node.js b/lib/internal/bootstrap/node.js
index d634b0405dbadd..b4ce9ea8f7bedd 100644
--- a/lib/internal/bootstrap/node.js
+++ b/lib/internal/bootstrap/node.js
@@ -85,6 +85,10 @@ const {
   },
 } = internalBinding('util');
 
+require('internal/process/arch'); // ensure the process arch is cached
+require('internal/process/platform'); // ensure the process platform is cached
+require('internal/process/version'); // ensure the process version is cached
+
 setupProcessObject();
 
 setupGlobalProxy();
diff --git a/lib/internal/navigator.js b/lib/internal/navigator.js
index e56b2da5e339e3..26d90185146394 100644
--- a/lib/internal/navigator.js
+++ b/lib/internal/navigator.js
@@ -27,7 +27,9 @@ const {
 } = internalBinding('os');
 
 const kInitialize = Symbol('kInitialize');
-const nodeVersion = process.version;
+const nodeVersion = require('internal/process/version');
+const platform = require('internal/process/platform');
+const arch = require('internal/process/arch');
 
 /**
  * @param {object} process
@@ -35,44 +37,44 @@ const nodeVersion = process.version;
  * @param {string} process.arch
  * @returns {string}
  */
-function getNavigatorPlatform(process) {
-  if (process.platform === 'darwin') {
+function getNavigatorPlatform({ platform, arch }) {
+  if (platform === 'darwin') {
     // On macOS, modern browsers return 'MacIntel' even if running on Apple Silicon.
     return 'MacIntel';
-  } else if (process.platform === 'win32') {
+  } else if (platform === 'win32') {
     // On Windows, modern browsers return 'Win32' even if running on a 64-bit version of Windows.
     // https://developer.mozilla.org/en-US/docs/Web/API/Navigator/platform#usage_notes
     return 'Win32';
-  } else if (process.platform === 'linux') {
-    if (process.arch === 'ia32') {
+  } else if (platform === 'linux') {
+    if (arch === 'ia32') {
       return 'Linux i686';
-    } else if (process.arch === 'x64') {
+    } else if (arch === 'x64') {
       return 'Linux x86_64';
     }
-    return `Linux ${process.arch}`;
-  } else if (process.platform === 'freebsd') {
-    if (process.arch === 'ia32') {
+    return `Linux ${arch}`;
+  } else if (platform === 'freebsd') {
+    if (arch === 'ia32') {
       return 'FreeBSD i386';
-    } else if (process.arch === 'x64') {
+    } else if (arch === 'x64') {
       return 'FreeBSD amd64';
     }
-    return `FreeBSD ${process.arch}`;
-  } else if (process.platform === 'openbsd') {
-    if (process.arch === 'ia32') {
+    return `FreeBSD ${arch}`;
+  } else if (platform === 'openbsd') {
+    if (arch === 'ia32') {
       return 'OpenBSD i386';
-    } else if (process.arch === 'x64') {
+    } else if (arch === 'x64') {
       return 'OpenBSD amd64';
     }
-    return `OpenBSD ${process.arch}`;
-  } else if (process.platform === 'sunos') {
-    if (process.arch === 'ia32') {
+    return `OpenBSD ${arch}`;
+  } else if (platform === 'sunos') {
+    if (arch === 'ia32') {
       return 'SunOS i86pc';
     }
-    return `SunOS ${process.arch}`;
-  } else if (process.platform === 'aix') {
+    return `SunOS ${arch}`;
+  } else if (platform === 'aix') {
     return 'AIX';
   }
-  return `${StringPrototypeToUpperCase(process.platform[0])}${StringPrototypeSlice(process.platform, 1)} ${process.arch}`;
+  return `${StringPrototypeToUpperCase(platform[0])}${StringPrototypeSlice(platform, 1)} ${arch}`;
 }
 
 class Navigator {
@@ -80,7 +82,7 @@ class Navigator {
   #availableParallelism;
   #userAgent;
   #platform;
-  #language;
+  #language = Intl?.Collator().resolvedOptions().locale || 'en-US';
   #languages;
 
   constructor() {
@@ -102,7 +104,6 @@ class Navigator {
    * @return {string}
    */
   get language() {
-    this.#language ??= Intl?.Collator().resolvedOptions().locale || 'en-US';
     return this.#language;
   }
 
@@ -126,7 +127,7 @@ class Navigator {
    * @return {string}
    */
   get platform() {
-    this.#platform ??= getNavigatorPlatform(process);
+    this.#platform ??= getNavigatorPlatform({ arch, platform });
     return this.#platform;
   }
 }
diff --git a/lib/internal/process/arch.js b/lib/internal/process/arch.js
new file mode 100644
index 00000000000000..ecf9b0c3ed6bb9
--- /dev/null
+++ b/lib/internal/process/arch.js
@@ -0,0 +1 @@
+module.exports = require('process').arch;
diff --git a/lib/internal/process/platform.js b/lib/internal/process/platform.js
new file mode 100644
index 00000000000000..9d18406d10d77f
--- /dev/null
+++ b/lib/internal/process/platform.js
@@ -0,0 +1 @@
+module.exports = require('process').platform;
diff --git a/lib/internal/process/version.js b/lib/internal/process/version.js
new file mode 100644
index 00000000000000..dd373f98250fce
--- /dev/null
+++ b/lib/internal/process/version.js
@@ -0,0 +1 @@
+module.exports = require('process').version;
diff --git a/test/parallel/test-navigator.js b/test/parallel/test-navigator.js
index 53949cea246d16..581f1530dca6eb 100644
--- a/test/parallel/test-navigator.js
+++ b/test/parallel/test-navigator.js
@@ -2,8 +2,28 @@
 
 'use strict';
 
-const common = require('../common');
 const assert = require('assert');
+
+{
+
+  assert.strictEqual(require.resolve('../common') in require.cache, false); // ensures `navigator` has not been evaluated yet
+
+  let called = false;
+  const { version, platform, arch } = process;
+  Object.defineProperty(process, 'arch', { get() { called += 'arch|'; return arch; } });
+  Object.defineProperty(process, 'platform', { get() { called = 'platform|'; return platform; } });
+  Object.defineProperty(process, 'version', { get() { called = 'version|'; return version; } });
+
+  navigator;
+
+  assert.strictEqual(called, false, 'evaluating navigator does not runtime-call `process.version`, `process.platform`, `platform.version`');
+
+  Object.defineProperty(process, 'arch', { value: arch });
+  Object.defineProperty(process, 'platform', { value: platform });
+  Object.defineProperty(process, 'version', { value: version });
+}
+
+const common = require('../common');
 const { getNavigatorPlatform } = require('internal/navigator');
 const { execFile } = require('child_process');