fix: memory leaks fix #1707

[maksimr]

Fixes #1707

[changeset-bot]

⚠️ No Changeset found

Latest commit: d1386ac

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[Copilot]

Pull Request Overview

This pull request fixes memory leaks by ensuring that observers and asynchronous tasks are properly torn down using AbortControllers and cleanup functions. Key changes include renaming and updating the mutation observer API, integrating AbortSignal across snapshot and mutation flows, and adding cleanup logic to abort pending operations.

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
packages/rrweb/src/record/shadow-dom-manager.ts	Renames observer variable to removeObserver for clarity and proper cleanup
packages/rrweb/src/record/observer.ts	Updates the API to return a teardown function and renames mutationObserver
packages/rrweb/src/record/mutation.ts	Introduces a new AbortController to cancel serialization and ensures cleanup
packages/rrweb/src/record/index.ts	Adds/reinitializes an AbortController before snapshot operations
packages/rrweb-snapshot/src/snapshot.ts	Adds AbortSignal support to various asynchronous load functions

Comments suppressed due to low confidence (2)

packages/rrweb-snapshot/src/snapshot.ts:1296

• Consider adding a comment to clarify why a new AbortController is created in the snapshot function, which will help future maintainers understand the design decision.

    signal = new AbortController().signal,

packages/rrweb/src/record/index.ts:380

• [nitpick] It would be helpful to document the lifecycle of the abortController here, explaining why abort() is called at this point to improve overall code maintainability.

    abortController.abort();

[maksimr]

@YunFeng0817 @Juice10 Hi, I hope you're doing well. Could you please look at this PR with timer and memory leaks. Thx! 🥂