Repository navigation
feat(agent-handoff): Add reusable setup prompt UI - #130465
evanpurkhiser wants to merge 1 commit into
Conversation
github-actions
Bot
added
the
Scope: Frontend
Story previewsPreview the stories changed in this PR on the Vercel deployment: Preview deployment: https://sentry-gcdlmr4q8.sentry.dev |
| Body = Fragment, | ||
| Footer = Fragment, |
There was a problem hiding this comment.
Not super in love with this. Maybe what would be better is if you have it like this
<AgentHandoff>
<AgentHandoff.Details />
<AgentHandoff.Action trigger={... defaults to button, can be ovverideen like dropdown menu trigger} />
</AgentHandoff>then we can compose this inside a dropdown instead
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
862a4a8 to
b231196
Compare
| } catch { | ||
| throw new Error(t('Unable to create your agent prompt. Please try again.')); | ||
| } | ||
| } |
There was a problem hiding this comment.
Copy reuses consumed handoff token
Medium Severity
After a successful copy, later clicks reuse the cached prompt until expiry or a pinning change. The backend handoff is single-use, so a second copy can paste an already-claimed token and leave setup failing for up to five minutes.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit b231196. Configure here.
b231196 to
8f4618f
Compare
8f4618f to
9c4ed49
Compare
9c4ed49 to
79ae131
Compare
79ae131 to
148dc33
Compare
623e679 to
f689e35
Compare
f689e35 to
626f738
Compare
626f738 to
c77aa54
Compare
c77aa54 to
1f8acef
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
There are 2 total unresolved issues (including 1 from previous review).
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 1f8acef. Configure here.
| path: {organizationIdOrSlug: organization.slug}, | ||
| }), | ||
| data: {flow, disableIpPinning}, | ||
| }); |
There was a problem hiding this comment.
Onboarding mint omits required context
High Severity
Copying always POSTs {flow: 'onboarding', disableIpPinning} with no context. The handoff API requires context.runId for the onboarding flow and rejects the request otherwise, so minting a prompt cannot succeed and the copy action fails every time.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 1f8acef. Configure here.
Provide reusable handoff content for Sentry setup surfaces: an ordered setup checklist, a preloaded Markdown guide preview, and a copy action that mints a private prompt only when clicked.
Keep IP pinning enabled by default, show the sharing warning when it is disabled, and support retrying preview or clipboard failures. Includes an inline story and component/hook coverage for preview loading, pinning choices, expiration, clipboard recovery, and dismissal.