Skip to content

Conversation

ChiragAgg5k
Copy link
Member

@ChiragAgg5k ChiragAgg5k commented Oct 11, 2025

Summary

  • Added TablesDB service support with --tables-db flag (new default)
  • Moved legacy Databases service to --databases flag for backward compatibility
  • Upgraded Appwrite Python SDK from 11.1.0 to 13.4.1
  • Updated README documentation to reflect new service flags
  • Bumped package version to 0.2.8

Changes

  • TablesDB Service: New --tables-db flag enables the TablesDB API (replaces Databases as default)
  • Legacy Support: Databases API moved to --databases flag for users who need the legacy service
  • SDK Upgrade: Updated to Appwrite SDK 13.4.1 for latest features and bug fixes
  • Documentation: Updated README table to show TablesDB as primary option and Databases as legacy

Migration Guide

Users can continue using the legacy Databases service by explicitly passing the --databases flag. New users will automatically get TablesDB by default.

Test Plan

  • Verified all service flags work correctly
  • Confirmed TablesDB is the new default when no flags are specified
  • Tested that --all flag includes both TablesDB and legacy Databases services

Summary by CodeRabbit

  • New Features
    • Introduced TablesDB service with a new --tables-db flag and made it the default when no services are specified.
    • Added CLI flags to enable additional APIs: --functions, --messaging, and --locale.
  • Refactor
    • Switched default from Databases to TablesDB; legacy Databases can be enabled via --databases (legacy).
  • Documentation
    • Updated CLI arguments table to reflect new flags and the legacy Databases entry.
  • Chores
    • Bumped version to 0.2.8 and updated dependency ranges.

- Add TablesDB service with --tables-db flag as new default
- Move Databases service to --databases flag (legacy support)
- Upgrade Appwrite SDK from 11.1.0 to 13.4.1
- Bump version to 0.2.8
@coderabbitai
Copy link

coderabbitai bot commented Oct 11, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

This change replaces the CLI's primary Databases integration with TablesDB: README updates --databases--tables-db, adds --functions, --messaging, and --locale flags, and reintroduces a legacy --databases flag. server.py now registers TablesDB by default (with an explicit --databases to enable the legacy Databases service), updates server_version to 0.2.8, and leaves other services unchanged. Package versions are bumped (pyproject.toml and server.json to 0.2.8) and the Appwrite SDK requirement is raised to >=13.4.1.

Possibly related PRs

Suggested reviewers

  • ItzNotABug

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title succinctly summarizes the two primary changes in the pull request by highlighting the addition of TablesDB support and the upgrade of the Appwrite SDK to version 13.4.1, which directly aligns with the PR’s objectives and scope. It is clear, concise, and immediately informs reviewers about the core enhancements without extraneous details.

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1fb80f3 and ab9d052.

📒 Files selected for processing (1)
  • src/mcp_server_appwrite/server.py (6 hunks)

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
README.md (1)

85-88: Update the default-service note to match TablesDB behavior.

This paragraph still claims the default server ships only with the Databases tools, but the code now enables TablesDB by default. Please rewrite the note so it reflects TablesDB as the default and clarifies that the legacy Databases API requires --databases.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cd595fd and 1fb80f3.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (4)
  • README.md (2 hunks)
  • pyproject.toml (1 hunks)
  • server.json (1 hunks)
  • src/mcp_server_appwrite/server.py (6 hunks)
🧰 Additional context used
🧬 Code graph analysis (1)
src/mcp_server_appwrite/server.py (2)
src/mcp_server_appwrite/tool_manager.py (1)
  • register_service (10-13)
src/mcp_server_appwrite/service.py (1)
  • Service (6-127)

@ChiragAgg5k ChiragAgg5k merged commit 0da7e3f into main Oct 11, 2025
@ChiragAgg5k ChiragAgg5k deleted the add-tablesdb-support branch October 11, 2025 09:57
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.

2 participants