ci(share): prunes stale share images on a schedule - #3149
Draft
seferturan wants to merge 2 commits into
Draft
Conversation
|
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| JavaScript | Aug 18, 2026 11:10a.m. | Review ↗ | |
| Code coverage | Aug 18, 2026 11:40a.m. | Review ↗ |
Important
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🚧 Drafted 🚧
effectiveReleaseDatefor shows isfirst_aired, so a long running show that premiered years ago falls into the slowest tier even while it is actively airing and its ratings churn weekly. trakt-workers usesshows.lastAiredfor exactly this reason, andlast_airedis not on the show summary response, so this needs its own decision🎶 Notes 🎶
AGGRESSIVE_RECHECK_SCHEDULEin trakt-workers: 12h under 14 days, 3d to 40 days, 7d to 120 days, 30d after thatimages/share/{og,feed,story}/{movie,show}/, derived from the same helper the endpoint builds paths with, so the prefixes cannot drift from where images actually landcachedAtandreleasedAtfrom object metadatacachedAtare skipped rather than deletedreleasedAtare pruned, so they come back with the full metadata set🛠 Fixed along the way 🛠
api.cloudflare.com, but the script talks to<account>.r2.cloudflarestorage.comandsetup-denoneeds jsr and npm. Copied the proven set fromsupply_chain.ymland added the R2 hostsetup-denoalready does onebuildTargetPrefixesandshareImagePrefixout ofbuildImagePathso each file keeps one exportimages/og/...prefixes🤔 Worth knowing 🤔
cachedAt, so they are skipped even if a prefix were ever wrong. That is the safety net, not the plan