Repository navigation
The guide and the build page start with installing the skill (for main) - #44
Merged
Merged
Conversation
Until now the site's prompts sent a chat to read the h5p-author skill from /h5p/skill/. That worked only in chats that now take the skill as a zip, so the skill is installed instead, and the prompts name it. - The guide's first step is the install, with a line for Claude, ChatGPT, Grok, Open WebUI and agents with a terminal. Grok took Claude's zip under Skills, then Import, in the owner's test of 2026-11-13, and built from the two downloaded files after its npx timed out. - The build page's three steps are install, ask, play. - No page links to /h5p/skill/: the footer and the home page's skills card go to the install step. The page and its files keep answering, for the copies of the skill already installed and the prompts people copied before.
There was a problem hiding this comment.
No issues found across 7 files
Tip: cubic could auto-approve low-risk PRs like this, if it thinks it's safe to merge. Learn more
View guided diff | Turn on auto-fix | Re-trigger cubic
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #42. This is #43's commit, unchanged, for
main.#43 was stacked on #41 and was merged ten seconds after it, while its base was still
feat/skill-zips. So it went into that branch, not intomain, and the site still shows the old prompts. This pull request brings the same commit (2dbce42) tomain.What changes
/h5p/create-with-ai/): step 1 is "Install the skill, once", with lines for Claude, ChatGPT, Grok, Open WebUI and agents with a terminal, so there are six steps. The seven prompts say "Use the h5p-author skill to make…" and carry no address. The lead no longer says "nothing to install". In Help, "The chat can't open the address" becomes "The chat doesn't use the skill"./h5p/skill/any more./h5p/skill/keeps answering, for installed copies of the skill and for old prompts. Only one sentence on it changes: it names Grok.vite.config.ts, say what the bundles are for now that Grok was seen building from them.Titles, descriptions and the home card's text are untouched: Claude, ChatGPT and Grok all take a zip.
Checked
The commit is the one #43 carried:
pnpm buildpasses, 66 page checks in Chromium at 1280 px and 390 px pass, and the 34 damaged-link checks from #39 hold.mainhas nothing but #41 since the commit's parent, so it applies as it is. Firefox and WebKit were not run.Summary by cubic
The guide and the build page now start with installing the h5p-author skill, instead of sending chats to read its page. The old prompts worked only in chats that now take the skill as a zip, so the prompts now name the installed skill and carry no address. This change previously landed in a feature branch by mistake; this brings it to main.
/h5p/skill/anymore: the footer's second guide and the home card's "For a chat" go to the install step. The skill page stays, for installed copies of the skill and old prompts.Written for commit 2dbce42. Summary will update on new commits.