fix: remove references to ingest_id and add download feature

Author: ix-56h

docs/api_models.md

@@ -63,7 +63,6 @@ class IngestSuccessResponse(BaseModel):
     summary: str = Field(..., description="Ingestion summary with token estimates")
     tree: str = Field(..., description="File tree structure")
     content: str = Field(..., description="Processed file content")
-    ingest_id: str = Field(..., description="Unique ingestion identifier")
     default_file_size: int = Field(..., description="File size slider position used")
     pattern_type: str = Field(..., description="Pattern type used")
     pattern: str = Field(..., description="Pattern used")
@@ -79,7 +78,6 @@ class IngestSuccessResponse(BaseModel):
   "summary": "Processed 50 files, estimated tokens: 15,000",
   "tree": "gitingest/\n├── src/\n│   ├── server/\n│   └── gitingest/\n└── README.md",
   "content": "Repository content here...",
-  "ingest_id": "abc123",
   "default_file_size": 243,
   "pattern_type": "exclude",
   "pattern": "*.md",

tests/test_api_ingest.py

@@ -32,12 +32,7 @@ def mock_static_files(mocker: MockerFixture) -> None:
     return mock_static
 
 
-@pytest.fixture(autouse=True)
-def mock_templates(mocker: MockerFixture) -> None:
-    """Mock Jinja2 template rendering to bypass actual file loading."""
-    mock_template = mocker.patch("starlette.templating.Jinja2Templates.TemplateResponse", autospec=True)
-    mock_template.return_value = "Mocked Template Response"
-    return mock_template
+
 
 
 @pytest.fixture(scope="module", autouse=True)
@@ -64,9 +59,17 @@ async def test_remote_repository_analysis(request: pytest.FixtureRequest) -> Non
         "token": "",
     }
 
-    response = client.post("/", data=form_data)
+    response = client.post("/api/ingest", data=form_data)
     assert response.status_code == status.HTTP_200_OK, f"Form submission failed: {response.text}"
-    assert "Mocked Template Response" in response.text
+    
+    # Check that response is JSON
+    response_data = response.json()
+    assert "result" in response_data
+    assert response_data["result"] is True
+    assert "repo_url" in response_data
+    assert "summary" in response_data
+    assert "tree" in response_data
+    assert "content" in response_data
 
 
 @pytest.mark.asyncio
@@ -81,9 +84,14 @@ async def test_invalid_repository_url(request: pytest.FixtureRequest) -> None:
         "token": "",
     }
 
-    response = client.post("/", data=form_data)
-    assert response.status_code == status.HTTP_200_OK, f"Request failed: {response.text}"
-    assert "Mocked Template Response" in response.text
+    response = client.post("/api/ingest", data=form_data)
+    # Should return 400 for invalid repository
+    assert response.status_code == status.HTTP_400_BAD_REQUEST, f"Request failed: {response.text}"
+    
+    # Check that response is JSON error
+    response_data = response.json()
+    assert "error" in response_data
+    assert "repo_url" in response_data
 
 
 @pytest.mark.asyncio
@@ -98,9 +106,16 @@ async def test_large_repository(request: pytest.FixtureRequest) -> None:
         "token": "",
     }
 
-    response = client.post("/", data=form_data)
-    assert response.status_code == status.HTTP_200_OK, f"Request failed: {response.text}"
-    assert "Mocked Template Response" in response.text
+    response = client.post("/api/ingest", data=form_data)
+    # This might fail with 400 if repo doesn't exist, or succeed with 200
+    assert response.status_code in [status.HTTP_200_OK, status.HTTP_400_BAD_REQUEST], f"Request failed: {response.text}"
+    
+    response_data = response.json()
+    if response.status_code == status.HTTP_200_OK:
+        assert "result" in response_data
+        assert response_data["result"] is True
+    else:
+        assert "error" in response_data
 
 
 @pytest.mark.asyncio
@@ -116,9 +131,15 @@ def make_request() -> None:
             "pattern": "",
             "token": "",
         }
-        response = client.post("/", data=form_data)
-        assert response.status_code == status.HTTP_200_OK, f"Request failed: {response.text}"
-        assert "Mocked Template Response" in response.text
+        response = client.post("/api/ingest", data=form_data)
+        assert response.status_code in [status.HTTP_200_OK, status.HTTP_400_BAD_REQUEST], f"Request failed: {response.text}"
+        
+        response_data = response.json()
+        if response.status_code == status.HTTP_200_OK:
+            assert "result" in response_data
+            assert response_data["result"] is True
+        else:
+            assert "error" in response_data
 
     with ThreadPoolExecutor(max_workers=5) as executor:
         futures = [executor.submit(make_request) for _ in range(5)]
@@ -138,9 +159,15 @@ async def test_large_file_handling(request: pytest.FixtureRequest) -> None:
         "token": "",
     }
 
-    response = client.post("/", data=form_data)
-    assert response.status_code == status.HTTP_200_OK, f"Request failed: {response.text}"
-    assert "Mocked Template Response" in response.text
+    response = client.post("/api/ingest", data=form_data)
+    assert response.status_code in [status.HTTP_200_OK, status.HTTP_400_BAD_REQUEST], f"Request failed: {response.text}"
+    
+    response_data = response.json()
+    if response.status_code == status.HTTP_200_OK:
+        assert "result" in response_data
+        assert response_data["result"] is True
+    else:
+        assert "error" in response_data
 
 
 @pytest.mark.asyncio
@@ -155,6 +182,16 @@ async def test_repository_with_patterns(request: pytest.FixtureRequest) -> None:
         "token": "",
     }
 
-    response = client.post("/", data=form_data)
-    assert response.status_code == status.HTTP_200_OK, f"Request failed: {response.text}"
-    assert "Mocked Template Response" in response.text
+    response = client.post("/api/ingest", data=form_data)
+    assert response.status_code in [status.HTTP_200_OK, status.HTTP_400_BAD_REQUEST], f"Request failed: {response.text}"
+    
+    response_data = response.json()
+    if response.status_code == status.HTTP_200_OK:
+        assert "result" in response_data
+        assert response_data["result"] is True
+        assert "pattern_type" in response_data
+        assert response_data["pattern_type"] == "include"
+        assert "pattern" in response_data
+        assert response_data["pattern"] == "*.md"
+    else:
+        assert "error" in response_data