feat: enable Prisma Next completion parameter hints - #2003
Conversation
Signed-off-by: Steven McClankerton <tatarintsev+clanker@prisma.io>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (3)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. Summary by CodeRabbit
WalkthroughThe Prisma Next client now sends Priority: ⬇️ Low Merge Risk: ⚪ Minimal · up to The PR adds an opt-in capability advertisement with matching focused coverage and documentation; no actionable merge-blocking risk remains. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
Signed-off-by: Steven McClankerton <tatarintsev+clanker@prisma.io>
Signed-off-by: Steven McClankerton <tatarintsev+clanker@prisma.io>
Enables completion-triggered parameter hints for the Prisma Next VS Code client via the existing built-in command. No custom commands, keybindings, or Tab/Shift+Tab interception. Legacy clients unchanged.
Companion to prisma/orm#30312; requires that server support.
Verification: language-server build, extension/test typechecks and targeted lint/format checks passed before removing the redundant test. Editor UI behavior has not been manually verified.