From fdafb4e842d431f4ad8d5db9751695b1b691176d Mon Sep 17 00:00:00 2001 From: ekko <152005280+EKKOLearnAI@users.noreply.github.com> Date: Tue, 21 Apr 2026 14:44:51 +0800 Subject: [PATCH] fix(docker): use official Node.js binary and remove unnecessary deps (#103) * feat(chat): polish syntax highlighting and tool payload rendering (#94) * [verified] feat(chat): polish syntax highlighting and tool payload rendering * [verified] fix(chat): tighten large tool payload rendering * docs: update data volume path in Docker docs Align documentation with docker-compose.yml change: hermes-web-ui-data -> hermes-web-ui, /app/dist/data -> /root/.hermes-web-ui Co-Authored-By: Claude Opus 4.6 * refactor: bundle server build and restructure service modules - Add build-server.mjs script for standalone server compilation - Add logger service with structured output - Restructure auth, gateway-manager, hermes-cli, hermes services - Update docker-compose volume mount path - Update tsconfig and entry point for bundled server Co-Authored-By: Claude Opus 4.6 * refactor: separate controllers from routes and centralize route registration - Extract business logic from route handlers into controllers/ - Add centralized route registry in routes/index.ts with public/auth/protected layers - Replace global auth whitelist with sequential middleware registration - Extract shared helpers to services/config-helpers.ts - Allow custom provider name to be user-editable in ProviderFormModal - Deduplicate custom providers by poolKey instead of base_url in getAvailable Co-Authored-By: Claude Opus 4.6 * fix: auth bypass via path case, SPA serving, and provider improvements - Fix auth bypass: path case-insensitive check for /api, /v1, /upload - Fix SPA returning 401: skip auth for non-API paths (static files) - Fix profile switch: use local loading state instead of shared store ref - Auto-append /v1 to base_url when fetching models (frontend + backend) - Guard .env writing to built-in providers only - Add builtin field to provider presets, enable base_url input in form - Print auth token to console on startup (pino only writes to file) Co-Authored-By: Claude Opus 4.6 * fix(docker): correct volume mount path and update Node.js to 23 - Fix webui volume mount from /root/.hermes-web-ui to /home/agent/.hermes-web-ui (container runs as agent user, homedir() returns /home/agent) - Update Node.js from 22 to 23 in Dockerfile Co-Authored-By: Claude Opus 4.6 * ci: add ARM64 support for Docker image build Add QEMU and multi-platform build (linux/amd64,linux/arm64) so the image works on Apple Silicon and ARM-based NAS devices. Co-Authored-By: Claude Opus 4.6 * fix(docker): use official Node.js binary and remove unnecessary deps - Replace NodeSource with official Node.js binary (fixes ARM64 GPG error) - Remove python3/python3-yaml (not needed, base image already has Python) - Keep make/g++ for node-pty native compilation Co-Authored-By: Claude Opus 4.6 --------- Co-authored-by: Zhicheng Han <43314240+hanzckernel@users.noreply.github.com> Co-authored-by: Claude Opus 4.6 --- Dockerfile | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/Dockerfile b/Dockerfile index e831bc9..991c195 100644 --- a/Dockerfile +++ b/Dockerfile @@ -6,13 +6,11 @@ USER root RUN apt-get update && apt-get install -y --no-install-recommends \ ca-certificates \ curl \ - gnupg \ - python3 \ - python3-yaml \ make \ g++ \ - && curl -fsSL https://deb.nodesource.com/setup_23.x | bash - \ - && apt-get install -y --no-install-recommends nodejs \ + && ARCH=$(dpkg --print-architecture) \ + && curl -fsSL "https://nodejs.org/dist/v23.11.0/node-v23.11.0-linux-${ARCH}.tar.xz" \ + | tar -xJ -C /usr/local --strip-components=1 \ && rm -rf /var/lib/apt/lists/* WORKDIR /app