Add User.Read scope for /me Graph call

src/Microsoft.Agents.A365.DevTools.Cli/Services/InteractiveGraphAuthService.cs

@@ -33,7 +33,8 @@ public sealed class InteractiveGraphAuthService
     {
         "https://graph.microsoft.com/Application.ReadWrite.All",
         "https://graph.microsoft.com/AgentIdentityBlueprint.ReadWrite.All",
-        "https://graph.microsoft.com/AgentIdentityBlueprint.UpdateAuthProperties.All"
+        "https://graph.microsoft.com/AgentIdentityBlueprint.UpdateAuthProperties.All",
+        "https://graph.microsoft.com/User.Read"
     };
 
     public InteractiveGraphAuthService(

src/Tests/Microsoft.Agents.A365.DevTools.Cli.Tests/Services/InteractiveGraphAuthServiceTests.cs

@@ -0,0 +1,102 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT License.
+
+using Microsoft.Agents.A365.DevTools.Cli.Services;
+using Microsoft.Extensions.Logging;
+using NSubstitute;
+using Xunit;
+
+namespace Microsoft.Agents.A365.DevTools.Cli.Tests.Services;
+
+public class InteractiveGraphAuthServiceTests
+{
+    /// <summary>
+    /// This test ensures that all required Graph API scopes are present in the RequiredScopes array.
+    /// If any of these scopes are removed, the test will fail to prevent accidental permission reduction.
+    /// 
+    /// These scopes are critical for Agent Blueprint creation and inheritable permissions configuration:
+    /// - Application.ReadWrite.All: Required for creating and managing app registrations
+    /// - AgentIdentityBlueprint.ReadWrite.All: Required for Agent Blueprint operations
+    /// - AgentIdentityBlueprint.UpdateAuthProperties.All: Required for updating blueprint auth properties
+    /// - User.Read: Basic user profile access for authentication context
+    /// </summary>
+    [Fact]
+    public void RequiredScopes_MustContainAllEssentialPermissions()
+    {
+        // Arrange
+        var expectedScopes = new[]
+        {
+            "https://graph.microsoft.com/Application.ReadWrite.All",
+            "https://graph.microsoft.com/AgentIdentityBlueprint.ReadWrite.All", 
+            "https://graph.microsoft.com/AgentIdentityBlueprint.UpdateAuthProperties.All",
+            "https://graph.microsoft.com/User.Read"
+        };
+
+        var logger = Substitute.For<ILogger<InteractiveGraphAuthService>>();
+        var service = new InteractiveGraphAuthService(logger, "12345678-1234-1234-1234-123456789abc");
+
+        // Act - Use reflection to access the private static RequiredScopes field
+        var requiredScopesField = typeof(InteractiveGraphAuthService)
+            .GetField("RequiredScopes", System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Static);
+
+        Assert.NotNull(requiredScopesField);
+        var actualScopes = (string[])requiredScopesField.GetValue(null)!;
+
+        // Assert
+        Assert.NotNull(actualScopes);
+        Assert.Equal(expectedScopes.Length, actualScopes.Length);
+
+        foreach (var expectedScope in expectedScopes)
+        {
+            Assert.Contains(expectedScope, actualScopes);
+        }
+    }
+
+    [Fact]
+    public void Constructor_WithValidGuidClientAppId_ShouldSucceed()
+    {
+        // Arrange
+        var logger = Substitute.For<ILogger<InteractiveGraphAuthService>>();
+        var validGuid = "12345678-1234-1234-1234-123456789abc";
+
+        // Act & Assert - Should not throw
+        var service = new InteractiveGraphAuthService(logger, validGuid);
+        Assert.NotNull(service);
+    }
+
+    [Theory]
+    [InlineData(null)]
+    [InlineData("")]
+    [InlineData("   ")]
+    public void Constructor_WithNullOrEmptyClientAppId_ShouldThrowArgumentNullException(string? clientAppId)
+    {
+        // Arrange
+        var logger = Substitute.For<ILogger<InteractiveGraphAuthService>>();
+
+        // Act & Assert
+        Assert.Throws<ArgumentNullException>(() => new InteractiveGraphAuthService(logger, clientAppId!));
+    }
+
+    [Theory]
+    [InlineData("not-a-guid")]
+    [InlineData("12345")]
+    [InlineData("invalid-format")]
+    public void Constructor_WithInvalidGuidClientAppId_ShouldThrowArgumentException(string clientAppId)
+    {
+        // Arrange
+        var logger = Substitute.For<ILogger<InteractiveGraphAuthService>>();
+
+        // Act & Assert
+        Assert.Throws<ArgumentException>(() => new InteractiveGraphAuthService(logger, clientAppId));
+    }
+
+    [Fact]
+    public void Constructor_WithNullLogger_ShouldThrowArgumentNullException()
+    {
+        // Arrange
+        var validGuid = "12345678-1234-1234-1234-123456789abc";
+
+        // Act & Assert
+        Assert.Throws<ArgumentNullException>(() => new InteractiveGraphAuthService(null!, validGuid));
+    }
+}