3 Commits

Author	SHA1	Message	Date
Daniel Bedeleanu	01321bf607	fix(phase1): restore API contract while preserving N+1 optimization ... The N+1 optimization in save_image() pre-lowercases the existing_files list before passing to get_unique_filename(). However, this broke the API contract: the function should handle any-case input to remain robust. Changed: get_unique_filename() now defensively lowercases the input list, ensuring collision detection works regardless of input case. Benefits: - Fixes implicit API contract change (function expected any-case input) - Maintains N+1 optimization (pre-lowercasing still works) - Supports both optimization and edge cases (direct function calls) - All 22 tests pass	2026-04-20 22:09:58 +03:00
Daniel Bedeleanu	2951ed81eb	fix(phase1): add logging, remove unused import, add error handling	2026-04-20 22:06:18 +03:00
Daniel Bedeleanu	ea49cd6e4a	feat(phase1): add image storage utilities ... - Create backend/services/image_storage.py with 4 core functions: - sanitize_filename(): remove unsafe chars, limit to 255 chars, convert to lowercase - get_unique_filename(): handle collisions with UUID suffix (format: {name}_{uuid8}_{variant}.jpg) - ensure_image_directories(): create /images/ root and category subdirs on startup - save_image(): save bytes to /images/{category}/{filename}, returns relative path - Create comprehensive test suite (22 tests) covering all functionality - Integrate ensure_image_directories() into FastAPI startup event - Directory structure: /images/{category}/{filename} - Collision handling: auto-suffix with UUID if filename exists - All tests passing, pathlib.Path for safe operations	2026-04-20 21:57:26 +03:00