Fix external authentication setting field identifiers

Author: sfmskywalker

src/modules/Elsa.ExternalAuthentication/Services/ExtensionDescriptorValidator.cs

@@ -10,6 +10,7 @@ namespace Elsa.ExternalAuthentication.Services;
 public sealed class ExtensionDescriptorValidator
 {
     private static readonly Regex IdentifierPattern = new("^[a-z][a-z0-9]*(?:[-.][a-z0-9]+)*$", RegexOptions.CultureInvariant);
+    private static readonly Regex SettingFieldNamePattern = new("^[a-z][A-Za-z0-9]*$", RegexOptions.CultureInvariant);
     private static readonly HashSet<string> SupportedValueTypes = new(StringComparer.Ordinal)
     {
         "string", "secret", "boolean", "integer", "number", "uri", "string-array", "json"
@@ -79,7 +80,7 @@ private static void ValidateFields(string extensionType, IReadOnlyCollection<Set
         var names = new HashSet<string>(StringComparer.Ordinal);
         foreach (var field in fields)
         {
-            if (!IsIdentifier(field.Name))
+            if (!IsSettingFieldName(field.Name))
                 failures.Add($"Extension '{extensionType}' has invalid field name '{field.Name}'.");
             else if (!names.Add(field.Name))
                 failures.Add($"Extension '{extensionType}' defines field '{field.Name}' more than once.");
@@ -123,4 +124,7 @@ field.Validation.MaximumLength is < 0 ||
 
     private static bool IsIdentifier(string? value) =>
         !string.IsNullOrWhiteSpace(value) && IdentifierPattern.IsMatch(value);
+
+    private static bool IsSettingFieldName(string? value) =>
+        !string.IsNullOrWhiteSpace(value) && SettingFieldNamePattern.IsMatch(value);
 }

test/unit/Elsa.ExternalAuthentication.UnitTests/Extensibility/ExtensionConformanceTests.cs

@@ -2,6 +2,7 @@
 using Elsa.ExternalAuthentication.Contracts;
 using Elsa.ExternalAuthentication.Models;
 using Elsa.ExternalAuthentication.Options;
+using Elsa.ExternalAuthentication.Policies;
 using Elsa.ExternalAuthentication.Services;
 
 namespace Elsa.ExternalAuthentication.UnitTests.Extensibility;
@@ -80,6 +81,23 @@ public void DescriptorValidatorRejectsMismatchedUnsafeAndIncompleteMetadata()
         Assert.Contains("custom-editor", exception.Message);
     }
 
+    [Fact]
+    public void DescriptorValidatorAcceptsLowerCamelCaseSettingFieldNames()
+    {
+        var descriptor = new ExtensionDescriptorValidator().Validate(new CreateUserUnlinkedIdentityPolicy());
+
+        Assert.Contains(descriptor.Fields, field => field.Name == "defaultRoleIds");
+    }
+
+    [Fact]
+    public void DescriptorValidatorKeepsExtensionTypesLowercaseAndStable()
+    {
+        var exception = Assert.Throws<InvalidOperationException>(() =>
+            new ExtensionDescriptorValidator().Validate(new ConformanceAdapter("InvalidType")));
+
+        Assert.Contains("stable identifier", exception.Message);
+    }
+
     [Fact]
     public async Task SettingsMigrationRunsAdapterOwnedForwardStepsAndRejectsUnsupportedVersions()
     {