gac 1.9.1__tar.gz → 1.9.3__tar.gz

{gac-1.9.1 → gac-1.9.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: gac
-Version: 1.9.1
+Version: 1.9.3
 Summary: AI-powered Git commit message generator with multi-provider support
 Project-URL: Homepage, https://github.com/cellwebb/gac
 Project-URL: Documentation, https://github.com/cellwebb/gac#readme
@@ -50,7 +50,8 @@ Description-Content-Type: text/markdown
 [![Python](https://img.shields.io/badge/python-3.10%20|%203.11%20|%203.12%20|%203.13%20|%203.14-blue.svg)](https://www.python.org/downloads/)
 [![Build Status](https://github.com/cellwebb/gac/actions/workflows/ci.yml/badge.svg)](https://github.com/cellwebb/gac/actions)
 [![codecov](https://codecov.io/gh/cellwebb/gac/branch/main/graph/badge.svg)](https://app.codecov.io/gh/cellwebb/gac)
-[![Code Style: black](https://img.shields.io/badge/code%20style-black-000000.svg)](https://github.com/psf/black)
+[![Ruff](https://img.shields.io/endpoint?url=https://raw.githubusercontent.com/astral-sh/ruff/main/assets/badge/v2.json)](https://github.com/astral-sh/ruff)
+[![mypy](https://img.shields.io/badge/mypy-checked-blue.svg)](https://mypy-lang.org/)
 [![Contributions welcome](https://img.shields.io/badge/contributions-welcome-brightgreen.svg)](docs/CONTRIBUTING.md)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](LICENSE)
 

{gac-1.9.1 → gac-1.9.3}/README.md

@@ -6,7 +6,8 @@
 [![Python](https://img.shields.io/badge/python-3.10%20|%203.11%20|%203.12%20|%203.13%20|%203.14-blue.svg)](https://www.python.org/downloads/)
 [![Build Status](https://github.com/cellwebb/gac/actions/workflows/ci.yml/badge.svg)](https://github.com/cellwebb/gac/actions)
 [![codecov](https://codecov.io/gh/cellwebb/gac/branch/main/graph/badge.svg)](https://app.codecov.io/gh/cellwebb/gac)
-[![Code Style: black](https://img.shields.io/badge/code%20style-black-000000.svg)](https://github.com/psf/black)
+[![Ruff](https://img.shields.io/endpoint?url=https://raw.githubusercontent.com/astral-sh/ruff/main/assets/badge/v2.json)](https://github.com/astral-sh/ruff)
+[![mypy](https://img.shields.io/badge/mypy-checked-blue.svg)](https://mypy-lang.org/)
 [![Contributions welcome](https://img.shields.io/badge/contributions-welcome-brightgreen.svg)](docs/CONTRIBUTING.md)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](LICENSE)
 

{gac-1.9.1 → gac-1.9.3}/pyproject.toml

@@ -205,6 +205,29 @@ markers = [
 ]
 addopts = "-m 'not integration'"
 
+[tool.mypy]
+python_version = "3.10"
+warn_return_any = true
+warn_unused_configs = true
+disallow_untyped_defs = false
+disallow_incomplete_defs = false
+check_untyped_defs = true
+no_implicit_optional = true
+warn_redundant_casts = true
+warn_unused_ignores = true
+warn_no_return = true
+warn_unreachable = true
+strict_equality = true
+show_error_codes = true
+
+[[tool.mypy.overrides]]
+module = "gac.providers.*"
+warn_return_any = false
+
+[[tool.mypy.overrides]]
+module = "halo"
+ignore_missing_imports = true
+
 [template.plugins.default]
 tests = true
 src-layout = true
@@ -215,4 +238,5 @@ dev = [
     "pytest-asyncio>=1.2.0",
     "pytest-cov>=7.0.0",
     "pre-commit",
+    "mypy>=1.8.0",
 ]

{gac-1.9.1 → gac-1.9.3}/src/gac/__version__.py

@@ -1,3 +1,3 @@
 """Version information for gac package."""
 
-__version__ = "1.9.1"
+__version__ = "1.9.3"

{gac-1.9.1 → gac-1.9.3}/src/gac/ai_utils.py

@@ -38,7 +38,7 @@ def extract_text_content(content: str | list[dict[str, str]] | dict[str, Any]) -
     elif isinstance(content, list):
         return "\n".join(msg["content"] for msg in content if isinstance(msg, dict) and "content" in msg)
     elif isinstance(content, dict) and "content" in content:
-        return content["content"]
+        return content["content"]  # type: ignore[no-any-return]
     return ""
 
 
@@ -144,7 +144,7 @@ def generate_with_retries(
                 spinner.succeed(f"Generated commit message with {provider} {model_name}")
 
             if content is not None and content.strip():
-                return content.strip()
+                return content.strip()  # type: ignore[no-any-return]
             else:
                 logger.warning(f"Empty or None content received from {provider} {model_name}: {repr(content)}")
                 raise AIError.model_error("Empty response from AI model")

{gac-1.9.1 → gac-1.9.3}/src/gac/cli.py

@@ -66,7 +66,7 @@ logger = logging.getLogger(__name__)
 def cli(
     ctx: click.Context,
     add_all: bool = False,
-    log_level: str = config["log_level"],
+    log_level: str = str(config["log_level"]),
     one_liner: bool = False,
     push: bool = False,
     show_prompt: bool = False,
@@ -74,7 +74,7 @@ def cli(
     quiet: bool = False,
     yes: bool = False,
     hint: str = "",
-    model: str = None,
+    model: str | None = None,
     version: bool = False,
     dry_run: bool = False,
     verbose: bool = False,

{gac-1.9.1 → gac-1.9.3}/src/gac/config.py

@@ -11,7 +11,7 @@ from dotenv import load_dotenv
 from gac.constants import EnvDefaults, Logging
 
 
-def load_config() -> dict[str, str | int | float | bool]:
+def load_config() -> dict[str, str | int | float | bool | None]:
     """Load configuration from $HOME/.gac.env, then ./.gac.env or ./.env, then environment variables."""
     user_config = Path.home() / ".gac.env"
     if user_config.exists():

{gac-1.9.1 → gac-1.9.3}/src/gac/constants.py

@@ -2,7 +2,6 @@
 
 import os
 from enum import Enum
-from re import Pattern
 
 
 class FileStatus(Enum):
@@ -48,7 +47,7 @@ class FilePatterns:
     """Patterns for identifying special file types."""
 
     # Regex patterns to detect binary file changes in git diffs (e.g., images or other non-text files)
-    BINARY: list[Pattern[str]] = [
+    BINARY: list[str] = [
         r"Binary files .* differ",
         r"GIT binary patch",
     ]
@@ -129,7 +128,7 @@ class CodePatternImportance:
 
     # Regex patterns to detect code structure changes in git diffs (e.g., class, function, import)
     # Note: The patterns are prefixed with "+" to match only added and modified lines
-    PATTERNS: dict[Pattern[str], float] = {
+    PATTERNS: dict[str, float] = {
         # Structure changes
         r"\+\s*(class|interface|enum)\s+\w+": 1.8,  # Class/interface/enum definitions
         r"\+\s*(def|function|func)\s+\w+\s*\(": 1.5,  # Function definitions

{gac-1.9.1 → gac-1.9.3}/src/gac/git.py

@@ -120,8 +120,8 @@ def run_pre_commit_hooks() -> bool:
     # Check if pre-commit is installed and configured
     try:
         # First check if pre-commit is installed
-        result = run_subprocess(["pre-commit", "--version"], silent=True, raise_on_error=False)
-        if not result:
+        version_check = run_subprocess(["pre-commit", "--version"], silent=True, raise_on_error=False)
+        if not version_check:
             logger.debug("pre-commit not installed, skipping hooks")
             return True
 
@@ -170,8 +170,8 @@ def run_lefthook_hooks() -> bool:
     # Check if lefthook is installed and configured
     try:
         # First check if lefthook is installed
-        result = run_subprocess(["lefthook", "--version"], silent=True, raise_on_error=False)
-        if not result:
+        version_check = run_subprocess(["lefthook", "--version"], silent=True, raise_on_error=False)
+        if not version_check:
             logger.debug("Lefthook not installed, skipping hooks")
             return True
 

{gac-1.9.1 → gac-1.9.3}/src/gac/init_cli.py

@@ -17,7 +17,7 @@ def _prompt_required_text(prompt: str) -> str | None:
             return None
         value = response.strip()
         if value:
-            return value
+            return value  # type: ignore[no-any-return]
         click.echo("A value is required. Please try again.")
 
 
@@ -34,7 +34,7 @@ def init() -> None:
     providers = [
         ("Anthropic", "claude-haiku-4-5"),
         ("Cerebras", "qwen-3-coder-480b"),
-        ("Chutes.ai", "zai-org/GLM-4.6-FP8"),
+        ("Chutes", "zai-org/GLM-4.6-FP8"),
         ("Gemini", "gemini-2.5-flash"),
         ("Groq", "meta-llama/llama-4-maverick-17b-128e-instruct"),
         ("LM Studio", "gemma3"),
@@ -51,7 +51,7 @@ def init() -> None:
     if not provider:
         click.echo("Provider selection cancelled. Exiting.")
         return
-    provider_key = provider.lower().replace(".", "").replace(" ", "-").replace("syntheticnew", "synthetic")
+    provider_key = provider.lower().replace(".", "").replace(" ", "-")
 
     is_ollama = provider_key == "ollama"
     is_lmstudio = provider_key == "lm-studio"
@@ -104,7 +104,9 @@ def init() -> None:
 
     api_key = questionary.password(api_key_prompt).ask()
     if api_key:
-        if is_zai:
+        if is_lmstudio:
+            api_key_name = "LMSTUDIO_API_KEY"
+        elif is_zai:
             api_key_name = "ZAI_API_KEY"
         else:
             api_key_name = f"{provider_key.upper()}_API_KEY"

{gac-1.9.1 → gac-1.9.3}/src/gac/main.py

@@ -57,18 +57,27 @@ def main(
         handle_error(GitError("Not in a git repository"), exit_program=True)
 
     if model is None:
-        model = config["model"]
-        if model is None:
+        model_from_config = config["model"]
+        if model_from_config is None:
             handle_error(
                 AIError.model_error(
                     "gac init hasn't been run yet. Please run 'gac init' to set up your configuration, then try again."
                 ),
                 exit_program=True,
             )
+        model = str(model_from_config)
 
-    temperature = config["temperature"]
-    max_output_tokens = config["max_output_tokens"]
-    max_retries = config["max_retries"]
+    temperature_val = config["temperature"]
+    assert temperature_val is not None
+    temperature = float(temperature_val)
+
+    max_tokens_val = config["max_output_tokens"]
+    assert max_tokens_val is not None
+    max_output_tokens = int(max_tokens_val)
+
+    max_retries_val = config["max_retries"]
+    assert max_retries_val is not None
+    max_retries = int(max_retries_val)
 
     if stage_all and (not dry_run):
         logger.info("Staging all changes")
@@ -168,8 +177,8 @@ def main(
 
     # Preprocess the diff before passing to build_prompt
     logger.debug(f"Preprocessing diff ({len(diff)} characters)")
-    model_id = model or config["model"]
-    processed_diff = preprocess_diff(diff, token_limit=Utility.DEFAULT_DIFF_TOKEN_LIMIT, model=model_id)
+    assert model is not None
+    processed_diff = preprocess_diff(diff, token_limit=Utility.DEFAULT_DIFF_TOKEN_LIMIT, model=model)
     logger.debug(f"Processed diff ({len(processed_diff)} characters)")
 
     system_prompt, user_prompt = build_prompt(
@@ -197,7 +206,9 @@ def main(
         # Count tokens for both prompts
         prompt_tokens = count_tokens(system_prompt, model) + count_tokens(user_prompt, model)
 
-        warning_limit = config.get("warning_limit_tokens", EnvDefaults.WARNING_LIMIT_TOKENS)
+        warning_limit_val = config.get("warning_limit_tokens", EnvDefaults.WARNING_LIMIT_TOKENS)
+        assert warning_limit_val is not None
+        warning_limit = int(warning_limit_val)
         if warning_limit and prompt_tokens > warning_limit:
             console.print(
                 f"[yellow]⚠️  WARNING: Prompt contains {prompt_tokens} tokens, which exceeds the warning limit of "

{gac-1.9.1 → gac-1.9.3}/src/gac/preprocess.py

@@ -143,7 +143,7 @@ def extract_binary_file_summary(section: str) -> str:
     return extract_filtered_file_summary(section, "[Binary file change]")
 
 
-def extract_filtered_file_summary(section: str, change_type: str = None) -> str:
+def extract_filtered_file_summary(section: str, change_type: str | None = None) -> str:
     """Extract a summary of filtered file changes from a diff section.
 
     Args:

{gac-1.9.1 → gac-1.9.3}/src/gac/security.py

@@ -179,7 +179,7 @@ def scan_diff_section(section: str) -> list[DetectedSecret]:
     Returns:
         List of detected secrets
     """
-    secrets = []
+    secrets: list[DetectedSecret] = []
     file_path = extract_file_path_from_diff_section(section)
 
     if not file_path:

{gac-1.9.1 → gac-1.9.3}/src/gac/utils.py

@@ -36,7 +36,7 @@ def setup_logging(
         level=log_level,
         format="%(asctime)s - %(name)s - %(levelname)s - %(message)s",
         datefmt="%Y-%m-%d %H:%M:%S",
-        **kwargs,
+        **kwargs,  # type: ignore[arg-type]
     )
 
     if suppress_noisy: