Fix MemosReader default endpoint and metadata

llama-index-integrations/readers/llama-index-readers-memos/llama_index/readers/memos/base.py

@@ -36,7 +36,7 @@ def load_data(self, params: Dict = {}) -> List[Document]:
         realUrl = self._memoUrl
 
         if not params:
-            realUrl = urljoin(self._memoUrl, "all", False)
+            realUrl = urljoin(f"{self._memoUrl}/", "all")
 
         try:
             req = requests.get(realUrl, params)
@@ -53,7 +53,7 @@ def load_data(self, params: Dict = {}) -> List[Document]:
             extra_info = {
                 "creator": memo["creator"],
                 "resource_list": memo["resourceList"],
-                id: memo["id"],
+                "id": memo["id"],
             }
             documents.append(Document(text=content, extra_info=extra_info))
 

llama-index-integrations/readers/llama-index-readers-memos/tests/test_readers_memos.py

@@ -1,7 +1,46 @@
+from unittest.mock import patch
+
 from llama_index.core.readers.base import BaseReader
 from llama_index.readers.memos import MemosReader
 
 
 def test_class():
     names_of_base_classes = [b.__name__ for b in MemosReader.__mro__]
     assert BaseReader.__name__ in names_of_base_classes
+
+
+def test_load_data_uses_memo_all_endpoint_by_default():
+    with patch("requests.get") as mock_get:
+        mock_get.return_value.json.return_value = {"data": []}
+
+        docs = MemosReader("https://example.com/").load_data()
+
+        assert docs == []
+        mock_get.assert_called_once_with("https://example.com/api/memo/all", {})
+
+
+def test_load_data_sets_string_id_metadata_key():
+    payload = {
+        "data": [
+            {
+                "id": 7,
+                "content": "hello",
+                "creator": "me",
+                "resourceList": [],
+            }
+        ]
+    }
+
+    with patch("requests.get") as mock_get:
+        mock_get.return_value.json.return_value = payload
+
+        docs = MemosReader("https://example.com/").load_data(params={"creator": "me"})
+
+        assert len(docs) == 1
+        assert docs[0].text == "hello"
+        assert docs[0].metadata["id"] == 7
+        assert docs[0].metadata["creator"] == "me"
+        assert docs[0].metadata["resource_list"] == []
+        mock_get.assert_called_once_with(
+            "https://example.com/api/memo", {"creator": "me"}
+        )