Minor fixes

src/Bicep.Core/Diagnostics/DiagnosticBuilder.cs

@@ -1001,12 +1001,12 @@ public ErrorDiagnostic VariableLoopsRuntimeDependencyNotAllowed(IEnumerable<stri
             public ErrorDiagnostic FunctionOnlyValidInModuleParameterAssignment(string functionName) => new(
                 TextSpan,
                 "BCP176",
-                $"Function \"{functionName}\" is not valid at this location. It can only be used in assigning value to a module parameter.");
+                $"Function \"{functionName}\" is not valid at this location. It can only be used when assigning a value to a module parameter.");
 
             public ErrorDiagnostic TypeNotValidInStringInterpolation(TypeSymbol type) => new(
                 TextSpan,
                 "BCP177",
-                $"Type \"{type.Name}\" cannot be used inside String interpolation.");
+                $"Type \"{type.Name}\" cannot be used inside string interpolation.");
         }
 
         public static DiagnosticBuilderInternal ForPosition(TextSpan span)

src/Bicep.Core/LanguageConstants.cs

@@ -59,6 +59,7 @@ public static class LanguageConstants
 
         public const string ParameterAllowedPropertyName = "allowed";
         public const string ParameterDefaultPropertyName = "default";
+        public const string ParameterModifierSecureName = "secure";
 
         public const string ModuleParamsPropertyName = "params";
         public const string ModuleOutputsPropertyName = "outputs";
@@ -126,7 +127,6 @@ public static TypeSymbol CreateParameterModifierType(TypeSymbol primitiveType, T
             return new ObjectType($"ParameterModifier<{allowedValuesType.Name}>", TypeSymbolValidationFlags.Default, CreateParameterModifierProperties(primitiveType, allowedValuesType), additionalPropertiesType: null);
         }
 
-        public const string ParameterModifierSecureName = "secure";
         private static IEnumerable<TypeProperty> CreateParameterModifierProperties(TypeSymbol primitiveType, TypeSymbol allowedValuesType)
         {
             /*

src/Bicep.Core/Syntax/ObjectSyntaxExtensions.cs

@@ -116,9 +116,7 @@ public static ObjectSyntax DeepMerge(this ObjectSyntax? sourceObject, ObjectSynt
                     : mergedObject);
         }
 
-        public static DecoratorSyntax? SafeGetDecoaratorByName(this StatementSyntax syntax, string name)
-        {
-            return syntax.Decorators.FirstOrDefault(d => d.Expression is FunctionCallSyntax func && string.Equals(func.Name.IdentifierName, name, System.StringComparison.OrdinalIgnoreCase));
-        }
+        public static DecoratorSyntax? SafeGetDecoratorByName(this StatementSyntax syntax, string name) =>
+            syntax.Decorators.FirstOrDefault(d => d.Expression is FunctionCallSyntaxBase func && string.Equals(func.Name.IdentifierName, name, LanguageConstants.IdentifierComparison));
     }
 }

src/Bicep.Core/Syntax/ParameterDeclarationSyntax.cs

@@ -85,7 +85,7 @@ public TypeSymbol GetAssignedType(ITypeManager typeManager, ArraySyntax? allowed
                     assignedType = UnionType.Create(allowedItemTypes);
                 }
                 else if ((Modifier is ObjectSyntax modifierSyntax && IsBooleanTrue(modifierSyntax.SafeGetPropertyByName(LanguageConstants.ParameterModifierSecureName)?.Value)) ||
-                    (this.SafeGetDecoaratorByName(LanguageConstants.ParameterModifierSecureName) is not null))
+                    (this.SafeGetDecoratorByName(LanguageConstants.ParameterModifierSecureName) is not null))
                 {
                     //parameter can accept either LooseString (see below) or reference to key vault secret
                     assignedType = LanguageConstants.SecureString;
@@ -103,7 +103,7 @@ public TypeSymbol GetAssignedType(ITypeManager typeManager, ArraySyntax? allowed
             {
                 assignedType = new TypedArrayType(UnionType.Create(allowedItemTypes), TypeSymbolValidationFlags.Default);
             }
-
+            
             return assignedType;
         }
     }

src/Bicep.Core/TypeSystem/Az/AzResourceTypeProvider.cs

@@ -227,7 +227,7 @@ private static IEnumerable<FunctionOverload> GetTypeBicepFunctions(ResourceTypeR
                 case "microsoft.keyvault/vaults":
                     yield return new FunctionOverloadBuilder("getSecret")
                         .WithReturnType(LanguageConstants.KeyVaultSecretReference)
-                        .WithDescription("References a secret from this key vault to be binded to a secure string module parameter")
+                        .WithDescription("Gets a reference to a key vault secret, which can be provided to a secure string module parameter")
                         .WithFlags(FunctionFlags.ModuleParamsAssignmentOnly)
                         .WithRequiredParameter("secretName", LanguageConstants.String, "Secret Name")
                         .WithOptionalParameter("secretVersion", LanguageConstants.String, "Secret Version")
@@ -272,4 +272,4 @@ public bool HasType(ResourceTypeReference typeReference)
         public IEnumerable<ResourceTypeReference> GetAvailableTypes()
             => availableResourceTypes;
     }
-}
+}