From 1661b355f362ea92c1bc7220964067a83304eea0 Mon Sep 17 00:00:00 2001 From: Matt Johnson Date: Thu, 16 Jul 2026 21:16:53 +0000 Subject: [PATCH] fix(entrypoint): drop three phantom history keys from the seeded default The config written on first boot -- what EVERY fresh Docker install starts from -- seeded three keys that do not exist on HistoryConfig and are silently discarded on load: auto_cleanup: true cleanup_interval_hours: 24 max_age_days: 30 HistoryConfig has only `database`, `max_messages_per_user`, and `conversation_timeout`. To be precise about the impact: history cleanup DOES work -- cleanup_expired() is wired at main.py:237 and _prune_history() honours max_messages_per_user. What never existed is the time-based retention model these keys describe (a 30-day age cutoff on a 24h interval). An operator setting max_age_days: 90 expecting 90-day retention was silently ignored. Not implemented here -- whether time-based retention should exist is a product decision, not a cleanup. This only stops the default config promising it. Co-Authored-By: Claude Opus 4.8 (1M context) --- work/docker-entrypoint.sh | 3 --- 1 file changed, 3 deletions(-) diff --git a/work/docker-entrypoint.sh b/work/docker-entrypoint.sh index 979db11..74f311b 100755 --- a/work/docker-entrypoint.sh +++ b/work/docker-entrypoint.sh @@ -34,9 +34,6 @@ history: database: /data/conversations.db max_messages_per_user: 50 conversation_timeout: 86400 - auto_cleanup: true - cleanup_interval_hours: 24 - max_age_days: 30 memory: enabled: true