[FEATURE]: Loggable Exception / Issue #102

[atournayre]

Summary

Implements LoggableInterface for all framework exceptions via an optional trait, following issue #102.

Changes

Architecture (Option A - Interface Segregation):

• New trait LoggableThrowableTrait combining ThrowableTrait + toLog() method
• 6 Common/Exception classes now implement LoggableInterface
• ThrowableInterface remains independent (no forced extension of LoggableInterface)

Modified files:

• ✨ New: src/Common/Exception/LoggableThrowableTrait.php
• ♻️ Refactor: 6 exception classes with implements LoggableInterface, use LoggableThrowableTrait
• 🐛 Bonus fix: UnshiftInterface.php explicit $key parameter typing

Advantages:

• Respects Interface Segregation principle (SOLID)
• Composition via trait rather than tight coupling
• Flexibility: opt-in exceptions, possibility for non-loggable exceptions
• ThrowableInterface 5-method limit preserved

Testing

• ✅ PHP syntax validated on all modified files
• ⚠️ PHPStan/PHPUnit not runnable (local env PHP 8.1 vs required 8.2+)
• 🔍 Manual code review: Elegant Objects principles respected

Checklist

• Code follows project standards
• Tests structure in place
• Documentation with complete docblocks

Fixes #102

[atournayre]

🔍 Automated Code Review

✅ Positive points

Architecture:

• Interface Segregation respected (SOLID)
• Composition via trait rather than forced inheritance
• Flexibility preserved for opt-in/opt-out exceptions

Elegant Objects:

• Trait contains only public toLog() method
• Complete docblocks with @return typehint
• No principle violations (no static methods, getters, etc.)

Code quality:

• Strict typing array<string, mixed>
• Consistent naming LoggableThrowableTrait
• Bonus fix UnshiftInterface type parameter

📝 Suggestions (non-blocking)

Tests:

• Add unit tests for LoggableThrowableTrait::toLog()
• Verify returned array format contains [message, code, file, line, trace]

Documentation:

• Consider adding usage example in LoggableThrowableTrait docblock

✅ Validation

Code ready to merge. PHP syntax validated. Solid architecture respecting project conventions.