Fixed
- Chunked Storage Error Reporting: Fixed bug where storing content exceeding
max_content_lengthwould returnsuccess: Truewith "Successfully stored 0 memory chunks" when all chunks failed (e.g., due to duplicates)- Now correctly returns
success: Falsewith descriptive error message when all chunks fail - Added
failed_chunksfield to chunked success response for partial failures - Includes failure reasons in error message (e.g., "Duplicate content detected")
- Added regression tests:
test_chunked_storage_all_chunks_failandtest_chunked_storage_partial_success
- Now correctly returns
Changed
- Simplified conditional expression per Gemini Code Assist review feedback
- Removed redundant
if failed_chunks else 0check (len([]) already returns 0)