Conversation
14b8623 to
0b4a538
Compare
bcea5f9 to
f409113
Compare
ae76979 to
cc1c580
Compare
|
I have not fixed the sideeffects being duplicated in our replay canvas integration, but I think it's fine to deal with them later after we are caught up with upstream |
| .turbo | ||
| dist | ||
| node_modules | ||
| yarn-error.log No newline at end of file |
There was a problem hiding this comment.
I think that's just github showing a newline being removed or something
There was a problem hiding this comment.
oh interesting, never seen that before :O
There was a problem hiding this comment.
Turns out it is needed as the simple-peer-light is a library but does not supply its own types so our TS complains.
There was a problem hiding this comment.
same as above, should this file be ignored instead?
c298lee
left a comment
There was a problem hiding this comment.
took a quick glance over and mostly lgtm, but there are a couple .d.ts files, should those be ignored?
chargome
left a comment
There was a problem hiding this comment.
So much going on here 😄 but I looks good from what I can tell, left some comments!
64b8b63 to
37dcc9c
Compare
* Chore: Add move most types from rrweb to @rrweb/types package * Split off type imports * Split off type import to its own line * Get vite to generate type definitions * Apply formatting changes * noEmit not allowed in tsconfig, moved it to build step * Migrate rrdom-nodejs build to vite * Apply formatting changes * Migrate rrweb-snapshot to vite * Unify configs * Chore: Migrate rrdom to vite Turns out what we where doing by overwriting `public textContent: string | undefined` as a getter in a subclass is something that isn't allowed in typescript. Because we where using `// @ts-ignore` to hide this error our bundler chose to allow the overwrite. Vite choses to disallow the overwrite making all subclasses' `textContent` undefined. To mitigate this we're using an abstract class, which does allow sub classes to decide if they wan't to use getters or not. * Chore: Migrate rrweb to vite WIP * build:browser was removed (for now) * BREAKING: moved rrweb-plugin-console to its own npm module This removes console from rrweb-all.js * Support cjs files in startServer * Move canvas-webrtc plugin to its own package * Chore: move sequential-id plugin to its own package * Chore: Configure rrweb's vite bundling * `Id` had lowercase `d` before, making it lowercase again * Test: Move console tests to their own package * remove unused utils from rrdom * pull in latest version of master something when wrong earlier when resolving merge conflicts, this should be correct * Fix type casting issue in diff.ts * Fix typo * Fix duplicate entries in package.json and tsconfig.json * Apply formatting changes * Update dependencies in package.json files * Update dependencies to use Vite 5.2.8 in package.json files * Get tests passing for rrdom `apply virtual style rules to node` tests need to be moved to rrweb to avoid circular dependencies * Fix image loading issue in integration tests * Move pack/unpack to its own @rrweb/packer module * Get tests to work in rrdom-nodejs * Port tests in rrweb-snapshot to vitest and fix them * Fix tests for rrweb-plugin-console-record * Add @rrweb/all package * Fix publint and attw errors for rrdom and @rrweb/types * Use shared vitest.config.ts in rrweb-snapshot package * Fix publint and attw issues for rrweb-snapshot * Export `ReplayPlugin` type directly from rrweb * Fix publint and attw issues for packages * Fix publint & attw issue. I was bumping into this issue: https://github.com/arethetypeswrong/arethetypeswrong.github.io/blob/3729bc2a3ca2ef7dda5c22fef81f89e1abe5dacf/docs/problems/NoResolution.md And had to choose one of these three methods described here: https://github.com/andrewbranch/example-subpath-exports-ts-compat?tab=readme-ov-file#typescript-friendly-strategies-for-packagejson-subpath-exports-compatibility And I ended up going for the method described here: https://github.com/andrewbranch/example-subpath-exports-ts-compat/tree/1ffe3425b0a7ad8ecdf3c373f76f431ee341366b/examples/node_modules/package-json-redirects#package-json-redirects The redirect method seemed the least invasive and most effective. * Fix publint & attw issue. I was bumping into this issue: https://github.com/arethetypeswrong/arethetypeswrong.github.io/blob/3729bc2a3ca2ef7dda5c22fef81f89e1abe5dacf/docs/problems/NoResolution.md And had to choose one of these three methods described here: https://github.com/andrewbranch/example-subpath-exports-ts-compat?tab=readme-ov-file#typescript-friendly-strategies-for-packagejson-subpath-exports-compatibility And I ended up going for the method described here: https://github.com/andrewbranch/example-subpath-exports-ts-compat/tree/1ffe3425b0a7ad8ecdf3c373f76f431ee341366b/examples/node_modules/package-json-redirects#package-json-redirects The redirect method seemed the least invasive and most effective. * move some rrdom tests that require rrweb to rrweb package * Use pre-jest 29 syntax for snapshotting * get rrweb passing publint and attw * const enum does not work with isolated modules flag * Fix script tag type in webgl.test.ts.snap and update rrweb.umd.cjs path in webgl.test.ts * Fix paths * Move tests for console record plugin and fix bundle path * Fix tests for rrweb * pack integration tests were moved to @rrweb/all * Update rrweb bundle path in test files * Fix flaky scroll emit from test * Migrate rrweb's tests over to vitest and make them pass * Make sure benchmarks & updating tests work * Remove jest from rrweb * Fix paths * always use rrweb's own cssom * Update tsconfig.json for rrweb-plugin-sequential-id-record Fixes this error: Error: @rrweb/rrweb-plugin-sequential-id-record:prepublish: tsconfig.json(9,5): error TS6377: Cannot write file '/home/runner/work/rrweb/rrweb/tsconfig.tsbuildinfo' because it will overwrite '.tsbuildinfo' file generated by referenced project '/home/runner/work/rrweb/rrweb/packages/rrweb' * Add tsbuildinfo config to extended tsconfig files * Move rrdom over to vitest * Apply formatting changes * Update rrweb imports to use the new package structure * extend rrweb-snapshot's tsconfig from monorepo base config * extend @rrweb/types's tsconfig from monorepo base config * extend rrdom's tsconfig from monorepo base config * extend rrdom-nodejs's tsconfig from monorepo base config * extend web-extension's tsconfig from monorepo base config * unify tsconfigs * Continue when tests fail * Add stricter type checking * Add check-types global command * remove jest * Remove unused code * Add check-types command to build script * Fix linting issues * Add setup Chrome action for CI/CD workflow * Update puppeteer version in package.json for rrweb * Update Chrome setup in CI/CD workflow * Update Chrome setup in CI/CD workflow * Add Chrome setup and test cache location * Update CI/CD workflow to test chrome cache location * Add chrome installation step to CI/CD workflow * Update Puppeteer configuration for headless testing * Update dependencies and workflow configuration * Use same version of chrome on CI as is run locally * Use version of chrome that seems to work with rrdom tests * Try using puppeteerrc to define chrome version * Add .cache directory to .gitignore * Move global flag to vitest config * Update puppeteer version to 20.9.0 * Update console log messages in rrweb-plugin-console-record for new puppeteer version * Remove redundant Chrome setup from CI/CD workflow * Add minification and umd for all built files * Update import paths for rrweb dist files * Add @rrweb/replay and @rrweb/record * Add script to lint packages * Apply formatting changes * exclude styles export from typescript package type checking * WIP Move rrweb-player over to vite * Apply formatting changes * chore: Update rrweb plugin import paths * Remove rollup from rrweb-player * Fix typing issues * Fix typing issues * chore: Update rrweb-player to use vite for build process * Apply formatting changes * chore: Export Player class in rrweb-player/src/main.ts Makes attw happy * Apply formatting changes * Gets wiped by yarn workspaces-to-typescript-project-references * Add .eslintignore and .eslintrc.cjs files for rrweb-player package * Apply formatting changes * Update dependencies in rrweb-player/package.json * Apply formatting changes * chore: Update eslint configuration for rrweb-player package * Apply formatting changes * chore: Remove unused files from rrweb-player package * Apply formatting changes * chore: Update rrweb-player import path to use rrweb-player.cjs * chore: Update addEventListener signature in rrweb-player * Apply formatting changes * Add .eslintignore and update .gitignore files for to root * Apply formatting changes * Update documentation * Update @rrweb/types package description * Apply formatting changes * Update build and run commands in CONTRIBUTING.md * Apply formatting changes * Update package versions to 2.0.0-alpha.13 * Apply formatting changes * Apply formatting changes * Fix import statement in media/index.ts * Apply formatting changes * chore: Update .gitignore to exclude build and dist directories * Apply formatting changes * Apply formatting changes * Migrate setTimeout to vitest * Apply formatting changes * Apply formatting changes * Fix isNativeShadowDom function signature in utils.ts * try out jsr * Apply formatting changes * Update package versions to 2.0.0-alpha.14 * Apply formatting changes * Fix name of rrwebSnapshot object * Apply formatting changes * Remove unused lock files * Apply formatting changes * Update rrweb bundle path to use umd.cjs format * Apply formatting changes * Trigger tests to run again * Rename snapshots for vitest * Apply formatting changes * Ping CI * Apply formatting changes * Ping CI * Apply formatting changes * Ignore files generated by svelte-kit for prettier * Correct Player object
…deo tests more stable (rrweb-io#1500) * Chore: Ignore generated files from .svelte-kit for prettier * Create serious-eggs-greet.md * Apply formatting changes * Trigger CI * Make video tests more stable * Apply formatting changes * Exclude ambient.d.ts from format as it isn't deterministic
* Chore: upgrade turbo & fix yarn dev
37dcc9c to
0fdc341
Compare
f0e6a9f to
6cee8d0
Compare
## Summary Updates stale yarn.lock resolutions to patched versions within their existing semver ranges, plus fixes a build breakage in rrweb-worker. **Lockfile refreshes** (no package.json changes): - **basic-ftp** 5.0.5 -> 5.3.0 (via puppeteer -> proxy-agent -> get-uri) - **node-forge** 1.3.1 -> 1.4.0 (via vite-plugin-web-extension -> web-ext-run -> @devicefarmer/adbkit) - **picomatch** 2.3.1 -> 2.3.2 (via chokidar, micromatch, @rollup/pluginutils) - **picomatch** 4.0.2 -> 4.0.4 (via @rollup/pluginutils) - **flatted** 3.3.2 -> 3.4.2 (via eslint -> flat-cache) **rrweb-worker build fix**: Replaces `rollup-plugin-typescript2` with `@rollup/plugin-typescript` — the old plugin can't parse `import type` syntax with the newer rollup version pulled in by the `@rollup/plugin-terser` v1 bump. ## Breaking changes - Lockfile refreshes: none — all within existing semver ranges - `rollup-plugin-typescript2` -> `@rollup/plugin-typescript`: drop-in replacement, same rollup plugin API. `@rollup/plugin-typescript` is the officially maintained plugin. Config options (`tsconfig`, `sourceMap`, `inlineSourceMap`, `inlineSources`) are compatible. Build verified passing. ## Dependabot alerts resolved - [Alert #249](https://github.com/getsentry/rrweb/security/dependabot/249) (high) — basic-ftp CRLF injection - [Alert #202](https://github.com/getsentry/rrweb/security/dependabot/202) (critical) — basic-ftp path traversal - [Alert #237](https://github.com/getsentry/rrweb/security/dependabot/237) (high) — node-forge basicConstraints bypass - [Alert #236](https://github.com/getsentry/rrweb/security/dependabot/236) (high) — node-forge Ed25519 signature forgery - [Alert #235](https://github.com/getsentry/rrweb/security/dependabot/235) (high) — node-forge RSA-PKCS signature forgery - [Alert #234](https://github.com/getsentry/rrweb/security/dependabot/234) (high) — node-forge DoS via BigInteger.modInverse - [Alert #164](https://github.com/getsentry/rrweb/security/dependabot/164) (high) — node-forge ASN.1 unbounded recursion - [Alert #163](https://github.com/getsentry/rrweb/security/dependabot/163) (medium) — node-forge ASN.1 OID integer truncation - [Alert #162](https://github.com/getsentry/rrweb/security/dependabot/162) (high) — node-forge ASN.1 desynchronization - [Alert #232](https://github.com/getsentry/rrweb/security/dependabot/232) (medium) — picomatch POSIX class method injection - [Alert #230](https://github.com/getsentry/rrweb/security/dependabot/230) (medium) — picomatch POSIX class method injection (4.x) - [Alert #229](https://github.com/getsentry/rrweb/security/dependabot/229) (high) — picomatch ReDoS via extglob quantifiers - [Alert #227](https://github.com/getsentry/rrweb/security/dependabot/227) (high) — flatted prototype pollution via parse 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Pulls in the following upstream commits:
Consists mostly build/meta changes, but there are some type changes and code changes (mostly removed unused code) as well .