code-review-graph-codeblackwell 2.3.6.post1__tar.gz → 2.4.0__tar.gz

{code_review_graph_codeblackwell-2.3.6.post1 → code_review_graph_codeblackwell-2.4.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: code-review-graph-codeblackwell
-Version: 2.3.6.post1
+Version: 2.4.0
 Summary: Local-first knowledge graph for token-efficient code review through MCP and CLI. Fork of tirth8205/code-review-graph with additional fixes; CLI and MCP server remain `code-review-graph`.
 Project-URL: Homepage, https://github.com/CodeBlackwell/code-review-graph-codeblackwell
 Project-URL: Repository, https://github.com/CodeBlackwell/code-review-graph-codeblackwell
@@ -191,7 +191,7 @@ Large monorepos are where token waste is most painful. The graph cuts through th
   <img src="diagrams/diagram9_language_coverage.png" alt="Language coverage organized by category: Web, Backend, Systems, Mobile, Scripting, Config, plus Jupyter and Databricks notebook support" width="90%" />
 </p>
 
-Parser support covers functions, classes, imports, call sites, inheritance, and test detection across the current parser surface, using Tree-sitter where available and targeted fallbacks where needed. Current support includes Python, JavaScript/TypeScript/TSX, Go, Rust, Java, C/C++, C#, Ruby, Kotlin, Swift, PHP, Scala, Solidity, Dart, R, Perl, Lua/Luau, Objective-C, shell scripts, Elixir, Zig, PowerShell, Julia, ReScript, GDScript, Nix, Verilog/SystemVerilog, SQL, Vue/Svelte SFCs, Astro files parsed through the TypeScript parser, Jupyter/Databricks notebooks (`.ipynb`), and Perl XS files (`.xs`).
+Parser support covers functions, classes, imports, call sites, inheritance, and test detection across the current parser surface, using Tree-sitter where available and targeted fallbacks where needed. Current support includes Python, JavaScript/TypeScript/TSX, Go, Rust, Java, C/C++, C#, Ruby, Kotlin, Swift, PHP, Scala, Solidity, Dart, R, Perl, Lua/Luau, Objective-C, shell scripts, Elixir, Zig, PowerShell, Julia, ReScript, GDScript, Nix, Verilog/SystemVerilog, SQL, CSS/SCSS/LESS stylesheets, Vue/Svelte SFCs (including `<style>` blocks), Astro files parsed through the TypeScript parser, Jupyter/Databricks notebooks (`.ipynb`), and Perl XS files (`.xs`).
 
 ### Add your own language (no fork needed)
 
@@ -320,7 +320,7 @@ The benchmark also runs an honest **co-change mode**: the predictor is seeded wi
 | Feature | Details |
 |---------|---------|
 | **Incremental updates** | Re-parses only changed files. Subsequent updates complete in under 2 seconds. |
-| **Broad language + notebook support** | Python, JavaScript/TypeScript/TSX, Go, Rust, Java, C/C++, C#, Ruby, Kotlin, Swift, PHP, Scala, Solidity, Dart, R, Perl, Lua/Luau, Objective-C, shell scripts, Elixir, Zig, PowerShell, Julia, ReScript, GDScript, Nix, Verilog/SystemVerilog, SQL, Vue/Svelte SFCs, Astro files parsed through the TypeScript parser, Jupyter/Databricks (.ipynb), and Perl XS (.xs) |
+| **Broad language + notebook support** | Python, JavaScript/TypeScript/TSX, Go, Rust, Java, C/C++, C#, Ruby, Kotlin, Swift, PHP, Scala, Solidity, Dart, R, Perl, Lua/Luau, Objective-C, shell scripts, Elixir, Zig, PowerShell, Julia, ReScript, GDScript, Nix, Verilog/SystemVerilog, SQL, CSS/SCSS/LESS, Vue/Svelte SFCs (including `<style>` blocks), Astro files parsed through the TypeScript parser, Jupyter/Databricks (.ipynb), and Perl XS (.xs) |
 | **Blast-radius analysis** | Shows which functions, classes, and files are likely affected by a change |
 | **Auto-update hooks** | Hooks and watch mode can update the graph on file saves and supported commit hooks |
 | **Semantic search** | Optional vector embeddings via sentence-transformers, Google Gemini, MiniMax, or any OpenAI-compatible endpoint (real OpenAI, Azure, new-api, LiteLLM, vLLM, LocalAI) |

{code_review_graph_codeblackwell-2.3.6.post1 → code_review_graph_codeblackwell-2.4.0}/README.md

@@ -129,7 +129,7 @@ Large monorepos are where token waste is most painful. The graph cuts through th
   <img src="diagrams/diagram9_language_coverage.png" alt="Language coverage organized by category: Web, Backend, Systems, Mobile, Scripting, Config, plus Jupyter and Databricks notebook support" width="90%" />
 </p>
 
-Parser support covers functions, classes, imports, call sites, inheritance, and test detection across the current parser surface, using Tree-sitter where available and targeted fallbacks where needed. Current support includes Python, JavaScript/TypeScript/TSX, Go, Rust, Java, C/C++, C#, Ruby, Kotlin, Swift, PHP, Scala, Solidity, Dart, R, Perl, Lua/Luau, Objective-C, shell scripts, Elixir, Zig, PowerShell, Julia, ReScript, GDScript, Nix, Verilog/SystemVerilog, SQL, Vue/Svelte SFCs, Astro files parsed through the TypeScript parser, Jupyter/Databricks notebooks (`.ipynb`), and Perl XS files (`.xs`).
+Parser support covers functions, classes, imports, call sites, inheritance, and test detection across the current parser surface, using Tree-sitter where available and targeted fallbacks where needed. Current support includes Python, JavaScript/TypeScript/TSX, Go, Rust, Java, C/C++, C#, Ruby, Kotlin, Swift, PHP, Scala, Solidity, Dart, R, Perl, Lua/Luau, Objective-C, shell scripts, Elixir, Zig, PowerShell, Julia, ReScript, GDScript, Nix, Verilog/SystemVerilog, SQL, CSS/SCSS/LESS stylesheets, Vue/Svelte SFCs (including `<style>` blocks), Astro files parsed through the TypeScript parser, Jupyter/Databricks notebooks (`.ipynb`), and Perl XS files (`.xs`).
 
 ### Add your own language (no fork needed)
 
@@ -258,7 +258,7 @@ The benchmark also runs an honest **co-change mode**: the predictor is seeded wi
 | Feature | Details |
 |---------|---------|
 | **Incremental updates** | Re-parses only changed files. Subsequent updates complete in under 2 seconds. |
-| **Broad language + notebook support** | Python, JavaScript/TypeScript/TSX, Go, Rust, Java, C/C++, C#, Ruby, Kotlin, Swift, PHP, Scala, Solidity, Dart, R, Perl, Lua/Luau, Objective-C, shell scripts, Elixir, Zig, PowerShell, Julia, ReScript, GDScript, Nix, Verilog/SystemVerilog, SQL, Vue/Svelte SFCs, Astro files parsed through the TypeScript parser, Jupyter/Databricks (.ipynb), and Perl XS (.xs) |
+| **Broad language + notebook support** | Python, JavaScript/TypeScript/TSX, Go, Rust, Java, C/C++, C#, Ruby, Kotlin, Swift, PHP, Scala, Solidity, Dart, R, Perl, Lua/Luau, Objective-C, shell scripts, Elixir, Zig, PowerShell, Julia, ReScript, GDScript, Nix, Verilog/SystemVerilog, SQL, CSS/SCSS/LESS, Vue/Svelte SFCs (including `<style>` blocks), Astro files parsed through the TypeScript parser, Jupyter/Databricks (.ipynb), and Perl XS (.xs) |
 | **Blast-radius analysis** | Shows which functions, classes, and files are likely affected by a change |
 | **Auto-update hooks** | Hooks and watch mode can update the graph on file saves and supported commit hooks |
 | **Semantic search** | Optional vector embeddings via sentence-transformers, Google Gemini, MiniMax, or any OpenAI-compatible endpoint (real OpenAI, Azure, new-api, LiteLLM, vLLM, LocalAI) |

{code_review_graph_codeblackwell-2.3.6.post1 → code_review_graph_codeblackwell-2.4.0}/code_review_graph/graph.py

@@ -1,7 +1,8 @@
 """SQLite-backed knowledge graph storage and query engine.
 
 Stores code structure as nodes (File, Class, Function, Type, Test) and
-edges (CALLS, IMPORTS_FROM, INHERITS, IMPLEMENTS, CONTAINS, TESTED_BY, DEPENDS_ON, REFERENCES).
+edges (CALLS, IMPORTS_FROM, INHERITS, IMPLEMENTS, CONTAINS, TESTED_BY, DEPENDS_ON,
+REFERENCES, OVERRIDES, STYLES, POTENTIAL_CONFLICT).
 Supports impact-radius queries and subgraph extraction.
 """
 
@@ -1310,6 +1311,32 @@ class GraphStore:
             nodes_by_id=nodes_by_id,
         )
 
+    def delete_edges_by_kind(self, kind: str) -> int:
+        """Delete all edges of the given kind. Returns count of deleted rows."""
+        cur = self._conn.execute("DELETE FROM edges WHERE kind = ?", (kind,))
+        self._invalidate_cache()
+        return cur.rowcount
+
+    def get_nodes_by_extra_pattern(
+        self, pattern: str, kind: str | None = None,
+    ) -> list[GraphNode]:
+        """Find nodes where extra JSON matches a LIKE pattern.
+
+        Args:
+            pattern: SQL LIKE pattern, e.g. '%"css_kind":"selector"%'.
+            kind: Optional node kind filter (Class, Function, etc.).
+        """
+        if kind:
+            rows = self._conn.execute(
+                "SELECT * FROM nodes WHERE extra LIKE ? AND kind = ?",
+                (pattern, kind),
+            ).fetchall()
+        else:
+            rows = self._conn.execute(
+                "SELECT * FROM nodes WHERE extra LIKE ?", (pattern,),
+            ).fetchall()
+        return [self._row_to_node(r) for r in rows]
+
     # --- Internal helpers ---
 
     def _build_networkx_graph(self) -> nx.DiGraph:
@@ -1406,6 +1433,21 @@ def _sanitize_name(s: str, max_len: int = 256) -> str:
     return cleaned[:max_len]
 
 
+def _sanitize_extra(extra: dict) -> dict:
+    """Sanitize string values in extra dict for safe display."""
+    if not extra:
+        return {}
+
+    def _sanitize_value(v):  # type: ignore[no-untyped-def]
+        if isinstance(v, str):
+            return _sanitize_name(v)
+        if isinstance(v, list):
+            return [_sanitize_value(item) for item in v]
+        return v
+
+    return {k: _sanitize_value(v) for k, v in extra.items()}
+
+
 def node_to_dict(n: GraphNode) -> dict:
     return {
         "id": n.id, "kind": n.kind, "name": _sanitize_name(n.name),
@@ -1414,6 +1456,7 @@ def node_to_dict(n: GraphNode) -> dict:
         "language": n.language,
         "parent_name": _sanitize_name(n.parent_name) if n.parent_name else n.parent_name,
         "is_test": n.is_test,
+        "extra": _sanitize_extra(n.extra),
     }
 
 
@@ -1424,4 +1467,5 @@ def edge_to_dict(e: GraphEdge) -> dict:
         "target": _sanitize_name(e.target_qualified),
         "file_path": e.file_path, "line": e.line,
         "confidence": e.confidence, "confidence_tier": e.confidence_tier,
+        "extra": _sanitize_extra(e.extra),
     }

{code_review_graph_codeblackwell-2.3.6.post1 → code_review_graph_codeblackwell-2.4.0}/code_review_graph/incremental.py

@@ -82,6 +82,30 @@ def _run_spring_resolver(store: GraphStore) -> Optional[dict]:
         return None
 
 
+def _run_css_resolver(store: GraphStore) -> Optional[dict]:
+    """Run cross-file CSS linking + conflict detection, swallowing any
+    failure so build never fails because of it. Returns stats or None.
+    """
+    try:
+        selectors = _get_css_selectors(store)
+        styles_count = link_css_styles(store, selectors)
+        conflicts_count = detect_cross_file_conflicts(store, selectors)
+        logger.info(
+            "CSS linking: %d STYLES edges, %d POTENTIAL_CONFLICT edges",
+            styles_count, conflicts_count,
+        )
+        return {"styles_edges": styles_count, "conflict_edges": conflicts_count}
+    except Exception as exc:  # noqa: BLE001 - best-effort post-pass
+        logger.warning("CSS resolver failed: %s", exc)
+        return None
+
+
+# File extensions whose changes can affect STYLES/POTENTIAL_CONFLICT edges.
+_CSS_RELEVANT_EXTENSIONS = (
+    ".css", ".scss", ".less", ".vue", ".svelte", ".tsx", ".jsx", ".ts", ".js",
+)
+
+
 def _run_temporal_resolver(store: GraphStore) -> Optional[dict]:
     """Run the Temporal workflow/activity call resolver, swallowing any failure so
     build never fails because of it. Returns stats or None on error.
@@ -728,7 +752,9 @@ def _single_hop_dependents(store: GraphStore, file_path: str) -> set[str]:
     nodes = store.get_nodes_by_file(file_path)
     for node in nodes:
         for e in store.get_edges_by_target(node.qualified_name):
-            if e.kind in ("CALLS", "IMPORTS_FROM", "INHERITS", "IMPLEMENTS"):
+            if e.kind in (
+                "CALLS", "IMPORTS_FROM", "INHERITS", "IMPLEMENTS", "STYLES",
+            ):
                 dependents.add(e.file_path)
 
     dependents.discard(file_path)
@@ -817,6 +843,194 @@ def _parse_single_file(
         return (rel_path, [], [], str(e), "")
 
 
+def _get_css_selectors(store: GraphStore) -> list:
+    """Fetch all CSS selector nodes (css_kind == "selector")."""
+    return store.get_nodes_by_extra_pattern(
+        '%"css_kind":%"selector"%', kind="Class",
+    )
+
+
+def link_css_styles(store: GraphStore, selectors: list | None = None) -> int:
+    """Create STYLES edges between class references and CSS selectors.
+
+    Queries the graph for:
+    1. All CSS selector nodes (css_kind == "selector")
+    2. All nodes with css_classes in extra (JSX functions, Vue/Svelte files)
+
+    Creates STYLES edges from the referencing component/function to
+    the matching CSS selector node.
+
+    Returns count of STYLES edges created.
+    """
+    from .parser import CodeParser, EdgeInfo  # noqa: F811 (avoid circular)
+
+    # Clean up previous STYLES edges
+    store.delete_edges_by_kind("STYLES")
+
+    # Build selector index: bare class name → list of (qualified_name, file_path)
+    if selectors is None:
+        selectors = _get_css_selectors(store)
+    selector_index: dict[str, list[tuple[str, str]]] = {}
+    for sel in selectors:
+        sel_name = sel.name.strip()
+        # Only index class selectors (starting with .)
+        if sel_name.startswith("."):
+            bare = sel_name.lstrip(".")
+            selector_index.setdefault(bare, []).append(
+                (sel.qualified_name, sel.file_path),
+            )
+        # For compound selectors, also index by last class segment
+        parts = sel_name.split()
+        if len(parts) > 1:
+            last = parts[-1].strip()
+            if last.startswith("."):
+                bare = last.lstrip(".")
+                selector_index.setdefault(bare, []).append(
+                    (sel.qualified_name, sel.file_path),
+                )
+
+    if not selector_index:
+        store.commit()
+        return 0
+
+    count = 0
+
+    # Link static css_classes references
+    class_nodes = store.get_nodes_by_extra_pattern('%"css_classes"%')
+    for node in class_nodes:
+        classes = node.extra.get("css_classes", [])
+        for cls_name in classes:
+            matches = selector_index.get(cls_name, [])
+            for sel_qn, sel_file in matches:
+                store.upsert_edge(EdgeInfo(
+                    kind="STYLES",
+                    source=node.qualified_name,
+                    target=sel_qn,
+                    file_path=node.file_path,
+                    line=node.line_start or 0,
+                    extra={
+                        "class_name": cls_name,
+                        "resolution": "static",
+                    },
+                ))
+                count += 1
+
+    # Link CSS Module references
+    module_nodes = store.get_nodes_by_extra_pattern('%"css_module_refs"%')
+    # Map of file_path → {import name: module path}, built once for all nodes
+    file_mod_imports: dict[str, dict[str, str]] = {}
+    if module_nodes:
+        for fn in store.get_nodes_by_extra_pattern(
+            '%"css_module_imports"%', kind="File",
+        ):
+            file_mod_imports[fn.file_path] = fn.extra.get(
+                "css_module_imports", {},
+            )
+    for node in module_nodes:
+        refs = node.extra.get("css_module_refs", [])
+        mod_imports = file_mod_imports.get(node.file_path, {})
+        for ref in refs:
+            imp_name = ref.get("import", "")
+            prop_name = ref.get("property", "")
+            if not imp_name or not prop_name:
+                continue
+            # Find the module path
+            if imp_name not in mod_imports:
+                continue
+            # Convert camelCase to kebab-case for selector lookup
+            kebab = CodeParser._camel_to_kebab(prop_name)
+            matches = selector_index.get(kebab, [])
+            for sel_qn, sel_file in matches:
+                store.upsert_edge(EdgeInfo(
+                    kind="STYLES",
+                    source=node.qualified_name,
+                    target=sel_qn,
+                    file_path=node.file_path,
+                    line=node.line_start or 0,
+                    extra={
+                        "class_name": kebab,
+                        "resolution": "css_module",
+                        "import_name": imp_name,
+                        "property": prop_name,
+                    },
+                ))
+                count += 1
+
+    store.commit()
+    return count
+
+
+def detect_cross_file_conflicts(
+    store: GraphStore, selectors: list | None = None,
+) -> int:
+    """Detect potential CSS conflicts across files.
+
+    Scans for CSS selectors across different files that target the same
+    class name. Creates POTENTIAL_CONFLICT edges when found.
+
+    Returns count of conflict edges created.
+    """
+    from .parser import EdgeInfo  # noqa: F811
+
+    store.delete_edges_by_kind("POTENTIAL_CONFLICT")
+
+    if selectors is None:
+        selectors = _get_css_selectors(store)
+
+    # Group by bare class name
+    by_class: dict[str, list[tuple[str, str, list]]] = {}
+    for sel in selectors:
+        sel_name = sel.name.strip()
+        if not sel_name.startswith("."):
+            continue
+        bare = sel_name.lstrip(".")
+        specificity = sel.extra.get("specificity", [0, 0, 0])
+        by_class.setdefault(bare, []).append(
+            (sel.qualified_name, sel.file_path, specificity),
+        )
+
+    count = 0
+    max_pairs_per_class = 50
+
+    for bare, entries in by_class.items():
+        # Only flag if selectors appear in 2+ different files
+        files_seen = {fp for _, fp, _ in entries}
+        if len(files_seen) < 2:
+            continue
+
+        pairs = 0
+        for i, (src_qn, src_fp, src_spec) in enumerate(entries):
+            for tgt_qn, tgt_fp, tgt_spec in entries[i + 1:]:
+                if src_fp == tgt_fp:
+                    continue
+                if pairs >= max_pairs_per_class:
+                    break
+                # Conflict confidence label; keyed as conflict_confidence
+                # because extra["confidence"] is reserved for the numeric
+                # edge-confidence score read by GraphStore.upsert_edge.
+                conflict_confidence = "high"  # same bare class name
+                store.upsert_edge(EdgeInfo(
+                    kind="POTENTIAL_CONFLICT",
+                    source=src_qn,
+                    target=tgt_qn,
+                    file_path="<cross-file>",
+                    line=0,
+                    extra={
+                        "class_name": f".{bare}",
+                        "source_specificity": src_spec,
+                        "target_specificity": tgt_spec,
+                        "conflict_confidence": conflict_confidence,
+                    },
+                ))
+                count += 1
+                pairs += 1
+            if pairs >= max_pairs_per_class:
+                break
+
+    store.commit()
+    return count
+
+
 def full_build(
     repo_root: Path,
     store: GraphStore,
@@ -901,6 +1115,7 @@ def full_build(
     _store_vcs_metadata(repo_root, store)
     store.commit()
 
+    css_stats = _run_css_resolver(store)
     rescript_stats = _run_rescript_resolver(store)
     spring_stats = _run_spring_resolver(store)
     temporal_stats = _run_temporal_resolver(store)
@@ -917,6 +1132,8 @@ def full_build(
         "total_nodes": total_nodes,
         "total_edges": total_edges,
         "disambiguated_nodes": disambiguated,
+        "styles_edges": (css_stats or {}).get("styles_edges"),
+        "conflict_edges": (css_stats or {}).get("conflict_edges"),
         "errors": errors,
         "rescript_resolution": rescript_stats,
         "spring_resolution": spring_stats,
@@ -1040,6 +1257,11 @@ def incremental_update(
     store.commit()
 
     # Only re-run language-specific resolvers when the relevant files changed.
+    css_changed = any(
+        rp.endswith(_CSS_RELEVANT_EXTENSIONS) for rp in all_files
+    )
+    css_stats = _run_css_resolver(store) if css_changed else None
+
     rescript_changed = any(
         rp.endswith((".res", ".resi")) for rp in all_files
     )
@@ -1058,6 +1280,8 @@ def incremental_update(
         "total_nodes": total_nodes,
         "total_edges": total_edges,
         "disambiguated_nodes": disambiguated,
+        "styles_edges": (css_stats or {}).get("styles_edges"),
+        "conflict_edges": (css_stats or {}).get("conflict_edges"),
         "changed_files": list(changed_files),
         "dependent_files": list(dependent_files),
         "errors": errors,

{code_review_graph_codeblackwell-2.3.6.post1 → code_review_graph_codeblackwell-2.4.0}/code_review_graph/main.py

@@ -229,6 +229,11 @@ def query_graph_tool(
     - children_of: Find nodes contained in a file or class
     - tests_for: Find tests for the target
     - inheritors_of: Find classes inheriting from the target
+    - overrides_of: Find CSS selectors that the target overrides
+    - overridden_by: Find CSS selectors that override the target
+    - styles_of: Find CSS selectors that style a component or function
+    - styled_by: Find components/functions using a given CSS selector
+    - conflicts_of: Find potential cross-file CSS conflicts for a selector
     - file_summary: Get all nodes in a file
 
     Args: