Skip to content

fix: require Console Python SDK 0.8.1 - #125

Merged
ChiragAgg5k merged 1 commit into
mainfrom
fix/console-sdk-0.8.1
Oct 6, 2026
Merged

ChiragAgg5k merged 1 commit into
mainfrom
fix/console-sdk-0.8.1

Conversation

@ChiragAgg5k

Copy link
Copy Markdown
Member

Bumps appwrite-console to >=0.8.1,<0.9. In 0.8.1 the SDK models accept null for Cloud fields that previously failed response parsing (appwrite-labs/cloud#6334). This should stop these Sentry issues once deployed:

Issue Model Fields
MCP-1G, MCP-1S Organization / OrganizationList paymentMethodId
MCP-1F PaymentMethodList mandateId, state, lastError
MCP-1P, MCP-1N Invoice / InvoiceList clientSecret, lastError
MCP-1W DedicatedDatabaseRestoration targetTime

The SDK upgrade also adds the analytics service and avatars.update_photo / delete_photo to the catalog:

  • Hosted OAuth: 41 services and 1016 tools (was 40 and 1007).
  • API-key stdio: still 26 services and 663 tools. avatars.update_photo and delete_photo accept only a session or JWT, never an API key, so they go in API_KEY_EXCLUDED_METHODS. analytics isn't on the API-key allowlist, so it stays hidden there.

Tests: test_console_sdk_accepts_null_billing_fields checks that the pinned SDK accepts null for the fields above. It fails 7 subtests on 0.8.0 and passes on 0.8.1. The catalog count tests and docs (docs/tool-surface.md, docs/appwrite-mcp-flow.svg) are updated.

Verification: ruff check, black --check and pyright are clean, and python -m unittest discover -s tests/unit passes.

@hansi-codes

hansi-codes Bot commented Oct 6, 2026

Copy link
Copy Markdown

🟢 Tier S · Ready to merge

The dependency and lockfile agree, catalog filtering remains consistent with service registration, and no concrete defects were found.

Raises the Console Python SDK requirement to >=0.8.1,<0.9 to accept nullable Cloud response fields. Updates the OAuth catalog counts and keeps the new avatar mutations and analytics service hidden from API-key mode, with regression coverage and matching documentation.

Verdict New comments Fixed Still open
✅ Approved 0 0 0
📂 Walkthrough · 6
File Change
pyproject.toml Requires appwrite-console >=0.8.1,<0.9.
uv.lock Locks appwrite-console to 0.8.1 and updates the package requirement.
src/mcp_server_appwrite/catalog_policy.py Excludes avatar photo updates and deletion from API-key catalogs.
tests/unit/test_server.py Updates OAuth catalog expectations and checks nullable SDK fields and API-key exclusions.
docs/tool-surface.md Documents the SDK version and expanded OAuth catalog.
docs/appwrite-mcp-flow.svg Updates the hosted catalog service and tool counts.

Reviewed b26e73f · 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 merged commit f639532 into main Oct 6, 2026
5 checks passed
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