Skip to content

Conversation

@sepideh-abedini
Copy link
Collaborator

Summary

Improve the logging in the project by using loguru and introducing logging annotations.

Clickup Ticket(s): https://app.clickup.com/t/868gqbcen

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • [ x] ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📝 Documentation update
  • 🔧 Refactoring (no functional changes)
  • ⚡ Performance improvement
  • 🧪 Test improvements
  • 🔒 Security fix

Changes Made

  • Removed custom logging configurations and used a unified loguru config
  • Add a separate log sink to save logs in a file
  • Add a separate logger to log interactions with LLM
  • Use @logger.catch decorator to catch exceptions and log errors
  • Added a custom @alog annotation for simple logging of method completion

Testing

  • [ x] Tests pass locally (uv run pytest tests/)
  • [ x] Type checking passes (uv run mypy <src_dir>)
  • [ x] Linting passes (uv run ruff check src_dir/)
  • Manual testing performed (describe below)

Checklist

  • [ x] Code follows the project's style guidelines
  • [ x] Self-review of code completed
  • [ x] Documentation updated (if applicable)
  • [ x] No sensitive information (API keys, credentials) exposed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants