8000 Merge pull request #10923 from getsentry/prepare-release/8.0.0-alpha.2 · jonator/sentry-javascript@ad50342 · GitHub
[go: up one dir, main page]
Skip to content

Commit ad50342

Browse files
mydeamydea
authored
Merge pull request getsentry#10923 from getsentry/prepare-release/8.0.0-alpha.2
meta(changelog): Update changelog for v8.0.0-alpha.2
2 parents b76033d + deb9da8 commit ad50342

File tree

187 files changed

+1575
-7690
lines changed
  • propagator.test.ts
  • trace.test.ts
  • utils
  • opentelemetry-node
  • profiling-node/test
  • react
  • remix
  • replay
  • svelte/test
  • sveltekit/src/vite
  • tracing-internal/src
  • types/src
  • utils/src/vendor
  • vercel-edge
  • vue/src
  • scripts
  • yarn.lock

    • Some content is hidden

    Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

    187 files changed

    +1575
    -7690
    lines changed

    ‎.craft.yml

    Lines changed: 0 additions & 5 deletions
    Original file line numberDiff line numberDiff line change
    @@ -91,9 +91,6 @@ targets:
    9191
    - name: npm
    9292
    id: '@sentry/serverless'
    9393
    includeNames: /^sentry-serverless-\d.*\.tgz$/
    94-
    - name: npm
    95-
    id: '@sentry/opentelemetry-node'
    96-
    includeNames: /^sentry-opentelemetry-node-\d.*\.tgz$/
    9794
    - name: npm
    9895
    id: '@sentry/bun'
    9996
    includeNames: /^sentry-bun-\d.*\.tgz$/
    @@ -194,8 +191,6 @@ targets:
    194191
    onlyIfPresent: /^sentry-svelte-\d.*\.tgz$/
    195192
    'npm:@sentry/sveltekit':
    196193
    onlyIfPresent: /^sentry-sveltekit-\d.*\.tgz$/
    197-
    'npm:@sentry/opentelemetry-node':
    198-
    onlyIfPresent: /^sentry-opentelemetry-node-\d.*\.tgz$/
    199194
    'npm:@sentry/bun':
    200195
    onlyIfPresent: /^sentry-bun-\d.*\.tgz$/
    201196
    'npm:@sentry/vercel-edge':

    ‎.github/ISSUE_TEMPLATE/bug.yml

    Lines changed: 1 addition & 2 deletions
    Original file line numberDiff line numberDiff line change
    @@ -31,7 +31,7 @@ body:
    3131
    setup.
    3232
    options:
    3333
    - '@sentry/browser'
    34-
    - '@sentry/astro'
    34+
    - '@sentry/astro'
    3535
    - '@sentry/angular'
    3636
    - '@sentry/angular-ivy'
    3737
    - '@sentry/bun'
    @@ -40,7 +40,6 @@ body:
    4040
    - '@sentry/gatsby'
    4141
    - '@sentry/nextjs'
    4242
    - '@sentry/node'
    43-
    - '@sentry/opentelemetry-node'
    4443
    - '@sentry/react'
    4544
    - '@sentry/remix'
    4645
    - '@sentry/serverless'

    ‎.github/workflows/build.yml

    Lines changed: 16 additions & 17 deletions
    Original file line numberDiff line numberDiff line change
    @@ -49,7 +49,7 @@ env:
    4949
    ${{ github.workspace }}/packages/utils/esm
    5050
    5151
    BUILD_CACHE_KEY: build-cache-${{ github.event.inputs.commit || github.sha }}
    52-
    BUILD_PROFILING_NODE_CACHE_TARBALL_KEY: profiling-node-tarball-${{ github.event.inputs.commit || github.sha }}
    52+
    BUILD_CACHE_TARBALL_KEY: tarball-${{ github.event.inputs.commit || github.sha }}
    5353

    5454
    # GH will use the first restore-key it finds that matches
    5555
    # So it will start by looking for one from the same branch, else take the newest one it can find elsewhere
    @@ -83,7 +83,7 @@ jobs:
    8383
    echo "COMMIT_MESSAGE=$(git log -n 1 --pretty=format:%s $COMMIT_SHA)" >> $GITHUB_ENV
    8484
    8585
    - name: Determine changed packages
    86-
    uses: dorny/paths-filter@v3.0.0
    86+
    uses: dorny/paths-filter@v3.0.1
    8787
    id: changed
    8888
    with:
    8989
    filters: |
    @@ -402,15 +402,15 @@ jobs:
    402402
    uses: ./.github/actions/restore-cache
    403403
    env:
    404404
    DEPENDENCY_CACHE_KEY: ${{ needs.job_build.outputs.dependency_cache_key }}
    405-
    - name: Pack tarballs
    406-
    # Profiling tarball is built separately as we assemble the precompiled binaries
    407-
    run: yarn build:tarball --ignore @sentry/profiling-node
    408405

    409-
    - name: Restore profiling tarball
    410-
    uses: actions/cache/restore@v4
    406+
    - name: Extract Profiling Node Prebuilt Binaries
    407+
    uses: actions/download-artifact@v3
    411408
    with:
    412-
    key: ${{ env.BUILD_PROFILING_NODE_CACHE_TARBALL_KEY }}
    413-
    path: ${{ github.workspace }}/packages/*/*.tgz
    409+
    name: profiling-node-binaries-${{ github.sha }}
    410+
    path: ${{ github.workspace }}/packages/profiling-node/lib/
    411+
    412+
    - name: Pack tarballs
    413+
    run: yarn build:tarball
    414414

    415415
    - name: Archive artifacts
    416416
    uses: actions/upload-artifact@v4
    @@ -985,8 +985,6 @@ jobs:
    985985
    key: nx-Linux-${{ github.ref }}-${{ env.HEAD_COMMIT }}
    986986
    # On develop branch, we want to _store_ the cache (so it can be used by other branches), but never _restore_ from it
    987987
    restore-keys: ${{ env.NX_CACHE_RESTORE_KEYS }}
    988-
    - name: Build tarballs
    989-
    run: yarn build:tarball --ignore @sentry/profiling-node
    990988

    991989
    # Rebuild profiling by compiling TS and pull the precompiled binary artifacts
    992990
    - name: Build Profiling Node
    @@ -1001,21 +999,22 @@ jobs:
    1001999
    # https://github.com/actions/upload-artifact/issues/478
    10021000
    if: |
    10031001
    (needs.job_get_metadata.outputs.changed_profiling_node_bindings == 'true') ||
    1002+
    (needs.job_get_metadata.outputs.is_release == 'true') ||
    10041003
    (github.event_name != 'pull_request')
    10051004
    uses: actions/download-artifact@v3
    10061005
    with:
    10071006
    name: profiling-node-binaries-${{ github.sha }}
    10081007
    path: ${{ github.workspace }}/packages/profiling-node/lib/
    10091008

    10101009
    - name: Build Profiling tarball
    1011-
    run: yarn build:tarball --scope @sentry/profiling-node
    1010+
    run: yarn build:tarball
    10121011
    # End rebuild profiling
    10131012

    10141013
    - name: Stores tarballs in cache
    10151014
    uses: actions/cache/save@v4
    10161015
    with:
    10171016
    path: ${{ github.workspace }}/packages/*/*.tgz
    1018-
    key: ${{ env.BUILD_PROFILING_NODE_CACHE_TARBALL_KEY }}
    1017+
    key: ${{ env.BUILD_CACHE_TARBALL_KEY }}
    10191018

    10201019
    job_e2e_tests:
    10211020
    name: E2E ${{ matrix.label || matrix.test-application }} Test
    @@ -1111,7 +1110,7 @@ jobs:
    11111110
    uses: actions/cache/restore@v4
    11121111
    with:
    11131112
    path: ${{ github.workspace }}/packages/*/*.tgz
    1114-
    key: ${{ env.BUILD_PROFILING_NODE_CACHE_TARBALL_KEY }}
    1113+
    key: ${{ env.BUILD_CACHE_TARBALL_KEY }}
    11151114

    11161115
    - name: Get node version
    11171116
    id: versions
    @@ -1202,13 +1201,13 @@ jobs:
    12021201
    with:
    12031202
    name: profiling-node-binaries-${{ github.sha }}
    12041203
    path: ${{ github.workspace }}/packages/profiling-node/lib/
    1205-
    - name: Build Profiling tarball
    1206-
    run: yarn build:tarball --scope @sentry/profiling-node
    1204+
    12071205
    - name: Restore tarball cache
    12081206
    uses: actions/cache/restore@v4
    12091207
    with:
    12101208
    path: ${{ github.workspace }}/packages/*/*.tgz
    1211-
    key: ${{ env.BUILD_PROFILING_NODE_CACHE_TARBALL_KEY }}
    1209+
    key: ${{ env.BUILD_CACHE_TARBALL_KEY }}
    1210+
    fail-on-cache-miss : true
    12121211

    12131212
    - name: Get node version
    12141213
    id: versions

    ‎.github/workflows/canary.yml

    Lines changed: 2 additions & 2 deletions
    Original file line numberDiff line numberDiff line change
    @@ -135,7 +135,7 @@ jobs:
    135135

    136136
    - name: Create Issue
    137137
    if: failure() && github.event_name == 'schedule'
    138-
    uses: JasonEtco/create-an-issue@e27dddc79c92bc6e4562f268fffa5ed752639abd
    138+
    uses: JasonEtco/create-an-issue@1b14a70e4d8dc185e5cc76d3bec9eab20257b2c5
    139139
    env:
    140140
    GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
    141141
    RUN_LINK: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}
    @@ -176,7 +176,7 @@ jobs:
    176176
    177177
    - name: Create Issue
    178178
    if: failure() && github.event_name == 'schedule'
    179-
    uses: JasonEtco/create-an-issue@e27dddc79c92bc6e4562f268fffa5ed752639abd
    179+
    uses: JasonEtco/create-an-issue@1b14a70e4d8dc185e5cc76d3bec9eab20257b2c5
    180180
    env:
    181181
    GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
    182182
    RUN_LINK: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}

    ‎.github/workflows/flaky-test-detector.yml

    Lines changed: 1 addition & 1 deletion
    Original file line numberDiff line numberDiff line change
    @@ -71,7 +71,7 @@ jobs:
    7171
    run: npx playwright install-deps
    7272

    7373
    - name: Determine changed tests
    74-
    uses: dorny/paths-filter@v3.0.0
    74+
    uses: dorny/paths-filter@v3.0.1
    7575
    id: changed
    7676
    with:
    7777
    list-files: json

    ‎.github/workflows/issue-package-label.yml

    Lines changed: 0 additions & 3 deletions
    Original file line numberDiff line numberDiff line change
    @@ -56,9 +56,6 @@ jobs:
    5656
    "@sentry.node": {
    5757
    "label": "Package: Node"
    5858
    },
    59-
    "@sentry.opentelemetry-node": {
    60-
    "label": "Package: otel-node"
    61-
    },
    6259
    "@sentry.react": {
    6360
    "label": "Package: react"
    6461
    },

    ‎CHANGELOG.md

    Lines changed: 33 additions & 0 deletions
    Original file line numberDiff line numberDiff line change
    @@ -4,6 +4,39 @@
    44

    55
    - "You miss 100 percent of the chances you don't take. — Wayne Gretzky" — Michael Scott
    66

    7+
    ## 8.0.0-alpha.2
    8+
    9+
    This alpha release fixes a build problem that prevented 8.0.0-alpha.1 from being properly released.
    10+
    11+
    ### Important Changes
    12+
    13+
    - **feat: Remove `@sentry/opentelemetry-node` package (#10906)**
    14+
    15+
    The `@sentry/opentelemetry-node` package has been removed. Instead, you can either use `@sentry/node` with built-in
    16+
    OpenTelemetry support, or use `@sentry/opentelemetry` to manually connect Sentry with OpenTelemetry.
    17+
    18+
    ### Removal/Refactoring of deprecated functionality
    19+
    20+
    - ref: Refactor some deprecated `startSpan` options (#10825)
    21+
    - feat(v8/core): remove void from transport return (#10794)
    22+
    - ref(integrations): Delete deprecated class integrations (#10887)
    23+
    24+
    ### Other Changes
    25+
    26+
    - feat(core): Use serialized spans in transaction event (#10912)
    27+
    - feat(deps): bump @sentry/cli from 2.28.6 to 2.29.1 (#10908)
    28+
    - feat(node): Allow to configure `skipOpenTelemetrySetup` (#10907)
    29+
    - feat(esm): Import rather than require `inspector` (#10910)
    30+
    - fix(browser): Don't use chrome variable name (#10874)
    31+
    - chore(sveltekit): Fix punctuation in a console.log (#10895)
    32+
    - fix(opentelemetry): Ensure DSC propagation works correctly (#10904)
    33+
    - feat(browser): Exclude span exports from non-performance CDN bundles (#10879)
    34+
    - ref: Refactor span status handling to be OTEL compatible (#10871)
    35+
    - feat(core): Fix span scope handling & transaction setting (#10886)
    36+
    - ref(ember): Avoid namespace import to hopefully resolve minification issue (#10885)
    37+
    38+
    Work in this release contributed by @harish-talview & @bfontaine. Thank you for your contributions!
    39+
    740
    ## 8.0.0-alpha.1
    841

    942
    This is the first Alpha release of the v8 cycle, which includes a variety of breaking changes.

    ‎MIGRATION.md

    Lines changed: 19 additions & 1 deletion
    Original file line numberDiff line numberDiff line change
    @@ -118,7 +118,7 @@ Sentry.init({
    118118
    tracesSampleRate: 1.0,
    119119
    });
    120120

    121-
    // Before (v8)
    121+
    // After (v8)
    122122
    const Sentry = require('@sentry/node');
    123123

    124124
    Sentry.init({
    @@ -272,6 +272,7 @@ Removed top-level exports: `tracingOrigins`, `MetricsAggregator`, `metricsAggreg
    272272
    - [Removal of `spanStatusfromHttpCode` in favour of `getSpanStatusFromHttpCode`](./MIGRATION.md#removal-of-spanstatusfromhttpcode-in-favour-of-getspanstatusfromhttpcode)
    273273
    - [Removal of `addGlobalEventProcessor` in favour of `addEventProcessor`](./MIGRATION.md#removal-of-addglobaleventprocessor-in-favour-of-addeventprocessor)
    274274
    - [Removal of `lastEventId()` method](./MIGRATION.md#deprecate-lasteventid)
    275+
    - [Remove `void` from transport return types](./MIGRATION.md#remove-void-from-transport-return-types)
    275276

    276277
    #### Deprecation of `Hub` and `getCurrentHub()`
    277278

    @@ -435,6 +436,23 @@ addEventProcessor(event => {
    435436

    436437
    The `lastEventId` function has been removed. See [below](./MIGRATION.md#deprecate-lasteventid) for more details.
    437438

    439+
    #### Remove `void` from transport return types
    440+
    441+
    The `send` method on the `Transport` interface now always requires a `TransportMakeRequestResponse` to be returned in
    442+
    the promise. This means that the `void` return type is no longer allowed.
    443+
    444+
    ```ts
    445+
    // Before (v7)
    446+
    interface Transport {
    447+
    send(event: Event): Promise<void | TransportMakeRequestResponse>;
    448+
    }
    449+
    450+
    // After (v8)
    451+
    interface Transport {
    452+
    send(event: Event): Promise<TransportMakeRequestResponse>;
    453+
    }
    454+
    ```
    455+
    438456
    ### Browser SDK (Browser, React, Vue, Angular, Ember, etc.)
    439457

    440458
    Removed top-level exports: `Offline`, `makeXHRTransport`, `BrowserTracing`

    ‎dev-packages/browser-integration-tests/suites/integrations/Breadcrumbs/dom/init.js

    Lines changed: 1 addition & 1 deletion
    Original file line numberDiff line numberDiff line change
    @@ -5,6 +5,6 @@ window.Sentry = Sentry;
    55
    Sentry.init({
    66
    dsn: 'https://public@dsn.ingest.sentry.io/1337',
    77
    defaultIntegrations: false,
    8-
    integrations: [new Sentry.Integrations.Breadcrumbs()],
    8+
    integrations: [Sentry.breadcrumbsIntegration()],
    99
    sampleRate: 1,
    1010
    });

    ‎dev-packages/browser-integration-tests/suites/integrations/Breadcrumbs/fetch/init.js

    Lines changed: 1 addition & 1 deletion
    Original file line numberDiff line numberDiff line change
    @@ -5,6 +5,6 @@ window.Sentry = Sentry;
    55
    Sentry.init({
    66
    dsn: 'https://public@dsn.ingest.sentry.io/1337',
    77
    defaultIntegrations: false,
    8-
    integrations: [new Sentry.Integrations.Breadcrumbs()],
    8+
    integrations: [Sentry.breadcrumbsIntegration()],
    99
    sampleRate: 1,
    1010
    });

    ‎dev-packages/browser-integration-tests/suites/integrations/Breadcrumbs/xhr/init.js

    Lines changed: 1 addition & 1 deletion
    Original file line numberDiff line numberDiff line change
    @@ -5,6 +5,6 @@ window.Sentry = Sentry;
    55
    Sentry.init({
    66
    dsn: 'https://public@dsn.ingest.sentry.io/1337',
    77
    defaultIntegrations: false,
    8-
    integrations: [new Sentry.Integrations.Breadcrumbs()],
    8+
    integrations: [Sentry.breadcrumbsIntegration()],
    99
    sampleRate: 1,
    1010
    });

    ‎dev-packages/browser-integration-tests/suites/manual-client/browser-context/init.js

    Lines changed: 10 additions & 10 deletions
    Original file line numberDiff line numberDiff line change
    @@ -1,23 +1,23 @@
    11
    import {
    2-
    Breadcrumbs,
    32
    BrowserClient,
    4-
    FunctionToString,
    5-
    HttpContext,
    63
    Hub,
    7-
    InboundFilters,
    8-
    LinkedErrors,
    4+
    breadcrumbsIntegration,
    95
    dedupeIntegration,
    106
    defaultStackParser,
    7+
    functionToStringIntegration,
    8+
    httpContextIntegration,
    9+
    inboundFiltersIntegration,
    10+
    linkedErrorsIntegration,
    1111
    makeFetchTransport,
    1212
    } from '@sentry/browser';
    1313

    1414
    const integrations = [
    15-
    new Breadcrumbs(),
    16-
    new FunctionToString(),
    15+
    breadcrumbsIntegration(),
    16+
    functionToStringIntegration(),
    1717
    dedupeIntegration(),
    18-
    new HttpContext(),
    19-
    new InboundFilters(),
    20-
    new LinkedErrors(),
    18+
    httpContextIntegration(),
    19+
    inboundFiltersIntegration(),
    20+
    linkedErrorsIntegration(),
    2121
    ];
    2222

    2323
    const client = new BrowserClient({

    ‎dev-packages/browser-integration-tests/suites/public-api/startSpan/basic/test.ts

    Lines changed: 3 additions & 3 deletions
    Original file line numberDiff line numberDiff line change
    @@ -1,5 +1,5 @@
    11
    import { expect } from '@playwright/test';
    2-
    import type { SerializedEvent } from '@sentry/types';
    2+
    import type { Event } from '@sentry/types';
    33

    44
    import { sentryTest } from '../../../../utils/fixtures';
    55
    import { getFirstSentryEnvelopeRequest, shouldSkipTracingTest } from '../../../../utils/helpers';
    @@ -10,7 +10,7 @@ sentryTest('should send a transaction in an envelope', async ({ getLocalTestPath
    1010
    }
    1111

    1212
    const url = await getLocalTestPath({ testDir: __dirname });
    13-
    const transaction = await getFirstSentryEnvelopeRequest<SerializedEvent>(page, url);
    13+
    const transaction = await getFirstSentryEnvelopeRequest<Event>(page, url);
    1414

    1515
    expect(transaction.transaction).toBe('parent_span');
    1616
    expect(transaction.spans).toBeDefined();
    @@ -22,7 +22,7 @@ sentryTest('should report finished spans as children of the root transaction', a
    2222
    }
    2323

    2424
    const url = await getLocalTestPath({ testDir: __dirname });
    25-
    const transaction = await getFirstSentryEnvelopeRequest<SerializedEvent>(page, url);
    25+
    const transaction = await getFirstSentryEnvelopeRequest<Event>(page, url);
    2626

    2727
    expect(transaction.spans).toHaveLength(1);
    2828

    ‎dev-packages/browser-integration-tests/suites/public-api/startTransaction/basic_usage/test.ts

    Lines changed: 3 additions & 3 deletions
    Original file line numberDiff line numberDiff line change
    @@ -1,5 +1,5 @@
    11
    import { expect } from '@playwright/test';
    2-
    import type { SerializedEvent } from '@sentry/types';
    2+
    import type { Event } from '@sentry/types';
    33

    44
    import { sentryTest } from '../../../../utils/fixtures';
    55
    import { getFirstSentryEnvelopeRequest, shouldSkipTracingTest } from '../../../../utils/helpers';
    @@ -10,7 +10,7 @@ sentryTest('should report a transaction in an envelope', async ({ getLocalTestPa
    1010
    }
    1111

    1212
    const url = await getLocalTestPath({ testDir: __dirname });
    13-
    const transaction = await getFirstSentryEnvelopeRequest<SerializedEvent>(page, url);
    13+
    const transaction = await getFirstSentryEnvelopeRequest<Event>(page, url);
    1414

    1515
    expect(transaction.transaction).toBe('root_span');
    1616
    expect(transaction.spans).toBeDefined();
    @@ -22,7 +22,7 @@ sentryTest('should report finished spans as children of the root span', async ({
    2222
    }
    2323

    2424
    const url = await getLocalTestPath({ testDir: __dirname });
    25-
    const transaction = await getFirstSentryEnvelopeRequest<SerializedEvent>(page, url);
    25+
    const transaction = await getFirstSentryEnvelopeRequest<Event>(page, url);
    2626

    2727
    const rootSpanId = transaction?.contexts?.trace?.span_id;
    2828

    ‎dev-packages/browser-integration-tests/suites/replay/errors/errorModeCustomTransport/init.js

    Lines changed: 0 additions & 23 deletions
    This file was deleted.

    0 commit comments

    Comments
     (0)
    0