Skip to content

Conversation

@saksharthakkar
Copy link
Contributor

@saksharthakkar saksharthakkar commented Jan 13, 2026

Summary

Add tool_type and display_name metadata to all tool types for completeness:

  • Integration tools
  • Context tools (semantic search, deep rag, batch transform)
  • MCP tools

Note: Process and escalation tools already have this metadata.

Why

This metadata is consumed by uipath-agents-python to create custom OTEL spans per tool type. See PR #87 which uses tool_type to identify integration tools and create proper child spans with IntegrationToolSpanAttributes.

Without this metadata, the callback code in uipath-agents-python cannot distinguish tool types for observability.

Test plan

  • Verify all tool types include tool_type and display_name metadata
  • Ensure existing functionality is unaffected

🤖 Generated with Claude Code

Add tool_type and display_name metadata to integration tools for better
identification and display purposes.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@saksharthakkar saksharthakkar changed the title feat: add metadata to integration tools (wip) feat: add metadata to integration tools Jan 13, 2026
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@saksharthakkar saksharthakkar force-pushed the feat/integration-tool-metadata branch from a2fa79d to 319224a Compare January 13, 2026 00:43
Add tool_type and display_name metadata to context tools (semantic
search, deep rag, batch transform) and mcp tools for completeness.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@saksharthakkar saksharthakkar changed the title (wip) feat: add metadata to integration tools feat: add tool_type metadata to all tools Jan 13, 2026
@saksharthakkar saksharthakkar marked this pull request as ready for review January 13, 2026 21:22
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copy link
Member

@JosephMar JosephMar left a comment

Choose a reason for hiding this comment

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

LGTM but tests are failing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants