more clippy linting fixes

Author: untra

.github/workflows/vscode-extension.yaml

@@ -37,6 +37,9 @@ jobs:
       - name: Install dependencies
         run: npm ci
 
+      - name: Copy generated types
+        run: npm run copy-types
+
       - name: Lint
         run: npm run lint
 

src/agents/agent_switcher.rs

@@ -292,6 +292,7 @@ mod tests {
     use crate::agents::tmux::MockTmuxClient;
     use crate::config::{Config, Delegator};
     use crate::templates::schema::{PermissionMode, ReviewType, StepSchema};
+    use std::collections::HashMap;
 
     fn make_step(agent: Option<&str>) -> StepSchema {
         StepSchema {
@@ -327,7 +328,7 @@ mod tests {
             llm_tool: tool.to_string(),
             model: model.to_string(),
             display_name: None,
-            model_properties: Default::default(),
+            model_properties: HashMap::default(),
             launch_config: None,
         }
     }

src/agents/hooks/mod.rs

@@ -226,7 +226,7 @@ mod tests {
         let signal = signal.unwrap();
         assert_eq!(signal.event, "stop");
         assert_eq!(signal.session_id, "test-session");
-        assert_eq!(signal.timestamp, 1234567890);
+        assert_eq!(signal.timestamp, 1_234_567_890);
     }
 
     #[test]

src/agents/monitor.rs

@@ -965,7 +965,7 @@ mod tests {
         std::fs::create_dir_all(&signal_dir).unwrap();
         let signal = HookSignal {
             event: "stop".to_string(),
-            timestamp: 1234567890,
+            timestamp: 1_234_567_890,
             session_id: agent_id.to_string(),
         };
         let signal_path = signal_dir.join(format!("{agent_id}.signal"));

src/api/anthropic.rs

@@ -252,14 +252,14 @@ mod tests {
     #[test]
     fn test_rate_limit_info_summary() {
         let info = RateLimitInfo {
-            input_tokens_limit: Some(100000),
+            input_tokens_limit: Some(100_000),
             input_tokens_remaining: Some(87000),
             ..Default::default()
         };
         assert_eq!(info.summary(), "87% input");
 
         let info = RateLimitInfo {
-            tokens_limit: Some(100000),
+            tokens_limit: Some(100_000),
             tokens_remaining: Some(45000),
             ..Default::default()
         };
@@ -275,7 +275,7 @@ mod tests {
     #[test]
     fn test_is_below_threshold() {
         let info = RateLimitInfo {
-            input_tokens_limit: Some(100000),
+            input_tokens_limit: Some(100_000),
             input_tokens_remaining: Some(15000), // 15%
             ..Default::default()
         };
@@ -286,7 +286,7 @@ mod tests {
     #[test]
     fn test_tokens_remaining_pct() {
         let info = RateLimitInfo {
-            tokens_limit: Some(100000),
+            tokens_limit: Some(100_000),
             tokens_remaining: Some(50000),
             ..Default::default()
         };

src/api/providers/ai/anthropic.rs

@@ -243,7 +243,7 @@ mod tests {
         let info = AnthropicProvider::parse_rate_limit_headers(&headers);
 
         assert_eq!(info.provider, "anthropic");
-        assert_eq!(info.tokens_limit, Some(100000));
+        assert_eq!(info.tokens_limit, Some(100_000));
         assert_eq!(info.tokens_remaining, Some(75000));
         assert_eq!(info.input_tokens_limit, Some(50000));
         assert_eq!(info.input_tokens_remaining, Some(40000));

src/api/providers/ai/mod.rs

@@ -155,12 +155,12 @@ mod tests {
     #[test]
     fn test_rate_limit_info_summary() {
         let mut info = RateLimitInfo::new("anthropic");
-        info.input_tokens_limit = Some(100000);
+        info.input_tokens_limit = Some(100_000);
         info.input_tokens_remaining = Some(87000);
         assert_eq!(info.summary(), "87% input");
 
         let mut info = RateLimitInfo::new("openai");
-        info.tokens_limit = Some(100000);
+        info.tokens_limit = Some(100_000);
         info.tokens_remaining = Some(45000);
         assert_eq!(info.summary(), "45% tokens");
 
@@ -173,7 +173,7 @@ mod tests {
     #[test]
     fn test_is_below_threshold() {
         let mut info = RateLimitInfo::new("anthropic");
-        info.input_tokens_limit = Some(100000);
+        info.input_tokens_limit = Some(100_000);
         info.input_tokens_remaining = Some(15000); // 15%
 
         assert!(info.is_below_threshold(0.2)); // Below 20%

src/queue/watcher.rs

@@ -86,7 +86,9 @@ impl QueueWatcher {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use notify::event::{AccessKind, CreateKind, DataChange, ModifyKind, RemoveKind};
+    use notify::event::{
+        AccessKind, CreateKind, DataChange, EventAttributes, ModifyKind, RemoveKind,
+    };
     use notify::EventKind;
     use std::path::PathBuf;
     use tempfile::TempDir;
@@ -108,7 +110,7 @@ mod tests {
         Event {
             kind,
             paths: vec![path],
-            attrs: Default::default(),
+            attrs: EventAttributes::default(),
         }
     }
 
@@ -187,7 +189,7 @@ mod tests {
         let event = Event {
             kind: EventKind::Create(CreateKind::File),
             paths: vec![],
-            attrs: Default::default(),
+            attrs: EventAttributes::default(),
         };
 
         let result = watcher.classify_event(event);

tests/git_integration.rs

@@ -90,7 +90,7 @@ fn test_ticket_id() -> String {
         .duration_since(std::time::UNIX_EPOCH)
         .unwrap()
         .as_millis();
-    format!("OPTEST-{}", timestamp % 100000)
+    format!("OPTEST-{}", timestamp % 100_000)
 }
 
 /// Get the path to the operator repository (current working directory)