fix: generate litestream v0.5 retention config#871
Conversation
Co-authored-by: WillBooster (Codex CLI) <agent@willbooster.com>
There was a problem hiding this comment.
Code Review
This pull request pins the Litestream version in the installation script and refactors the Litestream configuration in the Prisma command to move snapshot settings to the top level. However, a critical issue was found in the installation script where the default version was set to 0.5.12, which is a LiteFS version rather than a Litestream version (the latest stable being 0.3.13). This mismatch will cause the Docker build to fail with a 404 error.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Co-authored-by: WillBooster (Codex CLI) <agent@willbooster.com>
Co-authored-by: WillBooster (Codex CLI) <agent@willbooster.com>
Customer Summary
LITESTREAM_VERSIONoverride when needed.Technical Summary
snapshot.intervalandsnapshot.retentionkeys.retention-check-intervalgeneration because v0.5 enforces snapshot retention during snapshot compaction.ARCHvalue as the Litestream release asset architecture (x86_64orarm64) and uses it directly in the.debfilename.LITESTREAM_VERSIONandARCHoverrides for callers that need them.Why
sharedstill generated v0.3-style retention keys, so the intended72hretention was not applied./etc/litestream.ymlmatch the Litestream version currently installed by the Docker image.Testing
bash -n packages/wb/docker/bash/install-litestream.shlinux-arm64andlinux-x86_64withcurl -fsSI.snapshot:config is accepted by Litestream0.5.12vialitestream databases -config.yarn verifybunx @willbooster-private/agentic-workflows@latest skills check-pr-ci.