fix: remove duplicate step output from post-hoc trajectory replay

DSPy's verbose=True already shows each RLM iteration's reasoning and code
in real-time. The post-hoc on_step trajectory callback was replaying all
steps again after completion, causing every step to appear twice in output.

Agent-Id: agent-01069cef-6b08-41f1-a15d-29feb2ccaea1

Author: sng-asyncfunc

npx/python/cli/virtual_runner.py

@@ -263,13 +263,9 @@ async def review(self, url: str, question: str) -> tuple[str, list[str], dict]:
             if isinstance(sources, str):
                 sources = [s.strip() for s in sources.split(",") if s.strip()]
 
-            # Handle on_step callback with trajectory if available
-            if self.on_step and hasattr(result, 'trajectory') and result.trajectory:
-                for i, step in enumerate(result.trajectory):
-                    reasoning = step.get('reasoning', '')
-                    code = step.get('code', '')
-                    output = step.get('output', '')
-                    self.on_step(i + 1, reasoning, code, output)
+            # Note: DSPy's verbose=True already shows step-by-step progress in real-time.
+            # Post-hoc trajectory replay via on_step is intentionally removed to avoid
+            # duplicate output (steps were being shown twice).
         finally:
             # Cleanup
             if self._repo_tools:
@@ -321,13 +317,9 @@ async def review_local(self, path: str, question: str) -> tuple[str, list[str],
             if isinstance(sources, str):
                 sources = [s.strip() for s in sources.split(",") if s.strip()]
 
-            # Handle on_step callback with trajectory if available
-            if self.on_step and hasattr(result, 'trajectory') and result.trajectory:
-                for i, step in enumerate(result.trajectory):
-                    reasoning = step.get('reasoning', '')
-                    code = step.get('code', '')
-                    output = step.get('output', '')
-                    self.on_step(i + 1, reasoning, code, output)
+            # Note: DSPy's verbose=True already shows step-by-step progress in real-time.
+            # Post-hoc trajectory replay via on_step is intentionally removed to avoid
+            # duplicate output (steps were being shown twice).
         finally:
             # Cleanup
             if self._repo_tools: