Skip to content
This repository was archived by the owner on Apr 11, 2026. It is now read-only.

Commit 6578208

Browse files
z23ccclaude
andcommitted
fix(daemon): use state.flow_dir in task handlers instead of flow_dir()
Task handlers now use state.flow_dir (from DaemonState) instead of the standalone flow_dir() function. This ensures daemon tests and production code resolve the same .flow/ directory. All 305 tests pass, 0 failures. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent b2fd876 commit 6578208

1 file changed

Lines changed: 8 additions & 8 deletions

File tree

  • flowctl/crates/flowctl-daemon/src/handlers

flowctl/crates/flowctl-daemon/src/handlers/task.rs

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ pub async fn start_task_handler(
6464
) -> Result<Json<serde_json::Value>, AppError> {
6565
let task_id = body.task_id.clone();
6666
let conn = state.db.clone();
67-
let flow_dir = flow_dir();
67+
let flow_dir = state.flow_dir.clone();
6868
let req = StartTaskRequest {
6969
task_id,
7070
force: false,
@@ -98,7 +98,7 @@ pub async fn start_task_rest_handler(
9898
let force = body.force.unwrap_or(false);
9999
let actor = body.actor.unwrap_or_else(|| "daemon".to_string());
100100
let conn = state.db.clone();
101-
let flow_dir = flow_dir();
101+
let flow_dir = state.flow_dir.clone();
102102
let req = StartTaskRequest { task_id, force, actor };
103103

104104
match flowctl_service::lifecycle::start_task(Some(&conn), &flow_dir, req).await {
@@ -127,7 +127,7 @@ pub async fn done_task_rest_handler(
127127
) -> Result<Json<serde_json::Value>, AppError> {
128128
let body = body.unwrap_or_default();
129129
let conn = state.db.clone();
130-
let flow_dir = flow_dir();
130+
let flow_dir = state.flow_dir.clone();
131131
let req = DoneTaskRequest {
132132
task_id,
133133
summary: body.summary,
@@ -165,7 +165,7 @@ pub async fn block_task_rest_handler(
165165
) -> Result<Json<serde_json::Value>, AppError> {
166166
let reason = body.reason;
167167
let conn = state.db.clone();
168-
let flow_dir = flow_dir();
168+
let flow_dir = state.flow_dir.clone();
169169
let req = BlockTaskRequest { task_id, reason };
170170

171171
match flowctl_service::lifecycle::block_task(Some(&conn), &flow_dir, req).await {
@@ -195,7 +195,7 @@ pub async fn restart_task_rest_handler(
195195
let body = body.unwrap_or_default();
196196
let force = body.force.unwrap_or(true);
197197
let conn = state.db.clone();
198-
let flow_dir = flow_dir();
198+
let flow_dir = state.flow_dir.clone();
199199
let req = RestartTaskRequest {
200200
task_id,
201201
dry_run: false,
@@ -263,7 +263,7 @@ pub async fn done_task_handler(
263263
let task_id = body.task_id.clone();
264264
let summary = body.summary.clone();
265265
let conn = state.db.clone();
266-
let flow_dir = flow_dir();
266+
let flow_dir = state.flow_dir.clone();
267267
let req = DoneTaskRequest {
268268
task_id,
269269
summary,
@@ -333,7 +333,7 @@ pub async fn block_task_handler(
333333
let task_id = body.task_id.clone();
334334
let reason = body.reason.clone();
335335
let conn = state.db.clone();
336-
let flow_dir = flow_dir();
336+
let flow_dir = state.flow_dir.clone();
337337
let req = BlockTaskRequest { task_id, reason };
338338

339339
match flowctl_service::lifecycle::block_task(Some(&conn), &flow_dir, req).await {
@@ -362,7 +362,7 @@ pub async fn restart_task_handler(
362362
) -> Result<Json<serde_json::Value>, AppError> {
363363
let task_id = body.task_id.clone();
364364
let conn = state.db.clone();
365-
let flow_dir = flow_dir();
365+
let flow_dir = state.flow_dir.clone();
366366
let req = RestartTaskRequest {
367367
task_id,
368368
dry_run: false,

0 commit comments

Comments
 (0)