Support Custom Permissions through `develop add-permissions` #218

JesuTerraz · 2026-02-03T00:08:57Z

Builds on Setup Blueprint with Copilot Studio Permissions #210 by adding support for adding copilot studio required permissions to custom client ids.
Builds on Support Custom Resources with develop get-token #224 by merging similar logic for custom resources.

…sers/jterrazas/develop-add-permission-mcs

* Initial plan * Address code review feedback: logging, validation, and documentation fixes Co-authored-by: JesuTerraz <96103167+JesuTerraz@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: JesuTerraz <96103167+JesuTerraz@users.noreply.github.com>

…hub.com/microsoft/Agent365-devTools into users/jterrazas/resource-flag-get-token

…rrazas/develop-add-permission-mcs

github-actions · 2026-02-10T18:31:38Z

⚠️ Deprecation Warning: The deny-licenses option is deprecated for possible removal in the next major release. For more information, see issue 997.

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Copilot

Pull request overview

Adds support for targeting non-default resource APIs when using a365 develop add-permissions, aligning the command with the custom-resource support previously added to a365 develop get-token by centralizing resource keyword/ID resolution.

Changes:

Introduces ResourceResolutionHelper + ResolvedResource to resolve mcp/powerplatform keywords and custom resource GUIDs, with shared validation and error messages.
Extends develop add-permissions to accept --resource / --resource-id and routes both add-permissions and get-token through the shared resolver.
Adds unit tests for the new resolver and updates command tests; updates develop add-permissions documentation for the new resource support.

Reviewed changes

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

Show a summary per file

File	Description
src/Microsoft.Agents.A365.DevTools.Cli/Helpers/ResourceResolutionHelper.cs	New shared resource/keyword resolution + validation helper used by multiple commands.
src/Microsoft.Agents.A365.DevTools.Cli/Constants/ErrorMessages.cs	Adds standardized error messages for resource resolution failures.
src/Microsoft.Agents.A365.DevTools.Cli/Commands/DevelopSubcommands/GetTokenSubcommand.cs	Replaces local resource-resolution logic with `ResourceResolutionHelper`.
src/Microsoft.Agents.A365.DevTools.Cli/Commands/DevelopSubcommands/AddPermissionsSubcommand.cs	Adds `--resource` / `--resource-id` options and uses shared resource resolver.
src/Tests/Microsoft.Agents.A365.DevTools.Cli.Tests/Helpers/ResourceResolutionHelperTests.cs	New unit tests covering resource resolution behavior and error messaging.
src/Tests/Microsoft.Agents.A365.DevTools.Cli.Tests/Commands/AddPermissionsSubcommandTests.cs	Updates command-option expectations for new resource options.
docs/commands/develop/develop-addpermissions.md	Updates docs to describe multi-resource permission addition.

src/Microsoft.Agents.A365.DevTools.Cli/Commands/DevelopSubcommands/AddPermissionsSubcommand.cs

docs/commands/develop/develop-addpermissions.md

src/Microsoft.Agents.A365.DevTools.Cli/Constants/ErrorMessages.cs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

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

docs/commands/develop/develop-addpermissions.md

src/Microsoft.Agents.A365.DevTools.Cli/Commands/DevelopSubcommands/AddPermissionsSubcommand.cs

src/Microsoft.Agents.A365.DevTools.Cli/Commands/DevelopSubcommands/GetTokenSubcommand.cs

src/Tests/Microsoft.Agents.A365.DevTools.Cli.Tests/Helpers/ResourceResolutionHelperTests.cs

Copilot

Pull request overview

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

src/Microsoft.Agents.A365.DevTools.Cli/Helpers/ResourceResolutionHelper.cs

docs/commands/develop/develop-addpermissions.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

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

src/Microsoft.Agents.A365.DevTools.Cli/Commands/DevelopSubcommands/AddPermissionsSubcommand.cs

src/Microsoft.Agents.A365.DevTools.Cli/Commands/DevelopSubcommands/GetTokenSubcommand.cs

src/Microsoft.Agents.A365.DevTools.Cli/Helpers/ResourceResolutionHelper.cs

…github.com/microsoft/Agent365-devTools into users/jterrazas/develop-add-permission-mcs

sellakumaran

Had an offline discussion with Jesus and shared comments.

Copilot

Pull request overview

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

src/Microsoft.Agents.A365.DevTools.Cli/Commands/DevelopSubcommands/AddPermissionsSubcommand.cs

Jesus Terrazas and others added 19 commits January 27, 2026 15:44

initial changes

8ce6af3

adds tests

6cca5d9

Merge branch 'users/jterrazas/setup-copilotstudio-permissions' into u…

cd2e769

…sers/jterrazas/develop-add-permission-mcs

Merge branch 'main' into users/jterrazas/develop-add-permission-mcs

000c26b

add more than mcp permissions in add-permissions subcommand

5111635

remove unused setupall flag

386f550

Merge branch 'users/jterrazas/setup-copilotstudio-permissions' into u…

151caf7

…sers/jterrazas/develop-add-permission-mcs

initial code

84816a1

try cleaning diff

f77a2fb

remove test cases that don't test any code from source

9c4cc44

restore tests

06618a8

update endpoint resolution

4739fc5

Merge branch 'users/jterrazas/resource-flag-get-token' of https://git…

904ff12

…hub.com/microsoft/Agent365-devTools into users/jterrazas/resource-flag-get-token

Merge branch 'main' into users/jterrazas/resource-flag-get-token

4ecfd43

Merge branch 'main' into users/jterrazas/develop-add-permission-mcs

78755f2

Merge branch 'users/jterrazas/resource-flag-get-token' into users/jte…

6620c94

…rrazas/develop-add-permission-mcs

merge get-token code for shared functionality

cdfe391

Merge branch 'main' into users/jterrazas/develop-add-permission-mcs

c7d4966

Jesus Terrazas added 3 commits February 10, 2026 11:29

Add ResolveResource for duplicate code

96f1f34

update messages and comments

6c042bc

Update option description

d119463

JesuTerraz changed the title ~~Add CopilotStudio Permissions through develop add-permissions~~ Support Custom Permissions through develop add-permissions Feb 10, 2026

JesuTerraz marked this pull request as ready for review February 10, 2026 19:43

JesuTerraz requested review from a team as code owners February 10, 2026 19:43

Copilot AI review requested due to automatic review settings February 10, 2026 19:43

Copilot started reviewing on behalf of JesuTerraz February 10, 2026 19:44 View session

Copilot AI reviewed Feb 10, 2026

View reviewed changes

Update docs/commands/develop/develop-addpermissions.md

e7e2ea1

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings February 10, 2026 19:50

Copilot started reviewing on behalf of JesuTerraz February 10, 2026 19:51 View session

JesuTerraz and others added 2 commits February 10, 2026 11:54

Update src/Microsoft.Agents.A365.DevTools.Cli/Constants/ErrorMessages.cs

b29512c

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

update doc

8524a9f

Copilot AI reviewed Feb 10, 2026

View reviewed changes

Jesus Terrazas added 2 commits February 10, 2026 14:10

Update docs to be consistent about explicit scopes with resource flags

9053662

Specify resource / scope behavior a little more

2aed990

Copilot AI review requested due to automatic review settings February 10, 2026 22:14

Copilot started reviewing on behalf of JesuTerraz February 10, 2026 22:15 View session

Copilot AI reviewed Feb 10, 2026

View reviewed changes

src/Microsoft.Agents.A365.DevTools.Cli/Helpers/ResourceResolutionHelper.cs Outdated Show resolved Hide resolved

docs/commands/develop/develop-addpermissions.md Outdated Show resolved Hide resolved

Jesus Terrazas and others added 2 commits February 10, 2026 14:29

update tests

e38f0ea

Update docs/commands/develop/develop-addpermissions.md

71ba355

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings February 10, 2026 22:31

Copilot started reviewing on behalf of JesuTerraz February 10, 2026 22:31 View session

Copilot AI reviewed Feb 10, 2026

View reviewed changes

Jesus Terrazas added 2 commits February 10, 2026 14:45

add more example usage

928f4ad

Merge branch 'users/jterrazas/develop-add-permission-mcs' of https://…

bf65ea5

…github.com/microsoft/Agent365-devTools into users/jterrazas/develop-add-permission-mcs

sellakumaran requested changes Feb 11, 2026

View reviewed changes

Merge branch 'main' into users/jterrazas/develop-add-permission-mcs

1c7a0bf

Copilot AI review requested due to automatic review settings February 12, 2026 00:08

Copilot started reviewing on behalf of JesuTerraz February 12, 2026 00:09 View session

add missing using directive

377697b

Copilot AI reviewed Feb 12, 2026

View reviewed changes

Support Custom Permissions through develop add-permissions #218

Are you sure you want to change the base?

Support Custom Permissions through develop add-permissions #218

Uh oh!

Conversation

JesuTerraz commented Feb 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Feb 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Dependency Review

Scanned Files

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

sellakumaran left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Support Custom Permissions through `develop add-permissions` #218

Support Custom Permissions through `develop add-permissions` #218

JesuTerraz commented Feb 3, 2026 •

edited

Loading

github-actions bot commented Feb 10, 2026 •

edited

Loading