fix: properly use GitPython subcommands

Author: NicolasIRAGNE

src/gitingest/clone.py

@@ -12,7 +12,6 @@
     _add_token_to_url,
     check_repo_exists,
     checkout_partial_clone,
-    create_git_auth_header,
     create_git_repo,
     ensure_git_installed,
     git_auth_context,
@@ -101,19 +100,19 @@ async def clone_repo(config: CloneConfig, *, token: str | None = None) -> None:
             "no_checkout": True,
             "depth": 1,
         }
-        
+
         if partial_clone:
             # GitPython doesn't directly support --filter and --sparse in clone
             # We'll need to use git.Git() for the initial clone with these options
             git_cmd = git.Git()
-            cmd_args = ["clone", "--single-branch", "--no-checkout", "--depth=1"]
+            cmd_args = ["--single-branch", "--no-checkout", "--depth=1"]
             if partial_clone:
                 cmd_args.extend(["--filter=blob:none", "--sparse"])
             cmd_args.extend([clone_url, local_path])
-            git_cmd.execute(cmd_args)
+            git_cmd.clone(*cmd_args)
         else:
             git.Repo.clone_from(clone_url, local_path, **clone_kwargs)
-            
+
         logger.info("Git clone completed successfully")
     except git.GitCommandError as exc:
         msg = f"Git clone failed: {exc}"
@@ -128,7 +127,7 @@ async def clone_repo(config: CloneConfig, *, token: str | None = None) -> None:
     # Create repo object and perform operations
     try:
         repo = create_git_repo(local_path, url, token)
-        
+
         # Ensure the commit is locally available
         logger.debug("Fetching specific commit", extra={"commit": commit})
         repo.git.fetch("--depth=1", "origin", commit)

src/gitingest/utils/git_utils.py

@@ -420,7 +420,7 @@ async def checkout_partial_clone(config: CloneConfig, token: str | None) -> None
 
     try:
         repo = create_git_repo(config.local_path, config.url, token)
-        repo.git.execute(["sparse-checkout", "set", subpath])
+        repo.git.sparse_checkout("set", subpath)
     except git.GitCommandError as exc:
         msg = f"Failed to configure sparse-checkout: {exc}"
         raise RuntimeError(msg) from exc

tests/conftest.py

@@ -236,6 +236,8 @@ def _setup_gitpython_mocks(mocker: MockerFixture) -> dict[str, MagicMock]:
     mock_git_cmd.version.return_value = "git version 2.34.1"
     mock_git_cmd.config.return_value = "true"
     mock_git_cmd.execute.return_value = f"{DEMO_COMMIT}\trefs/heads/main\n"
+    mock_git_cmd.ls_remote.return_value = f"{DEMO_COMMIT}\trefs/heads/main\n"
+    mock_git_cmd.clone.return_value = ""
 
     # Mock git.Repo class
     mock_repo = MagicMock()
@@ -245,6 +247,7 @@ def _setup_gitpython_mocks(mocker: MockerFixture) -> dict[str, MagicMock]:
     mock_repo.git.submodule = MagicMock()
     mock_repo.git.execute = MagicMock()
     mock_repo.git.config = MagicMock()
+    mock_repo.git.sparse_checkout = MagicMock()
 
     # Mock git.Repo.clone_from
     mock_clone_from = MagicMock(return_value=mock_repo)

tests/test_clone.py

@@ -49,12 +49,12 @@ async def test_clone_with_commit(repo_exists_true: AsyncMock, gitpython_mocks: d
     await clone_repo(clone_config)
 
     repo_exists_true.assert_any_call(clone_config.url, token=None)
-
+    
     # Verify GitPython calls were made
     mock_git_cmd = gitpython_mocks["git_cmd"]
     mock_repo = gitpython_mocks["repo"]
     mock_clone_from = gitpython_mocks["clone_from"]
-
+    
     # Should have called version (for ensure_git_installed)
     mock_git_cmd.version.assert_called()