Skip to content

GenAI | Update execute tool to have name required#3595

Merged
lmolkova merged 3 commits intoopen-telemetry:mainfrom
keith-decker:toolcall-name-required
Apr 4, 2026
Merged

GenAI | Update execute tool to have name required#3595
lmolkova merged 3 commits intoopen-telemetry:mainfrom
keith-decker:toolcall-name-required

Conversation

@keith-decker
Copy link
Copy Markdown
Contributor

Fixes #

area: gen-ai

Changes

When implementing execute tool call in genai-utils for python-contrib, it became apparant that the name of the tool should be required instead of recommended.

Addresses comments in open-telemetry/opentelemetry-python-contrib#4356

Exploring existing genai frameworks shows a few times a tool call name would not be supplied would be during langchain tool streaming chunks, but the tool name would have been provided on the start of the tool call. Openai sdk still lists tool call name as optional in it's protobuf. It appears that a name is provided regardless.

Important

Pull requests acceptance are subject to the triage process as described in Issue and PR Triage Management.
PRs that do not follow the guidance above, may be automatically rejected and closed.

Merge requirement checklist

  • CONTRIBUTING.md guidelines followed.
  • Change log entry added, according to the guidelines in When to add a changelog entry.
    • If your PR does not need a change log, start the PR title with [chore]
  • Links to the prototypes or existing instrumentations (when adding or changing conventions)

Copy link
Copy Markdown
Member

@lmolkova lmolkova left a comment

Choose a reason for hiding this comment

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

Thanks!

Tool name is required in order to make a tool call - models request which tool to execute using name and server-side tool calls also have it.
It's always available - even with parameterless annotation/decorator - the name is the name of the function.

@lmolkova lmolkova moved this from Untriaged to Needs More Approval in Semantic Conventions Triage Apr 3, 2026
@lmolkova lmolkova added this pull request to the merge queue Apr 4, 2026
Merged via the queue into open-telemetry:main with commit 2f10c27 Apr 4, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants