-
Notifications
You must be signed in to change notification settings - Fork 1.1k
.Net: Fix DebuggerDisplay attribute to reference existing property #3326
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: rogerbarreto <19890735+rogerbarreto@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR fixes a debugger visualizer error in the AIAgent class by updating the DebuggerDisplay attribute to reference the existing Name property instead of the removed DisplayName property. The DisplayName property was removed in PR #2758, causing CS0103 compile errors when inspecting AIAgent instances in the debugger.
Changes:
- Updated
DebuggerDisplayattribute from{DisplayName,nq}to{Name,nq}inAIAgent.cs
Motivation and Context
The
[DebuggerDisplay]attribute onAIAgentreferencedDisplayName, which was removed in PR #2758. This caused CS0103 compile errors in the debugger visualizer.Description
Updated
[DebuggerDisplay("{DisplayName,nq}")]to[DebuggerDisplay("{Name,nq}")]inAIAgent.cs. TheNameproperty serves the same purpose as a human-readable identifier for debugging.Changed:
Contribution Checklist
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.