Skip to content

Conversation

@Zoe14
Copy link
Contributor

@Zoe14 Zoe14 commented Nov 17, 2025

Instead of having generic CodeAgent / ToolCallingAgent, now the log would show which specific agent


Note

Set run span name to the agent's name if present, otherwise fall back to the class name.

Written by Cursor Bugbot for commit 60e7f99. This will update automatically on new commits. Configure here.

@Zoe14 Zoe14 requested a review from a team as a code owner November 17, 2025 16:56
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Nov 17, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 17, 2025

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@Zoe14
Copy link
Contributor Author

Zoe14 commented Nov 17, 2025

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Nov 17, 2025
@caroger
Copy link
Collaborator

caroger commented Nov 18, 2025

hey @Zoe14 , thanks for the PR.

There is a simple formatting step you need to run to get your PR over the line - runuvx --from ruff==0.9.2 ruff check --fix . under the directoryopeninference/python/instrumentation/openinference-instrumentation-smolagents

@caroger caroger changed the title enhancement(smolagents): have more explicit span name fix(smolagents): give agent run span as more descriptive span name Nov 18, 2025
@caroger caroger self-requested a review November 18, 2025 07:07
@Zoe14
Copy link
Contributor Author

Zoe14 commented Dec 10, 2025

can we merge this ? @caroger

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Dec 10, 2025
@caroger caroger merged commit 6ffccce into Arize-ai:main Dec 10, 2025
12 of 13 checks passed
@github-actions github-actions bot mentioned this pull request Dec 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants