Repository navigation
feat(skills): add /skills/demos, grouped by what each Skill makes - #472
Merged
Merged
Conversation
The homepage section capped demos at six, so new ones fell off it. /skills/demos lists every demo under Films and launch videos, Landing pages, UI components, and Diagrams and explainers, each a prompt index with a shareable ?demo= link. The homepage keeps a teaser, films pinned first, and links the page. Indexable for "claude skills examples" (720/mo, KD 32), noindex below six demos.
Harlan's review: the trending page's full-width shell and header, the groups as compact headings in the left rail, no descriptions or jump chips, and one stage that shows the full prompt. The stage moves into DemoStage so the homepage teaser and the page share it. Adds Skill Demos to the header and the mobile menu.
…eir frame Harlan's review: page demos on /skills/demos show the real output in the sandboxed frame instead of a screenshot; films keep their own aspect ratio and never grow past their rendered width; the prompt label reads Prompt.
Contributor
🤖 MERGED
GitHub merged this pull request.
Review completed. No finding above 80/100. Selected findings run after merge and open separate pull requests. |
DemoStage reads liveUrl from HomeDemoItem, which the API already sends but the type never declared, so typecheck failed on #472.
2 of 6 tasks
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.
🔗 Linked issue
Follows #470 and #471.
❓ Type of change
📚 Description
The homepage demo section caps at six, so
make-interfaces-feel-betterandlemo-opuscarnever showed, and there was no room to record more./skills/demoslists every demo, grouped by what the Skill makes, which is the framing Harlan picked over the work-shaped tracks.makes, andshared/demo-groups.tsholds the list, the labels and the order; the recorder requires--makes. GLOSSARY.md notes that a group is not a track./skills/trendingboard shell: its header, a sticky left rail with the groups as compact headings and each demo as avatar, name and author, and one sticky stage that shows the full prompt, the output and the run command. Picking a demo writes?demo=owner/repo/namewithout moving the page, and a shared link opens on that demo. On phones the rail comes first and a pick scrolls to the stage.sandbox="allow-scripts", served under the CSP sandbox), not a screenshot. Films keep their own aspect ratio, never grow past their rendered width, and shrink to fit the screen; they no longer crop. The prompt label reads "Prompt" everywhere.DemoIndexand its stage intoDemoStage, which the page shares. The homepage keeps a teaser of six with the three films pinned first, and an "All 8 demos" link.page-admissions.tsfor "claude skills examples" (720 searches a month, KD 32, NuxtSEO 2026-10-06), and noindex below six demos. Cull path: remove the admission entry./skills/**stays out of the pages sitemap under the running SEO experiment, so the homepage link is how crawlers find it.skills-route-policy.tslists the page, or the middleware would 404 it before the router; the existing filesystem test caught that.