fix(codegraph-vector): remove unused imports and fix warnings

Remove unused imports and fix minor issues across codegraph-vector crate
to eliminate compilation warnings. All changes are non-breaking.

Changes:
- memory.rs: Remove unused MmapMut import
- openai_provider.rs: Remove unused error import from tracing
- onnx_provider.rs: Remove unused Array import from ndarray
- index.rs: Remove unused GpuResources import, make warn import conditional
- consistency.rs: Remove unused async_trait import, prefix unused node_id parameter
- incremental.rs: Remove unused async_trait, CodeNode, Hash, Hasher, Read imports
- persistent.rs: Remove unused error import, fix unnecessary mut in quantizer training
- storage.rs: Remove unused Read, Write, error, warn, Uuid imports, prefix unused config parameter
- lib.rs: Add allow attribute for ambiguous StorageStats glob re-exports

Summary:
- Reduced warnings from 21 to 0 for default build
- No functional changes or breaking changes
- Library compiles cleanly without warnings

Author: jruokola

crates/codegraph-vector/src/consistency.rs

@@ -1,4 +1,3 @@
-use async_trait::async_trait;
 use codegraph_core::{CodeGraphError, NodeId, Result};
 use parking_lot::{Mutex, RwLock};
 use serde::{Deserialize, Serialize};
@@ -642,7 +641,7 @@ impl ConsistencyManager {
     /// Check if a node is visible to a transaction
     pub fn is_visible(
         &self,
-        node_id: NodeId,
+        _node_id: NodeId,
         transaction_id: u64,
         committed_by: Option<u64>,
     ) -> bool {

crates/codegraph-vector/src/incremental.rs

@@ -1,13 +1,11 @@
-use async_trait::async_trait;
-use codegraph_core::{CodeGraphError, CodeNode, NodeId, Result};
+use codegraph_core::{CodeGraphError, NodeId, Result};
 use crossbeam_channel::{unbounded, Receiver, Sender};
 use dashmap::DashMap;
 use parking_lot::{Mutex, RwLock};
 use rayon::prelude::*;
 use serde::{Deserialize, Serialize};
 use std::collections::{HashMap, HashSet, VecDeque};
-use std::hash::{Hash, Hasher};
-use std::io::{Read, Write};
+use std::io::Write;
 use std::sync::Arc;
 use std::time::{Duration, SystemTime, UNIX_EPOCH};
 use tokio::sync::mpsc as tokio_mpsc;

crates/codegraph-vector/src/index.rs

@@ -9,7 +9,9 @@ use serde::{Deserialize, Serialize};
 use std::collections::HashMap;
 use std::path::PathBuf;
 use std::sync::Arc;
-use tracing::{debug, info, warn};
+use tracing::{debug, info};
+#[cfg(not(feature = "gpu"))]
+use tracing::warn;
 
 /// Configuration for different FAISS index types optimized for various use cases
 #[derive(Debug, Clone, Serialize, Deserialize)]
@@ -227,7 +229,7 @@ impl FaissIndexManager {
         if self.config.gpu_enabled {
             #[cfg(feature = "gpu")]
             {
-                use faiss::gpu::{GpuResources, StandardGpuResources};
+                use faiss::gpu::StandardGpuResources;
 
                 let gpu_resources = StandardGpuResources::new().map_err(|e| {
                     CodeGraphError::Vector(format!("Failed to initialize GPU resources: {}", e))

crates/codegraph-vector/src/lib.rs

@@ -86,6 +86,7 @@ pub use consistency::*;
 #[cfg(feature = "persistent")]
 pub use incremental::*;
 #[cfg(feature = "persistent")]
+#[allow(ambiguous_glob_reexports)]
 pub use persistent::*;
 #[cfg(feature = "persistent")]
 pub use storage::*;

crates/codegraph-vector/src/memory.rs

@@ -1,6 +1,6 @@
 use codegraph_core::{CodeGraphError, Result};
 #[cfg(feature = "persistent")]
-use memmap2::{MmapMut, MmapOptions};
+use memmap2::MmapOptions;
 use parking_lot::{Mutex, RwLock};
 use serde::{Deserialize, Serialize};
 use std::collections::HashMap;

crates/codegraph-vector/src/onnx_provider.rs

@@ -9,7 +9,7 @@ use codegraph_core::{CodeGraphError, CodeNode, Result};
 #[cfg(feature = "onnx")]
 use hf_hub::api::tokio::Api;
 #[cfg(feature = "onnx")]
-use ndarray::{s, Array, Array2, Axis};
+use ndarray::{s, Array2, Axis};
 #[cfg(feature = "onnx")]
 use ort::execution_providers::CoreMLExecutionProvider;
 #[cfg(feature = "onnx")]

crates/codegraph-vector/src/openai_provider.rs

@@ -18,7 +18,7 @@ use std::{
 };
 use tokenizers::Tokenizer;
 use tokio::time::timeout;
-use tracing::{debug, error, info, warn};
+use tracing::{debug, info, warn};
 
 /// Configuration for OpenAI embedding provider
 #[derive(Debug, Clone)]

crates/codegraph-vector/src/persistent.rs

@@ -9,7 +9,7 @@ use std::path::{Path, PathBuf};
 use std::sync::Arc;
 use std::time::{SystemTime, UNIX_EPOCH};
 use tokio::fs;
-use tracing::{debug, error, info, warn};
+use tracing::{debug, info, warn};
 
 /// Header for the memory-mapped vector storage file
 #[derive(Debug, Clone, Serialize, Deserialize)]
@@ -914,13 +914,13 @@ impl PersistentVectorStore {
         }
 
         // Train PQ if enabled
-        if let Some(mut pq) = self.pq_quantizer.write().as_mut() {
+        if let Some(pq) = self.pq_quantizer.write().as_mut() {
             pq.train(sample_vectors)?;
             info!("Trained product quantizer");
         }
 
         // Train SQ if enabled
-        if let Some(mut sq) = self.sq_quantizer.write().as_mut() {
+        if let Some(sq) = self.sq_quantizer.write().as_mut() {
             sq.train(sample_vectors)?;
             info!("Trained scalar quantizer");
         }

crates/codegraph-vector/src/storage.rs

@@ -8,11 +8,10 @@ use parking_lot::RwLock;
 use serde::{Deserialize, Serialize};
 use std::collections::HashMap;
 use std::fs::{File, OpenOptions};
-use std::io::{BufReader, BufWriter, Read, Write};
+use std::io::{BufReader, BufWriter};
 use std::path::{Path, PathBuf};
 use std::sync::Arc;
-use tracing::{debug, error, info, warn};
-use uuid::Uuid;
+use tracing::{debug, info};
 
 /// Metadata for persistent vector storage
 #[derive(Debug, Clone, Serialize, Deserialize)]
@@ -164,7 +163,7 @@ impl PersistentStorage {
     }
 
     /// Load FAISS index from disk with automatic decompression
-    pub fn load_index(&self, config: &IndexConfig) -> Result<IndexImpl> {
+    pub fn load_index(&self, _config: &IndexConfig) -> Result<IndexImpl> {
         let index_path = self.base_path.join(Self::INDEX_FILE);
 
         if !index_path.exists() {