Add comments for service selector enums

Signed-off-by: Hayden B <8418760+haydentherapper@users.noreply.github.com>

Author: Hayden-IO

gen/jsonschema/schemas/ClientTrustConfig.schema.json

@@ -218,7 +218,7 @@
                     ],
                     "type": "string",
                     "title": "Service Selector",
-                    "description": "ServiceSelector specifies how a client should select a set of Services to connect to. A client SHOULD throw an error if the value is SERVICE_SELECTOR_UNDEFINED."
+                    "description": "ServiceSelector specifies how a client SHOULD select a set of Services to connect to. A client SHOULD throw an error if the value is SERVICE_SELECTOR_UNDEFINED."
                 },
                 "count": {
                     "type": "integer",

gen/jsonschema/schemas/ServiceConfiguration.schema.json

@@ -13,7 +13,7 @@
                     ],
                     "type": "string",
                     "title": "Service Selector",
-                    "description": "ServiceSelector specifies how a client should select a set of Services to connect to. A client SHOULD throw an error if the value is SERVICE_SELECTOR_UNDEFINED."
+                    "description": "ServiceSelector specifies how a client SHOULD select a set of Services to connect to. A client SHOULD throw an error if the value is SERVICE_SELECTOR_UNDEFINED."
                 },
                 "count": {
                     "type": "integer",

gen/jsonschema/schemas/SigningConfig.schema.json

@@ -111,7 +111,7 @@
                     ],
                     "type": "string",
                     "title": "Service Selector",
-                    "description": "ServiceSelector specifies how a client should select a set of Services to connect to. A client SHOULD throw an error if the value is SERVICE_SELECTOR_UNDEFINED."
+                    "description": "ServiceSelector specifies how a client SHOULD select a set of Services to connect to. A client SHOULD throw an error if the value is SERVICE_SELECTOR_UNDEFINED."
                 },
                 "count": {
                     "type": "integer",

gen/pb-go/trustroot/v1/sigstore_trustroot.pb.go

@@ -315,7 +315,7 @@ pub struct ClientTrustConfig {
     #[prost(message, optional, tag = "3")]
     pub signing_config: ::core::option::Option<SigningConfig>,
 }
-/// ServiceSelector specifies how a client should select a set of
+/// ServiceSelector specifies how a client SHOULD select a set of
 /// Services to connect to. A client SHOULD throw an error if
 /// the value is SERVICE_SELECTOR_UNDEFINED.
 #[derive(
@@ -326,8 +326,17 @@ pub struct ClientTrustConfig {
 #[repr(i32)]
 pub enum ServiceSelector {
     Undefined = 0,
+    /// Clients SHOULD select all Services based on supported API version
+    /// and validity window.
     All = 1,
+    /// Clients SHOULD select one Service based on supported API version
+    /// and validity window. It is up to the client implementation to
+    /// decide how to select the Service, e.g. random or round-robin.
     Any = 2,
+    /// Clients SHOULD select a specific number of Services based on
+    /// supported API version and validity window, using the provided
+    /// `count`. It is up to the client implementation to decide how to
+    /// select the Service, e.g. random or round-robin.
     Exact = 3,
 }
 impl ServiceSelector {

gen/pb-python/sigstore_protobuf_specs/dev/sigstore/trustroot/v1/__init__.py

@@ -248,13 +248,22 @@ message Service {
         dev.sigstore.common.v1.TimeRange valid_for = 3;
 }
 
-// ServiceSelector specifies how a client should select a set of
+// ServiceSelector specifies how a client SHOULD select a set of
 // Services to connect to. A client SHOULD throw an error if
 // the value is SERVICE_SELECTOR_UNDEFINED.
 enum ServiceSelector {
         SERVICE_SELECTOR_UNDEFINED = 0;
+        // Clients SHOULD select all Services based on supported API version
+        // and validity window.
         ALL = 1;
+        // Clients SHOULD select one Service based on supported API version
+        // and validity window. It is up to the client implementation to
+        // decide how to select the Service, e.g. random or round-robin. 
         ANY = 2;
+        // Clients SHOULD select a specific number of Services based on
+        // supported API version and validity window, using the provided
+        // `count`. It is up to the client implementation to decide how to
+        // select the Service, e.g. random or round-robin. 
         EXACT = 3;
 }