feat: add/update ZeroClaw template (v0.7.0-beta.1047)#804
Open
theonlyhennygod wants to merge 1 commit intoDokploy:mainfrom
Open
feat: add/update ZeroClaw template (v0.7.0-beta.1047)#804theonlyhennygod 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
|
| @@ -5952,24 +5952,6 @@ | |||
| "media" | |||
There was a problem hiding this comment.
Unrelated removal of Strapi template entry
The diff removes the entire "strapi" entry from meta.json even though blueprints/strapi/ still exists with all its files intact. This will cause Strapi to disappear from the template registry entirely, breaking any users relying on it — with no justification in the PR description. This appears to be an accidental edit that should be reverted.
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.0-beta.1047) with a
docker-compose.yml,template.toml, logo, andmeta.jsonentry. The blueprint itself follows Dokploy conventions correctly (version 3.8,exposeoverports, named volumes, secret generation via${password:64}), but the PR also inadvertently removes the existing Strapi entry frommeta.jsonwhile theblueprints/strapi/directory and all its files remain untouched.meta.jsonentry should be restored — its removal is unrelated to this PR and would silently hide Strapi from the template registry.Confidence Score: 4/5
Not safe to merge until the accidental Strapi entry deletion from meta.json is reverted.
The ZeroClaw blueprint itself is well-formed, but the PR inadvertently removes an unrelated Strapi entry from meta.json, which would break Strapi's visibility in the template registry. This P1 issue must be fixed before merging.
meta.json — Strapi entry was accidentally removed and must be restored.
Comments Outside Diff (1)
blueprints/zeroclaw/zeroclaw.png, line 1 (link)AGENTS.md states "Add logo file (SVG preferred, ~128x128px) to blueprint folder." The current logo is a PNG file. Consider converting to SVG for better scaling and consistency with other templates.
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