Skip to content

feat(api): add S3-backed screenshot storage#147

Open
vvatelot wants to merge 4 commits into
mainfrom
feat/api-s3-screenshots
Open

feat(api): add S3-backed screenshot storage#147
vvatelot wants to merge 4 commits into
mainfrom
feat/api-s3-screenshots

Conversation

@vvatelot

@vvatelot vvatelot commented Jul 9, 2026

Copy link
Copy Markdown
Member

Support both filesystem and S3-compatible storage for screenshots, and default the Docker API stack to Garage-backed object storage so screenshot persistence is production-like by default.

Support both filesystem and S3-compatible storage for screenshots, and default the Docker API stack to Garage-backed object storage so screenshot persistence is production-like by default.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vvatelot vvatelot self-assigned this Jul 9, 2026
@github-actions github-actions Bot added documentation Improvements or additions to documentation API tests components labels Jul 9, 2026
vvatelot and others added 3 commits July 9, 2026 13:45
Add the missing Version import so ruff and mypy pass after the S3 screenshot storage changes, and refresh uv.lock for boto3.

Co-authored-by: Cursor <cursoragent@cursor.com>
Move S3 screenshot storage into an API-only polylith brick so ecoindex_cli and ecoindex_scraper no longer inherit boto3 and config dependencies from the shared utils package.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

Coverage PR 147

Branch coverage •
FileStmtsMissCoverMissing
components/ecoindex/config
   settings.py370100% 
components/ecoindex/models
   response_examples.py60100% 
components/ecoindex/screenshot_storage
   __init__.py20100% 
   storage.py652365%29–31, 33–34, 38–39, 43–44, 48, 85, 102–104, 108–113, 120–121, 125
TOTAL97429670% 

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API components documentation Improvements or additions to documentation tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant