mindesbunister 6f85fee1da critical: Fix test endpoint violating database-first pattern (Common Pitfall #29) ...

- Moved positionManager.addTrade() to AFTER database save succeeds
- Changed database error handling to return HTTP 500 (not silent fail)
- Test endpoint now enforces same pattern as execute endpoint
- Prevents untracked positions when database save fails
- Root cause of trade manual-1763391075992 compounding to -19.43

Before: Test endpoint added to Position Manager first, saved to DB after
After: Test endpoint saves to DB first, only adds to PM if DB succeeds
Impact: No more untracked positions from test trades with failed DB saves

2025-11-17 16:47:07 +01:00

12 KiB

Raw Blame History

View Raw