Conversation
merge dev to main (v3.0.0-beta.28)
merge dev to main (v3.0.0-beta.29)
merge dev to main (v3.0.0-beta.30)
merge dev to main (v3.0.0-beta.31)
merge dev to main (v3.0.0-beta.32)
merge dev to main (v3.0.0-beta.33)
merge dev to main (v3.0.0-beta.34)
merge dev to main (v3.0.0)
chore: update CI sample update script (#531)
merge dev to main
📝 WalkthroughWalkthroughThis PR adds Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Claude finished @ymc9's task —— View job Code Review Complete ✅I've completed the review of this pull request. Here's my comprehensive analysis: Review Tasks
📋 SummaryThis PR is titled "feat(cli): implement watch mode for generate" but the actual changes shown are regenerated schema files only. The core watch mode implementation was already merged in commit 🔍 Key FindingsWhat Changed:
Watch Mode Implementation Analysis:From reviewing the actual implementation files ( ✅ Strengths
|
There was a problem hiding this comment.
Pull request overview
This PR adds support for ExistsArgs type definitions across all generated TypeScript schema files. The changes enable the use of the new exists query operation in the ZenStack ORM type system.
- Adds
ExistsArgsimport from@zenstackhq/ormpackage - Generates corresponding
{ModelName}ExistsArgstype aliases for all models
Reviewed changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tests/runtimes/edge-runtime/schemas/input.ts | Added ExistsArgs types for User and Post models |
| tests/runtimes/bun/schemas/input.ts | Added ExistsArgs types for User and Post models |
| tests/regression/test/issue-503/input.ts | Added ExistsArgs types for InternalChat, Message, and Media models |
| tests/regression/test/issue-422/input.ts | Added ExistsArgs types for Session, User, and Profile models |
| tests/regression/test/issue-204/input.ts | Added ExistsArgs type for Foo model |
| tests/e2e/orm/schemas/typing/input.ts | Added ExistsArgs types for User, Post, Profile, Tag, Region, and Meta models |
| tests/e2e/orm/schemas/typed-json/input.ts | Added ExistsArgs type for User model |
| tests/e2e/orm/schemas/todo/input.ts | Added ExistsArgs types for Space, SpaceUser, User, List, and Todo models |
| tests/e2e/orm/schemas/procedures/input.ts | Added ExistsArgs type for User model |
| tests/e2e/orm/schemas/petstore/input.ts | Added ExistsArgs types for User, Pet, and Order models |
| tests/e2e/orm/schemas/omit/input.ts | Added ExistsArgs types for User, Post, Base, and Sub models |
| tests/e2e/orm/schemas/name-mapping/input.ts | Added ExistsArgs types for User and Post models |
| tests/e2e/orm/schemas/json/input.ts | Added ExistsArgs type for Foo model |
| tests/e2e/orm/schemas/delegate/input.ts | Added ExistsArgs types for User, Comment, Asset, Video, RatedVideo, Image, and Gallery models |
| samples/orm/zenstack/input.ts | Added ExistsArgs types for User, Profile, and Post models |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary by CodeRabbit
Release Notes
✏️ Tip: You can customize this high-level summary in your review settings.