Skip to content

[codex] Normalize LangGraph resume submit updates#182

Merged
blove merged 1 commit into
mainfrom
codex/langgraph-submit-normalizer
May 2, 2026
Merged

[codex] Normalize LangGraph resume submit updates#182
blove merged 1 commit into
mainfrom
codex/langgraph-submit-normalizer

Conversation

@blove
Copy link
Copy Markdown
Contributor

@blove blove commented May 2, 2026

Summary

  • Return the real stream manager promise from agent.submit() so callers can await stream completion.
  • Normalize submit({ resume, state }) and submit({ resume, message, state }) into LangGraph command.update while keeping input: null.
  • Merge submit-derived command updates with existing command.update and preserve advanced command fields like goto.

Root Cause

The public submit input type says message can be combined with resume and/or state, but the LangGraph adapter returned null as soon as resume was present. That dropped state/message updates during interrupt resume flows. submit() also returned an already-resolved promise instead of the stream manager promise.

Validation

  • npx nx test langgraph -- --runInBand=false --reporter=verbose
  • npx nx test langgraph -- --runInBand=false
  • npx nx lint langgraph (0 errors; existing warnings remain)
  • npx nx build langgraph
  • git diff --check

@vercel
Copy link
Copy Markdown

vercel Bot commented May 2, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cacheplane Ready Ready Preview, Comment May 2, 2026 5:24pm

Request Review

@blove blove merged commit 3a790f8 into main May 2, 2026
14 checks passed
@blove blove deleted the codex/langgraph-submit-normalizer branch May 2, 2026 17:27
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.

1 participant