From 2323a57c3974971cd786312f3ab6e406129ff567 Mon Sep 17 00:00:00 2001
From: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>
Date: Sat, 15 Jun 2024 21:19:44 +0300
Subject: [PATCH] asd

---
 CliFx/Schema/IInputSchema.cs                  | 15 ++---------
 CliFx/Schema/OptionSchema.cs                  |  4 +--
 CliFx/Schema/ParameterSchema.cs               |  4 +--
 ...opertyDescriptor.cs => PropertyBinding.cs} | 25 ++++++++++++-------
 4 files changed, 22 insertions(+), 26 deletions(-)
 rename CliFx/Schema/{PropertyDescriptor.cs => PropertyBinding.cs} (54%)

diff --git a/CliFx/Schema/IInputSchema.cs b/CliFx/Schema/IInputSchema.cs
index 790c690..8ca594c 100644
--- a/CliFx/Schema/IInputSchema.cs
+++ b/CliFx/Schema/IInputSchema.cs
@@ -9,20 +9,9 @@ namespace CliFx.Schema;
 public interface IInputSchema
 {
     /// <summary>
-    /// Information about the property that this input is bound to.
+    /// Describes the binding of this input to a CLR property.
     /// </summary>
-    PropertyDescriptor Property { get; }
-
-    /// <summary>
-    /// Whether this input is a scalar (single value) or a sequence (multiple values).
-    /// </summary>
-    bool IsScalar { get; }
-
-    /// <summary>
-    /// Valid values for this input, if applicable.
-    /// If the input does not have a predefined set of valid values, this property is <c>null</c>.
-    /// </summary>
-    IReadOnlyList<object?>? ValidValues { get; }
+    PropertyBinding Property { get; }
 
     /// <summary>
     /// Optional binding converter for this input.
diff --git a/CliFx/Schema/OptionSchema.cs b/CliFx/Schema/OptionSchema.cs
index 63cf1ff..7e93bdd 100644
--- a/CliFx/Schema/OptionSchema.cs
+++ b/CliFx/Schema/OptionSchema.cs
@@ -9,7 +9,7 @@ namespace CliFx.Schema;
 /// Describes a command's option.
 /// </summary>
 public class OptionSchema(
-    PropertyDescriptor property,
+    PropertyBinding property,
     bool isScalar,
     IReadOnlyList<object?>? validValues,
     string? name,
@@ -22,7 +22,7 @@ public class OptionSchema(
 ) : IInputSchema
 {
     /// <inheritdoc />
-    public PropertyDescriptor Property { get; } = property;
+    public PropertyBinding Property { get; } = property;
 
     /// <inheritdoc />
     public bool IsScalar { get; } = isScalar;
diff --git a/CliFx/Schema/ParameterSchema.cs b/CliFx/Schema/ParameterSchema.cs
index 2eb0819..b986de4 100644
--- a/CliFx/Schema/ParameterSchema.cs
+++ b/CliFx/Schema/ParameterSchema.cs
@@ -7,7 +7,7 @@ namespace CliFx.Schema;
 /// Describes a command's parameter.
 /// </summary>
 public class ParameterSchema(
-    PropertyDescriptor property,
+    PropertyBinding property,
     bool isScalar,
     IReadOnlyList<object?>? validValues,
     int order,
@@ -19,7 +19,7 @@ public class ParameterSchema(
 ) : IInputSchema
 {
     /// <inheritdoc />
-    public PropertyDescriptor Property { get; } = property;
+    public PropertyBinding Property { get; } = property;
 
     /// <inheritdoc />
     public bool IsScalar { get; } = isScalar;
diff --git a/CliFx/Schema/PropertyDescriptor.cs b/CliFx/Schema/PropertyBinding.cs
similarity index 54%
rename from CliFx/Schema/PropertyDescriptor.cs
rename to CliFx/Schema/PropertyBinding.cs
index 0fed85c..61d4f0c 100644
--- a/CliFx/Schema/PropertyDescriptor.cs
+++ b/CliFx/Schema/PropertyBinding.cs
@@ -1,24 +1,23 @@
 using System;
-using System.Diagnostics.CodeAnalysis;
+using System.Collections.Generic;
+using System.Linq;
 
 namespace CliFx.Schema;
 
 /// <summary>
 /// Describes a CLR property.
 /// </summary>
-public class PropertyDescriptor(
-    [DynamicallyAccessedMembers(DynamicallyAccessedMemberTypes.Interfaces)]
-    Type type,
+public class PropertyBinding(
+    Type propertyType,
     Func<object, object?> getValue,
     Action<object, object?> setValue
-)
+    )
 {
     /// <summary>
-    /// Property's CLR type.
+    /// Underlying property type.
     /// </summary>
-    [DynamicallyAccessedMembers(DynamicallyAccessedMemberTypes.Interfaces)]
-    public Type Type { get; } = type;
-
+    public Type PropertyType { get; } = propertyType;
+    
     /// <summary>
     /// Gets the current value of the property on the specified instance.
     /// </summary>
@@ -29,3 +28,11 @@ public class PropertyDescriptor(
     /// </summary>
     public void SetValue(object instance, object? value) => setValue(instance, value);
 }
+
+internal static class PropertyBindingExtensions
+{
+    public static IReadOnlyList<object?>? TryGetValidValues(this PropertyBinding binding) =>
+        binding.PropertyType.IsEnum
+            ? binding.PropertyType.GetEnumValuesAsUnderlyingType().Cast<object?>().ToArray()
+            : null;
+}
\ No newline at end of file