-
Notifications
You must be signed in to change notification settings - Fork 522
Fix settings grouping under PowerShell heading with subgroups in VS Code 1.101+ #5329
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
|
@copilot preserve the title property to be present, so that settings are still grouped by their extension name "PowerShell" but also in sub groups such as "Interface", but ensure they are all under the parent PowerShell header. |
Co-authored-by: JustinGrote <15258962+JustinGrote@users.noreply.github.com>
Co-authored-by: JustinGrote <15258962+JustinGrote@users.noreply.github.com>
… under PowerShell Co-authored-by: JustinGrote <15258962+JustinGrote@users.noreply.github.com>
Done! Added This properly scopes the subgroups under the PowerShell extension heading while maintaining the categorical organization you requested. |
|
@andyleejordan I tested and adding the IDs fixed the issue. We can either merge this or wait for it to be fixed in vscode. |
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 settings organization issue in VS Code 1.101+ where PowerShell extension settings were appearing as separate top-level items instead of being grouped under a unified PowerShell heading. The fix adds the id property to each configuration section to properly scope them under the extension while preserving subgroup structure.
Key Changes:
- Added
idproperty to all 6 configuration sections (Interface, Formatting, Editor Services, Pester, Terminal, Developer) - Used consistent naming pattern:
powershell.<sectionName>in camelCase - Maintained existing
titleproperties for subgroup display names
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
PR Summary
VS Code 1.101+ requires the
idproperty in configuration objects to properly scope settings subgroups under an extension's heading. Settings groups (Interface, Formatting, Editor Services, Pester, Terminal, Developer) were appearing as separate top-level items instead of under the PowerShell extension heading.Changes
Added
idproperty to each configuration section inpackage.jsonto properly scope them under the PowerShell extension while preserving thetitleproperty for subgroup display names:Before:
After:
VS Code now groups all settings under the extension's
displayName("PowerShell") as the parent heading, with each titled section appearing as a subgroup.Expected Settings UI Structure
Impact
PR Checklist
Fixes #5328
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.