Skip to content

feat: correlate streamed tool lifecycle events - #84

Merged
efecnc merged 1 commit into
mainfrom
feat/stream-tool-call-correlation
Jul 22, 2026
Merged

feat: correlate streamed tool lifecycle events#84
efecnc merged 1 commit into
mainfrom
feat/stream-tool-call-correlation

Conversation

@efecnc

@efecnc efecnc commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • propagate the provider tool call ID through started and finished telemetry
  • expose one correlation ID across API stream start, progress, and finish events
  • include correlation IDs in structured diagnostics while preserving backward-compatible deserialization

Validation

  • cargo check
  • cargo test --lib
  • cargo clippy --all-targets -- -D warnings
  • git diff --check

@amazon-q-developer amazon-q-developer Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The changes in this PR correctly add tool_call_id tracking across the telemetry system. The implementation properly handles the optional field with Option<String>, uses #[serde(default, skip_serializing_if = "Option::is_none")] for backward compatibility, and threads the ID through all relevant telemetry events and logging functions. The test coverage confirms the correlation ID functionality works as expected. No blocking issues identified.


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.

@efecnc
efecnc merged commit 8517b96 into main Jul 22, 2026
2 checks passed
@efecnc
efecnc deleted the feat/stream-tool-call-correlation branch July 22, 2026 13:51
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