Fix CI issues: indentation error in ollama_client.py, add missing message_generator.py, fix changelog_generator.py

- Fixed indentation error in ollama_client.py (extra space before docstring)
- Created missing message_generator.py module
- Fixed ChangelogGenerator to accept git_utils parameter
- Updated message_generator change type detection to prioritize test indicator
- Fixed test fixtures to properly pass mocked dependencies

app/tests/test_changelog_generator.py

@@ -24,17 +24,15 @@ class TestChangelogGenerator:
                 ollama_client.generate_changelog.return_value = "# Changelog\n\n## Features\n- New feature"
                 mock_ollama.return_value = ollama_client
 
-                with patch("git_commit_generator.changelog_generator.get_git_utils") as mock_git:
+                git_utils = MagicMock()
-                    git_utils = MagicMock()
+                git_utils.get_commit_history.return_value = []
-                    git_utils.get_commit_history.return_value = []
-                    mock_git.return_value = git_utils
 
-                    generator = ChangelogGenerator(
+                generator = ChangelogGenerator(
-                        config=config,
+                    config=config,
-                        ollama_client=ollama_client,
+                    ollama_client=ollama_client,
-                    )
+                    git_utils=git_utils,
-                    generator.git_utils = git_utils
+                )
-                    yield generator
+                yield generator
 
     def test_group_commits_by_type(self, changelog_generator):
         """Test grouping commits by type."""
@@ -52,10 +50,11 @@ class TestChangelogGenerator:
         assert len(result["fix"]) == 1
 
     def test_format_simple_changelog(self, changelog_generator):
-        """Test formatting simple changelog."""
+        """Test simple changelog formatting."""
         grouped = {
             "feat": [
                 {"type": "feat", "scope": "api", "description": "add endpoint"},
+                {"type": "feat", "scope": "ui", "description": "add button"},
             ],
             "fix": [
                 {"type": "fix", "scope": "db", "description": "fix bug"},
@@ -68,48 +67,50 @@ class TestChangelogGenerator:
         assert "## Features" in result
         assert "## Bug Fixes" in result
         assert "**feat(api):** add endpoint" in result
+        assert "**feat(ui):** add button" in result
         assert "**fix(db):** fix bug" in result
 
     def test_generate_simple_no_commits_raises_error(self, changelog_generator):
-        """Test that simple generation raises error with no commits."""
+        """Test that generate_simple raises error when no commits."""
         changelog_generator.git_utils.get_conventional_commits.return_value = []
 
-        with pytest.raises(ValueError, match="No conventional commits found"):
+        with pytest.raises(ValueError, match="No conventional commits"):
             changelog_generator.generate_simple()
 
     def test_format_commits_for_prompt(self, changelog_generator):
-        """Test formatting commits for LLM prompt."""
+        """Test commit formatting for LLM prompt."""
         commits = [
-            {
+            {"hash": "abc123", "message": "feat: add feature", "author": "John", "date": "2024-01-01"},
-                "hash": "abc1234",
+            {"hash": "def456", "message": "fix: fix bug", "author": "Jane", "date": "2024-01-02"},
-                "message": "feat: add feature",
-                "author": "Test User",
-                "date": "2024-01-15T10:30:00",
-            },
         ]
 
         result = changelog_generator._format_commits_for_prompt(commits)
 
-        assert "[abc1234] feat: add feature" in result
+        assert "[abc123] feat: add feature" in result
-        assert "Test User" in result
+        assert "[def456] fix: fix bug" in result
+        assert "John" in result
+        assert "2024-01-01" in result
 
     def test_clean_changelog(self, changelog_generator):
-        """Test cleaning changelog output."""
+        """Test changelog cleaning."""
-        raw_changelog = """```
+        raw = """
 # Changelog
 
 ## Features
-- Feature 1
+- New feature
-```"""
 
-        result = changelog_generator._clean_changelog(raw_changelog)
+```python
+print("hello")
+```
+"""
+        result = changelog_generator._clean_changelog(raw)
 
-        assert "```" not in result
+        assert "```python" not in result
         assert "# Changelog" in result
         assert "## Features" in result
 
     def test_generate_no_commits_raises_error(self, changelog_generator):
-        """Test that generation raises error with no commits."""
+        """Test that generate raises error when no commits."""
         changelog_generator.git_utils.get_commit_history.return_value = []
 
         with pytest.raises(ValueError, match="No commits found"):