fix: remove duplicate reverse property and dead _build_reverse method

The optimizer added a second 'reverse' property at the bottom of CallGraph
that shadowed the existing one (F811), leaving the original as dead code.
Remove the duplicate and the unnecessary _build_reverse helper; the existing
reverse property via _build_adjacency is sufficient.

Author: github-actions[bot]

codeflash/models/call_graph.py

@@ -99,7 +99,7 @@ def descendants(self, node: FunctionNode, max_depth: int | None = None) -> set[F
     def ancestors(self, node: FunctionNode, max_depth: int | None = None) -> set[FunctionNode]:
         visited: set[FunctionNode] = set()
         reverse_map = self.reverse
-        
+
         if max_depth is None:
             queue: deque[FunctionNode] = deque([node])
             while queue:
@@ -118,7 +118,7 @@ def ancestors(self, node: FunctionNode, max_depth: int | None = None) -> set[Fun
                     if edge.caller not in visited:
                         visited.add(edge.caller)
                         queue_with_depth.append((edge.caller, depth + 1))
-        
+
         return visited
 
     def subgraph(self, nodes: set[FunctionNode]) -> CallGraph:
@@ -161,20 +161,6 @@ def topological_order(self) -> list[FunctionNode]:
         result.reverse()
         return result
 
-    def _build_reverse(self) -> dict[FunctionNode, list[CallEdge]]:
-        rev: dict[FunctionNode, list[CallEdge]] = {}
-        for e in self.edges:
-            rev.setdefault(e.callee, []).append(e)
-        return rev
-
-    @property
-    def reverse(self) -> dict[FunctionNode, list[CallEdge]]:
-        rv = self._reverse
-        if rv is None:
-            rv = self._build_reverse()
-            self._reverse = rv
-        return rv
-
 
 def augment_with_trace(graph: CallGraph, trace_db_path: Path) -> CallGraph:
     import sqlite3