Skip to content

[codex] Add rrweb-snapshot utility boundaries#1838

Merged
Juice10 merged 6 commits into
masterfrom
codex/snapshot-rebuild-utils-boundary
May 20, 2026
Merged

[codex] Add rrweb-snapshot utility boundaries#1838
Juice10 merged 6 commits into
masterfrom
codex/snapshot-rebuild-utils-boundary

Conversation

@Juice10

@Juice10 Juice10 commented May 19, 2026

Copy link
Copy Markdown
Member

Summary

  • add snapshot-domain and rebuild-domain utility entrypoints in rrweb-snapshot
  • route snapshot and rebuild internals through those entrypoints without changing imported symbols
  • keep the legacy utils export public and document the compatibility path

Validation

@changeset-bot

changeset-bot Bot commented May 19, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: b46b6c9

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

@github-actions

Copy link
Copy Markdown
Contributor

Bundle Size Changes

No bundle size changes.

@eoghanmurray

Copy link
Copy Markdown
Contributor

Approving on the assumption that this is a good thing to have already present in the 2.0 release, but otherwise we should skip the noise and implement #1839 instead

eoghanmurray
eoghanmurray previously approved these changes May 20, 2026
@Juice10 Juice10 changed the base branch from codex/record-tree-shaking-postcss to master May 20, 2026 09:49
@Juice10 Juice10 dismissed eoghanmurray’s stale review May 20, 2026 09:49

The base branch was changed.

@Juice10 Juice10 merged commit a5bb326 into master May 20, 2026
11 checks passed
@Juice10 Juice10 deleted the codex/snapshot-rebuild-utils-boundary branch May 20, 2026 11:36
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.

2 participants