Skip to content

Weekly Permissions sync 2026-04-03#1490

Open
marabooy wants to merge 3 commits intomasterfrom
permissions-update/2026-04-03
Open

Weekly Permissions sync 2026-04-03#1490
marabooy wants to merge 3 commits intomasterfrom
permissions-update/2026-04-03

Conversation

@marabooy
Copy link
Copy Markdown
Contributor

@marabooy marabooy commented Apr 3, 2026

Weekly Permissions sync 2026-04-03

@marabooy marabooy requested a review from a team as a code owner April 3, 2026 00:33
Copilot AI review requested due to automatic review settings April 3, 2026 00:33
Copy link
Copy Markdown
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 updating the permissions provisioning metadata used by the repo’s permissions pipeline.

Changes:

  • Added two new placeholder permission keys (_A.DG, _A.DH) to permissions/new/provisioningInfo.json.
  • Each new key currently includes an Application-scheme deployment entry (hidden/enabled, empty id/environment).

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

Comment on lines +26514 to +26518
"_A.DG": [
{
"id": "",
"environment": "",
"isHidden": true,
Copy link

Copilot AI Apr 3, 2026

Choose a reason for hiding this comment

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

_A.DG is added as an underscore placeholder permission, but it only includes an Application variant. In this file, underscore placeholder permissions in the surrounding block (e.g., _A.DC_A.DF) consistently include both Application and DelegatedWork variants; omitting DelegatedWork here can cause lookups/exports for delegated scenarios to miss this key. Add the corresponding DelegatedWork entry (matching the other placeholder entries) or confirm/encode explicitly that this placeholder is application-only in a way that is consistent with the rest of the placeholder set.

Copilot uses AI. Check for mistakes.
"isHidden": true,
"isEnabled": true,
"resourceAppId": "00000002-0000-0000-c000-000000000000",
"scheme": "Application"
Copy link

Copilot AI Apr 3, 2026

Choose a reason for hiding this comment

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

_A.DH is added with only an Application variant. Given neighboring underscore placeholder permissions include both Application and DelegatedWork variants, consider adding the missing DelegatedWork entry so delegated permission flows don’t treat this key as absent.

Suggested change
"scheme": "Application"
"scheme": "Application"
},
{
"id": "",
"environment": "",
"isHidden": true,
"isEnabled": true,
"resourceAppId": "00000002-0000-0000-c000-000000000000",
"scheme": "DelegatedWork"

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