Conversation
There was a problem hiding this comment.
Pull request overview
This PR synchronizes the permissions configuration as part of the weekly update process dated 2025-11-27. The changes primarily focus on enabling Agent Registry permissions and adding new application/service principal management permissions.
Key changes:
- Enabling 18 Agent* permissions (Read/ReadWrite for AgentInstance, AgentCardManifest, AgentCollection) by adding UUIDs and changing visibility/enabled flags
- Adding 4 new permission entries with empty IDs: AppRegistration.Create, AppRegistration.DeleteRestore.All, AppRegistration.EnableDisable.All, and ServicePrincipal.DeleteRestore.All
- Correcting resourceAppId for one CopilotPackages permission and adding comprehensive permission definitions in permissions.json
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| permissions/new/provisioningInfo.json | Enables Agent* permissions with UUIDs, adds new AppRegistration/ServicePrincipal entries with empty IDs, corrects one resourceAppId, adds Group-Priority and MailboxConfigItem entries |
| permissions/new/permissions.json | Adds complete definitions for all Agent* permissions, MailboxConfigItem.Read, reorganizes Calendar.Read and CloudPC paths, adds Policy paths, adds Copilot report paths |
💡 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.
| "id": "", | ||
| "scheme": "DelegatedWork", | ||
| "environment": "", | ||
| "isHidden": true, | ||
| "isEnabled": true, |
There was a problem hiding this comment.
New permission entries AppRegistration.Create, AppRegistration.DeleteRestore.All, AppRegistration.EnableDisable.All, and ServicePrincipal.DeleteRestore.All (lines 1037-1087 and 13172-13186) have empty id and environment fields but are marked as isEnabled: true. This creates an inconsistent state where permissions are enabled without proper identifiers. Either populate these fields with valid UUIDs and environment values, or set isEnabled: false until the identifiers are available.
Weekly Permissions sync 2025-11-27