Skip to content

feat: Add Seanime and remove strapai duplicate entry#808

Open
EnnioMrk wants to merge 11 commits intoDokploy:canaryfrom
EnnioMrk:seanime
Open

feat: Add Seanime and remove strapai duplicate entry#808
EnnioMrk wants to merge 11 commits intoDokploy:canaryfrom
EnnioMrk:seanime

Conversation

@EnnioMrk
Copy link
Copy Markdown

@EnnioMrk EnnioMrk commented Apr 19, 2026

What is this PR about?

Add Seanime and remove the strapai duplicate entry

Checklist

Before submitting this PR, please make sure that:

Screenshots or Videos

image

Greptile Summary

This PR adds a new Seanime blueprint and removes a duplicate strapi entry from meta.json. All previously flagged concerns (missing version: "3.8", unpinned images, missing expose directive, empty [[config.mounts]] block, and version: "latest" in meta.json) have been addressed. The only remaining nit is that the version field in meta.json is 3.6.1-rootless while the actual image tag is v3.6.1-rootless.

Confidence Score: 5/5

Safe to merge — all prior blocking concerns have been resolved and only a trivial style nit remains.

All P0/P1 findings from previous review rounds are now fixed: images are pinned, the expose directive is present, version: 3.8 is declared, and the empty mounts block is gone. The only remaining finding is a missing v prefix in the meta.json version field, which is P2 and does not block merge.

No files require special attention.

Comments Outside Diff (1)

  1. blueprints/seanime/template.toml, line 13 (link)

    P2 Empty [[config.mounts]] block

    The [[config.mounts]] table array is declared but contains no fields. This may cause a TOML parse error or create a null/empty mount entry at deploy time. If no custom mounts are needed, remove this section entirely.

  2. meta.json, line 5969-5985 (link)

    P1 Unintentional removal of strapi entry

    The strapi template entry has been deleted from meta.json in this PR, even though the PR only intends to add Seanime. This looks like an accidental deletion caused by a rebase/merge conflict. The strapi entry should be restored.

Reviews (5): Last reviewed commit: "v1.0.6" | Re-trigger Greptile

Context used:

  • Context used - AGENTS.md (source)

Copilot AI review requested due to automatic review settings April 19, 2026 01:56
@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Apr 19, 2026
@dosubot dosubot Bot added the new-template label Apr 19, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 19, 2026

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

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

@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. and removed size:S This PR changes 10-29 lines, ignoring generated files. labels Apr 19, 2026
Comment thread blueprints/seanime/docker-compose.yml
Comment thread blueprints/seanime/docker-compose.yml Outdated
Comment thread blueprints/seanime/docker-compose.yml
Comment thread meta.json
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new Dokploy blueprint for deploying Seanime and registers it in the template index.

Changes:

  • Added a new blueprints/seanime/ template (Compose + Dokploy template.toml + logo).
  • Added a meta.json entry for the new seanime template.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.

File Description
meta.json Registers the new seanime template in the central catalog.
blueprints/seanime/docker-compose.yml Defines the Seanime service (plus an init step) and persistent mounts.
blueprints/seanime/template.toml Declares the domain mapping and Dokploy config for the blueprint.
blueprints/seanime/seanime.png Adds the template logo asset referenced by meta.json.
Comments suppressed due to low confidence (1)

blueprints/seanime/template.toml:13

  • config.mounts is declared but the mount entry is empty; the template validator requires each mount to include at least filePath (string) and content (string). Either remove the [[config.mounts]] section entirely or fill in the required fields so validation passes.

Comment thread blueprints/seanime/docker-compose.yml
Comment thread blueprints/seanime/docker-compose.yml
Comment thread meta.json Outdated
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated 4 comments.

Comment thread meta.json Outdated
Comment thread blueprints/seanime/docker-compose.yml Outdated
Comment thread blueprints/seanime/docker-compose.yml Outdated
Comment thread blueprints/seanime/docker-compose.yml Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@EnnioMrk
Copy link
Copy Markdown
Author

@greptileai

@EnnioMrk
Copy link
Copy Markdown
Author

@greptileai The deletion of strapi was intentional because it was duplicate.

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Apr 19, 2026

Based on the diff, only one strapi entry existed in meta.json before this PR, and it was removed without any other strapi entry being present. The search confirms strapi appears once in the base branch and zero times in the PR head.

If the duplicate was in a different JSON structure or a previous state of the file that predates this branch, that context isn't visible here. But from what I can see in this diff, removing the only strapi entry would cause Strapi to disappear from the template catalog entirely — which seems unintentional regardless of the motivation.

Could you point to where the duplicate entry is? If there are genuinely two strapi entries somewhere in the file, I'm happy to accept the correction.

Tip: You can give me more context with [.greptile/rules](https://www.greptile.com/docs/code-review/custom-standards) so I get this right next time.

@EnnioMrk
Copy link
Copy Markdown
Author

@greptileai The Diff only shows the deletion. The duplicate entry was already present.

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Apr 19, 2026

You're right, and I apologize for the false alarm. The search_code tool only indexes the default branch, so I can't directly verify the PR head — but your explanation is consistent with the evidence. If the base branch had a duplicate strapi entry (a common rebase/merge artifact), this deletion would be intentional cleanup, not accidental removal, and strapi would still be present elsewhere in the file.

I'll withdraw that blocker. The PR looks good to merge.

Tip: You can give me more context with .greptile/rules so I get this right next time.

@EnnioMrk
Copy link
Copy Markdown
Author

@greptileai update the original pr message to reflect this

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Apr 19, 2026

Looks like I don't have permission to update the PR directly. Here's the updated description you can paste in yourself:


What is this PR about?

New PR to add Seanime — a self-hosted anime streaming template.

All prior review findings have been resolved (pinned image, version: "3.8", expose, empty [[config.mounts]], and meta.json version).

Remaining Blocker

The strapi template entry was accidentally removed from meta.json, likely due to a merge/rebase conflict. This must be restored before merging.

Checklist

Before submitting this PR, please make sure that:

Screenshots or Videos

(keep your existing screenshot tag)

@EnnioMrk
Copy link
Copy Markdown
Author

@greptileai

Comment thread meta.json
@EnnioMrk EnnioMrk changed the title Add Seanime feat: Add Seanime and remove strapai duplicate entry Apr 19, 2026
@EnnioMrk
Copy link
Copy Markdown
Author

@greptileai

@EnnioMrk EnnioMrk marked this pull request as ready for review April 19, 2026 02:47
Copilot AI review requested due to automatic review settings April 19, 2026 02:47
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated no new comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-template size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants