Mimic git-quick-stats suggestReviewers()

* Adjust the previous function to mimic what suggestReviewers()
  does when you execute it in git-quick-stats. This matches the
  output of the bash version when run side-by-side. Also, renamed
  the function to look more similar to the original bash function.
  Updated test cases as appropriate

  Note that it does not handle the adjustable options such as
  $_since, $_until, etc yet.

* Added .toml file to prepare for dist building and pip distribution

Author: tomice

git_py_stats/interactive_mode.py

@@ -29,7 +29,7 @@ def handle_interactive_mode() -> None:
         '19': lambda: list_cmds.git_commits_per_hour(input("Enter author name: ")),
         '20': list_cmds.git_commits_per_timezone,
         '21': lambda: list_cmds.git_commits_per_timezone(input("Enter author name: ")),
-        '22': suggest_cmds.code_reviewers,
+        '22': suggest_cmds.suggest_reviewers,
     }
 
     while True:

git_py_stats/non_interactive_mode.py

@@ -34,7 +34,7 @@ def handle_non_interactive_mode(args) -> None:
         'commits_by_author_by_hour': lambda: list_cmds.git_commits_per_hour(args.commits_by_author_by_hour),
         'commits_by_timezone': list_cmds.git_commits_per_timezone,
         'commits_by_author_by_timezone': lambda: list_cmds.git_commits_per_timezone(args.commits_by_author_by_timezone),
-        'suggest_reviewers': suggest_cmds.code_reviewers,
+        'suggest_reviewers': suggest_cmds.suggest_reviewers,
     }
 
     # Call the appropriate function based on the command-line argument

git_py_stats/suggest_cmds.py

@@ -1,30 +1,73 @@
-"""
-Functions related to the 'Suggest' section.
-"""
-
-from collections import Counter
+import subprocess
+from collections import defaultdict
 
 from git_py_stats.git_operations import run_git_command
 
-
-def code_reviewers() -> None:
+def suggest_reviewers() -> None:
     """
     Suggests potential code reviewers based on commit history.
     """
-    cmd = ['git', 'log', '--format=%ae', '--no-merges']
-    output = run_git_command(cmd)
-    
-    if output:
-        emails = output.split('\n')
-        counter = Counter(emails)
-        reviewers = [email for email, count in counter.items()]
-        
-        if reviewers:
+
+    # Construct the git log command with all options. Original command is:
+    # git -c log.showSignature=false log --use-mailmap $_merges "$_since" "$_until" \
+    #     --pretty=%aN $_log_options $_pathspec | head -n 100 | sort | uniq -c \
+    #     | sort -nr
+    # Then some LC_ALL portion which is currently not important
+    # Then pipe it all into column -t -s
+    #
+    # For now, let's just hardcode some of this stuff
+    cmd = ['git', '-c', 'log.showSignature=false', 'log', '--use-mailmap', '--no-merges', '--pretty=%aN']
+
+    try:
+        # Execute the git command and get the output
+        output = run_git_command(cmd)
+
+        # Check if output is empty
+        if not output:
+            print('No data available.')
+            return
+
+        # Split the output into lines (each line is a commit author)
+        lines = output.splitlines()
+
+        # Mimic "head -n 100"
+        head_lines = lines[:100]
+
+        # Mimic "sort"
+        sorted_lines = sorted(head_lines)
+
+        # Mimic "uniq -c"
+        counted_authors = []
+        current_author = None
+        current_count = 0
+
+        # Iterate over sorted lines and count consecutive duplicates
+        for author in sorted_lines:
+            if author == current_author:
+                current_count += 1
+            else:
+                if current_author is not None:
+                    counted_authors.append((current_count, current_author))
+                current_author = author
+                current_count = 1
+
+        # Append the last counted author
+        if current_author is not None:
+            counted_authors.append((current_count, current_author))
+
+        # Sort by count descending, and by name ascending
+        sorted_authors = sorted(counted_authors, key=lambda x: (-x[0], x[1]))
+
+        # Print results similar to "column -t -s"
+        if sorted_authors:
             print("Suggested code reviewers based on git history:")
-            for reviewer in reviewers:
-                print(reviewer)
+            for count, author in sorted_authors:
+                # Pad output with 7 to match original code's behavior
+                # TODO: Bit of a magic number we can remove later
+                print(f"{count:7} {author}")
         else:
-            print('No potential reviewers found. Consider using a larger dataset.')
-    else:
-        print('No data available.')
+            print("No potential reviewers found.")
+
+    except subprocess.CalledProcessError as e:
+        print(f"Error executing git command: {e}")
 

git_py_stats/tests/test_suggest_cmds.py

@@ -11,19 +11,19 @@ class TestSuggestCmds(unittest.TestCase):
 
     # Don't print to stdout
     @patch('git_py_stats.suggest_cmds.print')
-    def test_code_reviewers(self, mock_print) -> None:
+    def test_suggest_reviewers(self, mock_print) -> None:
         """
-        Test case for code_reviewers in suggest_cmds
+        Test case for suggest_reviewers in suggest_cmds
 
-        Checks if `code_reviewers` executes without errors, and it uses
+        Checks if `suggest_reviewers` executes without errors, and it uses
         `unittest.mock.patch` to mock the print function to prevent actual
         output during testing.
 
         Verifies that the function returns `None` and calls the print function
         at least once, indicating that some output was generated.
         """
 
-        result = suggest_cmds.code_reviewers()
+        result = suggest_cmds.suggest_reviewers()
         self.assertIsNone(result)
         mock_print.assert_called()
 

pyproject.toml

@@ -0,0 +1,3 @@
+[build-system]
+requires = ["setuptools>=61.0"]
+build-backend = "setuptools.build_meta"