git-explain 1.1.3__tar.gz → 2.1.4__tar.gz

{git_explain-1.1.3 → git_explain-2.1.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: git-explain
-Version: 1.1.3
+Version: 2.1.4
 Summary: CLI that suggests git add/commit from diffs using Gemini
 Author: git-explain contributors
 License-Expression: MIT
@@ -26,6 +26,7 @@ Requires-Dist: python-dotenv>=1.0.0
 Requires-Dist: prompt_toolkit>=3.0.0
 Provides-Extra: dev
 Requires-Dist: pytest>=8.0.0; extra == "dev"
+Requires-Dist: ruff>=0.8.0; extra == "dev"
 Dynamic: license-file
 
 # git-explain
@@ -125,7 +126,7 @@ You’ll see a list of changed files, choose which to include, then get suggeste
 | `--ai` | Use Gemini for commit type/message (file paths only). |
 | `--with-diff` | With `--ai`: send full diff to the model for detailed messages. |
 | `--model NAME` | Override Gemini model (e.g. `--model gemini-2.0-flash`). |
-| `--staged-only` | Commit only what’s already staged (no `git add`). |
+| `--staged-only` | Commit only what’s already staged (no `git add`). Always one commit for the whole index—split-by-group mode is disabled, because Git would commit the entire index on the first step and later steps would have nothing left staged. |
 | `--cwd PATH` | Run as if current directory is `PATH`. |
 | `--install-completion [SHELL]` | Install shell completion (`bash`, `zsh`). |
 | `--show-completion [SHELL]` | Print completion script for `SHELL`. |
@@ -134,7 +135,7 @@ You’ll see a list of changed files, choose which to include, then get suggeste
 
 ## Workflow
 
-1. **Changed files** — Shows staged, unstaged, and untracked files. Untracked folders are grouped (e.g. `foo/ (untracked folder; 5 files)`).
+1. **Changed files** — Shows staged, unstaged, and untracked files. Untracked directories are expanded so you still see per-file paths.
 2. **Select files** — Enter numbers (e.g. `1,2,5-7`), `all`, or a path (e.g. `main.py`, `src/utils/`).
 3. **Commit mode** — If you selected 2+ files: choose `one` (single commit) or `split` (separate commits by docs/tests/config/code).
 4. **Suggested commands** — Panel with `git add` and `git commit` lines.

{git_explain-1.1.3 → git_explain-2.1.4}/README.md

@@ -95,7 +95,7 @@ You’ll see a list of changed files, choose which to include, then get suggeste
 | `--ai` | Use Gemini for commit type/message (file paths only). |
 | `--with-diff` | With `--ai`: send full diff to the model for detailed messages. |
 | `--model NAME` | Override Gemini model (e.g. `--model gemini-2.0-flash`). |
-| `--staged-only` | Commit only what’s already staged (no `git add`). |
+| `--staged-only` | Commit only what’s already staged (no `git add`). Always one commit for the whole index—split-by-group mode is disabled, because Git would commit the entire index on the first step and later steps would have nothing left staged. |
 | `--cwd PATH` | Run as if current directory is `PATH`. |
 | `--install-completion [SHELL]` | Install shell completion (`bash`, `zsh`). |
 | `--show-completion [SHELL]` | Print completion script for `SHELL`. |
@@ -104,7 +104,7 @@ You’ll see a list of changed files, choose which to include, then get suggeste
 
 ## Workflow
 
-1. **Changed files** — Shows staged, unstaged, and untracked files. Untracked folders are grouped (e.g. `foo/ (untracked folder; 5 files)`).
+1. **Changed files** — Shows staged, unstaged, and untracked files. Untracked directories are expanded so you still see per-file paths.
 2. **Select files** — Enter numbers (e.g. `1,2,5-7`), `all`, or a path (e.g. `main.py`, `src/utils/`).
 3. **Commit mode** — If you selected 2+ files: choose `one` (single commit) or `split` (separate commits by docs/tests/config/code).
 4. **Suggested commands** — Panel with `git add` and `git commit` lines.

{git_explain-1.1.3 → git_explain-2.1.4}/git_explain/cli.py

@@ -20,6 +20,8 @@ load_dotenv()
 app = typer.Typer()
 console = Console()
 
+_DIFF_INFER_MAX_CHARS = 50_000
+
 
 @dataclass(frozen=True)
 class Change:
@@ -264,36 +266,11 @@ def run(
         return
 
     norm_paths = [c.path.replace("\\", "/") for c in changes]
-    untracked_indices_by_root: dict[str, list[int]] = {}
-    for idx, (ch, np) in enumerate(zip(changes, norm_paths)):
-        if (
-            "Untracked" in ch.sections
-            and "Staged" not in ch.sections
-            and "Unstaged" not in ch.sections
-            and "/" in np
-        ):
-            root = np.split("/", 1)[0]
-            untracked_indices_by_root.setdefault(root, []).append(idx)
-    folder_groups = {
-        root: idxs for root, idxs in untracked_indices_by_root.items() if len(idxs) > 1
-    }
-
     display_items: list[tuple[str, list[int]]] = []
-    seen_untracked_roots: set[str] = set()
     for idx, ch in enumerate(changes):
-        np = norm_paths[idx]
-        root = np.split("/", 1)[0] if "/" in np else None
-        if root and root in folder_groups:
-            if root in seen_untracked_roots:
-                continue
-            seen_untracked_roots.add(root)
-            count = len(folder_groups[root])
-            label = f"{root}/ (untracked folder; {count} files)"
-            display_items.append((label, folder_groups[root]))
-        else:
-            sec = ",".join([s.lower() for s in ch.sections if s and s != "Meta"])
-            label = f"[{ch.status}] ({sec}) {ch.path}"
-            display_items.append((label, [idx]))
+        sec = ",".join([s.lower() for s in ch.sections if s and s != "Meta"])
+        label = f"[{ch.status}] ({sec}) {ch.path}"
+        display_items.append((label, [idx]))
 
     lines = []
     for idx, (label, _idxs) in enumerate(display_items, start=1):
@@ -347,8 +324,20 @@ def run(
 
     def suggest_for(
         change_items: list[tuple[str, str]], title: str
-    ) -> tuple[list[str], str, str, str]:
-        # Returns (paths, type, message, raw_text)
+    ) -> tuple[list[str], str, str, str, str | None]:
+        # Returns (paths, type, message, raw_text, ai_fallback_reason).
+        # ai_fallback_reason is set when --ai was used but heuristics were used instead.
+        paths_for_infer = [p for _, p in change_items]
+        infer_diff: str | None = None
+        if paths_for_infer:
+            raw_d = get_diff_for_paths(paths_for_infer, cwd=repo_root)
+            if raw_d.strip():
+                infer_diff = (
+                    raw_d[:_DIFF_INFER_MAX_CHARS]
+                    if len(raw_d) > _DIFF_INFER_MAX_CHARS
+                    else raw_d
+                )
+
         if ai:
             payload = _render_combined(has_commits, change_items, title=title)
             if with_diff:
@@ -357,42 +346,91 @@ def run(
                 if diff_text:
                     payload = payload + "\n\n## Diff\n" + diff_text
             try:
-                sug, raw = suggest_commands(payload, model=model, with_diff=with_diff)
+                sug, raw = suggest_commands(
+                    payload,
+                    model=model,
+                    with_diff=with_diff,
+                    unified_diff_for_infer=infer_diff,
+                )
                 if sug is None:
                     raise RuntimeError("Could not parse AI suggestion.")
-                return sug.add_args, sug.commit_type, sug.commit_message, raw
+                return sug.add_args, sug.commit_type, sug.commit_message, raw, None
             except Exception as e:
                 # Fall back to heuristics on quota / API errors
-                h = suggest_from_changes(changes=change_items, has_commits=has_commits)
+                h = suggest_from_changes(
+                    changes=change_items,
+                    has_commits=has_commits,
+                    diff_text=infer_diff,
+                )
                 return (
                     h.add_args,
                     h.commit_type,
                     h.commit_message,
-                    f"AI unavailable: {e}",
+                    "",
+                    str(e),
                 )
-        h = suggest_from_changes(changes=change_items, has_commits=has_commits)
-        return h.add_args, h.commit_type, h.commit_message, ""
+        h = suggest_from_changes(
+            changes=change_items,
+            has_commits=has_commits,
+            diff_text=infer_diff,
+        )
+        return h.add_args, h.commit_type, h.commit_message, "", None
 
     selected_pairs = [(ch.status, ch.path) for ch in selected]
     unique_paths = {p for _, p in selected_pairs}
 
     mode = "one"
     if len(unique_paths) > 1:
-        mode_input = (
-            typer.prompt("Commit mode: one or split", default="one").strip().lower()
-        )
-        if mode_input in ("one", "split"):
-            mode = mode_input
+        if staged_only:
+            console.print(
+                "[dim]Note:[/dim] split commits are not available with --staged-only: "
+                "each commit would need its own staging, but this mode skips git add. "
+                "Using a single commit for everything currently staged."
+            )
+        else:
+            mode_input = (
+                typer.prompt("Commit mode: one or split", default="one").strip().lower()
+            )
+            if mode_input in ("one", "split"):
+                mode = mode_input
 
     plan: list[tuple[str, list[str], str, str]] = []
+    ai_fallback_notes: list[tuple[str, str]] = []
     if mode == "split":
         groups = _group_changes(selected_pairs)
         for gname, items in groups.items():
-            paths, ctype, cmsg, _raw = suggest_for(items, title=gname.capitalize())
+            paths, ctype, cmsg, _raw, fb = suggest_for(items, title=gname.capitalize())
             plan.append((gname, paths, ctype, cmsg))
+            if fb:
+                ai_fallback_notes.append((gname, fb))
     else:
-        paths, ctype, cmsg, _raw = suggest_for(selected_pairs, title="Selected")
+        paths, ctype, cmsg, _raw, fb = suggest_for(selected_pairs, title="Selected")
         plan.append(("one", paths, ctype, cmsg))
+        if fb:
+            ai_fallback_notes.append(("", fb))
+
+    if ai and ai_fallback_notes:
+        lines = [
+            "[bold]You used --ai, but Gemini was not used for the suggestion below.[/bold]",
+            "Commit message(s) come from [bold]local heuristics[/bold] instead.",
+            "",
+        ]
+        if mode == "split":
+            for gname, reason in ai_fallback_notes:
+                lines.append(f"[dim]{gname}:[/dim] {reason}")
+        else:
+            lines.append(ai_fallback_notes[0][1])
+        lines.append("")
+        lines.append(
+            "[dim]Check API key (GEMINI_API_KEY / GOOGLE_API_KEY), quota, model name, and network.[/dim]"
+        )
+        console.print(
+            Panel(
+                "\n".join(lines),
+                title="[yellow]Warning: AI unavailable[/yellow]",
+                border_style="yellow",
+            )
+        )
 
     def _render_plan(pl: list[tuple[str, list[str], str, str]]) -> str:
         rendered: list[str] = []
@@ -466,7 +504,13 @@ def run(
     if do_apply:
         for name, paths, ctype, cmsg in plan:
             try:
-                apply_commands(repo_root, [] if staged_only else paths, ctype, cmsg)
+                apply_commands(
+                    repo_root,
+                    [] if staged_only else paths,
+                    ctype,
+                    cmsg,
+                    staged_only=staged_only,
+                )
                 console.print(f"[green]Commit created ({name}).[/green]")
             except subprocess.CalledProcessError as e:
                 console.print("[red]git command failed.[/red]")

git_explain-2.1.4/git_explain/commit_infer.py

@@ -0,0 +1,66 @@
+"""Infer FIX-style commits from unified diff text (behavior fixes vs refactors)."""
+
+from __future__ import annotations
+
+
+def infer_fix_subject_from_diff(diff_text: str | None) -> str | None:
+    """Return a short subject fragment after 'Fix …', or None if no strong signal.
+
+    Uses high-precision phrases so we do not flip real refactors to FIX.
+    """
+    if not diff_text or len(diff_text.strip()) < 12:
+        return None
+    low = diff_text.lower()
+
+    if "split commits are not available" in low and (
+        "staged-only" in low or "staged_only" in low
+    ):
+        return "staged-only mode with multi-file split commits"
+
+    if (
+        "nothing is currently staged" in low
+        and "--staged-only" in low
+        and "git add" in low
+    ):
+        return "clearer error when index is empty under --staged-only"
+
+    infer_signals = (
+        "refine_type_and_message_from_diff",
+        "infer_fix_subject_from_diff",
+        "unified_diff_for_infer",
+        "commit_infer.py",
+    )
+    if sum(1 for s in infer_signals if s in low) >= 2:
+        return "commit message classification using unified diffs"
+
+    return None
+
+
+def refine_type_and_message_from_diff(
+    commit_type: str,
+    commit_message: str,
+    diff_text: str | None,
+) -> tuple[str, str]:
+    """When diff shows a behavior fix, prefer FIX and a concrete subject.
+
+    Does not override DOCS, TEST(S), or CHORE. May override REFACTOR or FEAT
+    when the diff matches known bugfix patterns.
+    """
+    ct = (commit_type or "").upper()
+    if ct in ("DOCS", "TEST", "TESTS", "CHORE"):
+        return commit_type, commit_message
+
+    subject = infer_fix_subject_from_diff(diff_text)
+    if not subject:
+        return commit_type, commit_message
+
+    if ct == "FIX":
+        msg = (commit_message or "").strip()
+        if len(msg) < 8 or msg.lower() in {"fix", "fixes", "bugfix", "bug fix"}:
+            return "FIX", f"Fix {subject}"
+        return commit_type, commit_message
+
+    if ct in ("REFACTOR", "FEAT"):
+        return "FIX", f"Fix {subject}"
+
+    return commit_type, commit_message

{git_explain-1.1.3 → git_explain-2.1.4}/git_explain/gemini.py

@@ -8,6 +8,15 @@ from dataclasses import dataclass
 from google import genai
 from google.genai import types
 
+from git_explain.commit_infer import refine_type_and_message_from_diff
+from git_explain.path_topics import (
+    area_scope_suffix,
+    basename_fallback_topic,
+    infra_deploy_topics,
+    is_test_path,
+    test_subject_hints,
+)
+
 SYSTEM_PROMPT = """You are given a list of changed/added files under ## Staged, ## Unstaged, ## Untracked.
 Each file line is: <STATUS> <PATH> where STATUS is one of:
 - A = added/new file
@@ -20,13 +29,19 @@ Suggest one commit that includes ALL of these files.
 
 Rules:
 1. Line 1 must be: git add <path1> <path2> ... with EVERY PATH from the list (all sections). Do not omit any file. Do not truncate. Do not include status letters.
-2. Line 2 must be: git commit -m "[TYPE] Message" with TYPE one of: FEAT, FIX, DOCS, REFACTOR, TEST.
+2. Line 2 must be: git commit -m "[TYPE] Message" with TYPE one of: FEAT, FIX, DOCS, REFACTOR, TEST, CHORE.
 3. The message must be a short, specific summary of what the change does based on the file names (e.g. "Add README and feature status doc", "Fix Gemini model and add file-list mode"). Never use only generic words like "update", "changes", or "refactor" by themselves—always add what was updated (e.g. "Update docs and CLI prompt").
-4. Use imperative, no period at end. Maximum one short line.
+4. Infer concrete artifacts from paths when obvious: Dockerfiles, Docker Compose files, nginx configs, .env/.env.example templates, CI workflows—not vague summaries like "add changes" or "add files" with no subject. For test paths (e.g. tests/test_foo.py), name the area under test (e.g. "Expand tests for foo and bar")—not "update project files".
+5. Use [FIX] (or "fix:" with --with-diff) when the change corrects broken behavior, wrong CLI flow, or misleading errors—not [REFACTOR] for those cases.
+6. Use imperative, no period at end. Maximum one short line.
 
 Example for files README.md, FEATURES.md, git_explain/gemini.py:
 git add README.md FEATURES.md git_explain/gemini.py
 git commit -m "[DOCS] Add README and FEATURES doc, tune Gemini prompt"
+
+Example for Docker + nginx + env templates under api/ and apps/frontend/:
+git add api/app/Dockerfile apps/frontend/nginx.conf
+git commit -m "[CHORE] Add Docker and nginx config with env examples for api and frontend"
 """
 
 SYSTEM_PROMPT_WITH_DIFF = """You are given:
@@ -34,10 +49,12 @@ SYSTEM_PROMPT_WITH_DIFF = """You are given:
 2. The full diff (## Staged diff, ## Unstaged diff, ## Untracked) showing exact code changes.
 
 Use the diff to write a specific, detailed commit message. Do not use generic words like "update" or "changes"—describe what actually changed (e.g. "add opt-in --with-diff to send full diff to LLM for detailed messages", "tweak commit message edit flow to show suggestion before prompting to edit").
+Name concrete pieces from paths when helpful (Docker, nginx, env templates, workflows)—avoid empty phrases like "add changes" that do not say what was added.
+Prefer **fix:** when the diff corrects incorrect behavior or user-visible bugs; use **refactor:** only for internal restructuring without behavior change.
 
 Output format (conventional commits style):
 - Line 1: git add <path1> <path2> ... with EVERY path from the file list. Do not omit any.
-- Line 2: git commit -m "type: subject" where type is exactly one of: feat, fix, docs, refactor, test.
+- Line 2: git commit -m "type: subject" where type is exactly one of: feat, fix, docs, refactor, test, chore.
   The subject must be a short, specific summary in imperative mood, no period at end (e.g. "feat: allow editing commit message before apply", "fix: parse conventional commit line from AI").
 
 Example:
@@ -47,16 +64,39 @@ git commit -m "feat: add opt-in --with-diff for detailed AI commit messages"
 
 ADD_LINE_RE = re.compile(r"git\s+add\s+(.+)", re.IGNORECASE)
 COMMIT_LINE_RE = re.compile(
-    r'git\s+commit\s+-m\s+["\']\[(FEAT|FIX|DOCS|REFACTOR|TESTS)\]\s*(.+?)["\']',
+    r'git\s+commit\s+-m\s+["\']\[(FEAT|FIX|DOCS|REFACTOR|TESTS|CHORE)\]\s*(.+?)["\']',
     re.IGNORECASE,
 )
 # Conventional: "feat: subject" or "fix: subject" (use "tests" not "test")
 COMMIT_LINE_CONVENTIONAL_RE = re.compile(
-    r'git\s+commit\s+-m\s+["\'](feat|fix|docs|refactor|tests)\s*:\s*(.+?)["\']',
+    r'git\s+commit\s+-m\s+["\'](feat|fix|docs|refactor|tests|chore)\s*:\s*(.+?)["\']',
     re.IGNORECASE,
 )
 DEFAULT_MODEL = "gemini-2.5-flash"
 
+_VAGUE_VERB_NOUN = re.compile(
+    r"^(add|update|modify|make)\s+(changes?|updates?|stuff|things)\s*$",
+    re.IGNORECASE,
+)
+
+# After add/update/modify/make, these tails are too vague to keep as the final message.
+_VAGUE_TAIL_AFTER_VERB = frozenset(
+    {
+        "project files",
+        "the project",
+        "the codebase",
+        "codebase",
+        "code",
+        "files",
+        "file",
+        "some files",
+        "various files",
+        "multiple files",
+        "dependencies",
+        "deps",
+    }
+)
+
 _GENERIC_MESSAGES = {
     "update",
     "updates",
@@ -80,6 +120,30 @@ def _is_generic_message(message: str) -> bool:
         return True
     if msg in _GENERIC_MESSAGES:
         return True
+    if _VAGUE_VERB_NOUN.match(msg):
+        return True
+    parts = msg.split()
+    if (
+        len(parts) == 2
+        and parts[0]
+        in (
+            "add",
+            "update",
+            "modify",
+            "make",
+        )
+        and parts[1] in ("changes", "change", "updates", "update", "files", "file")
+    ):
+        return True
+    if len(parts) >= 2 and parts[0] in (
+        "add",
+        "update",
+        "modify",
+        "make",
+    ):
+        tail = " ".join(parts[1:]).strip()
+        if tail in _VAGUE_TAIL_AFTER_VERB:
+            return True
     # "update X" is okay, but bare "update" or "update stuff" isn't
     if re.fullmatch(
         r"(update|updates|change|changes|refactor|refactoring|misc)(\s+.+)?", msg
@@ -127,8 +191,12 @@ def _fallback_type_and_message_with_context(
 
     verb = "Add" if (added_any or has_commits is False) else "Update"
 
+    all_test_paths = bool(files) and all(is_test_path(f) for f in files)
+
     if docs_only:
         commit_type = "DOCS"
+    elif all_test_paths:
+        commit_type = "TEST"
     elif verb == "Add":
         commit_type = "FEAT"
     else:
@@ -139,6 +207,17 @@ def _fallback_type_and_message_with_context(
         topics.append("README")
     if any(f.endswith("features.md") for f in lower):
         topics.append("FEATURES doc")
+    topics.extend(infra_deploy_topics(files))
+    test_files = [f for f in files if is_test_path(f)]
+    if test_files:
+        all_tests_only = len(test_files) == len(files)
+        hints = test_subject_hints(files)
+        if all_tests_only and hints:
+            head = " and ".join(hints[:3])
+            tail = f" (+{len(hints) - 3} more)" if len(hints) > 3 else ""
+            topics.append(f"tests for {head}{tail}")
+        else:
+            topics.append("tests")
     if touches_docs and not docs_only:
         topics.append("docs")
     if any(f.startswith("git_explain/") for f in lower) or any(
@@ -154,13 +233,14 @@ def _fallback_type_and_message_with_context(
     if touches_packaging:
         topics.append("packaging config")
 
-    if not topics:
-        topics = ["project files"]
-
     # Dedupe while keeping order
     seen: set[str] = set()
     topics = [t for t in topics if not (t in seen or seen.add(t))]
 
+    if not topics:
+        fb = basename_fallback_topic(files)
+        topics = [fb] if fb else ["project files"]
+
     if len(topics) == 1:
         msg = f"{verb} {topics[0]}"
     elif len(topics) == 2:
@@ -168,6 +248,8 @@ def _fallback_type_and_message_with_context(
     else:
         msg = f"{verb} {topics[0]}, {topics[1]}, and {topics[2]}"
 
+    msg += area_scope_suffix(files)
+
     if verb == "Add" and (has_commits is False):
         # Make initial commits a little clearer but still "Add …"
         msg = msg.replace("Add ", "Add initial ", 1) if msg.startswith("Add ") else msg
@@ -221,9 +303,18 @@ def _get_client() -> genai.Client:
 
 
 def suggest_commands(
-    diff: str, model: str | None = None, with_diff: bool = False
+    diff: str,
+    model: str | None = None,
+    with_diff: bool = False,
+    *,
+    unified_diff_for_infer: str | None = None,
 ) -> tuple[Suggestion | None, str]:
-    """Call Gemini with the file list (and optionally full diff); return (suggestion, raw_response). suggestion is None if unparseable."""
+    """Call Gemini with the file list (and optionally full diff); return (suggestion, raw_response). suggestion is None if unparseable.
+
+    ``unified_diff_for_infer`` optional text (staged+unstaged unified diff) used to
+    refine REFACTOR/FEAT into FIX when the diff matches behavior-fix patterns
+    (e.g. ``--staged-only``), including when ``with_diff`` is False.
+    """
     if not diff or not diff.strip():
         return None, ""
     model = model or os.environ.get("GEMINI_MODEL") or DEFAULT_MODEL
@@ -319,6 +410,14 @@ def suggest_commands(
         commit_type, commit_message = _fallback_type_and_message_with_context(
             files=add_args, added_any=added_any, has_commits=has_commits
         )
+
+    infer_body = unified_diff_for_infer
+    if not (infer_body and infer_body.strip()) and with_diff and "\n## Diff" in diff:
+        infer_body = diff.split("\n## Diff", 1)[1]
+    commit_type, commit_message = refine_type_and_message_from_diff(
+        commit_type, commit_message, infer_body
+    )
+
     return Suggestion(
         add_args=add_args, commit_type=commit_type, commit_message=commit_message
     ), raw

{git_explain-1.1.3 → git_explain-2.1.4}/git_explain/heuristics.py

@@ -4,11 +4,19 @@ from __future__ import annotations
 
 import os
 
+from git_explain.commit_infer import refine_type_and_message_from_diff
 from git_explain.gemini import Suggestion
+from git_explain.path_topics import (
+    area_scope_suffix,
+    basename_fallback_topic,
+    infra_deploy_topics,
+    is_infra_deploy_path,
+    is_test_path,
+    test_subject_hints,
+)
 
 
 DOC_EXTS = {".md", ".rst", ".txt"}
-TEST_HINTS = ("test", "tests", "pytest", "unittest")
 CONFIG_FILES = {
     "pyproject.toml",
     "requirements.txt",
@@ -32,38 +40,30 @@ def _is_doc(path: str) -> bool:
     }
 
 
-def _is_test(path: str) -> bool:
+def _is_plain_config(path: str) -> bool:
     p = path.lower()
     base = os.path.basename(p)
-    if p.startswith("tests/") or "/tests/" in p:
-        return True
-    if (
-        base.startswith("test_")
-        or base.endswith("_test.py")
-        or base.endswith(".spec.ts")
-        or base.endswith(".spec.tsx")
-    ):
-        return True
-    return any(h in p for h in TEST_HINTS)
+    return base in CONFIG_FILES or os.path.splitext(p)[1] in CONFIG_EXTS
 
 
 def _is_config(path: str) -> bool:
-    p = path.lower()
-    base = os.path.basename(p)
-    return base in CONFIG_FILES or os.path.splitext(p)[1] in CONFIG_EXTS
+    """Packaging/config files plus Docker, Compose, nginx, env templates."""
+    return _is_plain_config(path) or is_infra_deploy_path(path)
 
 
 def suggest_from_changes(
     *,
     changes: list[tuple[str, str]],
     has_commits: bool | None,
+    diff_text: str | None = None,
 ) -> Suggestion:
     """Create a Suggestion from [(status, path)] without calling AI."""
     paths = [p for _, p in changes]
     added_any = any(s.upper() == "A" for s, _ in changes) or has_commits is False
+    modified_any = any(s.upper() == "M" for s, _ in changes)
 
     docs = [p for p in paths if _is_doc(p)]
-    tests = [p for p in paths if _is_test(p)]
+    tests = [p for p in paths if is_test_path(p)]
     configs = [p for p in paths if _is_config(p)]
     has_tests = bool(tests)
     has_configs = bool(configs)
@@ -75,7 +75,12 @@ def suggest_from_changes(
         tc = len([p for p in non_docs if p in tests or p in configs])
         mostly_tests_or_config = tc / max(1, len(non_docs)) >= 0.6
 
-    verb = "Add" if added_any else "Update"
+    if has_commits is False:
+        verb = "Add"
+    elif added_any and not modified_any:
+        verb = "Add"
+    else:
+        verb = "Update"
 
     if docs_only:
         commit_type = "DOCS"
@@ -96,20 +101,44 @@ def suggest_from_changes(
         topics.append("README")
     if any(os.path.basename(p).lower() == "features.md" for p in paths):
         topics.append("FEATURES doc")
+    topics.extend(infra_deploy_topics(paths))
     if tests:
-        topics.append("tests")
-    if configs:
+        all_tests_only = bool(paths) and len(tests) == len(paths)
+        hints = test_subject_hints(paths)
+        if all_tests_only and hints:
+            head = " and ".join(hints[:3])
+            tail = f" (+{len(hints) - 3} more)" if len(hints) > 3 else ""
+            topics.append(f"tests for {head}{tail}")
+        else:
+            topics.append("tests")
+    if any(_is_plain_config(p) for p in paths):
         topics.append("config")
-    if any("git_explain/" in p.replace("\\", "/").lower() for p in paths):
-        topics.append("git-explain CLI")
-
-    if not topics:
-        topics = ["changes"]
+    ge_paths = [p for p in paths if "git_explain/" in p.replace("\\", "/").lower()]
+    if ge_paths:
+        if len(ge_paths) <= 2:
+            topics.append("git-explain CLI")
+        else:
+            stems: list[str] = []
+            seen_stem: set[str] = set()
+            for p in sorted(ge_paths, key=lambda x: x.lower()):
+                stem, _ext = os.path.splitext(os.path.basename(p))
+                key = stem.lower()
+                if key not in seen_stem:
+                    seen_stem.add(key)
+                    stems.append(stem.replace("_", " "))
+            label = ", ".join(stems[:4])
+            if len(stems) > 4:
+                label += f" (+{len(stems) - 4} more)"
+            topics.append(label)
 
     # Dedupe while preserving order
     seen: set[str] = set()
     topics = [t for t in topics if not (t in seen or seen.add(t))]
 
+    if not topics:
+        fb = basename_fallback_topic(paths)
+        topics = [fb] if fb else ["project files"]
+
     if len(topics) == 1:
         message = f"{verb} {topics[0]}"
     elif len(topics) == 2:
@@ -117,7 +146,16 @@ def suggest_from_changes(
     else:
         message = f"{verb} {topics[0]}, {topics[1]}, and {topics[2]}"
 
+    message += area_scope_suffix(paths)
+
     if added_any and has_commits is False and message.startswith("Add "):
         message = message.replace("Add ", "Add initial ", 1)
 
+    if len(message) > 72:
+        message = message[:72].rstrip()
+
+    commit_type, message = refine_type_and_message_from_diff(
+        commit_type, message, diff_text
+    )
+
     return Suggestion(add_args=paths, commit_type=commit_type, commit_message=message)

git_explain-2.1.4/git_explain/path_topics.py

@@ -0,0 +1,178 @@
+"""Derive concrete commit-message topics from paths (Docker, nginx, env templates, areas)."""
+
+from __future__ import annotations
+
+import os
+
+
+def _norm(p: str) -> str:
+    return p.replace("\\", "/").strip()
+
+
+_TEST_HINTS = ("pytest", "unittest", "tests/", "/tests/")
+
+
+def is_test_path(path: str) -> bool:
+    """True if path looks like a test file (mirrors heuristics rules; paths normalized)."""
+    p = _norm(path).lower()
+    base = os.path.basename(p)
+    if p.startswith("tests/") or "/tests/" in p:
+        return True
+    if (
+        base.startswith("test_")
+        or base.endswith("_test.py")
+        or base.endswith(".spec.ts")
+        or base.endswith(".spec.tsx")
+    ):
+        return True
+    return any(h in p for h in _TEST_HINTS)
+
+
+def test_subject_hints(paths: list[str]) -> list[str]:
+    """Short labels from test filenames, e.g. test_gemini.py -> gemini (deduped, stable order)."""
+    hints: list[str] = []
+    seen: set[str] = set()
+    for raw in paths:
+        if not is_test_path(raw):
+            continue
+        base = os.path.basename(_norm(raw))
+        stem, _ext = os.path.splitext(base)
+        s = stem
+        low = s.lower()
+        if low.startswith("test_"):
+            core = s[5:]
+        elif low.endswith("_test"):
+            core = s[: -len("_test")]
+        else:
+            core = s
+        core = core.strip().replace("_", " ").strip()
+        if not core:
+            continue
+        key = core.lower()
+        if key not in seen:
+            seen.add(key)
+            hints.append(core)
+    return hints
+
+
+def is_infra_deploy_path(path: str) -> bool:
+    """True if path looks like Docker, Compose, nginx, or env-template deploy config."""
+    p = _norm(path).lower()
+    base = os.path.basename(p)
+    if base == "dockerfile" or base.endswith(".dockerfile"):
+        return True
+    if base == ".dockerignore":
+        return True
+    if base.startswith("docker-compose") and base.endswith((".yml", ".yaml")):
+        return True
+    if base in ("compose.yaml", "compose.yml"):
+        return True
+    if "nginx" in base and base.endswith(".conf"):
+        return True
+    if base.endswith((".env.example", ".env.sample")):
+        return True
+    if base in ("compose.env.example", ".env.example"):
+        return True
+    return False
+
+
+def infra_deploy_topics(paths: list[str]) -> list[str]:
+    """Ordered, deduplicated topic phrases (no leading verb)."""
+    has_dockerfile = False
+    has_dockerignore = False
+    has_compose = False
+    has_nginx = False
+    has_env_example = False
+
+    for raw in paths:
+        p = _norm(raw).lower()
+        base = os.path.basename(p)
+        if base == "dockerfile" or base.endswith(".dockerfile"):
+            has_dockerfile = True
+        if base == ".dockerignore":
+            has_dockerignore = True
+        if base.startswith("docker-compose") and base.endswith((".yml", ".yaml")):
+            has_compose = True
+        if base in ("compose.yaml", "compose.yml"):
+            has_compose = True
+        if "nginx" in base and base.endswith(".conf"):
+            has_nginx = True
+        if base.endswith((".env.example", ".env.sample")):
+            has_env_example = True
+        if base in ("compose.env.example", ".env.example"):
+            has_env_example = True
+
+    topics: list[str] = []
+    seen: set[str] = set()
+
+    def add(s: str) -> None:
+        if s not in seen:
+            seen.add(s)
+            topics.append(s)
+
+    if has_dockerfile or has_dockerignore:
+        add("Docker")
+    if has_compose:
+        add("Docker Compose")
+    if has_nginx:
+        add("nginx")
+    if has_env_example:
+        add("env examples")
+
+    return topics
+
+
+def area_scope_suffix(paths: list[str]) -> str:
+    """Return ' for api and frontend' style suffix, or ''."""
+    labels: list[str] = []
+    seen: set[str] = set()
+
+    for raw in paths:
+        p = _norm(raw)
+        if not p or p == ".":
+            continue
+        parts = [x for x in p.split("/") if x]
+        if len(parts) < 2:
+            continue
+        first, second = parts[0], parts[1]
+        if first.lower() in {"apps", "packages", "services"}:
+            label = second
+        else:
+            label = first
+        low = label.lower()
+        if low not in seen:
+            seen.add(low)
+            labels.append(label)
+
+    if not labels:
+        return ""
+
+    def fmt(lbl: str) -> str:
+        return "API" if lbl.lower() == "api" else lbl
+
+    labels = [fmt(x) for x in labels]
+    if len(labels) == 1:
+        return f" for {labels[0]}"
+    if len(labels) == 2:
+        return f" for {labels[0]} and {labels[1]}"
+    return f" for {labels[0]}, {labels[1]}, and {labels[2]}"
+
+
+def basename_fallback_topic(paths: list[str], max_names: int = 4) -> str | None:
+    """Short description from basenames when no other topic matched."""
+    bases: list[str] = []
+    seen: set[str] = set()
+    for raw in paths:
+        b = os.path.basename(_norm(raw))
+        if not b:
+            continue
+        key = b.lower()
+        if key not in seen:
+            seen.add(key)
+            bases.append(b)
+    if not bases:
+        return None
+    if len(bases) <= max_names:
+        return ", ".join(bases)
+    head = ", ".join(bases[:max_names])
+    return f"{head} (+{len(bases) - max_names} more)"

{git_explain-1.1.3 → git_explain-2.1.4}/git_explain/run.py

@@ -27,11 +27,17 @@ def apply_commands(
     add_args: list[str],
     commit_type: str,
     commit_message: str,
+    *,
+    staged_only: bool = False,
 ) -> None:
     """Stage selected paths and commit. Raises on failure.
 
     Uses `git add -A -- <paths...>` to properly handle deletes/renames.
     Verifies that something is staged before attempting the commit.
+
+    When ``staged_only`` is True, ``git add`` is skipped (``add_args`` should be
+    empty); the current index is committed as-is. Split multi-commit plans are
+    not supported in that mode because each ``git commit`` empties the index.
     """
     root = Path(repo_root)
     if add_args:
@@ -43,6 +49,11 @@ def apply_commands(
             text=True,
         )
     if not _has_staged_changes(root):
+        if staged_only:
+            raise RuntimeError(
+                "Nothing is currently staged. With --staged-only, git-explain does "
+                "not run git add; stage your changes first, then try again."
+            )
         raise RuntimeError("Nothing staged after git add; aborting commit.")
     full_message = f"[{commit_type}] {commit_message}"
     subprocess.run(

{git_explain-1.1.3 → git_explain-2.1.4}/git_explain.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: git-explain
-Version: 1.1.3
+Version: 2.1.4
 Summary: CLI that suggests git add/commit from diffs using Gemini
 Author: git-explain contributors
 License-Expression: MIT
@@ -26,6 +26,7 @@ Requires-Dist: python-dotenv>=1.0.0
 Requires-Dist: prompt_toolkit>=3.0.0
 Provides-Extra: dev
 Requires-Dist: pytest>=8.0.0; extra == "dev"
+Requires-Dist: ruff>=0.8.0; extra == "dev"
 Dynamic: license-file
 
 # git-explain
@@ -125,7 +126,7 @@ You’ll see a list of changed files, choose which to include, then get suggeste
 | `--ai` | Use Gemini for commit type/message (file paths only). |
 | `--with-diff` | With `--ai`: send full diff to the model for detailed messages. |
 | `--model NAME` | Override Gemini model (e.g. `--model gemini-2.0-flash`). |
-| `--staged-only` | Commit only what’s already staged (no `git add`). |
+| `--staged-only` | Commit only what’s already staged (no `git add`). Always one commit for the whole index—split-by-group mode is disabled, because Git would commit the entire index on the first step and later steps would have nothing left staged. |
 | `--cwd PATH` | Run as if current directory is `PATH`. |
 | `--install-completion [SHELL]` | Install shell completion (`bash`, `zsh`). |
 | `--show-completion [SHELL]` | Print completion script for `SHELL`. |
@@ -134,7 +135,7 @@ You’ll see a list of changed files, choose which to include, then get suggeste
 
 ## Workflow
 
-1. **Changed files** — Shows staged, unstaged, and untracked files. Untracked folders are grouped (e.g. `foo/ (untracked folder; 5 files)`).
+1. **Changed files** — Shows staged, unstaged, and untracked files. Untracked directories are expanded so you still see per-file paths.
 2. **Select files** — Enter numbers (e.g. `1,2,5-7`), `all`, or a path (e.g. `main.py`, `src/utils/`).
 3. **Commit mode** — If you selected 2+ files: choose `one` (single commit) or `split` (separate commits by docs/tests/config/code).
 4. **Suggested commands** — Panel with `git add` and `git commit` lines.

{git_explain-1.1.3 → git_explain-2.1.4}/git_explain.egg-info/SOURCES.txt

@@ -3,9 +3,11 @@ README.md
 pyproject.toml
 git_explain/__init__.py
 git_explain/cli.py
+git_explain/commit_infer.py
 git_explain/gemini.py
 git_explain/git.py
 git_explain/heuristics.py
+git_explain/path_topics.py
 git_explain/run.py
 git_explain.egg-info/PKG-INFO
 git_explain.egg-info/SOURCES.txt
@@ -14,6 +16,7 @@ git_explain.egg-info/entry_points.txt
 git_explain.egg-info/requires.txt
 git_explain.egg-info/top_level.txt
 tests/test_cli_utils.py
+tests/test_commit_infer.py
 tests/test_gemini.py
 tests/test_heuristics.py
 tests/test_run_apply.py

{git_explain-1.1.3 → git_explain-2.1.4}/git_explain.egg-info/requires.txt

@@ -6,3 +6,4 @@ prompt_toolkit>=3.0.0
 
 [dev]
 pytest>=8.0.0
+ruff>=0.8.0

{git_explain-1.1.3 → git_explain-2.1.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "git-explain"
-version = "v1.1.3"
+version = "v2.1.4"
 description = "CLI that suggests git add/commit from diffs using Gemini"
 readme = "README.md"
 requires-python = ">=3.10"
@@ -36,6 +36,7 @@ dependencies = [
 [project.optional-dependencies]
 dev = [
     "pytest>=8.0.0",
+    "ruff>=0.8.0",
 ]
 
 [project.scripts]

git_explain-2.1.4/tests/test_commit_infer.py

@@ -0,0 +1,72 @@
+from git_explain.commit_infer import (
+    infer_fix_subject_from_diff,
+    refine_type_and_message_from_diff,
+)
+from git_explain.heuristics import suggest_from_changes
+
+
+def test_infer_fix_subject_staged_only_split() -> None:
+    diff = """
++        if staged_only:
++            console.print(
++                "split commits are not available with --staged-only"
++            )
+"""
+    assert infer_fix_subject_from_diff(diff) is not None
+    assert "staged-only" in (infer_fix_subject_from_diff(diff) or "").lower()
+
+
+def test_infer_fix_subject_commit_classification_helpers_in_diff() -> None:
+    diff = """
+diff --git a/git_explain/commit_infer.py b/git_explain/commit_infer.py
++def refine_type_and_message_from_diff
++def infer_fix_subject_from_diff
+"""
+    subj = infer_fix_subject_from_diff(diff)
+    assert subj is not None
+    assert "diff" in subj.lower() or "classification" in subj.lower()
+
+
+def test_infer_fix_subject_empty_index_message() -> None:
+    diff = """
++            raise RuntimeError(
++                "Nothing is currently staged. With --staged-only, git-explain does "
++                "not run git add; stage your changes first, then try again."
++            )
+"""
+    assert infer_fix_subject_from_diff(diff) is not None
+
+
+def test_refine_refactor_to_fix() -> None:
+    diff = "split commits are not available with --staged-only"
+    ct, msg = refine_type_and_message_from_diff(
+        "REFACTOR", "Update git-explain CLI", diff
+    )
+    assert ct == "FIX"
+    assert "staged-only" in msg.lower()
+
+
+def test_refine_does_not_override_docs() -> None:
+    diff = "split commits are not available with --staged-only"
+    ct, msg = refine_type_and_message_from_diff("DOCS", "Update README", diff)
+    assert ct == "DOCS"
+    assert msg == "Update README"
+
+
+def test_suggest_from_changes_with_staged_only_diff() -> None:
+    diff = """
+## Unstaged diff
+diff --git a/git_explain/cli.py b/git_explain/cli.py
++        if staged_only:
++                "split commits are not available with --staged-only"
+"""
+    s = suggest_from_changes(
+        changes=[
+            ("M", "git_explain/cli.py"),
+            ("M", "git_explain/run.py"),
+        ],
+        has_commits=True,
+        diff_text=diff,
+    )
+    assert s.commit_type == "FIX"
+    assert "staged-only" in s.commit_message.lower()

git_explain-2.1.4/tests/test_gemini.py

@@ -0,0 +1,73 @@
+from git_explain.gemini import (
+    COMMIT_LINE_CONVENTIONAL_RE,
+    COMMIT_LINE_RE,
+    _fallback_type_and_message_with_context,
+    _is_generic_message,
+)
+
+
+def test_commit_line_re_matches_tests_not_test() -> None:
+    """COMMIT_LINE_RE should match [TESTS] but not [TEST]."""
+    line_tests = 'git commit -m "[TESTS] Add unit tests"'
+    m = COMMIT_LINE_RE.match(line_tests)
+    assert m is not None
+    assert m.group(1).upper() == "TESTS"
+    assert "Add unit tests" in m.group(2)
+
+    line_test = 'git commit -m "[TEST] Add unit test"'
+    m = COMMIT_LINE_RE.match(line_test)
+    assert m is None
+
+
+def test_commit_line_conventional_re_matches_tests() -> None:
+    """COMMIT_LINE_CONVENTIONAL_RE should match 'tests:' not 'test:'."""
+    line = 'git commit -m "tests: add unit tests"'
+    m = COMMIT_LINE_CONVENTIONAL_RE.match(line)
+    assert m is not None
+    assert m.group(1).lower() == "tests"
+
+    line_test = 'git commit -m "test: add unit test"'
+    m = COMMIT_LINE_CONVENTIONAL_RE.match(line_test)
+    assert m is None
+
+
+def test_commit_line_re_matches_other_types() -> None:
+    for line in [
+        'git commit -m "[FEAT] Add feature"',
+        'git commit -m "[FIX] Fix bug"',
+        'git commit -m "[DOCS] Update readme"',
+        'git commit -m "[REFACTOR] Simplify logic"',
+        'git commit -m "[CHORE] Add Docker and nginx config"',
+    ]:
+        m = COMMIT_LINE_RE.match(line)
+        assert m is not None, f"Expected match for {line}"
+
+
+def test_commit_line_conventional_matches_chore() -> None:
+    line = 'git commit -m "chore: add docker compose"'
+    m = COMMIT_LINE_CONVENTIONAL_RE.match(line)
+    assert m is not None
+    assert m.group(1).lower() == "chore"
+
+
+def test_is_generic_message_flags_vague_add_changes() -> None:
+    assert _is_generic_message("Add changes") is True
+    assert _is_generic_message("Update changes") is True
+    assert _is_generic_message("Add Docker and nginx for api") is False
+
+
+def test_is_generic_message_flags_update_project_files() -> None:
+    assert _is_generic_message("Update project files") is True
+    assert _is_generic_message("Add project files") is True
+    assert _is_generic_message("Update tests for gemini and heuristics") is False
+
+
+def test_fallback_uses_test_hints_for_test_files() -> None:
+    ctype, msg = _fallback_type_and_message_with_context(
+        files=["tests/test_gemini.py", "tests/test_heuristics.py"],
+        added_any=False,
+        has_commits=True,
+    )
+    assert ctype == "TEST"
+    assert "gemini" in msg.lower()
+    assert "heuristics" in msg.lower()

git_explain-2.1.4/tests/test_heuristics.py

@@ -0,0 +1,94 @@
+from git_explain.heuristics import suggest_from_changes
+
+
+def test_docs_only_is_docs() -> None:
+    s = suggest_from_changes(
+        changes=[("M", "README.md"), ("A", "FEATURES.md")],
+        has_commits=True,
+    )
+    assert s.commit_type == "DOCS"
+    assert s.commit_message.lower().startswith(
+        "add"
+    ) or s.commit_message.lower().startswith("update")
+
+
+def test_added_files_prefer_feat() -> None:
+    s = suggest_from_changes(
+        changes=[("A", "git_explain/cli.py"), ("M", "pyproject.toml")],
+        has_commits=True,
+    )
+    assert s.commit_type == "FEAT"
+    assert s.commit_message.lower().startswith(
+        "add"
+    ) or s.commit_message.lower().startswith("update")
+
+
+def test_many_git_explain_paths_use_module_names_not_umbrella_cli() -> None:
+    s = suggest_from_changes(
+        changes=[
+            ("M", "git_explain/cli.py"),
+            ("M", "git_explain/run.py"),
+            ("M", "git_explain/gemini.py"),
+        ],
+        has_commits=True,
+    )
+    m = s.commit_message.lower()
+    assert "git-explain cli" not in m
+    assert "cli" in m
+
+
+def test_mostly_tests_or_config_is_test() -> None:
+    s = suggest_from_changes(
+        changes=[
+            ("M", "tests/test_cli.py"),
+            ("M", "pyproject.toml"),
+            ("M", "requirements.txt"),
+        ],
+        has_commits=True,
+    )
+    assert s.commit_type == "TEST"
+
+
+def test_config_only_is_chore_not_test() -> None:
+    s = suggest_from_changes(
+        changes=[("M", ".gitignore"), ("M", "pyproject.toml")],
+        has_commits=True,
+    )
+    assert s.commit_type == "CHORE"
+
+
+def test_test_only_paths_get_specific_test_message() -> None:
+    s = suggest_from_changes(
+        changes=[
+            ("M", "tests/test_gemini.py"),
+            ("M", "tests/test_heuristics.py"),
+        ],
+        has_commits=True,
+    )
+    assert s.commit_type == "TEST"
+    m = s.commit_message.lower()
+    assert "gemini" in m
+    assert "heuristics" in m
+    assert "project files" not in m
+
+
+def test_docker_nginx_env_paths_get_specific_chore_message() -> None:
+    """Infra paths should not collapse to 'Add changes'."""
+    s = suggest_from_changes(
+        changes=[
+            ("A", "api/app/.env.example"),
+            ("A", "api/app/Dockerfile"),
+            ("A", "apps/frontend/.dockerignore"),
+            ("A", "apps/frontend/Dockerfile"),
+            ("A", "apps/frontend/nginx.conf"),
+            ("A", "compose.env.example"),
+        ],
+        has_commits=True,
+    )
+    assert s.commit_type == "CHORE"
+    m = s.commit_message.lower()
+    assert "docker" in m
+    assert "nginx" in m
+    assert "env" in m
+    assert "changes" not in m
+    assert "api" in m and "frontend" in m

git_explain-1.1.3/tests/test_gemini.py

@@ -1,37 +0,0 @@
-from git_explain.gemini import COMMIT_LINE_CONVENTIONAL_RE, COMMIT_LINE_RE
-
-
-def test_commit_line_re_matches_tests_not_test() -> None:
-    """COMMIT_LINE_RE should match [TESTS] but not [TEST]."""
-    line_tests = 'git commit -m "[TESTS] Add unit tests"'
-    m = COMMIT_LINE_RE.match(line_tests)
-    assert m is not None
-    assert m.group(1).upper() == "TESTS"
-    assert "Add unit tests" in m.group(2)
-
-    line_test = 'git commit -m "[TEST] Add unit test"'
-    m = COMMIT_LINE_RE.match(line_test)
-    assert m is None
-
-
-def test_commit_line_conventional_re_matches_tests() -> None:
-    """COMMIT_LINE_CONVENTIONAL_RE should match 'tests:' not 'test:'."""
-    line = 'git commit -m "tests: add unit tests"'
-    m = COMMIT_LINE_CONVENTIONAL_RE.match(line)
-    assert m is not None
-    assert m.group(1).lower() == "tests"
-
-    line_test = 'git commit -m "test: add unit test"'
-    m = COMMIT_LINE_CONVENTIONAL_RE.match(line_test)
-    assert m is None
-
-
-def test_commit_line_re_matches_other_types() -> None:
-    for line in [
-        'git commit -m "[FEAT] Add feature"',
-        'git commit -m "[FIX] Fix bug"',
-        'git commit -m "[DOCS] Update readme"',
-        'git commit -m "[REFACTOR] Simplify logic"',
-    ]:
-        m = COMMIT_LINE_RE.match(line)
-        assert m is not None, f"Expected match for {line}"

git_explain-1.1.3/tests/test_heuristics.py

@@ -1,41 +0,0 @@
-from git_explain.heuristics import suggest_from_changes
-
-
-def test_docs_only_is_docs() -> None:
-    s = suggest_from_changes(
-        changes=[("M", "README.md"), ("A", "FEATURES.md")],
-        has_commits=True,
-    )
-    assert s.commit_type == "DOCS"
-    assert s.commit_message.lower().startswith(
-        "add"
-    ) or s.commit_message.lower().startswith("update")
-
-
-def test_added_files_prefer_feat() -> None:
-    s = suggest_from_changes(
-        changes=[("A", "git_explain/cli.py"), ("M", "pyproject.toml")],
-        has_commits=True,
-    )
-    assert s.commit_type == "FEAT"
-    assert s.commit_message.lower().startswith("add")
-
-
-def test_mostly_tests_or_config_is_test() -> None:
-    s = suggest_from_changes(
-        changes=[
-            ("M", "tests/test_cli.py"),
-            ("M", "pyproject.toml"),
-            ("M", "requirements.txt"),
-        ],
-        has_commits=True,
-    )
-    assert s.commit_type == "TEST"
-
-
-def test_config_only_is_chore_not_test() -> None:
-    s = suggest_from_changes(
-        changes=[("M", ".gitignore"), ("M", "pyproject.toml")],
-        has_commits=True,
-    )
-    assert s.commit_type == "CHORE"