fix(client): validate max_depth >= 1 in traverse(); xfail strict=True

- traverse(): raise ValueError locally when max_depth < 1 before RPC
- test_traverse_inbound_direction: strict=True so CI catches unexpected
  XPASS when server adds inbound direction support
- test_get_labels_has_property_definitions: document why the assertion
  is intentionally weak (schema-free labels may have empty properties)

Author: polaz

coordinode/coordinode/client.py

@@ -396,6 +396,8 @@ async def traverse(
         key = direction.lower()
         if key not in _direction_map:
             raise ValueError(f"Invalid direction {direction!r}. Must be one of: 'outbound', 'inbound', 'both'.")
+        if max_depth < 1:
+            raise ValueError(f"max_depth must be >= 1, got {max_depth!r}.")
         direction_value = _direction_map[key]
 
         req = TraverseRequest(

tests/integration/test_sdk.py

@@ -234,6 +234,8 @@ def test_get_labels_has_property_definitions(client):
         labels = client.get_labels()
         found = next((lbl for lbl in labels if lbl.name == "PropLabel"), None)
         assert found is not None, "PropLabel not returned by get_labels()"
+        # Intentionally only check the type — CoordiNode is schema-free and may return
+        # an empty properties list even when the node was created with properties.
         assert isinstance(found.properties, list)
     finally:
         client.cypher("MATCH (n:PropLabel {tag: $tag}) DETACH DELETE n", params={"tag": tag})
@@ -279,7 +281,7 @@ def test_traverse_returns_neighbours(client):
 
 
 @pytest.mark.xfail(
-    strict=False,
+    strict=True,
     reason="CoordiNode Traverse RPC does not yet support inbound direction — server returns empty result set",
 )
 def test_traverse_inbound_direction(client):