Fix tmp file creation and add test

Author: cyclotruc

src/gitingest/config.py

@@ -1,5 +1,6 @@
 """ Configuration file for the project. """
 
+import tempfile
 from pathlib import Path
 
 MAX_FILE_SIZE = 10 * 1024 * 1024  # 10 MB
@@ -8,4 +9,5 @@
 MAX_TOTAL_SIZE_BYTES = 500 * 1024 * 1024  # 500 MB
 
 OUTPUT_FILE_PATH = "digest.txt"
-TMP_BASE_PATH = Path("/tmp/gitingest")
+
+TMP_BASE_PATH = Path(tempfile.gettempdir()) / "gitingest"

src/gitingest/repository_clone.py

@@ -1,7 +1,9 @@
 """ This module contains functions for cloning a Git repository to a local path. """
 
 import asyncio
+import os
 from dataclasses import dataclass
+from pathlib import Path
 
 from gitingest.utils import async_timeout
 
@@ -61,6 +63,8 @@ async def clone_repo(config: CloneConfig) -> tuple[bytes, bytes]:
     ------
     ValueError
         If the 'url' or 'local_path' parameters are missing, or if the repository is not found.
+    OSError
+        If there is an error creating the parent directory structure.
     """
     # Extract and validate query parameters
     url: str = config.url
@@ -74,6 +78,13 @@ async def clone_repo(config: CloneConfig) -> tuple[bytes, bytes]:
     if not local_path:
         raise ValueError("The 'local_path' parameter is required.")
 
+    # Create parent directory if it doesn't exist
+    parent_dir = Path(local_path).parent
+    try:
+        os.makedirs(parent_dir, exist_ok=True)
+    except OSError as e:
+        raise OSError(f"Failed to create parent directory {parent_dir}: {e}") from e
+
     # Check if the repository exists
     if not await _check_repo_exists(url):
         raise ValueError("Repository not found, make sure it is public")

tests/test_repository_clone.py

@@ -7,6 +7,7 @@
 
 import asyncio
 import os
+from pathlib import Path
 from unittest.mock import AsyncMock, patch
 
 import pytest
@@ -362,3 +363,36 @@ async def test_clone_branch_with_slashes(tmp_path):
                 clone_config.url,
                 clone_config.local_path,
             )
+
+
+@pytest.mark.asyncio
+async def test_clone_repo_creates_parent_directory(tmp_path: Path) -> None:
+    """
+    Test that clone_repo creates parent directories if they don't exist.
+
+    Given a local path with non-existent parent directories:
+    When `clone_repo` is called,
+    Then it should create the parent directories before attempting to clone.
+    """
+    nested_path = tmp_path / "deep" / "nested" / "path" / "repo"
+    clone_config = CloneConfig(
+        url="https://github.com/user/repo",
+        local_path=str(nested_path),
+    )
+
+    with patch("gitingest.repository_clone._check_repo_exists", return_value=True):
+        with patch("gitingest.repository_clone._run_git_command", new_callable=AsyncMock) as mock_exec:
+            await clone_repo(clone_config)
+
+            # Verify parent directory was created
+            assert nested_path.parent.exists()
+
+            # Verify git clone was called with correct parameters
+            mock_exec.assert_called_once_with(
+                "git",
+                "clone",
+                "--depth=1",
+                "--single-branch",
+                clone_config.url,
+                str(nested_path),
+            )