runkids/skillshare-app/skillshare-app-pr

Land a change in skillshare-app

The facts and rules live in the wiki. This skill only orders the steps. Before step 1, load python3 scripts/ai-context.py development, plus the one topic for the area you are changing (frontend, architecture, background-services).

Steps

1. Branch

1. Run git fetch and check git status in the main checkout. Leave the user's

uncommitted work alone.

1. Create a worktree on a new branch from origin/main, as "Shipping a change"

describes.

1. If several requests arrive together, handle them in one PR only when they touch the

same files. Otherwise use one PR each.

2. Change

1. Before editing, rg every use of anything you will change or remove: commands, the

bridge, settings fields, capabilities, tests, wiki and README.

1. When you remove a feature, remove all of it: UI, Rust commands, plugins,

capabilities, CliMeta fields, lockfiles, tests and docs. Then rg again until nothing is left.

1. Update the wiki topics and README that describe the old behavior in the same change.

3. Verify

1. Run every command in the "Verifying a change" table and fix whatever fails.
2. For anything visible, apply the branch to the main checkout and take screenshots in

noVNC. Read each image before you call it verified. Then revert the main checkout.

1. If the user sends a screenshot, read it first. Convert it with sips when it is not

a real PNG.

4. Pull request

1. Commit, push and run gh pr create with the What/Verification body.
2. The user has delegated merging, so run scripts/merge-pr.sh <pr> in the background

and keep working.

1. On a real failure, read gh run view <id> --log-failed, fix it, push a new commit,

and run the script again.

5. Finish

1. Pull main into the main checkout and remove the worktree.
2. Tell the user, in their language:

• what changed and the PR link
• what was verified, and how: tests or screenshots
• anything assumed or left out

1. Offer a release in one line. Release only when they ask, using

skillshare-app-release.

Rules

• Never --no-verify. Never force-push or amend a pushed commit; merge origin/main

into the branch instead.

• Report a failing check with its output. Never merge on red.
• One PR, one purpose. The commit body says why.