remove public setter requirement from modal component definition and add guard clause to inputs

Author: Cenngo

src/Discord.Net.Interactions/Builders/ModuleClassBuilder.cs

@@ -654,6 +654,8 @@ public static ModalInfo BuildModalInfo(Type modalType, InteractionService intera
 
         private static void BuildTextInputComponent(TextInputComponentBuilder builder, PropertyInfo propertyInfo, object defaultValue)
         {
+            EnsurePubliclySettable(propertyInfo);
+
             var attributes = propertyInfo.GetCustomAttributes();
 
             builder.Label = propertyInfo.Name;
@@ -690,6 +692,8 @@ private static void BuildTextInputComponent(TextInputComponentBuilder builder, P
 
         private static void BuildSelectMenuComponent(SelectMenuComponentBuilder builder, PropertyInfo propertyInfo, object defaultValue)
         {
+            EnsurePubliclySettable(propertyInfo);
+
             var attributes = propertyInfo.GetCustomAttributes();
 
             builder.Label = propertyInfo.Name;
@@ -742,6 +746,8 @@ private static void BuildSnowflakeSelectComponent<TInfo, TBuilder>(SnowflakeSele
             where TInfo : SnowflakeSelectComponentInfo
             where TBuilder : SnowflakeSelectComponentBuilder<TInfo, TBuilder>
         {
+            EnsurePubliclySettable(propertyInfo);
+
             var attributes = propertyInfo.GetCustomAttributes();
 
             builder.Label = propertyInfo.Name;
@@ -767,6 +773,9 @@ private static void BuildSnowflakeSelectComponent<TInfo, TBuilder>(SnowflakeSele
                         builder.Label = inputLabel.Label;
                         builder.Description = inputLabel.Description;
                         break;
+                    case ChannelTypesAttribute channelTypes when builder is ChannelSelectComponentBuilder channelSelectBuilder:
+                        channelSelectBuilder.WithChannelTypes(channelTypes.ChannelTypes);
+                        break;
                     default:
                         builder.WithAttributes(attribute);
                         break;
@@ -776,6 +785,8 @@ private static void BuildSnowflakeSelectComponent<TInfo, TBuilder>(SnowflakeSele
 
         private static void BuildFileUploadComponent(FileUploadComponentBuilder builder, PropertyInfo propertyInfo, object defaultValue)
         {
+            EnsurePubliclySettable(propertyInfo);
+
             var attributes = propertyInfo.GetCustomAttributes();
 
             builder.Label = propertyInfo.Name;
@@ -882,9 +893,18 @@ private static bool IsValidModalCommandDefinition(MethodInfo methodInfo)
 
         private static bool IsValidModalComponentDefinition(PropertyInfo propertyInfo)
         {
-            return propertyInfo.SetMethod?.IsPublic == true &&
-                propertyInfo.SetMethod?.IsStatic == false &&
-                propertyInfo.IsDefined(typeof(ModalComponentAttribute));
+            return propertyInfo.IsDefined(typeof(ModalComponentAttribute));
+        }
+
+        private static bool IsPubliclySettable(PropertyInfo propertyInfo)
+        {
+            return propertyInfo.SetMethod is { IsPublic: true, IsStatic: false };
+        }
+
+        private static void EnsurePubliclySettable(PropertyInfo propertyInfo)
+        {
+            if(!IsPubliclySettable(propertyInfo))
+                throw new InvalidOperationException($"The property {propertyInfo.Name} must be publicly settable.");
         }
 
         private static ConstructorInfo GetComplexParameterConstructor(TypeInfo typeInfo, ComplexParameterAttribute complexParameter)