refactor: support telemetry for ai agent related extensions - #9861
refactor: support telemetry for ai agent related extensions#9861JerryYang (JerryYangKai) wants to merge 1 commit into
Conversation
|
Azure Pipelines: Successfully started running 1 pipeline(s). 21 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
📋 Prioritization NoteThanks for the contribution! The linked issue isn't in the current milestone yet. |
There was a problem hiding this comment.
🟢 Approval recommended
The implementation is focused, documented, and adequately tested with no unresolved correctness issues.
Pull request overview
Centralizes best-effort telemetry reporting for Foundry extensions, addressing duplication identified in PR #9841.
Changes:
- Adds a shared telemetry reporter with timeout and privacy-safe logging.
- Adds coverage for forwarding, failures, cancellation, and concurrency.
- Documents reporter usage.
File summaries
| File | Description |
|---|---|
pkg/foundry/telemetry/reporter.go |
Implements the shared reporter. |
pkg/foundry/telemetry/reporter_test.go |
Tests reporter behavior. |
docs/extensions/extension-telemetry.md |
Documents extension usage. |
docs/extensions/extension-sdk-reference.md |
Adds SDK reference guidance. |
Review details
- Files reviewed: 4/4 changed files
- Comments generated: 0
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Azure Dev CLI Install InstructionsInstall scriptsMacOS/Linux
bash: pwsh: WindowsPowerShell install MSI install Standalone Binary
MSI
Documentationlearn.microsoft.com documentationtitle: Azure Developer CLI reference
|
|
/check-enforcer evaluate |
Based on the comment from this PR #9841
Move telemetry related logic in azd/pkg/foundry package