Bill/AI-Trader
1
0
Fork 0
mirror of https://github.com/Xe138/AI-Trader.git synced 2026-04-01 17:17:24 -04:00
Code Issues Packages Projects Releases Wiki Activity

fix: correct Database default path to match system-wide db_path

Browse Source 
Changes:
- Change Database.__init__ default from "data/trading.db" to "data/jobs.db"

Root Cause:
- Job creation uses "data/jobs.db" (via JobManager, SimulationWorker)
- BaseAgent's Database() was using "data/trading.db" by default
- This caused jobs table to exist in jobs.db but trading_days INSERT
  tried to reference job_id from trading.db, causing FK constraint failure

Impact:
- Fixes: "FOREIGN KEY constraint failed" when creating trading_day records
- Ensures all components use same database file for referential integrity
- Maintains DEV/PROD mode isolation via get_db_path()

Related: api/database.py:521
This commit is contained in:
Bill Ballou
2025-11-04 22:39:36 -05:00
parent 481126ceca
commit 9da65c2d53
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
api/database.py
View File

@@ -518,7 +518,7 @@ class Database:
""" """
if db_path is None: if db_path is None:
from tools.deployment_config import get_db_path from tools.deployment_config import get_db_path
db_path = get_db_path("data/trading.db") db_path = get_db_path("data/jobs.db")
self.db_path = db_path self.db_path = db_path
self.connection = sqlite3.connect(db_path, check_same_thread=False) self.connection = sqlite3.connect(db_path, check_same_thread=False)