fix: clear clippy 1.96 drift + bump rustls-webpki (unblock CI) (#97)

* fix: unblock CI — clippy build.rs commas + rustls-webpki advisory

Two pre-existing failures on main (toolchain + advisory-db drift) that block
every PR:
- crates/rpg-parser/build.rs: remove 3 unnecessary trailing commas in format!
  args (newer stable clippy flags them under -D warnings).
- Cargo.lock: bump rustls-webpki 0.103.11 -> 0.103.13, resolving
  RUSTSEC-2026-0104 (reachable panic in CRL parsing) and RUSTSEC-2026-0099.

* fix: clear all clippy 1.96 drift + bump rustls-webpki to 0.103.13

CI uses dtolnay/rust-toolchain@stable (now 1.96), which flags lints absent in
older clippy. Resolve every workspace lint under -D warnings:
- sort_by -> sort_by_key(Reverse(..)): graph.rs, cycles.rs, snapshot.rs, and the
  build.rs codegen for lang_registry.
- unnecessary trailing commas in format! args: build.rs, tools.rs.
- iterating a map's values: semantic_lifting.rs (file_index.values()).
- collapsible match arm: deps.rs (method_call_expression only).
- duration_suboptimal_units: provider.rs from_secs(120) -> from_mins(2).
Also bump rustls-webpki 0.103.11 -> 0.103.13 (RUSTSEC-2026-0099/0104).

Author: userFRM

Cargo.lock

@@ -526,7 +526,7 @@ impl RPGraph {
             }
         }
         let mut result: Vec<(String, Vec<String>)> = by_file.into_iter().collect();
-        result.sort_by(|a, b| b.1.len().cmp(&a.1.len()));
+        result.sort_by_key(|x| std::cmp::Reverse(x.1.len()));
         result
     }
 

crates/rpg-core/src/graph.rs

@@ -119,8 +119,8 @@ pub fn apply_features(
 pub fn aggregate_module_features(graph: &mut RPGraph) -> usize {
     let module_data: Vec<(String, Vec<String>)> = graph
         .file_index
-        .iter()
-        .filter_map(|(_, ids)| {
+        .values()
+        .filter_map(|ids| {
             let module_id = ids.iter().find(|id| {
                 graph
                     .entities

crates/rpg-encoder/src/semantic_lifting.rs

@@ -67,7 +67,7 @@ impl AnthropicProvider {
             model: model.unwrap_or_else(|| Self::DEFAULT_MODEL.to_string()),
             agent: ureq::Agent::new_with_config(
                 ureq::config::Config::builder()
-                    .timeout_global(Some(std::time::Duration::from_secs(120)))
+                    .timeout_global(Some(std::time::Duration::from_mins(2)))
                     .build(),
             ),
         }
@@ -193,7 +193,7 @@ impl OpenAiProvider {
             base_url: base_url.unwrap_or_else(|| Self::DEFAULT_BASE_URL.to_string()),
             agent: ureq::Agent::new_with_config(
                 ureq::config::Config::builder()
-                    .timeout_global(Some(std::time::Duration::from_secs(120)))
+                    .timeout_global(Some(std::time::Duration::from_mins(2)))
                     .build(),
             ),
         }

crates/rpg-lift/src/provider.rs

@@ -1867,10 +1867,10 @@ impl RpgServer {
                     } else {
                         cat_node.semantic_features.join(", ")
                     };
-                    result.push_str(&format!("  - {}/{}: {}\n", area_name, cat_name, cat_feats,));
+                    result.push_str(&format!("  - {}/{}: {}\n", area_name, cat_name, cat_feats));
                     for sub_name in cat_node.children.keys() {
                         result
-                            .push_str(&format!("    - {}/{}/{}\n", area_name, cat_name, sub_name,));
+                            .push_str(&format!("    - {}/{}/{}\n", area_name, cat_name, sub_name));
                     }
                 }
             }
@@ -2914,7 +2914,7 @@ impl RpgServer {
         );
 
         if result.primary_entities.is_empty() {
-            return Ok(format!("{}No entities found for: {}", notice, params.query,));
+            return Ok(format!("{}No entities found for: {}", notice, params.query));
         }
 
         Ok(format!(

crates/rpg-mcp/src/tools.rs

@@ -423,7 +423,7 @@ pub fn detect_cycles(graph: &RPGraph, config: &CycleConfig) -> CycleReport {
             stats
         })
         .collect();
-    area_breakdown.sort_by(|a, b| b.cycle_count.cmp(&a.cycle_count));
+    area_breakdown.sort_by_key(|x| std::cmp::Reverse(x.cycle_count));
 
     let areas_in_cycles = area_breakdown.len();
 

crates/rpg-nav/src/cycles.rs

@@ -299,7 +299,7 @@ fn build_hot_spots(graph: &RPGraph, top_n: usize) -> Vec<HotSpot> {
             (id.as_str(), connections)
         })
         .collect();
-    scored.sort_by(|a, b| b.1.cmp(&a.1));
+    scored.sort_by_key(|x| std::cmp::Reverse(x.1));
 
     scored
         .into_iter()

crates/rpg-nav/src/snapshot.rs

@@ -291,7 +291,7 @@ fn generate_lang_registry(defs: &[LangToml]) -> String {
          \x20           .filter(|(_, count)| **count > 0)\n\
          \x20           .map(|(idx, count)| (Self::from_index(idx), *count))\n\
          \x20           .collect();\n\
-         \x20       langs.sort_by(|a, b| b.1.cmp(&a.1));\n\
+         \x20       langs.sort_by_key(|x| ::std::cmp::Reverse(x.1));\n\
          \x20       langs.into_iter().map(|(lang, _)| lang).collect()\n\
          \x20   }\n\n\
          \x20   /// Count files per language in the project.\n\
@@ -373,7 +373,7 @@ fn generate_lang_registry(defs: &[LangToml]) -> String {
     );
     for def in defs {
         if !def.grammar.aliases.is_empty() {
-            code.push_str(&format!("        \"{}\" => match file_ext {{\n", def.name,));
+            code.push_str(&format!("        \"{}\" => match file_ext {{\n", def.name));
             for alias in &def.grammar.aliases {
                 let ext_patterns: Vec<String> = alias
                     .for_extensions
@@ -429,7 +429,7 @@ fn generate_lang_registry(defs: &[LangToml]) -> String {
         if let Some(ref builtin) = def.builtin
             && let Some(ref extractor) = builtin.dep_extractor
         {
-            code.push_str(&format!("        {} => Some(\"{}\"),\n", i, extractor,));
+            code.push_str(&format!("        {} => Some(\"{}\"),\n", i, extractor));
         }
     }
     code.push_str("        _ => None,\n    }\n}\n\n");
@@ -444,7 +444,7 @@ fn generate_lang_registry(defs: &[LangToml]) -> String {
         if let Some(ref builtin) = def.builtin
             && let Some(ref extractor) = builtin.entity_extractor
         {
-            code.push_str(&format!("        {} => Some(\"{}\"),\n", i, extractor,));
+            code.push_str(&format!("        {} => Some(\"{}\"),\n", i, extractor));
         }
     }
     code.push_str("        _ => None,\n    }\n}\n");

crates/rpg-parser/build.rs

@@ -360,22 +360,19 @@ fn collect_rust_calls(
                     }
                 }
             }
-            "method_call_expression" | "field_expression" => {
-                // obj.method() — tree-sitter-rust uses "method_call_expression" for x.foo()
-                // but some versions nest it differently
-                if child.kind() == "method_call_expression" {
-                    // The method name is in the "name" field
-                    if let Some(method_node) = child.child_by_field_name("name") {
-                        let callee = source[method_node.byte_range()].to_string();
-                        if !callee.is_empty() {
-                            let call_row = child.start_position().row;
-                            let caller = find_enclosing_scope(scopes, call_row)
-                                .unwrap_or_else(|| "<module>".to_string());
-                            calls.push(CallDep {
-                                caller_entity: caller,
-                                callee,
-                            });
-                        }
+            "method_call_expression" => {
+                // obj.method() — tree-sitter-rust uses "method_call_expression" for x.foo().
+                // The method name is in the "name" field.
+                if let Some(method_node) = child.child_by_field_name("name") {
+                    let callee = source[method_node.byte_range()].to_string();
+                    if !callee.is_empty() {
+                        let call_row = child.start_position().row;
+                        let caller = find_enclosing_scope(scopes, call_row)
+                            .unwrap_or_else(|| "<module>".to_string());
+                        calls.push(CallDep {
+                            caller_entity: caller,
+                            callee,
+                        });
                     }
                 }
             }