Use structuredClone instead of remeda's clone#14351
Merged
thdxr merged 4 commits intoanomalyco:devfrom Feb 20, 2026
Merged
Conversation
Avoids pathological cases when cloning large objects Fixes anomalyco#14344
Contributor
|
Hey! Your PR title Please update it to start with one of:
Where See CONTRIBUTING.md for details. |
Contributor
|
The following comment was made by an LLM, it may be inaccurate: Potential Related PR Found: PR #6623: "refactor: use structuredClone instead of JSON parse/stringify" This PR is related because it also involves refactoring to use |
rekram1-node
pushed a commit
that referenced
this pull request
Feb 20, 2026
Co-authored-by: Dax Raad <d@ironbay.co> Co-authored-by: Dax <mail@thdxr.com>
lacymorrow
added a commit
to lacymorrow/lash
that referenced
this pull request
Feb 22, 2026
* upstream/dev: (476 commits) release: v1.2.9 fix: issue from structuredClone addition by using unwrap (anomalyco#14359) tweak(ui): adjust session feed spacing tweak(ui): tone down reasoning emphasis tweak(ui): refine session feed spacing fix(desktop): restore settings header mask tweak(ui): nudge edited files chevron tweak(ui): stabilize collapsible chevron hover fix: add missing id/sessionID/messageID to MCP tool attachments (anomalyco#14345) Use structuredClone instead of remeda's clone (anomalyco#14351) remove unnecessary deep clones from session loop and LLM stream (anomalyco#14354) fix(web): correct config import path in Korean enterprise docs docs: update providers layout and Windows sidebar label core: remove User-Agent header assertion from LLM test to fix failing test release: v1.2.8 fix(app): black screen on launch with sidecar server chore: generate feat(opencode): support adaptive thinking for claude sonnet 4.6 (anomalyco#14283) feat(tui): add custom tool and mcp call responses visible and collapsable (anomalyco#10649) chore: cleanup ... # Conflicts: # bun.lock # packages/app/package.json # packages/console/app/package.json # packages/console/core/package.json # packages/console/function/package.json # packages/console/mail/package.json # packages/desktop/package.json # packages/enterprise/package.json # packages/extensions/zed/extension.toml # packages/function/package.json # packages/opencode/package.json # packages/opencode/src/cli/cmd/tui/component/prompt/index.tsx # packages/opencode/src/cli/cmd/tui/routes/home.tsx # packages/opencode/src/config/config.ts # packages/opencode/src/tool/lsp.ts # packages/opencode/src/tool/read.ts # packages/plugin/package.json # packages/sdk/js/package.json # packages/slack/package.json # packages/ui/package.json # packages/util/package.json # packages/web/package.json # sdks/vscode/package.json
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Avoids pathological cases when cloning large objects
Issue for this PR
Closes #14344
Type of change
What does this PR do?
Changes all
cloneusages tostructuredClone. Don't know the codebase well enough to know if that's correct behaviour or not, but it works for me™How did you verify your code works?
Haven't verified extensively, just tested that it gets past the errors seen in #14344 and continues to work otherwise
Screenshots / recordings
If this is a UI change, please include a screenshot or recording.
Checklist
If you do not follow this template your PR will be automatically rejected.