fix(ci): clear clippy warnings (-D warnings)

- needless borrow (cli.rs test) + map_or->is_some_and (cli main) via clippy --fix
- mcp tests: reword handler_env doc to drop the '+' markdown-list marker
  (doc_list_item_without_indentation), and #![allow(await_holding_lock)] on
  the tests module — the guard is intentionally held across .await to
  serialize handler tests on the current-thread runtime.

Author: Shahinyanm

crates/tj-cli/src/main.rs

@@ -4325,7 +4325,7 @@ fn build_dream_inputs(
         )?;
         let tasks: Vec<_> = candidates
             .into_iter()
-            .filter(|t| task_filter.map_or(true, |f| f == t.task_id))
+            .filter(|t| task_filter.is_none_or(|f| f == t.task_id))
             .collect();
         if tasks.is_empty() {
             continue;

crates/tj-cli/tests/cli.rs

@@ -4527,7 +4527,7 @@ fn export_memory_dry_run_prints_path_and_content_no_write() {
         .args(["export-memory", "--task", &task_id, "--dry-run"])
         .assert()
         .success()
-        .stdout(contains(&format!("memory/tj-{task_id}-ship-x.md")))
+        .stdout(contains(format!("memory/tj-{task_id}-ship-x.md")))
         .stdout(contains("name: ship-x"));
 
     // Nothing written under the sandboxed Claude config dir.

crates/tj-mcp/src/main.rs

@@ -743,12 +743,17 @@ async fn main() -> Result<()> {
 
 #[cfg(test)]
 mod tests {
+    // The handler tests intentionally hold the handler_env() mutex across
+    // `.await` to serialize access to the process-global PROJECT_DIR_OVERRIDE
+    // and XDG_DATA_HOME. On a current-thread runtime this is safe.
+    #![allow(clippy::await_holding_lock)]
+
     use super::*;
 
     /// Handler tests touch process-global state (PROJECT_DIR_OVERRIDE OnceLock
-    /// + XDG_DATA_HOME env), so they must run one at a time and share a single
-    /// project dir. This guard serializes them and lazily pins the override +
-    /// XDG to a single persistent tempdir for the whole test binary.
+    /// and the XDG_DATA_HOME env var), so they must run one at a time and share
+    /// a single project dir. This guard serializes them and lazily pins the
+    /// override and XDG to a single persistent tempdir for the whole test binary.
     fn handler_env() -> std::sync::MutexGuard<'static, ()> {
         use std::sync::{Mutex, OnceLock};
         static LOCK: OnceLock<Mutex<()>> = OnceLock::new();