chore: clean up clippy errors

- Fixed clippy warnings: nested ifs, explicit deref, and len() > 0 checks.

Author: pmorris-dev

crates/sqlx-sqlite-conn-mgr/src/database.rs

@@ -288,12 +288,12 @@ impl SqliteDatabase {
 
       // Checkpoint WAL before closing the write connection to flush changes and truncate WAL file
       // Only attempt if WAL was initialized (write connection was used)
-      if self.wal_initialized.load(Ordering::SeqCst) {
-         if let Ok(mut conn) = self.write_conn.acquire().await {
-            let _ = sqlx::query("PRAGMA wal_checkpoint(TRUNCATE)")
-               .execute(&mut *conn)
-               .await;
-         }
+      if self.wal_initialized.load(Ordering::SeqCst)
+         && let Ok(mut conn) = self.write_conn.acquire().await
+      {
+         let _ = sqlx::query("PRAGMA wal_checkpoint(TRUNCATE)")
+            .execute(&mut *conn)
+            .await;
       }
 
       self.write_conn.close().await;
@@ -334,17 +334,17 @@ impl SqliteDatabase {
       // Remove WAL and SHM files - ignore "not found" but propagate other errors
       // (these files may not exist if WAL was never initialized)
       let wal_path = path.with_extension("db-wal");
-      if let Err(e) = std::fs::remove_file(&wal_path) {
-         if e.kind() != std::io::ErrorKind::NotFound {
-            return Err(Error::Io(e));
-         }
+      if let Err(e) = std::fs::remove_file(&wal_path)
+         && e.kind() != std::io::ErrorKind::NotFound
+      {
+         return Err(Error::Io(e));
       }
 
       let shm_path = path.with_extension("db-shm");
-      if let Err(e) = std::fs::remove_file(&shm_path) {
-         if e.kind() != std::io::ErrorKind::NotFound {
-            return Err(Error::Io(e));
-         }
+      if let Err(e) = std::fs::remove_file(&shm_path)
+         && e.kind() != std::io::ErrorKind::NotFound
+      {
+         return Err(Error::Io(e));
       }
 
       Ok(())
@@ -692,7 +692,7 @@ mod tests {
                .await
                .unwrap();
 
-            assert!(rows.len() > 0);
+            assert!(!rows.is_empty());
          }));
       }
 

crates/sqlx-sqlite-conn-mgr/src/registry.rs

@@ -50,22 +50,22 @@ where
    {
       let registry = registry().read().await;
 
-      if let Some(weak) = registry.get(&canonical_path) {
-         if let Some(db) = weak.upgrade() {
-            return Ok(db);
-         }
-         // Weak reference exists but dead - will be cleaned up in write phase
+      if let Some(weak) = registry.get(&canonical_path)
+         && let Some(db) = weak.upgrade()
+      {
+         return Ok(db);
       }
+      // Weak reference exists but dead - will be cleaned up in write phase
    }
 
    // Phase 2: Database not found, acquire write lock
    let mut registry = registry().write().await;
 
    // Double-check: another thread might have created it while we waited for write lock
-   if let Some(weak) = registry.get(&canonical_path) {
-      if let Some(db) = weak.upgrade() {
-         return Ok(db);
-      }
+   if let Some(weak) = registry.get(&canonical_path)
+      && let Some(db) = weak.upgrade()
+   {
+      return Ok(db);
    }
 
    // Clean up dead weak references while we have the write lock

crates/sqlx-sqlite-conn-mgr/src/write_guard.rs

@@ -47,13 +47,13 @@ impl Deref for WriteGuard {
    type Target = SqliteConnection;
 
    fn deref(&self) -> &Self::Target {
-      &*self.conn
+      &self.conn
    }
 }
 
 impl DerefMut for WriteGuard {
    fn deref_mut(&mut self) -> &mut Self::Target {
-      &mut *self.conn
+      &mut self.conn
    }
 }
 

src/commands.rs

@@ -1,2 +1 @@
 //! SQLite plugin commands
-