[Ellipsis] fix: handle file size limits and encoding in _content_files_list

[ellipsis-dev]

This change addresses review comments left by @ErikBjare on PR #188: fix: limit image size, fixes exception when viewing large images (#185)

⚠️ We couldn't build/test your project to verify our changes. Add a Dockerfile to significantly improve code quality.

Summary:

Enhances _content_files_list in message.py to handle file size limits and improve base64 encoding efficiency for OpenAI and Anthropic providers.

Key points:

• Behavior:
  • Raises ValueError in _content_files_list if neither openai nor anthropic is true.
  • Checks file size in _content_files_list, appending a warning message if file exceeds 5MB.
  • Uses data variable for base64 encoding in _content_files_list to avoid redundant encoding.
• Misc:
  • Minor refactoring in _content_files_list to improve readability and efficiency.

---

You can configure Ellipsis to address comments with a direct commit or a side PR, see docs.

---

Something look wrong? If this Pull Request doesn't address the comments left on the above pull request, create a new PR review with more details. For more information, check the documentation.

Generated with ❤️ by ellipsis.dev