Skip to content

feat(isl): add PR review status indicators with legend tooltip#49

Open
sontiO wants to merge 5 commits intomainfrom
pr49
Open

feat(isl): add PR review status indicators with legend tooltip#49
sontiO wants to merge 5 commits intomainfrom
pr49

Conversation

The WorktreeRemoveOperation used 'sl wt rm' but the actual subcommand
is 'sl wt remove', causing 'abort: you need to specify a subcommand'
when attempting to delete a worktree from the UI.
The worktree dropdown showed deleted worktrees because 'sl wt list'
returns entries even after directories are removed from disk. Added
server-side validation using exists() to filter out stale entries.
- Move stack card avatars closer to title (remove margin-left: auto)
- Push action buttons to far right with margin-left: auto on actions div
- Reduce left/right padding on PR dashboard content for wider cards
- Bump version to v4.2.1
Add gitAuthorAvatarCache atom that cross-references diff summaries
(PR data with GitHub avatar URLs) with the commit DAG to build a
mapping from git author strings to avatar URLs. CommitAvatar now
uses this shared cache, so all commits by known PR authors show
their GitHub profile picture instead of colored initials.
Show review decision (approved, changes requested, review required) on PR
row status circles. Each state gets a distinct icon and color. Hovering
reveals an interactive legend showing all possible statuses.
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