Initial commit of endpoint registration migration

[joratz]

No description provided.

[Copilot]

Pull Request Overview

This PR migrates from Azure Bot Service bot registration to a new endpoint registration system using Agent Blueprint Identity. The changes replace Azure CLI-based bot creation with direct HTTP API calls to Agent365 endpoints for registering and deleting messaging endpoints.

Key changes:

• Replaced CreateOrUpdateBotWithAgentBlueprintAsync with CreateEndpointWithAgentBlueprintAsync that calls REST APIs directly
• Added DeleteEndpointWithAgentBlueprintAsync for endpoint cleanup
• Removed Bot Service provider registration and channel configuration logic
• Updated naming convention from "bot" to "endpoint" throughout the codebase

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 15 comments.

Show a summary per file

File	Description
BotConfigurator.cs	Refactored to use HTTP API calls for endpoint registration instead of Azure CLI bot commands; added new dependencies for config and auth services
BotConfiguratorTests.cs	Removed extensive bot-related tests; updated remaining tests to use new endpoint method signatures
SetupCommandTests.cs	Updated mock call verification to match new endpoint method signature
CleanupCommandTests.cs	Added BotConfigurator mock initialization to support new cleanup flow
SetupCommand.cs	Removed bot service provider registration; updated to call new endpoint registration method
CreateInstanceCommand.cs	Simplified bot configuration retrieval by using agent blueprint ID directly instead of querying Azure
CleanupCommand.cs	Updated cleanup logic to call new endpoint deletion method instead of Azure CLI bot deletion
Program.cs	Added BotConfigurator parameter to CleanupCommand initialization
Agent365Config.cs	Changed bot name suffix from "-bot" to "-endpoint"
ConfigConstants.cs	Added endpoint URL constants and helper methods for create/delete operations

Comments suppressed due to low confidence (1)

src/Microsoft.Agents.A365.DevTools.Cli/Commands/SetupCommand.cs:1

• Extra space after comma. Should be 'botName, location' instead of 'botName, location'.

// Copyright (c) Microsoft Corporation.

[Copilot]

Pull Request Overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 16 comments.

Comments suppressed due to low confidence (4)

src/Tests/Microsoft.Agents.A365.DevTools.Cli.Tests/Commands/CleanupCommandTests.cs:32

• This assignment to mockBotLogger is useless, since its value is never read.

        var mockBotLogger = Substitute.For<ILogger<IBotConfigurator>>();

src/Tests/Microsoft.Agents.A365.DevTools.Cli.Tests/Commands/CleanupCommandTests.cs:34

• This assignment to mockAuthService is useless, since its value is never read.

        var mockAuthService = Substitute.ForPartsOf<AuthenticationService>(mockAuthLogger);

src/Tests/Microsoft.Agents.A365.DevTools.Cli.Tests/Commands/CreateInstanceCommandTests.cs:34

• This assignment to mockAuthService is useless, since its value is never read.

        var mockAuthService = Substitute.ForPartsOf<AuthenticationService>(mockAuthLogger);

src/Tests/Microsoft.Agents.A365.DevTools.Cli.Tests/Commands/SetupCommandTests.cs:55

• This assignment to mockAuthService is useless, since its value is never read.

        var mockAuthService = Substitute.ForPartsOf<AuthenticationService>(mockAuthLogger);

[Copilot]

Pull Request Overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 8 comments.

[Copilot]

Pull Request Overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 15 comments.

Comments suppressed due to low confidence (2)

src/Tests/Microsoft.Agents.A365.DevTools.Cli.Tests/Commands/CleanupCommandTests.cs:32

• This assignment to mockBotLogger is useless, since its value is never read.

        var mockBotLogger = Substitute.For<ILogger<IBotConfigurator>>();

src/Tests/Microsoft.Agents.A365.DevTools.Cli.Tests/Commands/CleanupCommandTests.cs:34

• This assignment to mockAuthService is useless, since its value is never read.

        var mockAuthService = Substitute.ForPartsOf<AuthenticationService>(mockAuthLogger);

[Copilot]

Pull Request Overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 10 comments.

Comments suppressed due to low confidence (1)

src/Microsoft.Agents.A365.DevTools.Cli/Commands/CleanupCommand.cs:182

• This assignment to endpointName is useless, since its value is never read.

                    var endpointName = config.BotName.Length > 42
                        ? config.BotName.Substring(0, 42)
                        : config.BotName;

[Copilot]

Pull Request Overview

Copilot reviewed 19 out of 19 changed files in this pull request and generated 8 comments.

Comments suppressed due to low confidence (3)

src/Microsoft.Agents.A365.DevTools.Cli/Services/Helpers/EndpointHelper.cs:89

• Corrected spelling of 'Environment' to 'environment' in heading to maintain consistency.
  src/Tests/Microsoft.Agents.A365.DevTools.Cli.Tests/Commands/CleanupCommandTests.cs:32
• This assignment to mockBotLogger is useless, since its value is never read.

        var mockBotLogger = Substitute.For<ILogger<IBotConfigurator>>();

src/Tests/Microsoft.Agents.A365.DevTools.Cli.Tests/Commands/CleanupCommandTests.cs:34

• This assignment to mockAuthService is useless, since its value is never read.

        var mockAuthService = Substitute.ForPartsOf<AuthenticationService>(mockAuthLogger);

[Copilot]

Pull Request Overview

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

[Copilot]

Pull Request Overview

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