- cleanup log statements shown in the smartfix run

Author: dougj-contrast

src/git_handler.py

@@ -927,7 +927,7 @@ def get_issue_comments(issue_number: int, author: str = None) -> List[dict]:
         List[dict]: A list of comment data dictionaries or empty list if no comments or error
     """
     author_log = f"and author: {author}" if author  else ""
-    log(f"Getting comments for issue #{issue_number} {author_log}")
+    debug_log(f"Getting comments for issue #{issue_number} {author_log}")
     gh_env = get_gh_env()
     author_filter = f"| map(select(.author.login == \"{author}\")) " if author else ""
     jq_filter = f'.comments {author_filter}| sort_by(.createdAt) | reverse'
@@ -1095,7 +1095,7 @@ def get_claude_workflow_run_id() -> int:
     Returns:
         int: The workflow run ID if found, or None if no in-progress runs are found
     """
-    log("Getting in-progress Claude workflow run ID")
+    debug_log("Getting in-progress Claude workflow run ID")
 
     gh_env = get_gh_env()
     jq_filter = 'map(select(.event == "issues" or .event == "issue_comment") | select(.status == "in_progress") | select(.conclusion != "skipped")) | sort_by(.createdAt) | reverse | .[0]'
@@ -1131,7 +1131,7 @@ def get_claude_workflow_run_id() -> int:
 
         if workflow_run_id is not None:
             workflow_run_id = int(workflow_run_id)
-            log(f"Found in-progress Claude workflow run ID: {workflow_run_id}")
+            debug_log(f"Found in-progress Claude workflow run ID: {workflow_run_id}")
             return workflow_run_id
         else:
             debug_log("No databaseId found in workflow run data")
@@ -1197,7 +1197,7 @@ def create_claude_pr(title: str, body: str, base_branch: str, head_branch: str)
         # Run the command and capture the output (PR URL)
         pr_url = run_command(pr_command, env=gh_env, check=True)
         if pr_url:
-            log(f"Successfully created Claude PR: {pr_url}")
+            debug_log(f"Successfully created Claude PR: {pr_url}")
         return pr_url.strip() if pr_url else ""
 
     except Exception as e:

test/test_git_handler.py

@@ -1003,7 +1003,7 @@ def test_get_issue_comments_success(self, mock_log, mock_debug_log, mock_get_gh_
         self.assertIn('author.login == "claude"', command[jq_index])
         self.assertIn('sort_by(.createdAt) | reverse', command[jq_index])
 
-        mock_log.assert_any_call("Getting comments for issue #94 and author: claude")
+        mock_debug_log.assert_any_call("Getting comments for issue #94 and author: claude")
         mock_debug_log.assert_any_call("Found 1 comments on issue #94")
 
     @patch('src.git_handler.run_command')
@@ -1032,12 +1032,13 @@ def test_get_issue_comments_no_comments(self, mock_log, mock_debug_log, mock_get
                 # Assert
                 self.assertEqual(result, [])
                 mock_debug_log.assert_any_call(f"No comments found for issue #{issue_number}")
-                mock_log.assert_any_call(f"Getting comments for issue #{issue_number} and author: claude")
+                mock_debug_log.assert_any_call(f"Getting comments for issue #{issue_number} and author: claude")
 
     @patch('src.git_handler.run_command')
     @patch('src.git_handler.get_gh_env')
+    @patch('src.git_handler.debug_log')
     @patch('src.git_handler.log')
-    def test_get_issue_comments_json_error(self, mock_log, mock_get_gh_env, mock_run_command):
+    def test_get_issue_comments_json_error(self, mock_log, mock_debug_log, mock_get_gh_env, mock_run_command):
         """Test getting comments from an issue when JSON parsing error occurs"""
         # Setup
         issue_number = 42
@@ -1055,7 +1056,7 @@ def test_get_issue_comments_json_error(self, mock_log, mock_get_gh_env, mock_run
         # Assert
         self.assertEqual(result, [])
         mock_run_command.assert_called_once()
-        mock_log.assert_any_call(f"Getting comments for issue #{issue_number} and author: claude")
+        mock_debug_log.assert_any_call(f"Getting comments for issue #{issue_number} and author: claude")
         # Use assertIn rather than assert_any_call to check for partial match
         # since the actual error message includes JSON exception details
         log_calls = [call[0][0] for call in mock_log.call_args_list if call[1].get('is_error', False)]
@@ -1064,8 +1065,9 @@ def test_get_issue_comments_json_error(self, mock_log, mock_get_gh_env, mock_run
 
     @patch('src.git_handler.run_command')
     @patch('src.git_handler.get_gh_env')
+    @patch('src.git_handler.debug_log')
     @patch('src.git_handler.log')
-    def test_get_issue_comments_exception(self, mock_log, mock_get_gh_env, mock_run_command):
+    def test_get_issue_comments_exception(self, mock_log, mock_debug_log, mock_get_gh_env, mock_run_command):
         """Test getting comments from an issue when an exception occurs"""
         # Setup
         issue_number = 42
@@ -1083,7 +1085,7 @@ def test_get_issue_comments_exception(self, mock_log, mock_get_gh_env, mock_run_
         # Assert
         self.assertEqual(result, [])
         mock_run_command.assert_called_once()
-        mock_log.assert_any_call(f"Getting comments for issue #{issue_number} and author: claude")
+        mock_debug_log.assert_any_call(f"Getting comments for issue #{issue_number} and author: claude")
         mock_log.assert_any_call(f"Error getting comments for issue #{issue_number}: Command failed", is_error=True)
 
     @patch('src.git_handler.run_command')
@@ -1186,8 +1188,8 @@ def test_get_claude_workflow_run_id_success(self, mock_config, mock_debug_log, m
         self.assertEqual(command[command.index("--jq") + 1], 'map(select(.event == "issues" or .event == "issue_comment") | select(.status == "in_progress") | select(.conclusion != "skipped")) | sort_by(.createdAt) | reverse | .[0]')
 
 
-        mock_log.assert_any_call("Getting in-progress Claude workflow run ID")
-        mock_log.assert_any_call(f"Found in-progress Claude workflow run ID: 12345678")
+        mock_debug_log.assert_any_call("Getting in-progress Claude workflow run ID")
+        mock_debug_log.assert_any_call(f"Found in-progress Claude workflow run ID: 12345678")
 
     @patch('src.git_handler.run_command')
     @patch('src.git_handler.get_gh_env')
@@ -1326,7 +1328,7 @@ def success_run_command_side_effect(*args, **kwargs):
 
         # Verify appropriate logs were created
         mock_log.assert_any_call(f"Creating Claude PR with title: '{title}'")
-        mock_log.assert_any_call("Successfully created Claude PR: https://github.com/mock/repo/pull/123\n")
+        mock_debug_log.assert_any_call("Successfully created Claude PR: https://github.com/mock/repo/pull/123\n")
 
     @patch('src.git_handler.run_command')
     @patch('src.git_handler.get_gh_env')