Bill f1f76b9a99 fix: extract position dict from CallToolResult.structuredContent ...

Fix negative cash bug where ContextInjector._current_position never updated.

Root cause: MCP tools return mcp.types.CallToolResult objects, not plain
dicts. The isinstance(result, dict) check always failed, preventing
_current_position from accumulating trades within a session.

This caused all trades to calculate from initial $10,000 position instead
of previous trade's ending position, resulting in negative cash balances
when total purchases exceeded $10,000.

Solution: Extract position dict from CallToolResult.structuredContent field
before validating. Maintains backward compatibility by handling both
CallToolResult objects (production) and plain dicts (unit tests).

Impact:
- Fixes negative cash positions (e.g., -$8,768.68 after 11 trades)
- Enables proper intra-day position tracking
- Validates sufficient cash before each trade based on cumulative position
- Trade tool responses now properly accumulate all holdings

Testing:
- All existing unit tests pass (handle plain dict results)
- Production logs confirm structuredContent extraction works
- Debug logging shows _current_position now updates after each trade

2025-11-07 14:24:48 -05:00

..

base_agent

fix: normalize DeepSeek non-standard tool_calls format

2025-11-06 11:38:35 -05:00

mock_provider

fix: suppress unused parameter warnings in mock LangChain model

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

chat_model_wrapper.py

fix: resolve DeepSeek tool_calls validation errors (production ready)

2025-11-06 20:49:11 -05:00

context_injector.py

fix: extract position dict from CallToolResult.structuredContent

2025-11-07 14:24:48 -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

debug: add logging to verify conversation history capture

2025-11-05 00:46:30 -05:00