Skip to content

chore(rrvideo)!: Remove rrvideo package#276

Draft
chargome wants to merge 9 commits intosentry-v2from
cg/rm-rrvideo
Draft

chore(rrvideo)!: Remove rrvideo package#276
chargome wants to merge 9 commits intosentry-v2from
cg/rm-rrvideo

Conversation

@chargome
Copy link
Copy Markdown
Member

@chargome chargome commented Apr 1, 2026

We do not use this package internally and it just pulls in a bunch of deps that we do not need.

@chargome chargome self-assigned this Apr 1, 2026
chargome and others added 5 commits April 1, 2026 18:19
Puppeteer's bundled Chromium download was causing the yarn install
step to exceed the 10-minute timeout. Use browser-actions/setup-chrome
instead and point puppeteer at the system-installed Chrome binary.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Puppeteer's Chromium downloads need more time after the yarn.lock
changes in the vite/vitest bump. Revert the setup-chrome approach
since puppeteer v17/v20 need their own bundled Chromium revisions.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Vitest 2 dynamically imports jsdom even when tests don't use the
jsdom environment, causing unhandled errors that fail the test run.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
… timeout

jsdom 29 formats cssText with spaces inside braces (e.g.
`div { color: black; }` instead of `div {color: black;}`).
Use toContain for CSS assertions to be resilient to formatting.

Also increase timeout for the iframe stylesheet loading test which
is slow in CI.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant