From 382b5aa5f643db06062b791d0ca4f2a59390c6ba Mon Sep 17 00:00:00 2001 From: Thigazhezhilan J Date: Sat, 28 Mar 2026 13:45:51 +0530 Subject: [PATCH] Improve resume strategy error handling and status updates --- backend/app/services/strategy_service.py | 25 +++++++++++++++--------- 1 file changed, 16 insertions(+), 9 deletions(-) diff --git a/backend/app/services/strategy_service.py b/backend/app/services/strategy_service.py index c15bb64..6c6f46b 100644 --- a/backend/app/services/strategy_service.py +++ b/backend/app/services/strategy_service.py @@ -611,20 +611,27 @@ def resume_strategy(user_id: str): "broker": broker_state.get("broker"), } + if not engine_external: + engine_config = _build_engine_config(user_id, run_id, None) + if not engine_config: + return { + "status": "resume_failed", + "run_id": run_id, + "message": "Saved strategy configuration is incomplete.", + } + started = start_engine(engine_config) + if not started: + return { + "status": "resume_failed", + "run_id": run_id, + "message": "Strategy engine could not be started.", + } + reactivate_strategy_config(user_id, run_id) resume_run(user_id, run_id) _write_status(user_id, run_id, "RUNNING") update_run_status(user_id, run_id, "RUNNING", meta={"reason": "user_resume"}) - if not engine_external: - engine_config = _build_engine_config(user_id, run_id, None) - if not engine_config: - return {"status": "resume_failed", "run_id": run_id} - started = start_engine(engine_config) - if not started: - return {"status": "resume_failed", "run_id": run_id} - _write_status(user_id, run_id, "RUNNING") - try: user = get_user_by_id(user_id) if user: