feat: add/update ZeroClaw template (v0.7.1-beta.1049)#807
Open
theonlyhennygod wants to merge 1 commit intoDokploy:mainfrom
Open
feat: add/update ZeroClaw template (v0.7.1-beta.1049)#807theonlyhennygod wants to merge 1 commit intoDokploy:mainfrom
theonlyhennygod wants to merge 1 commit intoDokploy:mainfrom
Conversation
built with Refined Cloudflare Pages Action⚡ Cloudflare Pages Deployment
|
| @@ -0,0 +1,16 @@ | |||
| [variables] | |||
| main_domain = "${domain}" | |||
| api_key = "${password:64}" | |||
There was a problem hiding this comment.
Random value used for what may be an external LLM provider key
The template auto-generates a random 64-character string for api_key and injects it as API_KEY. Since PROVIDER is set to openrouter, this variable is almost certainly the user's LLM provider credential — a value only the user possesses. Deploying with a randomly-generated value will cause every instance to fail provider authentication silently. If API_KEY is instead ZeroClaw's own internal gateway token (unrelated to the upstream provider), please clarify this in a comment; otherwise the template should prompt the user to supply their own value rather than auto-generating one.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
ghcr.io/zeroclaw-labs/zeroclaw:${VERSION}Checklist
Testing
Links
Greptile Summary
This PR adds a new ZeroClaw template (v0.7.1-beta.1049) with a
docker-compose.yml,template.toml, logo, andmeta.jsonentry. However, it also unintentionally removes the existing Strapi entry frommeta.jsonwhile Strapi's blueprint files remain on disk, and theAPI_KEYvariable is auto-generated rather than accepting user input, which will likely break LLM provider authentication on every deployment.meta.json—blueprints/strapi/still exists; the registry entry must be restored.API_KEYauto-generated — withPROVIDER=openrouter, this is almost certainly an external provider credential that must be supplied by the user, not randomly generated.Confidence Score: 3/5
Not safe to merge — removes an existing template from the registry and likely ships a non-functional API key configuration.
Two P1 issues: the Strapi entry is silently dropped from meta.json (orphaning its blueprint files), and the auto-generated API_KEY will break LLM provider authentication for every ZeroClaw deployment. Both must be resolved before merging.
meta.json (Strapi removal) and blueprints/zeroclaw/template.toml (API_KEY generation)
Comments Outside Diff (2)
meta.json, line 5952-5971 (link)The Strapi template entry is deleted from
meta.json, but its blueprint files (blueprints/strapi/docker-compose.yml,template.toml,strapi.svg) still exist. This leaves Strapi's compose files orphaned and invisible in the template registry. This looks like an accidental side-effect — likely caused by runningdedupe-and-sort-meta.jsin a state where Strapi was already absent, or a merge conflict resolution that dropped the block. The Strapi entry should be restored.blueprints/zeroclaw/zeroclaw.png, line 1 (link)Per project conventions, logo files should be SVG (~128×128 px) for crisp rendering at all sizes. A PNG will work but may appear blurry in the UI at higher DPI. Consider providing an SVG version.
Context Used: AGENTS.md (source)
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Reviews (1): Last reviewed commit: "feat: add/update ZeroClaw template (v0.7..." | Re-trigger Greptile
Context used: