diff --git a/dev-packages/browser-integration-tests/suites/feedback/attachTo/init.js b/dev-packages/browser-integration-tests/suites/feedback/attachTo/init.js index 5eb27143fdc7..740fb69558ed 100644 --- 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 index ae36b0c69c7b..d0c83c526ca4 100644 --- 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 @@ - + diff --git a/dev-packages/browser-integration-tests/suites/feedback/attachTo/test.ts b/dev-packages/browser-integration-tests/suites/feedback/attachTo/test.ts index 8c605597020d..022e8f789c2a 100644 --- 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/dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/non-string-arg/test.ts b/dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/non-string-arg/test.ts index 681d5db8bf02..86227532efb1 100644 --- a/dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/non-string-arg/test.ts +++ b/dev-packages/browser-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/dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/rethrown/test.ts b/dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/rethrown/test.ts index 01b319e759b2..58a35d86a800 100644 --- a/dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/rethrown/test.ts +++ b/dev-packages/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/dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/syntax-errors/test.ts b/dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/syntax-errors/test.ts index 2b6cc09be8a2..46b68f8ccb24 100644 --- a/dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/syntax-errors/test.ts +++ b/dev-packages/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/dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/thrown-errors/test.ts b/dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/thrown-errors/test.ts index 17dd6c650b43..47607a22b289 100644 --- a/dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/thrown-errors/test.ts +++ b/dev-packages/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/dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/thrown-objects/test.ts b/dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/thrown-objects/test.ts index 4ed03991ff58..71159b605b4e 100644 --- a/dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/thrown-objects/test.ts +++ b/dev-packages/browser-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/dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/thrown-strings/test.ts b/dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/thrown-strings/test.ts index 326cf414f0f8..ff98e805f9aa 100644 --- a/dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/thrown-strings/test.ts +++ b/dev-packages/browser-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 index bb7b3b43c516..4c4388dab435 100644 --- 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(); } diff --git a/dev-packages/browser-integration-tests/suites/replay/sessionInactive/init.js b/dev-packages/browser-integration-tests/suites/replay/sessionInactive/init.js index 874580a83eff..3af8409da672 100644 --- a/dev-packages/browser-integration-tests/suites/replay/sessionInactive/init.js +++ b/dev-packages/browser-integration-tests/suites/replay/sessionInactive/init.js @@ -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 }; diff --git a/dev-packages/browser-integration-tests/suites/replay/throttleBreadcrumbs/test.ts b/dev-packages/browser-integration-tests/suites/replay/throttleBreadcrumbs/test.ts index 62bfda2b0a8c..b01a980ea961 100644 --- a/dev-packages/browser-integration-tests/suites/replay/throttleBreadcrumbs/test.ts +++ b/dev-packages/browser-integration-tests/suites/replay/throttleBreadcrumbs/test.ts @@ -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/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/error/test.ts b/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/error/test.ts index 581f0fd206dc..7eaab8b2872e 100644 --- a/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/error/test.ts +++ b/dev-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/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/interactions/subject.js b/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/interactions/subject.js index f9503ef6f261..0d4772ba535d 100644 --- a/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/interactions/subject.js +++ b/dev-packages/browser-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/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-animation-frame-disabled/assets/script.js b/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-animation-frame-disabled/assets/script.js index 9ac3d6fb33d2..195a094070be 100644 --- a/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-animation-frame-disabled/assets/script.js +++ b/dev-packages/browser-integration-tests/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/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-animation-frame-non-chromium/assets/script.js b/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-animation-frame-non-chromium/assets/script.js index 9ac3d6fb33d2..195a094070be 100644 --- a/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-animation-frame-non-chromium/assets/script.js +++ b/dev-packages/browser-integration-tests/suites/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/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-tasks-disabled/assets/script.js b/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-tasks-disabled/assets/script.js index 9ac3d6fb33d2..195a094070be 100644 --- a/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-tasks-disabled/assets/script.js +++ b/dev-packages/browser-integration-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/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-tasks-enabled/assets/script.js b/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-tasks-enabled/assets/script.js index 5a2aef02028d..b61592e05943 100644 --- a/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-tasks-enabled/assets/script.js +++ b/dev-packages/browser-integration-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/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-tasks-no-animation-frame/assets/script.js b/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-tasks-no-animation-frame/assets/script.js index 5a2aef02028d..b61592e05943 100644 --- a/dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-tasks-no-animation-frame/assets/script.js +++ b/dev-packages/browser-integration-tests/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/dev-packages/browser-integration-tests/suites/tracing/metrics/pageload-resource-spans/test.ts b/dev-packages/browser-integration-tests/suites/tracing/metrics/pageload-resource-spans/test.ts index e98cb5b3d9b2..fc74fa685bc7 100644 --- a/dev-packages/browser-integration-tests/suites/tracing/metrics/pageload-resource-spans/test.ts +++ b/dev-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(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 index 3163ef8d6a60..4e836bc57528 100644 --- 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( page: Page, diff --git a/dev-packages/browser-integration-tests/utils/replayEventTemplates.ts b/dev-packages/browser-integration-tests/utils/replayEventTemplates.ts index e711ea3bb0bb..51760544d868 100644 --- 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 index 088205fc4028..deaf37caad12 100644 --- 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'; 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 index bf3eca58a307..28b9d8cab43f 100644 --- 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 index bba794d9f027..b7279c9942a7 100644 --- 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 index 1e4526a891a3..36231f87a464 100644 --- 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 index 942e67ca4551..24ae70449ced 100644 --- 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 index bfe39fc89ee5..c096871cb755 100644 --- 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 index ca5ff99438fd..e519f33b988f 100644 --- 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 index 848802256853..1b8daa867e82 100644 --- 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 index f33a33170dc4..549f224d5cc4 100644 --- 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 index c6f328c9ff0f..3d612b9f94cc 100644 --- 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 index 73339494bbf8..29891e2d129e 100644 --- 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(), }); diff --git a/packages/astro/README.md b/packages/astro/README.md index bbcf09f03720..7ffa655a7f63 100644 --- a/packages/astro/README.md +++ b/packages/astro/README.md @@ -77,7 +77,7 @@ export const onRequest = sequence( The Sentry middleware enhances the data collected by Sentry on the server side by: -- Enabeling distributed tracing between client and server +- Enabling distributed tracing between client and server - Collecting performance spans for incoming requests - Enhancing captured errors with additional information diff --git a/packages/astro/src/integration/middleware/index.ts b/packages/astro/src/integration/middleware/index.ts index e220f8843070..695eed6c2237 100644 --- a/packages/astro/src/integration/middleware/index.ts +++ b/packages/astro/src/integration/middleware/index.ts @@ -7,7 +7,7 @@ import { handleRequest } from '../../server/middleware'; * to astro ^3.5.0 projects. * * It's not possible to pass options at this moment, so we'll call our middleware - * factory function with the default options. Users can deactiveate the automatic + * factory function with the default options. Users can deactivate the automatic * middleware registration in our integration and manually add it in their own * `/src/middleware.js` file. */ diff --git a/packages/astro/src/integration/types.ts b/packages/astro/src/integration/types.ts index 8020bcde7c76..ede2f49ff732 100644 --- a/packages/astro/src/integration/types.ts +++ b/packages/astro/src/integration/types.ts @@ -7,7 +7,7 @@ type SdkInitPaths = { * * If this option is not specified, the default location (`/sentry.client.config.(js|ts)`) * will be used to look up the config file. - * If there is no file at the default location either, the SDK will initalize with the options + * If there is no file at the default location either, the SDK will initialize with the options * specified in the `sentryAstro` integration or with default options. */ clientInitPath?: string; @@ -17,7 +17,7 @@ type SdkInitPaths = { * * If this option is not specified, the default location (`/sentry.server.config.(js|ts)`) * will be used to look up the config file. - * If there is no file at the default location either, the SDK will initalize with the options + * If there is no file at the default location either, the SDK will initialize with the options * specified in the `sentryAstro` integration or with default options. */ serverInitPath?: string; diff --git a/packages/browser-utils/src/instrument/dom.ts b/packages/browser-utils/src/instrument/dom.ts index d08350a48766..7172ca575225 100644 --- a/packages/browser-utils/src/instrument/dom.ts +++ b/packages/browser-utils/src/instrument/dom.ts @@ -93,7 +93,7 @@ export function instrumentDOM(): void { handlerForType.refCount++; } catch (e) { // Accessing dom properties is always fragile. - // Also allows us to skip `addEventListenrs` calls with no proper `this` context. + // Also allows us to skip `addEventListeners` calls with no proper `this` context. } } @@ -133,7 +133,7 @@ export function instrumentDOM(): void { } } catch (e) { // Accessing dom properties is always fragile. - // Also allows us to skip `addEventListenrs` calls with no proper `this` context. + // Also allows us to skip `addEventListeners` calls with no proper `this` context. } } diff --git a/packages/browser-utils/src/metrics/browserMetrics.ts b/packages/browser-utils/src/metrics/browserMetrics.ts index 5789bbe94513..59a58290c502 100644 --- a/packages/browser-utils/src/metrics/browserMetrics.ts +++ b/packages/browser-utils/src/metrics/browserMetrics.ts @@ -418,7 +418,7 @@ export function _addMeasureSpans( // spans created by the Next.js framework. // // To prevent this we will pin the start timestamp to the request start time - // This does make duration inaccruate, so if this does happen, we will add + // This does make duration inaccurate, so if this does happen, we will add // an attribute to the span const measureStartTimestamp = timeOrigin + Math.max(startTime, requestTime); const startTimeStamp = timeOrigin + startTime; diff --git a/packages/browser/src/integrations/breadcrumbs.ts b/packages/browser/src/integrations/breadcrumbs.ts index db30a48dda67..360668c4ba64 100644 --- a/packages/browser/src/integrations/breadcrumbs.ts +++ b/packages/browser/src/integrations/breadcrumbs.ts @@ -117,7 +117,7 @@ function _getSentryBreadcrumbHandler(client: Client): (event: SentryEvent) => vo } /** - * A HOC that creaes a function that creates breadcrumbs from DOM API calls. + * A HOC that creates a function that creates breadcrumbs from DOM API calls. * This is a HOC so that we get access to dom options in the closure. */ function _getDomBreadcrumbHandler( diff --git a/packages/browser/src/integrations/reportingobserver.ts b/packages/browser/src/integrations/reportingobserver.ts index 12a4039542b9..8212386493fb 100644 --- a/packages/browser/src/integrations/reportingobserver.ts +++ b/packages/browser/src/integrations/reportingobserver.ts @@ -65,7 +65,7 @@ const _reportingObserverIntegration = ((options: ReportingObserverOptions = {}) let details = 'No details available'; if (report.body) { - // Object.keys doesn't work on ReportBody, as all properties are inheirted + // Object.keys doesn't work on ReportBody, as all properties are inherited const plainBody: { [key: string]: unknown; } = {}; diff --git a/packages/browser/src/profiling/startProfileForSpan.ts b/packages/browser/src/profiling/startProfileForSpan.ts index 9dcdb81a9e8e..8d7d91fdddc9 100644 --- a/packages/browser/src/profiling/startProfileForSpan.ts +++ b/packages/browser/src/profiling/startProfileForSpan.ts @@ -41,7 +41,7 @@ export function startProfileForSpan(span: Span): void { // Whichever of the two (span.finish/timeout) is first to run, the profiling will be stopped and the gathered profile // will be processed when the original span is finished. Since onProfileHandler can be invoked multiple times in the // event of an error or user mistake (calling span.finish multiple times), it is important that the behavior of onProfileHandler - // is idempotent as we do not want any timings or profiles to be overriden by the last call to onProfileHandler. + // is idempotent as we do not want any timings or profiles to be overridden by the last call to onProfileHandler. // After the original finish method is called, the event will be reported through the integration and delegated to transport. const processedProfile: JSSelfProfile | null = null; diff --git a/packages/browser/src/profiling/utils.ts b/packages/browser/src/profiling/utils.ts index 48e87450b0e8..50272d9b5554 100644 --- a/packages/browser/src/profiling/utils.ts +++ b/packages/browser/src/profiling/utils.ts @@ -601,7 +601,7 @@ export function createProfilingEvent( // TODO (v8): We need to obtain profile ids in @sentry-internal/tracing, // but we don't have access to this map because importing this map would -// cause a circular dependancy. We need to resolve this in v8. +// cause a circular dependency. We need to resolve this in v8. const PROFILE_MAP: Map = new Map(); /** * diff --git a/packages/browser/src/tracing/request.ts b/packages/browser/src/tracing/request.ts index 4efe76a637a5..b41a0bf24cc6 100644 --- a/packages/browser/src/tracing/request.ts +++ b/packages/browser/src/tracing/request.ts @@ -138,7 +138,7 @@ export function instrumentOutgoingRequests(client: Client, _options?: Partial = {}; if (traceFetch) { - // Keeping track of http requests, whose body payloads resolved later than the intial resolved request + // Keeping track of http requests, whose body payloads resolved later than the initial resolved request // e.g. streaming using server sent events (SSE) client.addEventProcessor(event => { if (event.type === 'transaction' && event.spans) { diff --git a/packages/browser/src/transports/fetch.ts b/packages/browser/src/transports/fetch.ts index f9a7c258d4ff..d1ded0f7da84 100644 --- a/packages/browser/src/transports/fetch.ts +++ b/packages/browser/src/transports/fetch.ts @@ -29,7 +29,7 @@ export function makeFetchTransport( // Outgoing requests are usually cancelled when navigating to a different page, causing a "TypeError: Failed to // fetch" error and sending a "network_error" client-outcome - in Chrome, the request status shows "(cancelled)". // The `keepalive` flag keeps outgoing requests alive, even when switching pages. We want this since we're - // frequently sending events right before the user is switching pages (eg. whenfinishing navigation transactions). + // frequently sending events right before the user is switching pages (eg. when finishing navigation transactions). // Gotchas: // - `keepalive` isn't supported by Firefox // - As per spec (https://fetch.spec.whatwg.org/#http-network-or-cache-fetch): @@ -47,7 +47,7 @@ export function makeFetchTransport( } try { - // TODO: This may need a `suppresTracing` call in the future when we switch the browser SDK to OTEL + // TODO: This may need a `suppressTracing` call in the future when we switch the browser SDK to OTEL return nativeFetch(options.url, requestOptions).then(response => { pendingBodySize -= requestSize; pendingCount--; diff --git a/packages/browser/src/transports/offline.ts b/packages/browser/src/transports/offline.ts index a6755d8f43b5..7f5d22e466a9 100644 --- a/packages/browser/src/transports/offline.ts +++ b/packages/browser/src/transports/offline.ts @@ -8,7 +8,7 @@ import { makeFetchTransport } from './fetch'; // modified and simplified: https://github.com/jakearchibald/idb-keyval // // At commit: 0420a704fd6cbb4225429c536b1f61112d012fca -// Original licence: +// Original license: // Copyright 2016, Jake Archibald // diff --git a/packages/browser/test/loader.js b/packages/browser/test/loader.js index c80d5a1a4129..cfb749ae50a8 100644 --- a/packages/browser/test/loader.js +++ b/packages/browser/test/loader.js @@ -42,7 +42,7 @@ lazy ) { // We only want to lazy inject/load the sdk bundle if - // an error or promise rejection occured + // an error or promise rejection occurred // OR someone called `capture...` on the SDK injectSdk(onLoadCallbacks); } @@ -127,7 +127,7 @@ calledSentry = true; var call = data[i]; if (initAlreadyCalled === false && call.f !== 'init') { - // First call always has to be init, this is a conveniece for the user so call to init is optional + // First call always has to be init, this is a convenience for the user so call to init is optional SDK.init(); } initAlreadyCalled = true; diff --git a/packages/bun/test/sdk.test.ts b/packages/bun/test/sdk.test.ts index acef85f55eb5..a548cc2614c7 100644 --- a/packages/bun/test/sdk.test.ts +++ b/packages/bun/test/sdk.test.ts @@ -9,6 +9,6 @@ test("calling init shouldn't fail", () => { expect(true).toBe(true); }); -test('shuold return client from init', () => { +test('should return client from init', () => { expect(init({})).not.toBeUndefined(); }); diff --git a/packages/cloudflare/package.json b/packages/cloudflare/package.json index d5c4adbfb195..77b9bd14d5be 100644 --- a/packages/cloudflare/package.json +++ b/packages/cloudflare/package.json @@ -1,7 +1,7 @@ { "name": "@sentry/cloudflare", "version": "8.36.0", - "description": "Offical Sentry SDK for Cloudflare Workers and Pages", + "description": "Official Sentry SDK for Cloudflare Workers and Pages", "repository": "git://github.com/getsentry/sentry-javascript.git", "homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/cloudflare", "author": "Sentry", diff --git a/packages/cloudflare/src/async.ts b/packages/cloudflare/src/async.ts index 9662fc340a7e..c287a04474eb 100644 --- a/packages/cloudflare/src/async.ts +++ b/packages/cloudflare/src/async.ts @@ -8,7 +8,7 @@ import { AsyncLocalStorage } from 'node:async_hooks'; /** * Sets the async context strategy to use AsyncLocalStorage. * - * AsyncLocalStorage is only avalaible in the cloudflare workers runtime if you set + * AsyncLocalStorage is only available in the cloudflare workers runtime if you set * compatibility_flags = ["nodejs_compat"] or compatibility_flags = ["nodejs_als"] */ export function setAsyncLocalStorageAsyncContextStrategy(): void { diff --git a/packages/core/src/baseclient.ts b/packages/core/src/baseclient.ts index 6071f644c37c..db79df0d1c16 100644 --- a/packages/core/src/baseclient.ts +++ b/packages/core/src/baseclient.ts @@ -472,7 +472,7 @@ export abstract class BaseClient implements Client { public on(hook: string, callback: unknown): () => void { const hooks = (this._hooks[hook] = this._hooks[hook] || []); - // @ts-expect-error We assue the types are correct + // @ts-expect-error We assume the types are correct hooks.push(callback); // This function returns a callback execution handler that, when invoked, @@ -480,7 +480,7 @@ export abstract class BaseClient implements Client { // need to be unregistered to prevent self-referencing in callback closures, // ensuring proper garbage collection. return () => { - // @ts-expect-error We assue the types are correct + // @ts-expect-error We assume the types are correct const cbIndex = hooks.indexOf(callback); if (cbIndex > -1) { hooks.splice(cbIndex, 1); diff --git a/packages/core/src/currentScopes.ts b/packages/core/src/currentScopes.ts index 9472d48b71bf..317042c08054 100644 --- a/packages/core/src/currentScopes.ts +++ b/packages/core/src/currentScopes.ts @@ -16,7 +16,7 @@ export function getCurrentScope(): Scope { /** * Get the currently active isolation scope. - * The isolation scope is active for the current exection context. + * The isolation scope is active for the current execution context. */ export function getIsolationScope(): Scope { const carrier = getMainCarrier(); diff --git a/packages/core/src/envelope.ts b/packages/core/src/envelope.ts index d8b8c03e888b..efe55ab685ac 100644 --- a/packages/core/src/envelope.ts +++ b/packages/core/src/envelope.ts @@ -76,7 +76,7 @@ export function createEventEnvelope( /* Note: Due to TS, event.type may be `replay_event`, theoretically. In practice, we never call `createEventEnvelope` with `replay_event` type, - and we'd have to adjut a looot of types to make this work properly. + and we'd have to adjust a looot of types to make this work properly. We want to avoid casting this around, as that could lead to bugs (e.g. when we add another type) So the safe choice is to really guard against the replay_event type here. */ diff --git a/packages/core/src/fetch.ts b/packages/core/src/fetch.ts index 26a993ff0e12..29ff5d074cd0 100644 --- a/packages/core/src/fetch.ts +++ b/packages/core/src/fetch.ts @@ -23,7 +23,7 @@ import { getActiveSpan, spanToTraceHeader } from './utils/spanUtils'; type PolymorphicRequestHeaders = | Record | Array<[string, string]> - // the below is not preicsely the Header type used in Request, but it'll pass duck-typing + // the below is not precisely the Header type used in Request, but it'll pass duck-typing | { // eslint-disable-next-line @typescript-eslint/no-explicit-any [key: string]: any; diff --git a/packages/core/src/semanticAttributes.ts b/packages/core/src/semanticAttributes.ts index 2d24c52a15ea..2896bd81f93f 100644 --- a/packages/core/src/semanticAttributes.ts +++ b/packages/core/src/semanticAttributes.ts @@ -30,7 +30,7 @@ export const SEMANTIC_ATTRIBUTE_SENTRY_MEASUREMENT_UNIT = 'sentry.measurement_un export const SEMANTIC_ATTRIBUTE_SENTRY_MEASUREMENT_VALUE = 'sentry.measurement_value'; /** - * The id of the profile that this span occured in. + * The id of the profile that this span occurred in. */ export const SEMANTIC_ATTRIBUTE_PROFILE_ID = 'sentry.profile_id'; diff --git a/packages/core/src/tracing/errors.ts b/packages/core/src/tracing/errors.ts index 835038175331..a611dd1d80e4 100644 --- a/packages/core/src/tracing/errors.ts +++ b/packages/core/src/tracing/errors.ts @@ -36,7 +36,7 @@ function errorCallback(): void { const rootSpan = activeSpan && getRootSpan(activeSpan); if (rootSpan) { const message = 'internal_error'; - DEBUG_BUILD && logger.log(`[Tracing] Root span: ${message} -> Global error occured`); + DEBUG_BUILD && logger.log(`[Tracing] Root span: ${message} -> Global error occurred`); rootSpan.setStatus({ code: SPAN_STATUS_ERROR, message }); } } diff --git a/packages/core/src/tracing/idleSpan.ts b/packages/core/src/tracing/idleSpan.ts index c3d66a4b7593..3a9b705dd449 100644 --- a/packages/core/src/tracing/idleSpan.ts +++ b/packages/core/src/tracing/idleSpan.ts @@ -32,7 +32,7 @@ const FINISH_REASON_CANCELLED = 'cancelled'; // unused const FINISH_REASON_DOCUMENT_HIDDEN = 'documentHidden'; -// unusued in this file, but used in BrowserTracing +// unused in this file, but used in BrowserTracing const FINISH_REASON_INTERRUPTED = 'interactionInterrupted'; type IdleSpanFinishReason = diff --git a/packages/core/src/tracing/sentryNonRecordingSpan.ts b/packages/core/src/tracing/sentryNonRecordingSpan.ts index 867b5684d1da..c39730faf9c5 100644 --- a/packages/core/src/tracing/sentryNonRecordingSpan.ts +++ b/packages/core/src/tracing/sentryNonRecordingSpan.ts @@ -71,7 +71,7 @@ export class SentryNonRecordingSpan implements Span { /** * This should generally not be used, - * but we need it for being comliant with the OTEL Span interface. + * but we need it for being compliant with the OTEL Span interface. * * @hidden * @internal @@ -82,7 +82,7 @@ export class SentryNonRecordingSpan implements Span { /** * This should generally not be used, - * but we need it for being comliant with the OTEL Span interface. + * but we need it for being compliant with the OTEL Span interface. * * @hidden * @internal @@ -93,7 +93,7 @@ export class SentryNonRecordingSpan implements Span { /** * This should generally not be used, - * but we need it for being comliant with the OTEL Span interface. + * but we need it for being compliant with the OTEL Span interface. * * @hidden * @internal diff --git a/packages/core/src/utils/parameterize.ts b/packages/core/src/utils/parameterize.ts index 5783a61fa3c9..1ad880618a48 100644 --- a/packages/core/src/utils/parameterize.ts +++ b/packages/core/src/utils/parameterize.ts @@ -1,7 +1,7 @@ import type { ParameterizedString } from '@sentry/types'; /** - * Tagged template function which returns paramaterized representation of the message + * Tagged template function which returns parameterized representation of the message * For example: parameterize`This is a log statement with ${x} and ${y} params`, would return: * "__sentry_template_string__": 'This is a log statement with %s and %s params', * "__sentry_template_values__": ['first', 'second'] diff --git a/packages/core/test/lib/integration.test.ts b/packages/core/test/lib/integration.test.ts index 18b9d1791bc8..3a1a1ae19768 100644 --- a/packages/core/test/lib/integration.test.ts +++ b/packages/core/test/lib/integration.test.ts @@ -35,7 +35,7 @@ type TestCase = [ string, // test name Options['defaultIntegrations'], // default integrations Options['integrations'], // user-provided integrations - Array, // expected resulst + Array, // expected results ]; describe('getIntegrationsToSetup', () => { diff --git a/packages/core/test/lib/tracing/idleSpan.test.ts b/packages/core/test/lib/tracing/idleSpan.test.ts index ff45938646a1..7a59fc790f0a 100644 --- a/packages/core/test/lib/tracing/idleSpan.test.ts +++ b/packages/core/test/lib/tracing/idleSpan.test.ts @@ -115,7 +115,7 @@ describe('startIdleSpan', () => { setCurrentClient(client); client.init(); - // We want to accomodate a bit of drift there, so we ensure this starts earlier... + // We want to accommodate a bit of drift there, so we ensure this starts earlier... const baseTimeInSeconds = Math.floor(Date.now() / 1000) - 9999; const beforeSpanEnd = jest.fn((span: Span) => { @@ -169,7 +169,7 @@ describe('startIdleSpan', () => { setCurrentClient(client); client.init(); - // We want to accomodate a bit of drift there, so we ensure this starts earlier... + // We want to accommodate a bit of drift there, so we ensure this starts earlier... const baseTimeInSeconds = Math.floor(Date.now() / 1000) - 9999; const idleSpan = startIdleSpan({ name: 'idle span', startTime: baseTimeInSeconds }); @@ -249,7 +249,7 @@ describe('startIdleSpan', () => { setCurrentClient(client); client.init(); - // We want to accomodate a bit of drift there, so we ensure this starts earlier... + // We want to accommodate a bit of drift there, so we ensure this starts earlier... const finalTimeout = 99_999; const baseTimeInSeconds = Math.floor(Date.now() / 1000) - 9999; diff --git a/packages/core/test/lib/utils/handleCallbackErrors.test.ts b/packages/core/test/lib/utils/handleCallbackErrors.test.ts index 3c6bb1e19302..08c116e50ceb 100644 --- a/packages/core/test/lib/utils/handleCallbackErrors.test.ts +++ b/packages/core/test/lib/utils/handleCallbackErrors.test.ts @@ -69,7 +69,7 @@ describe('handleCallbackErrors', () => { }); describe('onFinally', () => { - it('triggers after successfuly sync callback', () => { + it('triggers after successful sync callback', () => { const onError = jest.fn(); const onFinally = jest.fn(); diff --git a/packages/core/test/lib/utils/spanUtils.test.ts b/packages/core/test/lib/utils/spanUtils.test.ts index 2a4850947e80..779e75006c86 100644 --- a/packages/core/test/lib/utils/spanUtils.test.ts +++ b/packages/core/test/lib/utils/spanUtils.test.ts @@ -238,7 +238,7 @@ describe('getRootSpan', () => { startSpan({ name: 'inner2' }, inner2 => { expect(getRootSpan(inner2)).toBe(root); - const inactiveSpan = startInactiveSpan({ name: 'inactived' }); + const inactiveSpan = startInactiveSpan({ name: 'inactive' }); expect(getRootSpan(inactiveSpan)).toBe(root); }); }); diff --git a/packages/eslint-config-sdk/src/base.js b/packages/eslint-config-sdk/src/base.js index 6daa79eaeed8..525b24b4a334 100644 --- a/packages/eslint-config-sdk/src/base.js +++ b/packages/eslint-config-sdk/src/base.js @@ -40,7 +40,7 @@ module.exports = { // Although for most codebases inferencing the return type is fine, we explicitly ask to annotate // all functions with a return type. This is so that intent is as clear as possible. We are guarding against - // cases where you accidently refactor a function's return type to be the wrong type. + // cases where you accidentally refactor a function's return type to be the wrong type. '@typescript-eslint/explicit-function-return-type': ['error', { allowExpressions: true }], // Consistent ordering of fields, methods and constructors for classes should be enforced @@ -131,7 +131,7 @@ module.exports = { ], // We want to prevent optional chaining & nullish coalescing usage in our files - // to prevent uncessary bundle size. Turned off in tests. + // to prevent unnecessary bundle size. Turned off in tests. '@sentry-internal/sdk/no-optional-chaining': 'error', '@sentry-internal/sdk/no-nullish-coalescing': 'error', @@ -258,7 +258,7 @@ module.exports = { quotes: ['error', 'single', { avoidEscape: true }], - // Remove uncessary usages of async await to prevent extra micro-tasks + // Remove unnecessary usages of async await to prevent extra micro-tasks 'no-return-await': 'error', }, }; diff --git a/packages/feedback/README.md b/packages/feedback/README.md index 336e74da6593..d2ed35cc8c7d 100644 --- a/packages/feedback/README.md +++ b/packages/feedback/README.md @@ -15,7 +15,7 @@ To view Feedback in Sentry, your ## Installation -Please read the [offical integration documentation](https://docs.sentry.io/platforms/javascript/user-feedback/) for +Please read the [official integration documentation](https://docs.sentry.io/platforms/javascript/user-feedback/) for installation instructions. ## Configuration diff --git a/packages/feedback/src/modal/components/Form.tsx b/packages/feedback/src/modal/components/Form.tsx index cc6f1afd32f6..d6cc94ec7f60 100644 --- a/packages/feedback/src/modal/components/Form.tsx +++ b/packages/feedback/src/modal/components/Form.tsx @@ -61,7 +61,7 @@ export function Form({ submitButtonLabel, isRequiredLabel, } = options; - // TODO: set a ref on the form, and whenever an input changes call proceessForm() and setError() + // TODO: set a ref on the form, and whenever an input changes call processForm() and setError() const [error, setError] = useState(null); const [showScreenshotInput, setShowScreenshotInput] = useState(false); diff --git a/packages/nextjs/rollup.npm.config.mjs b/packages/nextjs/rollup.npm.config.mjs index afe41659238f..7fbacde6fe3c 100644 --- a/packages/nextjs/rollup.npm.config.mjs +++ b/packages/nextjs/rollup.npm.config.mjs @@ -14,7 +14,7 @@ export default [ 'src/config/index.ts', ], - // prevent this internal nextjs code from ending up in our built package (this doesn't happen automatially because + // prevent this internal nextjs code from ending up in our built package (this doesn't happen automatically because // the name doesn't match an SDK dependency) packageSpecificConfig: { external: ['next/router', 'next/constants', 'next/headers', 'stacktrace-parser'], diff --git a/packages/nextjs/src/client/routing/appRouterRoutingInstrumentation.ts b/packages/nextjs/src/client/routing/appRouterRoutingInstrumentation.ts index 741849c481ab..c44ef444fdf7 100644 --- a/packages/nextjs/src/client/routing/appRouterRoutingInstrumentation.ts +++ b/packages/nextjs/src/client/routing/appRouterRoutingInstrumentation.ts @@ -36,7 +36,7 @@ const GLOBAL_OBJ_WITH_NEXT_ROUTER = GLOBAL_OBJ as typeof GLOBAL_OBJ & { nd?: { router?: NextRouter; }; - // Avalable from 13.4.4-canary.4 - https://github.com/vercel/next.js/pull/50210 + // Available from 13.4.4-canary.4 - https://github.com/vercel/next.js/pull/50210 next?: { router?: NextRouter; }; diff --git a/packages/nextjs/src/config/loaders/prefixLoader.ts b/packages/nextjs/src/config/loaders/prefixLoader.ts index a9ca78667b53..29920a7047be 100644 --- a/packages/nextjs/src/config/loaders/prefixLoader.ts +++ b/packages/nextjs/src/config/loaders/prefixLoader.ts @@ -24,7 +24,7 @@ export default function prefixLoader(this: LoaderThis, userCode: const { templatePrefix, replacements } = 'getOptions' in this ? this.getOptions() : this.query; const templatePath = path.resolve(__dirname, `../templates/${templatePrefix}PrefixLoaderTemplate.js`); - // make sure the template is included when runing `webpack watch` + // make sure the template is included when running `webpack watch` this.addDependency(templatePath); // Fill in placeholders diff --git a/packages/nextjs/src/config/loaders/wrappingLoader.ts b/packages/nextjs/src/config/loaders/wrappingLoader.ts index e298a459d16e..d18d0efd7634 100644 --- a/packages/nextjs/src/config/loaders/wrappingLoader.ts +++ b/packages/nextjs/src/config/loaders/wrappingLoader.ts @@ -82,7 +82,7 @@ export default function wrappingLoader( // Get the parameterized route name from this page's filepath const parameterizedPagesRoute = path - // Get the path of the file insde of the pages directory + // Get the path of the file inside of the pages directory .relative(pagesDir, this.resourcePath) // Replace all backslashes with forward slashes (windows) .replace(/\\/g, '/') @@ -124,7 +124,7 @@ export default function wrappingLoader( // Get the parameterized route name from this page's filepath const parameterizedPagesRoute = path - // Get the path of the file insde of the app directory + // Get the path of the file inside of the app directory .relative(appDir, this.resourcePath) // Replace all backslashes with forward slashes (windows) .replace(/\\/g, '/') @@ -281,7 +281,7 @@ async function wrapUserCode( } else if (id === WRAPPING_TARGET_MODULE_NAME) { return { code: userModuleCode, - map: userModuleSourceMap, // give rollup acces to original user module source map + map: userModuleSourceMap, // give rollup access to original user module source map }; } else { return null; @@ -342,7 +342,7 @@ async function wrapUserCode( // This is why we want to avoid unnecessarily creating default exports, even if they're just `undefined`. // For this reason we try to bundle/wrap the user code once including a re-export of `default`. // If the user code didn't have a default export, rollup will throw. - // We then try bundling/wrapping agian, but without including a re-export of `default`. + // We then try bundling/wrapping again, but without including a re-export of `default`. let rollupBuild; try { rollupBuild = await wrap(true); @@ -356,7 +356,7 @@ async function wrapUserCode( const finalBundle = await rollupBuild.generate({ format: 'esm', - sourcemap: 'hidden', // put source map data in the bundle but don't generate a source map commment in the output + sourcemap: 'hidden', // put source map data in the bundle but don't generate a source map comment in the output }); // The module at index 0 is always the entrypoint, which in this case is the proxy module. diff --git a/packages/nextjs/src/config/templates/apiWrapperTemplate.ts b/packages/nextjs/src/config/templates/apiWrapperTemplate.ts index 80b9a4f51d60..da03a0c413d4 100644 --- a/packages/nextjs/src/config/templates/apiWrapperTemplate.ts +++ b/packages/nextjs/src/config/templates/apiWrapperTemplate.ts @@ -25,7 +25,7 @@ type NextApiModule = ( const userApiModule = origModule as NextApiModule; // Default to undefined. It's possible for Next.js users to not define any exports/handlers in an API route. If that is -// the case Next.js wil crash during runtime but the Sentry SDK should definitely not crash so we need tohandle it. +// the case Next.js will crash during runtime but the Sentry SDK should definitely not crash so we need to handle it. let userProvidedHandler = undefined; if ('default' in userApiModule && typeof userApiModule.default === 'function') { @@ -65,6 +65,6 @@ if (wrappedHandler) { export default wrappedHandler; // Re-export anything exported by the page module we're wrapping. When processing this code, Rollup is smart enough to -// not include anything whose name matchs something we've explicitly exported above. +// not include anything whose name matches something we've explicitly exported above. // @ts-expect-error See above export * from '__SENTRY_WRAPPING_TARGET_FILE__'; diff --git a/packages/nextjs/src/config/templates/pageWrapperTemplate.ts b/packages/nextjs/src/config/templates/pageWrapperTemplate.ts index 7ac89ed1931c..2c3f22a597c7 100644 --- a/packages/nextjs/src/config/templates/pageWrapperTemplate.ts +++ b/packages/nextjs/src/config/templates/pageWrapperTemplate.ts @@ -50,6 +50,6 @@ export const getServerSideProps = export default pageComponent ? Sentry.wrapPageComponentWithSentry(pageComponent as unknown) : pageComponent; // Re-export anything exported by the page module we're wrapping. When processing this code, Rollup is smart enough to -// not include anything whose name matchs something we've explicitly exported above. +// not include anything whose name matches something we've explicitly exported above. // @ts-expect-error See above export * from '__SENTRY_WRAPPING_TARGET_FILE__'; diff --git a/packages/nextjs/src/config/templates/serverComponentWrapperTemplate.ts b/packages/nextjs/src/config/templates/serverComponentWrapperTemplate.ts index 364affc08058..d569715e74fc 100644 --- a/packages/nextjs/src/config/templates/serverComponentWrapperTemplate.ts +++ b/packages/nextjs/src/config/templates/serverComponentWrapperTemplate.ts @@ -99,7 +99,7 @@ export const generateViewport = serverComponentModule.generateViewport : undefined; // Re-export anything exported by the page module we're wrapping. When processing this code, Rollup is smart enough to -// not include anything whose name matchs something we've explicitly exported above. +// not include anything whose name matches something we've explicitly exported above. // @ts-expect-error See above export * from '__SENTRY_WRAPPING_TARGET_FILE__'; diff --git a/packages/nextjs/src/config/webpack.ts b/packages/nextjs/src/config/webpack.ts index 9656a74c4efe..79bd230a3ee7 100644 --- a/packages/nextjs/src/config/webpack.ts +++ b/packages/nextjs/src/config/webpack.ts @@ -683,7 +683,7 @@ function addOtelWarningIgnoreRule(newConfig: WebpackConfigObjectWithModuleRules) const ignoreRules = [ // Inspired by @matmannion: https://github.com/getsentry/sentry-javascript/issues/12077#issuecomment-2180307072 (warning, compilation) => { - // This is wapped in try-catch because we are vendoring types for this hook and we can't be 100% sure that we are accessing API that is there + // This is wrapped in try-catch because we are vendoring types for this hook and we can't be 100% sure that we are accessing API that is there try { if (!warning.module) { return false; diff --git a/packages/nextjs/src/index.types.ts b/packages/nextjs/src/index.types.ts index 8f09999ad738..be7bb441f053 100644 --- a/packages/nextjs/src/index.types.ts +++ b/packages/nextjs/src/index.types.ts @@ -1,7 +1,7 @@ /* eslint-disable @typescript-eslint/no-explicit-any */ // We export everything from both the client part of the SDK and from the server part. Some of the exports collide, -// which is not allowed, unless we redifine the colliding exports in this file - which we do below. +// which is not allowed, unless we redefine the colliding exports in this file - which we do below. export * from './config'; export * from './client'; export * from './server'; diff --git a/packages/nextjs/test/serverSdk.test.ts b/packages/nextjs/test/serverSdk.test.ts index 27230874d457..1129bcdbbf2b 100644 --- a/packages/nextjs/test/serverSdk.test.ts +++ b/packages/nextjs/test/serverSdk.test.ts @@ -54,7 +54,7 @@ describe('Server init()', () => { // Integrations are tested separately, and we can't be more specific here without depending on the order in // which integrations appear in the array, which we can't guarantee. // - // TODO: If we upgrde to Jest 28+, we can follow Jest's example matcher and create an + // TODO: If we upgrade to Jest 28+, we can follow Jest's example matcher and create an // `expect.ArrayContainingInAnyOrder`. See // https://github.com/facebook/jest/blob/main/examples/expect-extend/toBeWithinRange.ts. defaultIntegrations: expect.any(Array), diff --git a/packages/node/src/integrations/http/index.ts b/packages/node/src/integrations/http/index.ts index 975503956f21..224bd56485a1 100644 --- a/packages/node/src/integrations/http/index.ts +++ b/packages/node/src/integrations/http/index.ts @@ -75,7 +75,7 @@ interface HttpOptions { ) => void; /** - * You can pass any configuration through to the underlying instrumention. + * You can pass any configuration through to the underlying instrumentation. * Note that there are no semver guarantees for this! */ _experimentalConfig?: ConstructorParameters[0]; diff --git a/packages/node/src/integrations/tracing/prisma.ts b/packages/node/src/integrations/tracing/prisma.ts index e5d9e61a0229..c9e7acafb09d 100644 --- a/packages/node/src/integrations/tracing/prisma.ts +++ b/packages/node/src/integrations/tracing/prisma.ts @@ -41,7 +41,7 @@ const _prismaIntegration = (() => { * Prisma integration * * Capture tracing data for prisma. - * Note: This requieres to set: + * Note: This requires to set: * previewFeatures = ["tracing"] * For the prisma client. * See https://www.prisma.io/docs/concepts/components/prisma-client/opentelemetry-tracing for more details. diff --git a/packages/node/src/types.ts b/packages/node/src/types.ts index f6b08a394f8c..7235e2057c34 100644 --- a/packages/node/src/types.ts +++ b/packages/node/src/types.ts @@ -158,7 +158,7 @@ export interface CurrentScopes { * so in these cases we type this as `AbstractSpan` which could be either a regular `Span` or a `ReadableSpan`. * You'll have to make sur to check revelant fields before accessing them. * - * Note that technically, the `Span` exported from `@opentelemwetry/sdk-trace-base` matches this, + * Note that technically, the `Span` exported from `@opentelemetry/sdk-trace-base` matches this, * but we cannot be 100% sure that we are actually getting such a span, so this type is more defensive. */ export type AbstractSpan = WriteableSpan | ReadableSpan | Span; diff --git a/packages/node/test/integration/transactions.test.ts b/packages/node/test/integration/transactions.test.ts index 048496f363b4..5c77b81e6994 100644 --- a/packages/node/test/integration/transactions.test.ts +++ b/packages/node/test/integration/transactions.test.ts @@ -595,7 +595,7 @@ describe('Integration | Transactions', () => { jest.advanceTimersByTime(1); - // Child-spans have been added to the exporter, but they are pending since they are waiting for their parant + // Child-spans have been added to the exporter, but they are pending since they are waiting for their parent expect(exporter['_finishedSpans'].length).toBe(2); expect(beforeSendTransaction).toHaveBeenCalledTimes(0); diff --git a/packages/node/test/sdk/init.test.ts b/packages/node/test/sdk/init.test.ts index 1f904ca528f9..10a7a56ae8bd 100644 --- a/packages/node/test/sdk/init.test.ts +++ b/packages/node/test/sdk/init.test.ts @@ -143,7 +143,7 @@ describe('init()', () => { }); }); - it('returns intiated client', () => { + it('returns initialized client', () => { const client = init({ dsn: PUBLIC_DSN, skipOpenTelemetrySetup: true }); expect(client).toBeInstanceOf(NodeClient); diff --git a/packages/opentelemetry/src/spanExporter.ts b/packages/opentelemetry/src/spanExporter.ts index 5ba28f8b1607..a9fa207ade6b 100644 --- a/packages/opentelemetry/src/spanExporter.ts +++ b/packages/opentelemetry/src/spanExporter.ts @@ -209,7 +209,7 @@ function createTransactionForOtelSpan(span: ReadableSpan): TransactionEvent { const parentSpanIdFromTraceState = span.spanContext().traceState?.get(SENTRY_TRACE_STATE_PARENT_SPAN_ID); - // If parentSpanIdFromTraceState is defined at all, we want it to take presedence + // If parentSpanIdFromTraceState is defined at all, we want it to take precedence // In that case, an empty string should be interpreted as "no parent span id", // even if `span.parentSpanId` is set // this is the case when we are starting a new trace, where we have a virtual span based on the propagationContext @@ -337,7 +337,7 @@ function getSpanData(span: ReadableSpan): { } /** - * Remove custom `sentry.` attribtues we do not need to send. + * Remove custom `sentry.` attributes we do not need to send. * These are more carrier attributes we use inside of the SDK, we do not need to send them to the API. */ function removeSentryAttributes(data: Record): Record { diff --git a/packages/opentelemetry/src/types.ts b/packages/opentelemetry/src/types.ts index 30dd025a3116..341fbea88d38 100644 --- a/packages/opentelemetry/src/types.ts +++ b/packages/opentelemetry/src/types.ts @@ -18,7 +18,7 @@ export interface OpenTelemetrySpanContext extends StartSpanOptions { * so in these cases we type this as `AbstractSpan` which could be either a regular `Span` or a `ReadableSpan`. * You'll have to make sure to check relevant fields before accessing them. * - * Note that technically, the `Span` exported from `@opentelemwetry/sdk-trace-base` matches this, + * Note that technically, the `Span` exported from `@opentelemetry/sdk-trace-base` matches this, * but we cannot be 100% sure that we are actually getting such a span, so this type is more defensive. */ export type AbstractSpan = WriteableSpan | ReadableSpan | Span; diff --git a/packages/opentelemetry/src/utils/isSentryRequest.ts b/packages/opentelemetry/src/utils/isSentryRequest.ts index bbfcefc59d31..c910ec1607bb 100644 --- a/packages/opentelemetry/src/utils/isSentryRequest.ts +++ b/packages/opentelemetry/src/utils/isSentryRequest.ts @@ -6,7 +6,7 @@ import { spanHasAttributes } from './spanTypes'; /** * - * @param otelSpan Checks wheter a given OTEL Span is an http request to sentry. + * @param otelSpan Checks whether a given OTEL Span is an http request to sentry. * @returns boolean */ export function isSentryRequestSpan(span: AbstractSpan): boolean { diff --git a/packages/opentelemetry/test/asyncContextStrategy.test.ts b/packages/opentelemetry/test/asyncContextStrategy.test.ts index f3b664fcaf44..915d33671e68 100644 --- a/packages/opentelemetry/test/asyncContextStrategy.test.ts +++ b/packages/opentelemetry/test/asyncContextStrategy.test.ts @@ -85,7 +85,7 @@ describe('asyncContextStrategy', () => { const initialScope = getCurrentScope(); const initialIsolationScope = getIsolationScope(); - async function asycnSetExtra(scope: Scope, key: string, value: string): Promise { + async function asyncSetExtra(scope: Scope, key: string, value: string): Promise { await new Promise(resolve => setTimeout(resolve, 1)); scope.setExtra(key, value); } @@ -103,8 +103,8 @@ describe('asyncContextStrategy', () => { expect(scope1.getScopeData()).toEqual(initialScope.getScopeData()); expect(isolationScope1.getScopeData()).toEqual(initialIsolationScope.getScopeData()); - await asycnSetExtra(scope1, 'b', 'b'); - await asycnSetExtra(isolationScope1, 'bb', 'bb'); + await asyncSetExtra(scope1, 'b', 'b'); + await asyncSetExtra(isolationScope1, 'bb', 'bb'); await withScope(async () => { const scope2 = getCurrentScope(); @@ -115,7 +115,7 @@ describe('asyncContextStrategy', () => { expect(scope2.getScopeData()).toEqual(scope1.getScopeData()); - await asycnSetExtra(scope2, 'c', 'c'); + await asyncSetExtra(scope2, 'c', 'c'); expect(scope2.getScopeData().extra).toEqual({ a: 'a', @@ -217,7 +217,7 @@ describe('asyncContextStrategy', () => { const initialScope = getCurrentScope(); const initialIsolationScope = getIsolationScope(); - async function asycnSetExtra(scope: Scope, key: string, value: string): Promise { + async function asyncSetExtra(scope: Scope, key: string, value: string): Promise { await new Promise(resolve => setTimeout(resolve, 1)); scope.setExtra(key, value); } @@ -235,8 +235,8 @@ describe('asyncContextStrategy', () => { expect(scope1.getScopeData()).toEqual(initialScope.getScopeData()); expect(isolationScope1.getScopeData()).toEqual(initialIsolationScope.getScopeData()); - await asycnSetExtra(scope1, 'b', 'b'); - await asycnSetExtra(isolationScope1, 'bb', 'bb'); + await asyncSetExtra(scope1, 'b', 'b'); + await asyncSetExtra(isolationScope1, 'bb', 'bb'); await withScope(async () => { const scope2 = getCurrentScope(); @@ -247,7 +247,7 @@ describe('asyncContextStrategy', () => { expect(scope2.getScopeData()).toEqual(scope1.getScopeData()); - await asycnSetExtra(scope2, 'c', 'c'); + await asyncSetExtra(scope2, 'c', 'c'); expect(scope2.getScopeData().extra).toEqual({ a: 'a', diff --git a/packages/opentelemetry/test/integration/transactions.test.ts b/packages/opentelemetry/test/integration/transactions.test.ts index b8e7713cb4ca..bab3f03d1f64 100644 --- a/packages/opentelemetry/test/integration/transactions.test.ts +++ b/packages/opentelemetry/test/integration/transactions.test.ts @@ -476,7 +476,7 @@ describe('Integration | Transactions', () => { await new Promise(resolve => setTimeout(resolve, 10 * 60 * 1000)); }); - // Child-spans have been added to the exporter, but they are pending since they are waiting for their parant + // Child-spans have been added to the exporter, but they are pending since they are waiting for their parent expect(exporter['_finishedSpans'].length).toBe(2); expect(beforeSendTransaction).toHaveBeenCalledTimes(0); diff --git a/packages/opentelemetry/test/propagator.test.ts b/packages/opentelemetry/test/propagator.test.ts index 16848352239a..d3b9483674a1 100644 --- a/packages/opentelemetry/test/propagator.test.ts +++ b/packages/opentelemetry/test/propagator.test.ts @@ -533,7 +533,7 @@ describe('SentryPropagator', () => { ); }); - it('should NOT set baggage and sentry-trace header if instrumentation is supressed', () => { + it('should NOT set baggage and sentry-trace header if instrumentation is suppressed', () => { const spanContext = { traceId: 'd4cda95b652f4a1592b449d5929fda1b', spanId: '6e0c63257de34c92', diff --git a/packages/opentelemetry/test/utils/parseSpanDescription.test.ts b/packages/opentelemetry/test/utils/parseSpanDescription.test.ts index 91de028fdcbe..c44645c62888 100644 --- a/packages/opentelemetry/test/utils/parseSpanDescription.test.ts +++ b/packages/opentelemetry/test/utils/parseSpanDescription.test.ts @@ -142,7 +142,7 @@ describe('parseSpanDescription', () => { describe('descriptionForHttpMethod', () => { it.each([ [ - 'works withhout attributes', + 'works without attributes', 'GET', {}, 'test name', diff --git a/packages/profiling-node/README.md b/packages/profiling-node/README.md index 4357e23bb194..890022ae2e88 100644 --- a/packages/profiling-node/README.md +++ b/packages/profiling-node/README.md @@ -305,7 +305,7 @@ In that case it is possible that the function name may end up being reported to No. All instances of the profiler are scoped per thread In practice, this means that starting a transaction on thread A and delegating work to thread B will only result in sample stacks being collected from thread A. That said, nothing -should prevent you from starting a transaction on thread B concurrently which will result in two independant profiles +should prevent you from starting a transaction on thread B concurrently which will result in two independent profiles being sent to the Sentry backend. We currently do not do any correlation between such transactions, but we would be open to exploring the possibilities. Please file an issue if you have suggestions or specific use-cases in mind. diff --git a/packages/profiling-node/bindings/cpu_profiler.cc b/packages/profiling-node/bindings/cpu_profiler.cc index ad3ca8079e00..d51a1e747e93 100644 --- a/packages/profiling-node/bindings/cpu_profiler.cc +++ b/packages/profiling-node/bindings/cpu_profiler.cc @@ -351,7 +351,7 @@ v8::CpuProfile *SentryProfile::Stop(Profiler *profiler) { v8::NewStringType::kNormal) .ToLocalChecked()); - // Remove the meemory sampler + // Remove the memory sampler profiler->measurements_ticker.remove_heap_listener(id, memory_sampler_cb); profiler->measurements_ticker.remove_cpu_listener(id, cpu_sampler_cb); // If for some reason stopProfiling was called with an invalid profile title diff --git a/packages/profiling-node/scripts/check-build.js b/packages/profiling-node/scripts/check-build.js index 6892d90ba4b3..dda96e66b900 100644 --- a/packages/profiling-node/scripts/check-build.js +++ b/packages/profiling-node/scripts/check-build.js @@ -1,4 +1,4 @@ -// This is a build scripts, so some logging is desireable as it allows +// This is a build script, so some logging is desirable as it allows // us to follow the code path that triggered the error. /* eslint-disable no-console */ const fs = require('fs'); diff --git a/packages/profiling-node/scripts/copy-target.js b/packages/profiling-node/scripts/copy-target.js index ee3b75163724..8277f1d45290 100644 --- a/packages/profiling-node/scripts/copy-target.js +++ b/packages/profiling-node/scripts/copy-target.js @@ -1,4 +1,4 @@ -// This is a build scripts, so some logging is desireable as it allows +// This is a build script, so some logging is desirable as it allows // us to follow the code path that triggered the error. /* eslint-disable no-console */ const fs = require('fs'); diff --git a/packages/profiling-node/scripts/prune-profiler-binaries.js b/packages/profiling-node/scripts/prune-profiler-binaries.js index 925cedaee73a..22fc13ce28b6 100755 --- a/packages/profiling-node/scripts/prune-profiler-binaries.js +++ b/packages/profiling-node/scripts/prune-profiler-binaries.js @@ -1,6 +1,6 @@ #! /usr/bin/env node -// This is a build scripts, so some logging is desireable as it allows +// This is a build script, so some logging is desirable as it allows // us to follow the code path that triggered the error. /* eslint-disable no-console */ const fs = require('fs'); @@ -88,7 +88,7 @@ if (!SOURCE_DIR) { if (!PLATFORM && !ARCH && !STDLIB) { ARGV_ERRORS.push( - `❌ Sentry: Missing argument values, pruning requires either --target_platform, --target_arch or --targer_stdlib to be passed as argument values.\n Example: sentry-prune-profiler-binaries --target_platform=linux --target_arch=x64 --target_stdlib=glibc\n + `❌ Sentry: Missing argument values, pruning requires either --target_platform, --target_arch or --target_stdlib to be passed as argument values.\n Example: sentry-prune-profiler-binaries --target_platform=linux --target_arch=x64 --target_stdlib=glibc\n If you are unsure about the execution environment, you can opt to skip some values, but at least one value must be passed.`, ); } diff --git a/packages/profiling-node/src/cpu_profiler.ts b/packages/profiling-node/src/cpu_profiler.ts index fb739a939e77..51d0279ba31b 100644 --- a/packages/profiling-node/src/cpu_profiler.ts +++ b/packages/profiling-node/src/cpu_profiler.ts @@ -46,7 +46,7 @@ export function importCppBindingsModule(): PrivateV8CpuProfilerBindings { return require(`${binaryPath}.node`); } - // We need the fallthrough so that in the end, we can fallback to the require dynamice require. + // We need the fallthrough so that in the end, we can fallback to the dynamic require. // This is for cases where precompiled binaries were not provided, but may have been compiled from source. if (platform === 'darwin') { if (arch === 'x64') { diff --git a/packages/profiling-node/src/utils.ts b/packages/profiling-node/src/utils.ts index 0eb4fe4281b4..1ec575aaaf3f 100644 --- a/packages/profiling-node/src/utils.ts +++ b/packages/profiling-node/src/utils.ts @@ -364,7 +364,7 @@ export function findProfiledTransactionsFromEnvelope(envelope: Envelope): Event[ const event = item[j]; if (!event) { - // Shouldnt happen, but lets be safe + // Shouldn't happen, but lets be safe continue; } diff --git a/packages/profiling-node/test/bindings.test.ts b/packages/profiling-node/test/bindings.test.ts index c524a277bfa9..27361a87d941 100644 --- a/packages/profiling-node/test/bindings.test.ts +++ b/packages/profiling-node/test/bindings.test.ts @@ -14,7 +14,7 @@ const cases = [ ['/Users/jonas/code/src/file.ts', 'Users.jonas.code.src:file.ts'], // No extension ['/Users/jonas/code/src/file', 'Users.jonas.code.src:file'], - // Edge cases that shouldn't happen in practice, but try and handle them so we dont crash + // Edge cases that shouldn't happen in practice, but try and handle them so we don't crash ['/Users/jonas/code/src/file.js', 'Users.jonas.code.src:file'], ['', ''], ]; diff --git a/packages/profiling-node/test/cpu_profiler.test.ts b/packages/profiling-node/test/cpu_profiler.test.ts index 1e3903be6fc5..b370ae7f96c9 100644 --- a/packages/profiling-node/test/cpu_profiler.test.ts +++ b/packages/profiling-node/test/cpu_profiler.test.ts @@ -234,7 +234,7 @@ describe('Profiler bindings', () => { } expect(sample.timestamp).toBeDefined(); // No older than a minute and not in the future. Timestamp is in seconds so convert to ms - // as the constructor expectes ms. + // as the constructor expects ms. expect(new Date((sample.timestamp as number) * 1e3).getTime()).toBeGreaterThan(Date.now() - 60 * 1e3); expect(new Date((sample.timestamp as number) * 1e3).getTime()).toBeLessThanOrEqual(Date.now()); } diff --git a/packages/profiling-node/test/spanProfileUtils.test.ts b/packages/profiling-node/test/spanProfileUtils.test.ts index fd2c95ec79e4..c5619eaaae85 100644 --- a/packages/profiling-node/test/spanProfileUtils.test.ts +++ b/packages/profiling-node/test/spanProfileUtils.test.ts @@ -556,7 +556,7 @@ describe('continuous profiling', () => { expect(stopProfilingSpy).toHaveBeenCalledTimes(1); }); - it('manullly stopping a chunk doesnt restart the profiler', async () => { + it('manually stopping a chunk doesnt restart the profiler', async () => { const startProfilingSpy = jest.spyOn(CpuProfilerBindings, 'startProfiling'); const stopProfilingSpy = jest.spyOn(CpuProfilerBindings, 'stopProfiling'); diff --git a/packages/react/README.md b/packages/react/README.md index 5645b03d9fb0..066ab7f7c828 100644 --- a/packages/react/README.md +++ b/packages/react/README.md @@ -75,7 +75,7 @@ import React from 'react'; import * as Sentry from '@sentry/react'; function FallbackComponent() { - return
An error has occured
; + return
An error has occurred
; } class App extends React.Component { diff --git a/packages/react/src/profiler.tsx b/packages/react/src/profiler.tsx index 5008ffdc0010..611425b04df0 100644 --- a/packages/react/src/profiler.tsx +++ b/packages/react/src/profiler.tsx @@ -78,8 +78,8 @@ class Profiler extends React.Component { // and if the updateProps have changed. It is ok to not do a deep equality check here as it is expensive. // We are just trying to give baseline clues for further investigation. if (includeUpdates && this._mountSpan && updateProps !== this.props.updateProps) { - // See what props haved changed between the previous props, and the current props. This is - // set as data on the span. We just store the prop keys as the values could be potenially very large. + // See what props have changed between the previous props, and the current props. This is + // set as data on the span. We just store the prop keys as the values could be potentially very large. const changedProps = Object.keys(updateProps).filter(k => updateProps[k] !== this.props.updateProps[k]); if (changedProps.length > 0) { const now = timestampInSeconds(); diff --git a/packages/react/src/reactrouter.tsx b/packages/react/src/reactrouter.tsx index d52885dffa8d..1b9102273e9c 100644 --- a/packages/react/src/reactrouter.tsx +++ b/packages/react/src/reactrouter.tsx @@ -20,7 +20,7 @@ import type { ReactElement } from 'react'; import type { Action, Location } from './types'; -// We need to disable eslint no-explict-any because any is required for the +// We need to disable eslint no-explicit-any because any is required for the // react-router typings. type Match = { path: string; url: string; params: Record; isExact: boolean }; // eslint-disable-line @typescript-eslint/no-explicit-any diff --git a/packages/react/src/reactrouterv6.tsx b/packages/react/src/reactrouterv6.tsx index 400028a28789..1ec1ae4b7d35 100644 --- a/packages/react/src/reactrouterv6.tsx +++ b/packages/react/src/reactrouterv6.tsx @@ -170,7 +170,7 @@ function getNormalizedName( if ( // If the route defined on the element is something like // Product} /> - // We should check against the branch.pathname for the number of / seperators + // We should check against the branch.pathname for the number of / separators getNumberOfUrlSegments(pathBuilder) !== getNumberOfUrlSegments(branch.pathname) && // We should not count wildcard operators in the url segments calculation pathBuilder.slice(-2) !== '/*' @@ -266,7 +266,7 @@ export function withSentryReactRouterV6Routing

, R handleNavigation(location, routes, navigationType); } }, - // `props.children` is purpusely not included in the dependency array, because we do not want to re-run this effect + // `props.children` is purposely not included in the dependency array, because we do not want to re-run this effect // when the children change. We only want to start transactions when the location or navigation type change. [location, navigationType], ); diff --git a/packages/react/src/types.ts b/packages/react/src/types.ts index 9c81f63cc0d5..1a40ec4fce91 100644 --- a/packages/react/src/types.ts +++ b/packages/react/src/types.ts @@ -64,7 +64,7 @@ export type UseLocation = () => Location; export type UseNavigationType = () => Action; // For both of these types, use `any` instead of `RouteObject[]` or `RouteMatch[]`. -// Have to do this so we maintain backwards compatability between +// Have to do this so we maintain backwards compatibility between // react-router > 6.0.0 and >= 6.4.2. export type RouteObjectArrayAlias = any; export type RouteMatchAlias = any; diff --git a/packages/react/test/reactrouterv6.test.tsx b/packages/react/test/reactrouterv6.test.tsx index d584bcfa22c1..fbb135a5449c 100644 --- a/packages/react/test/reactrouterv6.test.tsx +++ b/packages/react/test/reactrouterv6.test.tsx @@ -268,7 +268,7 @@ describe('reactRouterV6BrowserTracingIntegration', () => { }); }); - it('works with paramaterized paths', () => { + it('works with parameterized paths', () => { const client = createMockBrowserClient(); setCurrentClient(client); @@ -663,7 +663,7 @@ describe('reactRouterV6BrowserTracingIntegration', () => { }); }); - it('works with paramaterized paths', () => { + it('works with parameterized paths', () => { const client = createMockBrowserClient(); setCurrentClient(client); diff --git a/packages/remix/src/index.types.ts b/packages/remix/src/index.types.ts index b3342ff35250..2ffec834558b 100644 --- a/packages/remix/src/index.types.ts +++ b/packages/remix/src/index.types.ts @@ -1,5 +1,5 @@ // We export everything from both the client part of the SDK and from the server part. Some of the exports collide, -// which is not allowed, unless we redifine the colliding exports in this file - which we do below. +// which is not allowed, unless we redefine the colliding exports in this file - which we do below. export * from './index.client'; export * from './index.server'; diff --git a/packages/replay-internal/README.md b/packages/replay-internal/README.md index 90ddb5b78110..b08aad64b21e 100644 --- a/packages/replay-internal/README.md +++ b/packages/replay-internal/README.md @@ -107,7 +107,7 @@ Sentry.init({ const client = Sentry.getClient(); -// Add replay integration, will start recoring +// Add replay integration, will start recording client?.addIntegration(replay); // Stop recording diff --git a/packages/replay-internal/src/coreHandlers/handleClick.ts b/packages/replay-internal/src/coreHandlers/handleClick.ts index 029588764610..6be19626315c 100644 --- a/packages/replay-internal/src/coreHandlers/handleClick.ts +++ b/packages/replay-internal/src/coreHandlers/handleClick.ts @@ -66,7 +66,7 @@ export class ClickDetector implements ReplayClickDetector { private _teardown: undefined | (() => void); private _threshold: number; - private _scollTimeout: number; + private _scrollTimeout: number; private _timeout: number; private _ignoreSelector: string; @@ -87,7 +87,7 @@ export class ClickDetector implements ReplayClickDetector { // We want everything in s, but options are in ms this._timeout = slowClickConfig.timeout / 1000; this._threshold = slowClickConfig.threshold / 1000; - this._scollTimeout = slowClickConfig.scrollTimeout / 1000; + this._scrollTimeout = slowClickConfig.scrollTimeout / 1000; this._replay = replay; this._ignoreSelector = slowClickConfig.ignoreSelector; this._addBreadcrumbEvent = _addBreadcrumbEvent; @@ -216,7 +216,7 @@ export class ClickDetector implements ReplayClickDetector { /** Generate matching breadcrumb(s) for the click. */ private _generateBreadcrumbs(click: Click): void { const replay = this._replay; - const hadScroll = click.scrollAfter && click.scrollAfter <= this._scollTimeout; + const hadScroll = click.scrollAfter && click.scrollAfter <= this._scrollTimeout; const hadMutation = click.mutationAfter && click.mutationAfter <= this._threshold; const isSlowClick = !hadScroll && !hadMutation; diff --git a/packages/replay-internal/src/coreHandlers/util/fetchUtils.ts b/packages/replay-internal/src/coreHandlers/util/fetchUtils.ts index f218f4ab9b35..8da058674516 100644 --- a/packages/replay-internal/src/coreHandlers/util/fetchUtils.ts +++ b/packages/replay-internal/src/coreHandlers/util/fetchUtils.ts @@ -26,7 +26,7 @@ import { /** * Capture a fetch breadcrumb to a replay. - * This adds additional data (where approriate). + * This adds additional data (where appropriate). */ export async function captureFetchBreadcrumbToReplay( breadcrumb: Breadcrumb & { data: FetchBreadcrumbData }, diff --git a/packages/replay-internal/src/coreHandlers/util/shouldSampleForBufferEvent.ts b/packages/replay-internal/src/coreHandlers/util/shouldSampleForBufferEvent.ts index 736d296e2d95..2144aab4182c 100644 --- a/packages/replay-internal/src/coreHandlers/util/shouldSampleForBufferEvent.ts +++ b/packages/replay-internal/src/coreHandlers/util/shouldSampleForBufferEvent.ts @@ -6,7 +6,7 @@ import { isSampled } from '../../util/isSampled'; /** * Determine if event should be sampled (only applies in buffer mode). - * When an event is captured by `hanldleGlobalEvent`, when in buffer mode + * When an event is captured by `handleGlobalEvent`, when in buffer mode * we determine if we want to sample the error or not. */ export function shouldSampleForBufferEvent(replay: ReplayContainer, event: Event): boolean { diff --git a/packages/replay-internal/src/coreHandlers/util/xhrUtils.ts b/packages/replay-internal/src/coreHandlers/util/xhrUtils.ts index 52b6cafdfec7..d7ac201c0756 100644 --- a/packages/replay-internal/src/coreHandlers/util/xhrUtils.ts +++ b/packages/replay-internal/src/coreHandlers/util/xhrUtils.ts @@ -25,7 +25,7 @@ import { /** * Capture an XHR breadcrumb to a replay. - * This adds additional data (where approriate). + * This adds additional data (where appropriate). */ export async function captureXhrBreadcrumbToReplay( breadcrumb: Breadcrumb & { data: XhrBreadcrumbData }, diff --git a/packages/replay-internal/src/eventBuffer/EventBufferCompressionWorker.ts b/packages/replay-internal/src/eventBuffer/EventBufferCompressionWorker.ts index 90a54bbf07f3..8ca7f3caccca 100644 --- a/packages/replay-internal/src/eventBuffer/EventBufferCompressionWorker.ts +++ b/packages/replay-internal/src/eventBuffer/EventBufferCompressionWorker.ts @@ -39,7 +39,7 @@ export class EventBufferCompressionWorker implements EventBuffer { /** * Ensure the worker is ready (or not). - * This will either resolve when the worker is ready, or reject if an error occured. + * This will either resolve when the worker is ready, or reject if an error occurred. */ public ensureReady(): Promise { return this._worker.ensureReady(); @@ -55,7 +55,7 @@ export class EventBufferCompressionWorker implements EventBuffer { /** * Add an event to the event buffer. * - * Returns true if event was successfuly received and processed by worker. + * Returns true if event was successfully received and processed by worker. */ public addEvent(event: RecordingEvent): Promise { const timestamp = timestampToMs(event.timestamp); diff --git a/packages/replay-internal/src/eventBuffer/WorkerHandler.ts b/packages/replay-internal/src/eventBuffer/WorkerHandler.ts index 2ccc3ee94b3c..062ed384674b 100644 --- a/packages/replay-internal/src/eventBuffer/WorkerHandler.ts +++ b/packages/replay-internal/src/eventBuffer/WorkerHandler.ts @@ -18,7 +18,7 @@ export class WorkerHandler { /** * Ensure the worker is ready (or not). - * This will either resolve when the worker is ready, or reject if an error occured. + * This will either resolve when the worker is ready, or reject if an error occurred. */ public ensureReady(): Promise { // Ensure we only check once diff --git a/packages/replay-internal/src/replay.ts b/packages/replay-internal/src/replay.ts index 0976b7d57870..d87a73a7e51d 100644 --- a/packages/replay-internal/src/replay.ts +++ b/packages/replay-internal/src/replay.ts @@ -640,7 +640,7 @@ export class ReplayContainer implements ReplayContainerInterface { /** * Always flush via `_debouncedFlush` so that we do not have flushes triggered * from calling both `flush` and `_debouncedFlush`. Otherwise, there could be - * cases of mulitple flushes happening closely together. + * cases of multiple flushes happening closely together. */ public flushImmediate(): Promise { this._debouncedFlush(); @@ -655,7 +655,7 @@ export class ReplayContainer implements ReplayContainerInterface { this._debouncedFlush.cancel(); } - /** Get the current sesion (=replay) ID */ + /** Get the current session (=replay) ID */ public getSessionId(): string | undefined { return this.session && this.session.id; } @@ -1155,7 +1155,7 @@ export class ReplayContainer implements ReplayContainerInterface { const timestamp = Date.now(); // Check total duration again, to avoid sending outdated stuff - // We leave 30s wiggle room to accomodate late flushing etc. + // We leave 30s wiggle room to accommodate late flushing etc. // This _could_ happen when the browser is suspended during flushing, in which case we just want to stop if (timestamp - this._context.initialTimestamp > this._options.maxReplayDuration + 30_000) { throw new Error('Session is too long, not sending replay'); diff --git a/packages/replay-internal/src/types/rrweb.ts b/packages/replay-internal/src/types/rrweb.ts index cb194e193a5d..60e562cadf55 100644 --- a/packages/replay-internal/src/types/rrweb.ts +++ b/packages/replay-internal/src/types/rrweb.ts @@ -20,7 +20,7 @@ export type ReplayEventType = /** * This is a partial copy of rrweb's eventWithTime type which only contains the properties - * we specifcally need in the SDK. + * we specifically need in the SDK. */ export type ReplayEventWithTime = { type: ReplayEventType; diff --git a/packages/replay-internal/src/util/addEvent.ts b/packages/replay-internal/src/util/addEvent.ts index 700627cf954f..3a245242e608 100644 --- a/packages/replay-internal/src/util/addEvent.ts +++ b/packages/replay-internal/src/util/addEvent.ts @@ -126,7 +126,7 @@ function maybeApplyCallback( } } catch (error) { DEBUG_BUILD && - logger.exception(error, 'An error occured in the `beforeAddRecordingEvent` callback, skipping the event...'); + logger.exception(error, 'An error occurred in the `beforeAddRecordingEvent` callback, skipping the event...'); return null; } diff --git a/packages/replay-internal/src/util/maskAttribute.ts b/packages/replay-internal/src/util/maskAttribute.ts index c74c2dc8121a..12daaeb97dff 100644 --- a/packages/replay-internal/src/util/maskAttribute.ts +++ b/packages/replay-internal/src/util/maskAttribute.ts @@ -25,7 +25,7 @@ export function maskAttribute({ return value; } - // unmaskTextSelector takes precendence + // unmaskTextSelector takes precedence if (privacyOptions.unmaskTextSelector && el.matches(privacyOptions.unmaskTextSelector)) { return value; } diff --git a/packages/replay-internal/test/integration/earlyEvents.test.ts b/packages/replay-internal/test/integration/earlyEvents.test.ts index 11dcb7295c0e..d89103166765 100644 --- a/packages/replay-internal/test/integration/earlyEvents.test.ts +++ b/packages/replay-internal/test/integration/earlyEvents.test.ts @@ -35,7 +35,7 @@ describe('Integration | early events', () => { expect(initialTimestamp).not.toEqual(earlyTimeStampSeconds * 1000); - // A performance entry that happend before should not extend the session when we manually started + // A performance entry that happened before should not extend the session when we manually started replay.replayPerformanceEntries.push({ type: 'largest-contentful-paint', name: 'largest-contentful-paint', diff --git a/packages/replay-internal/test/unit/coreHandlers/handleClick.test.ts b/packages/replay-internal/test/unit/coreHandlers/handleClick.test.ts index c2e46722941e..4cd75a609ec3 100644 --- a/packages/replay-internal/test/unit/coreHandlers/handleClick.test.ts +++ b/packages/replay-internal/test/unit/coreHandlers/handleClick.test.ts @@ -311,7 +311,7 @@ describe('Unit | coreHandlers | handleClick', () => { vi.advanceTimersByTime(500); - // Pretend a mutation happend + // Pretend a mutation happened detector['_lastMutation'] = BASE_TIMESTAMP / 1000 + 0.5; expect(mockAddBreadcrumbEvent).toHaveBeenCalledTimes(0); @@ -335,7 +335,7 @@ describe('Unit | coreHandlers | handleClick', () => { vi.advanceTimersByTime(1_000); - // Pretend a mutation happend + // Pretend a mutation happened detector['_lastMutation'] = BASE_TIMESTAMP / 1000 + 2; expect(mockAddBreadcrumbEvent).toHaveBeenCalledTimes(0); @@ -376,7 +376,7 @@ describe('Unit | coreHandlers | handleClick', () => { vi.advanceTimersByTime(1_000); - // Pretend a mutation happend + // Pretend a mutation happened detector['_lastMutation'] = BASE_TIMESTAMP / 1000 + 5; expect(mockAddBreadcrumbEvent).toHaveBeenCalledTimes(0); @@ -443,7 +443,7 @@ describe('Unit | coreHandlers | handleClick', () => { vi.advanceTimersByTime(100); - // Pretend a mutation happend + // Pretend a mutation happened detector['_lastScroll'] = BASE_TIMESTAMP / 1000 + 0.15; expect(mockAddBreadcrumbEvent).toHaveBeenCalledTimes(0); @@ -467,7 +467,7 @@ describe('Unit | coreHandlers | handleClick', () => { vi.advanceTimersByTime(300); - // Pretend a mutation happend + // Pretend a mutation happened detector['_lastScroll'] = BASE_TIMESTAMP / 1000 + 0.3; expect(mockAddBreadcrumbEvent).toHaveBeenCalledTimes(0); diff --git a/packages/replay-internal/test/unit/coreHandlers/handleKeyboardEvent.test.ts b/packages/replay-internal/test/unit/coreHandlers/handleKeyboardEvent.test.ts index 4e438173d3cd..be7764eaa327 100644 --- a/packages/replay-internal/test/unit/coreHandlers/handleKeyboardEvent.test.ts +++ b/packages/replay-internal/test/unit/coreHandlers/handleKeyboardEvent.test.ts @@ -21,7 +21,7 @@ describe('Unit | coreHandlers | handleKeyboardEvent', () => { }); it('returns null for event on contenteditable div', function () { - // JSOM does not support contentEditable properly :( + // JSDOM does not support contentEditable properly :( const target = document.createElement('div'); Object.defineProperty(target, 'isContentEditable', { get: function () { diff --git a/packages/replay-internal/test/unit/coreHandlers/util/networkUtils.test.ts b/packages/replay-internal/test/unit/coreHandlers/util/networkUtils.test.ts index 4158c8bfe7a9..00db91815a5f 100644 --- a/packages/replay-internal/test/unit/coreHandlers/util/networkUtils.test.ts +++ b/packages/replay-internal/test/unit/coreHandlers/util/networkUtils.test.ts @@ -38,7 +38,7 @@ describe('Unit | coreHandlers | util | networkUtils', () => { it('works with string body', () => { expect(getBodySize('abcd')).toBe(4); - // Emojis are correctly counted as mutliple characters + // Emojis are correctly counted as multiple characters expect(getBodySize('With emoji: 😈')).toBe(16); }); diff --git a/packages/replay-internal/test/unit/util/debounce.test.ts b/packages/replay-internal/test/unit/util/debounce.test.ts index a9a8f332a2f5..56d97621a030 100644 --- a/packages/replay-internal/test/unit/util/debounce.test.ts +++ b/packages/replay-internal/test/unit/util/debounce.test.ts @@ -35,7 +35,7 @@ describe('Unit | util | debounce', () => { expect(callback).not.toHaveBeenCalled(); vi.advanceTimersByTime(49); - // at this time, the callback shouldn't be invoked and with a new call, it should be devounced further. + // at this time, the callback shouldn't be invoked and with a new call, it should be debounced further. debouncedCallback(); expect(callback).not.toHaveBeenCalled(); diff --git a/packages/replay-worker/README.md b/packages/replay-worker/README.md index 29d39df52d67..ae8a751b3560 100644 --- a/packages/replay-worker/README.md +++ b/packages/replay-worker/README.md @@ -15,5 +15,5 @@ By extracting this into a dedicated (private, internal) package, we can streamli You can find an example worker for if you want to self-host the compression worker in [/examples](./examples/). -This is generated from the actual soure via `yarn build:examples`, which should be run manually whenever replay-worker +This is generated from the actual source via `yarn build:examples`, which should be run manually whenever replay-worker is updated. diff --git a/packages/solidstart/README.md b/packages/solidstart/README.md index ceda55838e8d..c43ac54c7037 100644 --- a/packages/solidstart/README.md +++ b/packages/solidstart/README.md @@ -28,7 +28,7 @@ functionality related to SolidStart. If the setup through the wizard doesn't work for you, you can also set up the SDK manually. -### 1. Prerequesits & Installation +### 1. Prerequisites & Installation Install the Sentry SolidStart SDK: diff --git a/packages/solidstart/rollup.npm.config.mjs b/packages/solidstart/rollup.npm.config.mjs index b0087a93c6fe..6723ed69845b 100644 --- a/packages/solidstart/rollup.npm.config.mjs +++ b/packages/solidstart/rollup.npm.config.mjs @@ -13,7 +13,7 @@ export default makeNPMConfigVariants( 'src/client/solidrouter.ts', 'src/server/solidrouter.ts', ], - // prevent this internal code from ending up in our built package (this doesn't happen automatially because + // prevent this internal code from ending up in our built package (this doesn't happen automatically because // the name doesn't match an SDK dependency) packageSpecificConfig: { external: ['solid-js/web', 'solid-js', '@sentry/solid', '@sentry/solid/solidrouter'], diff --git a/packages/svelte/src/sdk.ts b/packages/svelte/src/sdk.ts index 721051fc2418..aa40e5a25a98 100644 --- a/packages/svelte/src/sdk.ts +++ b/packages/svelte/src/sdk.ts @@ -49,7 +49,7 @@ export function detectAndReportSvelteKit(): void { /** * To actually detect a SvelteKit frontend, we search the DOM for a special - * div that's inserted by SvelteKit when the page is rendered. It's identifyed + * div that's inserted by SvelteKit when the page is rendered. It's identified * by its id, 'svelte-announcer', and it's used to improve page accessibility. * This div is not present when only using Svelte without SvelteKit. * diff --git a/packages/svelte/src/types.ts b/packages/svelte/src/types.ts index ed7fcfaa4928..8079019d8568 100644 --- a/packages/svelte/src/types.ts +++ b/packages/svelte/src/types.ts @@ -25,7 +25,7 @@ export type SentrySvelteConfigOptions = { export type SpanOptions = { /** - * If true, a span is recorded between a component's intialization and its + * If true, a span is recorded between a component's initialization and its * onMount lifecycle hook. This span tells how long it takes a component * to be created and inserted into the DOM. * diff --git a/packages/sveltekit/src/client/load.ts b/packages/sveltekit/src/client/load.ts index 3b3fba05fb07..6f0e84ef140f 100644 --- a/packages/sveltekit/src/client/load.ts +++ b/packages/sveltekit/src/client/load.ts @@ -45,7 +45,7 @@ function sendErrorToSentry(e: unknown): unknown { * * - catch errors happening during the execution of `load` * - create a load span if performance monitoring is enabled - * - attach tracing Http headers to `fech` requests if performance monitoring is enabled to get connected traces. + * - attach tracing Http headers to `fetch` requests if performance monitoring is enabled to get connected traces. * - add a fetch breadcrumb for every `fetch` request * * Note that tracing Http headers are only attached if the url matches the specified `tracePropagationTargets` diff --git a/packages/sveltekit/src/index.types.ts b/packages/sveltekit/src/index.types.ts index 4cccbf2a1ab7..622c05935922 100644 --- a/packages/sveltekit/src/index.types.ts +++ b/packages/sveltekit/src/index.types.ts @@ -1,5 +1,5 @@ // We export everything from both the client part of the SDK and from the server part. -// Some of the exports collide, which is not allowed, unless we redifine the colliding +// Some of the exports collide, which is not allowed, unless we redefine the colliding // exports in this file - which we do below. export * from './client'; export * from './vite'; diff --git a/packages/sveltekit/src/vite/autoInstrument.ts b/packages/sveltekit/src/vite/autoInstrument.ts index 708f3c5fc7dc..28e903d89db7 100644 --- a/packages/sveltekit/src/vite/autoInstrument.ts +++ b/packages/sveltekit/src/vite/autoInstrument.ts @@ -78,7 +78,7 @@ export function makeAutoInstrumentationPlugin(options: AutoInstrumentPluginOptio /** * We only want to apply our wrapper to files that * - * - Have no Sentry code yet in them. This is to avoid double-wrapping or interferance with custom + * - Have no Sentry code yet in them. This is to avoid double-wrapping or interfering with custom * Sentry calls. * - Actually declare a `load` function. The second check of course is not 100% accurate, but it's good enough. * Injecting our wrapper into files that don't declare a `load` function would result in a build-time warning diff --git a/packages/sveltekit/src/vite/detectAdapter.ts b/packages/sveltekit/src/vite/detectAdapter.ts index ac9dce2bf8a9..7717aa4bd4c2 100644 --- a/packages/sveltekit/src/vite/detectAdapter.ts +++ b/packages/sveltekit/src/vite/detectAdapter.ts @@ -41,7 +41,7 @@ export async function detectAdapter(debug?: boolean): Promise { const pkgFile = path.join(process.cwd(), 'package.json'); diff --git a/packages/sveltekit/src/vite/sourceMaps.ts b/packages/sveltekit/src/vite/sourceMaps.ts index d228eb6da30e..d0faf85addbd 100644 --- a/packages/sveltekit/src/vite/sourceMaps.ts +++ b/packages/sveltekit/src/vite/sourceMaps.ts @@ -105,7 +105,7 @@ export async function makeCustomSentryVitePlugins(options?: CustomSentryVitePlug const sourceMapsPreviouslyNotEnabled = !config.build?.sourcemap; if (debug && sourceMapsPreviouslyNotEnabled) { // eslint-disable-next-line no-console - console.log('[Source Maps Plugin] Enabeling source map generation'); + console.log('[Source Maps Plugin] Enabling source map generation'); if (!mergedOptions.sourcemaps?.filesToDeleteAfterUpload) { // eslint-disable-next-line no-console console.warn( diff --git a/packages/sveltekit/src/vite/svelteConfig.ts b/packages/sveltekit/src/vite/svelteConfig.ts index e50f8b10a2c3..9186e46caba6 100644 --- a/packages/sveltekit/src/vite/svelteConfig.ts +++ b/packages/sveltekit/src/vite/svelteConfig.ts @@ -46,7 +46,7 @@ export function getHooksFileName(svelteConfig: Config, hookType: 'client' | 'ser } /** - * Attempts to read a custom output directory that can be specidied in the options + * Attempts to read a custom output directory that can be specified in the options * of a SvelteKit adapter. If no custom output directory is specified, the default * directory is returned. */ diff --git a/packages/sveltekit/src/vite/types.ts b/packages/sveltekit/src/vite/types.ts index 7102971375d4..0f6717a2c7e9 100644 --- a/packages/sveltekit/src/vite/types.ts +++ b/packages/sveltekit/src/vite/types.ts @@ -178,7 +178,7 @@ export type SentrySvelteKitPluginOptions = { /** * The Sentry plugin will automatically instrument certain parts of your SvelteKit application at build time. - * Set this option to `false` to disable this behavior or what is instrumentated by passing an object. + * Set this option to `false` to disable this behavior or what is intrumented by passing an object. * * Auto instrumentation includes: * - Universal `load` functions in `+page.(js|ts)` files diff --git a/packages/sveltekit/test/client/browserTracingIntegration.test.ts b/packages/sveltekit/test/client/browserTracingIntegration.test.ts index 6d052cc77b50..a21dbbe66bae 100644 --- a/packages/sveltekit/test/client/browserTracingIntegration.test.ts +++ b/packages/sveltekit/test/client/browserTracingIntegration.test.ts @@ -291,7 +291,7 @@ describe('browserTracingIntegration', () => { // @ts-expect-error - the fakeClient doesn't satisfy Client but that's fine integration.afterAllSetup(fakeClient); - // window.location.pathame is "/" in tests + // window.location.pathname is "/" in tests // @ts-expect-error - page is a writable but the types say it's just readable navigating.set({ diff --git a/packages/sveltekit/test/server/handle.test.ts b/packages/sveltekit/test/server/handle.test.ts index 7b5b86d541b2..c25b926c7ace 100644 --- a/packages/sveltekit/test/server/handle.test.ts +++ b/packages/sveltekit/test/server/handle.test.ts @@ -163,7 +163,7 @@ describe('sentryHandle', () => { await sentryHandle()({ event: mockEvent(), resolve: async _ => { - // simulateing a nested load call: + // simulating a nested load call: await sentryHandle()({ event: mockEvent({ route: { id: 'api/users/details/[id]', isSubRequest: true } }), resolve: resolve(type, isError), @@ -210,7 +210,7 @@ describe('sentryHandle', () => { await sentryHandle()({ event: mockEvent(), resolve: async _ => { - // simulateing a nested load call: + // simulating a nested load call: await sentryHandle()({ event: mockEvent({ route: { id: 'api/users/details/[id]' } }), resolve: resolve(type, isError), diff --git a/packages/types/src/client.ts b/packages/types/src/client.ts index 94f28f9157aa..06e3109e1e14 100644 --- a/packages/types/src/client.ts +++ b/packages/types/src/client.ts @@ -303,7 +303,7 @@ export interface Client { */ on(hook: 'close', callback: () => void): () => void; - /** Fire a hook whener a span starts. */ + /** Fire a hook whenever a span starts. */ emit(hook: 'spanStart', span: Span): void; /** A hook that is called every time before a span is sampled. */ @@ -318,7 +318,7 @@ export interface Client { samplingDecision: { decision: boolean }, ): void; - /** Fire a hook whener a span ends. */ + /** Fire a hook whenever a span ends. */ emit(hook: 'spanEnd', span: Span): void; /** diff --git a/packages/types/src/feedback/config.ts b/packages/types/src/feedback/config.ts index 0b9548b5712e..4ec846c7d98d 100644 --- a/packages/types/src/feedback/config.ts +++ b/packages/types/src/feedback/config.ts @@ -168,7 +168,7 @@ export interface FeedbackTextConfiguration { isRequiredLabel: string; /** - * The label for the button that adds a screeshot and renders the image editor + * The label for the button that adds a screenshot and renders the image editor */ addScreenshotButtonLabel: string; diff --git a/packages/types/src/mechanism.ts b/packages/types/src/mechanism.ts index 9d3dc86e7382..8d34fdcb26eb 100644 --- a/packages/types/src/mechanism.ts +++ b/packages/types/src/mechanism.ts @@ -50,7 +50,7 @@ export interface Mechanism { exception_id?: number; /** - * References another exception via the `exception_id` field to indicate that this excpetion is a child of that + * References another exception via the `exception_id` field to indicate that this exception is a child of that * exception in the case of aggregate or linked errors. */ parent_id?: number; diff --git a/packages/types/src/scope.ts b/packages/types/src/scope.ts index a4b91f4b5d96..2d5c72230aef 100644 --- a/packages/types/src/scope.ts +++ b/packages/types/src/scope.ts @@ -235,7 +235,7 @@ export interface Scope { * Capture an exception for this scope. * * @param exception The exception to capture. - * @param hint Optinal additional data to attach to the Sentry event. + * @param hint Optional additional data to attach to the Sentry event. * @returns the id of the captured Sentry event. */ captureException(exception: unknown, hint?: EventHint): string; diff --git a/packages/utils/src/browser.ts b/packages/utils/src/browser.ts index ce00f2556d05..b3a5220e7c3c 100644 --- a/packages/utils/src/browser.ts +++ b/packages/utils/src/browser.ts @@ -143,7 +143,7 @@ export function getLocationHref(): string { /** * Gets a DOM element by using document.querySelector. * - * This wrapper will first check for the existance of the function before + * This wrapper will first check for the existence of the function before * actually calling it so that we don't have to take care of this check, * every time we want to access the DOM. * @@ -167,7 +167,7 @@ export function getDomElement(selector: string): E | null { /** * Given a DOM element, traverses up the tree until it finds the first ancestor node * that has the `data-sentry-component` or `data-sentry-element` attribute with `data-sentry-component` taking - * precendence. This attribute is added at build-time by projects that have the component name annotation plugin installed. + * precedence. This attribute is added at build-time by projects that have the component name annotation plugin installed. * * @returns a string representation of the component for the provided DOM element, or `null` if not found */ diff --git a/packages/utils/src/buildPolyfills/README.md b/packages/utils/src/buildPolyfills/README.md index bdae9c65c89d..3b18ad989133 100644 --- a/packages/utils/src/buildPolyfills/README.md +++ b/packages/utils/src/buildPolyfills/README.md @@ -8,7 +8,7 @@ which can lead to a great deal of duplication. For our builds, we have therefore polyfills injected in multiple places, we instead replace each injected function with an `import` or `require` statement. -Note that not all polyfills are currently used by the SDK, but all are included here for future compatitibility, should +Note that not all polyfills are currently used by the SDK, but all are included here for future compatibility, should they ever be needed. Also, since we're never going to be calling these directly from within another TS file, their types are fairly generic. In some cases testing required more specific types, which can be found in the test files. diff --git a/packages/utils/src/envelope.ts b/packages/utils/src/envelope.ts index 8bf29788edf0..945bd4d66366 100644 --- a/packages/utils/src/envelope.ts +++ b/packages/utils/src/envelope.ts @@ -117,7 +117,7 @@ export function serializeEnvelope(envelope: Envelope): string | Uint8Array { try { stringifiedPayload = JSON.stringify(payload); } catch (e) { - // In case, despite all our efforts to keep `payload` circular-dependency-free, `JSON.strinify()` still + // In case, despite all our efforts to keep `payload` circular-dependency-free, `JSON.stringify()` still // fails, we try again after normalizing it again with infinite normalization depth. This of course has a // performance impact but in this case a performance hit is better than throwing. stringifiedPayload = JSON.stringify(normalize(payload)); diff --git a/packages/utils/src/normalize.ts b/packages/utils/src/normalize.ts index 18b41f1c9357..e88b1edd8513 100644 --- a/packages/utils/src/normalize.ts +++ b/packages/utils/src/normalize.ts @@ -29,7 +29,7 @@ type ObjOrArray = { [key: string]: T }; * @param input The object to be normalized. * @param depth The max depth to which to normalize the object. (Anything deeper stringified whole.) * @param maxProperties The max number of elements or properties to be included in any single array or - * object in the normallized output. + * object in the normalized output. * @returns A normalized version of the object, or `"**non-serializable**"` if any errors are thrown during normalization. */ // eslint-disable-next-line @typescript-eslint/no-explicit-any diff --git a/packages/utils/src/object.ts b/packages/utils/src/object.ts index 0ff9b32da402..e0424f6aa120 100644 --- a/packages/utils/src/object.ts +++ b/packages/utils/src/object.ts @@ -98,7 +98,7 @@ export function urlEncode(object: { [key: string]: any }): string { * non-enumerable properties attached. * * @param value Initial source that we have to transform in order for it to be usable by the serializer - * @returns An Event or Error turned into an object - or the value argurment itself, when value is neither an Event nor + * @returns An Event or Error turned into an object - or the value argument itself, when value is neither an Event nor * an Error. */ export function convertToPlainObject( diff --git a/packages/utils/test/string.test.ts b/packages/utils/test/string.test.ts index bb49a7833a11..64870c07fcba 100644 --- a/packages/utils/test/string.test.ts +++ b/packages/utils/test/string.test.ts @@ -38,7 +38,7 @@ describe('isMatchingPattern()', () => { expect(isMatchingPattern('foobar', 'nope', true)).toEqual(false); }); - test('matches when `value` constains `pattern` but not vice-versa', () => { + test('matches when `value` contains `pattern` but not vice-versa', () => { expect(isMatchingPattern('foobar', 'foo')).toEqual(true); expect(isMatchingPattern('foobar', 'foobarbaz')).toEqual(false); }); @@ -83,7 +83,7 @@ describe('stringMatchesSomePattern()', () => { expect(stringMatchesSomePattern('foobar', ['baz', 'nope'], true)).toEqual(false); }); - test('matches when `testString` constains a pattern but not vice-versa', () => { + test('matches when `testString` contains a pattern but not vice-versa', () => { expect(stringMatchesSomePattern('foobar', ['foo', 'nope'])).toEqual(true); expect(stringMatchesSomePattern('foobar', ['foobarbaz', 'nope'])).toEqual(false); }); diff --git a/packages/utils/test/syncpromise.test.ts b/packages/utils/test/syncpromise.test.ts index 53dff943fa48..b3074b3c1b32 100644 --- a/packages/utils/test/syncpromise.test.ts +++ b/packages/utils/test/syncpromise.test.ts @@ -145,7 +145,7 @@ describe('SyncPromise', () => { expect(foo).toEqual(2); }); - test('calling the callback not immediatly', () => { + test('calling the callback not immediately', () => { jest.useFakeTimers(); expect.assertions(4); @@ -226,7 +226,7 @@ describe('SyncPromise', () => { }); }); - test('reject immediatly and do not call then', async () => { + test('reject immediately and do not call then', async () => { expect.assertions(1); return new SyncPromise((_, reject) => { diff --git a/packages/utils/test/url.test.ts b/packages/utils/test/url.test.ts index d58f7cf205a8..c6926fb74968 100644 --- a/packages/utils/test/url.test.ts +++ b/packages/utils/test/url.test.ts @@ -24,8 +24,8 @@ describe('stripQueryStringAndFragment', () => { describe('getNumberOfUrlSegments', () => { test.each([ ['regular path', '/projects/123/views/234', 4], - ['single param paramaterized path', '/users/:id/details', 3], - ['multi param paramaterized path', '/stores/:storeId/products/:productId', 4], + ['single param parameterized path', '/users/:id/details', 3], + ['multi param parameterized path', '/stores/:storeId/products/:productId', 4], ['regex path', String(/\/api\/post[0-9]/), 2], ])('%s', (_: string, input, output) => { expect(getNumberOfUrlSegments(input)).toEqual(output); diff --git a/packages/vercel-edge/package.json b/packages/vercel-edge/package.json index b80c0c279668..bc36f5880949 100644 --- a/packages/vercel-edge/package.json +++ b/packages/vercel-edge/package.json @@ -1,7 +1,7 @@ { "name": "@sentry/vercel-edge", "version": "8.36.0", - "description": "Offical Sentry SDK for the Vercel Edge Runtime", + "description": "Official Sentry SDK for the Vercel Edge Runtime", "repository": "git://github.com/getsentry/sentry-javascript.git", "homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/vercel-edge", "author": "Sentry", diff --git a/tsconfig-templates/README.md b/tsconfig-templates/README.md index 8c007dcbb7c2..f1b29fb1b3e9 100644 --- a/tsconfig-templates/README.md +++ b/tsconfig-templates/README.md @@ -3,5 +3,5 @@ Every package should get its own copy of the three files in this directory and the one in `test/` (which should go in an analogous spot in the package). Package-specific options should go in `tsconfig.json` and test-specific options in `tsconfig.test.json`. The `types` file shouldn't need to be modified, and only exists because tsconfigs don't support -multiple inheritence. The same goes for the file in `test/`, which only exists because VSCode only knows to look for a +multiple inheritance. The same goes for the file in `test/`, which only exists because VSCode only knows to look for a file named (exactly) `tsconfig.json`.