Skip to content

fix(structuredClone) - remove warning#255

Merged
gabrielseco merged 1 commit into
mainfrom
remove-warning-about-structureClone
May 25, 2026
Merged

fix(structuredClone) - remove warning#255
gabrielseco merged 1 commit into
mainfrom
remove-warning-about-structureClone

Conversation

@gabrielseco
Copy link
Copy Markdown
Collaborator

@gabrielseco gabrielseco commented May 25, 2026

I have users telling me this warning adds noise as they think they could have an error


Note

Low Risk
Logging-only change in a clone helper; clone and fallback behavior are unchanged.

Overview
Removes the console.warn when structuredClone throws in safeDeepClone, so the existing JSON parse/stringify fallback runs silently instead of looking like an application error to users.

Behavior is unchanged: still tries structuredClone first, then falls back on failure.

Reviewed by Cursor Bugbot for commit 108291b. Bugbot is set up for automated code reviews on this repo. Configure here.

@gabrielseco gabrielseco self-assigned this May 25, 2026
@gabrielseco gabrielseco marked this pull request as ready for review May 25, 2026 07:21
@sandrina-p sandrina-p self-requested a review May 25, 2026 08:56
@gabrielseco gabrielseco merged commit f2769e3 into main May 25, 2026
6 checks passed
@gabrielseco gabrielseco deleted the remove-warning-about-structureClone branch May 25, 2026 08:58
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