chore: regenerate SDK packages from updated schemas#821
Merged
pyramation merged 2 commits intomainfrom Mar 15, 2026
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Regenerates all three SDK packages (
constructive-sdk,constructive-react,constructive-cli) from the updated.graphqlschema files insdk/constructive-sdk/schemas/using theschemaDir-based codegen pipeline.Schemas processed:
admin,app(empty/skipped),auth,objects,publicKey changes reflected in the regenerated output:
private.graphqlschema removed (was 44k+ lines); its types are now consolidatedsearchTsv: FullTextremoved fromUserInput/UserPatchmutation inputs (tsvector columns are auto-populated by DB triggers and should not be manually set)SEARCH_TSV_ASC/SEARCH_TSV_DESCremoved fromUserOrderByenum (ordering by raw tsvector is meaningless)All generated code compiles cleanly (
pnpm buildpasses). CI is green (41/41).Review & Testing Checklist for Human
sdk/constructive-sdk/schemas/are correct and complete — these are the source of truth for all generated SDK code. Theprivate.graphqlremoval is the biggest change; confirm nothing was lost that should have been kept.publicschema-types and ORM —publichas 103 tables and is the largest target. Verify the generated types and ORM operations look correct for a few key tables (e.g.,User,Database,App).searchTsvremoval from mutation inputs is intentional — this aligns with the graphile-build-pg rc.8 upgrade (PR fix: upgrade graphile ecosystem to latest RC versions with compatibility fixes #820) where tsvector columns are now hidden from insert/update. If any downstream consumer relies on settingsearchTsvmanually, this would be a breaking change.Notes
pnpm generatein each SDK package. No hand-written code was modified.app.graphqlschema produced no tables (empty schema) and was skipped by the generator, as expected.Link to Devin session: https://app.devin.ai/sessions/7dae13db67bc4cc9b971e75984be2187
Requested by: @pyramation