4 Commits

Include renames

Author	SHA1	Message	Date
mathiasandClaude Opus 4.8	e4c701c6f1	feat(discovery): trigger a discovery pass on YouTube connect ... A newly connected account showed no videos until the next 2h scheduled pass — the gap that made onboarding look broken (a second user connected, saw nothing, read as failure). The connect callback now fires an out-of-band discovery pass for the connecting user, so videos appear promptly. Concurrency: scheduled and connect-triggered passes share one lock (serialize), preserving the single-fetcher invariant (ADR-018). A trigger interleaves between the scheduler's per-user passes rather than fetching concurrently or waiting for a whole pass. The trigger runs on the server ctx (survives the redirect) and is non-blocking for the request goroutine. Scope: connect-trigger only. The optional login-refresh / "Discover now" button from #6 are intentionally not built — an unconditional login hook risks 429 storms (per the ticket's own recommendation); defer until wanted. TDD: TestCallbackTriggersDiscovery, TestSerializeRunsOneAtATime, TestDiscoveryTriggerEnqueueRunsUser; new BDD scenario mapped. Refs #6 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-06-09 21:15:46 +02:00
mathiasandClaude Opus 4.8	a884e7e9c5	test(bdd): add scenario name-coverage gate (no godog) ... Close the gap where docs/use-cases/*.feature claimed to be the behavior spec but nothing executed them — so scenarios drifted (the stale "auto summarizes every new video" and "manual is the default" were proof). Decision (per issue #5 BDD-runner fork): no godog — keep .feature as design records, add a cheap name-coverage gate instead. TestScenarioCoverage parses every scenario and asserts each non-@pending one maps to an existing Go test in the scenarioCoverage manifest; it flags unmapped scenarios, missing/renamed tests, and stale entries. It checks the link, not that the test exercises the scenario (the deliberate trade for skipping godog). Also: - Fix the stale ADR-018 drift: "Manual is the default" -> auto is the default for new users; added an explicit default scenario + a plain manual scenario. - Tag 4 documented-but-unbuilt/untested scenarios @pending with reasons (Vimeo connect, BYO config flow, logout->welcome, re-register-after-delete) so they are tracked without a false coverage claim. - CLAUDE.md BDD section now describes the real setup (design records + the gate + @pending convention) instead of claiming an executable spec. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-06-08 22:59:25 +02:00
mathiasandClaude Opus 4.8	6421a1334a	feat(usecase): add subscription-driven watch loop with per-video dedup ... ProcessNewVideos walks a user's subscriptions and processes each newly seen video. Two remaining .feature scenarios are now covered: a channel the user is not subscribed to is never surfaced (so never processed), and a video already processed in this engine's lifetime is not summarized twice. Dedup is in-memory and per-user; durable cross-restart dedup stays the store's concern (no new port), per docs/data-model.md. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-06-02 16:25:52 +02:00
mathias	0064e34841	test: add acceptance tests for summarize-new-video (RED) ... Executable translation of docs/use-cases/summarize_new_video.feature: captioned video is summarized and delivered to the store sink; no-transcript video is skipped with reason "no transcript" and no delivery. Drives the engine through fake adapters (no live YouTube/brain). Intentionally RED until the engine is implemented — this is the swarm's target.	2026-06-02 11:05:32 +00:00