iris-vector-graph 1.45.2__tar.gz → 1.45.3__tar.gz

{iris_vector_graph-1.45.2 → iris_vector_graph-1.45.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: iris-vector-graph
-Version: 1.45.2
+Version: 1.45.3
 Summary: Transactional Graph + Vector retrieval system for InterSystems IRIS with hybrid search, openCypher, and GraphQL APIs
 Project-URL: Homepage, https://github.com/isc-tdyar/iris-vector-graph
 Project-URL: Documentation, https://github.com/isc-tdyar/iris-vector-graph/tree/main/docs
@@ -474,6 +474,11 @@ anchors = engine.get_kg_anchors(icd_codes=["J18.0", "E11.9"])
 
 ## Changelog
 
+### v1.45.3 (2026-04-04)
+- `translate_relationship_pattern`: inline property filters on relationship nodes were silently dropped — `MATCH (t)-[:R]->(c {id: 'x'})` returned all nodes instead of filtering. Fixed by applying `source_node.properties` and `target_node.properties` after JOIN construction.
+- `vector_search`: `TO_VECTOR(?, DOUBLE, {dim})` now includes explicit dimension in query cast, resolving type mismatch on IRIS 2025.1 when column dimension is known
+- 2 regression tests added (375 unit tests total)
+
 ### v1.45.2 (2026-04-03)
 - `embedded.py`: auto-fixes `sys.path` shadowing — ensures `/usr/irissys/lib/python` is first so the embedded `iris` module takes priority over pip-installed `intersystems_irispython`
 - `embedded.py`: clear error message when shadowed iris (no `iris.sql`) is detected, naming the root cause

{iris_vector_graph-1.45.2 → iris_vector_graph-1.45.3}/README.md

@@ -403,6 +403,11 @@ anchors = engine.get_kg_anchors(icd_codes=["J18.0", "E11.9"])
 
 ## Changelog
 
+### v1.45.3 (2026-04-04)
+- `translate_relationship_pattern`: inline property filters on relationship nodes were silently dropped — `MATCH (t)-[:R]->(c {id: 'x'})` returned all nodes instead of filtering. Fixed by applying `source_node.properties` and `target_node.properties` after JOIN construction.
+- `vector_search`: `TO_VECTOR(?, DOUBLE, {dim})` now includes explicit dimension in query cast, resolving type mismatch on IRIS 2025.1 when column dimension is known
+- 2 regression tests added (375 unit tests total)
+
 ### v1.45.2 (2026-04-03)
 - `embedded.py`: auto-fixes `sys.path` shadowing — ensures `/usr/irissys/lib/python` is first so the embedded `iris` module takes priority over pip-installed `intersystems_irispython`
 - `embedded.py`: clear error message when shadowed iris (no `iris.sql`) is detected, naming the root cause

{iris_vector_graph-1.45.2 → iris_vector_graph-1.45.3}/iris_vector_graph/cypher/translator.py

@@ -1141,13 +1141,31 @@ def translate_relationship_pattern(rel, source_node, target_node, context, metad
         else: edge_cond += f" AND {edge_alias}.p IN ({', '.join([context.add_join_param(t) for t in rel.types])})"
         
     context.join_clauses.append(f"{jt} {_table('rdf_edges')} {edge_alias} ON {edge_cond}")
-    
+
     if is_new_target and not target_alias.startswith('Stage'):
         context.join_clauses.append(f"{jt} {_table('nodes')} {target_alias} ON {target_on}")
     else:
         # If target node is already joined, add the connection as a WHERE condition
         context.where_conditions.append(target_on)
 
+    # Apply inline property filters from source and target nodes.
+    # These are silently dropped without this block — e.g. MATCH (t)-[:R]->(c {id: 'x'})
+    # returns all nodes instead of filtering, because the relationship path never applies them.
+    for prop_node, prop_alias in ((source_node, source_alias), (target_node, target_alias)):
+        if not prop_node.properties:
+            continue
+        for k, v in prop_node.properties.items():
+            val_sql = translate_expression(v, context, segment="where")
+            if k in ("node_id", "id"):
+                context.where_conditions.append(f"{prop_alias}.node_id = {val_sql}")
+            else:
+                p_alias = context.next_alias("p")
+                context.join_clauses.append(
+                    f"{jt} {_table('rdf_props')} {p_alias} "
+                    f"ON {p_alias}.s = {prop_alias}.node_id AND {p_alias}.\"key\" = {context.add_join_param(k)}"
+                )
+                context.where_conditions.append(f"{p_alias}.val = {val_sql}")
+
 
 def translate_where_clause(where, context):
     context.where_conditions.append(translate_boolean_expression(where.expression, context))

{iris_vector_graph-1.45.2 → iris_vector_graph-1.45.3}/iris_vector_graph/engine.py

@@ -1652,7 +1652,7 @@ class IRISGraphEngine:
                 raise ValueError(f"Column '{vector_col}' not found in table '{table}'")
             cursor.execute(f"SELECT COUNT(*) FROM {table}")
             row_count = int(cursor.fetchone()[0])
-            cursor.execute(f"SELECT {vector_col} FROM {table} WHERE ROWNUM = 1")
+            cursor.execute(f"SELECT TOP 1 {vector_col} FROM {table}")
             sample = cursor.fetchone()
             dimension = None
             if sample and sample[0]:
@@ -1691,7 +1691,19 @@ class IRISGraphEngine:
             query_vec_str = query_embedding
 
         extra = ", ".join(sanitize_identifier(c) for c in (return_cols or []) if c != id_col)
-        select_cols = f"t.{id_col}, VECTOR_COSINE(TO_VECTOR(t.{vector_col}), TO_VECTOR(?, DOUBLE)) AS score"
+
+        dim = None
+        if isinstance(query_embedding, list):
+            dim = len(query_embedding)
+        elif isinstance(query_embedding, str):
+            dim = query_embedding.count(",") + 1
+
+        if dim:
+            query_cast = f"TO_VECTOR(?, DOUBLE, {dim})"
+        else:
+            query_cast = "TO_VECTOR(?, DOUBLE)"
+
+        select_cols = f"t.{id_col}, VECTOR_COSINE(t.{vector_col}, {query_cast}) AS score"
         if extra:
             select_cols += f", {extra}"
 

{iris_vector_graph-1.45.2 → iris_vector_graph-1.45.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "iris-vector-graph"
-version = "1.45.2"
+version = "1.45.3"
 description = "Transactional Graph + Vector retrieval system for InterSystems IRIS with hybrid search, openCypher, and GraphQL APIs"
 readme = "README.md"
 license = "MIT"

{iris_vector_graph-1.45.2 → iris_vector_graph-1.45.3}/tests/unit/test_cypher_translator.py

@@ -101,3 +101,34 @@ def test_translate_pattern_operators():
     sql3 = translate_to_sql(parsed3)
     sql_str3 = "\n".join(sql3.sql) if isinstance(sql3.sql, list) else sql3.sql
     assert "LIKE" in sql_str3
+
+
+def test_inline_property_filter_on_relationship_target():
+    """Inline props on relationship target must generate WHERE conditions, not be silently dropped.
+
+    Bug: MATCH (t)-[:R]->(c:Label {id: 'x'}) returned all nodes instead of filtering,
+    because translate_relationship_pattern did not apply target_node.properties after joining.
+    """
+    query = "MATCH (t:IServiceTicket)-[:TICKET_FOR]->(c:Customer {id: 'Customer:Northwell'}) RETURN t.id as id"
+    parsed = parse_query(query)
+    sql_result = translate_to_sql(parsed)
+    sql_str = "\n".join(sql_result.sql) if isinstance(sql_result.sql, list) else sql_result.sql
+    # SQLQuery.parameters is a list of param-lists (one per statement); flatten to check
+    params = [p for plist in sql_result.parameters for p in plist]
+
+    # The literal is passed as a bind param — check the params list, not the SQL string
+    assert "Customer:Northwell" in params, \
+        f"Inline property filter on relationship target was dropped — literal not in params: {params}"
+    # Must use node_id equality for id/node_id keys (not rdf_props join)
+    assert "node_id" in sql_str
+
+
+def test_inline_property_filter_on_relationship_source():
+    """Inline props on relationship source must also generate WHERE conditions."""
+    query = "MATCH (t:IServiceTicket {status: 'Open'})-[:TICKET_FOR]->(c:Customer) RETURN c.id as cid"
+    parsed = parse_query(query)
+    sql_result = translate_to_sql(parsed)
+    params = [p for plist in sql_result.parameters for p in plist]
+
+    assert "Open" in params, \
+        f"Inline property filter on relationship source was dropped — literal not in params: {params}"