make it backwards compat

Author: AbhiPrasad

packages/browser/src/client.ts

@@ -85,9 +85,10 @@ export class BrowserClient extends Client<BrowserClientOptions> {
 
     super(opts);
 
-    const { sendDefaultPii, sendClientReports, enableLogs } = this._options;
+    const { sendDefaultPii, sendClientReports, enableLogs, _experiments } = this._options;
+    const shouldEnableLogs = enableLogs ?? _experiments?.enableLogs;
 
-    if (WINDOW.document && (sendClientReports || enableLogs)) {
+    if (WINDOW.document && (sendClientReports || shouldEnableLogs)) {
       WINDOW.document.addEventListener('visibilitychange', () => {
         if (WINDOW.document.visibilityState === 'hidden') {
           if (sendClientReports) {

packages/core/src/logs/console-integration.ts

@@ -33,8 +33,10 @@ const _consoleLoggingIntegration = ((options: Partial<CaptureConsoleOptions> = {
   return {
     name: INTEGRATION_NAME,
     setup(client) {
-      const { enableLogs, normalizeDepth = 3, normalizeMaxBreadth = 1_000 } = client.getOptions();
-      if (!enableLogs) {
+      const { enableLogs, _experiments, normalizeDepth = 3, normalizeMaxBreadth = 1_000 } = client.getOptions();
+      // eslint-disable-next-line deprecation/deprecation
+      const shouldEnableLogs = enableLogs ?? _experiments?.enableLogs;
+      if (!shouldEnableLogs) {
         DEBUG_BUILD && debug.warn('`enableLogs` is not enabled, ConsoleLogs integration disabled');
         return;
       }

packages/core/src/logs/exports.ts

@@ -124,11 +124,15 @@ export function _INTERNAL_captureLog(
     return;
   }
 
-  const { release, environment, enableLogs = false, beforeSendLog } = client.getOptions();
-  if (!enableLogs) {
+  const { release, environment, enableLogs = false, beforeSendLog, _experiments } = client.getOptions();
+  // eslint-disable-next-line deprecation/deprecation
+  const shouldEnableLogs = enableLogs ?? _experiments?.enableLogs;
+  if (!shouldEnableLogs) {
     DEBUG_BUILD && debug.warn('logging option not enabled, log will not be captured.');
     return;
   }
+  // eslint-disable-next-line deprecation/deprecation
+  const actualBeforeSendLog = beforeSendLog ?? _experiments?.beforeSendLog;
 
   const [, traceContext] = _getTraceInfoFromScope(client, currentScope);
 
@@ -168,7 +172,7 @@ export function _INTERNAL_captureLog(
   client.emit('beforeCaptureLog', processedLog);
 
   // We need to wrap this in `consoleSandbox` to avoid recursive calls to `beforeSendLog`
-  const log = beforeSendLog ? consoleSandbox(() => beforeSendLog(processedLog)) : processedLog;
+  const log = actualBeforeSendLog ? consoleSandbox(() => actualBeforeSendLog(processedLog)) : processedLog;
   if (!log) {
     client.recordDroppedEvent('before_send', 'log_item', 1);
     DEBUG_BUILD && debug.warn('beforeSendLog returned null, log will not be captured.');

packages/core/src/server-runtime-client.ts

@@ -49,7 +49,9 @@ export class ServerRuntimeClient<
 
     this._logWeight = 0;
 
-    if (this._options.enableLogs) {
+    // eslint-disable-next-line deprecation/deprecation
+    const shouldEnableLogs = this._options.enableLogs ?? this._options._experiments?.enableLogs;
+    if (shouldEnableLogs) {
       // eslint-disable-next-line @typescript-eslint/no-this-alias
       const client = this;
 

packages/core/src/types-hoist/options.ts

@@ -246,6 +246,31 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
   _experiments?: {
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
     [key: string]: any;
+
+    /**
+     * If logs support should be enabled.
+     *
+     * @deprecated Use the top-level `enableLogs` option instead.
+     *
+     * @default false
+     */
+    enableLogs?: boolean;
+
+    /**
+     * An event-processing callback for logs, guaranteed to be invoked after all other log
+     * processors. This allows a log to be modified or dropped before it's sent.
+     *
+     * 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.
+     *
+     * @deprecated Use the top-level `beforeSendLog` option instead.
+     *
+     * @default undefined
+     *
+     * @param log The log generated by the SDK.
+     * @returns A new log that will be sent | null.
+     */
+    beforeSendLog?: (log: Log) => Log | null;
   };
 
   /**

packages/node-core/src/sdk/client.ts

@@ -45,7 +45,10 @@ export class NodeClient extends ServerRuntimeClient<NodeClientOptions> {
 
     super(clientOptions);
 
-    if (this.getOptions().enableLogs) {
+    const { enableLogs, _experiments } = this.getOptions();
+    // eslint-disable-next-line deprecation/deprecation
+    const shouldEnableLogs = enableLogs ?? _experiments?.enableLogs;
+    if (shouldEnableLogs) {
       this._logOnExitFlushListener = () => {
         _INTERNAL_flushLogsBuffer(this);
       };