Skip to content

feat(events): add 'tutorial skipped' default event (ST-286) - #177

Open
iav-uk wants to merge 1 commit into
mainfrom
iav-286-tutorial-skip
Open

feat(events): add 'tutorial skipped' default event (ST-286)#177
iav-uk wants to merge 1 commit into
mainfrom
iav-286-tutorial-skip

Conversation

@iav-uk

@iav-uk iav-uk commented Aug 13, 2026

Copy link
Copy Markdown

No description provided.

@iav-uk
iav-uk requested a lite review from Copilot August 13, 2026 16:01
@linear-code

linear-code Bot commented Aug 13, 2026

Copy link
Copy Markdown

ST-286

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds first-class support for marking tutorials as skipped, exposing the capability across the public API surface (Go interfaces, protobuf/OpenAPI), and introducing a corresponding analytics/publisher event.

Changes:

  • Extend TutorialsSystem with a Skip(...) method and introduce a TUTORIAL_STATE_SKIPPED state.
  • Add a new RPC (RPC_ID_TUTORIALS_SKIP) plus TutorialSkipRequest to protobuf and OpenAPI.
  • Add a new publisher event constructor (tutorialSkipped) and document the change in the changelog.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
tutorials.go Adds Skip(...) to the exported TutorialsSystem interface.
hiro.proto Adds RPC id, request message, and tutorial state for “skipped”.
hiro-openapi.yml Exposes the new tutorials skip RPC endpoint and schemas.
events_satori.go Adds a new tutorialSkipped publisher event constructor.
CHANGELOG.md Documents the new skip functionality/event under Unreleased.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread events_satori.go
Comment thread tutorials.go
@iav-uk
iav-uk requested a review from a team August 13, 2026 17:03
@iav-uk
iav-uk marked this pull request as ready for review August 13, 2026 17:03
Comment thread hiro.proto
Comment thread events_satori.go
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.

3 participants