From a5bf4f6f4f37c53509f78c22a4851bddd9aabbc0 Mon Sep 17 00:00:00 2001
From: Zachary Ebenfeld <zackebenfeld@gmail.com>
Date: Fri, 18 Apr 2025 18:30:16 -0400
Subject: [PATCH 1/2] added jsdoc defaults from
 https://docs.sentry.io/platforms/javascript/guides/react/configuration/options/

---
 packages/core/src/types-hoist/options.ts | 38 +++++++++++++++++-------
 1 file changed, 28 insertions(+), 10 deletions(-)

diff --git a/packages/core/src/types-hoist/options.ts b/packages/core/src/types-hoist/options.ts
index 51ccc7cdab79..0401cf0f11e4 100644
--- a/packages/core/src/types-hoist/options.ts
+++ b/packages/core/src/types-hoist/options.ts
@@ -13,20 +13,25 @@ import type { BaseTransportOptions, Transport } from './transport';
 export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOptions> {
   /**
    * Enable debug functionality in the SDK itself
+   * @default false
    */
   debug?: boolean;
 
   /**
    * Specifies whether this SDK should send events to Sentry.
-   * Defaults to true.
+   * @default true
    */
   enabled?: boolean;
 
-  /** Attaches stacktraces to pure capture message / log integrations */
+  /**
+   * Attaches stacktraces to pure capture message / log integrations
+   * @default false
+   */
   attachStacktrace?: boolean;
 
   /**
    * Send SDK Client Reports. When calling `Sentry.init()`, this option defaults to `true`.
+   * @default true
    */
   sendClientReports?: boolean;
 
@@ -43,7 +48,10 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
    */
   release?: string;
 
-  /** The current environment of your application (e.g. "production"). */
+  /**
+   * The current environment of your application (e.g. "production").
+   * @default "production"
+   */
   environment?: string;
 
   /** Sets the distribution for all events */
@@ -51,6 +59,7 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
 
   /**
    * List of integrations that should be installed after SDK was initialized.
+   * @default []
    */
   integrations: Integration[];
 
@@ -101,8 +110,9 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
   initialScope?: CaptureContext;
 
   /**
-   * The maximum number of breadcrumbs sent with events. Defaults to 100.
+   * The maximum number of breadcrumbs sent with events.
    * Sentry has a maximum payload size of 1MB and any events exceeding that payload size will be dropped.
+   * @default 100
    */
   maxBreadcrumbs?: number;
 
@@ -111,10 +121,14 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
    *
    * 0.0 = 0% chance of a given event being sent (send no events) 1.0 = 100% chance of a given event being sent (send
    * all events)
+   * @default 1.0
    */
   sampleRate?: number;
 
-  /** Maximum number of chars a single value can have before it will be truncated. */
+  /**
+   * Maximum number of chars a single value can have before it will be truncated.
+   * @default 250
+   */
   maxValueLength?: number;
 
   /**
@@ -124,7 +138,7 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
    * - `user`
    * - `contexts`
    * - `extra`
-   * Defaults to `3`. Set to `0` to disable.
+   * @default 3
    */
   normalizeDepth?: number;
 
@@ -135,19 +149,21 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
    * - `user`
    * - `contexts`
    * - `extra`
-   * Defaults to `1000`
+   * @default 1000
    */
   normalizeMaxBreadth?: number;
 
   /**
    * A pattern for error messages which should not be sent to Sentry.
    * By default, all errors will be sent.
+   * @default []
    */
   ignoreErrors?: Array<string | RegExp>;
 
   /**
    * A pattern for transaction names which should not be sent to Sentry.
    * By default, all transactions will be sent.
+   * @default []
    */
   ignoreTransactions?: Array<string | RegExp>;
 
@@ -162,8 +178,7 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
    * Controls if potentially sensitive data should be sent to Sentry by default.
    * Note that this only applies to data that the SDK is sending by default
    * but not data that was explicitly set (e.g. by calling `Sentry.setUser()`).
-   *
-   * Defaults to `false`.
+   * @default false
    *
    * NOTE: This option currently controls only a few data points in a selected
    * set of SDKs. The goal for this option is to eventually control all sensitive
@@ -186,7 +201,8 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
     [key: string]: any;
     /**
-     * If logs support should be enabled. Defaults to false.
+     * If logs support should be enabled.
+     * @default false
      */
     enableLogs?: boolean;
     /**
@@ -208,6 +224,7 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
    * By default, all errors will be sent.
    *
    * Requires the use of the `InboundFilters` integration.
+   * @default []
    */
   allowUrls?: Array<string | RegExp>;
 
@@ -217,6 +234,7 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
    * By default, all errors will be sent.
    *
    * Requires the use of the `InboundFilters` integration.
+   * @default []
    */
   denyUrls?: Array<string | RegExp>;
 

From 55f574ac4182bb5712fceb27783dc5f4ff7367aa Mon Sep 17 00:00:00 2001
From: Abhijeet Prasad <aprasad@sentry.io>
Date: Tue, 22 Apr 2025 10:07:11 -0400
Subject: [PATCH 2/2] update more docstrings and add spacing

---
 packages/core/src/types-hoist/options.ts | 89 +++++++++++++++++++-----
 1 file changed, 71 insertions(+), 18 deletions(-)

diff --git a/packages/core/src/types-hoist/options.ts b/packages/core/src/types-hoist/options.ts
index 0401cf0f11e4..93f9be36ed25 100644
--- a/packages/core/src/types-hoist/options.ts
+++ b/packages/core/src/types-hoist/options.ts
@@ -12,53 +12,78 @@ import type { BaseTransportOptions, Transport } from './transport';
 
 export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOptions> {
   /**
-   * Enable debug functionality in the SDK itself
+   * Enable debug functionality in the SDK itself. If `debug` is set to `true` the SDK will attempt
+   * to print out useful debugging information about what the SDK is doing.
+   *
    * @default false
    */
   debug?: boolean;
 
   /**
-   * Specifies whether this SDK should send events to Sentry.
+   * Specifies whether this SDK should send events to Sentry. Setting this to `enabled: false`
+   * doesn't prevent all overhead from Sentry instrumentation. To disable Sentry completely,
+   * depending on environment, call `Sentry.init conditionally.
+   *
    * @default true
    */
   enabled?: boolean;
 
   /**
-   * Attaches stacktraces to pure capture message / log integrations
+   * When enabled, stack traces are automatically attached to all events captured with `Sentry.captureMessage`.
+   *
+   * Grouping in Sentry is different for events with stack traces and without. As a result, you will get
+   * new groups as you enable or disable this flag for certain events.
+   *
    * @default false
    */
   attachStacktrace?: boolean;
 
   /**
-   * Send SDK Client Reports. When calling `Sentry.init()`, this option defaults to `true`.
+   * Send SDK Client Reports, which are used to emit outcomes about events that the SDK dropped
+   * or failed to capture.
+   *
    * @default true
    */
   sendClientReports?: boolean;
 
   /**
-   * The Dsn used to connect to Sentry and identify the project. If omitted, the
-   * SDK will not send any data to Sentry.
+   * The DSN tells the SDK where to send the events. If this is not set, the SDK will not send any events to Sentry.
+   *
+   * @default undefined
    */
   dsn?: string;
 
   /**
-   * The release identifier used when uploading respective source maps. Specify
-   * this value to allow Sentry to resolve the correct source maps when
-   * processing events.
+   * Sets the release. Release names are strings, but some formats are detected by Sentry and might be
+   * rendered differently. Learn more about how to send release data so Sentry can tell you about
+   * regressions between releases and identify the potential source in the
+   * [releases documentation](https://docs.sentry.io/product/releases/)
+   *
+   * @default undefined
    */
   release?: string;
 
   /**
    * The current environment of your application (e.g. "production").
+   *
+   * Environments are case-sensitive. The environment name can't contain newlines, spaces or forward slashes,
+   * can't be the string "None", or exceed 64 characters. You can't delete environments, but you can hide them.
+   *
    * @default "production"
    */
   environment?: string;
 
-  /** Sets the distribution for all events */
+  /**
+   * Sets the distribution of the application. Distributions are used to disambiguate build or
+   * deployment variants of the same release of an application.
+   *
+   * @default undefined
+   */
   dist?: string;
 
   /**
    * List of integrations that should be installed after SDK was initialized.
+   *
    * @default []
    */
   integrations: Integration[];
@@ -70,8 +95,7 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
   transport: (transportOptions: TO) => Transport;
 
   /**
-   * A stack parser implementation
-   * By default, a stack parser is supplied for all supported platforms
+   * A stack parser implementation. By default, a stack parser is supplied for all supported platforms.
    */
   stackParser: StackParser;
 
@@ -84,10 +108,12 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
    * Sample rate to determine trace sampling.
    *
    * 0.0 = 0% chance of a given trace being sent (send no traces) 1.0 = 100% chance of a given trace being sent (send
-   * all traces)
+   * all traces).
    *
    * Tracing is enabled if either this or `tracesSampler` is defined. If both are defined, `tracesSampleRate` is
-   * ignored.
+   * ignored. Set this and `tracesSampler` to `undefined` to disable tracing.
+   *
+   * @default undefined
    */
   tracesSampleRate?: number;
 
@@ -106,27 +132,32 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
 
   /**
    * Initial data to populate scope.
+   *
+   * @default undefined
    */
   initialScope?: CaptureContext;
 
   /**
    * The maximum number of breadcrumbs sent with events.
    * Sentry has a maximum payload size of 1MB and any events exceeding that payload size will be dropped.
+   *
    * @default 100
    */
   maxBreadcrumbs?: number;
 
   /**
-   * A global sample rate to apply to all events.
+   * A global sample rate to apply to all error events.
    *
    * 0.0 = 0% chance of a given event being sent (send no events) 1.0 = 100% chance of a given event being sent (send
    * all events)
+   *
    * @default 1.0
    */
   sampleRate?: number;
 
   /**
    * Maximum number of chars a single value can have before it will be truncated.
+   *
    * @default 250
    */
   maxValueLength?: number;
@@ -138,6 +169,7 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
    * - `user`
    * - `contexts`
    * - `extra`
+   *
    * @default 3
    */
   normalizeDepth?: number;
@@ -149,6 +181,7 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
    * - `user`
    * - `contexts`
    * - `extra`
+   *
    * @default 1000
    */
   normalizeMaxBreadth?: number;
@@ -156,6 +189,10 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
   /**
    * A pattern for error messages which should not be sent to Sentry.
    * By default, all errors will be sent.
+   *
+   * Behavior of the `ignoreErrors` option is controlled by the `Sentry.eventFiltersIntegration` integration. If the
+   * event filters integration is not installed, the `ignoreErrors` option will not have any effect.
+   *
    * @default []
    */
   ignoreErrors?: Array<string | RegExp>;
@@ -163,6 +200,10 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
   /**
    * A pattern for transaction names which should not be sent to Sentry.
    * By default, all transactions will be sent.
+   *
+   * Behavior of the `ignoreTransactions` option is controlled by the `Sentry.eventFiltersIntegration` integration.
+   * If the event filters integration is not installed, the `ignoreTransactions` option will not have any effect.
+   *
    * @default []
    */
   ignoreTransactions?: Array<string | RegExp>;
@@ -171,6 +212,8 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
    * A URL to an envelope tunnel endpoint. An envelope tunnel is an HTTP endpoint
    * that accepts Sentry envelopes for forwarding. This can be used to force data
    * through a custom server independent of the type of data.
+   *
+   * @default undefined
    */
   tunnel?: string;
 
@@ -178,6 +221,7 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
    * Controls if potentially sensitive data should be sent to Sentry by default.
    * Note that this only applies to data that the SDK is sending by default
    * but not data that was explicitly set (e.g. by calling `Sentry.setUser()`).
+   *
    * @default false
    *
    * NOTE: This option currently controls only a few data points in a selected
@@ -191,6 +235,8 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
   /**
    * Set of metadata about the SDK that can be internally used to enhance envelopes and events,
    * and provide additional data about every request.
+   *
+   * @internal This option is not part of the public API and is subject to change at any time.
    */
   _metadata?: SdkMetadata;
 
@@ -202,6 +248,7 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
     [key: string]: any;
     /**
      * If logs support should be enabled.
+     *
      * @default false
      */
     enableLogs?: boolean;
@@ -212,6 +259,8 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
      * Note that you must return a valid log from this callback. If you do not wish to modify the log, simply return
      * it at the end. Returning `null` will cause the log to be dropped.
      *
+     * @default undefined
+     *
      * @param log The log generated by the SDK.
      * @returns A new log that will be sent | null.
      */
@@ -223,7 +272,9 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
    * This is the opposite of {@link Options.denyUrls}.
    * By default, all errors will be sent.
    *
-   * Requires the use of the `InboundFilters` integration.
+   * Behavior of the `allowUrls` option is controlled by the `Sentry.eventFiltersIntegration` integration.
+   * If the event filters integration is not installed, the `allowUrls` option will not have any effect.
+   *
    * @default []
    */
   allowUrls?: Array<string | RegExp>;
@@ -233,7 +284,9 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
    * To allow certain errors instead, use {@link Options.allowUrls}.
    * By default, all errors will be sent.
    *
-   * Requires the use of the `InboundFilters` integration.
+   * Behavior of the `denyUrls` option is controlled by the `Sentry.eventFiltersIntegration` integration.
+   * If the event filters integration is not installed, the `denyUrls` option will not have any effect.
+   *
    * @default []
    */
   denyUrls?: Array<string | RegExp>;
@@ -271,7 +324,7 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
    * Function to compute tracing sample rate dynamically and filter unwanted traces.
    *
    * Tracing is enabled if either this or `tracesSampleRate` is defined. If both are defined, `tracesSampleRate` is
-   * ignored.
+   * ignored. Set this and `tracesSampleRate` to `undefined` to disable tracing.
    *
    * Will automatically be passed a context object of default and optional custom data.
    *