Skip to content

Fix active project edge cases#138

Merged
andyrewlee merged 3 commits intomainfrom
stale-active
Feb 1, 2026
Merged

Fix active project edge cases#138
andyrewlee merged 3 commits intomainfrom
stale-active

Conversation

@andyrewlee
Copy link
Copy Markdown
Owner

@andyrewlee andyrewlee commented Feb 1, 2026

@andyrewlee andyrewlee changed the title Add tmux activity hysteresis and safe scans Fix active project edge cases Feb 1, 2026
Copy link
Copy Markdown

@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

Comment thread internal/app/app_tmux_activity.go
andyrewlee and others added 2 commits February 1, 2026 00:15
scanTmuxActivityNow() now increments the token before capturing it,
ensuring manual scan results aren't silently dropped when a periodic
tick fires in between. Stale ticks reschedule instead of returning nil,
which previously broke the tick chain permanently. Also removes the
unused activeWorkspaceIDsFromSessionActivity function and its test.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Covers the three workspace ID resolution sources (session field, tab
info fallback, session name fallback) and verifies non-chat and
below-threshold sessions are excluded. Uses pre-warmed activity states
so the test exercises activeWorkspaceIDsWithHysteresis without needing
a running tmux.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@andyrewlee andyrewlee merged commit 7248d0f into main Feb 1, 2026
1 check passed
@andyrewlee andyrewlee deleted the stale-active branch February 1, 2026 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant