Skip to content

chore(tests): Use verdaccio as node process instead of docker image - #20336

Merged
mydea merged 12 commits into
developfrom
fn/node-verdaccio
Apr 17, 2026
Merged

chore(tests): Use verdaccio as node process instead of docker image#20336
mydea merged 12 commits into
developfrom
fn/node-verdaccio

Conversation

@mydea

@mydeamydea commented Apr 15, 2026•
edited
Loading

Copy link
Copy Markdown
Member

Extracted out from #20270

This rewrites how we run verdaccio to run this as node process instead of via docker.

For this, I pinned the latest verdaccio version in the e2e package, and run it via node APIs as a detached process. This also means no more docker requirement to run this stuff locally. Works both with yarn test:run xxx for local usage as well as with yarn test:prepare for CI usage.

@mydeamydea self-assigned this Apr 15, 2026
@github-actions

github-actionsBot commented Apr 15, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor

size-limit report 📦

PathSize% ChangeChange
@sentry/browser25.78 kB--
@sentry/browser - with treeshaking flags24.27 kB--
@sentry/browser (incl. Tracing)43.61 kB--
@sentry/browser (incl. Tracing + Span Streaming)45.32 kB--
@sentry/browser (incl. Tracing, Profiling)48.51 kB--
@sentry/browser (incl. Tracing, Replay)82.74 kB--
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags72.25 kB--
@sentry/browser (incl. Tracing, Replay with Canvas)87.43 kB--
@sentry/browser (incl. Tracing, Replay, Feedback)99.67 kB--
@sentry/browser (incl. Feedback)42.59 kB--
@sentry/browser (incl. sendFeedback)30.45 kB--
@sentry/browser (incl. FeedbackAsync)35.45 kB--
@sentry/browser (incl. Metrics)27.07 kB--
@sentry/browser (incl. Logs)27.2 kB--
@sentry/browser (incl. Metrics & Logs)27.89 kB--
@sentry/react27.53 kB--
@sentry/react (incl. Tracing)45.88 kB--
@sentry/vue30.61 kB--
@sentry/vue (incl. Tracing)45.45 kB--
@sentry/svelte25.8 kB--
CDN Bundle28.46 kB--
CDN Bundle (incl. Tracing)44.69 kB--
CDN Bundle (incl. Logs, Metrics)29.83 kB--
CDN Bundle (incl. Tracing, Logs, Metrics)45.78 kB--
CDN Bundle (incl. Replay, Logs, Metrics)68.73 kB--
CDN Bundle (incl. Tracing, Replay)81.65 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)82.73 kB--
CDN Bundle (incl. Tracing, Replay, Feedback)87.17 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)88.23 kB--
CDN Bundle - uncompressed83.12 kB--
CDN Bundle (incl. Tracing) - uncompressed133.64 kB--
CDN Bundle (incl. Logs, Metrics) - uncompressed87.27 kB--
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed137.05 kB--
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed210.63 kB--
CDN Bundle (incl. Tracing, Replay) - uncompressed250.87 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed254.27 kB--
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed263.78 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed267.17 kB--
@sentry/nextjs (client)48.42 kB--
@sentry/sveltekit (client)44.06 kB--
@sentry/node-core57.94 kB+0.02%+6 B 🔺
@sentry/node174.77 kB-0.01%-9 B 🔽
@sentry/node - without tracing97.89 kB+0.03%+21 B 🔺
@sentry/aws-serverless115.12 kB+0.01%+8 B 🔺

View base workflow run

mydea
mydea commented Apr 16, 2026
View reviewed changes
Comment thread.github/actions/install-dependencies/action.yml

- name: Check dependency cache
uses: actions/cache@v4
uses: actions/cache@v5

@mydeamydeaApr 16, 2026

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

leftover I noticed I forgot to bump, not really related but this is non-breaking for us.

@mydea
mydea requested review from Lms24, isaacs and logaretmApril 16, 2026 13:09
@mydea
mydea marked this pull request as ready for review April 16, 2026 13:10
cursor[bot]
cursorBot reviewed Apr 16, 2026
View reviewed changes
Comment threaddev-packages/e2e-tests/lib/constants.ts
sentry[bot]
sentryBot reviewed Apr 16, 2026
View reviewed changes
Comment threaddev-packages/e2e-tests/registrySetup.ts
cursor[bot]
cursorBot reviewed Apr 16, 2026
View reviewed changes
Comment threaddev-packages/e2e-tests/registrySetup.ts
cursor[bot]
cursorBot reviewed Apr 16, 2026
View reviewed changes

@cursorcursorBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 64cbb91. Configure here.

Comment threaddev-packages/e2e-tests/verdaccio-runner.mjs
@mydea
mydea merged commit 313cf82 into developApr 17, 2026
706 of 711 checks passed
@mydea
mydea deleted the fn/node-verdaccio branch April 17, 2026 07:18
andreiborza pushed a commit that referenced this pull request Apr 20, 2026
@mydea@andreiborza
chore(tests): Use verdaccio as node process instead of docker image (#…
c6859d8
…20336)
Extracted out from
#20270
This rewrites how we run verdaccio to run this as node process instead
of via docker.
For this, I pinned the latest verdaccio version in the e2e package, and
run it via node APIs as a detached process. This also means no more
docker requirement to run this stuff locally. Works both with `yarn
test:run xxx` for local usage as well as with `yarn test:prepare` for CI
usage.
@sentry-release-botsentry-release-botBot mentioned this pull request Apr 23, 2026
Closed
46 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sentrysentry[bot]sentry[bot] left review comments
@cursorcursor[bot]cursor[bot] left review comments
@logaretmlogaretmlogaretm approved these changes
@isaacsisaacsAwaiting requested review from isaacs
@Lms24Lms24Awaiting requested review from Lms24

Assignees

@mydeamydea

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mydea@logaretm