fix: remove dead setStatus calls from engine
This commit is contained in:
@@ -75,7 +75,6 @@ export class TaskExecutor {
|
||||
console.log(`[executor] Starting ${task.id}: ${task.title}`);
|
||||
|
||||
try {
|
||||
await this.store.setStatus(task.id, "executing");
|
||||
// Check dependencies
|
||||
const allTasks = await this.store.listTasks();
|
||||
const unmetDeps = task.dependencies.filter((depId) => {
|
||||
@@ -147,7 +146,6 @@ export class TaskExecutor {
|
||||
}
|
||||
} catch (err: any) {
|
||||
console.error(`[executor] ✗ ${task.id} execution failed:`, err.message);
|
||||
await this.store.setStatus(task.id, "idle");
|
||||
this.options.onError?.(task, err);
|
||||
} finally {
|
||||
this.executing.delete(task.id);
|
||||
|
||||
@@ -78,14 +78,12 @@ export class Scheduler {
|
||||
});
|
||||
|
||||
if (unmetDeps.length > 0) {
|
||||
await this.store.setStatus(task.id, "blocked");
|
||||
this.options.onBlocked?.(task, unmetDeps);
|
||||
continue;
|
||||
}
|
||||
|
||||
// Dependencies met — check concurrency
|
||||
if (started >= available) {
|
||||
await this.store.setStatus(task.id, "queued");
|
||||
continue;
|
||||
}
|
||||
|
||||
|
||||
@@ -99,7 +99,6 @@ export class TriageProcessor {
|
||||
console.log(`[triage] Specifying ${task.id}: ${task.title}`);
|
||||
this.options.onSpecifyStart?.(task);
|
||||
|
||||
await this.store.setStatus(task.id, "specifying");
|
||||
|
||||
try {
|
||||
// Get the full task detail including current prompt
|
||||
@@ -132,7 +131,6 @@ export class TriageProcessor {
|
||||
}
|
||||
} catch (err: any) {
|
||||
console.error(`[triage] ✗ ${task.id} specification failed:`, err.message);
|
||||
await this.store.setStatus(task.id, "idle");
|
||||
this.options.onSpecifyError?.(task, err);
|
||||
} finally {
|
||||
this.processing.delete(task.id);
|
||||
|
||||
Reference in New Issue
Block a user