lib: make internal/options lazy

This way, internal modules can still require the module and cache the function getOptionValue() early (therefore these code can be included in the snapshots), but the options map won't be serialized from C++ land until the option values are actually queried. PR-URL: #38993 Refs: #35711 Refs: #38905 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Zijian Liu <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>

Author: joyeecheung

lib/internal/options.js

@@ -1,12 +1,32 @@
 'use strict';
 
 const{ getOptions, shouldNotRegisterESMLoader }=internalBinding('options');
-const{ options, aliases }=getOptions();
 
 letwarnOnAllowUnauthorized=true;
 
+letoptionsMap;
+letaliasesMap;
+
+// getOptions() would serialize the option values from C++ land.
+// It would error if the values are queried before bootstrap is
+// complete so that we don't accidentally include runtime-dependent
+// states into a runtime-independent snapshot.
+functiongetOptionsFromBinding(){
+if(!optionsMap){
+({options: optionsMap}=getOptions());
+}
+returnoptionsMap;
+}
+
+functiongetAliasesFromBinding(){
+if(!aliasesMap){
+({aliases: aliasesMap}=getOptions());
+}
+returnaliasesMap;
+}
+
 functiongetOptionValue(option){
-returnoptions.get(option)?.value;
+returngetOptionsFromBinding().get(option)?.value;
 }
 
 functiongetAllowUnauthorized(){
@@ -24,8 +44,12 @@ function getAllowUnauthorized(){
 }
 
 module.exports={
- options,
- aliases,
+getoptions(){
+returngetOptionsFromBinding();
+},
+getaliases(){
+returngetAliasesFromBinding();
+},
  getOptionValue,
  getAllowUnauthorized,
  shouldNotRegisterESMLoader