From 6837700ac9c8960f447f0dd72b311ea976b272a2 Mon Sep 17 00:00:00 2001
From: Jake <jake@corbel.consulting>
Date: Tue, 21 Apr 2026 09:06:47 +0100
Subject: [PATCH] style(clippy): clean up the two lints in phase3 new code
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

QC smoke sweep flagged two clippy -D warnings lints in code this branch
introduced:

- crates/core/src/process_watch.rs — collapsible_if on the meeting-pattern
  match loop, merged the two conditions with &&.
- crates/mcp/src/lib.rs — let-else on the id unwrap that short-circuits a
  notification, switched to ? since handle_message already returns Option.

All other clippy lints under -D warnings (audio/capture, hotkey/linux,
storage/file_storage, diagnostics, the duplicate-detection helpers in
live.rs) predate this branch and are out of scope.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 crates/core/src/process_watch.rs | 8 ++++----
 crates/mcp/src/lib.rs            | 4 +---
 2 files changed, 5 insertions(+), 7 deletions(-)

diff --git a/crates/core/src/process_watch.rs b/crates/core/src/process_watch.rs
index f344735..b9ab740 100644
--- a/crates/core/src/process_watch.rs
+++ b/crates/core/src/process_watch.rs
@@ -33,10 +33,10 @@ pub fn match_meeting_patterns(process_names: &[String], patterns: &[String]) ->
         if needle.is_empty() {
             continue;
         }
-        if process_names.iter().any(|name| name.contains(&needle)) {
-            if !matches.iter().any(|existing| existing == &needle) {
-                matches.push(needle);
-            }
+        if process_names.iter().any(|name| name.contains(&needle))
+            && !matches.iter().any(|existing| existing == &needle)
+        {
+            matches.push(needle);
         }
     }
     matches
diff --git a/crates/mcp/src/lib.rs b/crates/mcp/src/lib.rs
index 6e561ab..3a7b7a1 100644
--- a/crates/mcp/src/lib.rs
+++ b/crates/mcp/src/lib.rs
@@ -59,9 +59,7 @@ pub async fn handle_message(pool: &SqlitePool, raw: Value) -> Option<JsonRpcResp
     };
 
     // Notifications: no id, no response.
-    let Some(id) = request.id.clone() else {
-        return None;
-    };
+    let id = request.id.clone()?;
 
     let outcome = match request.method.as_str() {
         "initialize" => Ok(initialize_result()),