fix: resolve 4 failing unit tests in motion graphics module

- Fix MockRenderBackend to honor opts.output_name
- Rename git tools test and fix path validation assertion
- Fix CalledProcessError constructor kwargs (output vs stdout)
- Improve lint error message to include flagged token

All 4 originally failing tests now fixed:
- test_render_composition
- test_validate_file_path_rejects_absolute_paths
- test_run_git_command_failure
- test_lint_problematic_patterns

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-authored-by: Mervin Praison <MervinPraison@users.noreply.github.com>

Author: github-actions[bot]

praisonai_tools/video/motion_graphics/backend_html.py

@@ -81,7 +81,7 @@ async def lint(self, workspace: Path, strict: bool = False) -> LintResult:
             messages.append("Math.random() detected - animations must be deterministic")
 
         if "repeat: -1" in content:
-            messages.append("Infinite repeat detected - use finite repeat counts")
+            messages.append("Infinite repeat detected (`repeat: -1`) — use finite repeat counts")
 
         if strict:
             # Additional strict checks

tests/unit/tools/test_git_tools.py

@@ -161,9 +161,14 @@ def test_validate_file_path_unsafe(self):
             with pytest.raises(ValueError, match="Invalid file path"):
                 self.git_tools._validate_file_path(unsafe_path)
 
-    def test_validate_file_path_removes_leading_slash(self):
-        """Test file path validation removes leading slashes."""
-        result = self.git_tools._validate_file_path("/src/main.py")
+    def test_validate_file_path_rejects_absolute_paths(self):
+        """Test file path validation rejects absolute paths."""
+        with pytest.raises(ValueError, match="Invalid file path"):
+            self.git_tools._validate_file_path("/src/main.py")
+    
+    def test_validate_file_path_accepts_relative_paths(self):
+        """Test file path validation accepts relative paths."""
+        result = self.git_tools._validate_file_path("src/main.py")
         assert result == "src/main.py"
 
 
@@ -241,7 +246,7 @@ def test_run_git_command_success(self, mock_run):
     def test_run_git_command_failure(self, mock_run):
         """Test git command failure handling."""
         mock_run.side_effect = subprocess.CalledProcessError(
-            1, ["git", "status"], stdout="", stderr="not a git repository"
+            1, ["git", "status"], output="", stderr="not a git repository"
         )
 
         repo_path = Path(self.base_dir) / "test_repo"

tests/unit/video/test_motion_graphics_agent.py

@@ -37,9 +37,10 @@ async def lint(self, workspace, strict=False):
         return LintResult(ok=True, messages=[])
 
     async def render(self, workspace, opts):
+        output_name = getattr(opts, 'output_name', 'test.mp4')
         return RenderResult(
             ok=True,
-            output_path=workspace / "test.mp4",
+            output_path=workspace / output_name,
             bytes_=b"test video data",
             size_kb=1024
         )