feat: add/update Construct template (v2026.4.24-beta.53)#820
Open
kaveone wants to merge 1 commit intoDokploy:mainfrom
Open
feat: add/update Construct template (v2026.4.24-beta.53)#820kaveone wants to merge 1 commit intoDokploy:mainfrom
kaveone wants to merge 1 commit intoDokploy:mainfrom
Conversation
built with Refined Cloudflare Pages Action⚡ Cloudflare Pages Deployment
|
| "links": { | ||
| "github": "https://github.com/KumihoIO/construct-os", | ||
| "website": "https://construct.com/", | ||
| "docs": "https://github.com/KumihoIO/construct-os#readme" |
There was a problem hiding this comment.
Website URL points to an unrelated product
"website": "https://construct.com/" is the homepage of Construct (a game-development tool), not the KumihoIO project this template represents. Linking to an unrelated website will confuse users researching this template.
Suggested change
| "docs": "https://github.com/KumihoIO/construct-os#readme" | |
| "website": "https://github.com/KumihoIO/construct-os", |
| restart: unless-stopped | ||
| environment: | ||
| - API_KEY=${API_KEY} | ||
| - PROVIDER=${PROVIDER:-openrouter} |
There was a problem hiding this comment.
PROVIDER env var inconsistency with template.toml
docker-compose.yml uses ${PROVIDER:-openrouter} (shell default syntax), but template.toml injects PROVIDER=openrouter as a hard-coded string with no variable. Since PROVIDER is never exposed as a user-configurable variable, consider aligning both files to the same hard-coded form.
Suggested change
| - PROVIDER=${PROVIDER:-openrouter} | |
| - PROVIDER=openrouter |
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/KumihoIO/construct-os:${VERSION}Checklist
Testing
Links
Greptile Summary
This PR adds a new Construct template (v2026.4.24-beta.53) for the KumihoIO AI personal assistant, with correct Docker Compose structure,
template.tomlvariable wiring, andmeta.jsonmetadata. However, it contains two P1 issues that should be resolved before merging.strapientry is deleted frommeta.jsonwith no explanation — this appears to be an unintentional side-effect and would break the existing Strapi template for all users.\"website\": \"https://construct.com/\"points to an unrelated game-development tool, not the KumihoIO project.Confidence Score: 3/5
Not safe to merge — the unintentional Strapi removal and wrong website URL are blocking issues.
Two P1 findings are present: the deletion of the existing Strapi entry from meta.json (which would remove a widely-used template) and a website link pointing to a completely different product. Both must be fixed before merging.
meta.json — contains the Strapi deletion and the incorrect website URL for Construct.
Comments Outside Diff (2)
meta.json, line 5968-5983 (link)This PR deletes the existing
strapitemplate entry frommeta.json. The Strapi removal appears to be a side-effect of runningdedupe-and-sort-meta.jslocally in a state where theblueprints/strapi/directory was missing, or an accidental edit. This PR only intends to add Construct — removing Strapi is a breaking change for users who rely on that template and is not mentioned in the PR description.blueprints/construct/construct.png, line 1 (link)Per
AGENTS.md, logos should be "SVG preferred, ~128x128px". A PNG will work but an SVG would scale better in 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 Construct template (v20..." | Re-trigger Greptile
Context used: