-
-
Notifications
You must be signed in to change notification settings - Fork 16
chore: regenerate schemas #575
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.