Skip to content

[Improve] Show all linked pull requests in task surfaces - #1166

Merged
daniel-lxs merged 4 commits into
developfrom
feature/multi-pr-task-links-2w6elbv08zimg
Aug 8, 2026
Merged

[Improve] Show all linked pull requests in task surfaces#1166
daniel-lxs merged 4 commits into
developfrom
feature/multi-pr-task-links-2w6elbv08zimg

Conversation

@roomote-roomote

@roomote-roomote roomote-roomote Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Opened on behalf of Daniel Riccio. Follow up by mentioning @roomote-roomote, in the web UI, or in Slack.

What changed

Task workspace headers and Task Info render every linked pull request with its canonical URL. Slack thread and automation footers expose all active linked PRs instead of selecting one. Automation footer actions use unique IDs and stay within Slack's actions-block limit.

Why this change was made

Tasks can span multiple repositories and pull requests. Showing only the first or latest PR hid part of the work and made multi-PR tasks difficult to review.

Impact

Users can see and open the complete active PR set from the existing task surfaces, while compact task-list badges retain their existing behavior. Provider-specific PR URLs are preserved, and multi-PR automation footers remain valid Slack blocks.

Screenshots

Single linked pull request

Seeded task showing PR 101 in the task header and Task Info

Multiple linked pull requests

Seeded task showing PR 201 and PR 202 together in the task header and Task Info

Related PRs

@roomote-community

roomote-community Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

No code issues found. See task

  • packages/slack/src/automation-root-footer.ts:28 Reused Slack action IDs make multi-PR automation footers invalid.
  • packages/slack/src/automation-root-footer.ts:25 Too many PR buttons can exceed Slack's 25-element actions-block limit.

Reviewed 0009795

Comment thread packages/slack/src/automation-root-footer.ts Outdated
Comment thread packages/slack/src/automation-root-footer.ts Outdated
@roomote-roomote

Copy link
Copy Markdown
Contributor Author

Addressed both unresolved Slack automation-footer comments in commit 6d1dc655: PR action IDs are unique per button, and the PR button list is capped at 24 to preserve Slack's 25-element actions-block limit alongside Go to task. Added regression tests; Slack tests and typecheck pass.

@roomote-roomote

roomote-roomote Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

Updated the branch from develop and fixed the stale SDK footer assertion in commit 00097954. The required Test check is now green, all current checks are successful or intentionally skipped, and the seeded single-PR and multi-PR task states both render correctly. No repository-level blocker remains; the PR is still draft and must be marked ready for review before GitHub will allow merging.

@daniel-lxs
daniel-lxs marked this pull request as ready for review August 8, 2026 17:25
@daniel-lxs
daniel-lxs merged commit a6ee7e9 into develop Aug 8, 2026
18 checks passed
@daniel-lxs
daniel-lxs deleted the feature/multi-pr-task-links-2w6elbv08zimg branch August 8, 2026 17:25
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