Skip to content

fix(gateway): preserve context-only run config#1780

Closed
xsfX20 wants to merge 1 commit into
bytedance:mainfrom
xsfX20:fix/context-overrides-with-context-xsfx20
Closed

fix(gateway): preserve context-only run config#1780
xsfX20 wants to merge 1 commit into
bytedance:mainfrom
xsfX20:fix/context-overrides-with-context-xsfx20

Conversation

@xsfX20
Copy link
Copy Markdown
Contributor

@xsfX20 xsfX20 commented Apr 2, 2026

Summary

  • keep DeerFlow context overrides inside config["context"] when the incoming request already uses LangGraph's context mode
  • avoid reintroducing configurable in that path, which can violate LangGraph's context/configurable exclusivity
  • add a regression test covering the context-only merge path

Testing

  • git diff --check
  • uv run pytest -q tests/test_gateway_services.py (dependency bootstrap started, but failed on a python-telegram-bot wheel download timeout in this environment)

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 2, 2026

CLA assistant check
All committers have signed the CLA.

@xsfX20
Copy link
Copy Markdown
Contributor Author

xsfX20 commented May 11, 2026

Closing this from my side as superseded by the current main branch. The gateway code now has merge_run_context_overrides(), which intentionally writes whitelisted context values to both config["configurable"] and config["context"] for the newer LangGraph runtime path. Rebasing this older patch would conflict with that current design and reintroduce the older context-only behavior.

@xsfX20 xsfX20 closed this May 11, 2026
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