Skip to content

Weekly Permissions sync 2026-03-06#1463

Open
marabooy wants to merge 1 commit intomasterfrom
permissions-update/2026-03-06
Open

Weekly Permissions sync 2026-03-06#1463
marabooy wants to merge 1 commit intomasterfrom
permissions-update/2026-03-06

Conversation

@marabooy
Copy link
Contributor

@marabooy marabooy commented Mar 6, 2026

Weekly Permissions sync 2026-03-06

@marabooy marabooy requested a review from a team as a code owner March 6, 2026 01:29
Copilot AI review requested due to automatic review settings March 6, 2026 01:29
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Weekly sync of the permission metadata sources that back the repo’s permissions catalog (deployment/provisioning data and permission definitions).

Changes:

  • Added new provisioning entries (e.g., LifecyclePolicies-Guests.*) and updated visibility/IDs for several existing permission deployments (notably TenantGovernance-*, WindowsUpdates.Read.All).
  • Expanded/adjusted Microsoft Graph path mappings for new “inherited*” endpoints and several Network Access + Planner endpoints.
  • Added new permission definitions in permissions.json (e.g., TenantGovernance-*, WindowsUpdates.Read.All) and updated a couple of permission descriptions.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
permissions/new/provisioningInfo.json Adds/updates permission deployment records (IDs, hidden flags, new permissions).
permissions/new/permissions.json Updates permission definitions and path mappings; introduces new permission entries.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment on lines +1486 to 1493
"adminDescription": "Allows the app to read, update, create, and delete agent identity blueprints on behalf of the signed-in user.",
"requiresAdminConsent": true,
"privilegeLevel": 3
},
"Application": {
"adminDisplayName": "Read and write all agent identity blueprints.",
"adminDescription": "Allows the app to read, update, and delete agent identity blueprints without a signed-in user.",
"adminDescription": "Allows the app to read, update, create, and delete agent identity blueprints without a signed-in user.",
"requiresAdminConsent": true,
Copy link

Copilot AI Mar 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The updated adminDescription now claims this permission can "create" and "delete" agent identity blueprints, but the defined pathSets for AgentIdentityBlueprint.ReadWrite.All only include PATCH (update) operations. Either remove unsupported actions from the description or add the corresponding POST/DELETE paths/methods to match the described capabilities (or rely on the separate Create/DeleteRestore permissions).

Copilot uses AI. Check for mistakes.
Comment on lines +1745 to 1752
"adminDescription": "Allows the app to read, update, create, and delete agent identity blueprint principals on behalf of the signed-in user.",
"requiresAdminConsent": true,
"privilegeLevel": 3
},
"Application": {
"adminDisplayName": "Read and write all agent identity blueprint principals.",
"adminDescription": "Allows the app to read, update, and delete agent identity blueprint principals without a signed-in user.",
"adminDescription": "Allows the app to read, update, create, and delete agent identity blueprint principals without a signed-in user.",
"requiresAdminConsent": true,
Copy link

Copilot AI Mar 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The updated adminDescription now claims this permission can "create" and "delete" agent identity blueprint principals, but the pathSets for AgentIdentityBlueprintPrincipal.ReadWrite.All only include PATCH (update) operations. Please align the description with actual allowed operations, or add POST/DELETE paths/methods if this permission is intended to cover those actions (otherwise keep create/delete covered by the separate Create/DeleteRestore permissions).

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants