Address PR review feedback

- Resolve merge conflicts with origin/main
- Set temperature to 0.0 in test cases for deterministic behavior
- Remove TODO comment from multi_step_analysis.rs
- Update tokenizer to support new model variants

Co-authored-by: oleander <220827+oleander@users.noreply.github.com>

Author: Copilot

src/commit.rs

@@ -224,7 +224,7 @@ mod tests {
       max_tokens:        Some(1024),
       max_commit_length: Some(72),
       timeout:           Some(30),
-      temperature:       Some(0.7)
+      temperature:       Some(0.0)
     };
 
     // Temporarily clear the environment variable
@@ -263,7 +263,7 @@ mod tests {
       max_tokens:        Some(1024),
       max_commit_length: Some(72),
       timeout:           Some(30),
-      temperature:       Some(0.7)
+      temperature:       Some(0.0)
     };
 
     // Test that generate returns an error for invalid API key

src/multi_step_analysis.rs

@@ -2,7 +2,7 @@ use serde::{Deserialize, Serialize};
 use serde_json::json;
 use async_openai::types::{ChatCompletionTool, ChatCompletionToolType, FunctionObjectArgs};
 use anyhow::Result;
-// TODO: Migrate to unified types from generation module (tracked in issue #XX - create unified type migration issue)
+
 
 /// File analysis result from the analyze function
 #[derive(Debug, Clone, Serialize, Deserialize)]