Remove Agent Skills from web + docs, redirect to bundles (skills purge, stage 2)#125
Open
mgoldsborough wants to merge 2 commits into
Open
Remove Agent Skills from web + docs, redirect to bundles (skills purge, stage 2)#125mgoldsborough wants to merge 2 commits into
mgoldsborough wants to merge 2 commits into
Conversation
…bundles Stage 2 of the skills purge. mpak's web UI and docs are now bundle-only. Inbound skill links resolve on-domain to bundle equivalents. Web: - delete SkillsPage, SkillDetailPage, PublishSkillsPage, generated skill schema - App.tsx routes: /skills + /skills/* -> /bundles, /publish/skills -> /publish (React Router Navigate); nginx.conf adds matching server-side 301s - prune skills from nav, homepage hero/FAQ, About, publish gateway/page, lib/api (searchSkills/getSkill/getSkillDownloadUrl), BadgeSection (bundle-only) - SEO/build: prerender, sitemap, feed, og-image, manifest, llms.txt -> bundles only Docs: - delete docs/skills/* and cli/skills.mdx; drop Skills sidebar section - prune skill refs from index, ecosystem, quickstart, browsing, cli/*, registry/* Verified: web typecheck, 60 tests, biome lint, docs build, web build, prerender (10 routes, no skills) all green.
These 6 files under apps/docs/.astro/ were committed before the root .gitignore added the .astro/ rule (.gitignore:10), so they have been tracked by mistake and carried reorder churn on every docs build (this PR otherwise showed 38 lines of Map-reorder noise in content-modules.mjs). astro build regenerates the whole .astro/ dir, so untracking them is safe and removes the churn + future merge conflicts. No .gitignore change needed; the ignore rule already exists.
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
1 participant
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.
Stage 2 of 4: bundle-only web UI + docs
mpak is being refocused to handle only MCPB bundles. This removes all Agent Skills surfaces from the web app and docs site. Skill URLs are redirected on-domain (per decision: redirect to mpak.dev URLs, not off-domain).
Builds on the positioning refocus in #124.
Redirects (both layers)
/skills(listing)/bundles(listing)301+ React Router<Navigate>/skills/*(deep)/bundles(listing)301+ React Router<Navigate>/publish/skills/publish301+ React Router<Navigate>Note: there is no 1:1 skill→bundle mapping, so deep
/skills/@scope/nameURLs collapse to the/bundleslisting rather than to a specific bundle. This preserves the domain/path and avoids 404s on previously-indexed URLs; the tradeoff is that search engines may treat the many-to-one collapse as soft-404s for the deep URLs. No more specific target exists. nginx uses^~ /skills/prefix matching so/skills/@scope/nameredirects while/skillsfoodoes not; client-side<Navigate>matches the repo's existing convention (/browse→/bundles).Web
SkillsPage,SkillDetailPage,PublishSkillsPage, generatedskill.tsschemalib/apiskill methods,BadgeSection(now bundle-only)prerender,generate-sitemap,generate-feed,generate-og-image,manifest.json,llms.txt, committedsitemap.xml/feed.xmlall bundle-onlyDocs
docs/skills/*(5 pages) +cli/skills.mdx; removed the Skills sidebar sectionindex,ecosystem,quickstart,browsing,cli/{search,show,completion},registry/{api,access-model,how-it-works}apps/docs/.astro/cache files (already covered by.gitignore:10, committed before that rule; removes per-build reorder churn). astro build regenerates them.Surface staging (intentional)
The docs now omit
mpak skill *commands, but the CLI still ships them until stage 3 (#126), which deletes thempak skillnamespace. Docs narrower than the CLI is the safe direction; #126 closes the gap.Verification (all green)
typecheck✓ · 60 tests ✓ · biome lint ✓