Skip to content

Weekly Permissions sync 2026-02-26#1453

Open
marabooy wants to merge 1 commit intomasterfrom
permissions-update/2026-02-26
Open

Weekly Permissions sync 2026-02-26#1453
marabooy wants to merge 1 commit intomasterfrom
permissions-update/2026-02-26

Conversation

@marabooy
Copy link
Contributor

Weekly Permissions sync 2026-02-26

Copilot AI review requested due to automatic review settings February 26, 2026 01:18
@marabooy marabooy requested a review from a team as a code owner February 26, 2026 01:18
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 permissions sync update to keep permissions/new/provisioningInfo.json aligned with the latest permission deployment metadata.

Changes:

  • Filled in previously-empty id GUIDs for several AppManagementOnboarding* permission deployments.
  • Added new hidden permission deployments (e.g., AppManagementOnboarding*Test*, EngagementExport.Read.All, SecurityIdentitiesMigration.*).
  • Renamed CrossTenantRoleAssignment.{Create,Delete} provisioning keys to MS-CrossTenantRoleAssignment.{Create,Delete} and adjusted AgentIdentity.Create.All hidden flag for DelegatedWork.

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

Comment on lines +13424 to +13430
"SecurityIdentitiesMigration.Read.All": [
{
"scheme": "DelegatedWork",
"environment": "public",
"isHidden": true,
"isEnabled": true,
"resourceAppId": "8ee8fdad-f234-4243-8f3b-15c294843740"
Copy link

Copilot AI Feb 26, 2026

Choose a reason for hiding this comment

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

The new SecurityIdentitiesMigration.* deployment objects are missing the id property entirely. Elsewhere in this file each deployment object includes an id field (even when empty), so omitting it is inconsistent and may break tooling that expects a stable schema. Add an id field to both the DelegatedWork and Application objects (populate it if known, otherwise keep it as an empty string).

Copilot uses AI. Check for mistakes.
Comment on lines +13440 to +13446
"SecurityIdentitiesMigration.ReadWrite.All": [
{
"scheme": "DelegatedWork",
"environment": "public",
"isHidden": true,
"isEnabled": true,
"resourceAppId": "8ee8fdad-f234-4243-8f3b-15c294843740"
Copy link

Copilot AI Feb 26, 2026

Choose a reason for hiding this comment

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

SecurityIdentitiesMigration.ReadWrite.All deployment objects also omit the id field. Please add id to both entries (DelegatedWork and Application) to match the schema used throughout provisioningInfo.json.

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