Bill abb9cd0726 fix: capture tool messages in conversation history for summarizer ...

**Root Cause:**
The summarizer was not receiving tool execution results (buy/sell trades)
because they were never captured to conversation_history.

**What was captured:**
- User: 'Please analyze positions'
- Assistant: 'I will buy/sell...'
- Assistant: 'Done <FINISH_SIGNAL>'

**What was MISSING:**
- Tool: buy 14 NVDA at $185.24
- Tool: sell 1 GOOGL at $245.15

**Changes:**
- Added tool message capture in trading loop (line 649)
- Extract tool_name and tool_content from each tool message
- Capture to conversation_history before processing
- Changed message['tool_name'] to message['name'] for consistency

**Impact:**
Now the summarizer sees the actual tool results, not just the AI's
intentions. Combined with alpha.8's prompt improvements, summaries
will accurately reflect executed trades.

Fixes reasoning summaries that contradicted actual trades.

2025-11-05 00:44:24 -05:00

..

base_agent

fix: capture tool messages in conversation history for summarizer

2025-11-05 00:44:24 -05:00

mock_provider

fix: suppress unused parameter warnings in mock LangChain model

2025-11-01 11:16:51 -04:00

context_injector.py

chore: remove debug logging from ContextInjector

2025-11-05 00:31:16 -05:00

pnl_calculator.py

feat: add daily P&L calculator with weekend gap handling

2025-11-03 23:12:49 -05:00

reasoning_summarizer.py

fix: improve reasoning summary to explicitly mention trades

2025-11-05 00:41:59 -05:00