fix: poll status=="completed" in cloud add_document (#226)

The cloud backend previously polled tree_resp["retrieval_ready"]
as the ready signal. Empirically this flag is not a reliable
indicator — docs can reach status=="completed" without
retrieval_ready flipping, causing col.add() to wait until the 10
min timeout before giving up on otherwise-successful uploads.

The cloud API's canonical ready signal is status=="completed";
switch the poll to check that instead.

Author: rejojer

pageindex/backend/cloud.py

@@ -141,12 +141,13 @@ def add_document(self, collection: str, file_path: str) -> str:
 
         doc_id = resp["doc_id"]
 
-        # Poll until retrieval-ready
+        # Poll until indexing completes. The cloud API signals readiness via
+        # status == "completed"; retrieval_ready is not a reliable indicator.
         for _ in range(120):  # 10 min max
             tree_resp = self._request("GET", f"/doc/{self._enc(doc_id)}/", params={"type": "tree"})
-            if tree_resp.get("retrieval_ready"):
-                return doc_id
             status = tree_resp.get("status", "")
+            if status == "completed":
+                return doc_id
             if status == "failed":
                 raise CloudAPIError(f"Document {doc_id} indexing failed")
             time.sleep(5)