feat: add/update ZeroClaw template (v0.7.3-beta.1051)#809
Open
theonlyhennygod wants to merge 1 commit intoDokploy:mainfrom
Open
feat: add/update ZeroClaw template (v0.7.3-beta.1051)#809theonlyhennygod 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.
API_KEY auto-generated instead of user-supplied
The api_key variable is populated with a randomly generated 64-character string via the password helper, but it is paired with PROVIDER=openrouter in the env block — indicating it is the credential for an external LLM provider (OpenRouter, etc.), not a local gateway secret. A randomly generated value cannot authenticate against any real AI provider, so the deployment will fail at runtime until the user manually replaces it. Consider leaving the variable empty and requiring the user to enter their own key, or at minimum adding a comment that the value must be overridden post-deploy.
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 blueprint (v0.7.3-beta.1051) with a docker-compose, template.toml, logo, and
meta.jsonentry. Two blocking issues exist: the Strapi entry has been accidentally deleted frommeta.json(its blueprint files are still present), and theAPI_KEYfor the LLM provider is populated with an auto-generated random string rather than a real user-supplied provider credential, which will cause the service to fail on first use.Confidence Score: 3/5
Not safe to merge — accidental Strapi removal breaks an existing template, and the API_KEY misconfiguration will cause ZeroClaw to fail on first use.
Two P1 issues are present: the unintended removal of the Strapi registry entry (which breaks an existing template for all users) and the incorrect use of a generated password as the user's external LLM provider API key. Both need to be fixed before this can be merged.
meta.json(Strapi removal) andblueprints/zeroclaw/template.toml(API_KEY generation)Comments Outside Diff (2)
meta.json, line 5952-5969 (link)The
strapientry has been deleted frommeta.json, butblueprints/strapi/(withdocker-compose.yml,template.toml,strapi.svg) still exists on disk. Removing it from the registry makes Strapi invisible in the Dokploy template browser while its blueprint files remain, leaving it in a broken/orphaned state. This change is not mentioned in the PR description and appears accidental — likely caused by a merge conflict or a stalemeta.jsonbase. The entry should be restored.blueprints/zeroclaw/zeroclaw.png, line 1 (link)Per project conventions, logo files should be SVG (~128×128 px) for sharper rendering at all sizes. PNG will work but is not the preferred format. If an SVG source is available, swapping it in would be better.
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