report: refactor configuration management

This commit removes process.report.setOptions(). Instead of
using complex configuration synchronization between C++ and
JS, this commit introduces individual getters and setters.

PR-URL: https://github.com/nodejs/node/pull/26414
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Wyatt Preul <wpreul@gmail.com>

1 files changed, 5 insertions, 14 deletions

diff --git a/lib/internal/bootstrap/pre_execution.js b/lib/internal/bootstrap/pre_execution.js
index 3578e7c112..b6e606cf9d 100644
--- a/lib/internal/bootstrap/pre_execution.js
+++ b/lib/internal/bootstrap/pre_execution.js
@@ -40,14 +40,8 @@ function initializeReport() {
   if (!getOptionValue('--experimental-report')) {
     return;
   }
-  const {
-    config,
-    report,
-    syncConfig
-  } = require('internal/process/report');
+  const { report } = require('internal/process/report');
   process.report = report;
-  // Download the CLI / ENV config into JS land.
-  syncConfig(config, false);
 }
 
 function setupSignalHandlers() {
@@ -68,13 +62,10 @@ function initializeReportSignalHandlers() {
   if (!getOptionValue('--experimental-report')) {
     return;
   }
-  const {
-    config,
-    handleSignal
-  } = require('internal/process/report');
-  if (config.events.includes('signal')) {
-    process.on(config.signal, handleSignal);
-  }
+
+  const { addSignalHandler } = require('internal/process/report');
+
+  addSignalHandler();
 }
 
 function setupTraceCategoryState() {