feat: SDK update for version 12.2.0#90
Conversation
Greptile SummarySDK update for Appwrite Console version 12.2.0, bumping
Confidence Score: 4/5Safe to merge once the queries type mismatch in listRepositoryBranches is corrected; all other changes are well-formed. The src/services/vcs.ts — the queries type needs correcting at lines 313, 326, and 328–331. Important Files Changed
Reviews (12): Last reviewed commit: "chore: update Console SDK to 12.2.0" | Re-trigger Greptile |
| required: false, | ||
| min: null, // optional | ||
| max: null, // optional | ||
| default: null, // optional |
There was a problem hiding this comment.
Wrong property name for default value in example
The example uses default: null but the TypeScript method signature expects the property to be named xdefault. Callers who copy this snippet verbatim will get a TypeScript type error (since default is not in the params interface) and the default value will be silently dropped from the request. The same mismatch exists in docs/examples/databases/update-big-int-attribute.md, docs/examples/tablesdb/create-big-int-column.md, and docs/examples/tablesdb/update-big-int-column.md.
There was a problem hiding this comment.
this is also present in the main repo as well https://github.com/appwrite/sdk-for-console/blob/main/docs/examples/tablesdb/create-integer-column.md
default and not xdefault
| * @returns {Promise<Models.BranchList>} | ||
| */ | ||
| listRepositoryBranches(params: { installationId: string, providerRepositoryId: string }): Promise<Models.BranchList>; | ||
| listRepositoryBranches(params: { installationId: string, providerRepositoryId: string, search?: string, queries?: string }): Promise<Models.BranchList>; |
There was a problem hiding this comment.
The
queries parameter in listRepositoryBranches is typed as string (singular) but the JSDoc describes it as "Array of query strings" and every other queries parameter in this SDK (including the adjacent listRepositories in the same file) is typed as string[]. Passing a plain string will serialize incorrectly — the server expects an array, so query filters like Query.limit() will be silently ignored or cause a server-side error.
| listRepositoryBranches(params: { installationId: string, providerRepositoryId: string, search?: string, queries?: string }): Promise<Models.BranchList>; | |
| listRepositoryBranches(params: { installationId: string, providerRepositoryId: string, search?: string, queries?: string[] }): Promise<Models.BranchList>; |
This PR contains updates to the SDK for version 12.2.0.
Whats Changed
bigintcreate/update APIs for legacy Databases attributesbigintcreate/update APIs forTablesDBcolumnskey,resourceType,resourceId, andsecret