Skip to content

feat: cache URL + yt-dlp fetches via kit storage#9

Merged
jadb merged 2 commits into
mainfrom
feat-scrape-httpcache
Jun 19, 2026
Merged

feat: cache URL + yt-dlp fetches via kit storage#9
jadb merged 2 commits into
mainfrom
feat-scrape-httpcache

Conversation

@jadb

@jadb jadb commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

No description provided.

FOO_SCRAPE_CACHE=<path> routes fetches through kit/storage/httpcache
backed by a sqlite kv store; FOO_SCRAPE_CACHE_TTL tunes freshness
(default 24h). unset or open-failure falls back to default client —
caching is best-effort, never a hard dep of a scrape. mirrors the
FOO_SCRAPE_BUS_PEERS opt-in idiom.

bumps kit to v0.5.0-alpha.1 (adds storage/httpcache).
Copilot AI review requested due to automatic review settings June 19, 2026 10:06

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

route all yt-dlp invocations through runYTDLP, caching raw output in a
sqlite kv store keyed by argv. on by default (XDG cache dir, or
FOO_YOUTUBE_CACHE path; FOO_YOUTUBE_CACHE_TTL tunes freshness, default
24h); --no-cache bypasses. best-effort: store-open/read/write failure
degrades to a live exec.

extract an injectable ytRunner seam for the subprocess. tests replay
real-shape yt-dlp output via an xrr exec cassette (no live yt-dlp), plus
cache hit/miss + argv-key coverage. xrr pinned dev-only (test imports).
@jadb jadb changed the title feat(scrape): optional URL fetch caching via kit httpcache feat: cache URL + yt-dlp fetches via kit storage Jun 19, 2026
@jadb jadb merged commit 306e68c into main Jun 19, 2026
3 checks passed
@jadb jadb deleted the feat-scrape-httpcache branch June 19, 2026 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants