refactor: simplify context processors

Author: roxblnfk

src/Internal/ContextProcessor/BuiltInTypeProcessor.php

@@ -4,29 +4,26 @@
 
 namespace RoadRunner\PsrLogger\Internal\ContextProcessor;
 
+use RoadRunner\PsrLogger\Internal\ObjectProcessor;
+
 /**
  * Processor for DateTime objects.
  *
  * Converts DateTime and DateTimeImmutable objects to ISO 8601 format
  * for consistent structured logging.
  *
- * @internal This class is internal to the PSR Logger implementation and should not be used directly.
+ * @implements ObjectProcessor<\DateTimeInterface>
  *
- * @implements ContextProcessorInterface<\DateTimeInterface, string>
+ * @internal
  */
-class DateTimeProcessor implements ContextProcessorInterface
+final class DateTimeProcessor implements ObjectProcessor
 {
-    public function canProcess(mixed $value): bool
+    public function canProcess(object $value): bool
     {
         return $value instanceof \DateTimeInterface;
     }
 
-    /**
-     * @param \DateTimeInterface $value
-     * @param callable(mixed): mixed $recursiveProcessor
-     * @return string
-     */
-    public function process(mixed $value, callable $recursiveProcessor): mixed
+    public function process(object $value, callable $processor): mixed
     {
         return $value->format(\DateTimeInterface::ATOM);
     }

src/Internal/ContextProcessor/ContextProcessorInterface.php

@@ -4,30 +4,33 @@
 
 namespace RoadRunner\PsrLogger\Internal\ContextProcessor;
 
+use RoadRunner\PsrLogger\Internal\ObjectProcessor;
+
 /**
- * Fallback processor for unknown types.
- *
- * Returns the type name for any value that couldn't be processed
- * by more specific processors.
+ * Fallback processor for unknown objects.
  *
- * @internal This class is internal to the PSR Logger implementation and should not be used directly.
+ * @internal
  *
- * @implements ContextProcessorInterface<mixed, string>
+ * @implements ObjectProcessor<object>
  */
-class FallbackProcessor implements ContextProcessorInterface
+final class FallbackProcessor implements ObjectProcessor
 {
-    public function canProcess(mixed $value): bool
+    public function canProcess(object $value): bool
     {
-        // This processor can handle anything as a last resort
         return true;
     }
 
-    /**
-     * @param callable(mixed): mixed $recursiveProcessor
-     * @return string
-     */
-    public function process(mixed $value, callable $recursiveProcessor): mixed
+    public function process(object $value, callable $processor): string
     {
-        return \gettype($value);
+        $result = ['@class' => $value::class] + \get_object_vars($value);
+        foreach ($result as $k => &$v) {
+            if ($v === $value) {
+                unset($result[$k]);
+            }
+
+            $v = $processor($v);
+        }
+
+        return $result;
     }
 }

src/Internal/ContextProcessor/ContextProcessorManager.php

@@ -4,28 +4,23 @@
 
 namespace RoadRunner\PsrLogger\Internal\ContextProcessor;
 
+use RoadRunner\PsrLogger\Internal\ObjectProcessor;
+
 /**
- * Processor for objects implementing the Stringable interface.
- *
  * Converts Stringable objects to their string representation.
  *
- * @internal This class is internal to the PSR Logger implementation and should not be used directly.
+ * @internal
  *
- * @implements ContextProcessorInterface<\Stringable, string>
+ * @implements ObjectProcessor<\Stringable>
  */
-class StringableProcessor implements ContextProcessorInterface
+final class StringableProcessor implements ObjectProcessor
 {
-    public function canProcess(mixed $value): bool
+    public function canProcess(object $value): bool
     {
         return $value instanceof \Stringable;
     }
 
-    /**
-     * @param \Stringable $value
-     * @param callable(mixed): mixed $recursiveProcessor
-     * @return string
-     */
-    public function process(mixed $value, callable $recursiveProcessor): mixed
+    public function process(object $value, callable $processor): mixed
     {
         return (string) $value;
     }