chore: remove bundled agent skill cluster#97
Conversation
Drops the committed prisma-cli skill cluster and all of its supporting machinery: - skills/ source (skill SKILL.md files, journey tests, docs) - scripts/prepare-skills.mjs (pnpm prepare install hook) and the validate-skills lint/test scripts - lint:skills, prepare, and test:skills package scripts plus the skills and gray-matter devDependencies - .gitignore entries for the generated skill installs - README and CLI help references that pointed users at the cluster (app deploy longDescription + install example, packages/cli README section) and their test assertions
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (17)
💤 Files with no reviewable changes (17)
Summary by CodeRabbit
WalkthroughThis PR removes the agent skills infrastructure from the repository. The 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
luanvdw
left a comment
There was a problem hiding this comment.
@AmanVarshney01 looks good!
I checked for stale skill references and did not find any relevant remaining ones. Local validation on the PR head: pnpm install --frozen-lockfile, pnpm --filter @prisma/cli test -- app.test.ts (523 passed), pnpm lint (exit 0 with existing warnings), and git diff --check all passed.
What
Removes the committed
prisma-cliagent skill cluster and all of its supporting machinery from the repo.Why
The skill cluster (
skills/) and its install/validation tooling are no longer something we want bundled in this repo.Changes
skills/— the three SKILL.md clusters (prisma-cli,prisma-cli-deploy-nextjs,prisma-cli-feedback), journey tests, and docs.scripts/prepare-skills.mjs(thepnpm prepareinstall hook) andscripts/validate-skills.mjs+ its test.package.json— removedlint:skills,prepare, andtest:skillsscripts, plus theskillsandgray-matterdevDependencies. Lockfile regenerated..gitignore— removed entries for the generated skill installs (.agents/skills/,.claude/skills/,skills-lock.json).packages/cli/README.mdsections, and theapp deployhelplongDescription/ install example incommand-meta.tsthat pointed users at the cluster, plus the matching test assertions.Verification
pnpm install— clean (lockfile updated, skill deps dropped)pnpm lint— passes (pre-existing warnings only)pnpm test— 523 passed (42 files)