feat: SDK update for version 0.2.0 - #3
Conversation
| |---------|-------------| | ||
| | `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 | |
There was a problem hiding this 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.
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.There was a problem hiding this comment.
@ChiragAgg5k this is a legitimate issue
|
|
||
| ### 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. |
There was a problem hiding this 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.
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.
Greptile SummaryUpdates the Claude plugin to version 0.2.0 and refreshes its Appwrite integration.
Confidence Score: 2/5The 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
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 |
This PR contains updates to the plugin for version 0.2.0.
What's Changed
appwrite-apiandappwrite-docsMCP servers with the hosted server athttps://mcp.appwrite.io/userConfigfrom plugin manifest, no longer needed with OAuth-authenticated hosted MCP