Skip to content

feat: sync agent skills through Prisma CLI - #61

Merged
AmanVarshney01 merged 1 commit into
mainfrom
codex/use-cli-skills
Aug 24, 2026
Merged

feat: sync agent skills through Prisma CLI#61
AmanVarshney01 merged 1 commit into
mainfrom
codex/use-cli-skills

Conversation

@AmanVarshney01

Copy link
Copy Markdown
Member

Summary

  • install prisma@next in generated Node projects and use its local prisma binary for ORM and Composer scripts
  • configure the supported agent targets in prisma.config.ts, then run prisma init after dependency installation so the CLI syncs skills from installed Prisma packages
  • keep skills current with the CLI-managed postinstall hook; preserve Nuxt's required nuxt prepare step before syncing
  • align the generated stack with the skills-bearing Composer 0.12 release and its Alchemy/Effect versions
  • consume the consolidated CLI's structured endpoint frame in the Composer dev E2E test

This keeps the old ORM skills installer disabled (--skip-skills / --no-skill), so generated projects no longer source skills from prisma/skills. Deno remains on its existing ORM-only path until the consolidated CLI can start under Deno.

Verification

  • bun run check
  • bun run typecheck
  • bun run test:unit
  • bun run test:e2e
  • scaffolded and production-built a Nuxt app; verified its CLI-managed prisma-composer skill and chained postinstall
  • Composer local dev returned seeded users through the generated app
  • Deno minimal scaffold/build still passes

@coderabbitai

coderabbitai Bot commented Aug 24, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 3b5d6d7b-237d-4350-a785-30360727dbcf

📥 Commits

Reviewing files that changed from the base of the PR and between dceb0e7 and e34f1dd.

📒 Files selected for processing (9)
  • src/constants/dependencies.ts
  • src/tasks/install.ts
  • src/tasks/setup-prisma.ts
  • templates/create/_shared/pnpm-workspace.yaml.hbs
  • templates/create/_shared/prisma.config.ts.hbs
  • templates/create/nuxt/package.json.hbs
  • tests/dependencies.test.ts
  • tests/e2e/create-prisma.e2e.test.ts
  • tests/install.test.ts

Included review availability: 3 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.


Summary by CodeRabbit

  • New Features

    • Generated projects now initialize Prisma agent skills for supported development tools.
    • Prisma configuration includes agent skill settings and Composer skill synchronization.
    • Nuxt projects automatically synchronize skills after installation.
  • Bug Fixes

    • Updated Prisma, Alchemy, Effect, and Composer package versions.
    • Improved Prisma command execution across supported package managers.
    • Updated generated projects to use the current Prisma configuration interface.

Walkthrough

The dependency map removes @prisma/cli-engine and adds updated Prisma, Composer, Alchemy, and Effect packages. Generated projects invoke the Prisma CLI directly and add a failure-tolerant skills:sync script. Non-Deno setup initializes agent skills. Prisma and Nuxt templates include agent configuration and skill synchronization. Tests validate dependency versions, generated commands, templates, Composer endpoint parsing, and generated project output.

Merge Risk: ⚪ Minimal · up to e34f1

The PR updates generated-project dependency and skill synchronization behavior with stated checks and scaffold validations passing; no actionable merge-blocking risk remains after normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the primary change: synchronizing agent skills through the Prisma CLI.
Description check ✅ Passed The description directly explains the Prisma CLI skill synchronization changes, dependency updates, generated scripts, tests, and verification.
Docstring Coverage ✅ Passed Docstring check was indeterminate for this PR — some files could not be analyzed in time. Not blocking.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/use-cli-skills
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch codex/use-cli-skills

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

PR preview published

  • Version: 0.9.1-pr.61.227.1
  • Tag: pr61
  • Run with Bun: bunx create-prisma@pr61
  • Run with npm: npx create-prisma@pr61
  • Run with Yarn: yarn dlx create-prisma@pr61
  • Run with pnpm: pnpm dlx create-prisma@pr61
  • Run with Deno: deno run -A npm:create-prisma@pr61
  • Workflow run: https://github.com/prisma/create-prisma/actions/runs/32734448026

@AmanVarshney01
AmanVarshney01 merged commit 319b1a5 into main Aug 24, 2026
6 checks passed
@AmanVarshney01
AmanVarshney01 deleted the codex/use-cli-skills branch August 24, 2026 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant