Fix clippy warnings

AGENTS.md

@@ -148,6 +148,7 @@ mod tests {
 - Use `cargo fmt` for formatting
 - Use `cargo clippy` for linting
 - Ensure documentation for public APIs
+- _ALWAYS_ format and lint after making a change, and fix the linting errors
 
 ### 4. Commit Standards
 - Commit both code and tests together
@@ -207,4 +208,4 @@ mod tests {
 ### Technical Documentation
 - **docs/architecture.md**: Detailed technical specifications, implementation details, and developer-focused content
 - **specs/**: Implementation planning, API specifications, and historical context
-- **Code Comments**: Use for implementation details and complex logic explanations
+- **Code Comments**: Use for implementation details and complex logic explanations

banks2ff/src/adapters/gocardless/client.rs

@@ -119,12 +119,11 @@ impl TransactionSource for GoCardlessAdapter {
                     }
 
                     // Optimization: Stop if we found all wanted accounts
-                    if let Some(_) = wanted_set {
+                    if wanted_set.is_some()
-                        if found_count >= target_count && target_count > 0 {
+                        && found_count >= target_count && target_count > 0 {
                             info!("Found all {} wanted accounts. Stopping search.", target_count);
                             return Ok(accounts);
                         }
-                    }
                 }
             }
         }

banks2ff/src/core/sync.rs

@@ -166,21 +166,19 @@ async fn process_single_account(
                  if dry_run {
                      info!("[DRY RUN] Would create transaction {}", tx.internal_id);
                      stats.created += 1;
-                 } else {
+                 } else if let Err(e) = destination.create_transaction(&dest_id, &tx).await {
-                     if let Err(e) = destination.create_transaction(&dest_id, &tx).await {
+                     // Firefly might still reject it as duplicate if hash matches, even if we didn't find it via heuristic
-                         // Firefly might still reject it as duplicate if hash matches, even if we didn't find it via heuristic
+                     // (unlikely if heuristic is good, but possible)
-                         // (unlikely if heuristic is good, but possible)
+                     let err_str = e.to_string();
-                         let err_str = e.to_string();
+                     if err_str.contains("422") || err_str.contains("Duplicate") {
-                         if err_str.contains("422") || err_str.contains("Duplicate") {
+                         warn!("Duplicate rejected by Firefly: {}", tx.internal_id);
-                             warn!("Duplicate rejected by Firefly: {}", tx.internal_id);
+                         stats.duplicates += 1;
-                             stats.duplicates += 1;
-                         } else {
-                             tracing::error!("Failed to create transaction: {}", e);
-                             stats.errors += 1;
-                         }
                      } else {
-                         stats.created += 1;
+                         tracing::error!("Failed to create transaction: {}", e);
+                         stats.errors += 1;
                      }
+                 } else {
+                     stats.created += 1;
                  }
              }
          }