fix(rag): return 200 for partial indexing

Author: BjornMelin

docs/specs/active/0018-spec-rag-retriever-indexer.md

@@ -76,6 +76,10 @@ Index documents with automatic chunking and embedding generation.
 }
 ```
 
+**Status codes:**
+
+- Always returns `200 OK`, including partial indexing; inspect `success` and `failed` for per-item status.
+
 ### POST /api/rag/search
 
 Search documents with hybrid retrieval and optional reranking.

src/app/api/rag/index/route.ts

@@ -19,8 +19,8 @@ import { indexDocuments } from "@/lib/rag/indexer";
  * Index documents into the RAG store with automatic chunking and embedding.
  *
  * @param req - Request with documents array.
- * @returns Index result with counts and failed documents. Uses HTTP 207
- *          (Multi-Status) when indexing partially succeeds.
+ * @returns Index result with counts and failed documents. Always returns HTTP 200;
+ *          partial success is conveyed via `success: false` and per-item failures.
  *
  * @example
  * ```bash
@@ -49,7 +49,5 @@ export const POST = withApiGuards({
     supabase,
   });
 
-  return NextResponse.json(result, {
-    status: result.success ? 200 : 207, // 207 Multi-Status for partial success
-  });
+  return NextResponse.json(result, { status: 200 });
 });

src/lib/rag/__tests__/reranker.test.ts

@@ -117,7 +117,7 @@ describe("TogetherReranker", () => {
 
     expect(mockRerank).toHaveBeenCalledWith(
       expect.objectContaining({
-        topN: 1, // Should cap topN by config topN
+        topN: 1, // Should use min(requested topN, documents.length, config.topN)
       })
     );
     expect(result).toHaveLength(1);