feat: add/update ZeroClaw template (v0.7.1)#802
Open
theonlyhennygod wants to merge 1 commit intoDokploy:mainfrom
Open
feat: add/update ZeroClaw template (v0.7.1)#802theonlyhennygod 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.
Auto-generated key may not work as an external provider key
The template generates a random string for api_key (used as API_KEY in the compose env), but ZeroClaw connects to an external AI provider via PROVIDER=openrouter. For AI assistant services the API_KEY is typically the user's own provider credential — an auto-generated random value will cause the service to fail to authenticate with OpenRouter on startup.
If API_KEY is instead an internal gateway auth token (not the LLM provider credential), a comment in the template clarifying this would remove the ambiguity.
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.1) for a Rust-based autonomous AI assistant. The compose file and template structure are well-formed and follow conventions, but two issues need attention before merging.
meta.json: The diff deletes the entirestrapientry — Strapi's blueprint files still exist but it becomes invisible in the registry. This appears to be an unintended merge/rebase artifact and must be restored.API_KEYis auto-generated: ZeroClaw connects to an external AI provider (PROVIDER=openrouter), andAPI_KEYis almost certainly the user's provider credential. Generating a random value will cause authentication failures at startup; the user should supply their own key.Confidence Score: 4/5
Not safe to merge until the unintended Strapi removal and the auto-generated API key issue are resolved.
Two P1 findings: the Strapi meta.json entry is silently dropped (breaking that template's discoverability), and the API_KEY is auto-generated in a way that will likely cause ZeroClaw to fail connecting to its AI provider. Both need to be addressed 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 diff removes the entire
strapientry frommeta.jsonbut the PR is only meant to add/update ZeroClaw. Strapi's blueprint files still exist in the repository — removing it frommeta.jsonmakes it invisible to users of the template registry. This looks like an accidental conflict/rebase artifact and needs to be restored.blueprints/zeroclaw/zeroclaw.png, line 1 (link)AGENTS.mdstates "Add logo file (SVG preferred, ~128x128px) to blueprint folder." Using a PNG is not a hard blocker, but an SVG would scale better across the Dokploy UI. Consider replacing with an SVG if one is available upstream.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: