fix: resolve clippy lints for nightly-2026-04-24 and matrix-sdk compat

Fix all new clippy lints introduced by the nightly bump:
- collapsible_match: collapse inner `if` into match arm guards
- unnecessary_sort_by: use sort_by_key with Reverse for desc sorts
- question_mark: use `?` operator for early returns
- useless_conversion: remove redundant .into_iter()
- unnecessary cast: remove u32 -> u32 casts
- explicit_counter_loop: use enumerate() instead of manual counter

Also work around matrix-sdk 0.16 query-depth overflow on nightly
>= 2025-12-28 by removing `matrix` from default features and making
the metrics feature forward conditional (`moltis-matrix?/metrics`).
Matrix support is still available via `--features matrix`.

Author: penso

CHANGELOG.md

@@ -19,6 +19,29 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Security
 
+## [20260426.01] - 2026-04-26
+### Added
+- [cron] Add heartbeat wake cooldown to prevent exec re-fire loop (#871)
+
+
+### Changed
+- [config] Extract initialize_config() from discover_and_load()
+- Derive nightly toolchain from rust-toolchain.toml everywhere
+
+
+### Fixed
+- [tests] Stabilize flaky memory_config_get test
+- [ci] Pin NCCL version to match CUDA 12.4 container
+- [ci] Match NCCL dev headers to pre-installed runtime version
+- [ci] Skip libnccl-dev to work around llama-cpp-sys-2 linking bug
+- [ci] Remove pre-installed libnccl-dev from CUDA container
+- [providers] Link NCCL when llama-cpp-sys-2 compiles with GGML_USE_NCCL
+
+
+### Security
+- [ci] Use correct security list-keychains syntax for macOS codesign
+- [ci] Align macOS certificate import with working arbor pattern
+
 ## [20260425.09] - 2026-04-25
 ### Added
 - [cron] Add heartbeat wake cooldown to prevent exec re-fire loop (#871)

crates/agents/src/json_repair.rs

@@ -121,10 +121,8 @@ pub fn repair_json(input: &str) -> Option<serde_json::Value> {
             '"' => in_string = true,
             '{' => stack.push('}'),
             '[' => stack.push(']'),
-            '}' | ']' => {
-                if stack.last() == Some(&ch) {
-                    stack.pop();
-                }
+            '}' | ']' if stack.last() == Some(&ch) => {
+                stack.pop();
             },
             _ => {},
         }

crates/chat/src/types.rs

@@ -526,9 +526,11 @@ pub(crate) fn truncate_at_char_boundary(text: &str, max_bytes: usize) -> &str {
 /// Extract preview text from a single message JSON value.
 pub(crate) fn extract_preview_from_value(msg: &Value) -> Option<String> {
     fn message_text(msg: &Value) -> Option<String> {
-        let text = if let Some(s) = msg.get("content").and_then(|v| v.as_str()) {
+        let content = msg.get("content")?;
+        let text = if let Some(s) = content.as_str() {
             s.to_string()
-        } else if let Some(blocks) = msg.get("content").and_then(|v| v.as_array()) {
+        } else {
+            let blocks = content.as_array()?;
             blocks
                 .iter()
                 .filter_map(|b| {
@@ -540,8 +542,6 @@ pub(crate) fn extract_preview_from_value(msg: &Value) -> Option<String> {
                 })
                 .collect::<Vec<_>>()
                 .join(" ")
-        } else {
-            return None;
         };
         let trimmed = text.trim();
         if trimmed.is_empty() {

crates/cli/Cargo.toml

@@ -120,7 +120,7 @@ default = [
   "llm-compaction",
   "local-llm",
   "local-llm-metal",
-  "matrix",
+  # "matrix" excluded: matrix-sdk 0.16 compiler compat (nightly >= 2025-12-28)
   "mdns",
   "metrics",
   "ngrok",

crates/code-index/src/index.rs

@@ -657,7 +657,7 @@ impl CodeIndex {
                         } else {
                             raw_chunks
                                 .into_iter()
-                                .zip(embeddings.into_iter())
+                                .zip(embeddings)
                                 .enumerate()
                                 .map(|(idx, (chunk, embedding))| StoreChunk {
                                     file_path: chunk.file_path.clone(),

crates/config/src/defaults.rs

@@ -224,13 +224,13 @@ fn collect_shadowed_keys(
             (toml_edit::Item::Table(u), toml_edit::Item::Table(d)) => {
                 collect_shadowed_keys(u, d, &mut path.clone(), out);
             },
-            (toml_edit::Item::Value(u_val), toml_edit::Item::Value(d_val)) => {
+            (toml_edit::Item::Value(u_val), toml_edit::Item::Value(d_val))
                 // Only flag when the user value matches the default — that's
                 // a true shadow (frozen default).  Differing values are
                 // intentional overrides and should not be reported.
-                if u_val.to_string().trim() == d_val.to_string().trim() {
-                    out.push(path);
-                }
+                if u_val.to_string().trim() == d_val.to_string().trim() =>
+            {
+                out.push(path);
             },
             _ => {},
         }

crates/config/src/loader/config_io.rs

@@ -460,10 +460,10 @@ pub fn strip_default_values(effective: &mut toml_edit::Table, defaults: &toml_ed
                     }
                 }
             },
-            (toml_edit::Item::Value(eff_val), toml_edit::Item::Value(def_val)) => {
-                if values_equal(eff_val, def_val) {
-                    effective.remove(&key);
-                }
+            (toml_edit::Item::Value(eff_val), toml_edit::Item::Value(def_val))
+                if values_equal(eff_val, def_val) =>
+            {
+                effective.remove(&key);
             },
             _ => {
                 // Type mismatch (e.g. table vs value) → user override, keep it.
@@ -523,10 +523,10 @@ fn strip_new_default_values(
                     }
                 }
             },
-            (toml_edit::Item::Value(eff_val), toml_edit::Item::Value(def_val)) => {
-                if values_equal(eff_val, def_val) {
-                    effective.remove(&key);
-                }
+            (toml_edit::Item::Value(eff_val), toml_edit::Item::Value(def_val))
+                if values_equal(eff_val, def_val) =>
+            {
+                effective.remove(&key);
             },
             _ => {},
         }

crates/cron/src/heartbeat.rs

@@ -143,9 +143,9 @@ pub fn is_within_active_hours(start: &str, end: &str, timezone: &str) -> bool {
     let end_minutes = if end == "24:00" {
         24 * 60
     } else {
-        end_time.hour() as u32 * 60 + end_time.minute() as u32
+        end_time.hour() * 60 + end_time.minute()
     };
-    let start_minutes = start_time.hour() as u32 * 60 + start_time.minute() as u32;
+    let start_minutes = start_time.hour() * 60 + start_time.minute();
 
     let now_minutes = current_minutes(timezone);
 

crates/gateway/Cargo.toml

@@ -108,7 +108,8 @@ default = [
   "graphql",
   "local-llm",
   "local-llm-metal",
-  "matrix",
+  # "matrix" excluded from defaults: matrix-sdk 0.16 hits compiler query-depth
+  # limit on nightly >= 2025-12-28. Enable explicitly with --features matrix.
   "mdns",
   "metrics",
   "nostr",
@@ -153,7 +154,7 @@ metrics = [
   "dep:moltis-metrics",
   "moltis-chat/metrics",
   "moltis-discord/metrics",
-  "moltis-matrix/metrics",
+  "moltis-matrix?/metrics",
   "moltis-metrics/sqlite",
   "moltis-msteams/metrics",
   "moltis-nostr?/metrics",

crates/gateway/src/session.rs

@@ -118,9 +118,11 @@ fn filter_ui_history(messages: Vec<Value>) -> Vec<Value> {
 
 /// Extract text content from a single message Value.
 fn message_text(msg: &Value) -> Option<String> {
-    let text = if let Some(s) = msg.get("content").and_then(|v| v.as_str()) {
+    let content = msg.get("content")?;
+    let text = if let Some(s) = content.as_str() {
         s.to_string()
-    } else if let Some(blocks) = msg.get("content").and_then(|v| v.as_array()) {
+    } else {
+        let blocks = content.as_array()?;
         blocks
             .iter()
             .filter_map(|b| {
@@ -132,8 +134,6 @@ fn message_text(msg: &Value) -> Option<String> {
             })
             .collect::<Vec<_>>()
             .join(" ")
-    } else {
-        return None;
     };
     let trimmed = text.trim();
     if trimmed.is_empty() {