Release v2.8.2: Add --since flag back to gx who (Go + Python)

Author: mubbie

VERSION

@@ -1 +1 @@
-2.8.1
+2.8.2

cmd/who.go

@@ -21,6 +21,7 @@ func init() {
 		RunE:  runWho,
 	}
 	cmd.Flags().IntP("number", "n", 5, "Number of contributors to show")
+	cmd.Flags().String("since", "", "Only consider contributions after this date (e.g., 6months, 2024-01-01)")
 	cmd.Flags().Bool("no-limit", false, "Remove the 200-file cap for directory analysis")
 	rootCmd.AddCommand(cmd)
 }
@@ -32,9 +33,10 @@ func runWho(cmd *cobra.Command, args []string) error {
 	}
 
 	n, _ := cmd.Flags().GetInt("number")
+	since, _ := cmd.Flags().GetString("since")
 
 	if len(args) == 0 {
-		return whoRepo(n)
+		return whoRepo(n, since)
 	}
 
 	path := args[0]
@@ -45,9 +47,9 @@ func runWho(cmd *cobra.Command, args []string) error {
 	}
 	if info.IsDir() {
 		noLimit, _ := cmd.Flags().GetBool("no-limit")
-		return whoDir(path, n, noLimit)
+		return whoDir(path, n, since, noLimit)
 	}
-	return whoFile(path, n)
+	return whoFile(path, n, since)
 }
 
 // --- File-level: git blame (fast, ~100ms per file) ---
@@ -59,10 +61,15 @@ type blameAuthor struct {
 	lastDate string
 }
 
-func whoFile(path string, n int) error {
+func whoFile(path string, n int, since string) error {
 	sp := ui.StartSpinner(fmt.Sprintf("Analyzing %s...", path))
 
-	out, err := git.Run("blame", "--line-porcelain", path)
+	blameArgs := []string{"blame", "--line-porcelain"}
+	if since != "" {
+		blameArgs = append(blameArgs, "--since", since)
+	}
+	blameArgs = append(blameArgs, path)
+	out, err := git.Run(blameArgs...)
 	if err != nil {
 		sp.Stop()
 		ui.PrintError(fmt.Sprintf("Failed to blame %s: %s", path, err))
@@ -145,7 +152,7 @@ func whoFile(path string, n int) error {
 
 // --- Directory-level: concurrent blame across files ---
 
-func whoDir(dir string, n int, noLimit bool) error {
+func whoDir(dir string, n int, since string, noLimit bool) error {
 	sp := ui.StartSpinner(fmt.Sprintf("Analyzing %s...", dir))
 
 	filesOut, err := git.Run("ls-files", dir)
@@ -192,7 +199,12 @@ func whoDir(dir string, n int, noLimit bool) error {
 			sem <- struct{}{}
 			defer func() { <-sem }()
 
-			out := git.RunUnchecked("blame", "--line-porcelain", f)
+			bArgs := []string{"blame", "--line-porcelain"}
+			if since != "" {
+				bArgs = append(bArgs, "--since", since)
+			}
+			bArgs = append(bArgs, f)
+			out := git.RunUnchecked(bArgs...)
 			counts := map[string]int{}
 			emails := map[string]string{}
 			var name, email string
@@ -284,10 +296,14 @@ func whoDir(dir string, n int, noLimit bool) error {
 
 // --- Repo-level: git shortlog (fast, commits only) ---
 
-func whoRepo(n int) error {
+func whoRepo(n int, since string) error {
 	sp := ui.StartSpinner("Analyzing contributors...")
 
-	out, err := git.Run("shortlog", "-sne", "HEAD")
+	args := []string{"shortlog", "-sne", "HEAD"}
+	if since != "" {
+		args = append(args, "--since="+since)
+	}
+	out, err := git.Run(args...)
 	sp.Stop()
 	if err != nil || out == "" {
 		ui.PrintInfo("No contributors found.")

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "gx-git"
-version = "2.8.1"
+version = "2.8.2"
 description = "Git Productivity Toolkit"
 readme = "README.md"
 license = {text = "MIT"}

src/gx/commands/who.py

@@ -14,12 +14,15 @@
 from gx.utils.git import GitError, ensure_git_repo, run_git, time_ago
 
 
-def _blame_file(filepath: str, cwd: str | None = None) -> dict[str, dict]:
+def _blame_file(filepath: str, cwd: str | None = None, since: str | None = None) -> dict[str, dict]:
     """Run git blame on a file and return per-author stats.
 
     Returns {author: {"lines": int, "email": str, "last_ts": str}}.
     """
-    args = ["blame", "--line-porcelain", filepath]
+    args = ["blame", "--line-porcelain"]
+    if since:
+        args.extend(["--since", since])
+    args.append(filepath)
     try:
         output = run_git(args, cwd=cwd)
     except GitError:
@@ -70,9 +73,11 @@ def _get_tracked_files(directory: str, cwd: str | None = None) -> list[str]:
     return output.splitlines()
 
 
-def _repo_level(n: int) -> None:
+def _repo_level(n: int, since: str | None = None) -> None:
     """Show top contributors to the entire repo (fast, shortlog only)."""
     args = ["shortlog", "-sne", "HEAD"]
+    if since:
+        args.extend(["--since", since])
     output = run_git(args)
     if not output:
         print_info("No contributors found.")
@@ -107,13 +112,13 @@ def _repo_level(n: int) -> None:
     )
 
 
-def _file_level(filepath: str, n: int) -> None:
+def _file_level(filepath: str, n: int, since: str | None = None) -> None:
     """Show who knows a specific file best (blame-based)."""
     if not os.path.exists(filepath):
         print_error(f"File not found: {filepath}")
         raise typer.Exit(1)
 
-    stats = _blame_file(filepath)
+    stats = _blame_file(filepath, since=since)
     if not stats:
         print_info(f"No blame data for {filepath}")
         return
@@ -135,7 +140,7 @@ def _file_level(filepath: str, n: int) -> None:
     )
 
 
-def _dir_level(directory: str, n: int, no_limit: bool) -> None:
+def _dir_level(directory: str, n: int, since: str | None = None, no_limit: bool = False) -> None:
     """Show who knows a directory best (concurrent blame)."""
     files = _get_tracked_files(directory)
     if not files:
@@ -162,7 +167,7 @@ def _dir_level(directory: str, n: int, no_limit: bool) -> None:
         task = progress.add_task(f"Analyzing {len(files)} files...", total=len(files))
 
         def blame_one(f: str) -> tuple[str, dict[str, dict]]:
-            return f, _blame_file(f)
+            return f, _blame_file(f, since=since)
 
         with ThreadPoolExecutor(max_workers=8) as pool:
             futures = {pool.submit(blame_one, f): f for f in files}
@@ -200,6 +205,7 @@ def blame_one(f: str) -> tuple[str, dict[str, dict]]:
 def who(
     path: str = typer.Argument(None, help="File or directory to analyze (omit for repo-level)."),
     n: int = typer.Option(DEFAULT_WHO_LIMIT, "-n", "--number", help="Number of contributors to show."),
+    since: str = typer.Option(None, "--since", help="Only consider contributions after this date."),
     no_limit: bool = typer.Option(False, "--no-limit", help="Remove file cap for directory analysis."),
 ) -> None:
     """Show who knows a file, directory, or repo best."""
@@ -210,8 +216,8 @@ def who(
         raise typer.Exit(1)
 
     if path is None:
-        _repo_level(n)
+        _repo_level(n, since)
     elif os.path.isdir(path):
-        _dir_level(path, n, no_limit)
+        _dir_level(path, n, since, no_limit)
     else:
-        _file_level(path, n)
+        _file_level(path, n, since)