fix(client,demo): align type annotations with runtime; exec in docker; notebook fixes

- Update _build_property_definitions annotation to include tuple[dict, ...] (matches isinstance check)
- Update create_text_index annotation to include tuple[str, ...] in async and sync signatures
- Add `exec` before start-notebook.sh so Jupyter becomes PID 1 (proper signal handling)
- Extract _EMBEDDED_PIP_SPEC constant in notebook 03 to avoid drift on future bumps
- Fix query_facts LIMIT guard: skip appending LIMIT 20 when query already has any LIMIT clause (prevents invalid double-LIMIT with parameterized LIMIT $n)

Author: polaz

coordinode/coordinode/client.py

@@ -449,7 +449,7 @@ def _validate_property_dict(p: Any, idx: int) -> tuple[str, str, bool, bool]:
 
     @staticmethod
     def _build_property_definitions(
-        properties: list[dict[str, Any]] | None,
+        properties: list[dict[str, Any]] | tuple[dict[str, Any], ...] | None,
         property_type_cls: Any,
         property_definition_cls: Any,
     ) -> list[Any]:
@@ -613,7 +613,7 @@ async def create_text_index(
         self,
         name: str,
         label: str,
-        properties: str | list[str],
+        properties: str | list[str] | tuple[str, ...],
         *,
         language: str = "",
     ) -> TextIndexInfo:
@@ -984,7 +984,7 @@ def create_text_index(
         self,
         name: str,
         label: str,
-        properties: str | list[str],
+        properties: str | list[str] | tuple[str, ...],
         *,
         language: str = "",
     ) -> TextIndexInfo:

demo/docker-compose.yml

@@ -40,7 +40,7 @@ services:
     # /tmp/install-sdk.sh is copied into the image by Dockerfile.jupyter;
     # it must run after /sdk is mounted (i.e. at runtime, not during build).
     command: >-
-      bash -c "/tmp/install-sdk.sh && start-notebook.sh"
+      bash -c "/tmp/install-sdk.sh && exec start-notebook.sh"
     depends_on:
       coordinode:
         condition: service_healthy

demo/notebooks/03_langgraph_agent.ipynb

@@ -41,6 +41,10 @@
     "import os, sys, subprocess\n",
     "\n",
     "IN_COLAB = \"google.colab\" in sys.modules\n",
+    "_EMBEDDED_PIP_SPEC = (\n",
+    "    \"git+https://github.com/structured-world/coordinode-python.git\"\n",
+    "    \"@8da94d694ecaabee6f8380147d02f08220061bfa#subdirectory=coordinode-embedded\"\n",
+    ")\n",
     "\n",
     "# Install coordinode-embedded in Colab only (requires Rust build).\n",
     "if IN_COLAB and not os.environ.get(\"COORDINODE_ADDR\"):\n",
@@ -80,7 +84,7 @@
     "            \"pip\",\n",
     "            \"install\",\n",
     "            \"-q\",\n",
-    "            \"git+https://github.com/structured-world/coordinode-python.git@8da94d694ecaabee6f8380147d02f08220061bfa#subdirectory=coordinode-embedded\",\n",
+    "            _EMBEDDED_PIP_SPEC,\n",
     "        ],\n",
     "        check=True,\n",
     "        timeout=600,\n",
@@ -169,7 +173,7 @@
     "        except ImportError as exc:\n",
     "            raise RuntimeError(\n",
     "                \"coordinode-embedded is not installed. \"\n",
-    "                \"Run: pip install git+https://github.com/structured-world/coordinode-python.git@8da94d694ecaabee6f8380147d02f08220061bfa#subdirectory=coordinode-embedded\"\n",
+    "                f\"Run: pip install {_EMBEDDED_PIP_SPEC}\"\n",
     "                \" — or start a CoordiNode server and set COORDINODE_ADDR.\"\n",
     "            ) from exc\n",
     "\n",
@@ -258,6 +262,10 @@
     "        return f\"LIMIT {min(int(m.group(1)), 20)}\"\n",
     "    if _LIMIT_AT_END_RE.search(q):\n",
     "        q = _LIMIT_AT_END_RE.sub(_cap_limit, q)\n",
+    "    elif re.search(r\"\\bLIMIT\\b\", q, re.IGNORECASE):\n",
+    "        # Respect caller-provided non-terminal / parameterized LIMIT clauses\n",
+    "        # (e.g. LIMIT $n); appending a second LIMIT would produce invalid Cypher.\n",
+    "        pass\n",
     "    else:\n",
     "        q = q.rstrip().rstrip(\";\") + \" LIMIT 20\"\n",
     "    rows = client.cypher(q, params={\"sess\": SESSION})\n",