🔧 update: migrate runtime to node.js with bun tooling - #20
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
* 📦 new (verification): add honeypot enforcement Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * 🔧 update (verification): harden honeypot enforcement Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🟡 Changes recommended
Critical startup and production packaging issues remain unresolved.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
Migrates production execution from Bun to Node.js, replaces the SQLite driver, and updates Docker, CI, documentation, and branding.
Changes:
- Adds Node-based production builds and runtime configuration.
- Replaces
bun:sqlitewithbetter-sqlite3. - Updates deployment setup, CI, website content, and branding.
File summaries
| File | Description |
|---|---|
tsconfig.json |
Updates Node-compatible compiler settings. |
src/db/sqlite.ts |
Replaces Bun SQLite APIs. |
src/db/schema.ts |
Adapts schema operations to better-sqlite3. |
site/zentrynel-icon.svg |
Adds branded icon artwork. |
site/styles.css |
Styles updated website sections. |
site/index.html |
Updates marketing and setup content. |
README.md |
Updates database and invite documentation. |
package.json |
Defines Node production scripts and dependencies. |
Dockerfile |
Adds multi-stage Node runtime image. |
docker-entrypoint.sh |
Prepares persistent database storage. |
bun.lock |
Locks new dependencies. |
.github/workflows/build-flow.yml |
Adds Node version matrix. |
Review details
Suppressed comments (2)
Dockerfile:42
- This healthcheck always exits successfully without inspecting the bot process, so a hung or otherwise unhealthy Node application will still be reported as healthy by the container platform. Use a probe that reflects the bot's actual liveness, or remove this healthcheck instead of publishing a false status.
CMD node -e "process.exit(0)"
site/index.html:335
- The self-hosting section still advertises
Bun >= 1.0as the runtime (site/index.html:365), but production now launches Node (package.json:8and Dockerfile:44). That directs deployers to install the wrong runtime; update the runtime label to Node >=22 and show Bun only for the build/tooling steps.
<li class="setup-step">
- Files reviewed: 8/10 changed files
- Comments generated: 3
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
🔥 Container Build Complete - Patch BuildBuild Status: ✅ Success 📦 Pull ImageDocker Hub: docker pull wgtechlabs/zentrynel:patch-3a4b6dbGHCR: docker pull ghcr.io/wgtechlabs/zentrynel:patch-3a4b6db📋 Build Details
🏷️ Image Tags• 🔍 Testing Your Changes
🚀 Quick Start# Pull and run the container
Docker Hub: docker pull wgtechlabs/zentrynel:patch-3a4b6db
docker run <your-options> <image>🔒 Security Scan Results📋 Pre-Build Security Checks✅ Source Code Scan: 0 vulnerabilities found 🐳 Container Image Vulnerabilities
📊 Detailed Security ReportsView detailed vulnerability reports in the GitHub Security tab. 🤖 Powered by Container Build Flow Action vfb5c0662b33f7702bc1ccf85350689436989f606 |
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Summary
bun:sqlitewithbetter-sqlite3wgtechlabs/devin-discord-botValidation
bun run buildbun test(59 passing)git diff --checkDeployment
Railway production was temporarily rolled back to
wgtechlabs/zentrynel:0.11.0as a hot-patch safety measure. This PR targetsmaindirectly per request.