fix: relay structured extension errors across nested gRPC - #9636
Conversation
|
Azure Pipelines: Successfully started running 2 pipeline(s). 19 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
|
Azure Pipelines: Successfully started running 2 pipeline(s). 19 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.
Pull request overview
Preserves structured extension errors across nested gRPC boundaries without duplicating operation prefixes.
Changes:
- Relays and restores
ServiceErrorandLocalErrormetadata. - Preserves actionable guidance and existing host error handling.
- Adds host, wire-transport, and hosted-agent regression tests.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
cli/azd/internal/grpcserver/errors.go |
Serializes relayed extension errors. |
cli/azd/internal/grpcserver/server_test.go |
Tests host serialization and transport. |
cli/azd/extensions/azure.ai.agents/internal/exterrors/errors.go |
Restores relayed errors. |
cli/azd/extensions/azure.ai.agents/internal/exterrors/errors_test.go |
Tests restoration and precedence. |
cli/azd/extensions/azure.ai.agents/internal/project/service_target_agent_test.go |
Tests hosted-agent publish behavior. |
💡 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
|
Travis Angevine (trangevi)
left a comment
There was a problem hiding this comment.
Seems fine from the extension side, pending the copilot comment
|
/check-enforcer override |
Summary
azdext.ServiceErrorandazdext.LocalErrorvalues as a singleExtensionErrorgRPC status detail when nested host calls return extension-originated structured errors.azure.ai.agentswithout adding a second operation prefix.azcore.ResponseErrorhandling, gRPC status messages, authentication behavior, actionable guidance, and generic-error fallbacks.Closes #9635