Remove Agent Skills from backend: API, SDKs, CLI, drop tables (skills purge, stage 3)#126
Open
mgoldsborough wants to merge 1 commit into
Open
Remove Agent Skills from backend: API, SDKs, CLI, drop tables (skills purge, stage 3)#126mgoldsborough wants to merge 1 commit into
mgoldsborough wants to merge 1 commit into
Conversation
…ll tables Stage 3 of the skills purge. mpak's backend is now bundle-only. BREAKING CHANGE: the /v1/skills API, mpak skill CLI commands, skill SDK methods, and the skills/skill_versions tables are removed. Published skills, skill installs, and skill badge embeds are no longer served. Skill packaging/distribution moves to the skills.sh / npx skills ecosystem. Registry: - delete routes/v1/skills.ts, SkillRepository, utils/skill-content.ts - unregister /v1/skills, drop skills swagger tag and Discord skill branch - remove skills from the Prisma repositories decoration - Prisma: drop Skill/SkillVersion models + migration to DROP the tables - rewrite seed.ts to seed example bundles (Package/Version/Artifact/Scan) - delete skill tests, prune skill mocks from shared test helpers Packages: - schemas: delete skill schema + skill validators - sdk-typescript: remove skill methods/types - sdk-python: remove generated skill types (regenerate from spec long-term) - cli: delete src/commands/skills, mpak skill namespace, completions; unified mpak search is bundle-only Verified against a throwaway Postgres: migration applies cleanly, skill tables dropped, no drift, seed idempotent. Full suite green: typecheck (7/7), build (6/6), tests (registry 125 + all packages), biome CI (209 files), ruff (python SDK).
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.
Stage 3 of 4: bundle-only backend
The largest stage of the mpak refocus. Removes Agent Skills from the registry API, both SDKs, the CLI, and the database. mpak's backend now does one thing: serve security-scanned MCPB bundles.
Builds on #124 (positioning) and #125 (web + docs).
Registry (
apps/registry)routes/v1/skills.ts,SkillRepository,utils/skill-content.ts; unregister/v1/skills, drop theskillsswagger tag and the Discord skill branchskillsfrom the Prisma repositories decorationSkill/SkillVersionmodels + a migration thatDROPs the tables (canonical Prisma-generated SQL)seed.tsto seed example bundles (Package → Version → Artifact → completed SecurityScan with trust levels), replacing the old skill seedPackages
searchSkills/getSkill/getSkillDownload/getSkillVersionDownload/downloadSkillBundle+ skill typessrc/commands/skills/, thempak skillnamespace, and skill completions; unifiedmpak searchis now bundle-only (keptbundle search --type, which is the unrelated server-type filter)Verification (all green)
Migration + seed validated against a throwaway Postgres:
prisma migrate deployapplies the drop cleanly ·migrate statusreports no drift · skill tables confirmed goneStatic suite:
pnpm typecheck7/7 ·pnpm build6/6 ·pnpm test(registry 125 + all packages) ·biome ci(209 files) ·ruff(python SDK) ✓Net
+479 / −6,285across 60 files.Merge ordering
Independent of #125 at the code level (the web app has its own HTTP client, not the SDK), but for the cleanest deploy, land #125 (web/docs) before or with this PR so the UI isn't briefly calling removed
/v1/skillsendpoints. CHANGELOG history is preserved; the breaking change is recorded under[Unreleased].