linter fixes

Author: raginpirate

README.md

@@ -323,7 +323,15 @@ During the transition period the resolved schema **uses the `from` value** as th
 **Shorthand schema transition** (same transition for all operations):
 
 ```json
-{ "ucp_request": { "transition": { "from": "required", "to": "omit", "description": "Removed in v2." } } }
+{
+  "ucp_request": {
+    "transition": {
+      "from": "required",
+      "to": "omit",
+      "description": "Removed in v2."
+    }
+  }
+}
 ```
 
 ### Schema Composition

src/linter.rs

@@ -384,6 +384,7 @@ fn check_annotation_value(
                                 message: format!(
                                     "invalid {} value type: expected string or transition object, got {}",
                                     key,
+                                    json_type_name(val)
                                 ),
                             });
                         }

src/resolver.rs

@@ -4,7 +4,7 @@ use serde_json::{Map, Value};
 
 use crate::error::ResolveError;
 use crate::types::{
-    json_type_name, is_valid_schema_transition, Direction, ResolveOptions, SchemaTransitionInfo,
+    is_valid_schema_transition, json_type_name, Direction, ResolveOptions, SchemaTransitionInfo,
     Visibility, UCP_ANNOTATIONS,
 };
 
@@ -724,7 +724,9 @@ mod tests {
         assert!(result["properties"].get("id").is_some());
         let required = result["required"].as_array().unwrap();
         assert!(required.contains(&json!("id")));
-        let transition = result["properties"]["id"].get("x-ucp-schema-transition").unwrap();
+        let transition = result["properties"]["id"]
+            .get("x-ucp-schema-transition")
+            .unwrap();
         assert_eq!(transition["from"], "required");
         assert_eq!(transition["to"], "optional");
         assert_eq!(transition["description"], "Will become optional in v2.");
@@ -755,7 +757,9 @@ mod tests {
         assert!(result["properties"].get("id").is_some());
         let required = result["required"].as_array().unwrap();
         assert!(!required.contains(&json!("id")));
-        assert!(result["properties"]["id"].get("x-ucp-schema-transition").is_some());
+        assert!(result["properties"]["id"]
+            .get("x-ucp-schema-transition")
+            .is_some());
         assert_eq!(result["properties"]["id"]["deprecated"], true);
     }
 
@@ -790,7 +794,10 @@ mod tests {
         assert!(result["properties"].get("id").is_some());
         let required = result["required"].as_array().unwrap();
         assert!(required.contains(&json!("id")));
-        assert_eq!(result["properties"]["id"]["x-ucp-schema-transition"]["description"], "Removing in v2.");
+        assert_eq!(
+            result["properties"]["id"]["x-ucp-schema-transition"]["description"],
+            "Removing in v2."
+        );
     }
 
     #[test]

src/types.rs

@@ -92,9 +92,7 @@ impl Visibility {
 /// Returns true if (from, to) is a valid schema transition: both are visibility
 /// values (omit, optional, required) and from != to.
 pub fn is_valid_schema_transition(from: &str, to: &str) -> bool {
-    from != to
-        && Visibility::parse(from).is_some()
-        && Visibility::parse(to).is_some()
+    from != to && Visibility::parse(from).is_some() && Visibility::parse(to).is_some()
 }
 
 /// Options for schema resolution.