feat(core): Truncate commit message title

This commit introduces truncation logic for commit message titles to ensure they remain within a 50-character limit.  It prevents excessively long commit messages from creating formatting issues and improves readability.  A test has also been added to verify this truncation behavior.
Affected files:
- M smart_git_commit.py
- M tests/test_smart_git_commit.py

Author: CripterHack

smart_git_commit.py

@@ -421,11 +421,19 @@ def generate_commit_message(self) -> str:
         scope = "-".join(sorted(components)[:2]) if components else "general"
 
         # Create the subject line (first line of commit)
-        subject = f"{self.commit_type.value}({scope}): {self.name}"
-        if len(subject) > 50:
-            # Truncate if too long
-            subject = subject[:47] + "..."
-            
+        # Just limit to 50 characters for GitHub compatibility
+        max_subject_length = 50
+        
+        # Start with type and scope
+        prefix = f"{self.commit_type.value}({scope}): "
+        available_chars = max_subject_length - len(prefix)
+        
+        # Ensure the name is no longer than available chars
+        name = self.name if len(self.name) <= available_chars else self.name[:available_chars]
+        
+        # Combine to form subject
+        subject = f"{prefix}{name}"
+        
         # Create the body with file list
         body = self.description if self.description else f"Update {self.file_count} files in {scope}"
 
@@ -2230,6 +2238,7 @@ def _generate_ai_commit_message(self, group: CommitGroup) -> str:
         Use present tense (e.g., "Add feature" not "Added feature").
         Focus on technical details rather than trivial changes.
         Do not list the filenames again.
+        Keep the first sentence brief as it will be used in the subject line.
         
         Description:
         """

tests/test_smart_git_commit.py

@@ -157,6 +157,27 @@ def test_commit_message_generation(self):
         self.assertIn("Affected files:", message)
         self.assertIn("M app/main.py", message)
         self.assertIn("+ app/new_feature.py", message)
+    
+    def test_commit_message_title_truncation(self):
+        """Test that long commit message titles are properly limited to 50 characters."""
+        # Create a commit group with a very long name
+        very_long_name = "This is an extremely long commit title that exceeds the 50 character limit"
+        group = CommitGroup(name=very_long_name, commit_type=CommitType.FEAT)
+        group.add_change(GitChange(status="M", filename="app/main.py"))
+        
+        message = group.generate_commit_message()
+        
+        # Get the first line (subject)
+        subject = message.split('\n')[0]
+        
+        # Check that the subject is within the length limit
+        self.assertLessEqual(len(subject), 50)
+        
+        # The type and scope should be preserved
+        self.assertTrue(subject.startswith('feat(app):'))
+        
+        # No "Full title:" text should appear in the body
+        self.assertNotIn("Full title:", message)
 
 
 class TestMockGitRepository: