mirror of
https://github.com/meta-llama/llama-stack.git
synced 2025-12-03 09:53:45 +00:00
fix(tool-runtime): Remove authorization from list_runtime_tools in all providers
Updated all tool runtime provider implementations to remove the authorization parameter from list_runtime_tools(): - tavily_search.py - brave_search.py - wolfram_alpha.py - bing_search.py These providers were missing in the previous commit. Tool listing typically doesn't require authentication - only invoke_tool() needs the authorization parameter for authenticated tool execution. This ensures all tool runtime providers have consistent signatures matching the updated protocol definition.
This commit is contained in:
parent
18f197763b
commit
e6ebbd8a7b
4 changed files with 0 additions and 4 deletions
|
|
@ -52,7 +52,6 @@ class BingSearchToolRuntimeImpl(ToolGroupsProtocolPrivate, ToolRuntime, NeedsReq
|
|||
self,
|
||||
tool_group_id: str | None = None,
|
||||
mcp_endpoint: URL | None = None,
|
||||
authorization: str | None = None,
|
||||
) -> ListToolDefsResponse:
|
||||
return ListToolDefsResponse(
|
||||
data=[
|
||||
|
|
|
|||
|
|
@ -51,7 +51,6 @@ class BraveSearchToolRuntimeImpl(ToolGroupsProtocolPrivate, ToolRuntime, NeedsRe
|
|||
self,
|
||||
tool_group_id: str | None = None,
|
||||
mcp_endpoint: URL | None = None,
|
||||
authorization: str | None = None,
|
||||
) -> ListToolDefsResponse:
|
||||
return ListToolDefsResponse(
|
||||
data=[
|
||||
|
|
|
|||
|
|
@ -51,7 +51,6 @@ class TavilySearchToolRuntimeImpl(ToolGroupsProtocolPrivate, ToolRuntime, NeedsR
|
|||
self,
|
||||
tool_group_id: str | None = None,
|
||||
mcp_endpoint: URL | None = None,
|
||||
authorization: str | None = None,
|
||||
) -> ListToolDefsResponse:
|
||||
return ListToolDefsResponse(
|
||||
data=[
|
||||
|
|
|
|||
|
|
@ -52,7 +52,6 @@ class WolframAlphaToolRuntimeImpl(ToolGroupsProtocolPrivate, ToolRuntime, NeedsR
|
|||
self,
|
||||
tool_group_id: str | None = None,
|
||||
mcp_endpoint: URL | None = None,
|
||||
authorization: str | None = None,
|
||||
) -> ListToolDefsResponse:
|
||||
return ListToolDefsResponse(
|
||||
data=[
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue