Skip to content

Fix dashboard not updating for non-active workspaces #124

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

Jump to bottom
Open
andyrewlee wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fix dashboard not updating for non-active workspaces #124

andyrewlee wants to merge 2 commits into from

Conversation

@andyrewlee
Copy link
Owner

@andyrewlee andyrewlee commented Jan 29, 2026 •
edited by devin-ai-integration bot
Loading

  • Watch working tree directories recursively (excluding node_modules, .git, etc.)
  • Watch all workspaces at load time, not just on activation
  • Keep spinner alive while agents are active (len(activeWorkspaces) > 0)
  • Sync active workspace state on git status ticker as fallback
  • Handle dynamically created directories by adding watches on Create events
  • Prefer longest root match in findRoot() for nested workspaces
  • Make SetActiveWorkspaces return spinner command to fix race condition
Open with Devin

@andyrewlee @claude
Fix dashboard not updating for non-active workspaces
d147fb9 
- Watch working tree directories recursively (excluding node_modules, .git, etc.)
- Watch all workspaces at load time, not just on activation
- Keep spinner alive while agents are active (len(activeWorkspaces) > 0)
- Sync active workspace state on git status ticker as fallback
- Handle dynamically created directories by adding watches on Create events
- Prefer longest root match in findRoot() for nested workspaces
- Make SetActiveWorkspaces return spinner command to fix race condition

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
devin-ai-integration[bot]
devin-ai-integration bot reviewed Jan 29, 2026
View reviewed changes
Copy link

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

View issue and 5 additional flags in Devin Review.

Open in Devin Review

@andyrewlee @claude
Fix race condition when adding watch for new directories
9be68e0 
Check that the workspace root is still being watched after acquiring
the mutex, before adding the watch path. This prevents orphaned entries
in watchPaths if Unwatch is called between findRoot and the mutex lock.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@devin-ai-integration devin-ai-integration[bot] devin-ai-integration[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@andyrewlee