PR feedback

Author: mydea

packages/node/src/sdk/client.ts

@@ -156,7 +156,7 @@ export class NodeClient extends ServerRuntimeClient<NodeClientOptions> {
 }
 
 function applyDefaultOptions<T extends Partial<NodeClientOptions>>(options: T): T {
-  const release = getRelease(options.release);
+  const release = options.release ?? getSentryRelease();
   const spotlight =
     options.spotlight ?? envToBool(process.env.SENTRY_SPOTLIGHT, { strict: true }) ?? process.env.SENTRY_SPOTLIGHT;
   const tracesSampleRate = getTracesSampleRate(options.tracesSampleRate);
@@ -177,19 +177,6 @@ function applyDefaultOptions<T extends Partial<NodeClientOptions>>(options: T):
   };
 }
 
-function getRelease(release: NodeClientOptions['release']): string | undefined {
-  if (release !== undefined) {
-    return release;
-  }
-
-  const detectedRelease = getSentryRelease();
-  if (detectedRelease !== undefined) {
-    return detectedRelease;
-  }
-
-  return undefined;
-}
-
 /**
  * Tries to get a `tracesSampleRate`, possibly extracted from the environment variables.
  */

packages/node/src/sdk/index.ts

@@ -104,11 +104,7 @@ export function initWithoutDefaultIntegrations(options: NodeOptions = {}): NodeC
 
 /**
  * Initialize a Node client with the provided options and default integrations getter function.
- * This is an internal method the SDK uses under the hood to set up things - you should not use this as a user!
- * Instead, use `init()` to initialize the SDK.
- *
- * @hidden
- * @internal
+ * This is an internal method the SDK uses under the hood to set up things.
  */
 function _init(
   options: NodeOptions = {},