[CI][PR Subtask] Add PR author as collaborator to PR subtask #5218
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit af853df. Configure here.
| github-reviewer-user: ${{ github.event.requested_reviewer.login }} | ||
| github-pr-author-user: ${{ github.event.pull_request.user.login }} |
There was a problem hiding this comment.
Interestingly I noticed that while you are now correctly added a collaborator, somehow I was not, even though the task is assigned to me. This behavior is still fine since it achieves the goal (the PR author is now notified when the task is marked complete) so I'm not sure this is actually an issue - maybe it will mean that if you comment on the PR review task, I won't see it?
Feel free to add a comment on the PR review task and ping me when you did so, I'll check whether I got notified. If I didn't we should update the automation to also add the reviewer as a collaborator.
samsymons
left a comment
There was a problem hiding this comment.
Left one comment about something I'm not sure is an issue. Overall though this does achieve the success criteria so it's all good - feel free to merge the upstream apple-toolbox PR then merge this. Thanks!
8a5296e to
cc92420
Compare
Thanks Sam, I will fix it in the subsequent PR. |

Task/Issue URL: https://app.asana.com/1/137249556945/project/1206329551987282/task/1215310281455434?focus=true
Tech Design URL:
CC:
Description
This PR adds the PR author as a collaborator to each PR review task, so that comments/updates in the PR review task are automatically surfaced in their inbox.
This PR depends on Apple Toolbox PR duckduckgo/apple-toolbox#18
Testing Steps
Impact and Risks
What could go wrong?
Quality Considerations
Notes to Reviewer
Internal references:
Definition of Done | Engineering Expectations | Tech Design Template
Note
Low Risk
Internal CI wiring only; no app or user-facing runtime changes, though subtask collaborator assignment depends on the external apple-toolbox release.
Overview
When a reviewer is requested on a PR, the Create or Update PR Subtask step in
create_asana_pr_subtask.ymlnow passes the pull request author's GitHub login and user type intoduckduckgo/apple-toolbox’sasana-create-pr-subtaskaction (alongside the existing reviewer mapping). That lets the toolbox action add the PR author as a collaborator on the Asana PR review subtask so task comments and updates show up in their inbox.Behavior still runs only on
review_requestedand still depends on the companion change in apple-toolbox for the new inputs to take effect.Reviewed by Cursor Bugbot for commit cc92420. Bugbot is set up for automated code reviews on this repo. Configure here.