style: cargo fmt + fix clippy never_loop in catalog-gen

- cargo fmt across files touched in this PR.
- Replace the always-breaking while-let in strip_markdown with a
  straight if-let; same behavior, no clippy warning.

cargo clippy --lib --bins is now clean (one expect_used warning, not
denied). The remaining clippy errors live in pre-existing tests and
examples this PR doesn't touch.

Refs #14

Author: lightsofapollo

src/analysis.rs

@@ -646,10 +646,7 @@ impl SchemaAnalyzer {
         // A document may legitimately have no `components.schemas` (e.g. a
         // webhooks-only or paths-only spec). Return an empty map in that case
         // and let downstream codegen handle "no types to emit" gracefully.
-        let schemas = spec
-            .components
-            .as_ref()
-            .and_then(|c| c.schemas.as_ref());
+        let schemas = spec.components.as_ref().and_then(|c| c.schemas.as_ref());
         Ok(schemas
             .map(|m| {
                 m.iter()
@@ -3470,10 +3467,11 @@ impl SchemaAnalyzer {
         if no_properties {
             // Check for constraints that would make this a structured type.
             // After J5–J8, these are typed fields rather than `extra` lookups.
-            let has_structural_constraints =
-                details.required.as_ref()
-                    .map(|req| req.iter().any(|r| r != "type"))
-                    .unwrap_or(false)
+            let has_structural_constraints = details
+                .required
+                .as_ref()
+                .map(|req| req.iter().any(|r| r != "type"))
+                .unwrap_or(false)
                 || details.pattern_properties.is_some()
                 || details.property_names.is_some()
                 || details.min_properties.is_some()
@@ -3701,10 +3699,7 @@ impl SchemaAnalyzer {
                 // signal so a `stream: true` parameter and an event-stream
                 // content type produce a streaming variant by default.
                 if let Some(content) = response.content.as_ref() {
-                    if content
-                        .keys()
-                        .any(|ct| ct.starts_with("text/event-stream"))
-                    {
+                    if content.keys().any(|ct| ct.starts_with("text/event-stream")) {
                         op_info.supports_streaming = true;
                     }
                 }

src/bin/catalog-gen.rs

@@ -78,8 +78,11 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
 
     catalog.totals.object_count = catalog.objects.len();
     catalog.totals.field_count = catalog.objects.values().map(|o| o.fields.len()).sum();
-    catalog.totals.patterned_field_count =
-        catalog.objects.values().map(|o| o.patterned_fields.len()).sum();
+    catalog.totals.patterned_field_count = catalog
+        .objects
+        .values()
+        .map(|o| o.patterned_fields.len())
+        .sum();
     catalog.totals.json_schema_keyword_count = catalog.json_schema_2020_12_keywords.len();
     catalog.totals.parameter_style_combo_count = catalog.parameter_style_matrix.len();
 
@@ -301,17 +304,12 @@ fn strip_markdown(s: &str) -> String {
             }
             ']' => {
                 in_link_text = false;
-                while let Some(&n) = chars.peek() {
-                    if n == '(' {
-                        chars.next();
-                        for inner in chars.by_ref() {
-                            if inner == ')' {
-                                break;
-                            }
+                if let Some(&'(') = chars.peek() {
+                    chars.next();
+                    for inner in chars.by_ref() {
+                        if inner == ')' {
+                            break;
                         }
-                        break;
-                    } else {
-                        break;
                     }
                 }
             }
@@ -332,27 +330,69 @@ fn strip_markdown(s: &str) -> String {
 fn json_schema_2020_12_keywords() -> Vec<String> {
     [
         // Core
-        "$schema", "$vocabulary", "$id", "$ref", "$anchor", "$dynamicRef", "$dynamicAnchor",
-        "$defs", "$comment",
+        "$schema",
+        "$vocabulary",
+        "$id",
+        "$ref",
+        "$anchor",
+        "$dynamicRef",
+        "$dynamicAnchor",
+        "$defs",
+        "$comment",
         // Applicators
-        "allOf", "anyOf", "oneOf", "not", "if", "then", "else", "dependentSchemas",
-        "prefixItems", "items", "contains", "properties", "patternProperties",
-        "additionalProperties", "propertyNames",
+        "allOf",
+        "anyOf",
+        "oneOf",
+        "not",
+        "if",
+        "then",
+        "else",
+        "dependentSchemas",
+        "prefixItems",
+        "items",
+        "contains",
+        "properties",
+        "patternProperties",
+        "additionalProperties",
+        "propertyNames",
         // Validation
-        "type", "enum", "const",
-        "multipleOf", "maximum", "exclusiveMaximum", "minimum", "exclusiveMinimum",
-        "maxLength", "minLength", "pattern",
-        "maxItems", "minItems", "uniqueItems", "maxContains", "minContains",
-        "maxProperties", "minProperties", "required", "dependentRequired",
+        "type",
+        "enum",
+        "const",
+        "multipleOf",
+        "maximum",
+        "exclusiveMaximum",
+        "minimum",
+        "exclusiveMinimum",
+        "maxLength",
+        "minLength",
+        "pattern",
+        "maxItems",
+        "minItems",
+        "uniqueItems",
+        "maxContains",
+        "minContains",
+        "maxProperties",
+        "minProperties",
+        "required",
+        "dependentRequired",
         // Meta-data
-        "title", "description", "default", "deprecated", "readOnly", "writeOnly",
+        "title",
+        "description",
+        "default",
+        "deprecated",
+        "readOnly",
+        "writeOnly",
         "examples",
         // Format
         "format",
         // Content
-        "contentEncoding", "contentMediaType", "contentSchema",
+        "contentEncoding",
+        "contentMediaType",
+        "contentSchema",
         // Unevaluated
-        "unevaluatedItems", "unevaluatedProperties",
+        "unevaluatedItems",
+        "unevaluatedProperties",
     ]
     .iter()
     .map(|s| s.to_string())
@@ -409,12 +449,33 @@ fn parameter_style_matrix() -> Vec<StyleEntry> {
 
 fn appendices() -> Vec<AppendixEntry> {
     vec![
-        AppendixEntry { id: "A", title: "Revision History" },
-        AppendixEntry { id: "B", title: "Data Type Conversion" },
-        AppendixEntry { id: "C", title: "Using RFC6570-Based Serialization" },
-        AppendixEntry { id: "D", title: "Serializing Headers and Cookies" },
-        AppendixEntry { id: "E", title: "Percent-Encoding and Form Media Types" },
-        AppendixEntry { id: "F", title: "Base URI Determination and Reference Resolution" },
-        AppendixEntry { id: "G", title: "Parsing and Resolution Guidance" },
+        AppendixEntry {
+            id: "A",
+            title: "Revision History",
+        },
+        AppendixEntry {
+            id: "B",
+            title: "Data Type Conversion",
+        },
+        AppendixEntry {
+            id: "C",
+            title: "Using RFC6570-Based Serialization",
+        },
+        AppendixEntry {
+            id: "D",
+            title: "Serializing Headers and Cookies",
+        },
+        AppendixEntry {
+            id: "E",
+            title: "Percent-Encoding and Form Media Types",
+        },
+        AppendixEntry {
+            id: "F",
+            title: "Base URI Determination and Reference Resolution",
+        },
+        AppendixEntry {
+            id: "G",
+            title: "Parsing and Resolution Guidance",
+        },
     ]
 }

src/bin/file-beads.rs

@@ -110,7 +110,10 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
         if apply {
             create_label(&beads.repo, label)?;
         } else {
-            println!("  + label {} ({}, {})", label.name, label.color, label.description);
+            println!(
+                "  + label {} ({}, {})",
+                label.name, label.color, label.description
+            );
         }
     }
 
@@ -176,9 +179,15 @@ fn phase_string(v: &serde_yaml::Value) -> String {
 }
 
 fn phase_to_epic_map() -> BTreeMap<&'static str, &'static str> {
-    [("0", "E0"), ("1", "E1"), ("2", "E2"), ("2b", "E2"), ("3", "E3")]
-        .into_iter()
-        .collect()
+    [
+        ("0", "E0"),
+        ("1", "E1"),
+        ("2", "E2"),
+        ("2b", "E2"),
+        ("3", "E3"),
+    ]
+    .into_iter()
+    .collect()
 }
 
 fn phase_to_label_map() -> BTreeMap<&'static str, &'static str> {
@@ -193,11 +202,7 @@ fn phase_to_label_map() -> BTreeMap<&'static str, &'static str> {
     .collect()
 }
 
-fn bead_labels(
-    bead: &Bead,
-    phase_str: &str,
-    phase_to_label: &BTreeMap<&str, &str>,
-) -> Vec<String> {
+fn bead_labels(bead: &Bead, phase_str: &str, phase_to_label: &BTreeMap<&str, &str>) -> Vec<String> {
     let mut labels = vec!["bead".to_string()];
     if let Some(p) = phase_to_label.get(phase_str) {
         labels.push((*p).to_string());
@@ -374,10 +379,8 @@ fn compose_parallel_plan(beads: &[Bead]) -> String {
         let parallel = max_parallel_groups(group);
         for (i, batch) in parallel.iter().enumerate() {
             let ids: Vec<String> = batch.iter().map(|b| b.id.clone()).collect();
-            let files: BTreeSet<String> = batch
-                .iter()
-                .flat_map(|b| b.files.iter().cloned())
-                .collect();
+            let files: BTreeSet<String> =
+                batch.iter().flat_map(|b| b.files.iter().cloned()).collect();
             s.push_str(&format!(
                 "- Batch {}.{}: {} (touches {})\n",
                 d,
@@ -394,11 +397,7 @@ fn compose_parallel_plan(beads: &[Bead]) -> String {
 fn compute_depths(beads: &[Bead]) -> BTreeMap<String, usize> {
     let by_id: BTreeMap<&str, &Bead> = beads.iter().map(|b| (b.id.as_str(), b)).collect();
     let mut depth = BTreeMap::new();
-    fn dfs(
-        id: &str,
-        by_id: &BTreeMap<&str, &Bead>,
-        depth: &mut BTreeMap<String, usize>,
-    ) -> usize {
+    fn dfs(id: &str, by_id: &BTreeMap<&str, &Bead>, depth: &mut BTreeMap<String, usize>) -> usize {
         if let Some(d) = depth.get(id) {
             return *d;
         }
@@ -453,15 +452,23 @@ fn escape_md(s: &str) -> String {
 fn create_label(repo: &str, label: &Label) -> Result<(), Box<dyn std::error::Error>> {
     let status = Command::new("gh")
         .args([
-            "label", "create", &label.name,
-            "--repo", repo,
-            "--color", &label.color,
-            "--description", &label.description,
+            "label",
+            "create",
+            &label.name,
+            "--repo",
+            repo,
+            "--color",
+            &label.color,
+            "--description",
+            &label.description,
             "--force",
         ])
         .status()?;
     if !status.success() {
-        eprintln!("warning: label create failed for {} (may already exist)", label.name);
+        eprintln!(
+            "warning: label create failed for {} (may already exist)",
+            label.name
+        );
     }
     Ok(())
 }
@@ -474,9 +481,23 @@ fn create_issue(
 ) -> Result<u64, Box<dyn std::error::Error>> {
     // Dedup: skip if an open issue with the same title already exists.
     let existing = Command::new("gh")
-        .args(["issue", "list", "--repo", repo, "--state", "all", "--search", title, "--json", "number,title", "--limit", "20"])
+        .args([
+            "issue",
+            "list",
+            "--repo",
+            repo,
+            "--state",
+            "all",
+            "--search",
+            title,
+            "--json",
+            "number,title",
+            "--limit",
+            "20",
+        ])
         .output()?;
-    let existing_json: serde_json::Value = serde_json::from_slice(&existing.stdout).unwrap_or(serde_json::Value::Array(vec![]));
+    let existing_json: serde_json::Value =
+        serde_json::from_slice(&existing.stdout).unwrap_or(serde_json::Value::Array(vec![]));
     if let Some(arr) = existing_json.as_array() {
         for v in arr {
             if v.get("title").and_then(|t| t.as_str()) == Some(title) {
@@ -490,9 +511,12 @@ fn create_issue(
     let mut args = vec![
         "issue".to_string(),
         "create".to_string(),
-        "--repo".to_string(), repo.to_string(),
-        "--title".to_string(), title.to_string(),
-        "--body".to_string(), body.to_string(),
+        "--repo".to_string(),
+        repo.to_string(),
+        "--title".to_string(),
+        title.to_string(),
+        "--body".to_string(),
+        body.to_string(),
     ];
     for l in labels {
         args.push("--label".to_string());
@@ -504,7 +528,11 @@ fn create_issue(
         return Err(format!("gh issue create failed: {}", stderr).into());
     }
     let url = String::from_utf8_lossy(&output.stdout).trim().to_string();
-    let number = url.rsplit('/').next().and_then(|s| s.parse().ok()).unwrap_or(0);
+    let number = url
+        .rsplit('/')
+        .next()
+        .and_then(|s| s.parse().ok())
+        .unwrap_or(0);
     println!("  + #{} {}", number, title);
     Ok(number)
 }

src/cli.rs

@@ -278,7 +278,10 @@ fn parse_oas_version(s: &str) -> Option<(u32, u32)> {
     let mut parts = s.split('.');
     let major = parts.next()?.parse().ok()?;
     let minor_raw = parts.next()?;
-    let minor_digits: String = minor_raw.chars().take_while(|c| c.is_ascii_digit()).collect();
+    let minor_digits: String = minor_raw
+        .chars()
+        .take_while(|c| c.is_ascii_digit())
+        .collect();
     let minor = minor_digits.parse().ok()?;
     Some((major, minor))
 }

src/openapi.rs

@@ -587,12 +587,10 @@ impl Schema {
     pub fn schema_type(&self) -> Option<&SchemaType> {
         match self {
             Schema::Typed { schema_type, .. } => Some(schema_type),
-            Schema::TypedMulti { schema_types, .. } => {
-                schema_types
-                    .iter()
-                    .find(|t| **t != SchemaType::Null)
-                    .or_else(|| schema_types.first())
-            }
+            Schema::TypedMulti { schema_types, .. } => schema_types
+                .iter()
+                .find(|t| **t != SchemaType::Null)
+                .or_else(|| schema_types.first()),
             _ => None,
         }
     }

src/registry_generator.rs

@@ -175,7 +175,10 @@ impl CodeGenerator {
                 "HEAD" => quote! { HttpMethod::Head },
                 "OPTIONS" => quote! { HttpMethod::Options },
                 "TRACE" => quote! { HttpMethod::Trace },
-                other => panic!("unsupported HTTP method `{other}` for op `{}`", op.operation_id),
+                other => panic!(
+                    "unsupported HTTP method `{other}` for op `{}`",
+                    op.operation_id
+                ),
             };
             let path = &op.path;