From 5102a38fbe81bf3b10a24c0766454d98e2777a89 Mon Sep 17 00:00:00 2001
From: Michael Panchenko <mischa.panch@gmail.com>
Date: Sun, 6 Jul 2025 14:06:16 +0200
Subject: [PATCH] Don't include relative_path in symbol's children dict
 representation

---
 src/serena/symbol.py | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/src/serena/symbol.py b/src/serena/symbol.py
index 0f48bb54..8d684671 100644
--- a/src/serena/symbol.py
+++ b/src/serena/symbol.py
@@ -189,8 +189,11 @@ class SymbolLocation:
         if self.relative_path is not None:
             self.relative_path = self.relative_path.replace("/", os.path.sep)
 
-    def to_dict(self) -> dict[str, Any]:
-        return asdict(self)
+    def to_dict(self, include_relative_path: bool = True) -> dict[str, Any]:
+        result = asdict(self)
+        if not include_relative_path:
+            result.pop("relative_path", None)
+        return result
 
     def has_position_in_file(self) -> bool:
         return self.relative_path is not None and self.line is not None and self.column is not None
@@ -426,7 +429,8 @@ class Symbol(ToStringMixin):
         return result
 
     def to_dict(
-        self, kind: bool = False, location: bool = False, depth: int = 0, include_body: bool = False, include_children_body: bool = False
+        self, kind: bool = False, location: bool = False, depth: int = 0, include_body: bool = False, include_children_body: bool = False,
+        include_relative_path=True,
     ) -> dict[str, Any]:
         """
         Converts the symbol to a dictionary.
@@ -439,6 +443,8 @@ class Symbol(ToStringMixin):
             Note that the body of the children is part of the body of the parent symbol,
             so there is usually no need to set this to True unless you want process the output
             and pass the children without passing the parent body to the LM.
+        :param include_relative_path: whether to include the relative path of the symbol in the location
+            entry. Relative paths of the symbol's children are always excluded.
         :return: a dictionary representation of the symbol
         """
         result: dict[str, Any] = {"name": self.name, "name_path": self.get_name_path()}
@@ -447,7 +453,7 @@ class Symbol(ToStringMixin):
             result["kind"] = self.kind
 
         if location:
-            result["location"] = self.location.to_dict()
+            result["location"] = self.location.to_dict(include_relative_path=include_relative_path)
             body_start_line, body_end_line = self.get_body_line_numbers()
             result["body_location"] = {"start_line": body_start_line, "end_line": body_end_line}
 
@@ -466,6 +472,8 @@ class Symbol(ToStringMixin):
                         depth=depth - 1,
                         include_body=include_children_body,
                         include_children_body=include_children_body,
+                        # all children have the same relative path as the parent
+                        include_relative_path=False,
                     )
                 )
             return children