next bunch of psalm fixes

intuibase · intuibase · commit 8aa437ea384a · 2025-02-12T16:57:33.000+01:00
diff --git a/src/Instrumentation/CakePHP/tests/Integration/CommandTest.php b/src/Instrumentation/CakePHP/tests/Integration/CommandTest.php
@@ -6,6 +6,7 @@
 
 use Cake\Console\TestSuite\ConsoleIntegrationTestTrait;
 
+/** @psalm-suppress UnusedClass */
 class CommandTest extends TestCase
 {
     use ConsoleIntegrationTestTrait;
diff --git a/src/Instrumentation/Laravel/src/Hooks/Illuminate/Console/Command.php b/src/Instrumentation/Laravel/src/Hooks/Illuminate/Console/Command.php
@@ -24,6 +24,7 @@ public function instrument(): void
         $this->hookExecute();
     }
 
+    /** @psalm-suppress PossiblyUnusedReturnValue  */
     protected function hookExecute(): bool
     {
         return hook(
diff --git a/src/Instrumentation/Laravel/src/Hooks/Illuminate/Contracts/Console/Kernel.php b/src/Instrumentation/Laravel/src/Hooks/Illuminate/Contracts/Console/Kernel.php
@@ -32,6 +32,7 @@ public function instrument(): void
         }
     }
 
+    /** @psalm-suppress UnusedReturnValue  */
     private function hookHandle(): bool
     {
         return hook(
diff --git a/src/Instrumentation/Laravel/src/Hooks/Illuminate/Contracts/Http/Kernel.php b/src/Instrumentation/Laravel/src/Hooks/Illuminate/Contracts/Http/Kernel.php
@@ -33,6 +33,7 @@ public function instrument(): void
         $this->hookHandle();
     }
 
+    /** @psalm-suppress PossiblyUnusedReturnValue  */
     protected function hookHandle(): bool
     {
         return hook(
diff --git a/src/Instrumentation/Laravel/src/Hooks/Illuminate/Contracts/Queue/Queue.php b/src/Instrumentation/Laravel/src/Hooks/Illuminate/Contracts/Queue/Queue.php
@@ -31,6 +31,7 @@ public function instrument(): void
         $this->hookPushRaw();
     }
 
+    /** @psalm-suppress PossiblyUnusedReturnValue  */
     protected function hookBulk(): bool
     {
         return hook(
@@ -67,6 +68,7 @@ protected function hookBulk(): bool
         );
     }
 
+    /** @psalm-suppress PossiblyUnusedReturnValue  */
     protected function hookLater(): bool
     {
         return hook(
@@ -110,12 +112,13 @@ protected function hookLater(): bool
         );
     }
 
+    /** @psalm-suppress PossiblyUnusedReturnValue  */
     protected function hookPushRaw(): bool
     {
         return hook(
             QueueContract::class,
             'pushRaw',
-            pre: function (QueueContract $queue, array $params, string $class, string $function, ?string $filename, ?int $lineno) {
+            pre: function (QueueContract $queue, array $params, string $_class, string $_function, ?string $_filename, ?int $_lineno) {
                 /** @phan-suppress-next-line PhanParamTooFewUnpack */
                 $attributes = $this->buildMessageAttributes($queue, ...$params);
 
diff --git a/src/Instrumentation/Laravel/src/Hooks/Illuminate/Foundation/Application.php b/src/Instrumentation/Laravel/src/Hooks/Illuminate/Foundation/Application.php
@@ -24,10 +24,11 @@ class Application implements LaravelHook
 
     public function instrument(): void
     {
+        /** @psalm-suppress UnusedFunctionCall */
         hook(
             FoundationalApplication::class,
             '__construct',
-            post: function (FoundationalApplication $application, array $params, mixed $returnValue, ?Throwable $exception) {
+            post: function (FoundationalApplication $application, array $_params, mixed $_returnValue, ?Throwable $_exception) {
                 $this->registerWatchers($application, new CacheWatcher());
                 $this->registerWatchers($application, new ClientRequestWatcher($this->instrumentation));
                 $this->registerWatchers($application, new ExceptionWatcher());
diff --git a/src/Instrumentation/Laravel/src/Hooks/Illuminate/Foundation/Console/ServeCommand.php b/src/Instrumentation/Laravel/src/Hooks/Illuminate/Foundation/Console/ServeCommand.php
@@ -18,15 +18,16 @@ class ServeCommand implements LaravelHook
 
     public function instrument(): void
     {
+        /** @psalm-suppress UnusedFunctionCall */
         hook(
             FoundationServeCommand::class,
             'handle',
-            pre: static function (FoundationServeCommand $serveCommand, array $params, string $class, string $function, ?string $filename, ?int $lineno) {
+            pre: static function (FoundationServeCommand $_serveCommand, array $_params, string $_class, string $_function, ?string $_filename, ?int $_lineno) {
                 if (!property_exists(FoundationServeCommand::class, 'passthroughVariables')) {
                     return;
                 }
 
-                foreach ($_ENV as $key => $value) {
+                foreach ($_ENV as $key => $_value) {
                     if (str_starts_with($key, 'OTEL_') && !in_array($key, FoundationServeCommand::$passthroughVariables)) {
                         FoundationServeCommand::$passthroughVariables[] = $key;
                     }
diff --git a/src/Instrumentation/Laravel/src/Hooks/Illuminate/Queue/AttributesBuilder.php b/src/Instrumentation/Laravel/src/Hooks/Illuminate/Queue/AttributesBuilder.php
@@ -50,23 +50,23 @@ private function contextualMessageSystemAttributes(
         };
     }
 
-    private function beanstalkContextualAttributes(BeanstalkdQueue $queue, array $payload, ?string $queueName = null, array $options = [], mixed ...$params): array
+    private function beanstalkContextualAttributes(BeanstalkdQueue $queue, array $_payload, ?string $queueName = null, array $_options = [], mixed ...$_params): array
     {
         return [
             TraceAttributes::MESSAGING_SYSTEM => 'beanstalk',
             TraceAttributes::MESSAGING_DESTINATION_NAME => $queue->getQueue($queueName),
         ];
     }
 
-    private function redisContextualAttributes(RedisQueue $queue, array $payload, ?string $queueName = null, array $options = [], mixed ...$params): array
+    private function redisContextualAttributes(RedisQueue $queue, array $_payload, ?string $queueName = null, array $_options = [], mixed ...$_params): array
     {
         return [
             TraceAttributes::MESSAGING_SYSTEM => 'redis',
             TraceAttributes::MESSAGING_DESTINATION_NAME => $queue->getQueue($queueName),
         ];
     }
 
-    private function awsSqsContextualAttributes(SqsQueue $queue, array $payload, ?string $queueName = null, array $options = [], mixed ...$params): array
+    private function awsSqsContextualAttributes(SqsQueue $queue, array $_payload, ?string $queueName = null, array $_options = [], mixed ...$_params): array
     {
         return [
             TraceAttributes::MESSAGING_SYSTEM => TraceAttributeValues::MESSAGING_SYSTEM_AWS_SQS,
diff --git a/src/Instrumentation/Laravel/src/Hooks/Illuminate/Queue/Queue.php b/src/Instrumentation/Laravel/src/Hooks/Illuminate/Queue/Queue.php
@@ -21,12 +21,13 @@ public function instrument(): void
         $this->hookAbstractQueueCreatePayloadArray();
     }
 
+    /** @psalm-suppress PossiblyUnusedReturnValue */
     protected function hookAbstractQueueCreatePayloadArray(): bool
     {
         return hook(
             AbstractQueue::class,
             'createPayloadArray',
-            post: function (AbstractQueue $queue, array $params, array $payload, ?Throwable $exception): array {
+            post: function (AbstractQueue $_queue, array $_params, array $payload, ?Throwable $_exception): array {
                 TraceContextPropagator::getInstance()->inject($payload);
 
                 return $payload;
diff --git a/src/Instrumentation/Laravel/src/Hooks/Illuminate/Queue/SyncQueue.php b/src/Instrumentation/Laravel/src/Hooks/Illuminate/Queue/SyncQueue.php
@@ -25,6 +25,7 @@ public function instrument(): void
         $this->hookPush();
     }
 
+    /** @psalm-suppress PossiblyUnusedReturnValue */
     protected function hookPush(): bool
     {
         return hook(
diff --git a/src/Instrumentation/Laravel/src/Hooks/Illuminate/Queue/Worker.php b/src/Instrumentation/Laravel/src/Hooks/Illuminate/Queue/Worker.php
@@ -30,12 +30,13 @@ public function instrument(): void
         $this->hookWorkerGetNextJob();
     }
 
+    /** @psalm-suppress UnusedReturnValue */
     private function hookWorkerProcess(): bool
     {
         return hook(
             QueueWorker::class,
             'process',
-            pre: function (QueueWorker $worker, array $params, string $class, string $function, ?string $filename, ?int $lineno) {
+            pre: function (QueueWorker $worker, array $params, string $_class, string $_function, ?string $_filename, ?int $_lineno) {
                 $connectionName = $params[0];
                 /** @var Job $job */
                 $job = $params[1];
@@ -82,12 +83,13 @@ private function hookWorkerProcess(): bool
         );
     }
 
+    /** @psalm-suppress UnusedReturnValue */
     private function hookWorkerGetNextJob(): bool
     {
         return hook(
             QueueWorker::class,
             'getNextJob',
-            pre: function (QueueWorker $worker, array $params, string $class, string $function, ?string $filename, ?int $lineno) {
+            pre: function (QueueWorker $_worker, array $params, string $_class, string $_function, ?string $_filename, ?int $_lineno) {
                 /** @var \Illuminate\Contracts\Queue\Queue $connection */
                 $connection = $params[0];
                 $queue = $params[1];
@@ -109,7 +111,7 @@ private function hookWorkerGetNextJob(): bool
 
                 return $params;
             },
-            post: function (QueueWorker $worker, array $params, ?Job $job, ?Throwable $exception) {
+            post: function (QueueWorker $_worker, array $params, ?Job $job, ?Throwable $exception) {
                 $scope = Context::storage()->scope();
                 if (!$scope) {
                     return;
diff --git a/src/Instrumentation/Laravel/src/Hooks/LaravelHook.php b/src/Instrumentation/Laravel/src/Hooks/LaravelHook.php
@@ -8,6 +8,7 @@
 
 interface LaravelHook
 {
+    /** @psalm-suppress PossiblyUnusedReturnValue */
     public static function hook(CachedInstrumentation $instrumentation): LaravelHook;
 
     public function instrument(): void;
diff --git a/src/Instrumentation/Laravel/src/Hooks/LaravelHookTrait.php b/src/Instrumentation/Laravel/src/Hooks/LaravelHookTrait.php
@@ -17,6 +17,7 @@ protected function __construct(
 
     abstract public function instrument(): void;
 
+    /** @psalm-suppress PossiblyUnusedReturnValue */
     public static function hook(CachedInstrumentation $instrumentation): LaravelHook
     {
         /** @psalm-suppress RedundantPropertyInitializationCheck */
diff --git a/src/Instrumentation/Laravel/src/LaravelInstrumentation.php b/src/Instrumentation/Laravel/src/LaravelInstrumentation.php
@@ -11,6 +11,7 @@ class LaravelInstrumentation
 {
     public const NAME = 'laravel';
 
+    /** @psalm-suppress PossiblyUnusedMethod */
     public static function register(): void
     {
         $instrumentation = new CachedInstrumentation(
diff --git a/src/Instrumentation/Laravel/src/Watchers/CacheWatcher.php b/src/Instrumentation/Laravel/src/Watchers/CacheWatcher.php
@@ -27,6 +27,7 @@ public function register(Application $app): void
         $app['events']->listen(KeyForgotten::class, [$this, 'recordCacheForget']);
     }
 
+    /** @psalm-suppress PossiblyUnusedMethod */
     public function recordCacheHit(CacheHit $event): void
     {
         $this->addEvent('cache hit', [
@@ -35,6 +36,7 @@ public function recordCacheHit(CacheHit $event): void
         ]);
     }
 
+    /** @psalm-suppress PossiblyUnusedMethod */
     public function recordCacheMiss(CacheMissed $event): void
     {
         $this->addEvent('cache miss', [
@@ -44,6 +46,7 @@ public function recordCacheMiss(CacheMissed $event): void
     }
     /**
      * @psalm-suppress UndefinedPropertyFetch
+     * @psalm-suppress PossiblyUnusedMethod
      * @suppress PhanUndeclaredProperty
      */
     public function recordCacheSet(KeyWritten $event): void
@@ -67,6 +70,7 @@ public function recordCacheSet(KeyWritten $event): void
         ]);
     }
 
+    /** @psalm-suppress PossiblyUnusedMethod */
     public function recordCacheForget(KeyForgotten $event): void
     {
         $this->addEvent('cache forget', [
diff --git a/src/Instrumentation/Laravel/src/Watchers/ClientRequestWatcher.php b/src/Instrumentation/Laravel/src/Watchers/ClientRequestWatcher.php
@@ -42,11 +42,12 @@ public function register(Application $app): void
 
     /**
      * @psalm-suppress ArgumentTypeCoercion
+     * @psalm-suppress PossiblyUnusedMethod
      * @suppress PhanEmptyFQSENInCallable,PhanUndeclaredFunctionInCallable
      */
     public function recordRequest(RequestSending $request): void
     {
-        $parsedUrl = collect(parse_url($request->request->url()));
+        $parsedUrl = collect(parse_url($request->request->url()) ?: []);
         $processedUrl = $parsedUrl->get('scheme', 'http') . '://' . $parsedUrl->get('host') . $parsedUrl->get('path', '');
 
         if ($parsedUrl->has('query')) {
@@ -66,6 +67,7 @@ public function recordRequest(RequestSending $request): void
         $this->spans[$this->createRequestComparisonHash($request->request)] = $span;
     }
 
+    /** @psalm-suppress PossiblyUnusedMethod */
     public function recordConnectionFailed(ConnectionFailed $request): void
     {
         $requestHash = $this->createRequestComparisonHash($request->request);
@@ -81,6 +83,7 @@ public function recordConnectionFailed(ConnectionFailed $request): void
         unset($this->spans[$requestHash]);
     }
 
+    /** @psalm-suppress PossiblyUnusedMethod */
     public function recordResponse(ResponseReceived $request): void
     {
         $requestHash = $this->createRequestComparisonHash($request->request);
diff --git a/src/Instrumentation/Laravel/src/Watchers/ExceptionWatcher.php b/src/Instrumentation/Laravel/src/Watchers/ExceptionWatcher.php
@@ -21,7 +21,8 @@ public function register(Application $app): void
     }
     /**
      * Record an exception.
-     */
+     * @psalm-suppress PossiblyUnusedMethod
+    */
     public function recordException(MessageLogged $log): void
     {
         if (! isset($log->context['exception']) ||
diff --git a/src/Instrumentation/Laravel/src/Watchers/LogWatcher.php b/src/Instrumentation/Laravel/src/Watchers/LogWatcher.php
@@ -33,6 +33,7 @@ public function register(Application $app): void
     /**
      * Record a log.
      * @phan-suppress PhanDeprecatedFunction
+     * @psalm-suppress PossiblyUnusedMethod
      */
     public function recordLog(MessageLogged $log): void
     {
diff --git a/src/Instrumentation/Laravel/src/Watchers/QueryWatcher.php b/src/Instrumentation/Laravel/src/Watchers/QueryWatcher.php
@@ -27,8 +27,8 @@ public function register(Application $app): void
 
     /**
      * Record a query.
+     * @psalm-suppress PossiblyUnusedMethod
      */
-    /** @psalm-suppress UndefinedThisPropertyFetch */
     public function recordQuery(QueryExecuted $query): void
     {
         $nowInNs = (int) (microtime(true) * 1E9);
diff --git a/src/Instrumentation/Laravel/src/Watchers/RedisCommand/RedisCommandWatcher.php b/src/Instrumentation/Laravel/src/Watchers/RedisCommand/RedisCommandWatcher.php
@@ -37,8 +37,8 @@ public function register(Application $app): void
 
     /**
      * Record a Redis command.
+     * @psalm-suppress PossiblyUnusedMethod
      */
-    /** @psalm-suppress UndefinedThisPropertyFetch */
     public function recordRedisCommand(CommandExecuted $event): void
     {
         $nowInNs = (int) (microtime(true) * 1E9);
diff --git a/src/Instrumentation/Laravel/tests/Fixtures/Jobs/DummyJob.php b/src/Instrumentation/Laravel/tests/Fixtures/Jobs/DummyJob.php
@@ -12,11 +12,13 @@ class DummyJob implements ShouldQueue
 {
     use Queueable;
 
+    /** @psalm-suppress PossiblyUnusedMethod */
     public function __construct(
         private string $name,
     ) {
     }
 
+    /** @psalm-suppress PossiblyUnusedMethod */
     public function handle(LoggerInterface $logger): void
     {
         $logger->info("Task: {$this->name}");
diff --git a/src/Instrumentation/Laravel/tests/Integration/Console/CommandTest.php b/src/Instrumentation/Laravel/tests/Integration/Console/CommandTest.php
@@ -8,6 +8,7 @@
 use Illuminate\Contracts\Console\Kernel;
 use OpenTelemetry\Tests\Contrib\Instrumentation\Laravel\Integration\TestCase;
 
+/** @psalm-suppress UnusedClass */
 class CommandTest extends TestCase
 {
     public function test_command_tracing(): void
diff --git a/src/Instrumentation/Laravel/tests/Integration/Http/ClientTest.php b/src/Instrumentation/Laravel/tests/Integration/Http/ClientTest.php
@@ -13,6 +13,7 @@
 use OpenTelemetry\SemConv\TraceAttributes;
 use OpenTelemetry\Tests\Contrib\Instrumentation\Laravel\Integration\TestCase;
 
+/** @psalm-suppress UnusedClass */
 class ClientTest extends TestCase
 {
     public function test_it_records_requests(): void
diff --git a/src/Instrumentation/Laravel/tests/Integration/LaravelInstrumentationTest.php b/src/Instrumentation/Laravel/tests/Integration/LaravelInstrumentationTest.php
@@ -10,6 +10,7 @@
 use Illuminate\Support\Facades\Log;
 use OpenTelemetry\SemConv\TraceAttributes;
 
+/** @psalm-suppress UnusedClass */
 class LaravelInstrumentationTest extends TestCase
 {
     public function test_request_response(): void
diff --git a/src/Instrumentation/Laravel/tests/Integration/Queue/QueueTest.php b/src/Instrumentation/Laravel/tests/Integration/Queue/QueueTest.php
@@ -19,6 +19,7 @@
 use OpenTelemetry\Tests\Contrib\Instrumentation\Laravel\Integration\TestCase;
 use Psr\Log\LoggerInterface;
 
+/** @psalm-suppress UnusedClass */
 class QueueTest extends TestCase
 {
     private Queue $queue;
diff --git a/src/Instrumentation/Laravel/tests/Integration/TestCase.php b/src/Instrumentation/Laravel/tests/Integration/TestCase.php
@@ -18,6 +18,7 @@
 use OpenTelemetry\SDK\Trace\TracerProvider;
 use Orchestra\Testbench\TestCase as BaseTestCase;
 
+/** @psalm-suppress UnusedClass */
 abstract class TestCase extends BaseTestCase
 {
     protected ScopeInterface $scope;
diff --git a/src/Instrumentation/Psr14/tests/Fixture/SampleEventClass.php b/src/Instrumentation/Psr14/tests/Fixture/SampleEventClass.php
@@ -12,7 +12,8 @@ final class SampleEventClass
     /** @psalm-suppress PossiblyUnusedProperty */
     public string $item;
 
-    public function __construct(string $item = 'test') {
+    public function __construct(string $item = 'test')
+    {
         $this->item = $item;
     }
 }
diff --git a/src/Instrumentation/Psr3/src/Psr3Instrumentation.php b/src/Instrumentation/Psr3/src/Psr3Instrumentation.php
@@ -7,7 +7,6 @@
 use OpenTelemetry\API\Instrumentation\CachedInstrumentation;
 use OpenTelemetry\API\Instrumentation\ConfigurationResolver;
 use OpenTelemetry\API\Logs as API;
-use OpenTelemetry\API\Logs\Severity;
 use OpenTelemetry\API\Trace\Span;
 use OpenTelemetry\Context\Context;
 use function OpenTelemetry\Instrumentation\hook;
diff --git a/src/Instrumentation/Symfony/tests/Integration/HttpClientInstrumentationTest.php b/src/Instrumentation/Symfony/tests/Integration/HttpClientInstrumentationTest.php
@@ -5,8 +5,6 @@
 namespace OpenTelemetry\Tests\Instrumentation\Symfony\tests\Integration;
 
 use OpenTelemetry\API\Trace\StatusCode;
-use OpenTelemetry\SDK\Trace\EventInterface;
-use OpenTelemetry\SDK\Trace\ImmutableSpan;
 use OpenTelemetry\SemConv\TraceAttributes;
 use Symfony\Component\HttpClient\CurlHttpClient;
 use Symfony\Component\HttpClient\Exception\InvalidArgumentException;
diff --git a/src/Instrumentation/Symfony/tests/Integration/MessengerInstrumentationTest.php b/src/Instrumentation/Symfony/tests/Integration/MessengerInstrumentationTest.php
@@ -6,7 +6,6 @@
 
 use OpenTelemetry\API\Trace\SpanKind;
 use OpenTelemetry\Contrib\Instrumentation\Symfony\MessengerInstrumentation;
-use OpenTelemetry\SDK\Trace\ImmutableSpan;
 use Symfony\Component\Messenger\Envelope;
 use Symfony\Component\Messenger\MessageBus;
 use Symfony\Component\Messenger\MessageBusInterface;
diff --git a/src/Instrumentation/Symfony/tests/Integration/SymfonyInstrumentationTest.php b/src/Instrumentation/Symfony/tests/Integration/SymfonyInstrumentationTest.php
diff --git a/src/Logs/Monolog/src/Handler.php b/src/Logs/Monolog/src/Handler.php

Original file line number	Diff line number	Diff line change
`@@ -6,6 +6,7 @@`
`6`	`6`
`7`	`7`	`use Cake\Console\TestSuite\ConsoleIntegrationTestTrait;`
`8`	`8`
	`9`	`+/** @psalm-suppress UnusedClass */`
`9`	`10`	`class CommandTest extends TestCase`
`10`	`11`	`{`
`11`	`12`	`use ConsoleIntegrationTestTrait;`
Original file line number	Diff line number	Diff line change
`@@ -24,6 +24,7 @@ public function instrument(): void`
`24`	`24`	`$this->hookExecute();`
`25`	`25`	`}`
`26`	`26`
	`27`	`+ /** @psalm-suppress PossiblyUnusedReturnValue */`
`27`	`28`	`protected function hookExecute(): bool`
`28`	`29`	`{`
`29`	`30`	`return hook(`
Original file line number	Diff line number	Diff line change
`@@ -32,6 +32,7 @@ public function instrument(): void`
`32`	`32`	`}`
`33`	`33`	`}`
`34`	`34`
	`35`	`+ /** @psalm-suppress UnusedReturnValue */`
`35`	`36`	`private function hookHandle(): bool`
`36`	`37`	`{`
`37`	`38`	`return hook(`
Original file line number	Diff line number	Diff line change
`@@ -33,6 +33,7 @@ public function instrument(): void`
`33`	`33`	`$this->hookHandle();`
`34`	`34`	`}`
`35`	`35`
	`36`	`+ /** @psalm-suppress PossiblyUnusedReturnValue */`
`36`	`37`	`protected function hookHandle(): bool`
`37`	`38`	`{`
`38`	`39`	`return hook(`
Original file line number	Diff line number	Diff line change
`@@ -31,6 +31,7 @@ public function instrument(): void`
`31`	`31`	`$this->hookPushRaw();`
`32`	`32`	`}`
`33`	`33`
	`34`	`+ /** @psalm-suppress PossiblyUnusedReturnValue */`
`34`	`35`	`protected function hookBulk(): bool`
`35`	`36`	`{`
`36`	`37`	`return hook(`
`@@ -67,6 +68,7 @@ protected function hookBulk(): bool`
`67`	`68`	`);`
`68`	`69`	`}`
`69`	`70`
	`71`	`+ /** @psalm-suppress PossiblyUnusedReturnValue */`
`70`	`72`	`protected function hookLater(): bool`
`71`	`73`	`{`
`72`	`74`	`return hook(`
`@@ -110,12 +112,13 @@ protected function hookLater(): bool`
`110`	`112`	`);`
`111`	`113`	`}`
`112`	`114`
	`115`	`+ /** @psalm-suppress PossiblyUnusedReturnValue */`
`113`	`116`	`protected function hookPushRaw(): bool`
`114`	`117`	`{`
`115`	`118`	`return hook(`
`116`	`119`	`QueueContract::class,`
`117`	`120`	`'pushRaw',`
`118`		`- pre: function (QueueContract $queue, array $params, string $class, string $function, ?string $filename, ?int $lineno) {`
	`121`	`+ pre: function (QueueContract $queue, array $params, string $_class, string $_function, ?string $_filename, ?int $_lineno) {`
`119`	`122`	`/** @phan-suppress-next-line PhanParamTooFewUnpack */`
`120`	`123`	`$attributes = $this->buildMessageAttributes($queue, ...$params);`
`121`	`124`
Original file line number	Diff line number	Diff line change
`@@ -18,15 +18,16 @@ class ServeCommand implements LaravelHook`
`18`	`18`
`19`	`19`	`public function instrument(): void`
`20`	`20`	`{`
	`21`	`+ /** @psalm-suppress UnusedFunctionCall */`
`21`	`22`	`hook(`
`22`	`23`	`FoundationServeCommand::class,`
`23`	`24`	`'handle',`
`24`		`- pre: static function (FoundationServeCommand $serveCommand, array $params, string $class, string $function, ?string $filename, ?int $lineno) {`
	`25`	`+ pre: static function (FoundationServeCommand $_serveCommand, array $_params, string $_class, string $_function, ?string $_filename, ?int $_lineno) {`
`25`	`26`	`if (!property_exists(FoundationServeCommand::class, 'passthroughVariables')) {`
`26`	`27`	`return;`
`27`	`28`	`}`
`28`	`29`
`29`		`- foreach ($_ENV as $key => $value) {`
	`30`	`+ foreach ($_ENV as $key => $_value) {`
`30`	`31`	`if (str_starts_with($key, 'OTEL_') && !in_array($key, FoundationServeCommand::$passthroughVariables)) {`
`31`	`32`	`FoundationServeCommand::$passthroughVariables[] = $key;`
`32`	`33`	`}`
Original file line number	Diff line number	Diff line change
`@@ -50,23 +50,23 @@ private function contextualMessageSystemAttributes(`
`50`	`50`	`};`
`51`	`51`	`}`
`52`	`52`
`53`		`- private function beanstalkContextualAttributes(BeanstalkdQueue $queue, array $payload, ?string $queueName = null, array $options = [], mixed ...$params): array`
	`53`	`+ private function beanstalkContextualAttributes(BeanstalkdQueue $queue, array $_payload, ?string $queueName = null, array $_options = [], mixed ...$_params): array`
`54`	`54`	`{`
`55`	`55`	`return [`
`56`	`56`	`TraceAttributes::MESSAGING_SYSTEM => 'beanstalk',`
`57`	`57`	`TraceAttributes::MESSAGING_DESTINATION_NAME => $queue->getQueue($queueName),`
`58`	`58`	`];`
`59`	`59`	`}`
`60`	`60`
`61`		`- private function redisContextualAttributes(RedisQueue $queue, array $payload, ?string $queueName = null, array $options = [], mixed ...$params): array`
	`61`	`+ private function redisContextualAttributes(RedisQueue $queue, array $_payload, ?string $queueName = null, array $_options = [], mixed ...$_params): array`
`62`	`62`	`{`
`63`	`63`	`return [`
`64`	`64`	`TraceAttributes::MESSAGING_SYSTEM => 'redis',`
`65`	`65`	`TraceAttributes::MESSAGING_DESTINATION_NAME => $queue->getQueue($queueName),`
`66`	`66`	`];`
`67`	`67`	`}`
`68`	`68`
`69`		`- private function awsSqsContextualAttributes(SqsQueue $queue, array $payload, ?string $queueName = null, array $options = [], mixed ...$params): array`
	`69`	`+ private function awsSqsContextualAttributes(SqsQueue $queue, array $_payload, ?string $queueName = null, array $_options = [], mixed ...$_params): array`
`70`	`70`	`{`
`71`	`71`	`return [`
`72`	`72`	`TraceAttributes::MESSAGING_SYSTEM => TraceAttributeValues::MESSAGING_SYSTEM_AWS_SQS,`
Original file line number	Diff line number	Diff line change
`@@ -21,12 +21,13 @@ public function instrument(): void`
`21`	`21`	`$this->hookAbstractQueueCreatePayloadArray();`
`22`	`22`	`}`
`23`	`23`
	`24`	`+ /** @psalm-suppress PossiblyUnusedReturnValue */`
`24`	`25`	`protected function hookAbstractQueueCreatePayloadArray(): bool`
`25`	`26`	`{`
`26`	`27`	`return hook(`
`27`	`28`	`AbstractQueue::class,`
`28`	`29`	`'createPayloadArray',`
`29`		`- post: function (AbstractQueue $queue, array $params, array $payload, ?Throwable $exception): array {`
	`30`	`+ post: function (AbstractQueue $_queue, array $_params, array $payload, ?Throwable $_exception): array {`
`30`	`31`	`TraceContextPropagator::getInstance()->inject($payload);`
`31`	`32`
`32`	`33`	`return $payload;`
Original file line number	Diff line number	Diff line change
`@@ -25,6 +25,7 @@ public function instrument(): void`
`25`	`25`	`$this->hookPush();`
`26`	`26`	`}`
`27`	`27`
	`28`	`+ /** @psalm-suppress PossiblyUnusedReturnValue */`
`28`	`29`	`protected function hookPush(): bool`
`29`	`30`	`{`
`30`	`31`	`return hook(`