Skip to content

Commit 7e3207d

Browse files
authored
fix(wasm): Add @sentry/core as a dependency (#10283)
By using functional integrations in #10230, we started importing from `@sentry/core` in the WASM integration. However, we didn't register `@sentry/core` as a dependency, making rollup bundle core into the package output. This changed the `build/npm` directory structure, making our entry points in `package.json` invalid. This PR fixes things by simply registering core as a dependency of wasm. If we move WASM to core (which I strongly think we should do), we'll be able to get rid of this again.
1 parent 5f0b506 commit 7e3207d

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

.github/workflows/build.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -102,6 +102,7 @@ jobs:
102102
- 'packages/replay/**'
103103
- 'packages/replay-canvas/**'
104104
- 'packages/feedback/**'
105+
- 'packages/wasm/**'
105106
browser_integration:
106107
- *shared
107108
- *browser

packages/wasm/package.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,8 @@
3131
"dependencies": {
3232
"@sentry/browser": "7.94.1",
3333
"@sentry/types": "7.94.1",
34-
"@sentry/utils": "7.94.1"
34+
"@sentry/utils": "7.94.1",
35+
"@sentry/core": "7.94.1"
3536
},
3637
"scripts": {
3738
"build": "run-p build:transpile build:bundle build:types",

0 commit comments

Comments
 (0)