raise error when commit hash is not available/provided

Author: ix-56h

src/gitingest/utils/s3_utils.py

@@ -45,7 +45,7 @@ def generate_s3_file_path(
     user_name: str,
     repo_name: str,
     branch: str | None,
-    commit: str | None,
+    commit: str,
     include_patterns: set[str] | None,
     ignore_patterns: set[str],
 ) -> str:
@@ -66,8 +66,8 @@ def generate_s3_file_path(
         Repository name.
     branch : str | None
         Branch name (if available).
-    commit : str | None
-        Commit hash (should always be available now).
+    commit : str
+        Commit hash.
     include_patterns : set[str] | None
         Set of patterns specifying which files to include.
     ignore_patterns : set[str]
@@ -98,10 +98,7 @@ def generate_s3_file_path(
 
     patterns_hash = hashlib.sha256(patterns_str.encode()).hexdigest()[:16]
 
-    # Commit should always be available now, but provide fallback just in case
-    commit_id = commit or "HEAD"
-
-    return f"ingest/{git_source}/{user_name}/{repo_name}/{branch_name}/{commit_id}/{patterns_hash}.txt"
+    return f"ingest/{git_source}/{user_name}/{repo_name}/{branch_name}/{commit}/{patterns_hash}.txt"
 
 
 def create_s3_client() -> boto_client:  # type: ignore[name-defined]

src/server/query_processor.py

@@ -88,11 +88,7 @@ async def process_query(
 
         # Get the current commit hash if not already available
         if not query.commit:
-            try:
-                query.commit = await get_current_commit_hash(clone_config.local_path)
-            except RuntimeError:
-                # If we can't get the commit hash, use a default
-                query.commit = "HEAD"
+            query.commit = await get_current_commit_hash(clone_config.local_path)
 
         summary, tree, content = ingest_query(query)