fix(client): validate property name in _build_property_definitions; sync docstring types

- _build_property_definitions: raise ValueError with index when property entry
  is not a dict with a non-empty 'name' key (was KeyError)
- LabelInfo.schema_mode inline annotation: 0=unspecified (not 0=unspecified/strict)
- create_label docstring: add "list" and "map" to the type strings list
- _first_label docstring: clarify min() is intentional for determinism;
  labels(n)[0] is unstable due to non-guaranteed label ordering

Author: polaz

coordinode/coordinode/client.py

@@ -105,7 +105,7 @@ def __init__(self, proto_label: Any) -> None:
         self.name: str = proto_label.name
         self.version: int = proto_label.version
         self.properties: list[PropertyDefinitionInfo] = [PropertyDefinitionInfo(p) for p in proto_label.properties]
-        # schema_mode: 0=unspecified/strict, 1=strict, 2=validated, 3=flexible
+        # schema_mode: 0=unspecified, 1=strict, 2=validated, 3=flexible
         self.schema_mode: int = getattr(proto_label, "schema_mode", 0)
 
     def __repr__(self) -> str:
@@ -391,12 +391,14 @@ def _build_property_definitions(
             "map": PropertyType.PROPERTY_TYPE_MAP,
         }
         result = []
-        for p in properties or []:
+        for idx, p in enumerate(properties or []):
+            name = p.get("name") if isinstance(p, dict) else None
+            if not isinstance(name, str) or not name:
+                raise ValueError(f"Property at index {idx} must be a dict with a non-empty 'name' key; got {p!r}")
             type_str = str(p.get("type", "string")).lower()
             if type_str not in type_map:
                 raise ValueError(
-                    f"Unknown property type {type_str!r} for property {p['name']!r}. "
-                    f"Expected one of: {sorted(type_map)}"
+                    f"Unknown property type {type_str!r} for property {name!r}. Expected one of: {sorted(type_map)}"
                 )
             result.append(
                 PropertyDefinition(
@@ -423,7 +425,7 @@ async def create_label(
                 ``name`` (str), ``type`` (str), ``required`` (bool),
                 ``unique`` (bool).  Type strings: ``"string"``,
                 ``"int64"``, ``"float64"``, ``"bool"``, ``"bytes"``,
-                ``"timestamp"``, ``"vector"``.
+                ``"timestamp"``, ``"vector"``, ``"list"``, ``"map"``.
             schema_mode: ``"strict"`` (default — reject undeclared props),
                 ``"validated"`` (allow extra props without interning),
                 ``"flexible"`` (no enforcement).

langchain-coordinode/langchain_coordinode/graph.py

@@ -285,8 +285,10 @@ def _first_label(labels: Any) -> str | None:
     possible. ``min()`` gives a deterministic result regardless of how many
     labels are present.
 
-    TODO: replace with ``labels(n)[0]`` in Cypher once subscript-on-function
-    lands in the published Docker image.
+    Note: once subscript-on-function support lands in the published Docker
+    image (tracked in G010 / GAPS.md), this Python helper could be replaced
+    by an inline Cypher expression — but keep the deterministic ``min()``
+    rule rather than index 0, since label ordering is not guaranteed stable.
     """
     if isinstance(labels, list) and labels:
         return str(min(labels))