[PB-5721] fix(thumbnail): use file bucket instead of request bucket for storage

[douglas-xt]

What does this PR do?

Fixes thumbnail storage bucket inconsistency by ensuring thumbnails always use the file's bucket instead of the bucket from the request DTO.

Why are we doing this?

Thumbnails were being created using the bucketId from the request DTO, which could differ from the actual file's bucket. This caused issues in workspace environments where:

• Files exist in workspace buckets
• The request might contain a different bucket ID
• Thumbnails need to be in the same bucket as their parent files

How are we doing this?

1. Changed createThumbnail to use file.bucket instead of createThumbnailDto.bucketId
2. Fixed deletion of existing thumbnails to use existingThumbnail.bucketId (where it actually exists) instead of the new bucket for backward compatibility
3. Updated unit tests to verify:
   • Thumbnails are created/updated with file.bucket
   • Old thumbnails are deleted from their original bucket
   • Migration scenarios where DTO bucket differs from file bucket

Any background context you want to provide?

This fix ensures proper bucket isolation between personal and workspace files, and handles migration of thumbnails created before this fix was implemented.

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[jzunigax2]

fix small lint error here, missing coma