fix(search): compare notebook scope ids as strings

The driver returns `id` as RecordID objects, which are unhashable and
never equal to the request strings, so every real scoped request raised
TypeError before reaching the search. Normalize to str before comparing
and add a regression test that mocks RecordID rows.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014kRBoPiBD1NKeKHCo6EyrE

Author: lfnovo

open_notebook/domain/notebook.py

@@ -807,7 +807,9 @@ async def resolve_notebook_scope(notebook_ids: List[str]) -> List[str]:
         logger.error(f"Error resolving notebook scope: {str(e)}")
         logger.exception(e)
         raise DatabaseOperationError("Failed to resolve notebook scope")
-    found = {row["id"] for row in rows}
+    # The driver returns ids as RecordID objects (unhashable, and never equal
+    # to the request strings), so normalize to strings before comparing.
+    found = {str(row["id"]) for row in rows}
     missing = [nb_id for nb_id in notebook_ids if nb_id not in found]
     if missing:
         raise NotFoundError(f"Notebook(s) not found: {', '.join(missing)}")

tests/test_search_api.py

@@ -435,6 +435,23 @@ async def test_missing_notebooks_raise_not_found_listing_them(self):
                     ["notebook:a", "notebook:zzz"]
                 )
 
+    @pytest.mark.asyncio
+    async def test_record_id_rows_from_driver_match_request_strings(self):
+        """The real driver returns RecordID objects, not strings (regression)."""
+        from surrealdb import RecordID
+
+        from open_notebook.domain import notebook as notebook_module
+
+        with patch.object(
+            notebook_module,
+            "repo_query",
+            new_callable=AsyncMock,
+            return_value=[{"id": RecordID("notebook", "a")}],
+        ):
+            assert await notebook_module.resolve_notebook_scope(["notebook:a"]) == [
+                "notebook:a"
+            ]
+
 
 class TestNotebookScopedSearchDomain:
     """text_search / vector_search bind the scope as record ids (#574, #87)."""