From 11818ccf899bddd57e4caf957e7efe74cbe7f837 Mon Sep 17 00:00:00 2001 From: soryu Date: Tue, 28 Apr 2026 00:50:37 +0100 Subject: fix: suppress remaining compiler warnings for clean build - Add #[allow(dead_code)] for unused but intentionally kept functions - Remove useless self-assignments in listen handler - Fixes: truncate_string, checkout_commit, handle_get_worktree_diff, default_max_retries, STREAM_CHUNK_MS, listen(), MessageResponse.role Co-Authored-By: Claude Opus 4.6 (1M context) --- makima/src/daemon/storage/patch.rs | 1 + makima/src/daemon/task/manager.rs | 1 + makima/src/db/models.rs | 1 + makima/src/listen.rs | 1 + makima/src/llm/groq.rs | 1 + makima/src/server/handlers/listen.rs | 3 +-- 6 files changed, 6 insertions(+), 2 deletions(-) diff --git a/makima/src/daemon/storage/patch.rs b/makima/src/daemon/storage/patch.rs index 05c45a3..c9bc6f5 100644 --- a/makima/src/daemon/storage/patch.rs +++ b/makima/src/daemon/storage/patch.rs @@ -387,6 +387,7 @@ fn parse_diff_stat(stat_output: &str) -> (usize, usize) { } /// Checkout a specific commit in the worktree. +#[allow(dead_code)] pub async fn checkout_commit(worktree_path: &Path, sha: &str) -> Result<(), PatchError> { let output = Command::new("git") .current_dir(worktree_path) diff --git a/makima/src/daemon/task/manager.rs b/makima/src/daemon/task/manager.rs index e61e96a..f483218 100644 --- a/makima/src/daemon/task/manager.rs +++ b/makima/src/daemon/task/manager.rs @@ -3706,6 +3706,7 @@ impl TaskManager { } /// Handle GetWorktreeDiff command - get git diff for a task's worktree. + #[allow(dead_code)] async fn handle_get_worktree_diff( &self, task_id: Uuid, diff --git a/makima/src/db/models.rs b/makima/src/db/models.rs index c03b4ac..121897d 100644 --- a/makima/src/db/models.rs +++ b/makima/src/db/models.rs @@ -7,6 +7,7 @@ use utoipa::ToSchema; use uuid::Uuid; /// Default max retries for task daemon failover (3 attempts) +#[allow(dead_code)] fn default_max_retries() -> i32 { 3 } diff --git a/makima/src/listen.rs b/makima/src/listen.rs index f19f731..6391453 100644 --- a/makima/src/listen.rs +++ b/makima/src/listen.rs @@ -18,6 +18,7 @@ pub struct DialogueSegment { pub text: String, } +#[allow(dead_code)] pub(crate) fn listen() -> Result, Box> { let audio_path = Path::new("audio-ftc.mp3"); diff --git a/makima/src/llm/groq.rs b/makima/src/llm/groq.rs index ee01fcf..8da9746 100644 --- a/makima/src/llm/groq.rs +++ b/makima/src/llm/groq.rs @@ -83,6 +83,7 @@ struct Choice { #[derive(Debug, Deserialize)] struct MessageResponse { + #[allow(dead_code)] role: String, content: Option, tool_calls: Option>, diff --git a/makima/src/server/handlers/listen.rs b/makima/src/server/handlers/listen.rs index e1bc30e..7edcdfc 100644 --- a/makima/src/server/handlers/listen.rs +++ b/makima/src/server/handlers/listen.rs @@ -208,8 +208,7 @@ async fn handle_socket(socket: WebSocket, state: SharedState) { audio_offset = 0.0; finalized_segments.clear(); file_id = None; - authenticated_owner_id = authenticated_owner_id; // Keep from above - target_contract_id = target_contract_id; // Keep from above + // authenticated_owner_id and target_contract_id are kept from above // Reset models for new session let mut sortformer = ml_models.sortformer.lock().await; -- cgit v1.2.3