PR review

Author: timfish

packages/node/src/sdk/client.ts

@@ -21,9 +21,10 @@ export class NodeClient extends ServerRuntimeClient<NodeClientOptions> {
   private _logOnExitFlushListener: (() => void) | undefined;
 
   public constructor(options: NodeClientOptions) {
-    const serverName = options.skipIncludingServerName
-      ? undefined
-      : options.serverName || global.process.env.SENTRY_NAME || os.hostname();
+    const serverName =
+      options.includeServerName === false
+        ? undefined
+        : options.serverName || global.process.env.SENTRY_NAME || os.hostname();
 
     const clientOptions: ServerRuntimeClientOptions = {
       ...options,

packages/node/src/types.ts

@@ -62,12 +62,14 @@ export interface BaseNodeOptions {
   profileLifecycle?: 'manual' | 'trace';
 
   /**
-   * If set to true, the SDK will not automatically detect the `serverName`.
+   * If set to `false`, the SDK will not automatically detect the `serverName`.
    *
    * This is useful if you are using the SDK in a CLI app or Electron where the
    * hostname might be considered PII.
+   *
+   * @default true
    */
-  skipIncludingServerName?: boolean;
+  includeServerName?: boolean;
 
   /** Sets an optional server name (device name) */
   serverName?: string;