Disable highlighting when agent is working or thinking - #5
Merged
Conversation
Prevents cells from appearing actionable while the agent is actively processing tasks, ensuring UI accurately reflects when user actions are actually possible. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
This was referenced Apr 18, 2026
This was referenced Apr 26, 2026
agent-era-ai
added a commit
that referenced
this pull request
Apr 26, 2026
- Add terminal test that drives the full memoryStore → kanban render path, asserting the merged glyph and "Merged" label appear on a card whose worktree has a MERGED PR seeded in memoryStore.prStatus. This is the integration coverage requirements criterion #5 called for and the gap that hid the bug twice. - Trim the multi-line comment on isItemPRMerged to a single line per AGENTS.md style. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
agent-era-ai
added a commit
that referenced
this pull request
Apr 26, 2026
* fix(tracker): wire kanban merged-state lookup to GitHubContext The kanban read PR status via wt.pr.is_merged, but WorktreeInfo.pr was declared and never assigned anywhere — PR data lives on GitHubContext.pullRequests, keyed by worktree path. Two prior fixes (PR #221, PR #224) shipped reading the same broken field and were dead on arrival. Switch the lookup to pullRequests[wt.path], extract a small isItemPRMerged helper with regression tests, and delete the dead pr field plus the two unused getters that depended on it. Also scope the kanban's PR auto-refresh to its own visible worktrees via setVisibleWorktrees, so it doesn't keep polling whatever set the worktree list left in place. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * Address PR review comments - Add terminal test that drives the full memoryStore → kanban render path, asserting the merged glyph and "Merged" label appear on a card whose worktree has a MERGED PR seeded in memoryStore.prStatus. This is the integration coverage requirements criterion #5 called for and the gap that hid the bug twice. - Trim the multi-line comment on isItemPRMerged to a single line per AGENTS.md style. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
5 tasks
3 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
🤖 Generated with Claude Code