feat: add/update Construct template (v2026.5.3-beta.90)#846
feat: add/update Construct template (v2026.5.3-beta.90)#846kaveone wants to merge 1 commit intoDokploy:mainfrom
Conversation
built with Refined Cloudflare Pages Action⚡ Cloudflare Pages Deployment
|
| @@ -0,0 +1,17 @@ | |||
| [variables] | |||
| main_domain = "${domain}" | |||
| api_key = "${password:64}" | |||
There was a problem hiding this comment.
Auto-generated key replaces required external API key
${password:64} generates a random 64-character string, but API_KEY must be a real LLM provider credential (e.g., an OpenRouter API key). A randomly generated value will be passed to the container, making all LLM calls fail silently from the start. Users would need to discover this and manually replace the value post-deployment. Consider removing the default or replacing with a placeholder that signals user input is required (e.g., api_key = ""), and add a comment or description so users know to supply their actual key.
| [variables] | ||
| main_domain = "${domain}" | ||
| api_key = "${password:64}" | ||
| provider = "openrouter" |
There was a problem hiding this comment.
Provider hardcoded, defeating "swap anything" claim
The provider variable is set to "openrouter" and wired directly into the env without any user-configurable default. Since the project advertises "swap anything" for the LLM provider, users will be surprised they cannot change this without editing files directly. Consider leaving it unset or exposing it as a clearly user-overridable variable (e.g., provider = "openrouter" with an accompanying comment listing supported values).
|
Superseded by Construct v2026.5.7; closing in favor of the newer PR. |
|
Superseded by v2026.5.7. |
Summary
ghcr.io/kumihoio/construct-os:${VERSION}Checklist
Testing
Links
Greptile Summary
This PR adds the Construct AI assistant template (v2026.5.3-beta.90) and removes a duplicate Strapi entry from
meta.json— the base branch had two identical Strapi objects, so the dedup is correct. The docker-compose structure follows all conventions (v3.8,exposeonly,restart: unless-stopped, named volume).Confidence Score: 3/5
The P1 issue (auto-generated random key replacing a required external API credential) should be resolved before merging — the service will deploy but be non-functional by default.
One clear P1 defect: the template generates a random password for API_KEY instead of prompting the user to supply a real LLM provider key, so the service starts but cannot make any API calls. The remaining findings are P2 (logo format, provider flexibility). The Strapi removal is valid deduplication.
blueprints/construct/template.toml — the api_key variable default needs rethinking.
Comments Outside Diff (1)
blueprints/construct/construct.png, line 1 (link)Per the project's AGENTS.md contribution guide, SVG is the preferred logo format (~128×128px). A PNG logo will work functionally, but SVG scales better across different display sizes and DPIs used in the Dokploy UI.
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: