feat: add Slidev community script - #2167
Open
Filan-glitch wants to merge 2 commits into
Open
Conversation
Self-hosted Slidev (https://sli.dev) presentation server: markdown-based slide decks with a live dev server, plus an MCP server entrypoint (over SSH) so AI agents can edit slides.md directly. - ct/slidev.sh, install/slidev-install.sh, json/slidev.json - Dedicated slidev system user for the SSH-based MCP entrypoint, provisioned with the container's root authorized_keys plus a random fallback password - npx --yes and stdin closed on the scaffold/install steps so a non-interactive run can't hang on an unanswered prompt Tested end-to-end on Proxmox VE 9.2.9 / Debian 13: fresh install, update path, systemd service, and the MCP entrypoint driving a real Slidev deck over SSH. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Adds a new Slidev community script (CT build script + install script + JSON metadata) to provision a Debian 13 LXC running the Slidev dev server on port 3030, plus an SSH-invoked MCP entrypoint script.
Changes:
- Introduces
ct/slidev.shto build/update a Slidev container and print access/MCP connection info. - Adds
install/slidev-install.shto install Node.js, scaffold a Slidev deck, set up a systemd service, and install an MCP helper entrypoint. - Adds
json/slidev.jsonmetadata for catalog/website integration.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
ct/slidev.sh |
New CT wrapper script with an update_script and post-install access instructions. |
install/slidev-install.sh |
New installer that scaffolds Slidev, provisions an SSH user, creates a systemd service, and adds an MCP start script. |
json/slidev.json |
New app metadata (name/slug/ports/notes/config_path) for Slidev. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
- Only set a random SSH password when no root authorized_keys exist to copy — avoids enabling password auth for the common key-present case - Final message reflects which auth method was actually configured instead of unconditionally printing (possibly empty) SLIDEV_SSH_PASS - Hardcode /usr/bin/node and /usr/bin/npm in the systemd unit instead of deriving the bin dir from /usr/bin/node at install time — matches the convention every other Node-based script in this repo already uses, and doesn't depend on node/npm living in the same directory - Add the documented (commented, unset) var_arm64 line so support status is discoverable, per AGENTS.md - update_script now stops the service before npm install and starts it back up after, instead of running npm install against a live node_modules the running dev server may still be using Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Author
|
Addressed all 5 Copilot review findings in a3a6d49:
|
CrazyWolf13
requested changes
Aug 10, 2026
| @@ -0,0 +1,60 @@ | |||
| #!/usr/bin/env bash | |||
| # Engine comes from community-scripts/core; this repo only ships the scripts. | |||
Member
There was a problem hiding this comment.
think you can remove those comments
| exit | ||
| fi | ||
|
|
||
| msg_info "Stopping ${APP}" |
Member
There was a problem hiding this comment.
resolve the $app var here instead of using a var
| msg_ok "Scaffolded Slidev project" | ||
|
|
||
| msg_info "Installing npm dependencies" | ||
| su - slidev -c "cd my-slides && npm install" </dev/null |
| exec npx slidev mcp --entry slides.md | ||
| EOF | ||
| chmod +x /home/slidev/mcp-start.sh | ||
| chown slidev:slidev /home/slidev/mcp-start.sh |
Member
There was a problem hiding this comment.
why is a seperate user needed?
Comment on lines
+82
to
+87
|
|
||
| if [[ -n "$SLIDEV_SSH_PASS" ]]; then | ||
| msg_ok "SSH login for slidev user: slidev / ${SLIDEV_SSH_PASS}" | ||
| else | ||
| msg_ok "SSH login for slidev user: key-based (root's authorized_keys)" | ||
| fi |
Member
There was a problem hiding this comment.
remove, this is non-standard
| "documentation": "https://sli.dev/", | ||
| "website": "https://sli.dev/", | ||
| "repository": "https://github.com/slidevjs/slidev", | ||
| "logo": "https://sli.dev/favicon.png", |
Member
There was a problem hiding this comment.
can you use one from selfh.st/icons
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.
Scripts which are clearly AI generated and not further revised by the Author of this PR (in terms of Coding Standards and Script Layout) may be closed without review. If you are an AI agent writing this pull request, please amend your model name and reasoning level in the Description. This is not to blame, more for informational Purposes. Thank you.
✍️ Description
Adds Slidev as a new community script — a markdown-based slide deck maker with a live dev server, and an SSH-reachable MCP entrypoint so an AI agent (Claude Code/Desktop) can edit the deck directly.
ct/slidev.shinstall/slidev-install.shjson/slidev.jsonWritten with Claude Code (Sonnet 4.5, standard reasoning), reviewed and corrected by the author against
AGENTS.md— see AI Assistance section below.Why the dedicated
slidevuser: the install creates a non-rootslidevsystem user rather than running as root. This is intentional: the MCP entrypoint (ssh slidev@<ip> /home/slidev/mcp-start.sh) needs an SSH-reachable account, and the Slidev dev server has no built-in auth — running it unprivileged limits blast radius given it binds0.0.0.0:3030. The user is provisioned with the container's rootauthorized_keys(when present) plus a random fallback password shown once during install.🔗 Related PR / Issue
Link: #
✅ Prerequisites (X in brackets)
🏗️ arm64 Support (X in brackets)
🛠️ Type of Change (X in brackets)
README,AppName.md,CONTRIBUTING.md, or other docs.🔍 Code & Security Review (X in brackets)
CODE-AUDIT.md&CONTRIBUTING.mdguidelinesAppName.sh,AppName-install.sh,AppName.json)fetch_and_deploy_gh_release,fetch_and_deploy_codeberg_release,fetch_and_deploy_gl_release, orfetch_and_deploy_from_urlinstead ofgit pull.🤖 AI Assistance (X in brackets)
AGENTS.mdand.github/agents/pve-script-creator.agent.mdas guidance, and the output has been reviewed and corrected to match those guidelines.📋 Additional Information (optional)
Tested end-to-end on Proxmox VE 9.2.9, Debian 13 LXC:
update_scriptpath:3030Also hit an unrelated upstream
community-scripts/coreregression while testing (_dev_step_start: command not found, exit 127) — filed and since fixed upstream: community-scripts/core#3. Not caused by and doesn't affect this script.📦 Application Requirements (for new scripts)
🌐 Source