feat: replace ingest_id with digest_url for downloads

- Update `IngestResponse` schema to include `digest_url` instead of `ingest_id`.
- Modify API and UI logic to handle `digest_url` for file downloads.
- Disable local file download endpoint when S3 is enabled.

Author: MickaelCa

src/server/models.py

@@ -71,8 +71,8 @@ class IngestSuccessResponse(BaseModel):
         Short form of repository URL (user/repo).
     summary : str
         Summary of the ingestion process including token estimates.
-    ingest_id : str
-        Ingestion id used to download full context.
+    digest_url : str
+        URL to download the full digest content (either S3 URL or local download endpoint).
     tree : str
         File tree structure of the repository.
     content : str
@@ -89,7 +89,7 @@ class IngestSuccessResponse(BaseModel):
     repo_url: str = Field(..., description="Original repository URL")
     short_repo_url: str = Field(..., description="Short repository URL (user/repo)")
     summary: str = Field(..., description="Ingestion summary with token estimates")
-    ingest_id: str = Field(..., description="Ingestion id used to download full context")
+    digest_url: str = Field(..., description="URL to download the full digest content")
     tree: str = Field(..., description="File tree structure")
     content: str = Field(..., description="Processed file content")
     default_max_file_size: int = Field(..., description="File size slider position used")

src/server/query_processor.py

@@ -125,11 +125,21 @@ async def process_query(
         summary=summary,
     )
 
+    # Generate digest_url based on S3 configuration
+    if is_s3_enabled():
+        digest_url = getattr(query, "s3_url", None)
+        if not digest_url:
+            # This should not happen if S3 upload was successful
+            msg = "S3 is enabled but no S3 URL was generated"
+            raise RuntimeError(msg)
+    else:
+        digest_url = f"/api/download/file/{query.id}"
+
     return IngestSuccessResponse(
         repo_url=input_text,
         short_repo_url=short_repo_url,
         summary=summary,
-        ingest_id=str(query.id),
+        digest_url=digest_url,
         tree=tree,
         content=content,
         default_max_file_size=slider_position,

src/server/routers/ingest.py

@@ -10,7 +10,7 @@
 from gitingest.config import TMP_BASE_PATH
 from server.models import IngestRequest
 from server.routers_utils import COMMON_INGEST_RESPONSES, _perform_ingestion
-from server.s3_utils import get_s3_url_for_ingest_id, is_s3_enabled
+from server.s3_utils import is_s3_enabled
 from server.server_config import MAX_DISPLAY_SIZE
 from server.server_utils import limiter
 
@@ -101,29 +101,31 @@ async def download_ingest(
     """Download the first text file produced for an ingest ID.
 
     **This endpoint retrieves the first ``*.txt`` file produced during the ingestion process**
-    and returns it as a downloadable file. If S3 is enabled and the file is stored in S3,
-    it redirects to the S3 URL. Otherwise, it serves the local file.
+    and returns it as a downloadable file. When S3 is enabled, this endpoint is disabled
+    and clients should use the S3 URL provided in the ingest response instead.
 
     **Parameters**
 
     - **ingest_id** (`UUID`): Identifier that the ingest step emitted
 
     **Returns**
 
-    - **RedirectResponse**: Redirect to S3 URL if S3 is enabled and file exists in S3
     - **FileResponse**: Streamed response with media type ``text/plain`` for local files
 
     **Raises**
 
+    - **HTTPException**: **503** - endpoint is disabled when S3 is enabled
     - **HTTPException**: **404** - digest directory is missing or contains no ``*.txt`` file
     - **HTTPException**: **403** - the process lacks permission to read the directory or file
 
     """
-    # Check if S3 is enabled and file exists in S3
+    # Disable download endpoint when S3 is enabled
     if is_s3_enabled():
-        s3_url = get_s3_url_for_ingest_id(ingest_id)
-        if s3_url:
-            return RedirectResponse(url=s3_url, status_code=302)
+        raise HTTPException(
+            status_code=status.HTTP_503_SERVICE_UNAVAILABLE,
+            detail="Download endpoint is disabled when S3 is enabled. "
+            "Use the S3 URL provided in the ingest response instead.",
+        )
 
     # Fall back to local file serving
     # Normalize and validate the directory path

src/static/js/utils.js

@@ -172,8 +172,8 @@ function handleSuccessfulResponse(data) {
     // Show results section
     showResults();
 
-    // Store the ingest_id for download functionality
-    window.currentIngestId = data.ingest_id;
+    // Store the digest_url for download functionality
+    window.currentDigestUrl = data.digest_url;
 
     // Set plain text content for summary, tree, and content
     document.getElementById('result-summary').value = data.summary || '';
@@ -271,9 +271,9 @@ function copyFullDigest() {
 }
 
 function downloadFullDigest() {
-    // Check if we have an ingest_id
-    if (!window.currentIngestId) {
-        console.error('No ingest_id available for download');
+    // Check if we have a digest_url
+    if (!window.currentDigestUrl) {
+        console.error('No digest_url available for download');
 
         return;
     }
@@ -289,10 +289,10 @@ function downloadFullDigest() {
         Downloading...
     `;
 
-    // Create a download link to the server endpoint
+    // Create a download link using the digest_url
     const a = document.createElement('a');
 
-    a.href = `/api/download/file/${window.currentIngestId}`;
+    a.href = window.currentDigestUrl;
     a.download = 'digest.txt';
     document.body.appendChild(a);
     a.click();