Skip to content

feat(agent-manager): show checkmark on approved PR badges#8657

Merged
marius-kilocode merged 2 commits into
mainfrom
lime-tractor
Apr 9, 2026
Merged

feat(agent-manager): show checkmark on approved PR badges#8657
marius-kilocode merged 2 commits into
mainfrom
lime-tractor

Conversation

@marius-kilocode

Copy link
Copy Markdown
Collaborator

Summary

  • Replace the branch icon with a checkmark (check-small) on PR badges when review === "approved", making merge-ready PRs visually distinct at a glance
  • Add 10 Storybook stories covering every PR badge state: approved, pending review, changes requested, checks failing, checks pending, draft, merged, closed, no review, and approved+checks failing
  • Skip the pulsing checks-pending story in visual regression (non-deterministic animation frame)

Before/After

The branch icon (🔀) on every PR badge is identical regardless of review state. After this change, approved PRs show a small checkmark () instead, so you can scan a list of sessions and immediately see which are merge-ready.

marius-kilocode and others added 2 commits April 9, 2026 12:42
Replace the branch icon with a check-small icon when
pr.review === "approved", making merge-ready PRs visually
distinct at a glance. Add 10 Storybook stories covering every
PR badge color/state combination for visual regression.
@marius-kilocode marius-kilocode enabled auto-merge (squash) April 9, 2026 10:48
@marius-kilocode marius-kilocode merged commit b64d2ba into main Apr 9, 2026
14 checks passed
@marius-kilocode marius-kilocode deleted the lime-tractor branch April 9, 2026 10:54
jliounis pushed a commit to jliounis/kilocode that referenced this pull request May 18, 2026
)

* feat(agent-manager): show checkmark on approved PR badges

Replace the branch icon with a check-small icon when
pr.review === "approved", making merge-ready PRs visually
distinct at a glance. Add 10 Storybook stories covering every
PR badge color/state combination for visual regression.

* chore: update kilo-vscode visual regression baselines

---------
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.

2 participants