sql-glider 0.1.6__tar.gz → 0.1.8__tar.gz

{sql_glider-0.1.6 → sql_glider-0.1.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sql-glider
-Version: 0.1.6
+Version: 0.1.8
 Summary: SQL Utility Toolkit for better understanding, use, and governance of your queries in a native environment.
 Project-URL: Homepage, https://github.com/rycowhi/sql-glider/
 Project-URL: Repository, https://github.com/rycowhi/sql-glider/

{sql_glider-0.1.6 → sql_glider-0.1.8}/src/sqlglider/_version.py

@@ -28,7 +28,7 @@ version_tuple: VERSION_TUPLE
 commit_id: COMMIT_ID
 __commit_id__: COMMIT_ID
 
-__version__ = version = '0.1.6'
-__version_tuple__ = version_tuple = (0, 1, 6)
+__version__ = version = '0.1.8'
+__version_tuple__ = version_tuple = (0, 1, 8)
 
 __commit_id__ = commit_id = None

{sql_glider-0.1.6 → sql_glider-0.1.8}/src/sqlglider/graph/builder.py

@@ -114,11 +114,11 @@ class GraphBuilder:
                     )
 
                     # Add edge (source contributes_to target) - deduplicate
-                    edge_key = (item.source_name, item.output_name)
+                    edge_key = (item.source_name.lower(), item.output_name.lower())
                     if edge_key not in self._edge_set:
                         edge = GraphEdge(
-                            source_node=item.source_name,
-                            target_node=item.output_name,
+                            source_node=item.source_name.lower(),
+                            target_node=item.output_name.lower(),
                             file_path=file_path_str,
                             query_index=query_index,
                         )
@@ -280,17 +280,18 @@ class GraphBuilder:
         Returns:
             rustworkx node index
         """
-        if identifier in self._node_index_map:
-            return self._node_index_map[identifier]
+        key = identifier.lower()
+        if key in self._node_index_map:
+            return self._node_index_map[key]
 
         node = GraphNode.from_identifier(
-            identifier=identifier,
+            identifier=key,
             file_path=file_path,
             query_index=query_index,
         )
 
         node_idx = self.graph.add_node(node.model_dump())
-        self._node_index_map[identifier] = node_idx
+        self._node_index_map[key] = node_idx
         return node_idx
 
     def build(self) -> LineageGraph:

{sql_glider-0.1.6 → sql_glider-0.1.8}/src/sqlglider/graph/serialization.py

@@ -61,15 +61,15 @@ def to_rustworkx(graph: LineageGraph) -> Tuple[rx.PyDiGraph, Dict[str, int]]:
     rx_graph: rx.PyDiGraph = rx.PyDiGraph()
     node_map: Dict[str, int] = {}
 
-    # Add nodes
+    # Add nodes (use lowercase keys for case-insensitive lookup)
     for node in graph.nodes:
         idx = rx_graph.add_node(node.model_dump())
-        node_map[node.identifier] = idx
+        node_map[node.identifier.lower()] = idx
 
-    # Add edges
+    # Add edges (use lowercase for lookup to match node keys)
     for edge in graph.edges:
-        source_idx = node_map.get(edge.source_node)
-        target_idx = node_map.get(edge.target_node)
+        source_idx = node_map.get(edge.source_node.lower())
+        target_idx = node_map.get(edge.target_node.lower())
         if source_idx is not None and target_idx is not None:
             rx_graph.add_edge(source_idx, target_idx, edge.model_dump())
 

{sql_glider-0.1.6 → sql_glider-0.1.8}/tests/sqlglider/graph/test_builder.py

@@ -578,3 +578,56 @@ class TestGraphBuilderCreateViewWithCTEAndWindowFunction:
         assert "raw_sales.customer_id" in node_ids
         assert "raw_sales.product_id" in node_ids
         assert "raw_sales.quantity" in node_ids
+
+
+class TestGraphBuilderCaseInsensitive:
+    """Tests for case-insensitive node deduplication during graph build."""
+
+    def test_nodes_deduplicated_across_files_different_case(self, tmp_path):
+        """Nodes with different casing should be deduplicated into one node."""
+        file1 = tmp_path / "query1.sql"
+        file1.write_text("SELECT Customer_ID FROM Orders;")
+
+        file2 = tmp_path / "query2.sql"
+        file2.write_text("SELECT customer_id FROM orders;")
+
+        builder = GraphBuilder()
+        builder.add_file(file1)
+        builder.add_file(file2)
+        graph = builder.build()
+
+        # Should have deduplicated nodes regardless of case
+        identifiers_lower = [n.identifier.lower() for n in graph.nodes]
+        assert identifiers_lower.count("orders.customer_id") == 1
+
+    def test_edges_deduplicated_across_files_different_case(self, tmp_path):
+        """Edges with different casing should be deduplicated."""
+        file1 = tmp_path / "query1.sql"
+        file1.write_text("SELECT Customer_Name FROM Customers;")
+
+        file2 = tmp_path / "query2.sql"
+        file2.write_text("SELECT customer_name FROM customers;")
+
+        builder = GraphBuilder()
+        builder.add_file(file1)
+        builder.add_file(file2)
+        graph = builder.build()
+
+        # Edges should be deduplicated
+        edge_pairs = [
+            (e.source_node.lower(), e.target_node.lower()) for e in graph.edges
+        ]
+        for pair in edge_pairs:
+            assert edge_pairs.count(pair) == 1
+
+    def test_node_index_map_uses_lowercase_keys(self, tmp_path):
+        """The internal node index map should use lowercase keys."""
+        sql_file = tmp_path / "query.sql"
+        sql_file.write_text("SELECT Customer_ID FROM Orders;")
+
+        builder = GraphBuilder()
+        builder.add_file(sql_file)
+
+        # All keys in the node index map should be lowercase
+        for key in builder.node_index_map:
+            assert key == key.lower(), f"Node index map key '{key}' is not lowercase"

{sql_glider-0.1.6 → sql_glider-0.1.8}/tests/sqlglider/graph/test_query.py

@@ -196,7 +196,7 @@ class TestGraphQuerierUpstream:
         # Query with different case
         result = querier.find_upstream("TARGET.COLUMN")
 
-        assert result.query_column == "Target.Column"  # Original case preserved
+        assert result.query_column == "target.column"  # Normalized to lowercase
         assert len(result) == 1
 
 
@@ -302,7 +302,7 @@ class TestGraphQuerierDownstream:
         # Query with different case
         result = querier.find_downstream("source.column")
 
-        assert result.query_column == "Source.Column"  # Original case preserved
+        assert result.query_column == "source.column"  # Normalized to lowercase
         assert len(result) == 1