Skip to content

fix(advanced): bump langchain-quickjs to 0.3.12, drop the exit workaround - #1202

Open
radu-mocanu wants to merge 1 commit into
mainfrom
fix/quickjs-0.3.12-drop-atexit
Open

radu-mocanu wants to merge 1 commit into
mainfrom
fix/quickjs-0.3.12-drop-atexit

Conversation

@radu-mocanu

Copy link
Copy Markdown
Collaborator

Summary

  • require langchain-quickjs>=0.3.12 and remove the atexit REPL close: upstream no longer blocks interpreter shutdown, so jobs whose run ended in an interrupt or a tool error still exit
  • this brings deepagents 0.7.34 and langchain-core 1.7.5+
  • main agent refusals still fault with the provider CHAT_MODEL error instead of upstream's new ModelRefusalError
  • subagent refusals now come back to the parent as a task result instead of failing the run

Why

the hang was fixed upstream in langchain-ai/deepagents#6397, so the local workaround is dead weight.

@radu-mocanu
radu-mocanu marked this pull request as ready for review November 9, 2026 10:28
Copilot AI balanced review requested due to automatic review settings November 9, 2026 10:29

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No active review issues were identified for the dependency update and refusal-handling changes.

No open issues 🎉

What changed in this PR

Updates the advanced-agent dependencies to use the upstream interpreter-shutdown fix, removes the local exit workaround, and adapts refusal handling to the updated libraries.

Changes:

  • Requires langchain-quickjs>=0.3.12 and langchain-core>=1.7.5, refreshes locked dependencies, and bumps the SDK to 0.19.4.
  • Removes REPL cleanup at exit while preserving provider-specific main-agent refusal handling and upstream subagent refusal behavior.
  • Adds refusal regression tests and updates compatibility with dependency APIs.
File Description
uv.lock Refreshes resolved dependencies and SDK metadata.
tests/​agent/​multimodal/​test_utils.py Updates Anthropic formatting calls for the new signature.
tests/​agent/​advanced/​test_payload_handler_middleware.py Adds refusal-handling and middleware-wiring tests.
tests/​agent/​advanced/​test_code_interpreter.py Removes the obsolete exit-cleanup registration test.
src/​uipath_langchain/​agent/​tools/​base_uipath_structured_tool.py Uses LangChain’s signature-parameter helper.
src/​uipath_langchain/​agent/​advanced/​code_interpreter.py Removes the local atexit REPL cleanup workaround.
src/​uipath_langchain/​agent/​advanced/​agent.py Adds main-agent refusal-response middleware.
pyproject.toml Raises dependency minimums and bumps the SDK version.

Review effort: Balanced

@radu-mocanu
radu-mocanu force-pushed the fix/quickjs-0.3.12-drop-atexit branch from 0b7e066 to c6f6cc9 Compare November 9, 2026 10:37
@radu-mocanu
radu-mocanu force-pushed the fix/quickjs-0.3.12-drop-atexit branch from c6f6cc9 to bd5360e Compare November 10, 2026 09:02
@sonarqubecloud

Copy link
Copy Markdown

This branch has not been deployed

No deployments
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