Skip to content

feat: SDK update for version 0.2.0 - #3

Open
atharvadeosthale wants to merge 1 commit into
mainfrom
dev
Open

feat: SDK update for version 0.2.0#3
atharvadeosthale wants to merge 1 commit into
mainfrom
dev

Conversation

@atharvadeosthale

@atharvadeosthale atharvadeosthale commented Jul 29, 2026

Copy link
Copy Markdown
Member

This PR contains updates to the plugin for version 0.2.0.

What's Changed

  • Breaking: Replaced local appwrite-api and appwrite-docs MCP servers with the hosted server at https://mcp.appwrite.io/
  • Removed: userConfig from plugin manifest, no longer needed with OAuth-authenticated hosted MCP
  • Updated: LICENSE changed from MIT to BSD-3-Clause
  • Updated: Descriptions reference the hosted Appwrite MCP server
  • Updated: SDK skills content refreshed

@atharvadeosthale atharvadeosthale changed the title feat: ClaudePlugin SDK update for version 0.2.0 feat: SDK update for version 0.2.0 Jul 29, 2026
Comment thread commands/deploy-site.md
|---------|-------------|
| `appwrite sites list-templates` | List available site templates |
| `appwrite sites get-template --template-id <ID>` | Get template details |
| `appwrite sites create-template-deployment --site-id <ID>` | Deploy from a template |

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Template deployment lacks source arguments

When a user follows this reference, create-template-deployment receives only --site-id, omitting the repository, owner, root directory, reference type, and reference required to identify the template source, so the CLI rejects the command for missing arguments.

Prompt To Fix With AI
This is a comment left during a code review.
Path: commands/deploy-site.md
Line: 139

Comment:
**Template deployment lacks source arguments**

When a user follows this reference, `create-template-deployment` receives only `--site-id`, omitting the repository, owner, root directory, reference type, and reference required to identify the template source, so the CLI rejects the command for missing arguments.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Fix in Claude Code Fix in Codex

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@ChiragAgg5k this is a legitimate issue

Image

Comment thread skills/cli/SKILL.md

### Function variables

Do not define function variables in `appwrite.config.json`. Put them in a `.env` file inside the configured function `path`. Variables are saved after they are pushed, so use `--with-variables` only when you want to create, replace, or remove the remote variables from the local `.env` file.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Variable sync can erase secrets

When --with-variables is used with a missing, empty, or unparseable .env, the CLI deletes every remote variable before finding no local values to recreate, causing production function or site configuration to be erased. The function and site guidance should explicitly warn about this destructive behavior before recommending the flag.

Prompt To Fix With AI
This is a comment left during a code review.
Path: skills/cli/SKILL.md
Line: 278

Comment:
**Variable sync can erase secrets**

When `--with-variables` is used with a missing, empty, or unparseable `.env`, the CLI deletes every remote variable before finding no local values to recreate, causing production function or site configuration to be erased. The function and site guidance should explicitly warn about this destructive behavior before recommending the flag.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Fix in Claude Code Fix in Codex

@greptile-apps

greptile-apps Bot commented Jul 29, 2026

Copy link
Copy Markdown

Greptile Summary

Updates the Claude plugin to version 0.2.0 and refreshes its Appwrite integration.

  • Replaces the local API/docs MCP servers and credential configuration with one hosted OAuth-enabled MCP server.
  • Expands the CLI skill with split configuration, updated function/site fields, query flags, webhooks, project management, and deployment-variable guidance.
  • Adds function and site command references, updates SDK permission terminology, and changes the repository license to BSD-3-Clause.

Confidence Score: 2/5

The template-deployment command and destructive variable-sync guidance need correction before merging.

The template example omits required source arguments, and variable synchronization can delete all remote function or site variables when the local environment file is missing, empty, or invalid.

Files Needing Attention: commands/deploy-site.md and skills/cli/SKILL.md

Important Files Changed

Filename Overview
.mcp.json Replaces two existing MCP servers with a single hosted HTTP server; no concrete configuration defect was established.
.claude-plugin/plugin.json Bumps the plugin version, updates licensing metadata, and removes credential-based user configuration for the hosted OAuth migration.
skills/cli/SKILL.md Expands the CLI guidance, but the variable synchronization instructions omit that a missing or invalid local environment file causes all remote variables to be deleted.
commands/deploy-site.md Adds site logs and template commands, but the template-deployment command omits required source arguments and cannot run as documented.
commands/deploy-function.md Adds execution-body, variable-management, and local-development references without an established defect.
README.md Documents the hosted MCP server, browser OAuth workflow, version update, and BSD-3-Clause licensing.

Fix All in Claude Code Fix All in Codex

Prompt To Fix All With AI
### Issue 1
commands/deploy-site.md:139
**Template deployment lacks source arguments**

When a user follows this reference, `create-template-deployment` receives only `--site-id`, omitting the repository, owner, root directory, reference type, and reference required to identify the template source, so the CLI rejects the command for missing arguments.

### Issue 2
skills/cli/SKILL.md:278
**Variable sync can erase secrets**

When `--with-variables` is used with a missing, empty, or unparseable `.env`, the CLI deletes every remote variable before finding no local values to recreate, causing production function or site configuration to be erased. The function and site guidance should explicitly warn about this destructive behavior before recommending the flag.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Reviews (1): Last reviewed commit: "chore: update ClaudePlugin SDK to 0.2.0" | Re-trigger Greptile

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