feat(sentry): upgrade to v4.18.0 and refactor LogsHandler (#988)

- Upgrade sentry/sentry from ^4.16.0 to ^4.18.0 in root and sentry package
- Refactor LogsHandler to use parent class's handle() method by overriding compileAttributes()
- Add #[Override] attribute for better type safety
- Simplify code by removing duplicate logic and leveraging parent implementation
- Improve code maintainability by following DRY principles

Co-authored-by: Deeka Wong <[email protected]>

Author: huangdijia

composer.json

@@ -33,7 +33,7 @@
         "hyperf/http-server": "~3.1.0",
         "hyperf/support": "~3.1.0",
         "hyperf/tappable": "~3.1.0",
-        "sentry/sentry": "^4.16.0",
+        "sentry/sentry": "^4.18.0",
         "symfony/polyfill-php85": "^1.33"
     },
     "suggest": {

src/Monolog/LogsHandler.php

@@ -13,10 +13,9 @@
 
 use Hyperf\Collection\Arr;
 use Monolog\LogRecord;
+use Override;
 use Sentry\Logs\LogLevel;
-use Sentry\Logs\Logs;
 use Sentry\Monolog\CompatibilityLogLevelTrait;
-use Throwable;
 
 class LogsHandler extends \Sentry\Monolog\LogsHandler
 {
@@ -35,35 +34,19 @@ public function __construct(
     }
 
     /**
-     * @param array<string, mixed>|LogRecord $record
+     * @param array<string,mixed>|LogRecord $record
+     *
+     * @return array<string,mixed>
      */
-    public function handle($record): bool
+    #[Override]
+    protected function compileAttributes($record): array
     {
-        if (! $this->isHandling($record)) {
-            return false;
-        }
-
-        // Do not collect logs for exceptions, they should be handled separately by the `Handler` or `captureException`
-        if (
-            isset($record['context']['exception'])
-            && $record['context']['exception'] instanceof Throwable
-        ) {
-            return false;
-        }
-
-        Logs::getInstance()->aggregator()->add(
-            self::getSentryLogLevelFromMonologLevel($record['level']),
-            $record['message'],
-            [],
-            array_merge(
-                Arr::dot($record['context'] ?? [], 'context.'),
-                Arr::dot($record['extra'] ?? [], 'extra.'),
-                ['logger.channel' => $record['channel'] ?? ''],
-                ['logger.group' => $this->group],
-                ['sentry.origin' => 'auto.log.monolog']
-            )
+        return array_merge(
+            Arr::dot($record['context'] ?? [], 'context.'),
+            Arr::dot($record['extra'] ?? [], 'extra.'),
+            ['logger.channel' => $record['channel'] ?? ''],
+            ['logger.group' => $this->group],
+            ['sentry.origin' => 'auto.log.monolog']
         );
-
-        return $this->bubble === false;
     }
 }