lgit-cli 4.0.1__tar.gz → 4.1.0__tar.gz

{lgit_cli-4.0.1 → lgit_cli-4.1.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: lgit-cli
-Version: 4.0.1
+Version: 4.1.0
 Summary: AI-powered git commit message generator using Claude and other LLMs via OpenAI-compatible APIs
 Project-URL: Homepage, https://github.com/can1357/llm-git
 Project-URL: Repository, https://github.com/can1357/llm-git
@@ -208,7 +208,7 @@ analysis_model = "gpt-4o"
 summary_model = "gpt-4o-mini"
 ```
 
-The client uses tool calling for all models, falling back to parsing JSON content from the model response when a provider ignores `tool_choice`. This works across OpenRouter, LiteLLM, Anthropic, and OpenAI endpoints.
+The client asks all providers for markdown/plain-text responses and parses the same response format across OpenRouter, LiteLLM, Anthropic, and OpenAI endpoints.
 
 ### Commit Types
 

{lgit_cli-4.0.1 → lgit_cli-4.1.0}/README.md

@@ -184,7 +184,7 @@ analysis_model = "gpt-4o"
 summary_model = "gpt-4o-mini"
 ```
 
-The client uses tool calling for all models, falling back to parsing JSON content from the model response when a provider ignores `tool_choice`. This works across OpenRouter, LiteLLM, Anthropic, and OpenAI endpoints.
+The client asks all providers for markdown/plain-text responses and parses the same response format across OpenRouter, LiteLLM, Anthropic, and OpenAI endpoints.
 
 ### Commit Types
 

{lgit_cli-4.0.1 → lgit_cli-4.1.0}/lgit/__init__.py

@@ -8,7 +8,7 @@ from typing import TYPE_CHECKING, Any
 try:
     __version__ = metadata.version("lgit-cli")
 except metadata.PackageNotFoundError:
-    __version__ = "4.0.1"
+    __version__ = "4.1.0"
 
 _CORE_MODEL_EXPORTS = (
     "Mode",

{lgit_cli-4.0.1 → lgit_cli-4.1.0}/lgit/analysis.py

@@ -2,22 +2,13 @@
 
 from __future__ import annotations
 
+from collections import Counter
 from collections.abc import Sequence
 from dataclasses import dataclass, field
 from pathlib import Path
+from typing import Protocol
 
-try:  # Import defensively while shared model work lands independently.
-    from .models import ScopeCandidate
-except Exception:  # pragma: no cover - bootstrap fallback
-
-    @dataclass(slots=True)
-    class ScopeCandidate:  # type: ignore[no-redef]
-        """Candidate conventional-commit scope with confidence metadata."""
-
-        path: str
-        percentage: float
-        confidence: float
-
+from .models import ScopeCandidate
 
 PLACEHOLDER_DIRS = {
     "src",
@@ -39,15 +30,23 @@ PLACEHOLDER_DIRS = {
 SKIP_DIRS = {".test", "tests", "benches", "examples", "target", "build", "node_modules", ".github"}
 
 
+class _ScopeConfig(Protocol):
+    """Config attributes read during scope analysis; satisfied by ``CommitConfig``."""
+
+    excluded_files: list[str]
+    wide_change_threshold: float
+    wide_change_abstract: bool
+
+
 @dataclass(slots=True)
 class ScopeAnalyzer:
     """Accumulates changed-line totals per meaningful path component."""
 
-    component_lines: dict[str, int] = field(default_factory=dict)
+    component_lines: Counter[str] = field(default_factory=Counter)
     total_lines: int = 0
 
     @classmethod
-    def from_numstat(cls, numstat: str, config: object | None = None) -> ScopeAnalyzer:
+    def from_numstat(cls, numstat: str, config: _ScopeConfig | None = None) -> ScopeAnalyzer:
         """Build an analyzer from git diff --numstat output."""
 
         analyzer = cls()
@@ -55,7 +54,7 @@ class ScopeAnalyzer:
             analyzer.process_numstat_line(line, config)
         return analyzer
 
-    def process_numstat_line(self, line: str, config: object | None = None) -> None:
+    def process_numstat_line(self, line: str, config: _ScopeConfig | None = None) -> None:
         """Process one added/deleted/path numstat row."""
 
         parts = line.split("\t")
@@ -76,7 +75,7 @@ class ScopeAnalyzer:
         for component in extract_components_from_path(path):
             if any("." in segment for segment in component.split("/")):
                 continue
-            self.component_lines[component] = self.component_lines.get(component, 0) + lines_changed
+            self.component_lines[component] += lines_changed
 
     def build_scope_candidates(self) -> list[ScopeCandidate]:
         """Return sorted candidates with percentage and confidence scores."""
@@ -98,7 +97,7 @@ class ScopeAnalyzer:
         return candidates
 
     @staticmethod
-    def is_wide_change(candidates: Sequence[ScopeCandidate], config: object | None = None) -> bool:
+    def is_wide_change(candidates: Sequence[ScopeCandidate], config: _ScopeConfig | None = None) -> bool:
         """Return true when no scope dominates or many roots are touched."""
 
         threshold = float(getattr(config, "wide_change_threshold", 0.5))
@@ -108,14 +107,14 @@ class ScopeAnalyzer:
         return len(roots) >= 3
 
     @staticmethod
-    def extract_scope(numstat: str, config: object | None = None) -> tuple[list[ScopeCandidate], int]:
+    def extract_scope(numstat: str, config: _ScopeConfig | None = None) -> tuple[list[ScopeCandidate], int]:
         """Return candidates plus total changed lines from numstat."""
 
         analyzer = ScopeAnalyzer.from_numstat(numstat, config)
         return analyzer.build_scope_candidates(), analyzer.total_lines
 
     @staticmethod
-    def count_changed_lines(numstat: str, config: object | None = None) -> int:
+    def count_changed_lines(numstat: str, config: _ScopeConfig | None = None) -> int:
         """Count changed non-binary, non-excluded lines in numstat."""
 
         return ScopeAnalyzer.from_numstat(numstat, config).total_lines
@@ -124,8 +123,8 @@ class ScopeAnalyzer:
     def analyze_wide_change(numstat: str) -> str | None:
         """Detect an abstract category for cross-cutting changes."""
 
-        paths = [_path_from_numstat_line(line) for line in numstat.splitlines()]
-        paths = [path for path in paths if path]
+        raw_paths = (_path_from_numstat_line(line) for line in numstat.splitlines())
+        paths = [path for path in raw_paths if path]
         if not paths:
             return None
 
@@ -175,7 +174,7 @@ def extract_scope_candidates(
     source: object,
     target: str | None = None,
     dir: str = ".",
-    config: object | None = None,
+    config: _ScopeConfig | None = None,
 ) -> tuple[str, bool]:
     """Extract a scope prompt string and wide-change flag.
 
@@ -222,16 +221,6 @@ def extract_scope_candidates(
     return scope_str, is_wide
 
 
-def extract_scope_candidates_from_numstat(
-    numstat: str,
-    config: object | None = None,
-) -> tuple[list[ScopeCandidate], bool, int]:
-    """Return raw candidate objects, wide-change flag, and total lines."""
-
-    candidates, total = ScopeAnalyzer.extract_scope(numstat, config)
-    return candidates, ScopeAnalyzer.is_wide_change(candidates, config), total
-
-
 def extract_path_from_rename(path_part: str) -> str:
     """Return the destination path from git numstat rename syntax."""
 
@@ -305,7 +294,7 @@ def _is_file_segment(segment: str) -> bool:
     return "." in segment and not segment.startswith(".") and segment.rfind(".") > 0
 
 
-def _is_excluded(path: str, config: object | None) -> bool:
+def _is_excluded(path: str, config: _ScopeConfig | None) -> bool:
     excluded = getattr(config, "excluded_files", ())
     return any(path.endswith(str(pattern)) for pattern in excluded)