From 44e928640235295b087efacfda25e00e9cd2f516 Mon Sep 17 00:00:00 2001 From: Anubhav Dhawan Date: Tue, 29 Apr 2025 15:56:10 +0000 Subject: [PATCH 1/2] docs: Update docstring to be more accurate --- packages/toolbox-core/src/toolbox_core/tool.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/toolbox-core/src/toolbox_core/tool.py b/packages/toolbox-core/src/toolbox_core/tool.py index d26d9287..73aa2403 100644 --- a/packages/toolbox-core/src/toolbox_core/tool.py +++ b/packages/toolbox-core/src/toolbox_core/tool.py @@ -64,7 +64,7 @@ def __init__( description: The description of the remote tool. params: The args of the tool. required_authn_params: A dict of required authenticated parameters to a list - of services that provide values for them. + of alternative services that can provide values for them. auth_service_token_getters: A dict of authService -> token (or callables that produce a token) bound_params: A mapping of parameter names to bind to specific values or From f54c901a71cbde2acb92af86b2ba103001143810 Mon Sep 17 00:00:00 2001 From: Anubhav Dhawan Date: Tue, 29 Apr 2025 16:56:22 +0000 Subject: [PATCH 2/2] docs: Make the docstring consistent across helpers. --- packages/toolbox-core/src/toolbox_core/tool.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/packages/toolbox-core/src/toolbox_core/tool.py b/packages/toolbox-core/src/toolbox_core/tool.py index 73aa2403..73ca0009 100644 --- a/packages/toolbox-core/src/toolbox_core/tool.py +++ b/packages/toolbox-core/src/toolbox_core/tool.py @@ -63,7 +63,7 @@ def __init__( name: The name of the remote tool. description: The description of the remote tool. params: The args of the tool. - required_authn_params: A dict of required authenticated parameters to a list + required_authn_params: A map of required authenticated parameters to a list of alternative services that can provide values for them. auth_service_token_getters: A dict of authService -> token (or callables that produce a token) @@ -134,8 +134,8 @@ def __copy( name: The name of the remote tool. description: The description of the remote tool. params: The args of the tool. - required_authn_params: A dict of required authenticated parameters that need - an auth_service_token_getter set for them yet. + required_authn_params: A map of required authenticated parameters to a list + of alternative services that can provide values for them. auth_service_token_getters: A dict of authService -> token (or callables that produce a token) bound_params: A mapping of parameter names to bind to specific values or