Skip to content

feat: add creating-product-tours skill#135

Open
raquelmsmith wants to merge 2 commits into
mainfrom
posthog-code/add-creating-product-tours-skill
Open

feat: add creating-product-tours skill#135
raquelmsmith wants to merge 2 commits into
mainfrom
posthog-code/add-creating-product-tours-skill

Conversation

@raquelmsmith
Copy link
Copy Markdown
Member

Summary

Adds a new creating-product-tours skill that walks an agent through building an in-app product tour for the user's own application using PostHog feature flags for targeting and PostHog events for tracking (started / step viewed / completed / dismissed).

The skill is shaped like other self-contained docs-only skills (quack, posthog-best-practices) — a single config.yaml plus a description.md that drops in verbatim as SKILL.md. No external docs fetched. It ships in the posthog-all mega-plugin.

How it was tested

  • pnpm run build — the generator picks up the new group and emits creating-product-tours.zip (4.4 KB) with the right SKILL.md frontmatter (name, description, metadata).
  • pnpm test — all 42 tests still pass.
  • bash scripts/scan-prompt-injection.sh dist/skills — passes; no prompt-injection patterns flagged in any of the 92 archives.
  • Spot-checked the generated SKILL.md against the source description.md and confirmed the frontmatter is correct.

Notes

The build emits a [WARN] No plugin mapping for group "creating-product-tours" line, which matches the behavior of the existing quack and posthog-best-practices top-level groups. The skill still ends up in the mega-plugin. Happy to add a dedicated plugin entry to marketplace.yaml if reviewers want a standalone posthog-product-tours plugin instead.


Created with PostHog Code

Adds a docs-only skill that walks an agent through building an in-app
product tour using PostHog feature flags for targeting and PostHog events
for tracking. Surfaces in the mega-plugin alongside posthog-best-practices
and quack.

Generated-By: PostHog Code
Task-Id: f1029b67-83d1-466e-a4fd-56f132ae88d8
@github-actions
Copy link
Copy Markdown

🧙 Wizard CI

Run the Wizard CI and test your changes against wizard-workbench example apps by replying with a GitHub comment using one of the following commands:

Test all apps:

  • /wizard-ci all

Test all apps in a directory:

  • /wizard-ci basic-integration
  • /wizard-ci misc
  • /wizard-ci revenue

Test an individual app:

  • /wizard-ci basic-integration/android
  • /wizard-ci basic-integration/angular
  • /wizard-ci basic-integration/astro
Show more apps
  • /wizard-ci basic-integration/django
  • /wizard-ci basic-integration/fastapi
  • /wizard-ci basic-integration/flask
  • /wizard-ci basic-integration/javascript-node
  • /wizard-ci basic-integration/javascript-web
  • /wizard-ci basic-integration/laravel
  • /wizard-ci basic-integration/next-js
  • /wizard-ci basic-integration/nuxt
  • /wizard-ci basic-integration/python
  • /wizard-ci basic-integration/rails
  • /wizard-ci basic-integration/react-native
  • /wizard-ci basic-integration/react-router
  • /wizard-ci basic-integration/sveltekit
  • /wizard-ci basic-integration/swift
  • /wizard-ci basic-integration/tanstack-router
  • /wizard-ci basic-integration/tanstack-start
  • /wizard-ci basic-integration/vue
  • /wizard-ci misc/quack-quack
  • /wizard-ci revenue/stripe

Results will be posted here when complete.

@raquelmsmith raquelmsmith requested review from a team and adboio May 18, 2026 19:42
@raquelmsmith raquelmsmith marked this pull request as ready for review May 18, 2026 19:42
@raquelmsmith
Copy link
Copy Markdown
Member Author

@adboio what did I (aka my agent) miss here?

Copy link
Copy Markdown
Member

@edwinyjlim edwinyjlim left a comment

Choose a reason for hiding this comment

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

this is good on the context mill side

for the wizard, I'll need to create a Q&A interface to handle free text input. new feature!

i can work with @adboio to take this over the finish line

Comment on lines +58 to +59
import { useEffect, useState, useCallback } from "react";
import posthog from "posthog-js";
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

if we want to support more frameworks outside of react we can split skills out into variants in the config.yaml

Comment on lines +7 to +14
Ask the user these questions before writing any code:

1. **Goal** — what should the user be able to do after completing this tour? (e.g., "create their first dashboard", "connect a data source")
2. **Steps** — list each step: what element should it point to, what does it say, how does the user advance (click a button in the tooltip, or click the highlighted element itself)?
3. **Audience** — who sees it? All users, new users only, users on a specific plan, users who haven't done something yet?
4. **Trigger** — auto-show on page load, trigger from a button, or chain from another tour?
5. **Show frequency** — once ever, once per session, or every time the user visits?
6. **Tech stack** — React, Vue, vanilla JS? This determines component shape.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

oooh we need to build Q&A UI on the wizard-side, we don't have handle free text user input yet

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Do you want to do that now or should I somehow update this skill so that it doesn't require that? idk how I'd do that since the tour itself needs a lot of this detail in order to be good

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

yep I'm working on wizard PR now, will update when it's ready

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