code-review-graph 2.2.3__tar.gz → 2.2.4__tar.gz

{code_review_graph-2.2.3 → code_review_graph-2.2.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: code-review-graph
-Version: 2.2.3
+Version: 2.2.4
 Summary: Persistent incremental knowledge graph for token-efficient, context-aware code reviews with Claude Code
 Project-URL: Homepage, https://code-review-graph.com
 Project-URL: Repository, https://github.com/tirth8205/code-review-graph
@@ -21,7 +21,7 @@ Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
 Classifier: Topic :: Software Development :: Quality Assurance
 Requires-Python: >=3.10
-Requires-Dist: fastmcp<2,>=0.1.0
+Requires-Dist: fastmcp<3,>=2.14.0
 Requires-Dist: mcp<2,>=1.0.0
 Requires-Dist: networkx<4,>=3.2
 Requires-Dist: tree-sitter-language-pack<1,>=0.3.0

{code_review_graph-2.2.3 → code_review_graph-2.2.4}/code_review_graph/cli.py

@@ -32,6 +32,7 @@ import argparse
 import json
 import logging
 import os
+from importlib.metadata import PackageNotFoundError
 from importlib.metadata import version as pkg_version
 from pathlib import Path
 
@@ -40,7 +41,7 @@ def _get_version() -> str:
     """Get the installed package version."""
     try:
         return pkg_version("code-review-graph")
-    except Exception:
+    except PackageNotFoundError:
         return "dev"
 
 

{code_review_graph-2.2.3 → code_review_graph-2.2.4}/code_review_graph/eval/benchmarks/search_quality.py

@@ -3,6 +3,7 @@
 from __future__ import annotations
 
 import logging
+import sqlite3
 from pathlib import Path
 
 logger = logging.getLogger(__name__)
@@ -18,7 +19,8 @@ def run(repo_path: Path, store, config: dict) -> list[dict]:
         try:
             from code_review_graph.search import hybrid_search
             search_results = hybrid_search(store, query, limit=20)
-        except Exception:
+        except (ImportError, sqlite3.OperationalError) as exc:
+            logger.debug("hybrid_search unavailable, using fallback: %s", exc)
             # Fallback to basic search
             search_results = [
                 {"qualified_name": n.qualified_name}

{code_review_graph-2.2.3 → code_review_graph-2.2.4}/code_review_graph/eval/runner.py

@@ -51,6 +51,7 @@ def load_config(name: str) -> dict:
 
 def load_all_configs() -> list[dict]:
     """Load all benchmark configs from the configs directory."""
+    _require_yaml()
     configs = []
     for p in sorted(CONFIGS_DIR.glob("*.yaml")):
         with open(p) as f:

{code_review_graph-2.2.3 → code_review_graph-2.2.4}/code_review_graph/graph.py

@@ -769,7 +769,9 @@ class GraphStore:
                 r["qualified_name"]: r["community_id"]
                 for r in rows
             }
-        except Exception:
+        except sqlite3.OperationalError as exc:
+            # community_id column may not exist yet on pre-v6 schemas
+            logger.debug("get_all_community_ids: schema not yet migrated (%s)", exc)
             return {}
 
     def get_node_ids_by_files(
@@ -844,7 +846,9 @@ class GraphStore:
             return self._conn.execute(
                 "SELECT id, name FROM communities"
             ).fetchall()
-        except Exception:
+        except sqlite3.OperationalError as exc:
+            # communities table doesn't exist yet on pre-v4 schemas
+            logger.debug("get_communities_raw: table missing (%s)", exc)
             return []
 
     def get_community_member_qns(

{code_review_graph-2.2.3 → code_review_graph-2.2.4}/code_review_graph/incremental.py

@@ -14,7 +14,7 @@ import os
 import re
 import subprocess
 import time
-from pathlib import Path
+from pathlib import Path, PurePosixPath
 from typing import Optional
 
 from .graph import GraphStore
@@ -26,7 +26,11 @@ _MAX_PARSE_WORKERS = int(os.environ.get(
 
 logger = logging.getLogger(__name__)
 
-# Default ignore patterns (in addition to .gitignore)
+# Default ignore patterns (in addition to .gitignore).
+#
+# `<dir>/**` patterns are matched at any depth by _should_ignore, so
+# `node_modules/**` also excludes `packages/app/node_modules/react/index.js`
+# inside monorepos. See: #91
 DEFAULT_IGNORE_PATTERNS = [
     ".code-review-graph/**",
     "node_modules/**",
@@ -39,6 +43,21 @@ DEFAULT_IGNORE_PATTERNS = [
     "build/**",
     ".next/**",
     "target/**",
+    # PHP / Laravel / Composer
+    "vendor/**",
+    "bootstrap/cache/**",
+    "public/build/**",
+    # Ruby / Bundler
+    ".bundle/**",
+    # Java / Kotlin / Gradle
+    ".gradle/**",
+    "*.jar",
+    # Dart / Flutter
+    ".dart_tool/**",
+    ".pub-cache/**",
+    # General
+    "coverage/**",
+    ".cache/**",
     "*.min.js",
     "*.min.css",
     "*.map",
@@ -148,8 +167,31 @@ def _load_ignore_patterns(repo_root: Path) -> list[str]:
 
 
 def _should_ignore(path: str, patterns: list[str]) -> bool:
-    """Check if a path matches any ignore pattern."""
-    return any(fnmatch.fnmatch(path, p) for p in patterns)
+    """Check if a path matches any ignore pattern.
+
+    Handles nested occurrences of ``<dir>/**`` patterns: for example,
+    ``node_modules/**`` also matches ``packages/app/node_modules/foo.js``
+    inside monorepos. ``fnmatch`` alone treats ``*`` as not crossing ``/``
+    and only matches the prefix, so we additionally test each path segment
+    against the bare prefix of ``<dir>/**`` patterns. See: #91
+    """
+    # Direct fnmatch first (cheap)
+    if any(fnmatch.fnmatch(path, p) for p in patterns):
+        return True
+    # Then: treat simple single-segment "dir/**" patterns as
+    # "this directory at any depth".
+    parts = PurePosixPath(path).parts
+    for p in patterns:
+        if not p.endswith("/**"):
+            continue
+        prefix = p[:-3]
+        # Only single-segment dir patterns (no "/" inside the prefix)
+        # qualify for nested matching.
+        if "/" in prefix or not prefix:
+            continue
+        if prefix in parts:
+            return True
+    return False
 
 
 def _is_binary(path: Path) -> bool:

{code_review_graph-2.2.3 → code_review_graph-2.2.4}/code_review_graph/main.py

@@ -6,6 +6,7 @@ Communicates via stdio (standard MCP transport).
 
 from __future__ import annotations
 
+import sys
 from typing import Optional
 
 from fastmcp import FastMCP
@@ -48,6 +49,23 @@ from .tools import (
 # transport with concurrent requests, replace with contextvars.ContextVar.
 _default_repo_root: str | None = None
 
+
+def _resolve_repo_root(repo_root: Optional[str]) -> Optional[str]:
+    """Resolve repo_root for a tool call.
+
+    Order of precedence:
+    1. Explicit ``repo_root`` passed by the MCP client (highest).
+    2. ``--repo`` CLI flag passed to ``code-review-graph serve``
+       (captured in ``_default_repo_root``).
+    3. None — the underlying impl will fall back to the server's cwd.
+
+    Previously, only ``get_docs_section_tool`` consulted ``_default_repo_root``,
+    so ``serve --repo <X>`` had no effect for the other 21 tools. See: #222
+    follow-up.
+    """
+    return repo_root if repo_root else _default_repo_root
+
+
 mcp = FastMCP(
     "code-review-graph",
     instructions=(
@@ -82,7 +100,7 @@ def build_or_update_graph_tool(
             When None (default), falls back to CRG_RECURSE_SUBMODULES env var.
     """
     return build_or_update_graph(
-        full_rebuild=full_rebuild, repo_root=repo_root, base=base,
+        full_rebuild=full_rebuild, repo_root=_resolve_repo_root(repo_root), base=base,
         postprocess=postprocess, recurse_submodules=recurse_submodules,
     )
 
@@ -106,7 +124,7 @@ def run_postprocess_tool(
         repo_root: Repository root path. Auto-detected if omitted.
     """
     return run_postprocess(
-        flows=flows, communities=communities, fts=fts, repo_root=repo_root,
+        flows=flows, communities=communities, fts=fts, repo_root=_resolve_repo_root(repo_root),
     )
 
 
@@ -131,7 +149,7 @@ def get_minimal_context_tool(
     """
     return get_minimal_context(
         task=task, changed_files=changed_files,
-        repo_root=repo_root, base=base,
+        repo_root=_resolve_repo_root(repo_root), base=base,
     )
 
 
@@ -157,7 +175,7 @@ def get_impact_radius_tool(
     """
     return get_impact_radius(
         changed_files=changed_files, max_depth=max_depth,
-        repo_root=repo_root, base=base, detail_level=detail_level,
+        repo_root=_resolve_repo_root(repo_root), base=base, detail_level=detail_level,
     )
 
 
@@ -187,7 +205,7 @@ def query_graph_tool(
         detail_level: "standard" for full output, "minimal" for compact summary. Default: standard.
     """
     return query_graph(
-        pattern=pattern, target=target, repo_root=repo_root,
+        pattern=pattern, target=target, repo_root=_resolve_repo_root(repo_root),
         detail_level=detail_level,
     )
 
@@ -220,7 +238,7 @@ def get_review_context_tool(
     return get_review_context(
         changed_files=changed_files, max_depth=max_depth,
         include_source=include_source, max_lines_per_file=max_lines_per_file,
-        repo_root=repo_root, base=base, detail_level=detail_level,
+        repo_root=_resolve_repo_root(repo_root), base=base, detail_level=detail_level,
     )
 
 
@@ -249,7 +267,7 @@ def semantic_search_nodes_tool(
         detail_level: "standard" for full output, "minimal" for compact summary. Default: standard.
     """
     return semantic_search_nodes(
-        query=query, kind=kind, limit=limit, repo_root=repo_root, model=model,
+        query=query, kind=kind, limit=limit, repo_root=_resolve_repo_root(repo_root), model=model,
         detail_level=detail_level,
     )
 
@@ -274,7 +292,7 @@ def embed_graph_tool(
         model: Embedding model name (HuggingFace ID or local path).
                Falls back to CRG_EMBEDDING_MODEL env var, then all-MiniLM-L6-v2.
     """
-    return embed_graph(repo_root=repo_root, model=model)
+    return embed_graph(repo_root=_resolve_repo_root(repo_root), model=model)
 
 
 @mcp.tool()
@@ -289,7 +307,7 @@ def list_graph_stats_tool(
     Args:
         repo_root: Repository root path. Auto-detected if omitted.
     """
-    return list_graph_stats(repo_root=repo_root)
+    return list_graph_stats(repo_root=_resolve_repo_root(repo_root))
 
 
 @mcp.tool()
@@ -332,7 +350,7 @@ def find_large_functions_tool(
     """
     return find_large_functions(
         min_lines=min_lines, kind=kind, file_path_pattern=file_path_pattern,
-        limit=limit, repo_root=repo_root,
+        limit=limit, repo_root=_resolve_repo_root(repo_root),
     )
 
 
@@ -359,7 +377,7 @@ def list_flows_tool(
         repo_root: Repository root path. Auto-detected if omitted.
     """
     return list_flows(
-        repo_root=repo_root, sort_by=sort_by, limit=limit, kind=kind,
+        repo_root=_resolve_repo_root(repo_root), sort_by=sort_by, limit=limit, kind=kind,
         detail_level=detail_level,
     )
 
@@ -386,7 +404,7 @@ def get_flow_tool(
     """
     return get_flow(
         flow_id=flow_id, flow_name=flow_name,
-        include_source=include_source, repo_root=repo_root,
+        include_source=include_source, repo_root=_resolve_repo_root(repo_root),
     )
 
 
@@ -408,7 +426,7 @@ def get_affected_flows_tool(
         repo_root: Repository root path. Auto-detected if omitted.
     """
     return get_affected_flows_func(
-        changed_files=changed_files, base=base, repo_root=repo_root,
+        changed_files=changed_files, base=base, repo_root=_resolve_repo_root(repo_root),
     )
 
 
@@ -434,7 +452,7 @@ def list_communities_tool(
         repo_root: Repository root path. Auto-detected if omitted.
     """
     return list_communities_func(
-        repo_root=repo_root, sort_by=sort_by, min_size=min_size,
+        repo_root=_resolve_repo_root(repo_root), sort_by=sort_by, min_size=min_size,
         detail_level=detail_level,
     )
 
@@ -462,7 +480,7 @@ def get_community_tool(
     """
     return get_community_func(
         community_name=community_name, community_id=community_id,
-        include_members=include_members, repo_root=repo_root,
+        include_members=include_members, repo_root=_resolve_repo_root(repo_root),
     )
 
 
@@ -479,7 +497,7 @@ def get_architecture_overview_tool(
     Args:
         repo_root: Repository root path. Auto-detected if omitted.
     """
-    return get_architecture_overview_func(repo_root=repo_root)
+    return get_architecture_overview_func(repo_root=_resolve_repo_root(repo_root))
 
 
 @mcp.tool()
@@ -509,7 +527,7 @@ def detect_changes_tool(
     return detect_changes_func(
         base=base, changed_files=changed_files,
         include_source=include_source, max_depth=max_depth,
-        repo_root=repo_root, detail_level=detail_level,
+        repo_root=_resolve_repo_root(repo_root), detail_level=detail_level,
     )
 
 
@@ -545,7 +563,7 @@ def refactor_tool(
     """
     return refactor_func(
         mode=mode, old_name=old_name, new_name=new_name,
-        kind=kind, file_pattern=file_pattern, repo_root=repo_root,
+        kind=kind, file_pattern=file_pattern, repo_root=_resolve_repo_root(repo_root),
     )
 
 
@@ -568,7 +586,7 @@ def apply_refactor_tool(
         repo_root: Repository root path. Auto-detected if omitted.
     """
     return apply_refactor_func(
-        refactor_id=refactor_id, repo_root=repo_root,
+        refactor_id=refactor_id, repo_root=_resolve_repo_root(repo_root),
     )
 
 
@@ -587,7 +605,7 @@ def generate_wiki_tool(
         repo_root: Repository root path. Auto-detected if omitted.
         force: If True, regenerate all pages even if content unchanged. Default: False.
     """
-    return generate_wiki_func(repo_root=repo_root, force=force)
+    return generate_wiki_func(repo_root=_resolve_repo_root(repo_root), force=force)
 
 
 @mcp.tool()
@@ -604,7 +622,9 @@ def get_wiki_page_tool(
         community_name: Community name to look up.
         repo_root: Repository root path. Auto-detected if omitted.
     """
-    return get_wiki_page_func(community_name=community_name, repo_root=repo_root)
+    return get_wiki_page_func(
+        community_name=community_name, repo_root=_resolve_repo_root(repo_root),
+    )
 
 
 @mcp.tool()
@@ -691,9 +711,21 @@ def pre_merge_check(base: str = "HEAD~1") -> list[dict]:
 
 
 def main(repo_root: str | None = None) -> None:
-    """Run the MCP server via stdio."""
+    """Run the MCP server via stdio.
+
+    On Windows, Python 3.8+ defaults to ``ProactorEventLoop``, which
+    interacts poorly with ``concurrent.futures.ProcessPoolExecutor``
+    (used by ``full_build``) over a stdio MCP transport — the combination
+    produces silent hangs on ``build_or_update_graph_tool`` and
+    ``embed_graph_tool``. Switching to ``WindowsSelectorEventLoopPolicy``
+    before fastmcp starts its loop avoids the deadlock.
+    See: #46, #136
+    """
     global _default_repo_root
     _default_repo_root = repo_root
+    if sys.platform == "win32":
+        import asyncio
+        asyncio.set_event_loop_policy(asyncio.WindowsSelectorEventLoopPolicy())
     mcp.run(transport="stdio")
 
 

{code_review_graph-2.2.3 → code_review_graph-2.2.4}/code_review_graph/migrations.py

@@ -238,7 +238,7 @@ def run_migrations(conn: sqlite3.Connection) -> None:
             MIGRATIONS[version](conn)
             _set_schema_version(conn, version)
             conn.commit()
-        except Exception:
+        except sqlite3.Error:
             conn.rollback()
             logger.error("Migration v%d failed, rolling back", version, exc_info=True)
             raise

{code_review_graph-2.2.3 → code_review_graph-2.2.4}/code_review_graph/parser.py

@@ -304,12 +304,16 @@ class CodeParser:
         self._module_file_cache: dict[str, Optional[str]] = {}
         self._export_symbol_cache: dict[str, Optional[str]] = {}
         self._tsconfig_resolver = TsconfigResolver()
+        # Per-parse cache of Dart pubspec root lookups; see #87
+        self._dart_pubspec_cache: dict[tuple[str, str], Optional[Path]] = {}
 
     def _get_parser(self, language: str):  # type: ignore[arg-type]
         if language not in self._parsers:
             try:
                 self._parsers[language] = tslp.get_parser(language)  # type: ignore[arg-type]
-            except Exception:
+            except (LookupError, ValueError, ImportError) as exc:
+                # language not packaged, or grammar load failed
+                logger.debug("tree-sitter parser unavailable for %s: %s", language, exc)
                 return None
         return self._parsers[language]
 
@@ -924,6 +928,18 @@ class CodeParser:
             ):
                 continue
 
+            # --- Dart call detection (see #87) ---
+            # tree-sitter-dart does not wrap calls in a single
+            # ``call_expression`` node; instead the pattern is
+            # ``identifier + selector > argument_part`` as siblings inside
+            # the parent.  Scan child's children here and emit CALLS edges
+            # for any we find; nested calls are handled by the main recursion.
+            if language == "dart":
+                self._extract_dart_calls_from_children(
+                    child, source, file_path, edges,
+                    enclosing_class, enclosing_func,
+                )
+
             # --- JS/TS variable-assigned functions (const foo = () => {}) ---
             if (
                 language in ("javascript", "typescript", "tsx")
@@ -1014,6 +1030,84 @@ class CodeParser:
                 _depth=_depth + 1,
             )
 
+    def _extract_dart_calls_from_children(
+        self,
+        parent,
+        source: bytes,
+        file_path: str,
+        edges: list[EdgeInfo],
+        enclosing_class: Optional[str],
+        enclosing_func: Optional[str],
+    ) -> None:
+        """Detect Dart call sites from a parent node's children (#87 bug 1).
+
+        tree-sitter-dart does not emit a single ``call_expression`` node for
+        Dart calls.  Instead it produces ``identifier`` / method-selector
+        siblings followed by a ``selector`` whose child is ``argument_part``:
+
+            identifier "print"
+            selector
+              argument_part
+
+        And for method calls like ``obj.foo()`` the middle selector is a
+        ``unconditional_assignable_selector`` holding the method name:
+
+            identifier "obj"
+            selector
+              unconditional_assignable_selector "."
+                identifier "foo"
+            selector
+              argument_part
+
+        This walker scans the immediate children of ``parent`` for either
+        shape and emits a ``CALLS`` edge.  Nested calls are picked up as
+        ``_extract_from_tree`` recurses into child nodes.
+        """
+        call_name: Optional[str] = None
+        for sub in parent.children:
+            if sub.type == "identifier":
+                call_name = sub.text.decode("utf-8", errors="replace")
+                continue
+            if sub.type == "selector":
+                # Case A: selector > unconditional_assignable_selector > identifier
+                # (updates call_name to the method name)
+                method_name: Optional[str] = None
+                has_arguments = False
+                for ssub in sub.children:
+                    if ssub.type == "unconditional_assignable_selector":
+                        for ident in ssub.children:
+                            if ident.type == "identifier":
+                                method_name = ident.text.decode(
+                                    "utf-8", errors="replace"
+                                )
+                                break
+                    elif ssub.type == "argument_part":
+                        has_arguments = True
+                if method_name is not None:
+                    call_name = method_name
+                if has_arguments and call_name:
+                    src_qn = (
+                        self._qualify(enclosing_func, file_path, enclosing_class)
+                        if enclosing_func else file_path
+                    )
+                    edges.append(EdgeInfo(
+                        kind="CALLS",
+                        source=src_qn,
+                        target=call_name,
+                        file_path=file_path,
+                        line=parent.start_point[0] + 1,
+                    ))
+                    # After emitting for this call, clear call_name so we
+                    # don't re-emit on any trailing chained selector.
+                    call_name = None
+                continue
+            # Non-identifier, non-selector children don't change the
+            # pending call name (``return``, ``await``, ``yield``, etc.)
+            # but anything unexpected should reset it to avoid spurious
+            # edges across unrelated siblings.
+            if sub.type not in ("return", "await", "yield", "this", "const", "new"):
+                call_name = None
+
     def _extract_r_constructs(
         self,
         child,
@@ -1577,6 +1671,14 @@ class CodeParser:
         if not name:
             return False
 
+        # Go methods: attach to their receiver type as the enclosing class,
+        # so `func (s *T) Foo()` becomes a member of T rather than a
+        # top-level function. See: #190
+        if language == "go" and child.type == "method_declaration":
+            receiver_type = self._get_go_receiver_type(child)
+            if receiver_type:
+                enclosing_class = receiver_type
+
         # Extract annotations/decorators for test detection
         decorators: tuple[str, ...] = ()
         deco_list: list[str] = []
@@ -2469,9 +2571,60 @@ class CodeParser:
                 target = base.with_suffix(".dart")
                 if target.is_file():
                     return str(target.resolve())
+            elif module.startswith("package:"):
+                # ``package:<name>/<sub_path>`` — resolve to the current repo's
+                # ``lib/<sub_path>`` iff a ``pubspec.yaml`` declaring that
+                # package name is found in an ancestor directory. See: #87
+                try:
+                    uri_body = module[len("package:"):]
+                    pkg_name, _, sub_path = uri_body.partition("/")
+                    if not sub_path:
+                        return None
+                    pubspec_root = self._find_dart_pubspec_root(
+                        caller_dir, pkg_name
+                    )
+                    if pubspec_root is not None:
+                        target = pubspec_root / "lib" / sub_path
+                        if target.is_file():
+                            return str(target.resolve())
+                except (OSError, ValueError):
+                    return None
+            # ``dart:core`` / ``dart:async`` etc. are SDK libraries we do
+            # not track; fall through to return None.
 
         return None
 
+    def _find_dart_pubspec_root(
+        self, start: Path, pkg_name: str,
+    ) -> Optional[Path]:
+        """Walk up from ``start`` to find a ``pubspec.yaml`` whose ``name:``
+        matches ``pkg_name``. Returns the directory containing that pubspec,
+        or None if no match is found. Result is cached per (start, pkg_name)
+        pair so repeated lookups within one parse pass are cheap.
+        """
+        cache_key = (str(start), pkg_name)
+        cached = self._dart_pubspec_cache.get(cache_key)
+        if cached is not None or cache_key in self._dart_pubspec_cache:
+            return cached
+        current = start
+        # Avoid infinite loops on weird symlinks.
+        for _ in range(20):
+            pubspec = current / "pubspec.yaml"
+            if pubspec.is_file():
+                try:
+                    text = pubspec.read_text(encoding="utf-8", errors="replace")
+                except OSError:
+                    text = ""
+                m = re.search(r"^name:\s*([\w-]+)", text, re.MULTILINE)
+                if m and m.group(1) == pkg_name:
+                    self._dart_pubspec_cache[cache_key] = current
+                    return current
+            if current.parent == current:
+                break
+            current = current.parent
+        self._dart_pubspec_cache[cache_key] = None
+        return None
+
     def _resolve_call_target(
         self,
         call_name: str,
@@ -2684,6 +2837,36 @@ class CodeParser:
                     return self._get_name(child, language, kind)
         return None
 
+    def _get_go_receiver_type(self, node) -> Optional[str]:
+        """Extract the receiver type from a Go method_declaration.
+
+        For ``func (s *T) Foo() {...}`` returns ``"T"``. For ``func (T) Foo()``
+        also returns ``"T"``. Returns None if no receiver is present.
+
+        The receiver is always the first ``parameter_list`` child of a
+        Go ``method_declaration`` and contains a single ``parameter_declaration``
+        whose type is either a ``type_identifier`` or a ``pointer_type``
+        wrapping one. See: #190
+        """
+        for child in node.children:
+            if child.type != "parameter_list":
+                continue
+            for param in child.children:
+                if param.type != "parameter_declaration":
+                    continue
+                for sub in param.children:
+                    if sub.type == "type_identifier":
+                        return sub.text.decode("utf-8", errors="replace")
+                    if sub.type == "pointer_type":
+                        for ptr_child in sub.children:
+                            if ptr_child.type == "type_identifier":
+                                return ptr_child.text.decode(
+                                    "utf-8", errors="replace"
+                                )
+            # First parameter_list is always the receiver; stop searching.
+            return None
+        return None
+
     def _get_params(self, node, language: str, source: bytes) -> Optional[str]:
         """Extract parameter list as a string."""
         for child in node.children:

{code_review_graph-2.2.3 → code_review_graph-2.2.4}/code_review_graph/registry.py

@@ -192,7 +192,7 @@ class ConnectionPool:
                 evict_key, evict_conn = self._pool.popitem(last=False)
                 try:
                     evict_conn.close()
-                except Exception:
+                except sqlite3.Error:
                     logger.debug("Failed to close evicted connection: %s", evict_key)
                 logger.debug("Evicted connection: %s", evict_key)
 
@@ -212,7 +212,7 @@ class ConnectionPool:
             for key, conn in self._pool.items():
                 try:
                     conn.close()
-                except Exception:
+                except sqlite3.Error:
                     logger.debug("Failed to close connection: %s", key)
             self._pool.clear()
 

{code_review_graph-2.2.3 → code_review_graph-2.2.4}/code_review_graph/tools/context.py

@@ -3,6 +3,7 @@
 from __future__ import annotations
 
 import logging
+import sqlite3
 import subprocess
 from pathlib import Path
 from typing import Any
@@ -85,7 +86,10 @@ def get_minimal_context(
                         for f in analysis.get("changed_functions", [])[:5]
                     ]
                     test_gap_count = len(analysis.get("test_gaps", []))
-            except Exception:
+            except (
+                ImportError, OSError, ValueError,
+                sqlite3.Error, subprocess.SubprocessError,
+            ):
                 logger.debug("Risk analysis failed in get_minimal_context", exc_info=True)
 
         # 3. Top 3 communities
@@ -95,8 +99,8 @@ def get_minimal_context(
                 "SELECT name FROM communities ORDER BY size DESC LIMIT 3"
             ).fetchall()
             communities = [r[0] for r in rows]
-        except Exception:  # nosec B110 — table may not exist yet
-            pass
+        except sqlite3.OperationalError:  # nosec B110 — table may not exist yet
+            logger.debug("communities table not yet populated")
 
         # 4. Top 3 critical flows
         flows: list[str] = []
@@ -105,8 +109,8 @@ def get_minimal_context(
                 "SELECT name FROM flows ORDER BY criticality DESC LIMIT 3"
             ).fetchall()
             flows = [r[0] for r in rows]
-        except Exception:  # nosec B110 — table may not exist yet
-            pass
+        except sqlite3.OperationalError:  # nosec B110 — table may not exist yet
+            logger.debug("flows table not yet populated")
 
         # 5. Suggest next tools based on task keywords
         task_lower = task.lower()

{code_review_graph-2.2.3 → code_review_graph-2.2.4}/code_review_graph/tools/query.py

@@ -291,6 +291,16 @@ def query_graph(
                     if child:
                         results.append(node_to_dict(child))
                     edges_out.append(edge_to_dict(e))
+            # Fallback: INHERITS/IMPLEMENTS edges store unqualified base names
+            # (e.g. "Animal") while qn is fully qualified
+            # (e.g. "sample.dart::Animal"). Search by plain name too. See: #87
+            if not results and node:
+                for kind in ("INHERITS", "IMPLEMENTS"):
+                    for e in store.search_edges_by_target_name(node.name, kind=kind):
+                        child = store.get_node(e.source_qualified)
+                        if child:
+                            results.append(node_to_dict(child))
+                        edges_out.append(edge_to_dict(e))
 
         elif pattern == "file_summary":
             abs_path = str(root / target)

{code_review_graph-2.2.3 → code_review_graph-2.2.4}/code_review_graph/tsconfig_resolver.py

@@ -52,7 +52,7 @@ class TsconfigResolver:
                 base_dir = Path(tsconfig_dir).resolve()
 
             return self._match_and_probe(import_str, paths, base_dir)
-        except Exception:
+        except (OSError, ValueError, TypeError):
             logger.debug(
                 "TsconfigResolver: unexpected error for %s", file_path, exc_info=True,
             )

{code_review_graph-2.2.3 → code_review_graph-2.2.4}/code_review_graph/visualization.py

@@ -15,6 +15,7 @@ from __future__ import annotations
 
 import json
 import logging
+import sqlite3
 from collections import Counter, defaultdict
 from dataclasses import asdict
 from pathlib import Path
@@ -142,14 +143,16 @@ def export_graph_data(store: GraphStore) -> dict:
     try:
         from code_review_graph.flows import get_flows
         flows = get_flows(store, limit=100)
-    except Exception:
+    except (ImportError, sqlite3.OperationalError) as exc:
+        logger.debug("flows unavailable for export: %s", exc)
         flows = []
 
     # Include communities (graceful fallback if table doesn't exist)
     try:
         from code_review_graph.communities import get_communities
         communities = get_communities(store)
-    except Exception:
+    except (ImportError, sqlite3.OperationalError) as exc:
+        logger.debug("communities unavailable for export: %s", exc)
         communities = []
 
     return {
@@ -863,7 +866,12 @@ simulation.on("tick", function() {
     .attr("x", function(d) { return d.x + (KIND_RADIUS[d.kind] || 6) + 5; })
     .attr("y", function(d) { return d.y; });
 });
-nodes.forEach(function(n) { if (n.kind === "File") collapsedFiles.add(n.qualified_name); });
+// Only auto-collapse File nodes on very large graphs, otherwise all edges
+// become invisible because they connect to Functions/Classes that are now
+// hidden beneath collapsed Files. See: #132
+if (N > 2000) {
+  nodes.forEach(function(n) { if (n.kind === "File") collapsedFiles.add(n.qualified_name); });
+}
 updateNodes();
 function fitGraph() {
   var b = gRoot.node().getBBox();

{code_review_graph-2.2.3 → code_review_graph-2.2.4}/code_review_graph/wiki.py

@@ -8,6 +8,7 @@ from __future__ import annotations
 
 import logging
 import re
+import sqlite3
 from collections import Counter
 from pathlib import Path
 from typing import Any
@@ -118,7 +119,8 @@ def _generate_community_page(store: GraphStore, community: dict[str, Any]) -> st
                 lines.append(f"- *... and {len(community_flows) - 10} more flows.*")
         else:
             lines.append("No execution flows pass through this community.")
-    except Exception:
+    except sqlite3.OperationalError as exc:
+        logger.debug("wiki: flows table unavailable: %s", exc)
         lines.append("Execution flow data not available.")
     lines.append("")
 
@@ -158,7 +160,8 @@ def _generate_community_page(store: GraphStore, community: dict[str, Any]) -> st
         if not outgoing_targets and not incoming_sources:
             lines.append("No cross-community dependencies detected.")
             lines.append("")
-    except Exception:
+    except sqlite3.OperationalError as exc:
+        logger.debug("wiki: dependency edges unavailable: %s", exc)
         lines.append("Dependency data not available.")
         lines.append("")
 
@@ -194,9 +197,24 @@ def generate_wiki(
 
     page_entries: list[tuple[str, str, int]] = []  # (slug, name, size)
 
+    # Track slugs we've already used in THIS run so two communities that
+    # slugify to the same filename don't overwrite each other (#222 follow-up).
+    # Previously "Data Processing" and "data processing" both became
+    # "data-processing.md", causing silent data loss and inflated "updated"
+    # counters (each collision was counted as an update while only one file
+    # made it to disk).
+    used_slugs: set[str] = set()
+
     for comm in communities:
         name = comm["name"]
-        slug = _slugify(name)
+        base_slug = _slugify(name)
+        slug = base_slug
+        suffix = 2
+        while slug in used_slugs:
+            slug = f"{base_slug}-{suffix}"
+            suffix += 1
+        used_slugs.add(slug)
+
         filename = f"{slug}.md"
         filepath = wiki_path / filename
 

{code_review_graph-2.2.3 → code_review_graph-2.2.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "code-review-graph"
-version = "2.2.3"
+version = "2.2.4"
 description = "Persistent incremental knowledge graph for token-efficient, context-aware code reviews with Claude Code"
 readme = {file = "README.md", content-type = "text/markdown"}
 license = "MIT"
@@ -26,7 +26,10 @@ classifiers = [
 ]
 dependencies = [
     "mcp>=1.0.0,<2",
-    "fastmcp>=0.1.0,<2",
+    # fastmcp 2.14+ fixes CVE-2025-62800/62801/66416 and drops the
+    # transitive fakeredis dep that was broken by a rename upstream.
+    # See: #139, #195
+    "fastmcp>=2.14.0,<3",
     "tree-sitter>=0.23.0,<1",
     "tree-sitter-language-pack>=0.3.0,<1",
     "networkx>=3.2,<4",