git-commit-msg-ai 1.2.0__tar.gz → 1.4.0__tar.gz

{git_commit_msg_ai-1.2.0 → git_commit_msg_ai-1.4.0}/PKG-INFO

@@ -1,14 +1,16 @@
 Metadata-Version: 2.4
 Name: git-commit-msg-ai
-Version: 1.2.0
+Version: 1.4.0
 Summary: AI-powered git commit message generator following Conventional Commits
 License-Expression: MIT
-Requires-Python: >=3.9
+Requires-Python: >=3.10
 Description-Content-Type: text/markdown
 Requires-Dist: anthropic
 Provides-Extra: dev
 Requires-Dist: mypy; extra == "dev"
 Requires-Dist: ruff; extra == "dev"
+Requires-Dist: pytest; extra == "dev"
+Requires-Dist: pytest-cov; extra == "dev"
 
 # git-commit-msg-ai
 
@@ -16,7 +18,7 @@ AI-powered git commit message generator that follows the [Conventional Commits](
 
 ## Prerequisites
 
-- Python 3.9+
+- Python 3.10+
 - An Anthropic API key set as an environment variable:
 
   ```sh
@@ -52,7 +54,7 @@ The tool will:
 ```
 
 - **a** — commits immediately with the generated message
-- **e** — opens the message in your `$EDITOR`, lets you modify it, then commits
+- **e** — opens the message in your `$EDITOR` (defaults to `notepad` on Windows, `vi` on Linux/macOS), lets you modify it, then commits
 - **r** — exits without committing
 
 ## Commit message format

{git_commit_msg_ai-1.2.0 → git_commit_msg_ai-1.4.0}/README.md

@@ -4,7 +4,7 @@ AI-powered git commit message generator that follows the [Conventional Commits](
 
 ## Prerequisites
 
-- Python 3.9+
+- Python 3.10+
 - An Anthropic API key set as an environment variable:
 
   ```sh
@@ -40,7 +40,7 @@ The tool will:
 ```
 
 - **a** — commits immediately with the generated message
-- **e** — opens the message in your `$EDITOR`, lets you modify it, then commits
+- **e** — opens the message in your `$EDITOR` (defaults to `notepad` on Windows, `vi` on Linux/macOS), lets you modify it, then commits
 - **r** — exits without committing
 
 ## Commit message format

{git_commit_msg_ai-1.2.0 → git_commit_msg_ai-1.4.0}/git_commit_msg_ai/ai_client.py

@@ -3,6 +3,8 @@ from typing import Final, cast
 
 import anthropic
 
+from git_commit_msg_ai.exceptions import AIError
+
 SYSTEM_PROMPT: Final[str] = textwrap.dedent("""\
     You are a Git commit message generator. Output only the commit message, nothing else. Follow the Conventional Commits specification:
     - First line: <type>(<optional scope>)<!>: <short subject> (50 chars max); append ! before the colon if the commit introduces a breaking change
@@ -16,17 +18,25 @@ MAX_TOKENS: Final[int] = 1024
 
 
 def generate_commit_message(diff: str) -> str:
-    anthropic_client = anthropic.Anthropic()
-
-    anthropic_api_response = anthropic_client.messages.create(
-        model=MODEL,
-        max_tokens=MAX_TOKENS,
-        system=SYSTEM_PROMPT,
-        messages=[{'role': 'user', 'content': diff}],
-    )
+    try:
+        anthropic_client = anthropic.Anthropic()
+
+        anthropic_api_response = anthropic_client.messages.create(
+            model=MODEL,
+            max_tokens=MAX_TOKENS,
+            system=SYSTEM_PROMPT,
+            messages=[{'role': 'user', 'content': diff}],
+        )
+    except anthropic.AuthenticationError:
+        raise AIError('Anthropic API key is missing or invalid. Set the ANTHROPIC_API_KEY environment variable.')
+    except anthropic.RateLimitError:
+        raise AIError('Anthropic API rate limit reached. Wait a moment and try again.')
+    except anthropic.APIConnectionError:
+        raise AIError('Could not reach the Anthropic API. Check your network connection.')
+    except anthropic.APIStatusError as error:
+        raise AIError(f'Anthropic API returned an error: {error.status_code}.')
 
     anthropic_api_response_message = anthropic_api_response.content[0]
     commit_message = cast(anthropic.types.TextBlock, anthropic_api_response_message).text.strip()
 
     return commit_message
-

git_commit_msg_ai-1.4.0/git_commit_msg_ai/cli.py

@@ -0,0 +1,35 @@
+import sys
+
+from git_commit_msg_ai import ai_client, editor, git_ops
+from git_commit_msg_ai.exceptions import GitCommitAIError
+
+
+def main() -> None:
+    try:
+        diff = git_ops.get_staged_diff()
+        commit_message = ai_client.generate_commit_message(diff)
+        print(commit_message)
+
+        print()
+        user_selection = input('[a]ccept / [e]dit / [r]eject: ').strip().lower()
+
+        if user_selection == 'a':
+            git_ops.commit(commit_message)
+        elif user_selection == 'e':
+            updated_commit_message = editor.open_in_editor(commit_message)
+            git_ops.commit(updated_commit_message)
+        elif user_selection == 'r':
+            print('User rejected the generated commit message. No commit made.')
+        else:
+            print('Invalid selection.')
+            sys.exit(1)
+    except GitCommitAIError as error:
+        print(str(error))
+        sys.exit(1)
+    except (KeyboardInterrupt, EOFError):
+        print('Aborted.')
+        sys.exit(1)
+
+
+if __name__ == '__main__':  # pragma: no cover
+    main()

git_commit_msg_ai-1.4.0/git_commit_msg_ai/editor.py

@@ -0,0 +1,43 @@
+import os
+import platform
+import subprocess
+import tempfile
+
+from git_commit_msg_ai.exceptions import EditorError
+
+
+def get_default_editor() -> str:
+    current_platform = platform.system()
+    is_windows = current_platform == 'Windows'
+
+    return 'notepad' if is_windows else 'vi'
+
+
+def open_in_editor(initial_text: str) -> str:
+    try:
+        with tempfile.NamedTemporaryFile(suffix='.txt', mode='w', delete=False) as temp_file:
+            temp_file.write(initial_text)
+            temp_file_path = temp_file.name
+    except OSError:
+        raise EditorError('Could not create a temporary file.')
+
+    platform_default_editor = get_default_editor()
+    editor_command = os.environ.get('EDITOR', platform_default_editor)
+
+    try:
+        try:
+            subprocess.run([editor_command, temp_file_path], check=True)
+        except FileNotFoundError:
+            raise EditorError(f'Editor "{editor_command}" was not found. Set the EDITOR environment variable to a valid editor.')
+        except subprocess.CalledProcessError:
+            raise EditorError(f'Editor "{editor_command}" exited with an error.')
+
+        try:
+            with open(temp_file_path) as temp_file:
+                edited_text = temp_file.read().strip()
+        except OSError:
+            raise EditorError('Could not read the edited commit message from the temporary file.')
+    finally:
+        os.unlink(temp_file_path)
+
+    return edited_text

git_commit_msg_ai-1.4.0/git_commit_msg_ai/exceptions.py

@@ -0,0 +1,14 @@
+class GitCommitAIError(Exception):
+    """Base exception for the application."""
+
+
+class GitError(GitCommitAIError):
+    """Raised when a git operation fails."""
+
+
+class AIError(GitCommitAIError):
+    """Raised when the Anthropic API call fails."""
+
+
+class EditorError(GitCommitAIError):
+    """Raised when opening or reading from the editor fails."""

git_commit_msg_ai-1.4.0/git_commit_msg_ai/git_ops.py

@@ -0,0 +1,28 @@
+import subprocess
+
+from git_commit_msg_ai.exceptions import GitError
+
+
+def get_staged_diff() -> str:
+    try:
+        staged_diff = subprocess.check_output(['git', 'diff', '--cached']).decode('utf-8')
+    except FileNotFoundError:
+        raise GitError('git is not installed or not on PATH.')
+    except subprocess.CalledProcessError:
+        raise GitError('Failed to get staged diff. Are you inside a git repository?')
+    except UnicodeDecodeError:
+        raise GitError('Staged diff contains bytes that could not be decoded as UTF-8.')
+
+    if not staged_diff:
+        raise GitError('No staged changes found. Stage files with git add before running.')
+
+    return staged_diff
+
+
+def commit(message: str) -> None:
+    try:
+        subprocess.run(['git', 'commit', '-m', message], check=True)
+    except FileNotFoundError:
+        raise GitError('git is not installed or not on PATH.')
+    except subprocess.CalledProcessError:
+        raise GitError('git commit failed.')

{git_commit_msg_ai-1.2.0 → git_commit_msg_ai-1.4.0}/git_commit_msg_ai.egg-info/PKG-INFO

@@ -1,14 +1,16 @@
 Metadata-Version: 2.4
 Name: git-commit-msg-ai
-Version: 1.2.0
+Version: 1.4.0
 Summary: AI-powered git commit message generator following Conventional Commits
 License-Expression: MIT
-Requires-Python: >=3.9
+Requires-Python: >=3.10
 Description-Content-Type: text/markdown
 Requires-Dist: anthropic
 Provides-Extra: dev
 Requires-Dist: mypy; extra == "dev"
 Requires-Dist: ruff; extra == "dev"
+Requires-Dist: pytest; extra == "dev"
+Requires-Dist: pytest-cov; extra == "dev"
 
 # git-commit-msg-ai
 
@@ -16,7 +18,7 @@ AI-powered git commit message generator that follows the [Conventional Commits](
 
 ## Prerequisites
 
-- Python 3.9+
+- Python 3.10+
 - An Anthropic API key set as an environment variable:
 
   ```sh
@@ -52,7 +54,7 @@ The tool will:
 ```
 
 - **a** — commits immediately with the generated message
-- **e** — opens the message in your `$EDITOR`, lets you modify it, then commits
+- **e** — opens the message in your `$EDITOR` (defaults to `notepad` on Windows, `vi` on Linux/macOS), lets you modify it, then commits
 - **r** — exits without committing
 
 ## Commit message format

{git_commit_msg_ai-1.2.0 → git_commit_msg_ai-1.4.0}/git_commit_msg_ai.egg-info/SOURCES.txt

@@ -4,10 +4,16 @@ git_commit_msg_ai/__init__.py
 git_commit_msg_ai/ai_client.py
 git_commit_msg_ai/cli.py
 git_commit_msg_ai/editor.py
+git_commit_msg_ai/exceptions.py
 git_commit_msg_ai/git_ops.py
 git_commit_msg_ai.egg-info/PKG-INFO
 git_commit_msg_ai.egg-info/SOURCES.txt
 git_commit_msg_ai.egg-info/dependency_links.txt
 git_commit_msg_ai.egg-info/entry_points.txt
 git_commit_msg_ai.egg-info/requires.txt
-git_commit_msg_ai.egg-info/top_level.txt
+git_commit_msg_ai.egg-info/top_level.txt
+tests/test_ai_client.py
+tests/test_cli.py
+tests/test_editor.py
+tests/test_exceptions.py
+tests/test_git_ops.py

{git_commit_msg_ai-1.2.0 → git_commit_msg_ai-1.4.0}/git_commit_msg_ai.egg-info/requires.txt

@@ -3,3 +3,5 @@ anthropic
 [dev]
 mypy
 ruff
+pytest
+pytest-cov

{git_commit_msg_ai-1.2.0 → git_commit_msg_ai-1.4.0}/pyproject.toml

@@ -4,30 +4,40 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "git-commit-msg-ai"
-version = "1.2.0"
+version = "1.4.0"
 description = "AI-powered git commit message generator following Conventional Commits"
 readme = "README.md"
 license = "MIT"
-requires-python = ">=3.9"
+requires-python = ">=3.10"
 dependencies = ["anthropic"]
 
 [project.scripts]
 git-commit-msg-ai = "git_commit_msg_ai.cli:main"
 
 [project.optional-dependencies]
-dev = ["mypy", "ruff"]
+dev = ["mypy", "ruff", "pytest", "pytest-cov"]
 
 [tool.ruff]
-target-version = "py39"
+target-version = "py310"
 line-length = 320
 
 [tool.ruff.lint]
 select = ["E", "F", "I"]
 
 [tool.mypy]
-python_version = "3.9"
+python_version = "3.10"
 strict = true
 
+[tool.pytest.ini_options]
+testpaths = ["tests"]
+addopts = "--cov --cov-report=term-missing"
+
+[tool.coverage.run]
+source = ["git_commit_msg_ai"]
+
+[tool.coverage.report]
+show_missing = true
+
 [tool.setuptools.packages.find]
 where = ["."]
 include = ["git_commit_msg_ai*"]

git_commit_msg_ai-1.4.0/tests/test_ai_client.py

@@ -0,0 +1,74 @@
+from unittest.mock import MagicMock, patch
+
+import anthropic
+import pytest
+
+from git_commit_msg_ai.ai_client import generate_commit_message
+from git_commit_msg_ai.exceptions import AIError
+
+
+def _make_api_response(text: str) -> MagicMock:
+    mock_text_block = MagicMock(spec=anthropic.types.TextBlock)
+    mock_text_block.text = text
+
+    mock_response = MagicMock(spec=anthropic.types.Message)
+    mock_response.content = [mock_text_block]
+
+    return mock_response
+
+
+def _make_status_error(exception_class: type[anthropic.APIStatusError], status_code: int) -> anthropic.APIStatusError:
+    mock_response = MagicMock()
+    mock_response.status_code = status_code
+
+    return exception_class('error', response=mock_response, body={})
+
+
+class TestGenerateCommitMessage:
+    def test_returns_stripped_commit_message(self) -> None:
+        with patch('git_commit_msg_ai.ai_client.anthropic.Anthropic') as mock_anthropic_class:
+            mock_client = MagicMock()
+            mock_client.messages.create.return_value = _make_api_response('  feat: add feature  ')
+            mock_anthropic_class.return_value = mock_client
+
+            result = generate_commit_message('diff content')
+
+        assert result == 'feat: add feature'
+
+    def test_raises_ai_error_on_authentication_error(self) -> None:
+        with patch('git_commit_msg_ai.ai_client.anthropic.Anthropic') as mock_anthropic_class:
+            mock_client = MagicMock()
+            mock_client.messages.create.side_effect = _make_status_error(anthropic.AuthenticationError, 401)
+            mock_anthropic_class.return_value = mock_client
+
+            with pytest.raises(AIError, match='ANTHROPIC_API_KEY'):
+                generate_commit_message('diff content')
+
+    def test_raises_ai_error_on_rate_limit_error(self) -> None:
+        with patch('git_commit_msg_ai.ai_client.anthropic.Anthropic') as mock_anthropic_class:
+            mock_client = MagicMock()
+            mock_client.messages.create.side_effect = _make_status_error(anthropic.RateLimitError, 429)
+            mock_anthropic_class.return_value = mock_client
+
+            with pytest.raises(AIError, match='rate limit'):
+                generate_commit_message('diff content')
+
+    def test_raises_ai_error_on_api_connection_error(self) -> None:
+        mock_request = MagicMock()
+
+        with patch('git_commit_msg_ai.ai_client.anthropic.Anthropic') as mock_anthropic_class:
+            mock_client = MagicMock()
+            mock_client.messages.create.side_effect = anthropic.APIConnectionError(request=mock_request)
+            mock_anthropic_class.return_value = mock_client
+
+            with pytest.raises(AIError, match='network'):
+                generate_commit_message('diff content')
+
+    def test_raises_ai_error_on_api_status_error_with_status_code(self) -> None:
+        with patch('git_commit_msg_ai.ai_client.anthropic.Anthropic') as mock_anthropic_class:
+            mock_client = MagicMock()
+            mock_client.messages.create.side_effect = _make_status_error(anthropic.APIStatusError, 500)
+            mock_anthropic_class.return_value = mock_client
+
+            with pytest.raises(AIError, match='500'):
+                generate_commit_message('diff content')

git_commit_msg_ai-1.4.0/tests/test_cli.py

@@ -0,0 +1,139 @@
+from unittest.mock import MagicMock, patch
+
+import pytest
+
+from git_commit_msg_ai.cli import main
+from git_commit_msg_ai.exceptions import AIError, GitError
+
+COMMIT_MESSAGE = 'feat: add feature'
+EDITED_COMMIT_MESSAGE = 'edited commit message'
+
+
+class TestMain:
+    def _run_with_input(self, user_input: str, commit_message: str = COMMIT_MESSAGE) -> tuple[MagicMock, MagicMock, MagicMock]:
+        mock_git_ops = MagicMock()
+        mock_git_ops.get_staged_diff.return_value = 'diff content'
+
+        mock_ai_client = MagicMock()
+        mock_ai_client.generate_commit_message.return_value = commit_message
+
+        mock_editor = MagicMock()
+        mock_editor.open_in_editor.return_value = EDITED_COMMIT_MESSAGE
+
+        with patch('git_commit_msg_ai.cli.git_ops', mock_git_ops):
+            with patch('git_commit_msg_ai.cli.ai_client', mock_ai_client):
+                with patch('git_commit_msg_ai.cli.editor', mock_editor):
+                    with patch('builtins.input', return_value=user_input):
+                        main()
+
+        return mock_git_ops, mock_ai_client, mock_editor
+
+    def test_accept_calls_commit_with_generated_message(self) -> None:
+        mock_git_ops, _, _ = self._run_with_input('a', COMMIT_MESSAGE)
+
+        mock_git_ops.commit.assert_called_once_with(COMMIT_MESSAGE)
+
+    def test_edit_opens_editor_then_commits_edited_message(self) -> None:
+        mock_git_ops, _, mock_editor = self._run_with_input('e', COMMIT_MESSAGE)
+
+        mock_editor.open_in_editor.assert_called_once_with(COMMIT_MESSAGE)
+        mock_git_ops.commit.assert_called_once_with(EDITED_COMMIT_MESSAGE)
+
+    def test_reject_does_not_commit(self) -> None:
+        mock_git_ops, _, _ = self._run_with_input('r')
+
+        mock_git_ops.commit.assert_not_called()
+
+    def test_invalid_input_exits_with_code_1(self) -> None:
+        mock_git_ops = MagicMock()
+        mock_git_ops.get_staged_diff.return_value = 'diff content'
+
+        mock_ai_client = MagicMock()
+        mock_ai_client.generate_commit_message.return_value = 'msg'
+
+        with patch('git_commit_msg_ai.cli.git_ops', mock_git_ops):
+            with patch('git_commit_msg_ai.cli.ai_client', mock_ai_client):
+                with patch('git_commit_msg_ai.cli.editor'):
+                    with patch('builtins.input', return_value='x'):
+                        with pytest.raises(SystemExit) as exc_info:
+                            main()
+
+        assert exc_info.value.code == 1
+
+    def test_git_error_from_get_staged_diff_exits_with_message(self, capsys: pytest.CaptureFixture[str]) -> None:
+        mock_git_ops = MagicMock()
+        mock_git_ops.get_staged_diff.side_effect = GitError('staged error')
+
+        with patch('git_commit_msg_ai.cli.git_ops', mock_git_ops):
+            with pytest.raises(SystemExit) as exc_info:
+                main()
+
+        assert exc_info.value.code == 1
+        assert 'staged error' in capsys.readouterr().out
+
+    def test_ai_error_from_generate_commit_message_exits_with_message(self, capsys: pytest.CaptureFixture[str]) -> None:
+        mock_git_ops = MagicMock()
+        mock_git_ops.get_staged_diff.return_value = 'diff content'
+
+        mock_ai_client = MagicMock()
+        mock_ai_client.generate_commit_message.side_effect = AIError('ai error')
+
+        with patch('git_commit_msg_ai.cli.git_ops', mock_git_ops):
+            with patch('git_commit_msg_ai.cli.ai_client', mock_ai_client):
+                with pytest.raises(SystemExit) as exc_info:
+                    main()
+
+        assert exc_info.value.code == 1
+        assert 'ai error' in capsys.readouterr().out
+
+    def test_git_error_from_commit_exits_with_message(self, capsys: pytest.CaptureFixture[str]) -> None:
+        mock_git_ops = MagicMock()
+        mock_git_ops.get_staged_diff.return_value = 'diff content'
+        mock_git_ops.commit.side_effect = GitError('commit error')
+
+        mock_ai_client = MagicMock()
+        mock_ai_client.generate_commit_message.return_value = 'msg'
+
+        with patch('git_commit_msg_ai.cli.git_ops', mock_git_ops):
+            with patch('git_commit_msg_ai.cli.ai_client', mock_ai_client):
+                with patch('git_commit_msg_ai.cli.editor'):
+                    with patch('builtins.input', return_value='a'):
+                        with pytest.raises(SystemExit) as exc_info:
+                            main()
+
+        assert exc_info.value.code == 1
+        assert 'commit error' in capsys.readouterr().out
+
+    def test_keyboard_interrupt_exits_with_aborted_message(self, capsys: pytest.CaptureFixture[str]) -> None:
+        mock_git_ops = MagicMock()
+        mock_git_ops.get_staged_diff.return_value = 'diff content'
+
+        mock_ai_client = MagicMock()
+        mock_ai_client.generate_commit_message.return_value = 'msg'
+
+        with patch('git_commit_msg_ai.cli.git_ops', mock_git_ops):
+            with patch('git_commit_msg_ai.cli.ai_client', mock_ai_client):
+                with patch('git_commit_msg_ai.cli.editor'):
+                    with patch('builtins.input', side_effect=KeyboardInterrupt):
+                        with pytest.raises(SystemExit) as exc_info:
+                            main()
+
+        assert exc_info.value.code == 1
+        assert 'Aborted.' in capsys.readouterr().out
+
+    def test_eof_error_exits_with_aborted_message(self, capsys: pytest.CaptureFixture[str]) -> None:
+        mock_git_ops = MagicMock()
+        mock_git_ops.get_staged_diff.return_value = 'diff content'
+
+        mock_ai_client = MagicMock()
+        mock_ai_client.generate_commit_message.return_value = 'msg'
+
+        with patch('git_commit_msg_ai.cli.git_ops', mock_git_ops):
+            with patch('git_commit_msg_ai.cli.ai_client', mock_ai_client):
+                with patch('git_commit_msg_ai.cli.editor'):
+                    with patch('builtins.input', side_effect=EOFError):
+                        with pytest.raises(SystemExit) as exc_info:
+                            main()
+
+        assert exc_info.value.code == 1
+        assert 'Aborted.' in capsys.readouterr().out

git_commit_msg_ai-1.4.0/tests/test_editor.py

@@ -0,0 +1,130 @@
+import os
+import subprocess
+import tempfile
+from contextlib import ExitStack
+from unittest.mock import MagicMock, patch
+
+import pytest
+
+from git_commit_msg_ai.editor import get_default_editor, open_in_editor
+from git_commit_msg_ai.exceptions import EditorError
+
+TEMP_FILE_PATH = os.path.join(tempfile.gettempdir(), 'tmpfile.txt')
+
+
+def _make_named_temporary_file_mock() -> MagicMock:
+    mock_temp_file = MagicMock()
+    mock_temp_file.name = TEMP_FILE_PATH
+    mock_temp_file.__enter__ = MagicMock(return_value=mock_temp_file)
+    mock_temp_file.__exit__ = MagicMock(return_value=False)
+
+    return mock_temp_file
+
+
+def _make_open_mock(text: str) -> MagicMock:
+    mock_file = MagicMock()
+    mock_file.read.return_value = text
+    mock_file.__enter__ = MagicMock(return_value=mock_file)
+    mock_file.__exit__ = MagicMock(return_value=False)
+
+    return mock_file
+
+
+class TestOpenInEditor:
+    def test_returns_stripped_edited_text(self) -> None:
+        with ExitStack() as stack:
+            stack.enter_context(patch('git_commit_msg_ai.editor.tempfile.NamedTemporaryFile', return_value=_make_named_temporary_file_mock()))
+            stack.enter_context(patch('git_commit_msg_ai.editor.subprocess.run'))
+            stack.enter_context(patch('git_commit_msg_ai.editor.open', return_value=_make_open_mock('  edited text  ')))
+            stack.enter_context(patch('git_commit_msg_ai.editor.os.unlink'))
+
+            result = open_in_editor('initial text')
+
+        assert result == 'edited text'
+
+    def test_uses_editor_env_var(self) -> None:
+        with ExitStack() as stack:
+            stack.enter_context(patch('git_commit_msg_ai.editor.tempfile.NamedTemporaryFile', return_value=_make_named_temporary_file_mock()))
+            mock_run = stack.enter_context(patch('git_commit_msg_ai.editor.subprocess.run'))
+            stack.enter_context(patch('git_commit_msg_ai.editor.open', return_value=_make_open_mock('text')))
+            stack.enter_context(patch('git_commit_msg_ai.editor.os.unlink'))
+            stack.enter_context(patch('git_commit_msg_ai.editor.os.environ.get', return_value='vim'))
+
+            open_in_editor('initial text')
+
+        mock_run.assert_called_once_with(['vim', TEMP_FILE_PATH], check=True)
+
+    def test_falls_back_to_platform_default_when_editor_not_set(self) -> None:
+        sentinel_editor = 'test-sentinel-editor'
+
+        with ExitStack() as stack:
+            stack.enter_context(patch('git_commit_msg_ai.editor.tempfile.NamedTemporaryFile', return_value=_make_named_temporary_file_mock()))
+            mock_run = stack.enter_context(patch('git_commit_msg_ai.editor.subprocess.run'))
+            stack.enter_context(patch('git_commit_msg_ai.editor.open', return_value=_make_open_mock('text')))
+            stack.enter_context(patch('git_commit_msg_ai.editor.os.unlink'))
+            stack.enter_context(patch('git_commit_msg_ai.editor.get_default_editor', return_value=sentinel_editor))
+            stack.enter_context(patch.dict('os.environ', {}, clear=True))
+
+            open_in_editor('initial text')
+
+        mock_run.assert_called_once_with([sentinel_editor, TEMP_FILE_PATH], check=True)
+
+    def test_raises_editor_error_on_temp_file_creation_failure(self) -> None:
+        with ExitStack() as stack:
+            stack.enter_context(patch('git_commit_msg_ai.editor.tempfile.NamedTemporaryFile', side_effect=OSError))
+            mock_unlink = stack.enter_context(patch('git_commit_msg_ai.editor.os.unlink'))
+
+            with pytest.raises(EditorError, match='temporary file'):
+                open_in_editor('initial text')
+
+        mock_unlink.assert_not_called()
+
+    def test_raises_editor_error_when_editor_not_found(self) -> None:
+        with ExitStack() as stack:
+            stack.enter_context(patch('git_commit_msg_ai.editor.tempfile.NamedTemporaryFile', return_value=_make_named_temporary_file_mock()))
+            stack.enter_context(patch('git_commit_msg_ai.editor.subprocess.run', side_effect=FileNotFoundError))
+            mock_unlink = stack.enter_context(patch('git_commit_msg_ai.editor.os.unlink'))
+            stack.enter_context(patch('git_commit_msg_ai.editor.os.environ.get', return_value='vim'))
+
+            with pytest.raises(EditorError, match='was not found'):
+                open_in_editor('initial text')
+
+        mock_unlink.assert_called_once_with(TEMP_FILE_PATH)
+
+    def test_raises_editor_error_when_editor_exits_with_error(self) -> None:
+        with ExitStack() as stack:
+            stack.enter_context(patch('git_commit_msg_ai.editor.tempfile.NamedTemporaryFile', return_value=_make_named_temporary_file_mock()))
+            stack.enter_context(patch('git_commit_msg_ai.editor.subprocess.run', side_effect=subprocess.CalledProcessError(1, 'vim')))
+            mock_unlink = stack.enter_context(patch('git_commit_msg_ai.editor.os.unlink'))
+            stack.enter_context(patch('git_commit_msg_ai.editor.os.environ.get', return_value='vim'))
+
+            with pytest.raises(EditorError, match='exited with an error'):
+                open_in_editor('initial text')
+
+        mock_unlink.assert_called_once_with(TEMP_FILE_PATH)
+
+    def test_raises_editor_error_when_file_read_fails(self) -> None:
+        with ExitStack() as stack:
+            stack.enter_context(patch('git_commit_msg_ai.editor.tempfile.NamedTemporaryFile', return_value=_make_named_temporary_file_mock()))
+            stack.enter_context(patch('git_commit_msg_ai.editor.subprocess.run'))
+            stack.enter_context(patch('git_commit_msg_ai.editor.open', side_effect=OSError))
+            mock_unlink = stack.enter_context(patch('git_commit_msg_ai.editor.os.unlink'))
+
+            with pytest.raises(EditorError, match='Could not read'):
+                open_in_editor('initial text')
+
+        mock_unlink.assert_called_once_with(TEMP_FILE_PATH)
+
+
+class TestGetDefaultEditor:
+    def test_returns_notepad_on_windows(self) -> None:
+        with patch('git_commit_msg_ai.editor.platform.system', return_value='Windows'):
+            result = get_default_editor()
+
+        assert result == 'notepad'
+
+    def test_returns_vi_on_non_windows(self) -> None:
+        with patch('git_commit_msg_ai.editor.platform.system', return_value='Linux'):
+            result = get_default_editor()
+
+        assert result == 'vi'

git_commit_msg_ai-1.4.0/tests/test_exceptions.py

@@ -0,0 +1,25 @@
+from git_commit_msg_ai.exceptions import AIError, EditorError, GitCommitAIError, GitError
+
+
+def test_git_error_is_subclass_of_base() -> None:
+    assert issubclass(GitError, GitCommitAIError)
+
+
+def test_ai_error_is_subclass_of_base() -> None:
+    assert issubclass(AIError, GitCommitAIError)
+
+
+def test_editor_error_is_subclass_of_base() -> None:
+    assert issubclass(EditorError, GitCommitAIError)
+
+
+def test_git_error_carries_message() -> None:
+    assert str(GitError('git failed')) == 'git failed'
+
+
+def test_ai_error_carries_message() -> None:
+    assert str(AIError('api failed')) == 'api failed'
+
+
+def test_editor_error_carries_message() -> None:
+    assert str(EditorError('editor failed')) == 'editor failed'

git_commit_msg_ai-1.4.0/tests/test_git_ops.py

@@ -0,0 +1,70 @@
+import subprocess
+from unittest.mock import MagicMock, patch
+
+import pytest
+
+from git_commit_msg_ai.exceptions import GitError
+from git_commit_msg_ai.git_ops import commit, get_staged_diff
+
+
+class TestGetStagedDiff:
+    def test_returns_decoded_diff(self) -> None:
+        with patch('git_commit_msg_ai.git_ops.subprocess.check_output') as mock_check_output:
+            mock_check_output.return_value = b'diff content'
+
+            result = get_staged_diff()
+
+        assert result == 'diff content'
+
+    def test_raises_git_error_when_git_not_found(self) -> None:
+        with patch('git_commit_msg_ai.git_ops.subprocess.check_output') as mock_check_output:
+            mock_check_output.side_effect = FileNotFoundError
+
+            with pytest.raises(GitError, match='not installed'):
+                get_staged_diff()
+
+    def test_raises_git_error_on_called_process_error(self) -> None:
+        with patch('git_commit_msg_ai.git_ops.subprocess.check_output') as mock_check_output:
+            mock_check_output.side_effect = subprocess.CalledProcessError(1, 'git')
+
+            with pytest.raises(GitError, match='git repository'):
+                get_staged_diff()
+
+    def test_raises_git_error_on_unicode_decode_error(self) -> None:
+        mock_bytes = MagicMock()
+        mock_bytes.decode.side_effect = UnicodeDecodeError('utf-8', b'', 0, 1, 'reason')
+
+        with patch('git_commit_msg_ai.git_ops.subprocess.check_output') as mock_check_output:
+            mock_check_output.return_value = mock_bytes
+
+            with pytest.raises(GitError, match='UTF-8'):
+                get_staged_diff()
+
+    def test_raises_git_error_when_diff_is_empty(self) -> None:
+        with patch('git_commit_msg_ai.git_ops.subprocess.check_output') as mock_check_output:
+            mock_check_output.return_value = b''
+
+            with pytest.raises(GitError, match='No staged changes'):
+                get_staged_diff()
+
+
+class TestCommit:
+    def test_calls_subprocess_run_with_correct_args(self) -> None:
+        with patch('git_commit_msg_ai.git_ops.subprocess.run') as mock_run:
+            commit('my message')
+
+        mock_run.assert_called_once_with(['git', 'commit', '-m', 'my message'], check=True)
+
+    def test_raises_git_error_when_git_not_found(self) -> None:
+        with patch('git_commit_msg_ai.git_ops.subprocess.run') as mock_run:
+            mock_run.side_effect = FileNotFoundError
+
+            with pytest.raises(GitError, match='not installed'):
+                commit('my message')
+
+    def test_raises_git_error_on_called_process_error(self) -> None:
+        with patch('git_commit_msg_ai.git_ops.subprocess.run') as mock_run:
+            mock_run.side_effect = subprocess.CalledProcessError(1, 'git')
+
+            with pytest.raises(GitError, match='git commit failed'):
+                commit('my message')

git_commit_msg_ai-1.2.0/git_commit_msg_ai/cli.py

@@ -1,22 +0,0 @@
-from git_commit_msg_ai import ai_client, editor, git_ops
-
-
-def main() -> None:
-    diff = git_ops.get_staged_diff()
-    commit_message = ai_client.generate_commit_message(diff)
-    print(commit_message)
-
-    print()
-    user_selection = input('[a]ccept / [e]dit / [r]eject: ').strip().lower()
-
-    if user_selection == 'a':
-        git_ops.commit(commit_message)
-    elif user_selection == 'e':
-        updated_commit_message = editor.open_in_editor(commit_message)
-        git_ops.commit(updated_commit_message)
-    elif user_selection == 'r':
-        print('User rejected the generated commit message. No commit made.')
-
-
-if __name__ == '__main__':
-    main()

git_commit_msg_ai-1.2.0/git_commit_msg_ai/editor.py

@@ -1,19 +0,0 @@
-import os
-import subprocess
-import tempfile
-
-
-def open_in_editor(initial_text: str) -> str:
-    with tempfile.NamedTemporaryFile(suffix='.txt', mode='w', delete=False) as temp_file:
-        temp_file.write(initial_text)
-        temp_file_path = temp_file.name
-
-    editor_command = os.environ.get('EDITOR', 'notepad')
-    subprocess.run([editor_command, temp_file_path])
-
-    with open(temp_file_path) as temp_file:
-        edited_text = temp_file.read().strip()
-
-    os.unlink(temp_file_path)
-
-    return edited_text

git_commit_msg_ai-1.2.0/git_commit_msg_ai/git_ops.py

@@ -1,12 +0,0 @@
-import subprocess
-
-
-def get_staged_diff() -> str:
-    staged_diff = subprocess.check_output(['git', 'diff', '--cached']).decode()
-
-    return staged_diff
-
-
-def commit(message: str) -> None:
-    subprocess.run(['git', 'commit', '-m', message])
-