Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Move all user prompts to start of process#22

Draft
donnchawp wants to merge 1 commit intoiamsanmith:mainfrom
donnchawp:move-prompts-upfront
Draft

Move all user prompts to start of process#22
donnchawp wants to merge 1 commit intoiamsanmith:mainfrom
donnchawp:move-prompts-upfront

Conversation

@donnchawp
Copy link

@donnchawp donnchawp commented Feb 26, 2026

Summary

  • Move all interactive prompts to the beginning of main(), before any file copying or processing starts, so MetaSort can run unattended after initial setup
  • Prompts moved:
    • WhatsApp/Screenshot separation (from media_cleaning.rs)
    • Metadata embedding source selection (from metadata_embed.rs)
    • Unpaired media handling — skip vs guess from filename (from metadata_extraction.rs)
  • Rename ask_and_separate_whatsapp_screenshots to separate_whatsapp_screenshots
  • Remove unused io imports from metadata_embed.rs and metadata_extraction.rs

Test plan

  • Run MetaSort and verify all prompts appear before file copying begins
  • Verify no further prompts interrupt processing after initial setup
  • Verify WhatsApp/Screenshot separation still works correctly
  • Verify metadata embedding uses filename-based dates when option 2 is selected
  • Verify unpaired media files are guessed or skipped based on upfront choice

Collect all interactive input (WhatsApp/Screenshot separation and
metadata embedding source) upfront before file copying and processing
begins, so the tool can run unattended after initial setup.

Also rename ask_and_separate_whatsapp_screenshots to
separate_whatsapp_screenshots since it no longer prompts the user.
@donnchawp donnchawp force-pushed the move-prompts-upfront branch from 9cd9ca0 to 89e9ef7 Compare February 26, 2026 10:45
@marcodeppe
Copy link

LOL I was about to file an issue for this exact functionality! 🙇‍♂️
As I explained in #24 I had to use a slow ext USB HDD due to space issues and let everything run over night. If I had been able to give directions in the beginning it might be done by now :)

@donnchawp
Copy link
Author

Yeah, I had 700GB of files and I got up one morning to see it prompting about WhatsApp and screenshot images. I just need to test it properly on a smaller archive and then I'll make it ready for review.

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