Skip to content

Better exception logging#91

Merged
lotif merged 4 commits intomainfrom
marcelo/better-exceptions
Mar 26, 2026
Merged

Better exception logging#91
lotif merged 4 commits intomainfrom
marcelo/better-exceptions

Conversation

@lotif
Copy link
Collaborator

@lotif lotif commented Mar 26, 2026

Summary

Some teams are having errors in the LLM as a judge part and it's hard to debug without an exception trace. Adding some traces here to the logs to improve debugging.

Clickup Ticket(s): NA

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ 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

Adding a few logging statements on exceptions.

Testing

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

Manual testing details:

Screenshots/Recordings

Related Issues

Deployment Notes

Checklist

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

@lotif lotif merged commit 4f8dc47 into main Mar 26, 2026
3 checks passed
@lotif lotif deleted the marcelo/better-exceptions branch March 26, 2026 17:27
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