langchain-ai/deepagentsjs
1
0
Fork 0
mirror of https://github.com/langchain-ai/deepagentsjs.git synced 2026-07-19 16:33:38 -04:00
Code Issues 42 Packages Projects Releases 10 Wiki Activity

[PR #67] [CLOSED] fix: exclude files from subagent state updates #96

New Issue
Closed
opened 2026-02-16 06:17:09 -05:00 by yindo · 0 comments
yindo commented 2026-02-16 06:17:09 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/deepagentsjs/pull/67
Author: @antonnak
Created: 11/25/2025
Status: ❌ Closed

Base: main ← Head: fix/exclude-files-from-subagent-state

📝 Commits (1)

  • 57e3efb fix: exclude files from subagent state updates

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 src/middleware/subagents.ts (+1 -1)

📄 Description

Problem

When multiple subagents run in parallel and complete in the same LangGraph step,
both try to update the parent's files channel simultaneously, causing:

InvalidUpdateError: Invalid update for channel "files" with values [{},{}]:
LastValue can only receive one value per step.

Root Cause

EXCLUDED_STATE_KEYS filters state when subagents return results to the parent
via returnCommandWithStateUpdate(). Currently it excludes messages, todos,
and jumpTo, but NOT files.

When using external backends (like Azure Blob, NFS, or even StoreBackend), files
are already persisted externally. Passing the files state back to the parent
is unnecessary and causes concurrent update errors when subagents run in parallel.

Solution

Add "files" to EXCLUDED_STATE_KEYS. This is consistent with how messages
and todos are handled - they have their own reducers and don't need to be
passed through subagent state updates.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/deepagentsjs/pull/67 **Author:** [@antonnak](https://github.com/antonnak) **Created:** 11/25/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `fix/exclude-files-from-subagent-state` --- ### 📝 Commits (1) - [`57e3efb`](https://github.com/langchain-ai/deepagentsjs/commit/57e3efb0ac3eae816abd305b237474557e8bad98) fix: exclude files from subagent state updates ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `src/middleware/subagents.ts` (+1 -1) </details> ### 📄 Description ## Problem When multiple subagents run in parallel and complete in the same LangGraph step, both try to update the parent's `files` channel simultaneously, causing: InvalidUpdateError: Invalid update for channel "files" with values [{},{}]: LastValue can only receive one value per step. ## Root Cause `EXCLUDED_STATE_KEYS` filters state when subagents return results to the parent via `returnCommandWithStateUpdate()`. Currently it excludes `messages`, `todos`, and `jumpTo`, but NOT `files`. When using external backends (like Azure Blob, NFS, or even StoreBackend), files are already persisted externally. Passing the `files` state back to the parent is unnecessary and causes concurrent update errors when subagents run in parallel. ## Solution Add `"files"` to `EXCLUDED_STATE_KEYS`. This is consistent with how `messages` and `todos` are handled - they have their own reducers and don't need to be passed through subagent state updates. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 06:17:09 -05:00
yindo closed this issue 2026-02-16 06:17:09 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
enhancement
help wanted
needs-response
pull-request
Mirrored from GitHub Pull Request
 stale
triage: high-impact
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/deepagentsjs#96
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?