codegraphcontext 0.6.5__tar.gz → 0.6.7__tar.gz

{codegraphcontext-0.6.5/src/codegraphcontext.egg-info → codegraphcontext-0.6.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: codegraphcontext
-Version: 0.6.5
+Version: 0.6.7
 Summary: An MCP server that indexes local code into a graph database to provide context to AI assistants.
 Author-email: Shashank Shekhar Singh <shashankshekharsingh1205@gmail.com>
 License: MIT License
@@ -258,7 +258,12 @@ flowchart LR
 ---
 
 ## Star History
-[![Star History Chart](https://api.star-history.com/svg?repos=CodeGraphContext/CodeGraphContext&type=Date)](https://www.star-history.com/#CodeGraphContext/CodeGraphContext&Date)
+[Star History Chart ↗](https://www.star-history.com/#CodeGraphContext/CodeGraphContext&Date)
+<!-- The inline chart image is temporarily removed: api.star-history.com currently
+     renders "GitHub restricted access to star data" instead of the chart
+     (GitHub-side restriction, see PR #1642). Restore the image once the
+     official service recovers:
+     [![Star History Chart](https://api.star-history.com/svg?repos=CodeGraphContext/CodeGraphContext&type=Date)](https://www.star-history.com/#CodeGraphContext/CodeGraphContext&Date) -->
 
 ---
 

{codegraphcontext-0.6.5 → codegraphcontext-0.6.7}/README.md

@@ -159,7 +159,12 @@ flowchart LR
 ---
 
 ## Star History
-[![Star History Chart](https://api.star-history.com/svg?repos=CodeGraphContext/CodeGraphContext&type=Date)](https://www.star-history.com/#CodeGraphContext/CodeGraphContext&Date)
+[Star History Chart ↗](https://www.star-history.com/#CodeGraphContext/CodeGraphContext&Date)
+<!-- The inline chart image is temporarily removed: api.star-history.com currently
+     renders "GitHub restricted access to star data" instead of the chart
+     (GitHub-side restriction, see PR #1642). Restore the image once the
+     official service recovers:
+     [![Star History Chart](https://api.star-history.com/svg?repos=CodeGraphContext/CodeGraphContext&type=Date)](https://www.star-history.com/#CodeGraphContext/CodeGraphContext&Date) -->
 
 ---
 

{codegraphcontext-0.6.5 → codegraphcontext-0.6.7}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "codegraphcontext"
-version = "0.6.5"
+version = "0.6.7"
 description = "An MCP server that indexes local code into a graph database to provide context to AI assistants."
 authors = [{ name = "Shashank Shekhar Singh", email = "shashankshekharsingh1205@gmail.com" }]
 readme = "README.md"

{codegraphcontext-0.6.5 → codegraphcontext-0.6.7}/src/codegraphcontext/tools/indexing/persistence/writer.py

@@ -154,14 +154,30 @@ def _cypher_label(label: str, backend: str) -> str:
 
 
 def _called_context_clause(called_label: str) -> str:
-    """Match CALLS targets that store scope in context, class_context, or module_context."""
-    if called_label in ("Function", "Variable"):
+    """Match CALLS targets that store scope in context, class_context, or module_context.
+
+    Resolution stores the resolved scope in `called_context`, but which node
+    property carries that scope differs by parser: Python/JS use `context`,
+    C++/PHP methods use `class_context`, and Rust functions carry ONLY
+    `module_context` (#1510). Matching `context` alone filtered every
+    correctly-resolved Rust module-scoped call out at write time.
+
+    Variable is restricted to `context`: the Kùzu Variable table declares no
+    class_context/module_context columns, and referencing a missing column
+    binder-errors the whole batch on typed backends.
+    """
+    if called_label == "Function":
         return (
             'AND (row.called_context = "" OR row.called_context IS NULL '
             "OR called.context = row.called_context "
             "OR called.class_context = row.called_context "
             "OR called.module_context = row.called_context)"
         )
+    if called_label == "Variable":
+        return (
+            'AND (row.called_context = "" OR row.called_context IS NULL '
+            "OR called.context = row.called_context)"
+        )
     return ""
 
 
@@ -1006,10 +1022,12 @@ class GraphWriter:
                         precise_batch.append(row)
 
                 # Define which labels have a 'context' property in the schema
-                labels_with_context = {"Function", "Variable"}
-                called_context_clause = ""
-                if called_label in labels_with_context:
-                    called_context_clause = 'AND (row.called_context = "" OR called.context = row.called_context)'
+                # The label-aware helper matches context, class_context or
+                # module_context depending on what the label's parsers emit —
+                # the inline context-only predicate silently dropped Rust
+                # module-scoped calls whose functions carry only
+                # module_context (#1510).
+                called_context_clause = _called_context_clause(called_label)
 
                 # ...and which have a 'line_number'. File and Directory do not:
                 # the Kùzu File table is (path, name, relative_path, package_name,

{codegraphcontext-0.6.5 → codegraphcontext-0.6.7}/src/codegraphcontext/tools/indexing/schema.py

@@ -137,6 +137,51 @@ def create_graph_schema(driver: Any, db_manager: Any) -> None:
                 ddl.run(
                     "CREATE CONSTRAINT annotation_unique IF NOT EXISTS FOR (a:Annotation) REQUIRE (a.name, a.path, a.line_number) IS UNIQUE"
                 )
+                # The relationship passes and code_finder look these nodes up
+                # by (name, path) with no line_number: write_inheritance_links
+                # matches `(child:<Label> {name: row.child_name, path:
+                # row.path})`, and the same for the parent, for every label
+                # pair it enumerates; write_function_call_groups matches the
+                # callee by (name, path) in both the batched and the single-row
+                # path; and code_finder's who_calls_function,
+                # what_does_function_call, find_class_hierarchy,
+                # find_all_callers and find_all_callees do the same.
+                #
+                # Neo4j uses a composite index only when EVERY indexed property
+                # has a predicate, so the four-property identity constraints
+                # above cannot serve those lookups -- the planner falls back to
+                # NodeByLabelScan + Filter, once per row. These plain
+                # two-property indexes restore an index seek. They do not
+                # collide with the identity constraints: a constraint's backing
+                # index covers a different property set, so the two coexist.
+                #
+                # FalkorDB is excluded (this is inside `not is_falkordb`): it
+                # indexes per attribute, not per property tuple, and the
+                # composite CREATE INDEX statements in its own branch already
+                # cover name and path. A second (name, path) index would add
+                # nothing there and only re-issue an index creation FalkorDB
+                # rejects as "already indexed".
+                for _label, _var in POSITIONAL_IDENTITY_LABELS.items():
+                    ddl.run(
+                        f"CREATE INDEX {_label.lower()}_name_path IF NOT EXISTS "
+                        f"FOR ({_var}:{_label}) ON ({_var}.name, {_var}.path)"
+                    )
+                # EnumMember, Object, Mixin, Extension and Parameter had index
+                # statements only in the is_falkordb branch, so on Neo4j the
+                # writer's `MATCH (m:EnumMember {name: …, path: …})` (and the
+                # per-label CONTAINS linking generally) was an unavoidable
+                # label scan — found while auditing #1653's lookup sites.
+                for _extra_label, _extra_var in (
+                    ("EnumMember", "em"),
+                    ("Object", "o"),
+                    ("Mixin", "mx"),
+                    ("Extension", "ex"),
+                    ("Parameter", "p"),
+                ):
+                    ddl.run(
+                        f"CREATE INDEX {_extra_label.lower()}_name_path IF NOT EXISTS "
+                        f"FOR ({_extra_var}:{_extra_label}) ON ({_extra_var}.name, {_extra_var}.path)"
+                    )
 
             ddl.run("CREATE INDEX function_lang IF NOT EXISTS FOR (f:Function) ON (f.lang)")
             ddl.run("CREATE INDEX class_lang IF NOT EXISTS FOR (c:Class) ON (c.lang)")

{codegraphcontext-0.6.5 → codegraphcontext-0.6.7}/src/codegraphcontext/tools/languages/kotlin.py

@@ -169,7 +169,10 @@ class KotlinTreeSitterParser:
                 return (
                     self._get_node_text(name_node) if name_node else None,
                     curr.type,
-                    curr.start_point[0] + 1,
+                    # The name node's line, not the node's: an annotated
+                    # declaration starts at its annotation, and consumers join
+                    # this against the function's line_number (#1660).
+                    (name_node.start_point[0] + 1) if name_node else (curr.start_point[0] + 1),
                 )
             if curr.type in ("class_declaration", "interface_declaration", "object_declaration"):
                 for child in curr.children:
@@ -177,7 +180,7 @@ class KotlinTreeSitterParser:
                          return (
                             self._get_node_text(child),
                             curr.type,
-                            curr.start_point[0] + 1,
+                            child.start_point[0] + 1,
                         )
                 # Check for secondary constructors
                 if curr.type == "secondary_constructor":
@@ -1330,6 +1333,12 @@ class KotlinTreeSitterParser:
                             
                     if name_node:
                         func_name = self._get_node_text(name_node)
+                        # Annotations/modifiers are children of the declaration
+                        # node, so node.start_point is the ANNOTATION line for
+                        # any annotated function. The name node sits on the
+                        # declaration proper — the established convention
+                        # (python decorated_definition, csharp #1659, #1660).
+                        start_line = name_node.start_point[0] + 1
                         
                         params_node = None
                         for child in node.children:
@@ -1457,6 +1466,10 @@ class KotlinTreeSitterParser:
                         for child in node.children:
                             if child.type in ("type_identifier", "simple_identifier"):
                                 class_name = self._get_node_text(child)
+                                # Same convention as functions: an annotated
+                                # class node starts at its annotation line;
+                                # report the declaration line (#1660).
+                                start_line = child.start_point[0] + 1
                                 break
                             
                     source_text = self._get_node_text(node)

{codegraphcontext-0.6.5 → codegraphcontext-0.6.7}/src/codegraphcontext/tools/languages/php.py

@@ -162,7 +162,10 @@ class PhpTreeSitterParser:
                 return (
                     self._get_node_text(name_node) if name_node else None,
                     curr.type,
-                    curr.start_point[0] + 1,
+                    # Name-node line: attributed declarations start at their
+                    # #[Attribute], and consumers join this against the
+                    # declaration's line_number (#1660).
+                    (name_node.start_point[0] + 1) if name_node else (curr.start_point[0] + 1),
                 )
             curr = curr.parent
         return None, None, None
@@ -189,7 +192,7 @@ class PhpTreeSitterParser:
             if curr.type in ("class_declaration", "interface_declaration", "trait_declaration", "enum_declaration"):
                 name_node = curr.child_by_field_name("name")
                 name = self._get_node_text(name_node) if name_node else None
-                return (name, curr.start_point[0] + 1) if name else None
+                return (name, name_node.start_point[0] + 1) if name else None
             curr = curr.parent
         return None
 
@@ -245,6 +248,11 @@ class PhpTreeSitterParser:
                     name_node = node.child_by_field_name("name")
                     if name_node:
                         func_name = self._get_node_text(name_node)
+                        # #[Attribute] lines are part of the declaration node,
+                        # so node.start_point is the attribute line; report the
+                        # declaration proper (python/csharp/kotlin convention,
+                        # #1660).
+                        start_line = name_node.start_point[0] + 1
                         
                         params_node = node.child_by_field_name("parameters")
                         parameters = []
@@ -321,6 +329,9 @@ class PhpTreeSitterParser:
                     name_node = node.child_by_field_name("name")
                     if name_node:
                         type_name = self._get_node_text(name_node)
+                        # Same convention as functions: attributes are children
+                        # of the declaration node (#1660).
+                        start_line = name_node.start_point[0] + 1
                         source_text = self._get_node_text(node)
                         
                         bases = []

{codegraphcontext-0.6.5 → codegraphcontext-0.6.7}/src/codegraphcontext/tools/languages/ruby.py

@@ -49,6 +49,14 @@ RUBY_QUERIES = {
             left: (instance_variable) @name
             right: (_) @value
         )
+        (assignment
+            left: (class_variable) @name
+            right: (_) @value
+        )
+        (assignment
+            left: (global_variable) @name
+            right: (_) @value
+        )
     """,
     "comments": """
         (comment) @comment
@@ -537,7 +545,12 @@ class RubyTreeSitterParser:
                 while current and current.type != 'assignment':
                     current = current.parent
                 if current:
-                    assignment_id = id(current)
+                    # id() of a tree-sitter node is a fresh wrapper each
+                    # access (n1 == n2 while id(n1) != id(n2)), which split
+                    # the @name/@value captures of ONE assignment into two
+                    # buckets — value was always None (#1660). Key on the
+                    # node's span instead.
+                    assignment_id = (current.start_byte, current.end_byte)
                     if assignment_id not in captures_by_assignment:
                         captures_by_assignment[assignment_id] = {'node': current, 'name': None, 'value': None}
                     captures_by_assignment[assignment_id]['name'] = self._get_node_text(node)
@@ -547,7 +560,7 @@ class RubyTreeSitterParser:
                 while current and current.type != 'assignment':
                     current = current.parent
                 if current:
-                    assignment_id = id(current)
+                    assignment_id = (current.start_byte, current.end_byte)
                     if assignment_id not in captures_by_assignment:
                         captures_by_assignment[assignment_id] = {'node': current, 'name': None, 'value': None}
                     captures_by_assignment[assignment_id]['value'] = self._get_node_text(node)
@@ -560,10 +573,12 @@ class RubyTreeSitterParser:
             if name:
                 # Determine variable type based on name prefix
                 var_type = "local"
-                if name.startswith("@"):
-                    var_type = "instance"
-                elif name.startswith("@@"):
+                # @@ must be tested before @: startswith("@") matches
+                # "@@count" too, which left the class branch dead (#1660).
+                if name.startswith("@@"):
                     var_type = "class"
+                elif name.startswith("@"):
+                    var_type = "instance"
                 elif name.startswith("$"):
                     var_type = "global"
 

{codegraphcontext-0.6.5 → codegraphcontext-0.6.7/src/codegraphcontext.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: codegraphcontext
-Version: 0.6.5
+Version: 0.6.7
 Summary: An MCP server that indexes local code into a graph database to provide context to AI assistants.
 Author-email: Shashank Shekhar Singh <shashankshekharsingh1205@gmail.com>
 License: MIT License
@@ -258,7 +258,12 @@ flowchart LR
 ---
 
 ## Star History
-[![Star History Chart](https://api.star-history.com/svg?repos=CodeGraphContext/CodeGraphContext&type=Date)](https://www.star-history.com/#CodeGraphContext/CodeGraphContext&Date)
+[Star History Chart ↗](https://www.star-history.com/#CodeGraphContext/CodeGraphContext&Date)
+<!-- The inline chart image is temporarily removed: api.star-history.com currently
+     renders "GitHub restricted access to star data" instead of the chart
+     (GitHub-side restriction, see PR #1642). Restore the image once the
+     official service recovers:
+     [![Star History Chart](https://api.star-history.com/svg?repos=CodeGraphContext/CodeGraphContext&type=Date)](https://www.star-history.com/#CodeGraphContext/CodeGraphContext&Date) -->
 
 ---