Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 28 additions & 2 deletions permissions/new/permissions.json
Original file line number Diff line number Diff line change
Expand Up @@ -37100,7 +37100,7 @@
],
"paths": {
"/identity/conditionalaccess/claimProviders": "least=DelegatedWork",
"/identity/conditionalaccess/claimProviders/validate": "least=DelegatedWork"
"/identity/conditionalaccess/claimProviders/{name}/validate": "least=DelegatedWork"
}
},
{
Expand Down Expand Up @@ -38342,7 +38342,7 @@
],
"paths": {
"/identity/conditionalaccess/claimProviders": "",
"/identity/conditionalaccess/claimProviders/validate": ""
"/identity/conditionalaccess/claimProviders/{name}/validate": ""
}
},
{
Expand Down Expand Up @@ -48242,6 +48242,7 @@
"/planner/tasks/{id}/assignedtotaskboardformat": "least=DelegatedWork",
"/planner/tasks/{id}/buckettaskboardformat": "least=DelegatedWork",
"/planner/tasks/{id}/details": "least=DelegatedWork",
"/planner/tasks/{id}/messages": "least=DelegatedWork",
"/planner/tasks/{id}/progresstaskboardformat": "least=DelegatedWork",
"/planner/tasks/delta": "least=DelegatedWork",
"/teams/{id}/channels/{id}/planner/plans": "least=DelegatedWork",
Expand Down Expand Up @@ -48775,6 +48776,7 @@
"/planner/plans/{id}/unarchive": "least=DelegatedWork",
"/planner/rosters": "least=DelegatedWork",
"/planner/rosters/{id}/assignSensitivityLabel": "least=DelegatedWork",
"/planner/tasks/{id}/messages": "least=DelegatedWork",
"/planner/undo": "least=DelegatedWork"
}
},
Expand All @@ -48791,6 +48793,30 @@
"/planner/rosters/{id}/members/{id}": "least=DelegatedWork"
}
},
{
"schemeKeys": [
"DelegatedWork"
],
"methods": [
"DELETE",
"POST"
],
"paths": {
"/planner/tasks/{taskId}/messages/{id}/reactions": "least=DelegatedWork"
Copy link

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

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

The path parameter uses '{taskId}' while other planner paths in this file consistently use '{id}' for task identifiers (e.g., '/planner/tasks/{id}/messages/{id}' on line 48817). This inconsistency could cause confusion. Consider using '{id}' for consistency, or use more descriptive parameter names like '{taskId}' and '{messageId}' to disambiguate.

Copilot uses AI. Check for mistakes.
}
},
{
"schemeKeys": [
"DelegatedWork"
],
"methods": [
"DELETE",
"PATCH"
],
"paths": {
"/planner/tasks/{id}/messages/{id}": "least=DelegatedWork"
Copy link

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

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

The path '/planner/tasks/{id}/messages/{id}' uses the same parameter name '{id}' for both the task and message identifiers, which is ambiguous. Consider using distinct parameter names like '{taskId}' and '{messageId}' to clearly identify which resource each parameter refers to.

Copilot uses AI. Check for mistakes.
}
},
{
"schemeKeys": [
"DelegatedWork",
Expand Down
92 changes: 92 additions & 0 deletions permissions/new/provisioningInfo.json
Original file line number Diff line number Diff line change
Expand Up @@ -1104,6 +1104,88 @@
"resourceAppId": "00000003-0000-0000-c000-000000000000"
}
],
"AppManagementOnboarding.Read.All": [
{
"id": "",
"scheme": "DelegatedWork",
"environment": "public",
"isHidden": true,
"isEnabled": true,
"resourceAppId": "c7441e5a-180f-453a-bc4e-126159ca0174"
},
{
"id": "",
"scheme": "Application",
"environment": "public",
"isHidden": true,
"isEnabled": true,
"resourceAppId": "c7441e5a-180f-453a-bc4e-126159ca0174"
}
],
"AppManagementOnboarding.ReadWrite.All": [
{
"id": "",
"scheme": "DelegatedWork",
"environment": "public",
"isHidden": true,
"isEnabled": true,
"resourceAppId": "c7441e5a-180f-453a-bc4e-126159ca0174"
},
{
"id": "",
"scheme": "Application",
"environment": "public",
"isHidden": true,
"isEnabled": true,
"resourceAppId": "c7441e5a-180f-453a-bc4e-126159ca0174"
}
],
"AppManagementOnboarding.Submit": [
{
"id": "",
"scheme": "DelegatedWork",
"environment": "public",
"isHidden": true,
"isEnabled": true,
"resourceAppId": "c7441e5a-180f-453a-bc4e-126159ca0174"
}
],
"AppManagementOnboardingBlob.Read.All": [
{
"id": "",
"scheme": "DelegatedWork",
"environment": "public",
"isHidden": true,
"isEnabled": true,
"resourceAppId": "c7441e5a-180f-453a-bc4e-126159ca0174"
},
{
"id": "",
"scheme": "Application",
"environment": "public",
"isHidden": true,
"isEnabled": true,
"resourceAppId": "c7441e5a-180f-453a-bc4e-126159ca0174"
}
],
"AppManagementOnboardingBlob.ReadWrite.All": [
{
"id": "",
"scheme": "DelegatedWork",
"environment": "public",
"isHidden": true,
"isEnabled": true,
"resourceAppId": "c7441e5a-180f-453a-bc4e-126159ca0174"
},
{
"id": "",
"scheme": "Application",
"environment": "public",
"isHidden": true,
"isEnabled": true,
"resourceAppId": "c7441e5a-180f-453a-bc4e-126159ca0174"
}
],
"AppRoleAssignment.ReadWrite.All": [
{
"id": "84bccea3-f856-4a8a-967b-dbe0a3d53a64",
Expand Down Expand Up @@ -10360,6 +10442,16 @@
"resourceAppId": ""
}
],
"Policy.ReadWrite.AuthMethod-Features": [
Copy link

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

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

The permission name 'Policy.ReadWrite.AuthMethod-Features' uses a hyphen, which is inconsistent with the naming convention used by adjacent permissions like 'Policy.ReadWrite.AuthenticationMethod' and 'Policy.ReadWrite.AuthenticationFlows' that use camelCase. Consider using 'Policy.ReadWrite.AuthMethodFeatures' instead for consistency.

Suggested change
"Policy.ReadWrite.AuthMethod-Features": [
"Policy.ReadWrite.AuthMethodFeatures": [

Copilot uses AI. Check for mistakes.
{
"id": "",
"scheme": "Application",
"environment": "",
"isHidden": true,
"isEnabled": true,
"resourceAppId": "ea890292-c8c8-4433-b5ea-b09d0668e1a6"
}
],
"Policy.ReadWrite.Authorization": [
{
"id": "edd3c878-b384-41fd-95ad-e7407dd775be",
Expand Down