Skip to content

chore: delete podcast feature - #692

Merged
mrviduus merged 1 commit into
mainfrom
chore/delete-podcast
Oct 4, 2026
Merged

mrviduus merged 1 commit into
mainfrom
chore/delete-podcast

Conversation

@mrviduus

@mrviduus mrviduus commented Oct 4, 2026

Copy link
Copy Markdown
Owner

Why

Architecture review 2026-10: podcast generation was live but undocumented and unused. Owner decided to delete it.

What

  • Deleted: PodcastWorker, PodcastScriptBuilder, AudioAssembler, PodcastEndpoints, PodcastPrompt, PodcastGenerationJob + PodcastJobStatus, DbSet/config, podcast eval golden + dataset, web BookPodcastPlayer + CSS, admin Podcast section, related tests.
  • Migration DropPodcast — drops podcast_generation_jobs (Down recreates it).
  • Worker no longer references TextStack.Tts; ffmpeg removed from Worker.Dockerfile (only AudioAssembler used it).
  • podcast.script LLM route, model-registry seed and route-alarm default removed.
  • Docs describing the current state updated.

Verified

  • dotnet build 0 errors; UnitTests 1436/1436; AiEvals 53 passed / 4 skipped; dotnet format clean.
  • admin build ok; web tsc --noEmit ok. Mobile had nothing to remove.
  • Migration not applied locally.

Leftovers (not in this PR)

  • Old podcast mp3s may remain under prod storage/books/{editionId}/ — not deleted.
  • models rows for podcast.script stay (seeder runs only on an empty table).
  • SentryRouteAlarm now watches no task by default.

Rollback

Revert + MIGRATE_TARGET to the previous migration (table recreated empty; old job rows are gone).

🤖 Generated with Claude Code

@mrviduus
mrviduus enabled auto-merge (squash) October 4, 2026 19:30
@mrviduus
mrviduus force-pushed the chore/delete-podcast branch from 68fd2e3 to 4c3c272 Compare October 4, 2026 20:01
@mrviduus
mrviduus disabled auto-merge October 4, 2026 20:07
Worker + script builder + ffmpeg audio assembly, API/admin endpoints,
podcast.script route/eval/seed, web player, admin Generate button.
DropPodcast migration drops podcast_generation_jobs. ffmpeg and TTS
registration removed from Worker. Stored mp3s untouched.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@mrviduus
mrviduus force-pushed the chore/delete-podcast branch from 4c3c272 to 89e61a5 Compare October 4, 2026 21:17
@mrviduus
mrviduus merged commit b2fbcae into main Oct 4, 2026
10 checks passed
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.

1 participant