From ab53d9639ffc3c857d215af0777b5cb0b6025215 Mon Sep 17 00:00:00 2001 From: tiennm99 Date: Fri, 22 May 2026 17:27:26 +0700 Subject: [PATCH] fix(deploy): include stats in CI ModulesCSV override MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit SAM CLI's --parameter-overrides replaces samconfig.toml's overrides rather than merging them, so every CI deploy reset the stack's ModulesCSV to the prior value (which predated the stats module). The live Lambda's MODULES env was missing stats, so /stats was dispatched but never handled — the module factory was never called. --- .github/workflows/deploy.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/deploy.yml b/.github/workflows/deploy.yml index 3943636..8904389 100644 --- a/.github/workflows/deploy.yml +++ b/.github/workflows/deploy.yml @@ -59,7 +59,7 @@ jobs: # CI needs in the deployed stack must be listed here explicitly. # Telegram user IDs are public (visible to anyone the bot DMs), so # they live in this committed workflow rather than a secret. - OVERRIDES="CronSharedSecret=$CRON_SECRET BotOwnerID=1064111334 AdminUserIDs=1064111334" + OVERRIDES="CronSharedSecret=$CRON_SECRET BotOwnerID=1064111334 AdminUserIDs=1064111334 ModulesCSV=util,misc,wordle,loldle,lolschedule,twentyq,trading,stats" if [ -n "$ALERT_EMAIL" ]; then OVERRIDES="$OVERRIDES AlertEmail=$ALERT_EMAIL" fi