Skip to content

Github issue #1805 - Fix Real-Time Subtask Display via Worktree-Aware FileWatcher #1839

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
VDT-91 wants to merge 6 commits into
base: develop
Choose a base branch
from
+17 −10
Open

Github issue #1805 - Fix Real-Time Subtask Display via Worktree-Aware FileWatcher #1839

Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
f98190c
auto-claude: subtask-1-1 - Update TASK_START handler to use worktree-…
VDT-91 Feb 14, 2026
048d177
auto-claude: subtask-1-2 - Update TASK_UPDATE_STATUS auto-start handl…
VDT-91 Feb 14, 2026
ba9180c
auto-claude: subtask-1-3 - TASK_RECOVER_STUCK handler already updated
VDT-91 Feb 14, 2026
48a1903
auto-claude: subtask-2-2 - Verify TypeScript compilation and existing…
VDT-91 Feb 14, 2026
5ede277
Merge branch 'develop' into auto-claude/202-root-cause-high-confidenc…
VDT-91 Feb 15, 2026
4700009
Merge branch 'develop' into auto-claude/202-root-cause-high-confidenc…
VDT-91 Feb 15, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 17 additions & 10 deletions apps/frontend/src/main/ipc-handlers/task/execution-handlers.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -204,13 +204,12 @@ export function registerTaskExecutionHandlers(
console.warn(`[TASK_START] Reset ${resetResult.resetCount} stuck subtask(s) before starting`);
}

// Start file watcher for this task
// Start file watcher for this task - use worktree path if available
const specsBaseDir = getSpecsDir(project.autoBuildPath);
const specDir = path.join(
project.path,
specsBaseDir,
task.specId
);
const worktreePath = findTaskWorktree(project.path, task.specId);
const specDir = worktreePath
? path.join(worktreePath, specsBaseDir, task.specId)
: path.join(project.path, specsBaseDir, task.specId);
Comment on lines 208 to +212
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot Feb 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

This logic to determine the specDir by checking for a worktree is duplicated in three places in this file (here, lines 712-716, and lines 1153-1157). To improve maintainability and adhere to the DRY (Don't Repeat Yourself) principle, this should be extracted into a single helper function.

For example, you could create a function like getSpecDirForWatcher:

function getSpecDirForWatcher(project: Project, task: Task): string {
  const specsBaseDir = getSpecsDir(project.autoBuildPath);
  const worktreePath = findTaskWorktree(project.path, task.specId);
  const basePath = worktreePath || project.path;
  return path.join(basePath, specsBaseDir, task.specId);
}

And then call it in all three places:
const specDir = getSpecDirForWatcher(project, task);

This makes the code cleaner and avoids potential bugs if this logic needs to be updated in the future.

fileWatcher.watch(taskId, specDir);

// Check if spec.md exists (indicates spec creation was already done or in progress)
Expand Down Expand Up @@ -709,8 +708,13 @@ export function registerTaskExecutionHandlers(
console.warn(`[TASK_UPDATE_STATUS] Reset ${resetResult.resetCount} stuck subtask(s) before starting`);
}

// Start file watcher for this task
fileWatcher.watch(taskId, specDir);
// Start file watcher for this task - use worktree path if available
const specsBaseDirForWatcher = getSpecsDir(project.autoBuildPath);
const worktreePath = findTaskWorktree(project.path, task.specId);
const specDirForWatcher = worktreePath
? path.join(worktreePath, specsBaseDirForWatcher, task.specId)
: path.join(project.path, specsBaseDirForWatcher, task.specId);
fileWatcher.watch(taskId, specDirForWatcher);
Comment on lines +716 to +717

This comment was marked as outdated.

Sign in to view


// Check if spec.md exists
const specFilePath = path.join(specDir, AUTO_BUILD_PATHS.SPEC_FILE);

This comment was marked as outdated.

Sign in to view

Expand Down Expand Up @@ -1145,9 +1149,12 @@ export function registerTaskExecutionHandlers(
}

// Start the task execution
// Start file watcher for this task
// Start file watcher for this task - use worktree path if available
const specsBaseDir = getSpecsDir(project.autoBuildPath);
const specDirForWatcher = path.join(project.path, specsBaseDir, task.specId);
const worktreePath = findTaskWorktree(project.path, task.specId);
const specDirForWatcher = worktreePath
? path.join(worktreePath, specsBaseDir, task.specId)
: path.join(project.path, specsBaseDir, task.specId);
fileWatcher.watch(taskId, specDirForWatcher);

// Check if spec.md exists to determine whether to run spec creation or task execution
Expand Down
Loading