gac 2.3.0__tar.gz → 2.4.1__tar.gz

{gac-2.3.0 → gac-2.4.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: gac
-Version: 2.3.0
+Version: 2.4.1
 Summary: LLM-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
@@ -41,6 +41,9 @@ Requires-Dist: twine; extra == 'dev'
 Description-Content-Type: text/markdown
 
 <!-- markdownlint-disable MD013 -->
+<!-- markdownlint-disable MD033 MD036 -->
+
+<div align="center">
 
 # 🚀 Git Auto Commit (gac)
 
@@ -50,9 +53,11 @@ Description-Content-Type: text/markdown
 [![codecov](https://codecov.io/gh/cellwebb/gac/branch/main/graph/badge.svg)](https://app.codecov.io/gh/cellwebb/gac)
 [![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)
+[![Contributions welcome](https://img.shields.io/badge/contributions-welcome-brightgreen.svg)](docs/en/CONTRIBUTING.md)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](LICENSE)
 
+**English** | [简体中文](README.zh-CN.md) | [繁體中文](README.zh-TW.md) | [日本語](README.ja.md) | [한국어](README.ko.md) | [हिन्दी](README.hi.md) | [Français](README.fr.md) | [Русский](README.ru.md) | [Español](README.es.md) | [Português](README.pt.md) | [Deutsch](README.de.md) | [Nederlands](README.nl.md)
+
 **LLM-powered commit messages that understand your code!**
 
 **Automate your commits!** Replace `git commit -m "..."` with `gac` for contextual, well-formatted commit messages generated by large language models!
@@ -67,12 +72,19 @@ Intelligent, contextual messages that explain the **why** behind your changes:
 
 ---
 
+</div>
+
+<!-- markdownlint-enable MD033 MD036 -->
+
+<!-- markdownlint-enable MD033 MD036 -->
+
 ## Quick Start
 
 ### Use gac without installing
 
 ```bash
-uvx gac init  # Configure your LLM provider
+uvx gac init   # Configure your provider, model, and language
+uvx gac model  # Re-run provider/model setup without language prompts
 uvx gac  # Generate and commit with LLM
 ```
 
@@ -83,6 +95,7 @@ That's it! Review the generated message and confirm with `y`.
 ```bash
 uv tool install gac
 gac init
+gac model
 gac
 ```
 
@@ -220,6 +233,8 @@ The edit feature (`e`) provides rich in-place terminal editing, allowing you to:
 
 Run `gac init` to configure your provider interactively, or set environment variables:
 
+Need to change providers or models later without touching language settings? Use `gac model` for a streamlined flow that skips the language prompts.
+
 ```bash
 # Example configuration
 GAC_MODEL=anthropic:your-model-name
@@ -231,16 +246,24 @@ See `.gac.env.example` for all available options.
 
 **Want commit messages in another language?** Run `gac language` to select from 25+ languages including Español, Français, 日本語, and more.
 
-**Want to customize commit message style?** See [docs/CUSTOM_SYSTEM_PROMPTS.md](docs/CUSTOM_SYSTEM_PROMPTS.md) for guidance on writing custom system prompts.
+**Want to customize commit message style?** See [docs/CUSTOM_SYSTEM_PROMPTS.md](docs/en/CUSTOM_SYSTEM_PROMPTS.md) for guidance on writing custom system prompts.
+
+---
+
+## Project Analytics
+
+📊 **[View live usage analytics and statistics →](https://clickpy.clickhouse.com/dashboard/gac)**
+
+Track real-time installation metrics and package download statistics.
 
 ---
 
 ## Getting Help
 
-- **Full documentation**: [USAGE.md](USAGE.md) - Complete CLI reference
-- **Custom prompts**: [CUSTOM_SYSTEM_PROMPTS.md](docs/CUSTOM_SYSTEM_PROMPTS.md) - Customize commit message style
-- **Troubleshooting**: [TROUBLESHOOTING.md](docs/TROUBLESHOOTING.md) - Common issues and solutions
-- **Contributing**: [CONTRIBUTING.md](docs/CONTRIBUTING.md) - Development setup and guidelines
+- **Full documentation**: [docs/USAGE.md](docs/en/USAGE.md) - Complete CLI reference
+- **Custom prompts**: [docs/CUSTOM_SYSTEM_PROMPTS.md](docs/en/CUSTOM_SYSTEM_PROMPTS.md) - Customize commit message style
+- **Troubleshooting**: [docs/TROUBLESHOOTING.md](docs/en/TROUBLESHOOTING.md) - Common issues and solutions
+- **Contributing**: [docs/CONTRIBUTING.md](docs/en/CONTRIBUTING.md) - Development setup and guidelines
 
 ---
 
@@ -250,7 +273,7 @@ See `.gac.env.example` for all available options.
 
 Made with ❤️ for developers who want better commit messages
 
-[⭐ Star us on GitHub](https://github.com/cellwebb/gac) • [🐛 Report issues](https://github.com/cellwebb/gac/issues) • [📖 Full docs](USAGE.md)
+[⭐ Star us on GitHub](https://github.com/cellwebb/gac) • [🐛 Report issues](https://github.com/cellwebb/gac/issues) • [📖 Full docs](docs/en/USAGE.md)
 
 </div>
 

{gac-2.3.0 → gac-2.4.1}/README.md

@@ -1,4 +1,7 @@
 <!-- markdownlint-disable MD013 -->
+<!-- markdownlint-disable MD033 MD036 -->
+
+<div align="center">
 
 # 🚀 Git Auto Commit (gac)
 
@@ -8,9 +11,11 @@
 [![codecov](https://codecov.io/gh/cellwebb/gac/branch/main/graph/badge.svg)](https://app.codecov.io/gh/cellwebb/gac)
 [![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)
+[![Contributions welcome](https://img.shields.io/badge/contributions-welcome-brightgreen.svg)](docs/en/CONTRIBUTING.md)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](LICENSE)
 
+**English** | [简体中文](README.zh-CN.md) | [繁體中文](README.zh-TW.md) | [日本語](README.ja.md) | [한국어](README.ko.md) | [हिन्दी](README.hi.md) | [Français](README.fr.md) | [Русский](README.ru.md) | [Español](README.es.md) | [Português](README.pt.md) | [Deutsch](README.de.md) | [Nederlands](README.nl.md)
+
 **LLM-powered commit messages that understand your code!**
 
 **Automate your commits!** Replace `git commit -m "..."` with `gac` for contextual, well-formatted commit messages generated by large language models!
@@ -25,12 +30,19 @@ Intelligent, contextual messages that explain the **why** behind your changes:
 
 ---
 
+</div>
+
+<!-- markdownlint-enable MD033 MD036 -->
+
+<!-- markdownlint-enable MD033 MD036 -->
+
 ## Quick Start
 
 ### Use gac without installing
 
 ```bash
-uvx gac init  # Configure your LLM provider
+uvx gac init   # Configure your provider, model, and language
+uvx gac model  # Re-run provider/model setup without language prompts
 uvx gac  # Generate and commit with LLM
 ```
 
@@ -41,6 +53,7 @@ That's it! Review the generated message and confirm with `y`.
 ```bash
 uv tool install gac
 gac init
+gac model
 gac
 ```
 
@@ -178,6 +191,8 @@ The edit feature (`e`) provides rich in-place terminal editing, allowing you to:
 
 Run `gac init` to configure your provider interactively, or set environment variables:
 
+Need to change providers or models later without touching language settings? Use `gac model` for a streamlined flow that skips the language prompts.
+
 ```bash
 # Example configuration
 GAC_MODEL=anthropic:your-model-name
@@ -189,16 +204,24 @@ See `.gac.env.example` for all available options.
 
 **Want commit messages in another language?** Run `gac language` to select from 25+ languages including Español, Français, 日本語, and more.
 
-**Want to customize commit message style?** See [docs/CUSTOM_SYSTEM_PROMPTS.md](docs/CUSTOM_SYSTEM_PROMPTS.md) for guidance on writing custom system prompts.
+**Want to customize commit message style?** See [docs/CUSTOM_SYSTEM_PROMPTS.md](docs/en/CUSTOM_SYSTEM_PROMPTS.md) for guidance on writing custom system prompts.
+
+---
+
+## Project Analytics
+
+📊 **[View live usage analytics and statistics →](https://clickpy.clickhouse.com/dashboard/gac)**
+
+Track real-time installation metrics and package download statistics.
 
 ---
 
 ## Getting Help
 
-- **Full documentation**: [USAGE.md](USAGE.md) - Complete CLI reference
-- **Custom prompts**: [CUSTOM_SYSTEM_PROMPTS.md](docs/CUSTOM_SYSTEM_PROMPTS.md) - Customize commit message style
-- **Troubleshooting**: [TROUBLESHOOTING.md](docs/TROUBLESHOOTING.md) - Common issues and solutions
-- **Contributing**: [CONTRIBUTING.md](docs/CONTRIBUTING.md) - Development setup and guidelines
+- **Full documentation**: [docs/USAGE.md](docs/en/USAGE.md) - Complete CLI reference
+- **Custom prompts**: [docs/CUSTOM_SYSTEM_PROMPTS.md](docs/en/CUSTOM_SYSTEM_PROMPTS.md) - Customize commit message style
+- **Troubleshooting**: [docs/TROUBLESHOOTING.md](docs/en/TROUBLESHOOTING.md) - Common issues and solutions
+- **Contributing**: [docs/CONTRIBUTING.md](docs/en/CONTRIBUTING.md) - Development setup and guidelines
 
 ---
 
@@ -208,7 +231,7 @@ See `.gac.env.example` for all available options.
 
 Made with ❤️ for developers who want better commit messages
 
-[⭐ Star us on GitHub](https://github.com/cellwebb/gac) • [🐛 Report issues](https://github.com/cellwebb/gac/issues) • [📖 Full docs](USAGE.md)
+[⭐ Star us on GitHub](https://github.com/cellwebb/gac) • [🐛 Report issues](https://github.com/cellwebb/gac/issues) • [📖 Full docs](docs/en/USAGE.md)
 
 </div>
 

{gac-2.3.0 → gac-2.4.1}/src/gac/__version__.py

@@ -1,3 +1,3 @@
 """Version information for gac package."""
 
-__version__ = "2.3.0"
+__version__ = "2.4.1"

{gac-2.3.0 → gac-2.4.1}/src/gac/cli.py

@@ -17,6 +17,7 @@ from gac.constants import Languages, Logging
 from gac.diff_cli import diff as diff_cli
 from gac.errors import handle_error
 from gac.init_cli import init as init_cli
+from gac.init_cli import model as model_cli
 from gac.language_cli import language as language_cli
 from gac.main import main
 from gac.utils import setup_logging
@@ -154,9 +155,10 @@ def cli(
 
 
 cli.add_command(config_cli)
+cli.add_command(diff_cli)
 cli.add_command(init_cli)
 cli.add_command(language_cli)
-cli.add_command(diff_cli)
+cli.add_command(model_cli)
 
 
 @click.command(context_settings=language_cli.context_settings)

{gac-2.3.0 → gac-2.4.1}/src/gac/config.py

@@ -41,6 +41,7 @@ def load_config() -> dict[str, str | int | float | bool | None]:
         "system_prompt_path": os.getenv("GAC_SYSTEM_PROMPT_PATH"),
         "language": os.getenv("GAC_LANGUAGE"),
         "translate_prefixes": os.getenv("GAC_TRANSLATE_PREFIXES", "false").lower() in ("true", "1", "yes", "on"),
+        "rtl_confirmed": os.getenv("GAC_RTL_CONFIRMED", "false").lower() in ("true", "1", "yes", "on"),
     }
 
     return config

{gac-2.3.0 → gac-2.4.1}/src/gac/git.py

@@ -14,6 +14,63 @@ from gac.utils import run_subprocess
 logger = logging.getLogger(__name__)
 
 
+def run_subprocess_with_encoding_fallback(
+    command: list[str], silent: bool = False, timeout: int = 60
+) -> subprocess.CompletedProcess:
+    """Run subprocess with encoding fallback, returning full CompletedProcess object.
+
+    This is used for cases where we need both stdout and stderr separately,
+    like pre-commit and lefthook hook execution.
+
+    Args:
+        command: List of command arguments
+        silent: If True, suppress debug logging
+        timeout: Command timeout in seconds
+
+    Returns:
+        CompletedProcess object with stdout, stderr, and returncode
+    """
+    from gac.utils import get_safe_encodings
+
+    encodings = get_safe_encodings()
+    last_exception: Exception | None = None
+
+    for encoding in encodings:
+        try:
+            if not silent:
+                logger.debug(f"Running command: {' '.join(command)} (encoding: {encoding})")
+
+            result = subprocess.run(
+                command,
+                capture_output=True,
+                text=True,
+                check=False,
+                timeout=timeout,
+                encoding=encoding,
+                errors="replace",
+            )
+            return result
+        except UnicodeError as e:
+            last_exception = e
+            if not silent:
+                logger.debug(f"Failed to decode with {encoding}: {e}")
+            continue
+        except subprocess.TimeoutExpired:
+            raise
+        except Exception as e:
+            if not silent:
+                logger.debug(f"Command error: {e}")
+            # Try next encoding for non-timeout errors
+            last_exception = e
+            continue
+
+    # If we get here, all encodings failed
+    if last_exception:
+        raise subprocess.CalledProcessError(1, command, "", f"Encoding error: {last_exception}") from last_exception
+    else:
+        raise subprocess.CalledProcessError(1, command, "", "All encoding attempts failed")
+
+
 def run_git_command(args: list[str], silent: bool = False, timeout: int = 30) -> str:
     """Run a git command and return the output."""
     command = ["git"] + args
@@ -132,20 +189,20 @@ def get_diff(staged: bool = True, color: bool = True, commit1: str | None = None
 
 def get_repo_root() -> str:
     """Get absolute path of repository root."""
-    result = subprocess.check_output(["git", "rev-parse", "--show-toplevel"])
-    return result.decode().strip()
+    result = run_git_command(["rev-parse", "--show-toplevel"])
+    return result
 
 
 def get_current_branch() -> str:
     """Get name of current git branch."""
-    result = subprocess.check_output(["git", "rev-parse", "--abbrev-ref", "HEAD"])
-    return result.decode().strip()
+    result = run_git_command(["rev-parse", "--abbrev-ref", "HEAD"])
+    return result
 
 
 def get_commit_hash() -> str:
     """Get SHA-1 hash of current commit."""
-    result = subprocess.check_output(["git", "rev-parse", "HEAD"])
-    return result.decode().strip()
+    result = run_git_command(["rev-parse", "HEAD"])
+    return result
 
 
 def run_pre_commit_hooks() -> bool:
@@ -170,7 +227,7 @@ def run_pre_commit_hooks() -> bool:
         # Run pre-commit hooks on staged files
         logger.info("Running pre-commit hooks...")
         # Run pre-commit and capture both stdout and stderr
-        result = subprocess.run(["pre-commit", "run"], capture_output=True, text=True, check=False)
+        result = run_subprocess_with_encoding_fallback(["pre-commit", "run"])
 
         if result.returncode == 0:
             # All hooks passed
@@ -220,7 +277,7 @@ def run_lefthook_hooks() -> bool:
         # Run lefthook hooks on staged files
         logger.info("Running Lefthook hooks...")
         # Run lefthook and capture both stdout and stderr
-        result = subprocess.run(["lefthook", "run", "pre-commit"], capture_output=True, text=True, check=False)
+        result = run_subprocess_with_encoding_fallback(["lefthook", "run", "pre-commit"])
 
         if result.returncode == 0:
             # All hooks passed

{gac-2.3.0 → gac-2.4.1}/src/gac/init_cli.py

@@ -1,16 +1,58 @@
 """CLI for initializing gac configuration interactively."""
 
+import os
 from pathlib import Path
 
 import click
 import questionary
-from dotenv import dotenv_values, set_key
+from dotenv import dotenv_values, load_dotenv, set_key
 
 from gac.constants import Languages
 
 GAC_ENV_PATH = Path.home() / ".gac.env"
 
 
+def _should_show_rtl_warning_for_init() -> bool:
+    """Check if RTL warning should be shown based on init's GAC_ENV_PATH.
+
+    Returns:
+        True if warning should be shown, False if user previously confirmed
+    """
+    if GAC_ENV_PATH.exists():
+        load_dotenv(GAC_ENV_PATH)
+        rtl_confirmed = os.getenv("GAC_RTL_CONFIRMED", "false").lower() in ("true", "1", "yes", "on")
+        return not rtl_confirmed
+    return True  # Show warning if no config exists
+
+
+def _show_rtl_warning_for_init(language_name: str) -> bool:
+    """Show RTL language warning for init command and save preference to GAC_ENV_PATH.
+
+    Args:
+        language_name: Name of the RTL language
+
+    Returns:
+        True if user wants to proceed, False if they cancel
+    """
+
+    terminal_width = 80  # Use default width
+    title = "⚠️  RTL Language Detected".center(terminal_width)
+
+    click.echo()
+    click.echo(click.style(title, fg="yellow", bold=True))
+    click.echo()
+    click.echo("Right-to-left (RTL) languages may not display correctly in gac due to terminal limitations.")
+    click.echo("However, the commit messages will work fine and should be readable in Git clients")
+    click.echo("that properly support RTL text (like most web interfaces and modern tools).\n")
+
+    proceed = questionary.confirm("Do you want to proceed anyway?").ask()
+    if proceed:
+        # Remember that user has confirmed RTL acceptance
+        set_key(str(GAC_ENV_PATH), "GAC_RTL_CONFIRMED", "true")
+        click.echo("✓ RTL preference saved - you won't see this warning again")
+    return proceed if proceed is not None else False
+
+
 def _prompt_required_text(prompt: str) -> str | None:
     """Prompt until a non-empty string is provided or the user cancels."""
     while True:
@@ -23,20 +65,20 @@ def _prompt_required_text(prompt: str) -> str | None:
         click.echo("A value is required. Please try again.")
 
 
-@click.command()
-def init() -> None:
-    """Interactively set up $HOME/.gac.env for gac."""
-    click.echo("Welcome to gac initialization!\n")
-
-    # Load existing environment values
-    existing_env = {}
+def _load_existing_env() -> dict[str, str]:
+    """Ensure the env file exists and return its current values."""
+    existing_env: dict[str, str] = {}
     if GAC_ENV_PATH.exists():
         click.echo(f"$HOME/.gac.env already exists at {GAC_ENV_PATH}. Values will be updated.")
-        existing_env = dict(dotenv_values(str(GAC_ENV_PATH)))
+        existing_env = {k: v for k, v in dotenv_values(str(GAC_ENV_PATH)).items() if v is not None}
     else:
         GAC_ENV_PATH.touch()
         click.echo(f"Created $HOME/.gac.env at {GAC_ENV_PATH}.")
+    return existing_env
 
+
+def _configure_model(existing_env: dict[str, str]) -> bool:
+    """Run the provider/model/API key configuration flow."""
     providers = [
         ("Anthropic", "claude-haiku-4-5"),
         ("Cerebras", "zai-glm-4.6"),
@@ -63,7 +105,7 @@ def init() -> None:
     provider = questionary.select("Select your provider:", choices=provider_names).ask()
     if not provider:
         click.echo("Provider selection cancelled. Exiting.")
-        return
+        return False
     provider_key = provider.lower().replace(".", "").replace(" ", "-").replace("(", "").replace(")", "")
 
     is_ollama = provider_key == "ollama"
@@ -77,7 +119,7 @@ def init() -> None:
         endpoint_id = _prompt_required_text("Enter the Streamlake inference endpoint ID (required):")
         if endpoint_id is None:
             click.echo("Streamlake configuration cancelled. Exiting.")
-            return
+            return False
         model_to_save = endpoint_id
     else:
         model_suggestion = dict(providers)[provider]
@@ -88,7 +130,7 @@ def init() -> None:
         model = questionary.text(model_prompt, default=model_suggestion).ask()
         if model is None:
             click.echo("Model entry cancelled. Exiting.")
-            return
+            return False
         model_to_save = model.strip() if model.strip() else model_suggestion
 
     set_key(str(GAC_ENV_PATH), "GAC_MODEL", f"{provider_key}:{model_to_save}")
@@ -98,7 +140,7 @@ def init() -> None:
         base_url = _prompt_required_text("Enter the custom Anthropic-compatible base URL (required):")
         if base_url is None:
             click.echo("Custom Anthropic base URL entry cancelled. Exiting.")
-            return
+            return False
         set_key(str(GAC_ENV_PATH), "CUSTOM_ANTHROPIC_BASE_URL", base_url)
         click.echo(f"Set CUSTOM_ANTHROPIC_BASE_URL={base_url}")
 
@@ -112,7 +154,7 @@ def init() -> None:
         base_url = _prompt_required_text("Enter the custom OpenAI-compatible base URL (required):")
         if base_url is None:
             click.echo("Custom OpenAI base URL entry cancelled. Exiting.")
-            return
+            return False
         set_key(str(GAC_ENV_PATH), "CUSTOM_OPENAI_BASE_URL", base_url)
         click.echo(f"Set CUSTOM_OPENAI_BASE_URL={base_url}")
     elif is_ollama:
@@ -120,7 +162,7 @@ def init() -> None:
         url = questionary.text(f"Enter the Ollama API URL (default: {url_default}):", default=url_default).ask()
         if url is None:
             click.echo("Ollama URL entry cancelled. Exiting.")
-            return
+            return False
         url_to_save = url.strip() if url.strip() else url_default
         set_key(str(GAC_ENV_PATH), "OLLAMA_API_URL", url_to_save)
         click.echo(f"Set OLLAMA_API_URL={url_to_save}")
@@ -129,7 +171,7 @@ def init() -> None:
         url = questionary.text(f"Enter the LM Studio API URL (default: {url_default}):", default=url_default).ask()
         if url is None:
             click.echo("LM Studio URL entry cancelled. Exiting.")
-            return
+            return False
         url_to_save = url.strip() if url.strip() else url_default
         set_key(str(GAC_ENV_PATH), "LMSTUDIO_API_URL", url_to_save)
         click.echo(f"Set LMSTUDIO_API_URL={url_to_save}")
@@ -189,7 +231,13 @@ def init() -> None:
         else:
             click.echo("No API key entered. You can add one later by editing ~/.gac.env")
 
-    # Language selection
+    return True
+
+
+def _configure_language(existing_env: dict[str, str]) -> None:
+    """Run the language configuration flow."""
+    from gac.language_cli import is_rtl_text
+
     click.echo("\n")
     existing_language = existing_env.get("GAC_LANGUAGE")
 
@@ -216,7 +264,11 @@ def init() -> None:
             # Proceed with language selection
             display_names = [lang[0] for lang in Languages.LANGUAGES]
             language_selection = questionary.select(
-                "Select a language for commit messages:", choices=display_names, use_shortcuts=True, use_arrow_keys=True
+                "Select a language for commit messages:",
+                choices=display_names,
+                use_shortcuts=True,
+                use_arrow_keys=True,
+                use_jk_keys=False,
             ).ask()
 
             if not language_selection:
@@ -237,10 +289,30 @@ def init() -> None:
                         language_value = None
                     else:
                         language_value = custom_language.strip()
+
+                        # Check if the custom language appears to be RTL
+                        if is_rtl_text(language_value):
+                            if not _should_show_rtl_warning_for_init():
+                                click.echo(
+                                    f"\nℹ️  Using RTL language {language_value} (RTL warning previously confirmed)"
+                                )
+                            else:
+                                if not _show_rtl_warning_for_init(language_value):
+                                    click.echo("Language selection cancelled. Keeping existing language.")
+                                    language_value = None
                 else:
                     # Find the English name for the selected language
                     language_value = next(lang[1] for lang in Languages.LANGUAGES if lang[0] == language_selection)
 
+                    # Check if predefined language is RTL
+                    if is_rtl_text(language_value):
+                        if not _should_show_rtl_warning_for_init():
+                            click.echo(f"\nℹ️  Using RTL language {language_value} (RTL warning previously confirmed)")
+                        else:
+                            if not _show_rtl_warning_for_init(language_value):
+                                click.echo("Language selection cancelled. Keeping existing language.")
+                                language_value = None
+
                 if language_value:
                     # Ask about prefix translation
                     prefix_choice = questionary.select(
@@ -266,7 +338,11 @@ def init() -> None:
         # No existing language - proceed with normal flow
         display_names = [lang[0] for lang in Languages.LANGUAGES]
         language_selection = questionary.select(
-            "Select a language for commit messages:", choices=display_names, use_shortcuts=True, use_arrow_keys=True
+            "Select a language for commit messages:",
+            choices=display_names,
+            use_shortcuts=True,
+            use_arrow_keys=True,
+            use_jk_keys=False,
         ).ask()
 
         if not language_selection:
@@ -287,10 +363,28 @@ def init() -> None:
                     language_value = None
                 else:
                     language_value = custom_language.strip()
+
+                    # Check if the custom language appears to be RTL
+                    if is_rtl_text(language_value):
+                        if not _should_show_rtl_warning_for_init():
+                            click.echo(f"\nℹ️  Using RTL language {language_value} (RTL warning previously confirmed)")
+                        else:
+                            if not _show_rtl_warning_for_init(language_value):
+                                click.echo("Language selection cancelled. Using English (default).")
+                                language_value = None
             else:
                 # Find the English name for the selected language
                 language_value = next(lang[1] for lang in Languages.LANGUAGES if lang[0] == language_selection)
 
+                # Check if predefined language is RTL
+                if is_rtl_text(language_value):
+                    if not _should_show_rtl_warning_for_init():
+                        click.echo(f"\nℹ️  Using RTL language {language_value} (RTL warning previously confirmed)")
+                    else:
+                        if not _show_rtl_warning_for_init(language_value):
+                            click.echo("Language selection cancelled. Using English (default).")
+                            language_value = None
+
             if language_value:
                 # Ask about prefix translation
                 prefix_choice = questionary.select(
@@ -313,4 +407,29 @@ def init() -> None:
                 click.echo(f"Set GAC_LANGUAGE={language_value}")
                 click.echo(f"Set GAC_TRANSLATE_PREFIXES={'true' if translate_prefixes else 'false'}")
 
+    return
+
+
+@click.command()
+def init() -> None:
+    """Interactively set up $HOME/.gac.env for gac."""
+    click.echo("Welcome to gac initialization!\n")
+
+    existing_env = _load_existing_env()
+    if not _configure_model(existing_env):
+        return
+    _configure_language(existing_env)
+
     click.echo(f"\ngac environment setup complete. You can edit {GAC_ENV_PATH} to update values later.")
+
+
+@click.command()
+def model() -> None:
+    """Interactively update provider/model/API key without language prompts."""
+    click.echo("Welcome to gac model configuration!\n")
+
+    existing_env = _load_existing_env()
+    if not _configure_model(existing_env):
+        return
+
+    click.echo(f"\nModel configuration complete. You can edit {GAC_ENV_PATH} to update values later.")

gac-2.4.1/src/gac/language_cli.py

@@ -0,0 +1,250 @@
+"""CLI for selecting commit message language interactively."""
+
+import os
+import unicodedata
+from pathlib import Path
+
+import click
+import questionary
+from dotenv import load_dotenv, set_key
+
+from gac.constants import Languages
+
+GAC_ENV_PATH = Path.home() / ".gac.env"
+
+
+def should_show_rtl_warning() -> bool:
+    """Check if RTL warning should be shown based on saved preference.
+
+    Returns:
+        True if warning should be shown, False if user previously confirmed
+    """
+    # Load the current config to check RTL confirmation
+    if GAC_ENV_PATH.exists():
+        load_dotenv(GAC_ENV_PATH)
+        rtl_confirmed = os.getenv("GAC_RTL_CONFIRMED", "false").lower() in ("true", "1", "yes", "on")
+        return not rtl_confirmed
+    return True  # Show warning if no config exists
+
+
+def is_rtl_text(text: str) -> bool:
+    """Detect if text contains RTL characters or is a known RTL language.
+
+    Args:
+        text: Text to analyze
+
+    Returns:
+        True if text contains RTL script characters or is RTL language
+    """
+    # Known RTL language names (case insensitive)
+    rtl_languages = {
+        "arabic",
+        "ar",
+        "العربية",
+        "hebrew",
+        "he",
+        "עברית",
+        "persian",
+        "farsi",
+        "fa",
+        "urdu",
+        "ur",
+        "اردو",
+        "pashto",
+        "ps",
+        "kurdish",
+        "ku",
+        "کوردی",
+        "yiddish",
+        "yi",
+        "ייִדיש",
+    }
+
+    # Check if it's a known RTL language name or code (case insensitive)
+    if text.lower().strip() in rtl_languages:
+        return True
+
+    rtl_scripts = {"Arabic", "Hebrew", "Thaana", "Nko", "Syriac", "Mandeic", "Samaritan", "Mongolian", "Phags-Pa"}
+
+    for char in text:
+        if unicodedata.name(char, "").startswith(("ARABIC", "HEBREW")):
+            return True
+        script = unicodedata.name(char, "").split()[0] if unicodedata.name(char, "") else ""
+        if script in rtl_scripts:
+            return True
+    return False
+
+
+def center_text(text: str, width: int = 80) -> str:
+    """Center text within specified width, handling display width properly.
+
+    Args:
+        text: Text to center
+        width: Terminal width to center within (default 80)
+
+    Returns:
+        Centered text with proper padding
+    """
+    import unicodedata
+
+    def get_display_width(s: str) -> int:
+        """Get the display width of a string, accounting for wide characters."""
+        width = 0
+        for char in s:
+            # East Asian characters are typically 2 columns wide
+            if unicodedata.east_asian_width(char) in ("W", "F"):
+                width += 2
+            else:
+                width += 1
+        return width
+
+    # Handle multi-line text
+    lines = text.split("\n")
+    centered_lines = []
+
+    for line in lines:
+        # Strip existing whitespace to avoid double padding
+        stripped_line = line.strip()
+        if stripped_line:
+            # Calculate padding using display width for accurate centering
+            display_width = get_display_width(stripped_line)
+            padding = max(0, (width - display_width) // 2)
+            centered_line = " " * padding + stripped_line
+            centered_lines.append(centered_line)
+        else:
+            centered_lines.append("")
+
+    return "\n".join(centered_lines)
+
+
+def get_terminal_width() -> int:
+    """Get the current terminal width.
+
+    Returns:
+        Terminal width in characters, or default if can't be determined
+    """
+    try:
+        import shutil
+
+        return shutil.get_terminal_size().columns
+    except (OSError, AttributeError):
+        return 80  # Fallback to 80 columns
+
+
+def show_rtl_warning(language_name: str) -> bool:
+    """Show RTL language warning and ask for confirmation.
+
+    Args:
+        language_name: Name of the RTL language
+
+    Returns:
+        True if user wants to proceed, False if they cancel
+    """
+    terminal_width = get_terminal_width()
+
+    # Center just the title
+    title = center_text("⚠️  RTL Language Detected", terminal_width)
+
+    click.echo()
+    click.echo(click.style(title, fg="yellow", bold=True))
+    click.echo()
+    click.echo("Right-to-left (RTL) languages may not display correctly in gac due to terminal limitations.")
+    click.echo("However, the commit messages will work fine and should be readable in Git clients")
+    click.echo("that properly support RTL text (like most web interfaces and modern tools).\n")
+
+    proceed = questionary.confirm("Do you want to proceed anyway?").ask()
+    if proceed:
+        # Remember that user has confirmed RTL acceptance
+        set_key(str(GAC_ENV_PATH), "GAC_RTL_CONFIRMED", "true")
+        click.echo("✓ RTL preference saved - you won't see this warning again")
+    return proceed if proceed is not None else False
+
+
+@click.command()
+def language() -> None:
+    """Set the language for commit messages interactively."""
+    click.echo("Select a language for your commit messages:\n")
+
+    display_names = [lang[0] for lang in Languages.LANGUAGES]
+    selection = questionary.select(
+        "Choose your language:", choices=display_names, use_shortcuts=True, use_arrow_keys=True, use_jk_keys=False
+    ).ask()
+
+    if not selection:
+        click.echo("Language selection cancelled.")
+        return
+
+    # Ensure .gac.env exists
+    if not GAC_ENV_PATH.exists():
+        GAC_ENV_PATH.touch()
+        click.echo(f"Created {GAC_ENV_PATH}")
+
+    # Handle English - set explicitly
+    if selection == "English":
+        set_key(str(GAC_ENV_PATH), "GAC_LANGUAGE", "English")
+        set_key(str(GAC_ENV_PATH), "GAC_TRANSLATE_PREFIXES", "false")
+        click.echo("✓ Set language to English")
+        click.echo("  GAC_LANGUAGE=English")
+        click.echo("  GAC_TRANSLATE_PREFIXES=false")
+        click.echo(f"\n  Configuration saved to {GAC_ENV_PATH}")
+        return
+
+    # Handle custom input
+    if selection == "Custom":
+        custom_language = questionary.text("Enter the language name (e.g., 'Spanish', 'Français', '日本語'):").ask()
+        if not custom_language or not custom_language.strip():
+            click.echo("No language entered. Cancelled.")
+            return
+        language_value = custom_language.strip()
+
+        # Check if the custom language appears to be RTL
+        if is_rtl_text(language_value):
+            if not should_show_rtl_warning():
+                click.echo(f"\nℹ️  Using RTL language {language_value} (RTL warning previously confirmed)")
+            else:
+                if not show_rtl_warning(language_value):
+                    click.echo("Language selection cancelled.")
+                    return
+
+    else:
+        # Find the English name for the selected language
+        language_value = next(lang[1] for lang in Languages.LANGUAGES if lang[0] == selection)
+
+        # Check if predefined language is RTL
+        if is_rtl_text(language_value):
+            if not should_show_rtl_warning():
+                click.echo(f"\nℹ️  Using RTL language {language_value} (RTL warning previously confirmed)")
+            else:
+                if not show_rtl_warning(language_value):
+                    click.echo("Language selection cancelled.")
+                    return
+
+    # Ask about prefix translation
+    click.echo()  # Blank line for spacing
+    prefix_choice = questionary.select(
+        "How should conventional commit prefixes be handled?",
+        choices=[
+            "Keep prefixes in English (feat:, fix:, etc.)",
+            f"Translate prefixes into {language_value}",
+        ],
+    ).ask()
+
+    if not prefix_choice:
+        click.echo("Prefix translation selection cancelled.")
+        return
+
+    translate_prefixes = prefix_choice.startswith("Translate prefixes")
+
+    # Set the language and prefix translation preference in .gac.env
+    set_key(str(GAC_ENV_PATH), "GAC_LANGUAGE", language_value)
+    set_key(str(GAC_ENV_PATH), "GAC_TRANSLATE_PREFIXES", "true" if translate_prefixes else "false")
+
+    click.echo(f"\n✓ Set language to {selection}")
+    click.echo(f"  GAC_LANGUAGE={language_value}")
+    if translate_prefixes:
+        click.echo("  GAC_TRANSLATE_PREFIXES=true")
+        click.echo("\n  Prefixes will be translated (e.g., 'corrección:' instead of 'fix:')")
+    else:
+        click.echo("  GAC_TRANSLATE_PREFIXES=false")
+        click.echo(f"\n  Prefixes will remain in English (e.g., 'fix: <{language_value} description>')")
+    click.echo(f"\n  Configuration saved to {GAC_ENV_PATH}")

{gac-2.3.0 → gac-2.4.1}/src/gac/utils.py

@@ -1,7 +1,9 @@
 """Utility functions for gac."""
 
+import locale
 import logging
 import subprocess
+import sys
 
 from rich.console import Console
 from rich.theme import Theme
@@ -65,18 +67,47 @@ def print_message(message: str, level: str = "info") -> None:
     console.print(message, style=level)
 
 
-def run_subprocess(
+def get_safe_encodings() -> list[str]:
+    """Get a list of safe encodings to try for subprocess calls, in order of preference.
+
+    Returns:
+        List of encoding strings to try, with UTF-8 first
+    """
+    encodings = ["utf-8"]
+
+    # Add locale encoding as fallback
+    locale_encoding = locale.getpreferredencoding(False)
+    if locale_encoding and locale_encoding not in encodings:
+        encodings.append(locale_encoding)
+
+    # Windows-specific fallbacks
+    if sys.platform == "win32":
+        windows_encodings = ["cp65001", "cp936", "cp1252"]  # UTF-8, GBK, Windows-1252
+        for enc in windows_encodings:
+            if enc not in encodings:
+                encodings.append(enc)
+
+    # Final fallback to system default
+    if "utf-8" not in encodings:
+        encodings.append("utf-8")
+
+    return encodings
+
+
+def run_subprocess_with_encoding(
     command: list[str],
+    encoding: str,
     silent: bool = False,
     timeout: int = 60,
     check: bool = True,
     strip_output: bool = True,
     raise_on_error: bool = True,
 ) -> str:
-    """Run a subprocess command safely and return the output.
+    """Run subprocess with a specific encoding, handling encoding errors gracefully.
 
     Args:
         command: List of command arguments
+        encoding: Specific encoding to use
         silent: If True, suppress debug logging
         timeout: Command timeout in seconds
         check: Whether to check return code (for compatibility)
@@ -91,7 +122,7 @@ def run_subprocess(
         subprocess.CalledProcessError: If the command fails and raise_on_error is True
     """
     if not silent:
-        logger.debug(f"Running command: {' '.join(command)}")
+        logger.debug(f"Running command: {' '.join(command)} (encoding: {encoding})")
 
     try:
         result = subprocess.run(
@@ -100,6 +131,8 @@ def run_subprocess(
             text=True,
             check=False,
             timeout=timeout,
+            encoding=encoding,
+            errors="replace",  # Replace problematic characters instead of crashing
         )
 
         should_raise = result.returncode != 0 and (check or raise_on_error)
@@ -123,6 +156,11 @@ def run_subprocess(
         if raise_on_error:
             raise
         return ""
+    except UnicodeError as e:
+        # This should be rare with errors="replace", but handle it just in case
+        if not silent:
+            logger.debug(f"Encoding error with {encoding}: {e}")
+        raise
     except Exception as e:
         if not silent:
             logger.debug(f"Command error: {e}")
@@ -132,6 +170,69 @@ def run_subprocess(
         return ""
 
 
+def run_subprocess(
+    command: list[str],
+    silent: bool = False,
+    timeout: int = 60,
+    check: bool = True,
+    strip_output: bool = True,
+    raise_on_error: bool = True,
+) -> str:
+    """Run a subprocess command safely and return the output, trying multiple encodings.
+
+    Args:
+        command: List of command arguments
+        silent: If True, suppress debug logging
+        timeout: Command timeout in seconds
+        check: Whether to check return code (for compatibility)
+        strip_output: Whether to strip whitespace from output
+        raise_on_error: Whether to raise an exception on error
+
+    Returns:
+        Command output as string
+
+    Raises:
+        GacError: If the command times out
+        subprocess.CalledProcessError: If the command fails and raise_on_error is True
+
+    Note:
+        Tries multiple encodings in order: utf-8, locale encoding, platform-specific fallbacks
+        This prevents UnicodeDecodeError on systems with non-UTF-8 locales (e.g., Chinese Windows)
+    """
+    encodings = get_safe_encodings()
+    last_exception = None
+
+    for encoding in encodings:
+        try:
+            return run_subprocess_with_encoding(
+                command=command,
+                encoding=encoding,
+                silent=silent,
+                timeout=timeout,
+                check=check,
+                strip_output=strip_output,
+                raise_on_error=raise_on_error,
+            )
+        except UnicodeError as e:
+            last_exception = e
+            if not silent:
+                logger.debug(f"Failed to decode with {encoding}: {e}")
+            continue
+        except (subprocess.CalledProcessError, GacError, subprocess.TimeoutExpired):
+            # These are not encoding-related errors, so don't retry with other encodings
+            raise
+
+    # If we get here, all encodings failed with UnicodeError
+    if not silent:
+        logger.error(f"Failed to decode command output with any encoding: {encodings}")
+
+    # Raise the last UnicodeError we encountered
+    if last_exception:
+        raise subprocess.CalledProcessError(1, command, "", f"Encoding error: {last_exception}") from last_exception
+    else:
+        raise subprocess.CalledProcessError(1, command, "", "All encoding attempts failed")
+
+
 def edit_commit_message_inplace(message: str) -> str | None:
     """Edit commit message in-place using rich terminal editing.
 

gac-2.3.0/src/gac/language_cli.py

@@ -1,82 +0,0 @@
-"""CLI for selecting commit message language interactively."""
-
-from pathlib import Path
-
-import click
-import questionary
-from dotenv import set_key
-
-from gac.constants import Languages
-
-GAC_ENV_PATH = Path.home() / ".gac.env"
-
-
-@click.command()
-def language() -> None:
-    """Set the language for commit messages interactively."""
-    click.echo("Select a language for your commit messages:\n")
-
-    display_names = [lang[0] for lang in Languages.LANGUAGES]
-    selection = questionary.select(
-        "Choose your language:", choices=display_names, use_shortcuts=True, use_arrow_keys=True
-    ).ask()
-
-    if not selection:
-        click.echo("Language selection cancelled.")
-        return
-
-    # Ensure .gac.env exists
-    if not GAC_ENV_PATH.exists():
-        GAC_ENV_PATH.touch()
-        click.echo(f"Created {GAC_ENV_PATH}")
-
-    # Handle English - set explicitly
-    if selection == "English":
-        set_key(str(GAC_ENV_PATH), "GAC_LANGUAGE", "English")
-        set_key(str(GAC_ENV_PATH), "GAC_TRANSLATE_PREFIXES", "false")
-        click.echo("✓ Set language to English")
-        click.echo("  GAC_LANGUAGE=English")
-        click.echo("  GAC_TRANSLATE_PREFIXES=false")
-        click.echo(f"\n  Configuration saved to {GAC_ENV_PATH}")
-        return
-
-    # Handle custom input
-    if selection == "Custom":
-        custom_language = questionary.text("Enter the language name (e.g., 'Spanish', 'Français', '日本語'):").ask()
-        if not custom_language or not custom_language.strip():
-            click.echo("No language entered. Cancelled.")
-            return
-        language_value = custom_language.strip()
-    else:
-        # Find the English name for the selected language
-        language_value = next(lang[1] for lang in Languages.LANGUAGES if lang[0] == selection)
-
-    # Ask about prefix translation
-    click.echo()  # Blank line for spacing
-    prefix_choice = questionary.select(
-        "How should conventional commit prefixes be handled?",
-        choices=[
-            "Keep prefixes in English (feat:, fix:, etc.)",
-            f"Translate prefixes into {language_value}",
-        ],
-    ).ask()
-
-    if not prefix_choice:
-        click.echo("Prefix translation selection cancelled.")
-        return
-
-    translate_prefixes = prefix_choice.startswith("Translate prefixes")
-
-    # Set the language and prefix translation preference in .gac.env
-    set_key(str(GAC_ENV_PATH), "GAC_LANGUAGE", language_value)
-    set_key(str(GAC_ENV_PATH), "GAC_TRANSLATE_PREFIXES", "true" if translate_prefixes else "false")
-
-    click.echo(f"\n✓ Set language to {selection}")
-    click.echo(f"  GAC_LANGUAGE={language_value}")
-    if translate_prefixes:
-        click.echo("  GAC_TRANSLATE_PREFIXES=true")
-        click.echo("\n  Prefixes will be translated (e.g., 'corrección:' instead of 'fix:')")
-    else:
-        click.echo("  GAC_TRANSLATE_PREFIXES=false")
-        click.echo(f"\n  Prefixes will remain in English (e.g., 'fix: <{language_value} description>')")
-    click.echo(f"\n  Configuration saved to {GAC_ENV_PATH}")