From dc7e9ad6508e45299769d68c36c4470ef6704f08 Mon Sep 17 00:00:00 2001 From: Deeka Wong <8337659+huangdijia@users.noreply.github.com> Date: Sun, 28 Sep 2025 08:09:34 +0800 Subject: [PATCH] refactor(sentry): simplify hub initialization in Tracer - Replace complex tap-based hub cloning with direct SentrySdk::init() - Remove unused imports: HubInterface and tap function - Simplify startTransaction method while maintaining functionality - Improve code readability and reduce dependencies --- src/sentry/src/Tracing/Tracer.php | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/sentry/src/Tracing/Tracer.php b/src/sentry/src/Tracing/Tracer.php index 0286f2d97..d7f46bf65 100644 --- a/src/sentry/src/Tracing/Tracer.php +++ b/src/sentry/src/Tracing/Tracer.php @@ -14,7 +14,6 @@ use FriendsOfHyperf\Sentry\Feature; use Hyperf\Engine\Coroutine; use Sentry\SentrySdk; -use Sentry\State\HubInterface; use Sentry\State\Scope; use Sentry\Tracing\SpanContext; use Sentry\Tracing\SpanStatus; @@ -23,7 +22,6 @@ use Sentry\Tracing\TransactionSource; use Throwable; -use function Hyperf\Tappable\tap; use function Sentry\trace; class Tracer @@ -37,9 +35,8 @@ public function __construct(protected Feature $feature) */ public function startTransaction(TransactionContext $transactionContext, array $customSamplingContext = []): Transaction { - $hub = SentrySdk::setCurrentHub( - tap(clone SentrySdk::getCurrentHub(), fn (HubInterface $hub) => $hub->pushScope()) - ); + $hub = SentrySdk::init(); + $hub->pushScope(); $transactionContext->setData(['coroutine.id' => Coroutine::id()] + $transactionContext->getData());