Compare commits

..

2 Commits

Author SHA1 Message Date
2d41717b2b docs: update v0.4.1 changelog (IF_TRADE fix only)
Reverted ChatDeepSeek integration approach as it conflicts with
OpenRouter unified gateway architecture.

The system uses OPENAI_API_BASE (OpenRouter) with a single
OPENAI_API_KEY for all AI providers, not direct provider connections.

v0.4.1 now only includes the IF_TRADE initialization fix.
2025-11-06 11:20:22 -05:00
7c4874715b fix: initialize IF_TRADE to True (trades expected by default)
Root cause: IF_TRADE was initialized to False and never updated when
trades executed, causing 'No trading' message to always display.

Design documents (2025-02-11-complete-schema-migration) specify
IF_TRADE should start as True, with trades setting it to False only
after completion.

Fixes sporadic issue where all trading sessions reported 'No trading'
despite successful buy/sell actions.
2025-11-06 07:33:33 -05:00
3 changed files with 36 additions and 7 deletions

View File

@@ -7,13 +7,16 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
## [Unreleased]
## [0.4.1] - 2025-11-05
## [0.4.1] - 2025-11-06
### Fixed
- Resolved Pydantic validation errors when using DeepSeek Chat v3.1 through systematic debugging
- Root cause: Initial implementation incorrectly converted tool_calls arguments from strings to dictionaries, causing LangChain's `parse_tool_call()` to fail and create invalid_tool_calls with wrong format
- Solution: Removed unnecessary conversion logic - DeepSeek already returns arguments in correct format (JSON strings)
- `ToolCallArgsParsingWrapper` now acts as a simple passthrough proxy (kept for backward compatibility)
- Fixed "No trading" message always displaying despite trading activity by initializing `IF_TRADE` to `True` (trades expected by default)
- Root cause: `IF_TRADE` was initialized to `False` in runtime config but never updated when trades executed
### Note
- ChatDeepSeek integration was reverted as it conflicts with OpenRouter unified gateway architecture
- System uses `OPENAI_API_BASE` (OpenRouter) with single `OPENAI_API_KEY` for all providers
- Sporadic DeepSeek validation errors appear to be transient and do not require code changes
## [0.4.0] - 2025-11-05

View File

@@ -80,7 +80,7 @@ class RuntimeConfigManager:
initial_config = {
"TODAY_DATE": date,
"SIGNATURE": model_sig,
"IF_TRADE": False,
"IF_TRADE": True, # FIX: Trades are expected by default
"JOB_ID": job_id,
"TRADING_DAY_ID": trading_day_id
}

View File

@@ -63,7 +63,7 @@ class TestRuntimeConfigCreation:
assert config["TODAY_DATE"] == "2025-01-16"
assert config["SIGNATURE"] == "gpt-5"
assert config["IF_TRADE"] is False
assert config["IF_TRADE"] is True
assert config["JOB_ID"] == "test-job-123"
def test_create_runtime_config_unique_paths(self):
@@ -108,6 +108,32 @@ class TestRuntimeConfigCreation:
# Config file should exist
assert os.path.exists(config_path)
def test_create_runtime_config_if_trade_defaults_true(self):
"""Test that IF_TRADE initializes to True (trades expected by default)"""
from api.runtime_manager import RuntimeConfigManager
with tempfile.TemporaryDirectory() as temp_dir:
manager = RuntimeConfigManager(data_dir=temp_dir)
config_path = manager.create_runtime_config(
job_id="test-job-123",
model_sig="test-model",
date="2025-01-16",
trading_day_id=1
)
try:
# Read the config file
with open(config_path, 'r') as f:
config = json.load(f)
# Verify IF_TRADE is True by default
assert config["IF_TRADE"] is True, "IF_TRADE should initialize to True"
finally:
# Cleanup
if os.path.exists(config_path):
os.remove(config_path)
@pytest.mark.unit
class TestRuntimeConfigCleanup: