-
Notifications
You must be signed in to change notification settings - Fork 8
Setup Blueprint with Copilot Studio Permissions #210
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Setup Blueprint with Copilot Studio Permissions #210
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds support for configuring Copilot Studio permissions in agent blueprints, enabling them to invoke Copilot Studio copilots via the Power Platform API. The implementation follows the existing pattern for permission subcommands and centralizes validation logic for better code reuse.
Changes:
- Added
CopilotStudioSubcommandto configureCopilotStudio.Copilots.Invokedelegated permission for Power Platform API - Refactored validation logic into a shared
ValidationHelperclass to reduce code duplication across permission subcommands - Updated documentation and tests to reflect the new command
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
CopilotStudioSubcommand.cs |
New subcommand implementing CopilotStudio permission configuration with dry-run support |
PermissionsSubcommand.cs |
Registered new copilotstudio subcommand and removed validation methods moved to helper |
ValidationHelper.cs |
New shared helper centralizing validation logic for blueprint and MCP prerequisites |
MosConstants.cs |
Added Power Platform API permission constants for CopilotStudio.Copilots.Invoke |
CopilotStudioSubcommandTests.cs |
Comprehensive unit tests for the new subcommand |
PermissionsSubcommandTests.cs |
Updated tests to expect 3 subcommands instead of 2 |
SubcommandValidationTests.cs |
Updated to use ValidationHelper methods |
setup-permissions-copilotstudio.md |
New documentation for the copilotstudio command |
Readme-Usage.md |
Added copilotstudio command to usage examples |
src/Microsoft.Agents.A365.DevTools.Cli/Commands/SetupSubcommands/CopilotStudioSubcommand.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 9 out of 9 changed files in this pull request and generated no new comments.
|
Is someone runs setup-all will it add this permission, should we make it optional somehow? so that only if needed we add it |
This permission isn't added to the setup all flow: Agent365-devTools/src/Microsoft.Agents.A365.DevTools.Cli/Commands/SetupSubcommands/AllSubcommand.cs Lines 353 to 406 in 386f550
This permission would only be added if running |
This pull request adds support for configuring Copilot Studio permissions in the agent setup CLI, allowing agent blueprints to invoke Copilot Studio copilots via the Power Platform API. It introduces a new
copilotstudiosubcommand undera365 setup permissions, updates documentation, centralizes validation logic, and ensures the new functionality is covered by unit tests.Copilot Studio Permissions Support
CopilotStudioSubcommandimplementing thecopilotstudiosubcommand, which configures the delegatedCopilotStudio.Copilots.Invokepermission for the Power Platform API, including dry-run and verbose options. (src/Microsoft.Agents.A365.DevTools.Cli/Commands/SetupSubcommands/CopilotStudioSubcommand.cs)a365 setup permissionsnow supportscopilotstudioalongsidemcpandbot. (src/Microsoft.Agents.A365.DevTools.Cli/Commands/SetupSubcommands/PermissionsSubcommand.cs)Documentation Updates
docs/commands/setup/setup-permissions-copilotstudio.md,Readme-Usage.md) [1] [2]Centralized Validation Logic
src/Microsoft.Agents.A365.DevTools.Cli/Helpers/ValidationHelper.cs, removal fromPermissionsSubcommand.cs) [1] [2]Constants and Configuration
src/Microsoft.Agents.A365.DevTools.Cli/Constants/MosConstants.cs) [1] [2] [3]Testing
src/Tests/Microsoft.Agents.A365.DevTools.Cli.Tests/Commands/CopilotStudioSubcommandTests.cs,src/Tests/Microsoft.Agents.A365.DevTools.Cli.Tests/Commands/PermissionsSubcommandTests.cs) [1] [2] [3]