Skip to content

Conversation

@mm503
Copy link
Contributor

@mm503 mm503 commented Aug 20, 2025

Hi @mxpv ,

I did some manual cleanup of downloaded episodes and that broke the consistency between the DB and reality.

So I updated the cleanup logic to be more tolerant to these situations - and remove file references from the DB if the file is no longer present.

Example of how it worked in my setup:

time="2025-08-20T15:44:44Z" level=info msg="running cleaner" count=20 feed_id=world_sci_festival
time="2025-08-20T15:44:44Z" level=info msg="deleting \"Infinite Beginnings? Time in Cutting Edge Cosmology\"" episode_id=_DMbOtpRSk0 feed_id=world_sci_festival
time="2025-08-20T15:44:44Z" level=info msg="episode was not found - file does not exist" episode_id=_DMbOtpRSk0 feed_id=world_sci_festival
time="2025-08-20T15:44:44Z" level=info msg="deleting \"Consciousness, Free Will, and Psychedelics: Exploring Mysteries of the Mind\"" episode_id=C9bqa8hRonE feed_id=world_sci_festival
time="2025-08-20T15:44:44Z" level=info msg="episode was not found - file does not exist" episode_id=C9bqa8hRonE feed_id=world_sci_festival
time="2025-08-20T15:44:44Z" level=info msg="deleting \"Straight Talk on Quantum Computing\"" episode_id=AVYRW9Qdp7Q feed_id=world_sci_festival
time="2025-08-20T15:44:44Z" level=info msg="episode was not found - file does not exist" episode_id=AVYRW9Qdp7Q feed_id=world_sci_festival
...

(Consequent runs were clean)

@mm503 mm503 force-pushed the feat/improve_cleanup_handling branch from 62cc782 to 48a2562 Compare August 21, 2025 00:05
I did some manual cleanup and that broke consistency with what's in the DB.
The improved logic updates the DB based on current state.
@mm503 mm503 force-pushed the feat/improve_cleanup_handling branch from 48a2562 to df2e7fa Compare August 21, 2025 00:07
Repository owner deleted a comment from claude bot Aug 21, 2025
@mxpv
Copy link
Owner

mxpv commented Aug 21, 2025

Thanks!

@mxpv mxpv merged commit dd2b7bd into mxpv:main Aug 21, 2025
5 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.

2 participants