@contrast/agent 4.10.5 → 4.12.0

package/lib/core/config/options.js

@@ -33,8 +33,11 @@ Copyright: 2022 Contrast Security, Inc
  * @module lib/core/config/options
  */
 'use strict';
-const { Command, Option } = require('commander');
+const { Command } = require('commander');
+
 const program = new Command();
+program.allowUnknownOption();
+
 const os = require('os');
 const url = require('url');
 const path = require('path');
@@ -42,7 +45,7 @@ const util = require('util');
 const _ = require('lodash');
 
 const configPathEnvVars = {
-  path: 'CONTRAST_CONFIG_PATH'
+  path: 'CONTRAST_CONFIG_PATH',
 };
 
 /**
@@ -111,9 +114,8 @@ const misc = [
     // special case this guy because it should be settable via ENV
     env: configPathEnvVars.path,
     arg: '<path>',
-    desc:
-      'set config file location. defaults to <app_root>/contrast_security.yaml'
-  }
+    desc: 'set config file location. defaults to <app_root>/contrast_security.yaml',
+  },
 ];
 
 const api = [
@@ -122,21 +124,21 @@ const api = [
     arg: '[false]',
     fn: castBoolean,
     default: true,
-    desc: 'set false to disable reporting'
+    desc: 'set false to disable reporting',
   },
   {
     name: 'api.api_key',
     env: 'CONTRASTSECURITY_API_KEY',
     arg: '<key>',
     desc: 'the organization API key',
-    required: true
+    required: true,
   },
   {
     name: 'api.service_key',
     env: 'CONTRASTSECURITY_SECRET_KEY',
     arg: '<key>',
     desc: 'account service key',
-    required: true
+    required: true,
   },
   {
     name: 'api.url',
@@ -175,71 +177,66 @@ const api = [
       return value;
     },
     desc: 'url to report on',
-    required: true
+    required: true,
   },
   {
     name: 'api.user_name',
     env: 'CONTRASTSECURITY_UID',
     arg: '<name>',
     desc: 'account user name',
-    required: true
+    required: true,
   },
 
   {
     name: 'api.certificate.enable',
     arg: '[false]',
     default: 'false',
-    desc:
-      'If set to false, the certificate configuration in this section will be ignored.'
+    desc: 'If set to false, the certificate configuration in this section will be ignored.',
   },
   {
     name: 'api.certificate.ca_file',
     arg: '<path>',
-    desc:
-      'Set the absolute or relative path to a CA for communication with Contrast UI using a self-signed certificate.'
+    desc: 'Set the absolute or relative path to a CA for communication with Contrast UI using a self-signed certificate.',
   },
   {
     name: 'api.certificate.cert_file',
     arg: '<path>',
-    desc:
-      'Set the absolute or relative path to the Certificate PEM file for communication with Contrast UI'
+    desc: 'Set the absolute or relative path to the Certificate PEM file for communication with Contrast UI',
   },
   {
     name: 'api.certificate.key_file',
     arg: '<path>',
-    desc:
-      'Set the absolute or relative path to the Key PEM file for communication with Contrast UI'
+    desc: 'Set the absolute or relative path to the Key PEM file for communication with Contrast UI',
   },
   {
     name: 'api.certificate.key_password',
     arg: '<passphrase>',
-    desc: 'If the Key file requires a password, set it here'
+    desc: 'If the Key file requires a password, set it here',
   },
   {
     name: 'api.certificate.ignore_cert_errors',
     arg: '[true]',
     default: true,
-    desc:
-      'When set to `true`, the agent ignores certificate verification errors when the agent communicates with the Contrast UI.'
+    desc: 'When set to `true`, the agent ignores certificate verification errors when the agent communicates with the Contrast UI.',
   },
   {
     name: 'api.proxy.enable',
     arg: '[true]',
     default: false,
-    desc: 'if false, no proxy is being used for communication of data'
+    desc: 'if false, no proxy is being used for communication of data',
   },
   {
     name: 'api.proxy.url',
     arg: '<url>',
-    desc: 'url of proxy for communicating agent data'
+    desc: 'url of proxy for communicating agent data',
   },
   {
     name: 'api.timeout_ms',
     arg: '<ms>',
     default: 10000,
     fn: parseNum,
-    desc: 'http timeout value (in ms)'
-  }
+    desc: 'http timeout value (in ms)',
+  },
 ];
 
 const agent = [
@@ -248,8 +245,7 @@ const agent = [
     arg: '[false]',
     fn: castBoolean,
     default: true,
-    desc:
-      'if false, create a new log file on startup instead of appending and rolling daily'
+    desc: 'if false, create a new log file on startup instead of appending and rolling daily',
   },
   {
     name: 'agent.logger.level',
@@ -257,175 +253,163 @@ const agent = [
     fn: lowercase,
     enum: ['error', 'warn', 'info', 'debug', 'trace'],
     default: 'error',
-    desc:
-      'logging level (error, warn, info, debug, trace). overrides FeatureSet:logLevel'
+    desc: 'logging level (error, warn, info, debug, trace). overrides FeatureSet:logLevel',
   },
   {
     name: 'agent.logger.path',
     default: 'node-contrast',
     fn: toAbsolutePath,
     arg: '<path>',
-    desc: 'where contrast will put its debug log'
+    desc: 'where contrast will put its debug log',
   },
   {
     name: 'agent.logger.stdout',
     arg: '[false]',
     fn: castBoolean,
     default: true,
-    desc: 'if false, suppress output to STDOUT'
+    desc: 'if false, suppress output to STDOUT',
   },
   {
     name: 'agent.node.dataflow_monitor',
     arg: '[true]',
     default: false,
     fn: castBoolean,
-    desc:
-      'throw all propagation event details into the AsyncStorage context and spit them to a log file when the request ends'
+    desc: 'throw all propagation event details into the AsyncStorage context and spit them to a log file when the request ends',
   },
   {
     name: 'agent.node.array_request_sampling.enable',
     arg: '[false]',
     default: false,
     fn: castBoolean,
-    desc: 'enable sampling of array members for dataflow'
+    desc: 'enable sampling of array members for dataflow',
   },
   {
     name: 'agent.node.array_request_sampling.threshold',
     arg: '<num>',
     default: 50,
     fn: parseNum,
-    desc:
-      'after reaching the threshold, the agent will not continue to track array members for dataflow'
+    desc: 'after reaching the threshold, the agent will not continue to track array members for dataflow',
   },
   {
     name: 'agent.node.array_request_sampling.interval',
     arg: '<num>',
     default: 5,
     fn: parseNum,
-    desc:
-      'adjust the array member sampling interval. until the threshold is reached, the agent will track 1 in every <interval> member for dataflow'
+    desc: 'adjust the array member sampling interval. until the threshold is reached, the agent will track 1 in every <interval> member for dataflow',
   },
   {
     name: 'agent.node.enable_catch_log',
     arg: '[true]',
     fn: castBoolean,
     default: false,
-    desc:
-      'enable source rewriting of try/catch blocks so that Contrast can log caught errors for debugging'
+    desc: 'enable source rewriting of try/catch blocks so that Contrast can log caught errors for debugging',
   },
   {
     name: 'agent.node.enable_property_descriptor_tracking',
     arg: '[false]',
     fn: castBoolean,
     default: false,
-    desc:
-      'enable tracking the value property returned by getOwnPropertyDescriptor'
+    desc: 'enable tracking the value property returned by getOwnPropertyDescriptor',
   },
   {
     name: 'agent.node.enable_rewrite',
     arg: '[false]',
     fn: castBoolean,
     default: true,
-    desc: 'if false, disable source rewriting (not recommended)'
+    desc: 'if false, disable source rewriting (not recommended)',
   },
   {
     name: 'agent.node.rewrite_cache.enable',
     arg: '[true]',
     fn: castBoolean,
     default: true,
-    desc:
-      'if false, disable caching source rewriting (degrades startup performance)'
+    desc: 'if false, disable caching source rewriting (degrades startup performance)',
   },
   {
     name: 'agent.node.rewrite_cache.path',
     arg: '<path>',
-    desc: 'set the location of the source rewriting cache'
+    desc: 'set the location of the source rewriting cache',
   },
   {
     name: 'agent.node.enable_source_maps',
     arg: '[false]',
     fn: castBoolean,
     default: true,
-    desc: 'enable source map support in reporting'
+    desc: 'enable source map support in reporting',
   },
   {
     name: 'agent.node.enable_native_rewrite_log',
     arg: '[true]',
     fn: castBoolean,
     default: false,
-    desc: 'log contents of native modules that have been rewritten'
+    desc: 'log contents of native modules that have been rewritten',
   },
   {
     name: 'agent.node.enable_rewrite_log',
     arg: '[true]',
     fn: castBoolean,
     default: false,
-    desc:
-      'log contents of modules that have been rewritten for debugging purposes'
+    desc: 'log contents of modules that have been rewritten for debugging purposes',
   },
   {
     name: 'agent.node.enable_sources_in_deadzones',
     arg: '[false]',
     fn: castBoolean,
     default: false,
-    desc: 'create Source Events in deadzoned code'
+    desc: 'create Source Events in deadzoned code',
   },
   {
     name: 'agent.node.library_usage.read_extraneous_libraries',
     arg: '[true]',
     fn: castBoolean,
     default: false,
-    desc: 'if true, read libraries marked by npm as extraneous'
+    desc: 'if true, read libraries marked by npm as extraneous',
   },
   {
     name: 'agent.node.library_usage.reporting.interval',
     arg: '<num>',
     fn: parseNum,
     default: 1,
-    desc:
-      'frequency of collecting code events for library usage in milliseconds, defaults to 1 ms'
+    desc: 'frequency of collecting code events for library usage in milliseconds, defaults to 1 ms',
   },
   {
     name: 'agent.node.library_usage.reporting.enable',
     arg: '[false]',
     default: true,
     fn: castBoolean,
-    desc:
-      'add enhanced library usage features (i.e. scanning for composition of dependencies, reporting usage)'
+    desc: 'add enhanced library usage features (i.e. scanning for composition of dependencies, reporting usage)',
   },
   {
     name: 'agent.node.app_root',
     arg: '<path>',
-    desc: "set location to look for the app's package.json"
+    desc: "set location to look for the app's package.json",
   },
   {
     name: 'agent.node.req_perf_logging',
     arg: '[false]',
     default: false,
-    desc:
-      'enable debug logging of instrumentation overhead on a per request basis'
+    desc: 'enable debug logging of instrumentation overhead on a per request basis',
   },
   {
     name: 'agent.node.skip_nested_tracking',
     arg: '[true]',
     fn: castBoolean,
     default: false,
-    desc:
-      'do not traverse nested properties to look for tracking during propagation (not recommended)'
+    desc: 'do not traverse nested properties to look for tracking during propagation (not recommended)',
   },
   {
     name: 'agent.node.speedracer_input_analysis',
     arg: '[false]',
     default: true,
     fn: castBoolean,
-    desc: 'whether to use speedracer for input analysis when enabled'
+    desc: 'whether to use speedracer for input analysis when enabled',
   },
   {
     name: 'agent.node.native_input_analysis',
     arg: '[true]',
     default: false,
     fn: castBoolean,
-    desc: 'do agent-native input analysis prior to any external analysis'
+    desc: 'do agent-native input analysis prior to any external analysis',
   },
   {
     name: 'agent.node.unsafe.deadzones',
@@ -435,91 +419,85 @@ const agent = [
         return str.split(',').map((s) => s.trim());
       }
     },
-    desc:
-      'Add a comma-separated list of values to the deadzone regardless of its presence in the policy file.',
-    default: ''
+    desc: 'Add a comma-separated list of values to the deadzone regardless of its presence in the policy file.',
+    default: '',
   },
   {
     name: 'agent.node.unsafe.sr_max_body_size',
     arg: '<size in mb>',
     default: 16,
-    desc:
-      'set the maximum body size that will be sent to speedracer for input analysis'
+    desc: 'set the maximum body size that will be sent to speedracer for input analysis',
   },
   {
     name: 'agent.heap_dump.enable',
     arg: '[true]',
     fn: castBoolean,
     default: false,
-    desc: 'Every n seconds create a heap snapshot of the current process'
+    desc: 'Every n seconds create a heap snapshot of the current process',
   },
   {
     name: 'agent.heap_dump.path',
     arg: '<path>',
     default: 'contrast_heap_dumps',
-    desc:
-      "Set the location to which to save the heap dump files. If relative, the path is determined based on the process' working directory."
+    desc: "Set the location to which to save the heap dump files. If relative, the path is determined based on the process' working directory.",
   },
   {
     name: 'agent.heap_dump.delay_ms',
     arg: '<time>',
     default: 10000,
-    desc:
-      'Set the amount of time to wait, in milliseconds, after agent startup to begin taking heap dumps.'
+    desc: 'Set the amount of time to wait, in milliseconds, after agent startup to begin taking heap dumps.',
   },
   {
     name: 'agent.heap_dump.window_ms',
     arg: '<number>',
     default: 30000,
-    desc: 'How frequently the heap snapshot is created'
+    desc: 'How frequently the heap snapshot is created',
   },
   {
     name: 'agent.heap_dump.count',
     arg: '<number>',
     default: 5,
-    desc: 'Set the number of heap dumps to take before disabling this feature.'
+    desc: 'Set the number of heap dumps to take before disabling this feature.',
   },
   {
     name: 'agent.stack_trace_limit',
     arg: '<limit>',
     default: 10,
     fn: parseNum,
-    desc:
-      'set limit for stack trace size (larger limits will improve accuracy but increase memory usage)'
+    desc: 'set limit for stack trace size (larger limits will improve accuracy but increase memory usage)',
   },
   {
     name: 'agent.trust_custom_validators',
     arg: '<trust-custom-validators>',
     default: false,
-    desc: `trust incoming strings when they pass custom validators (Mongoose, Joi)`
+    desc: `trust incoming strings when they pass custom validators (Mongoose, Joi)`,
   },
   {
     name: 'agent.traverse_and_track',
     arg: '<traverse-and-track>',
     default: false,
-    desc: 'source membrane alternative'
+    desc: 'source membrane alternative',
   },
   {
     name: 'agent.polling.app_activity_ms',
     arg: '<ms>',
     default: 30000,
     fn: parseNum,
-    desc: 'how often (in ms), application activity messages are sent'
+    desc: 'how often (in ms), application activity messages are sent',
   },
   {
     name: 'agent.polling.app_update_ms',
     arg: '<ms>',
     default: 60000,
     fn: parseNum,
-    desc:
-      'how often (in ms), application update messages (libraries, technologies, etc.) are sent'
+    desc: 'how often (in ms), application update messages (libraries, technologies, etc.) are sent',
   },
   {
     name: 'agent.route_coverage.enable',
     arg: '[true]',
     default: true,
     fn: castBoolean,
-    desc: 'if false, do not send coverage data to the Contrast UI'
+    desc: 'if false, do not send coverage data to the Contrast UI',
   },
   {
     name: 'agent.security_logger.level',
@@ -529,133 +507,100 @@ const agent = [
     // set emergency for whatever reason
     enum: ['alert', 'crit', 'err', 'warning', 'notice', 'info', 'debug'],
     default: 'debug',
-    desc:
-      'security logging level (alert, crit, err, warning, notice, info, debug)'
+    desc: 'security logging level (alert, crit, err, warning, notice, info, debug)',
   },
   {
     name: 'agent.security_logger.path',
     default: 'security',
     fn: toAbsolutePath,
     arg: '<path>',
-    desc: 'where to log security events'
+    desc: 'where to log security events',
   },
   {
     name: 'agent.security_logger.syslog.enable',
     fn: castBoolean,
-    desc: 'Set to true to enable Syslog logging'
+    desc: 'Set to true to enable Syslog logging',
   },
   {
     name: 'agent.security_logger.syslog.ip',
-    desc:
-      'Set the IP address of the Syslog server to which the agent should send messages',
-    arg: '<ip>'
+    desc: 'Set the IP address of the Syslog server to which the agent should send messages',
+    arg: '<ip>',
   },
   {
     name: 'agent.security_logger.syslog.port',
-    desc:
-      'Set the port of the Syslog server to which the agent should send messages',
-    arg: '<port>'
+    desc: 'Set the port of the Syslog server to which the agent should send messages',
+    arg: '<port>',
   },
   {
     name: 'agent.security_logger.syslog.facility',
     desc: 'Set the facility code of the messages the agent sends to Syslog',
     enum: [
-      0,
-      1,
-      2,
-      3,
-      4,
-      5,
-      6,
-      7,
-      8,
-      9,
-      10,
-      11,
-      12,
-      13,
-      14,
-      15,
-      16,
-      17,
-      18,
-      19,
-      20,
-      21,
-      22,
-      23
+      0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20,
+      21, 22, 23,
     ],
-    arg: '<facility>'
+    arg: '<facility>',
   },
   {
     name: 'agent.security_logger.syslog.severity_exploited',
-    desc:
-      'Set the log level of Exploited attacks. Value options are ALERT/CRIT/ERROR/WARNING/NOTICE/INFO/DEBUG',
+    desc: 'Set the log level of Exploited attacks. Value options are ALERT/CRIT/ERROR/WARNING/NOTICE/INFO/DEBUG',
     enum: ['alert', 'crit', 'err', 'warning', 'notice', 'info', 'debug'],
     arg: '<level>',
-    fn: lowercase
+    fn: lowercase,
   },
   {
     name: 'agent.security_logger.syslog.severity_blocked',
-    desc:
-      'Set the log level of Exploited attacks. Value options are ALERT/CRIT/ERROR/WARNING/NOTICE/INFO/DEBUG',
+    desc: 'Set the log level of Exploited attacks. Value options are ALERT/CRIT/ERROR/WARNING/NOTICE/INFO/DEBUG',
     enum: ['alert', 'crit', 'err', 'warning', 'notice', 'info', 'debug'],
     arg: '<level>',
-    fn: lowercase
+    fn: lowercase,
   },
   {
     name: 'agent.security_logger.syslog.severity_probed',
-    desc:
-      'Set the log level of Probed attacks. Value options are ALERT/CRIT/ERROR/WARNING/NOTICE/INFO/DEBUG',
+    desc: 'Set the log level of Probed attacks. Value options are ALERT/CRIT/ERROR/WARNING/NOTICE/INFO/DEBUG',
     enum: ['alert', 'crit', 'err', 'warning', 'notice', 'info', 'debug'],
     arg: '<level>',
-    fn: lowercase
+    fn: lowercase,
   },
   {
     name: 'agent.logger.log_outbound_http',
     arg: '[true]',
     default: false,
-    desc: 'log all outbound http responses'
+    desc: 'log all outbound http responses',
   },
   {
     name: 'agent.logger.backups',
     arg: '<number>',
-    desc:
-      "set the max number of log files or days to keep files. For days, add 'd' after number"
+    desc: "set the max number of log files or days to keep files. For days, add 'd' after number",
   },
   {
     name: 'agent.logger.roll_size',
     arg: '<size>',
-    desc:
-      "set the maximum file size in bytes for logs. Add 'k', 'm', 'g' after number for Kb, Mb, Gb"
+    desc: "set the maximum file size in bytes for logs. Add 'k', 'm', 'g' after number for Kb, Mb, Gb",
   },
   {
     name: 'agent.service.socket',
     arg: '<socket>',
-    desc:
-      'If this property is defined, the service is listening on a Unix socket at the defined path'
+    desc: 'If this property is defined, the service is listening on a Unix socket at the defined path',
   },
   {
     name: 'agent.service.host',
     arg: '<host>',
     default: '127.0.0.1',
-    desc:
-      'set the the hostname or IP address of the Contrast service to which the Contrast agent should report'
+    desc: 'set the the hostname or IP address of the Contrast service to which the Contrast agent should report',
   },
   {
     name: 'agent.service.port',
     arg: '<port>',
     default: 30555,
-    desc:
-      'set the the port of the Contrast service to which the Contrast agent should report'
+    desc: 'set the the port of the Contrast service to which the Contrast agent should report',
   },
   {
     name: 'agent.service.grpc',
     arg: '[true]',
     default: false,
     fn: castBoolean,
-    desc: 'set to `true` to enable communication to speedracer via GRPC.'
-  }
+    desc: 'set to `true` to enable communication to speedracer via GRPC.',
+  },
 ];
 
 const application = [
@@ -674,69 +619,63 @@ const application = [
 
       return undefined;
     },
-    desc:
-      'string containing args to pass verbatim to the application, eg --application.args "-A -S -D -F foo bar"'
+    desc: 'string containing args to pass verbatim to the application, eg --application.args "-A -S -D -F foo bar"',
   },
   {
     name: 'application.code',
     arg: '<code>',
-    desc:
-      'add the application code this application should use in the Contrast UI'
+    desc: 'add the application code this application should use in the Contrast UI',
   },
   {
     name: 'application.group',
     arg: '<tags>',
-    desc: "how to report the application's group for auto-grouping"
+    desc: "how to report the application's group for auto-grouping",
   },
   {
     name: 'application.language',
     arg: '<language>',
     default: 'Node',
-    desc:
-      "override the reported application language (if different from 'Node')"
+    desc: "override the reported application language (if different from 'Node')",
   },
   {
     name: 'application.name',
     arg: '<name>',
     env: 'CONTRASTSECURITY_APP_NAME',
-    desc: 'override the reported application name. (default: package.json:name)'
+    desc: 'override the reported application name. (default: package.json:name)',
   },
   {
     name: 'application.path',
     arg: '<path>',
     default: '/',
-    desc: 'override the reported application path'
+    desc: 'override the reported application path',
   },
   {
     name: 'application.tags',
     arg: '<tags>',
-    desc:
-      'comma-separated list of tags to apply to each application reported by the agent'
+    desc: 'comma-separated list of tags to apply to each application reported by the agent',
   },
   {
     name: 'application.metadata',
     args: '<metadata>',
-    desc:
-      'comma-separated list of key=value pairs that are applied to each application reported by the agent.'
+    desc: 'comma-separated list of key=value pairs that are applied to each application reported by the agent.',
   },
   {
     name: 'application.version',
     arg: '<version>',
-    desc:
-      "override the reported application version (if different from 'version' field in the application's package.json)"
+    desc: "override the reported application version (if different from 'version' field in the application's package.json)",
   },
   {
     name: 'application.session_id',
     arg: '<session_id>',
     default: null,
-    desc: 'provide the ID of a session existing within Contrast UI'
+    desc: 'provide the ID of a session existing within Contrast UI',
   },
   {
     name: 'application.session_metadata',
     arg: '<session_metadata>',
     default: null,
-    desc: 'provide metadata used to create a new session within Contrast UI'
-  }
+    desc: 'provide metadata used to create a new session within Contrast UI',
+  },
 ];
 
 const inventory = [
@@ -746,22 +685,20 @@ const inventory = [
     arg: '[false]',
     fn: castBoolean,
     default: true,
-    desc:
-      'Set to false to disable reading or reporting libraries used by the application'
+    desc: 'Set to false to disable reading or reporting libraries used by the application',
   },
   {
     name: 'inventory.tags',
     arg: '<tags>',
-    desc:
-      'comma-separated list of tags to apply to each application library reported by the agent'
+    desc: 'comma-separated list of tags to apply to each application library reported by the agent',
   },
   {
     name: 'inventory.enable',
     arg: '[false]',
     fn: castBoolean,
     default: true,
-    desc: 'Set to `false` to disable inventory features in the agent.'
-  }
+    desc: 'Set to `false` to disable inventory features in the agent.',
+  },
 ];
 
 const assess = [
@@ -769,43 +706,41 @@ const assess = [
     name: 'assess.enable',
     arg: '[false]',
     fn: castBoolean,
-    desc:
-      'if false, disable assess for this agent. A restart is required to re-enable'
+    desc: 'if false, disable assess for this agent. A restart is required to re-enable',
   },
   {
     name: 'assess.enable_preflight',
     arg: '[false]',
     fn: castBoolean,
     default: true,
-    desc: 'if false, disable preflight spooling of traces (not recommended)'
+    desc: 'if false, disable preflight spooling of traces (not recommended)',
   },
   {
     name: 'assess.enable_propagators',
     arg: '[false]',
     fn: castBoolean,
     default: true,
-    desc: 'if false, disable dataflow propagation (not recommended)'
+    desc: 'if false, disable dataflow propagation (not recommended)',
   },
   {
     name: 'assess.sampling.enable',
     arg: '[false]',
     fn: castBoolean,
     default: true,
-    desc: 'if false, disable sampling'
+    desc: 'if false, disable sampling',
   },
   {
     name: 'assess.sampling.baseline',
     arg: '<rule limit>',
     fn: parseNum,
     default: 5,
-    desc: 'max number of times to report the same rule for a single'
+    desc: 'max number of times to report the same rule for a single',
   },
   {
     name: 'assess.tags',
     arg: '<tags>',
-    desc:
-      'comma-separated list of tags to apply to each application vulnerability reported by the agent'
-  }
+    desc: 'comma-separated list of tags to apply to each application vulnerability reported by the agent',
+  },
 ];
 
 const protect = [
@@ -813,112 +748,104 @@ const protect = [
     name: 'protect.enable',
     arg: '[false]',
     fn: castBoolean,
-    desc: 'if false, disable protect for this agent'
+    desc: 'if false, disable protect for this agent',
   },
   {
     name: 'protect.enable_rep',
     arg: '[false]',
     default: true,
     fn: castBoolean,
-    desc: 'if false, disable Runtime Exploit Prevention(REP) for this agent'
+    desc: 'if false, disable Runtime Exploit Prevention(REP) for this agent',
   },
   {
     name: 'protect.auth.mode',
     arg: '<mode>',
     default: 'OFF',
     enum: ['OFF', 'MONITOR'],
-    desc:
-      'whether to report authentication framework login attempts (OFF, MONITOR)'
+    desc: 'whether to report authentication framework login attempts (OFF, MONITOR)',
   },
   {
     name: 'protect.samples.blocked',
     arg: '<count>',
     fn: parseNum,
     default: 25,
-    desc:
-      'limit the reporting of "blocked" protect events to this number (per report cycle)'
+    desc: 'limit the reporting of "blocked" protect events to this number (per report cycle)',
   },
   {
     name: 'protect.samples.blocked_at_perimeter',
     arg: '<count>',
     fn: parseNum,
     default: 25,
-    desc:
-      'limit the reporting of "blocked-at-perim" protect events to this number (per report cycle)'
+    desc: 'limit the reporting of "blocked-at-perim" protect events to this number (per report cycle)',
   },
   {
     name: 'protect.samples.exploited',
     arg: '<count>',
     fn: parseNum,
     default: 100,
-    desc:
-      'limit the reporting of "effective" protect events to this number (per report cycle)'
+    desc: 'limit the reporting of "effective" protect events to this number (per report cycle)',
   },
   {
     name: 'protect.samples.probed',
     arg: '<count>',
     fn: parseNum,
     default: 50,
-    desc:
-      'limit the reporting of "ineffective" protect events to this number (per report cycle)'
+    desc: 'limit the reporting of "ineffective" protect events to this number (per report cycle)',
   },
   {
     name: 'protect.rules.disabled_rules',
     arg: '<rules>',
     default: '',
-    desc: 'comma-delimited list of rule IDs to disable'
+    desc: 'comma-delimited list of rule IDs to disable',
   },
   ...Object.values(require('../../constants').RULES).map((ruleId) => ({
     name: `protect.rules.${ruleId}.mode`,
     arg: '<mode>',
     enum: ['monitor', 'block', 'block_at_perimeter', 'off'],
-    desc: `the mode in which to run the ${ruleId} rule`
-  }))
+    desc: `the mode in which to run the ${ruleId} rule`,
+  })),
 ];
 
 const server = [
   {
     name: 'server.build',
     arg: '<version>',
-    desc: 'set reported server build option'
+    desc: 'set reported server build option',
   },
   {
     name: 'server.environment',
     arg: '<name>',
     fn: uppercase,
     // enum: ['QA', 'PRODUCTION', 'DEVELOPMENT'], none of the other agents validate this
-    desc:
-      'environment the server is running in (QA, PRODUCTION, or DEVELOPMENT)'
+    desc: 'environment the server is running in (QA, PRODUCTION, or DEVELOPMENT)',
   },
   {
     name: 'server.name',
     arg: '<name>',
     default: os.hostname(),
-    desc: 'override the reported server name'
+    desc: 'override the reported server name',
   },
   {
     name: 'server.path',
     arg: '<name>',
-    desc: 'override the reported server path'
+    desc: 'override the reported server path',
   },
   {
     name: 'server.tags',
     arg: '<tags>',
-    desc:
-      'comma-separated list of tags to apply to each server reported by the agent'
+    desc: 'comma-separated list of tags to apply to each server reported by the agent',
   },
   {
     name: 'server.type',
     arg: '<type>',
     default: util.format('node.js %s', process.version),
-    desc: 'override the reported server type'
+    desc: 'override the reported server type',
   },
   {
     name: 'server.version',
     arg: '<version>',
-    desc:
-      "override the reported server version (if different from 'version' field in the application's package.json)"
-  }
+    desc: "override the reported server version (if different from 'version' field in the application's package.json)",
+  },
 ];
 
 let dev = [];
@@ -928,68 +855,52 @@ if (process.env.CONTRAST_DEV) {
       name: 'agent.node.dev.global_agent',
       arg: '[true]',
       default: true,
-      desc:
-        'add global.contrast_agent so that apps can reference agent and its emitters'
+      desc: 'add global.contrast_agent so that apps can reference agent and its emitters',
     },
     {
       name: 'agent.node.dev.global_tracker',
       arg: '[true]',
       default: true,
-      desc:
-        'add global.contrast_tracker so that apps can reference dataflow metadata'
+      desc: 'add global.contrast_tracker so that apps can reference dataflow metadata',
     },
     {
       name: 'agent.node.dev.metrics',
       arg: '[false]',
       default: false,
-      desc:
-        'add tracking and periodically outputting diagnostic metrics to a file'
+      desc: 'add tracking and periodically outputting diagnostic metrics to a file',
     },
     {
       name: 'agent.node.dev.screener.enable',
       arg: '[false]',
       default: false,
-      desc: 'use message queue for (most) reporting rather than send to SR'
+      desc: 'use message queue for (most) reporting rather than send to SR',
     },
     {
       name: 'agent.node.dev.screener.mq_location',
       arg: '<location>',
       default: 'amqp://localhost',
-      desc:
-        'location of the messaging queue server to which to publish messages'
+      desc: 'location of the messaging queue server to which to publish messages',
     },
     {
       name: 'agent.node.dev.screener.exchange_name',
       arg: '<name>',
-      desc:
-        'name of the exchange to which to publish messages (unique per test run)'
+      desc: 'name of the exchange to which to publish messages (unique per test run)',
     },
     {
       name: 'agent.node.dev.custom_policy_path',
       arg: '<path>',
       fn: toAbsolutePath,
-      desc: 'provide an additional, custom policy'
+      desc: 'provide an additional, custom policy',
     },
     {
       name: 'agent.node.unsafe.generate_umbrella',
       arg: '[true]',
       fn: castBoolean,
       default: false,
-      desc:
-        'Generate Umbrella test-cases based on what was found by the agent into umbrella_test.json'
-    }
+      desc: 'Generate Umbrella test-cases based on what was found by the agent into umbrella_test.json',
+    },
   ];
 }
-const sails = [
-  {
-    name: 'pathToSails',
-    arg: '<path>'
-  },
-  {
-    name: 'gdsrc',
-    arg: '<path>'
-  }
-];
 
 const options = [].concat(
   misc,
@@ -1000,7 +911,7 @@ const options = [].concat(
   inventory,
   protect,
   server,
-  dev
+  dev,
 );
 
 // run
@@ -1031,19 +942,6 @@ options.forEach((option) => {
   program.option(name, option.desc);
 });
 
-// In NODE-2059 it was discovered that a module was appending config options that the
-// agent didn't recognize and was causing the application to not load properly.
-// The agent doesn't need to do anything with these options. It just needs to not
-// throw an error when it encounters them but we also don't need them displayed on
-// the agent's config option list. The newest version of Commander lets us do exactly this.
-// This is structured so that if anything like this is discovered again, they can be
-// added in easily.
-const hiddenOptions = [].concat(sails);
-
-hiddenOptions.forEach((option) => {
-  program.addOption(new Option(`--${option.name} ${option.arg}`).hideHelp());
-});
-
 function getDefault(optionName) {
   let option;
   options.forEach((entry) => {