Skip to content

chore(cli): build against sdk-for-go v7.13.0 - #2075

Merged
ChiragAgg5k merged 1 commit into
mainfrom
chore/cli-go-sdk-v7.13.0
Oct 16, 2026
Merged

ChiragAgg5k merged 1 commit into
mainfrom
chore/cli-go-sdk-v7.13.0

Conversation

@ChiragAgg5k

@ChiragAgg5k ChiragAgg5k commented Oct 16, 2026 •

Copy link
Copy Markdown
Member

The 2.5.x spec now has createQuery on databases, documentsDB and tablesDB, total on list endpoints in activities, analytics, backups, domains, mongo, mysql and postgresql, and emails on messaging.createEmail/updateEmail. sdk-for-go v7.12.0 has none of them, so the CLI generated in cloud failed to compile (service.WithListEventsTotal undefined, undefined: databases.CreateQueryOption, …). sdk-for-go v7.13.0 adds them.

This bumps the pin in templates/cli/go.mod.twig. project updatePasskeyPolicy stays excluded in example.php: v7.13.0 still takes a required rpId.

Verification

  • The CLI generated by cloud for 2.5.x, built against v7.13.0: go build ./..., go vet ./... and go test ./... pass (they fail to compile on v7.12.0).
  • php example.php cli, gofmt -l examples/cli, go build/vet/test ./... in examples/cli pass.

Related: appwrite-labs/cloud#6538

@hansi-codes

hansi-codes Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

🟢 Tier S · Ready to merge

The new SDK version is available and contains the APIs this CLI pin is intended to provide; no regressions were found.

The CLI template now pins generated modules to sdk-for-go v7.13.0. The generation example comment reflects the new pin while continuing to exclude updatePasskeyPolicy because its parameter contract is still incompatible.

Verdict New comments Fixed Still open
✅ Approved 0 0 0
📂 Walkthrough · 2
File Change
templates/cli/go.mod.twig Bumped the generated CLI's sdk-for-go dependency to v7.13.0.
example.php Updated the comment describing why the passkey policy method remains excluded.

Reviewed 7ccc9db · Details · Comment @hansi-codes review to re-run, or mention @hansi-codes with a question.

@hansi-codes hansi-codes Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Tier S · Looks good to merge. Summary

@ChiragAgg5k ChiragAgg5k self-assigned this Oct 16, 2026
@ChiragAgg5k
ChiragAgg5k merged commit ef54435 into main Oct 16, 2026
61 checks passed
@ChiragAgg5k
ChiragAgg5k deleted the chore/cli-go-sdk-v7.13.0 branch October 16, 2026 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant