Skip to content

feat(attributes): Complete the gen_ai.tool.definitions alias group - #590

Merged
JPeer264 merged 0 commit into
jp/conv-fs-error-transformfrom
jp/conv-gen-ai-tool-definitions-alias
Aug 27, 2026
Merged

feat(attributes): Complete the gen_ai.tool.definitions alias group#590
JPeer264 merged 0 commit into
jp/conv-fs-error-transformfrom
jp/conv-gen-ai-tool-definitions-alias

Conversation

@JPeer264

Copy link
Copy Markdown
Member

gen_ai.request.available_tools, ai.tools and ai.prompt.tools all replace onto gen_ai.tool.definitions, but none of the four named the others. gen_ai.request.available_tools even carried an empty alias array and ai.tools had none at all, which is why the symmetry test stayed green. All four now list each other.

Replaced by getsentry/sentry-javascript#22820

@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown

Semver Impact of This PR

🟡 Minor (new features)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

Attributes

  • Complete the gen_ai.tool.definitions alias group by JPeer264 in #590
  • Transform fs_error into error.type by JPeer264 in #589
  • Complete net.* aliases and transform net.transport by JPeer264 in #588
  • Transform http.target into url.path and url.query by JPeer264 in #587
  • Add url.path.params.<key> (deprecated) in favor of url.path.parameter.<key> by JPeer264 in #586
  • Add graphql.source (deprecated) in favor of graphql.document by JPeer264 in #584
  • Add legacy Vercel AI span attributes (deprecated) by JPeer264 in #583
  • Add legacy AMQP and Redis span attributes (deprecated) by JPeer264 in #581
  • Add graphql.processing.type by andreiborza in #572

Other

  • (span-names) Add url.domainrule for http.client spans by Lms24 in #575

Bug Fixes 🐛

Names

  • Add missing gen_ai ops to span name rules by Lms24 in #579
  • Include tool name in gen_ai.execute_tool span names by Lms24 in #580
  • Correct graphql span name examples by andreiborza in #573

Internal Changes 🔧

  • (attributes) Add search alias for segment name by nsdeschenes in #577

🤖 This preview updates automatically when you update the PR.

@JPeer264
JPeer264 force-pushed the jp/conv-gen-ai-tool-definitions-alias branch 3 times, most recently from c4a5a07 to 56d2bd6 Compare August 27, 2026 09:21
@JPeer264
JPeer264 force-pushed the jp/conv-gen-ai-tool-definitions-alias branch from 56d2bd6 to 244c92e Compare August 27, 2026 09:26
@JPeer264
JPeer264 force-pushed the jp/conv-gen-ai-tool-definitions-alias branch 2 times, most recently from 666dd9b to cf20f28 Compare August 27, 2026 16:30
@JPeer264
JPeer264 marked this pull request as ready for review August 27, 2026 16:37
@JPeer264
JPeer264 requested review from a team, Lms24, cleptric, mjq and nsdeschenes as code owners August 27, 2026 16:37
@JPeer264
JPeer264 merged commit ab9324d into main Aug 27, 2026
@JPeer264
JPeer264 deleted the jp/conv-gen-ai-tool-definitions-alias branch August 27, 2026 16:41
@JPeer264
JPeer264 force-pushed the jp/conv-gen-ai-tool-definitions-alias branch from cf20f28 to ab9324d Compare August 27, 2026 16:41
@JPeer264

Copy link
Copy Markdown
Member Author

how did that merge?

@JPeer264

Copy link
Copy Markdown
Member Author

stacks are confusing if you mess with them. It did not land on main.

@JPeer264
JPeer264 restored the jp/conv-gen-ai-tool-definitions-alias branch August 27, 2026 16:45
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.

1 participant