Support Custom Permissions through `develop add-permissions` by JesuTerraz · Pull Request #218 · microsoft/Agent365-devTools

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

JesuTerraz · 2026-02-13T20:22:59Z

Closing as some of this work will be picked up when adding generic permissions support as discussed with Sella

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

JesuTerraz closed this Feb 13, 2026

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!

JesuTerraz commented Feb 13, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

JesuTerraz commented Feb 3, 2026 •

edited

Loading

github-actions bot commented Feb 10, 2026 •

edited

Loading