Skip to content

feat(langchain): support secure parameters in langchain tools and clients - #777

Open
anubhav756 wants to merge 1 commit into
anubhav-secure-params-adkfrom
anubhav-secure-params-langchain
Open

feat(langchain): support secure parameters in langchain tools and clients#777
anubhav756 wants to merge 1 commit into
anubhav-secure-params-adkfrom
anubhav-secure-params-langchain

Conversation

@anubhav756

@anubhav756 anubhav756 commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Description

Adds secure parameters support to LangChain tool wrappers and client loaders across sync and async environments.

Key Changes

  • Configures args_schema on ToolboxTool and AsyncToolboxTool to strictly reflect regular parameters.
  • Added bind_secure_param and bind_secure_params to ToolboxTool and AsyncToolboxTool.
  • Updated ToolboxClient and AsyncToolboxClient methods (load_tool, aload_tool, load_toolset, aload_toolset) to accept and forward secure_params.
  • Added unit tests in tests/test_tools.py, tests/test_async_tools.py, tests/test_client.py, and tests/test_async_client.py.

@anubhav756 anubhav756 changed the title feat(langchain): support secure parameters in langchain tools and cli… feat(langchain): support secure parameters in langchain tools and clients Aug 20, 2026
@anubhav756
anubhav756 marked this pull request as ready for review August 20, 2026 06:15
@anubhav756
anubhav756 requested a review from a team as a code owner August 20, 2026 06:15
@anubhav756
anubhav756 force-pushed the anubhav-secure-params-langchain branch from eb017e3 to 20636c6 Compare August 20, 2026 06:25
@anubhav756
anubhav756 force-pushed the anubhav-secure-params-langchain branch from 20636c6 to 85e7cad Compare August 20, 2026 10:08
@twishabansal twishabansal added the priority: p2 Moderately-important priority. Fix may not be included in next release. label Aug 25, 2026
@anubhav756
anubhav756 force-pushed the anubhav-secure-params-langchain branch from 85e7cad to a1500ab Compare August 25, 2026 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: p2 Moderately-important priority. Fix may not be included in next release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants