getsentry · AbhiPrasad · Nov 4, 2024 · Oct 30, 2024 · Oct 31, 2024 · Oct 31, 2024
diff --git a/dev-packages/browser-integration-tests/suites/feedback/attachTo/init.js b/dev-packages/browser-integration-tests/suites/feedback/attachTo/init.js
@@ -14,4 +14,4 @@ Sentry.init({
   integrations: [feedback],
 });
 
-feedback.attachTo('#custom-feedback-buttom');
+feedback.attachTo('#custom-feedback-button');
diff --git a/dev-packages/browser-integration-tests/suites/feedback/attachTo/template.html b/dev-packages/browser-integration-tests/suites/feedback/attachTo/template.html
@@ -4,6 +4,6 @@
     <meta charset="utf-8" />
   </head>
   <body>
-    <button type="button" id="custom-feedback-buttom">Show feedback!</button>
+    <button type="button" id="custom-feedback-button">Show feedback!</button>
   </body>
 </html>
diff --git a/dev-packages/browser-integration-tests/suites/feedback/attachTo/test.ts b/dev-packages/browser-integration-tests/suites/feedback/attachTo/test.ts
@@ -26,7 +26,7 @@ sentryTest('should capture feedback with custom button', async ({ getLocalTestUr
   const url = await getLocalTestUrl({ testDir: __dirname });
 
   await page.goto(url);
-  await page.locator('#custom-feedback-buttom').click();
+  await page.locator('#custom-feedback-button').click();
   await page.waitForSelector(':visible:text-is("Report a Bug")');
 
   expect(await page.locator(':visible:text-is("Report a Bug")').count()).toEqual(1);

diff --git a/...rowser-integration-tests/suites/public-api/instrumentation/onError/non-string-arg/test.ts b/...rowser-integration-tests/suites/public-api/instrumentation/onError/non-string-arg/test.ts
@@ -8,7 +8,7 @@ sentryTest(
   'should catch onerror calls with non-string first argument gracefully',
   async ({ getLocalTestPath, page, browserName }) => {
     if (browserName === 'webkit') {
-      // This test fails on Webkit as erros thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
+      // This test fails on Webkit as errors thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
       sentryTest.skip();
     }
 

diff --git a/...ages/browser-integration-tests/suites/public-api/instrumentation/onError/rethrown/test.ts b/...ages/browser-integration-tests/suites/public-api/instrumentation/onError/rethrown/test.ts
@@ -8,7 +8,7 @@ sentryTest(
   'should NOT catch an exception already caught [but rethrown] via Sentry.captureException',
   async ({ getLocalTestPath, page, browserName }) => {
     if (browserName === 'webkit') {
-      // This test fails on Webkit as erros thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
+      // This test fails on Webkit as errors thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
       sentryTest.skip();
     }
 

diff --git a/...browser-integration-tests/suites/public-api/instrumentation/onError/syntax-errors/test.ts b/...browser-integration-tests/suites/public-api/instrumentation/onError/syntax-errors/test.ts
@@ -6,7 +6,7 @@ import { getFirstSentryEnvelopeRequest, runScriptInSandbox } from '../../../../.
 
 sentryTest('should catch syntax errors', async ({ getLocalTestPath, page, browserName }) => {
   if (browserName === 'webkit') {
-    // This test fails on Webkit as erros thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
+    // This test fails on Webkit as errors thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
     sentryTest.skip();
   }
 

diff --git a/...browser-integration-tests/suites/public-api/instrumentation/onError/thrown-errors/test.ts b/...browser-integration-tests/suites/public-api/instrumentation/onError/thrown-errors/test.ts
@@ -6,7 +6,7 @@ import { getFirstSentryEnvelopeRequest, runScriptInSandbox } from '../../../../.
 
 sentryTest('should catch thrown errors', async ({ getLocalTestPath, page, browserName }) => {
   if (browserName === 'webkit') {
-    // This test fails on Webkit as erros thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
+    // This test fails on Webkit as errors thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
     sentryTest.skip();
   }
 

diff --git a/...rowser-integration-tests/suites/public-api/instrumentation/onError/thrown-objects/test.ts b/...rowser-integration-tests/suites/public-api/instrumentation/onError/thrown-objects/test.ts
@@ -6,7 +6,7 @@ import { getFirstSentryEnvelopeRequest, runScriptInSandbox } from '../../../../.
 
 sentryTest('should catch thrown objects', async ({ getLocalTestPath, page, browserName }) => {
   if (browserName === 'webkit') {
-    // This test fails on Webkit as erros thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
+    // This test fails on Webkit as errors thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
     sentryTest.skip();
   }
 

diff --git a/...rowser-integration-tests/suites/public-api/instrumentation/onError/thrown-strings/test.ts b/...rowser-integration-tests/suites/public-api/instrumentation/onError/thrown-strings/test.ts
@@ -6,7 +6,7 @@ import { getFirstSentryEnvelopeRequest, runScriptInSandbox } from '../../../../.
 
 sentryTest('should catch thrown strings', async ({ getLocalTestPath, page, browserName }) => {
   if (browserName === 'webkit') {
-    // This test fails on Webkit as erros thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
+    // This test fails on Webkit as errors thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
     sentryTest.skip();
   }
 

diff --git a/dev-packages/browser-integration-tests/suites/public-api/startSpan/error-sync/test.ts b/dev-packages/browser-integration-tests/suites/public-api/startSpan/error-sync/test.ts
@@ -12,7 +12,7 @@ sentryTest(
   'should capture an error within a sync startSpan callback',
   async ({ getLocalTestPath, page, browserName }) => {
     if (browserName === 'webkit') {
-      // This test fails on Webkit as erros thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
+      // This test fails on Webkit as errors thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
       sentryTest.skip();
     }
 

@@ -18,5 +18,5 @@ Sentry.init({
 
 window.Replay._replay.timeouts = {
   sessionIdlePause: 1000, // this is usually 5min, but we want to test this with shorter times
-  sessionIdleExpire: 900000, // defayult: 15min
+  sessionIdleExpire: 900000, // default: 15min
 };
@@ -30,7 +30,7 @@ sentryTest(
     const spans = [...res0.performanceSpans, ...res1.performanceSpans];
     expect(breadcrumbs.filter(breadcrumb => breadcrumb.category === 'replay.throttled').length).toBe(1);
     // replay.throttled breadcrumb does *not* use the throttledAddEvent as we
-    // alwants want that breadcrumb to be present in replay
+    // always want that breadcrumb to be present in replay
     expect(breadcrumbs.length + spans.length).toBe(THROTTLE_LIMIT + 1);
   },
 );
diff --git a/...packages/browser-integration-tests/suites/tracing/browserTracingIntegration/error/test.ts b/...packages/browser-integration-tests/suites/tracing/browserTracingIntegration/error/test.ts
@@ -11,7 +11,7 @@ sentryTest(
   'should put the pageload transaction name onto an error event caught during pageload',
   async ({ getLocalTestPath, page, browserName }) => {
     if (browserName === 'webkit') {
-      // This test fails on Webkit as erros thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
+      // This test fails on Webkit as errors thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
       sentryTest.skip();
     }
 

diff --git a/...rowser-integration-tests/suites/tracing/browserTracingIntegration/interactions/subject.js b/...rowser-integration-tests/suites/tracing/browserTracingIntegration/interactions/subject.js
@@ -1,12 +1,12 @@
 const blockUI = e => {
   const startTime = Date.now();
 
-  function getElasped() {
+  function getElapsed() {
     const time = Date.now();
     return time - startTime;
   }
 
-  while (getElasped() < 70) {
+  while (getElapsed() < 70) {
     //
   }
 

diff --git a/...s/suites/tracing/browserTracingIntegration/long-animation-frame-disabled/assets/script.js b/...s/suites/tracing/browserTracingIntegration/long-animation-frame-disabled/assets/script.js
@@ -1,12 +1,12 @@
 (() => {
   const startTime = Date.now();
 
-  function getElasped() {
+  function getElapsed() {
     const time = Date.now();
     return time - startTime;
   }
 
-  while (getElasped() < 101) {
+  while (getElapsed() < 101) {
     //
   }
 })();
diff --git a/...ites/tracing/browserTracingIntegration/long-animation-frame-non-chromium/assets/script.js b/...ites/tracing/browserTracingIntegration/long-animation-frame-non-chromium/assets/script.js
@@ -1,12 +1,12 @@
 (() => {
   const startTime = Date.now();
 
-  function getElasped() {
+  function getElapsed() {
     const time = Date.now();
     return time - startTime;
   }
 
-  while (getElasped() < 101) {
+  while (getElapsed() < 101) {
     //
   }
 })();
diff --git a/...ation-tests/suites/tracing/browserTracingIntegration/long-tasks-disabled/assets/script.js b/...ation-tests/suites/tracing/browserTracingIntegration/long-tasks-disabled/assets/script.js
@@ -1,12 +1,12 @@
 (() => {
   const startTime = Date.now();
 
-  function getElasped() {
+  function getElapsed() {
     const time = Date.now();
     return time - startTime;
   }
 
-  while (getElasped() < 101) {
+  while (getElapsed() < 101) {
     //
   }
 })();
diff --git a/...ration-tests/suites/tracing/browserTracingIntegration/long-tasks-enabled/assets/script.js b/...ration-tests/suites/tracing/browserTracingIntegration/long-tasks-enabled/assets/script.js
@@ -1,12 +1,12 @@
 (() => {
   const startTime = Date.now();
 
-  function getElasped() {
+  function getElapsed() {
     const time = Date.now();
     return time - startTime;
   }
 
-  while (getElasped() < 105) {
+  while (getElapsed() < 105) {
     //
   }
 })();
diff --git a/...s/suites/tracing/browserTracingIntegration/long-tasks-no-animation-frame/assets/script.js b/...s/suites/tracing/browserTracingIntegration/long-tasks-no-animation-frame/assets/script.js
@@ -1,12 +1,12 @@
 (() => {
   const startTime = Date.now();
 
-  function getElasped() {
+  function getElapsed() {
     const time = Date.now();
     return time - startTime;
   }
 
-  while (getElasped() < 105) {
+  while (getElapsed() < 105) {
     //
   }
 })();
diff --git a/...packages/browser-integration-tests/suites/tracing/metrics/pageload-resource-spans/test.ts b/...packages/browser-integration-tests/suites/tracing/metrics/pageload-resource-spans/test.ts
@@ -20,7 +20,7 @@ sentryTest('should add resource spans to pageload transaction', async ({ getLoca
   const eventData = await getFirstSentryEnvelopeRequest<Event>(page, url);
   const resourceSpans = eventData.spans?.filter(({ op }) => op?.startsWith('resource'));
 
-  // Webkit 16.0 (which is linked to Playwright 1.27.1) consistently creates 2 consectutive spans for `css`,
+  // Webkit 16.0 (which is linked to Playwright 1.27.1) consistently creates 2 consecutive spans for `css`,
   // so we need to check for 3 or 4 spans.
   if (browser.browserType().name() === 'webkit') {
     expect(resourceSpans?.length).toBeGreaterThanOrEqual(3);

diff --git a/dev-packages/browser-integration-tests/utils/helpers.ts b/dev-packages/browser-integration-tests/utils/helpers.ts
@@ -275,8 +275,8 @@ export function shouldSkipMetricsTest(): boolean {
 
 /**
  * Waits until a number of requests matching urlRgx at the given URL arrive.
- * If the timout option is configured, this function will abort waiting, even if it hasn't reveived the configured
- * amount of requests, and returns all the events recieved up to that point in time.
+ * If the timeout option is configured, this function will abort waiting, even if it hasn't received the configured
+ * amount of requests, and returns all the events received up to that point in time.
  */
 async function getMultipleRequests<T>(
   page: Page,

diff --git a/dev-packages/browser-integration-tests/utils/replayEventTemplates.ts b/dev-packages/browser-integration-tests/utils/replayEventTemplates.ts
@@ -44,7 +44,7 @@ const DEFAULT_REPLAY_EVENT = {
  * This is useful for testing multi-segment replays to not repeat most of the properties that don't change
  * throughout the replay segments.
  *
- * Note: The benfit of this approach over expect.objectContaining is that,
+ * Note: The benefit of this approach over expect.objectContaining is that,
  *       we'll catch if properties we expect to stay the same actually change.
  *
  * @param customExpectedReplayEvent overwrite the default values with custom values (e.g. segment_id)

diff --git a/dev-packages/e2e-tests/test-applications/astro-4/src/pages/index.astro b/dev-packages/e2e-tests/test-applications/astro-4/src/pages/index.astro
@@ -8,7 +8,7 @@ import Layout from '../layouts/Layout.astro';
 		<ul role="list" style="display: flex; flex-direction: column;">
 			<a href="/ssr-error">SSR Error</a>
 			<a href="/endpoint-error">Endpoint Error</a>
-			<a href="/client-error">Cliennt Error</a>
+			<a href="/client-error">Client Error</a>
 			<a href="/test-ssr">SSR page</a>
 			<a href="/test-static" title="static page">Static Page</a>
 		</ul>

diff --git a/dev-packages/e2e-tests/test-applications/nextjs-14/tests/generation-functions.test.ts b/dev-packages/e2e-tests/test-applications/nextjs-14/tests/generation-functions.test.ts
@@ -1,7 +1,7 @@
 import { expect, test } from '@playwright/test';
 import { waitForError, waitForTransaction } from '@sentry-internal/test-utils';
 
-test('Should emit a span for a generateMetadata() function invokation', async ({ page }) => {
+test('Should emit a span for a generateMetadata() function invocation', async ({ page }) => {
   const testTitle = 'should-emit-span';
 
   const transactionPromise = waitForTransaction('nextjs-14', async transactionEvent => {
@@ -29,7 +29,7 @@ test('Should emit a span for a generateMetadata() function invokation', async ({
   expect(pageTitle).toBe(testTitle);
 });
 
-test('Should send a transaction and an error event for a faulty generateMetadata() function invokation', async ({
+test('Should send a transaction and an error event for a faulty generateMetadata() function invocation', async ({
   page,
 }) => {
   const testTitle = 'should-emit-error';
@@ -59,7 +59,7 @@ test('Should send a transaction and an error event for a faulty generateMetadata
   expect(transactionEvent.tags?.['my-global-scope-isolated-tag']).not.toBeDefined();
 });
 
-test('Should send a transaction event for a generateViewport() function invokation', async ({ page }) => {
+test('Should send a transaction event for a generateViewport() function invocation', async ({ page }) => {
   const testTitle = 'floob';
 
   const transactionPromise = waitForTransaction('nextjs-14', async transactionEvent => {
@@ -83,7 +83,7 @@ test('Should send a transaction event for a generateViewport() function invokati
   );
 });
 
-test('Should send a transaction and an error event for a faulty generateViewport() function invokation', async ({
+test('Should send a transaction and an error event for a faulty generateViewport() function invocation', async ({
   page,
 }) => {
   const testTitle = 'blargh';
@@ -109,7 +109,7 @@ test('Should send a transaction and an error event for a faulty generateViewport
   expect(errorEvent.transaction).toBe('Page.generateViewport (/generation-functions)');
 });
 
-test('Should send a transaction event with correct status for a generateMetadata() function invokation with redirect()', async ({
+test('Should send a transaction event with correct status for a generateMetadata() function invocation with redirect()', async ({
   page,
 }) => {
   const testTitle = 'redirect-foobar';
@@ -126,7 +126,7 @@ test('Should send a transaction event with correct status for a generateMetadata
   expect((await transactionPromise).contexts?.trace?.status).toBe('ok');
 });
 
-test('Should send a transaction event with correct status for a generateMetadata() function invokation with notfound()', async ({
+test('Should send a transaction event with correct status for a generateMetadata() function invocation with notfound()', async ({
   page,
 }) => {
   const testTitle = 'notfound-foobar';

diff --git a/dev-packages/e2e-tests/test-applications/node-otel-custom-sampler/src/instrument.ts b/dev-packages/e2e-tests/test-applications/node-otel-custom-sampler/src/instrument.ts
@@ -12,7 +12,7 @@ Sentry.init({
   debug: !!process.env.DEBUG,
   tunnel: `http://localhost:3031/`, // proxy server
   skipOpenTelemetrySetup: true,
-  // By defining _any_ sample rate, tracing intergations will be added by default
+  // By defining _any_ sample rate, tracing integrations will be added by default
   tracesSampleRate: 0,
 });
 

diff --git a/dev-packages/e2e-tests/test-applications/node-otel-without-tracing/tests/errors.test.ts b/dev-packages/e2e-tests/test-applications/node-otel-without-tracing/tests/errors.test.ts
@@ -20,7 +20,7 @@ test('Sends correct error event', async ({ baseURL }) => {
     url: 'http://localhost:3030/test-exception/123',
   });
 
-  // This is unparametrized here because we do not have the express instrumentation
+  // This is unparameterized here because we do not have the express instrumentation
   expect(errorEvent.transaction).toEqual('GET /test-exception/123');
 
   expect(errorEvent.contexts?.trace).toEqual({

diff --git a/dev-packages/e2e-tests/test-applications/react-router-6/tests/sse.test.ts b/dev-packages/e2e-tests/test-applications/react-router-6/tests/sse.test.ts
@@ -65,7 +65,7 @@ test('Waits for sse streaming when sse has been explicitly aborted', async ({ pa
   expect(resolveDuration).toBe(0);
   expect(resolveBodyDuration).toBe(0);
 
-  // validate abort eror was thrown by inspecting console
+  // validate abort error was thrown by inspecting console
   const consoleBreadcrumb = rootSpan.breadcrumbs?.find(breadcrumb => breadcrumb.category === 'console');
   expect(consoleBreadcrumb?.message).toBe('Could not fetch sse AbortError: BodyStreamBuffer was aborted');
 });

diff --git a/dev-packages/node-integration-tests/suites/express/tracing/tracesSampler/server.js b/dev-packages/node-integration-tests/suites/express/tracing/tracesSampler/server.js
@@ -8,7 +8,7 @@ Sentry.init({
   tracesSampler: samplingContext => {
     // The name we get here is inferred at span creation time
     // At this point, we sadly do not have a http.route attribute yet,
-    // so we infer the name from the unparametrized route instead
+    // so we infer the name from the unparameterized route instead
     return (
       samplingContext.name === 'GET /test/123' &&
       samplingContext.attributes['sentry.op'] === 'http.server' &&

diff --git a/dev-packages/rollup-utils/plugins/extractPolyfillsPlugin.mjs b/dev-packages/rollup-utils/plugins/extractPolyfillsPlugin.mjs
@@ -13,7 +13,7 @@ const POLYFILL_NAMES = new Set([
 ]);
 
 /**
- * Create a plugin which will replace function definitions of any of the above funcions with an `import` or `require`
+ * Create a plugin which will replace function definitions of any of the above functions with an `import` or `require`
  * statement pulling them in from a central source. Mimics tsc's `importHelpers` option.
  */
 export function makeExtractPolyfillsPlugin() {
@@ -40,7 +40,7 @@ export function makeExtractPolyfillsPlugin() {
         return null;
       }
 
-      // The index.js file of the tuils package will include identifiers named after polyfills so we would inject the
+      // The index.js file of the utils package will include identifiers named after polyfills so we would inject the
       // polyfills, however that would override the exports so we should just skip that file.
       const isUtilsPackage = process.cwd().endsWith(`packages${path.sep}utils`);
       if (isUtilsPackage && sourceFile === 'index.js') {

diff --git a/dev-packages/size-limit-gh-action/index.mjs b/dev-packages/size-limit-gh-action/index.mjs
@@ -130,7 +130,7 @@ async function run() {
     const sizeLimitComment = await fetchPreviousComment(octokit, repo, pr);
 
     if (sizeLimitComment) {
-      core.debug('Found existing size limit comment, udpating it instead of creating a new one...');
+      core.debug('Found existing size limit comment, updating it instead of creating a new one...');
     }
 
     const shouldComment =

diff --git a/docs/creating-a-new-sdk.md b/docs/creating-a-new-sdk.md
@@ -51,7 +51,7 @@ Some of the following concepts may be relevant to your SDK:
 - **Redirects**: If possible, we want to skip redirects. This means that if a user navigates to `/`, and this redirects
   the user internally to `/dashboard`, we only want to capture a single `/` navigation/pageload.
 - **Route Params**: Routes should be parametrized, which means that instead of `/users/123` we want to capture
-  `/users/:id` or simmilar.
+  `/users/:id` or similar.
 - **Query Params**: Query params should generally be removed from the route.
 
 #### Component Tracking
@@ -88,7 +88,7 @@ own instrumentation to capture `http.server` spans.
 Some of the following concepts may be relevant to your SDK:
 
 - **Route Params**: Routes should be parametrized, which means that instead of `/users/123` we want to capture
-  `/users/:id` or simmilar.
+  `/users/:id` or similar.
 - **Query Params**: Query params should generally be removed from the route.
 
 #### Middleware Tracking

diff --git a/docs/migration/feedback.md b/docs/migration/feedback.md
@@ -1,6 +1,6 @@
 # End of Feedback Beta
 
-With the release of 8.0.0, Sentry Feedback is now out of Beta. This means that the usual stabilty guarantees apply.
+With the release of 8.0.0, Sentry Feedback is now out of Beta. This means that the usual stability guarantees apply.
 
 Feedback 8.0.0 requires server version 24.4.2 and above.
 
@@ -89,7 +89,7 @@ function attachTo(button: HTMLElement) {
       onFormClose: () => {
         widget.close();
       },
-      onFormSubmited: () => {
+      onFormSubmitted: () => {
         widget.removeFromDom();
       }
     });

diff --git a/docs/v8-node.md b/docs/v8-node.md
@@ -136,7 +136,7 @@ const { SentrySpanProcessor, SentryPropagator, SentryContextManager, SentrySampl
 // We need a custom span processor
 provider.addSpanProcessor(new SentrySpanProcessor());
 // We need a custom propagator and context manager
-provier.register({
+provider.register({
   propagator: new SentryPropagator(),
   contextManager: new SentryContextManager(),
 });