chore(deps): update JavaScript SDK to v7.69.0 (#3247)

Co-authored-by: GitHub <[email protected]>
Co-authored-by: Krystof Woldrich <[email protected]>

CHANGELOG.md

@@ -10,9 +10,9 @@
 - Bump Cocoa SDK from v8.10.0 to v8.11.0 ([#3245](https://github.com/getsentry/sentry-react-native/pull/3245))
   - [changelog](https://github.com/getsentry/sentry-cocoa/blob/main/CHANGELOG.md#8110)
   - [diff](https://github.com/getsentry/sentry-cocoa/compare/8.10.0...8.11.0)
-- Bump JavaScript SDK from v7.63.0 to v7.68.0 ([#3277](https://github.com/getsentry/sentry-react-native/pull/3277))
-  - [changelog](https://github.com/getsentry/sentry-javascript/blob/develop/CHANGELOG.md#7680)
-  - [diff](https://github.com/getsentry/sentry-javascript/compare/7.63.0...7.68.0)
+- Bump JavaScript SDK from v7.63.0 to v7.69.0 ([#3277](https://github.com/getsentry/sentry-react-native/pull/3277), [#3247](https://github.com/getsentry/sentry-react-native/pull/3247))
+  - [changelog](https://github.com/getsentry/sentry-javascript/blob/develop/CHANGELOG.md#7690)
+  - [diff](https://github.com/getsentry/sentry-javascript/compare/7.63.0...7.69.0)
 - Bump Android SDK from v6.28.0 to v6.29.0 ([#3271](https://github.com/getsentry/sentry-react-native/pull/3271))
   - [changelog](https://github.com/getsentry/sentry-java/blob/main/CHANGELOG.md#6290)
   - [diff](https://github.com/getsentry/sentry-java/compare/6.28.0...6.29.0)

package.json

@@ -56,18 +56,18 @@
     "react-native": ">=0.65.0"
   },
   "dependencies": {
-    "@sentry/browser": "7.68.0",
+    "@sentry/browser": "7.69.0",
     "@sentry/cli": "2.20.7",
-    "@sentry/core": "7.68.0",
-    "@sentry/hub": "7.68.0",
-    "@sentry/integrations": "7.68.0",
-    "@sentry/react": "7.68.0",
-    "@sentry/types": "7.68.0",
-    "@sentry/utils": "7.68.0"
+    "@sentry/core": "7.69.0",
+    "@sentry/hub": "7.69.0",
+    "@sentry/integrations": "7.69.0",
+    "@sentry/react": "7.69.0",
+    "@sentry/types": "7.69.0",
+    "@sentry/utils": "7.69.0"
   },
   "devDependencies": {
-    "@sentry-internal/eslint-config-sdk": "7.68.0",
-    "@sentry-internal/eslint-plugin-sdk": "7.68.0",
+    "@sentry-internal/eslint-config-sdk": "7.69.0",
+    "@sentry-internal/eslint-plugin-sdk": "7.69.0",
     "@sentry/typescript": "^5.20.1",
     "@sentry/wizard": "3.9.1",
     "@types/jest": "^29.5.3",

sample-new-architecture/src/Screens/PerformanceTimingScreen.tsx

@@ -15,14 +15,14 @@ const PerformanceTimingScreen = () => {
   const [dateTime, setDateTime] = React.useState(0);
 
   React.useEffect(() => {
-    // @ts-ignore
+    // @ts-expect-error
     if (typeof global.performance !== 'undefined') {
       initialDate.current = Date.now();
-      // @ts-ignore
+      // @ts-expect-error
       initialPerformance.current = global.performance.now();
 
       const interval = setInterval(() => {
-        // @ts-ignore
+        // @ts-expect-error
         setPerformanceTime(global.performance.now());
         setDateTime(Date.now());
       }, 1000);

src/js/profiling/integration.ts

@@ -98,7 +98,6 @@ export class HermesProfiling implements Integration {
     const client = this._getCurrentHub && this._getCurrentHub().getClient();
     const options = client && client.getOptions();
 
-    // @ts-ignore not part of the browser options yet
     const profilesSampleRate =
       options && options._experiments && typeof options._experiments.profilesSampleRate === 'number'
         ? options._experiments.profilesSampleRate

src/js/tracing/reactnativetracing.ts

@@ -160,11 +160,7 @@ export class ReactNativeTracing implements Integration {
   /**
    *  Registers routing and request instrumentation.
    */
-  public setupOnce(
-    // @ts-ignore TODO
-    addGlobalEventProcessor: (callback: EventProcessor) => void,
-    getCurrentHub: () => Hub,
-  ): void {
+  public setupOnce(addGlobalEventProcessor: (callback: EventProcessor) => void, getCurrentHub: () => Hub): void {
     const hub = getCurrentHub();
     const client = hub.getClient();
     const clientOptions = client && client.getOptions();
@@ -175,7 +171,6 @@ export class ReactNativeTracing implements Integration {
       traceXHR,
       // eslint-disable-next-line deprecation/deprecation
       tracingOrigins,
-      // @ts-ignore TODO
       shouldCreateSpanForRequest,
       // eslint-disable-next-line deprecation/deprecation
       tracePropagationTargets: thisOptionsTracePropagationTargets,

src/js/tracing/transaction.ts

@@ -1,5 +1,4 @@
-import type { IdleTransaction } from '@sentry/core';
-import type { BeforeFinishCallback } from '@sentry/core/types/tracing/idletransaction';
+import type { BeforeFinishCallback, IdleTransaction } from '@sentry/core';
 import { logger } from '@sentry/utils';
 
 /**

src/js/wrapper.ts

@@ -574,7 +574,7 @@ export const NATIVE: SentryNativeWrapper = {
 
       if (NATIVE.platform === 'android') {
         if ('message' in event) {
-          // @ts-ignore Android still uses the old message object, without this the serialization of events will break.
+          // @ts-expect-error Android still uses the old message object, without this the serialization of events will break.
           event.message = { message: event.message };
         }
       }

test/client.test.ts

@@ -102,7 +102,7 @@ describe('Tests ReactNativeClient', () => {
       });
 
       await expect(client.eventFromMessage('test')).resolves.toBeDefined();
-      // @ts-ignore: Is Mocked
+      // @ts-expect-error: Is Mocked
       await expect(RN.LogBox.ignoreLogs).toBeCalled();
     });
 

test/integrations/release.test.ts

@@ -38,15 +38,15 @@ describe('Tests the Release integration', () => {
 
     let eventProcessor: EventProcessor = () => null;
 
-    // @ts-ignore Mock
+    // @ts-expect-error Mock
     addGlobalEventProcessor.mockImplementation(e => (eventProcessor = e));
     releaseIntegration.setupOnce();
 
     expect(addGlobalEventProcessor).toBeCalled();
 
     const client = getCurrentHub().getClient();
 
-    // @ts-ignore Mock
+    // @ts-expect-error Mock
     client.getOptions.mockImplementation(() => ({}));
 
     const event = await eventProcessor({}, {});
@@ -60,13 +60,13 @@ describe('Tests the Release integration', () => {
 
     let eventProcessor: EventProcessor = () => null;
 
-    // @ts-ignore Mock
+    // @ts-expect-error Mock
     addGlobalEventProcessor.mockImplementation(e => (eventProcessor = e));
     releaseIntegration.setupOnce();
 
     const client = getCurrentHub().getClient();
 
-    // @ts-ignore Mock
+    // @ts-expect-error Mock
     client.getOptions.mockImplementation(() => ({
       dist: 'options_dist',
     }));
@@ -82,13 +82,13 @@ describe('Tests the Release integration', () => {
 
     let eventProcessor: EventProcessor = () => null;
 
-    // @ts-ignore Mock
+    // @ts-expect-error Mock
     addGlobalEventProcessor.mockImplementation(e => (eventProcessor = e));
     releaseIntegration.setupOnce();
 
     const client = getCurrentHub().getClient();
 
-    // @ts-ignore Mock
+    // @ts-expect-error Mock
     client.getOptions.mockImplementation(() => ({
       release: 'options_release',
     }));
@@ -104,15 +104,15 @@ describe('Tests the Release integration', () => {
 
     let eventProcessor: EventProcessor = () => null;
 
-    // @ts-ignore Mock
+    // @ts-expect-error Mock
     addGlobalEventProcessor.mockImplementation(e => (eventProcessor = e));
     releaseIntegration.setupOnce();
 
     expect(addGlobalEventProcessor).toBeCalled();
 
     const client = getCurrentHub().getClient();
 
-    // @ts-ignore Mock
+    // @ts-expect-error Mock
     client.getOptions.mockImplementation(() => ({
       dist: 'options_dist',
       release: 'options_release',
@@ -129,15 +129,15 @@ describe('Tests the Release integration', () => {
 
     let eventProcessor: EventProcessor = () => null;
 
-    // @ts-ignore Mock
+    // @ts-expect-error Mock
     addGlobalEventProcessor.mockImplementation(e => (eventProcessor = e));
     releaseIntegration.setupOnce();
 
     expect(addGlobalEventProcessor).toBeCalled();
 
     const client = getCurrentHub().getClient();
 
-    // @ts-ignore Mock
+    // @ts-expect-error Mock
     client.getOptions.mockImplementation(() => ({
       dist: 'options_dist',
       release: 'options_release',

test/sdk.test.ts

@@ -217,7 +217,7 @@ describe('Tests the SDK functionality', () => {
         (NATIVE.isNativeAvailable as jest.Mock).mockImplementation(() => false);
         init({});
         expect(NATIVE.isNativeAvailable).toBeCalled();
-        // @ts-ignore enableNative not publicly available here.
+        // @ts-expect-error enableNative not publicly available here.
         expect(usedOptions()?.enableNative).toEqual(false);
         expect(usedOptions()?.transport).toEqual(makeFetchTransport);
       });
@@ -226,7 +226,7 @@ describe('Tests the SDK functionality', () => {
         (NATIVE.isNativeAvailable as jest.Mock).mockImplementation(() => false);
         init({ enableNative: true });
         expect(NATIVE.isNativeAvailable).toBeCalled();
-        // @ts-ignore enableNative not publicly available here.
+        // @ts-expect-error enableNative not publicly available here.
         expect(usedOptions()?.enableNative).toEqual(false);
         expect(usedOptions()?.transport).toEqual(makeFetchTransport);
       });
@@ -235,7 +235,7 @@ describe('Tests the SDK functionality', () => {
         (NATIVE.isNativeAvailable as jest.Mock).mockImplementation(() => false);
         init({ enableNative: false });
         expect(NATIVE.isNativeAvailable).not.toBeCalled();
-        // @ts-ignore enableNative not publicly available here.
+        // @ts-expect-error enableNative not publicly available here.
         expect(usedOptions()?.enableNative).toEqual(false);
         expect(usedOptions()?.transport).toEqual(makeFetchTransport);
       });
@@ -248,7 +248,7 @@ describe('Tests the SDK functionality', () => {
         });
         expect(usedOptions()?.transport).toEqual(mockTransport);
         expect(NATIVE.isNativeAvailable).toBeCalled();
-        // @ts-ignore enableNative not publicly available here.
+        // @ts-expect-error enableNative not publicly available here.
         expect(usedOptions()?.enableNative).toEqual(false);
       });
     });

test/touchevents.test.tsx

@@ -60,7 +60,7 @@ describe('TouchEventBoundary._onTouchStart', () => {
       },
     };
 
-    // @ts-ignore Calling private member
+    // @ts-expect-error Calling private member
     boundary._onTouchStart(event);
 
     expect(addBreadcrumb).not.toBeCalled();
@@ -97,7 +97,7 @@ describe('TouchEventBoundary._onTouchStart', () => {
       },
     };
 
-    // @ts-ignore Calling private member
+    // @ts-expect-error Calling private member
     boundary._onTouchStart(event);
 
     expect(addBreadcrumb).toBeCalledWith({
@@ -156,7 +156,7 @@ describe('TouchEventBoundary._onTouchStart', () => {
       },
     };
 
-    // @ts-ignore Calling private member
+    // @ts-expect-error Calling private member
     boundary._onTouchStart(event);
 
     expect(addBreadcrumb).toBeCalledWith({
@@ -206,7 +206,7 @@ describe('TouchEventBoundary._onTouchStart', () => {
       },
     };
 
-    // @ts-ignore Calling private member
+    // @ts-expect-error Calling private member
     boundary._onTouchStart(event);
 
     expect(addBreadcrumb).toBeCalledWith({

test/tracing/reactnativetracing.test.ts

@@ -126,11 +126,11 @@ describe('ReactNativeTracing', () => {
           expect(transaction.op).toBe(UI_LOAD);
 
           expect(
-            // @ts-ignore access private for test
+            // @ts-expect-error access private for test
             transaction._measurements[APP_START_COLD].value,
           ).toEqual(timeOriginMilliseconds - appStartTimeMilliseconds);
           expect(
-            // @ts-ignore access private for test
+            // @ts-expect-error access private for test
             transaction._measurements[APP_START_COLD].unit,
           ).toBe('millisecond');
         }
@@ -163,11 +163,11 @@ describe('ReactNativeTracing', () => {
           expect(transaction.op).toBe(UI_LOAD);
 
           expect(
-            // @ts-ignore access private for test
+            // @ts-expect-error access private for test
             transaction._measurements[APP_START_WARM].value,
           ).toEqual(timeOriginMilliseconds - appStartTimeMilliseconds);
           expect(
-            // @ts-ignore access private for test
+            // @ts-expect-error access private for test
             transaction._measurements[APP_START_WARM].unit,
           ).toBe('millisecond');
         }
@@ -198,12 +198,12 @@ describe('ReactNativeTracing', () => {
 
         if (transaction) {
           expect(
-            // @ts-ignore access private for test
+            // @ts-expect-error access private for test
             transaction._measurements[APP_START_WARM],
           ).toBeUndefined();
 
           expect(
-            // @ts-ignore access private for test
+            // @ts-expect-error access private for test
             transaction._measurements[APP_START_COLD],
           ).toBeUndefined();
         }
@@ -271,7 +271,7 @@ describe('ReactNativeTracing', () => {
         // trigger idle transaction to finish and call before finish callbacks
         jest.advanceTimersByTime(DEFAULT_IDLE_TIMEOUT);
 
-        // @ts-ignore access private for test
+        // @ts-expect-error access private for test
         expect(routeTransaction._measurements[APP_START_COLD].value).toBe(
           timeOriginMilliseconds - appStartTimeMilliseconds,
         );
@@ -327,7 +327,7 @@ describe('ReactNativeTracing', () => {
         // trigger idle transaction to finish and call before finish callbacks
         jest.advanceTimersByTime(DEFAULT_IDLE_TIMEOUT);
 
-        // @ts-ignore access private for test
+        // @ts-expect-error access private for test
         expect(routeTransaction._measurements[APP_START_WARM].value).toBe(
           timeOriginMilliseconds - appStartTimeMilliseconds,
         );
@@ -383,7 +383,7 @@ describe('ReactNativeTracing', () => {
         // trigger idle transaction to finish and call before finish callbacks
         jest.advanceTimersByTime(DEFAULT_IDLE_TIMEOUT);
 
-        // @ts-ignore access private for test
+        // @ts-expect-error access private for test
         expect(routeTransaction._measurements).toMatchObject({});
 
         expect(routeTransaction.op).not.toBe(UI_LOAD);

test/vendor/buffer/utf8ToBytes.test.ts

@@ -73,9 +73,9 @@ describe('Buffer utf8 tests', () => {
   describe('test strings', () => {
     for (const input of testCases) {
       it(`should encode "${input}"`, () => {
-        // @ts-ignore The test run in node where Buffer is available
+        // @ts-expect-error The test run in node where Buffer is available
         const actual = Buffer.from(utf8ToBytes(input));
-        // @ts-ignore The test run in node where Buffer is available
+        // @ts-expect-error The test run in node where Buffer is available
         const expected = Buffer.from(input, 'utf8');
 
         expect(actual).toEqual(expected);