Skip to content

feat(telemetry): optionally capture tool trace details - #95

Open
ahmadalguydi wants to merge 3 commits into
open-gitagent:mainfrom
ahmadalguydi:codex/capture-tool-trace-details-26
Open

feat(telemetry): optionally capture tool trace details#95
ahmadalguydi wants to merge 3 commits into
open-gitagent:mainfrom
ahmadalguydi:codex/capture-tool-trace-details-26

Conversation

@ahmadalguydi

Copy link
Copy Markdown

Summary

  • add opt-in bounded tool argument/result capture to gitagent.tool.execute spans
  • support both SDK configuration (captureToolContent: true) and CLI environment configuration (GITAGENT_OTEL_CAPTURE_TOOL_CONTENT=true)
  • keep content capture disabled by default to preserve the existing privacy posture
  • document the new attributes and cover enabled/disabled behavior

This addresses the missing tool-level detail described in #26 while avoiding accidental prompt, credential, or tool-result export for existing users.

Verification

  • npm run build
  • npm test (67 tests across 18 suites)
  • focused telemetry tests (10 passed)
  • git diff --check

@ahmadalguydi
ahmadalguydi marked this pull request as ready for review July 29, 2026 06:16
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