Skip to content

feat(skills): report agentic onboarding progress - #331

Merged
evanpurkhiser merged 1 commit into
mainfrom
evanpurkhiser/feat-skills-report-agentic-onboarding-progress
Aug 26, 2026
Merged

evanpurkhiser merged 1 commit into
mainfrom
evanpurkhiser/feat-skills-report-agentic-onboarding-progress

Conversation

@evanpurkhiser

@evanpurkhiser evanpurkhiser commented Aug 13, 2026 •

Copy link
Copy Markdown
Member

Keep Sentry’s getting-started UI synchronized with agent-led setup through silent, privacy-focused progress updates.

The guidance detects onboarding tokens in the initial prompt, defines stage and run-state transitions, handles update failures without interrupting setup, and reports multi-project and verification metadata at the relevant stages.

@evanpurkhiser
evanpurkhiser force-pushed the evanpurkhiser/feat-skills-report-agentic-onboarding-progress branch 4 times, most recently from 2f3d290 to 2af072e Compare August 14, 2026 00:13
@evanpurkhiser evanpurkhiser changed the title feat(skills): Report agentic onboarding progress feat(skills): report agentic onboarding progress Aug 14, 2026
@evanpurkhiser
evanpurkhiser marked this pull request as ready for review August 14, 2026 00:19
Comment thread src/skills/sentry-get-started/SKILL.md
Comment thread src/references/first-error-setup.md
Comment thread src/skills/sentry-get-started/SKILL.md
@evanpurkhiser
evanpurkhiser force-pushed the evanpurkhiser/feat-skills-report-agentic-onboarding-progress branch from b8f8e39 to dd365e9 Compare August 14, 2026 00:30
Comment thread src/references/first-error-setup.md
Comment thread src/skills/sentry-get-started/SKILL.md
@evanpurkhiser
evanpurkhiser force-pushed the evanpurkhiser/feat-skills-report-agentic-onboarding-progress branch 3 times, most recently from e29f0fb to 7aeb69c Compare August 14, 2026 00:44
Keep Sentry's getting-started UI synchronized with agent-led setup while limiting updates to privacy-safe operational state.
@evanpurkhiser
evanpurkhiser force-pushed the evanpurkhiser/feat-skills-report-agentic-onboarding-progress branch from 925b49b to a708bb8 Compare August 26, 2026 22:34
@evanpurkhiser
evanpurkhiser enabled auto-merge (squash) August 26, 2026 22:35
Comment on lines +36 to +45
> [!NOTE]
> If you are sending onboarding status updates, this stage is `create_project`: select
> an existing Sentry project or create one and obtain its DSN. Report `waiting` before
> requesting creation approval.
> Keep it `active` while selecting or provisioning every planned project, and report
> each validated slug in `update.extra.projectSlugs` as its DSN becomes usable.
> Report `completed` once all planned projects are usable, including all known validated
> slugs. For one existing project, complete it with `update.extra.projectSlugs` and
> `update.eventNote: Project already existed.` Do not complete the stage per project.
> This stage is unskippable.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bug: The create_project stage instructions omit the initial 'Report active' step, which is present in all other stages, breaking the expected agent state machine.
Severity: HIGH

Suggested Fix

Add an instruction at the beginning of the create_project stage documentation, similar to other stages, that directs the agent to 'Report active' before starting the project creation or selection process. This will ensure the state machine is entered correctly.

Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's
not valid.

Location: src/references/first-error-setup.md#L36-L45

Potential issue: The documentation for the `create_project` stage omits the instruction
for an agent to report an `active` status at the beginning of the process. All other
stages explicitly start with this instruction. An LLM agent following these instructions
will not report the initial `active` status, causing it to skip a step in its state
machine. This will lead to incorrect state transitions (e.g., starting with `waiting`
instead of `active`) when a user creates a new project, breaking the expected state
machine logic.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit a708bb8. Configure here.

unavailable, silently stop sending progress updates for the rest of the session and
continue setup normally.
Also stop if the reply says not to continue (`continueUpdates: false` or
`Continue updates: no`). Never report a status-call failure as an onboarding failure.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Auth failure permanently stops progress updates

High Severity

Discovery of onboarding_status_update is written as an immediate imperative, and any failed discovery permanently disables progress updates for the rest of the session. First-time getting-started users often have MCP disconnected, so search_sentry_tools fails before auth. The unauthenticated path never re-enables updates after the user connects, so the onboarding UI stays frozen even though setup continues.

Additional Locations (2)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit a708bb8. Configure here.

@evanpurkhiser
evanpurkhiser merged commit ddfc765 into main Aug 26, 2026
9 checks passed
@evanpurkhiser
evanpurkhiser deleted the evanpurkhiser/feat-skills-report-agentic-onboarding-progress branch August 26, 2026 22:45
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