Skip to content

Update CLI agent skill guidance#1501

Merged
ChiragAgg5k merged 1 commit intomasterfrom
update-cli-agent-skill
May 5, 2026
Merged

Update CLI agent skill guidance#1501
ChiragAgg5k merged 1 commit intomasterfrom
update-cli-agent-skill

Conversation

@ChiragAgg5k
Copy link
Copy Markdown
Member

What changed

  • Updated the generated Appwrite CLI agent skill for current CLI workflows.
  • Added multi-file appwrite.config.json guidance using includes for functions, sites, and webhooks.
  • Documented flag-based list queries, webhooks, project settings commands, deployment activation, completions, and update commands.
  • Replaced config vars examples with .env plus --with-variables guidance for functions and sites.

Testing

  • docker run --rm -v $(pwd):/app -w /app php:8.3-cli php example.php agent-skills
  • composer lint-twig
  • git diff --check

Notes

  • examples/agent-skills is generated successfully but ignored by this repository's examples/* rule, so the PR tracks the source template change.

@ChiragAgg5k ChiragAgg5k force-pushed the update-cli-agent-skill branch 2 times, most recently from c5bbd85 to 032abe8 Compare May 5, 2026 08:01
@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 5, 2026

Greptile Summary

  • Expands the CLI agent skill template with includes-based multi-file appwrite.config.json configuration, a new Webhooks section, Project Management commands, Maintenance Commands, and flag-based list query examples, all with updated field names that reflect the current Appwrite Cloud API (node-22, buildSpecification, etc.).
  • Replaces the deprecated per-function vars / deployment / live config fields and the old Homebrew install formula with the correct tap-based install (brew tap appwrite/appwrite && brew install appwrite/appwrite/appwrite) and .env + --with-variables variable workflow.
  • Issues flagged in previous review threads (duplicate appwrite login, duplicate appwrite completion install, and the webhook JSON format mismatch) have been addressed in this revision.

Confidence Score: 5/5

Safe to merge; all previously flagged issues are resolved and no new P0/P1 defects were found.

The only remaining finding is a P2 style inconsistency (missing comment in site variables block). No runtime-breaking, schema-breaking, or security issues were identified in the new content.

No files require special attention.

Important Files Changed

Filename Overview
templates/agent-skills/cli.md.twig Major expansion of CLI skill guidance: adds includes-based multi-file config, webhooks, project management, deployment activation, flag-based queries, and .env-based variable workflow; previously flagged issues (duplicate login, duplicate completion install, webhook array format) are all resolved.

Reviews (5): Last reviewed commit: "Update CLI agent skill guidance" | Re-trigger Greptile

Comment thread templates/agent-skills/cli.md.twig Outdated
Comment thread templates/agent-skills/cli.md.twig Outdated
Comment thread templates/agent-skills/cli.md.twig Outdated
@ChiragAgg5k ChiragAgg5k force-pushed the update-cli-agent-skill branch 3 times, most recently from 435be11 to a9b50f0 Compare May 5, 2026 08:21
Comment thread templates/agent-skills/cli.md.twig
@ChiragAgg5k ChiragAgg5k force-pushed the update-cli-agent-skill branch from a9b50f0 to 1628b0e Compare May 5, 2026 09:05
@ChiragAgg5k ChiragAgg5k merged commit 0c7627f into master May 5, 2026
57 checks passed
@ChiragAgg5k ChiragAgg5k deleted the update-cli-agent-skill branch May 5, 2026 10:43
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