Skip to content

Added Template GoClaw#1303

Open
moskoweb wants to merge 11 commits intoeasypanel-io:mainfrom
moskoweb:goclaw
Open

Added Template GoClaw#1303
moskoweb wants to merge 11 commits intoeasypanel-io:mainfrom
moskoweb:goclaw

Conversation

@moskoweb
Copy link
Copy Markdown

Multi-agent AI gateway with teams, delegation & orchestration. Single Go binary, 11+ LLM providers, 5 channels.

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 19, 2026

Deploy Preview for easypanel-templates failed.

Name Link
🔨 Latest commit 4c1e77c
🔍 Latest deploy log https://app.netlify.com/projects/easypanel-templates/deploys/69e1628e7ed7040008668c4d

@Ahson-Shaikh
Copy link
Copy Markdown
Collaborator

Ahson-Shaikh commented Mar 23, 2026

Thanks for the GoClaw template contribution — it’s a strong start and the overall structure is good.
Before we move this to production, a few changes are required to make it reliable and consistent with our template standards.

Required changes before merge

Fix schema consistency for providerApiKey
Right now providerApiKey is marked as required, but the same field is described as optional with an empty default.
Please make this consistent. If optional is intended, remove it from required.

Pin container images to immutable versions
The template currently uses mutable tags (:full, :latest).
For production templates, please pin exact versions (or digests) for both GoClaw app and GoClaw UI images.
Align runtime layout with upstream behavior (or document the divergence clearly)
Current env/mount path layout differs from upstream compose defaults (data/skills/workspace/session structure).
Please either align with upstream defaults, or explicitly document why this change is necessary and confirm it has been validated end-to-end.

Correct metadata link labeling
The Website link currently points to the GitHub repository.
Please either provide a real website URL or relabel/remove it to avoid duplicate GitHub references.

Recommended improvements (non-blocking but strongly suggested)

Add X-Forwarded-Proto and X-Forwarded-Host headers in nginx proxy config.
Double-check Postgres defaults (user/db naming and DSN assumptions) for future compatibility.
Validate and share smoke-test results for: fresh install, UI access, websocket route, provider key wiring, and restart persistence.

Once the required items are updated, I can re-review quickly for production readiness.

@Ahson-Shaikh
Copy link
Copy Markdown
Collaborator

For serviceName and other related serviceNames, we prefer serviceName: input.appServiceName and input.appServiceName-db and so on for any other service.

@moskoweb
Copy link
Copy Markdown
Author

I made the suggested adjustments.

Copy link
Copy Markdown
Collaborator

@Ahson-Shaikh Ahson-Shaikh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have you also tested the PR on Easypanel?

Comment thread templates/goclaw/index.ts
serviceName: `${input.appServiceName}-db`,
password: databasePassword,
image: "pgvector/pgvector:pg18",
env: "PGDATA=/var/lib/postgresql/data",
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you need to specify this? The managed postgres instance would have its mounts already defined

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was necessary because PGVector 18 changed the file folder, and Easypanel didn't from version 16 to 18, and without that, it gave an error about the data loading folder.

Comment thread templates/goclaw/index.ts Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants