Skip to content

feat(blueprint): add Verdaccio - private node.js package registry#735

Merged
Siumauricio merged 1 commit intoDokploy:canaryfrom
wladiarce:add-verdaccio-template
Mar 5, 2026
Merged

feat(blueprint): add Verdaccio - private node.js package registry#735
Siumauricio merged 1 commit intoDokploy:canaryfrom
wladiarce:add-verdaccio-template

Conversation

@wladiarce
Copy link
Copy Markdown
Contributor

@wladiarce wladiarce commented Mar 3, 2026

What is this PR about?

New PR of [Template Name]

Checklist

Before submitting this PR, please make sure that:

Issues related (if applicable)

Close automatically the related issues using the keywords: closes #ISSUE_NUMBER

Screenshots or Videos

image image

Greptile Summary

This PR adds a Verdaccio (private Node.js package registry) blueprint to the Dokploy templates collection. The template includes a docker-compose.yml, a template.toml with a fully embedded Verdaccio config (via [[config.mounts]]), an SVG logo, and a meta.json entry. The overall approach and configuration are correct; however, the docker-compose.yml violates two mandatory project conventions defined in AGENTS.md:

  1. ports used instead of expose: Dokploy handles all external routing; using ports directly exposes port 4873 on the host rather than routing through Dokploy's managed reverse proxy.
  2. Missing restart policy: The service does not define restart: unless-stopped (or restart: always), meaning the container will not automatically recover from crashes or host reboots.

Both issues are explicitly prohibited/required by the project's Docker Compose conventions and must be fixed before this template can be safely deployed.

Confidence Score: 2/5

  • Not safe to merge — the docker-compose.yml violates two mandatory project conventions that will prevent safe deployment in Dokploy.
  • The PR contains two verified violations of mandatory Docker Compose conventions defined in AGENTS.md. The use of ports instead of expose bypasses Dokploy's routing system and exposes the service directly on the host, while the missing restart policy means the service cannot recover automatically. Both must be fixed before deployment. The other files (meta.json, template.toml, and SVG) are correct.
  • blueprints/verdaccio/docker-compose.yml requires fixing: replace ports with expose and add restart: unless-stopped

Last reviewed commit: fc72f73

(3/5) Reply to the agent's comments like "Can you suggest a fix for this @greptileai?" or ask follow-up questions!

Context used:

  • Rule from dashboard - AGENTS.md (source)

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. new-template labels Mar 3, 2026
@dosubot
Copy link
Copy Markdown

dosubot bot commented Mar 3, 2026

Related Documentation

Checked 7 published document(s) in 1 knowledge base(s). No updates required.

How did I do? Any feedback?  Join Discord

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 3, 2026

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
templates ✅ Ready (View Log) Visit Preview fc72f73

Copy link
Copy Markdown

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

4 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

Comment on lines +8 to +9
ports:
- 4873
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

ports must be replaced with expose

Per the project's Docker Compose conventions (AGENTS.md), ports must never be included in blueprint files. Dokploy handles external routing via its own proxy, so using ports would expose port 4873 directly on the host instead of routing through Dokploy's managed network. Use expose instead to only announce the port to the Docker network.

Suggested change
ports:
- 4873
expose:
- 4873

Context Used: Rule from dashboard - AGENTS.md (source)

Comment on lines +4 to +5
verdaccio:
image: verdaccio/verdaccio:6
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Missing restart policy

Per the project's Docker Compose conventions (AGENTS.md), every service must include restart: unless-stopped or restart: always. Without this, the container will not automatically recover after a crash or host reboot.

Suggested change
verdaccio:
image: verdaccio/verdaccio:6
verdaccio:
image: verdaccio/verdaccio:6
restart: unless-stopped

Context Used: Rule from dashboard - AGENTS.md (source)

@Siumauricio Siumauricio merged commit 58d4a57 into Dokploy:canary Mar 5, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-template size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants