Skip to content

fix: consolidate PR summary status details#2406

Merged
neversettle17-101 merged 8 commits into
mainfrom
ao/agent-orchestrator-34/pr-summary-parts
Jul 5, 2026
Merged

fix: consolidate PR summary status details#2406
neversettle17-101 merged 8 commits into
mainfrom
ao/agent-orchestrator-34/pr-summary-parts

Conversation

@neversettle17-101

Copy link
Copy Markdown
Collaborator

Summary

  • replace separate PR status strip + Needs attention panel with a single three-part summary: CI, Merge, Review
  • attach failing checks, merge conflicts/blocked reasons, and review comment links under their corresponding part
  • add richer web-preview mock SCM summaries so linked CI/conflict/review examples render in preview mode

Verification

  • npm run typecheck
  • npm run test -- --run src/renderer/lib/pr-display.test.ts

Notes

@neversettle17-101 neversettle17-101 force-pushed the ao/agent-orchestrator-34/pr-summary-parts branch from 538f949 to ff72eb4 Compare July 4, 2026 08:39
@neversettle17-101

neversettle17-101 commented Jul 4, 2026

Copy link
Copy Markdown
Collaborator Author

Before:
Screenshot 2026-07-05 at 1 11 36 PM

After:
Screenshot 2026-07-05 at 1 18 41 PM

Screenshot 2026-07-05 at 1 10 59 PM Screenshot 2026-07-04 at 1 23 58 PM

@neversettle17-101 neversettle17-101 left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Changes requested. The consolidated summary looks generally aligned with the intended CI / Merge / Review grouping, but the new maxLinks path can hide actionable links without a correct overflow count in the compact board/table views.

Comment thread frontend/src/renderer/components/PRSummaryDisplay.tsx

@neversettle17-101 neversettle17-101 left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Changes requested. The follow-up fixes the compact maxLinks={2} case, but overflow is still lost whenever the source has more than the helper-level three links, so the summary can still hide actionable checks/files/reviewers without any +N indicator.

Comment thread frontend/src/renderer/components/PRSummaryDisplay.tsx Outdated

@illegalcall illegalcall left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approved

@neversettle17-101 neversettle17-101 force-pushed the ao/agent-orchestrator-34/pr-summary-parts branch from fd3fa85 to c989df0 Compare July 5, 2026 13:15
@neversettle17-101 neversettle17-101 merged commit 21294e6 into main Jul 5, 2026
3 checks passed
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