From b5895c0a05490a8450d4b79f2d37d5726ab8ace4 Mon Sep 17 00:00:00 2001
From: MToan <58007568+taminhtoan2601@users.noreply.github.com>
Date: Tue, 7 Jul 2026 07:53:48 +0200
Subject: [PATCH] fix(docker): add tzdata to runtime image for zoneinfo support
 (#1372)

Python skill scripts using zoneinfo.ZoneInfo (and Go time.LoadLocation with
named zones) fail on the published images because alpine:3.23 ships without
tzdata: ZoneInfoNotFoundError: 'No time zone found with key Europe/Paris'.

Add tzdata to the base runtime apk install so every image variant (base,
latest, full, claude-cli) has the system zoneinfo database.
---
 Dockerfile | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/Dockerfile b/Dockerfile
index a51aebc3..d734b39e 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -80,11 +80,12 @@ ARG ENABLE_CLAUDE_CLI=false
 # requirements-skills.txt: additional deps only for ENABLE_FULL_SKILLS.
 COPY docker/requirements-base.txt docker/requirements-skills.txt /tmp/
 
-# Install ca-certificates + wget (healthcheck) + optional runtimes.
+# Install ca-certificates + wget (healthcheck) + tzdata (zoneinfo for Go
+# time.LoadLocation and Python zoneinfo in skill scripts) + optional runtimes.
 # ENABLE_FULL_SKILLS=true pre-installs all skill deps (larger image, no on-demand install needed).
 # Otherwise, skill packages are installed on-demand via the admin UI.
 RUN set -eux; \
-    apk add --no-cache ca-certificates wget su-exec; \
+    apk add --no-cache ca-certificates wget su-exec tzdata; \
     if [ "$ENABLE_SANDBOX" = "true" ]; then \
         apk add --no-cache docker-cli; \
     fi; \