Fix/silent image skip on thumbnail failure

[Sanskriti0805]

Problem

In the image_util_prepare_image_records function, when image_util_generate_thumbnail() returns False (indicating thumbnail generation failure), the code simply skips adding the image to image_records without logging any warning or error message. The user has no visibility into which images were skipped or why.

Impact

• Images missing from database without user notification
• No visibility into which images failed to process
• Difficult to debug image import issues
• Users may not realize images are missing until they search for them

Solution

Added explicit warning logging when thumbnail generation fails, so users are informed about skipped images.

Testing

• Tested with images that fail thumbnail generation (corrupted files, unsupported formats, etc.)
• Verified warning messages appear in logs
• Confirmed that valid images still process correctly
• Ran prettier on frontend files
• Ran pre-commit hooks (ruff, black) on backend files

Summary by CodeRabbit

Release Notes

fixes issue #843

• Bug Fixes
  • Enhanced image processing error reporting. When image thumbnail generation fails, the system now logs explicit warning messages instead of silently skipping the file. This provides administrators with better visibility into processing failures and helps identify problematic images during import operations.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[github-actions]

⚠️ No issue was linked in the PR description.
Please make sure to link an issue (e.g., 'Fixes #issue_number')

[coderabbitai]

📝 Walkthrough

Walkthrough

The image_util_prepare_image_records function now logs a warning message when thumbnail generation fails, explicitly indicating that the image will be skipped and not added to the database. Previously, failed thumbnails were silently skipped without notification.

Changes

Cohort / File(s)	Summary
Thumbnail failure logging backend/app/utils/images.py	Added warning log when thumbnail function returns False, noting that the image will be skipped and excluded from database records

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A rabbit logs each stumble on the path,
When thumbnails fail to render in their wrath,
"Skip this one!" it squeaks with careful care,
No silent errors lurking unaware—
Transparency hops through the code so bright! 🐰📝

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly summarizes the main change: adding logging/visibility (fixing the 'silent' skip) when thumbnail generation fails.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 917daff and 855db66.

📒 Files selected for processing (1)

• backend/app/utils/images.py

🔇 Additional comments (1)

backend/app/utils/images.py (1)

179-184: LGTM! Excellent observability improvement.

The warning message clearly communicates when images are skipped due to thumbnail generation failures, directly addressing the issue described in the PR. The message includes both the affected image path and the consequence (not being added to the database), which helps with debugging and user awareness.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

⚠️ No issue was linked in the PR description.
Please make sure to link an issue (e.g., 'Fixes #issue_number')