feat: add/update ZeroClaw template (v0.7.0-beta.1030)#796
Open
theonlyhennygod wants to merge 1 commit intoDokploy:mainfrom
Open
feat: add/update ZeroClaw template (v0.7.0-beta.1030)#796theonlyhennygod 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
|
| [config] | ||
| env = [ | ||
| "API_KEY=${api_key}", | ||
| "PROVIDER=openrouter", |
There was a problem hiding this comment.
PROVIDER hardcoded; docker-compose.yml allows override
template.toml hardcodes PROVIDER=openrouter, but docker-compose.yml uses ${PROVIDER:-openrouter} which accepts an env var override. Users deploying through the Dokploy template UI have no way to change the provider. If multi-provider support is intentional, consider exposing PROVIDER as a configurable variable in template.toml.
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 the ZeroClaw template (v0.7.0-beta.1030) — a Rust-based AI assistant. The new blueprint files (
docker-compose.yml,template.toml, logo) are well-formed and follow repository conventions, but themeta.jsondiff also removes the existing Strapi template entry, which appears unintentional and would break Strapi availability for all Dokploy users.strapientry is deleted frommeta.jsonwith no explanation. This looks like a merge conflict artifact and should be restored before merging.Confidence Score: 4/5
Not safe to merge until the accidental Strapi deletion from meta.json is reverted.
The ZeroClaw blueprint itself is correct and complete, but the unintended removal of the Strapi entry from meta.json is a present defect that would silently break an existing well-known template for all users.
meta.json — the Strapi entry deletion must be reverted.
Comments Outside Diff (2)
meta.json, line 5952-5958 (link)The PR removes the entire
strapientry frommeta.json(18 lines deleted starting at the original line 5955), but the PR description and title make no mention of removing Strapi. This looks like a merge conflict artifact or accidental deletion. Removing it would make the Strapi template unavailable in the registry for all Dokploy users.blueprints/zeroclaw/zeroclaw.png, line 1 (link)The AGENTS.md style guide states "SVG preferred, ~128x128px" for blueprint logos. A PNG will work but an SVG would scale better across different display sizes and is the expected format for this repository.
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