remove prints

Author: acunniffe

src/commands/git_ai_handlers.rs

@@ -120,7 +120,9 @@ fn print_help() {
     eprintln!("  ci                 Continuous integration utilities");
     eprintln!("    github                 GitHub CI helpers");
     eprintln!("  squash-authorship  Generate authorship log for squashed commits");
-    eprintln!("    <base_branch> <new_sha> <old_sha>  Required: base branch, new commit SHA, old commit SHA");
+    eprintln!(
+        "    <base_branch> <new_sha> <old_sha>  Required: base branch, new commit SHA, old commit SHA"
+    );
     eprintln!("    --dry-run             Show what would be done without making changes");
     eprintln!("  git-path           Print the path to the underlying git executable");
     eprintln!("  version, -v, --version     Print the git-ai version");
@@ -326,14 +328,7 @@ fn handle_checkpoint(args: &[String]) {
         .map(|r| r.checkpoint_kind)
         .unwrap_or(CheckpointKind::Human);
 
-    println!("checkpoint_kind: {:?}", checkpoint_kind);
-    println!("files for mock_ai: {:?}", get_all_files_for_mock_ai(&final_working_dir));
-
     if CheckpointKind::Human == checkpoint_kind && agent_run_result.is_none() {
-        println!(
-            "get_all_files_for_mock_ai HUMAN Checkpoints: {:?}",
-            get_all_files_for_mock_ai(&final_working_dir)
-        );
         agent_run_result = Some(AgentRunResult {
             agent_id: AgentId {
                 tool: "mock_ai".to_string(),

src/git/repo_storage.rs

@@ -157,8 +157,6 @@ impl PersistedWorkingLog {
     }
 
     pub fn set_dirty_files(&mut self, dirty_files: Option<HashMap<String, String>>) {
-        println!("pre_transform dirty_files: {:?}", dirty_files);
-
         let normalized_dirty_files = dirty_files.map(|map| {
             map.into_iter()
                 .map(|(file_path, content)| {
@@ -170,8 +168,6 @@ impl PersistedWorkingLog {
         });
 
         self.dirty_files = normalized_dirty_files;
-
-        println!("setdirty_files: {:?}", self.dirty_files);
     }
 
     pub fn reset_working_log(&self) -> Result<(), GitAiError> {

tests/repos/test_file.rs

@@ -727,12 +727,10 @@ impl<'a> TestFile<'a> {
     fn write_and_checkpoint(&self, author_type: &AuthorType) {
         let contents = self.contents();
         fs::write(&self.file_path, contents).unwrap();
-        let _ = if author_type == &AuthorType::Ai {
-            let a = self.repo.git_ai(&["checkpoint", "mock_ai"]);
-            println!("checkpoint result: {:?}", a);
-            a
+        if author_type == &AuthorType::Ai {
+            self.repo.git_ai(&["checkpoint", "mock_ai"]).unwrap();
         } else {
-            self.repo.git_ai(&["checkpoint"])
+            self.repo.git_ai(&["checkpoint"]).unwrap();
         };
     }
 

tests/repos/test_repo.rs

@@ -40,7 +40,9 @@ impl TestRepo {
     }
 
     pub fn canonical_path(&self) -> PathBuf {
-        self.path.canonicalize().expect("failed to canonicalize test repo path")
+        self.path
+            .canonicalize()
+            .expect("failed to canonicalize test repo path")
     }
 
     pub fn stats(&self) -> Result<CommitStats, String> {
@@ -228,7 +230,7 @@ impl TestRepo {
 
 impl Drop for TestRepo {
     fn drop(&mut self) {
-        // fs::remove_dir_all(self.path.clone()).expect("failed to remove test repo");
+        fs::remove_dir_all(self.path.clone()).expect("failed to remove test repo");
     }
 }