Fix PHP8.4 nullable types deprecations

src/Activity/ActivityMethod.php

@@ -33,7 +33,7 @@ final class ActivityMethod
     #[Immutable]
     public ?string $name = null;
 
-    public function __construct(string $name = null)
+    public function __construct(?string $name = null)
     {
         $this->name = $name;
     }

src/Activity/ActivityOptions.php

@@ -125,7 +125,7 @@ public function __construct()
     /**
      * @return $this
      */
-    public function mergeWith(MethodRetry $retry = null): self
+    public function mergeWith(?MethodRetry $retry = null): self
     {
         $self = clone $this;
 

src/Activity/ActivityOptionsInterface.php

@@ -11,5 +11,5 @@
  */
 interface ActivityOptionsInterface
 {
-    public function mergeWith(MethodRetry $retry = null): self;
+    public function mergeWith(?MethodRetry $retry = null): self;
 }

src/Activity/LocalActivityOptions.php

@@ -81,7 +81,7 @@ public function __construct()
     /**
      * @return $this
      */
-    public function mergeWith(MethodRetry $retry = null): self
+    public function mergeWith(?MethodRetry $retry = null): self
     {
         $self = clone $this;
 

src/Client/GRPC/BaseClient.php

@@ -91,10 +91,10 @@ public static function create(string $address): static
      */
     public static function createSSL(
         string $address,
-        string $crt = null,
-        string $clientKey = null,
-        string $clientPem = null,
-        string $overrideServerName = null,
+        ?string $crt = null,
+        ?string $clientKey = null,
+        ?string $clientPem = null,
+        ?string $overrideServerName = null,
     ): static {
         if (!\extension_loaded('grpc')) {
             throw new \RuntimeException('The gRPC extension is required to use Temporal Client.');

src/Client/ScheduleClient.php

@@ -48,8 +48,8 @@ final class ScheduleClient implements ScheduleClientInterface
 
     public function __construct(
         ServiceClientInterface $serviceClient,
-        ClientOptions $options = null,
-        DataConverterInterface $converter = null,
+        ?ClientOptions $options = null,
+        ?DataConverterInterface $converter = null,
     ) {
         $this->client = $serviceClient;
         $this->clientOptions = $options ?? new ClientOptions();
@@ -62,8 +62,8 @@ public function __construct(
 
     public static function create(
         ServiceClientInterface $serviceClient,
-        ClientOptions $options = null,
-        DataConverterInterface $converter = null,
+        ?ClientOptions $options = null,
+        ?DataConverterInterface $converter = null,
     ): ScheduleClientInterface {
         return new self($serviceClient, $options, $converter);
     }

src/Client/WorkflowClient.php

@@ -66,9 +66,9 @@ class WorkflowClient implements WorkflowClientInterface
 
     public function __construct(
         ServiceClientInterface $serviceClient,
-        ClientOptions $options = null,
-        DataConverterInterface $converter = null,
-        PipelineProvider $interceptorProvider = null,
+        ?ClientOptions $options = null,
+        ?DataConverterInterface $converter = null,
+        ?PipelineProvider $interceptorProvider = null,
     ) {
         $this->client = $serviceClient;
         $this->interceptorPipeline = ($interceptorProvider ?? new SimplePipelineProvider())
@@ -83,9 +83,9 @@ public function __construct(
      */
     public static function create(
         ServiceClientInterface $serviceClient,
-        ClientOptions $options = null,
-        DataConverterInterface $converter = null,
-        PipelineProvider $interceptorProvider = null,
+        ?ClientOptions $options = null,
+        ?DataConverterInterface $converter = null,
+        ?PipelineProvider $interceptorProvider = null,
     ): self {
         return new self($serviceClient, $options, $converter, $interceptorProvider);
     }
@@ -207,7 +207,7 @@ public function startWithSignal(
 
     public function newWorkflowStub(
         string $class,
-        WorkflowOptions $options = null,
+        ?WorkflowOptions $options = null,
     ): object {
         $workflow = $this->reader->fromClass($class);
 
@@ -220,7 +220,7 @@ public function newWorkflowStub(
 
     public function newUntypedWorkflowStub(
         string $workflowType,
-        WorkflowOptions $options = null,
+        ?WorkflowOptions $options = null,
     ): WorkflowStubInterface {
         $options ??= new WorkflowOptions();
 

src/Client/WorkflowClientInterface.php

@@ -62,7 +62,7 @@ public function startWithSignal(
      */
     public function newWorkflowStub(
         string $class,
-        WorkflowOptions $options = null,
+        ?WorkflowOptions $options = null,
     ): object;
 
     /**
@@ -78,7 +78,7 @@ public function newWorkflowStub(
      */
     public function newUntypedWorkflowStub(
         string $workflowType,
-        WorkflowOptions $options = null,
+        ?WorkflowOptions $options = null,
     ): WorkflowStubInterface;
 
     /**

src/Client/WorkflowOptions.php

@@ -166,7 +166,7 @@ public function __construct()
      *
      * @return self return a new {@see self} instance with merged options
      */
-    public function mergeWith(MethodRetry $retry = null, CronSchedule $cron = null): self
+    public function mergeWith(?MethodRetry $retry = null, ?CronSchedule $cron = null): self
     {
         $self = clone $this;
 

src/Common/RetryOptions.php

@@ -106,7 +106,7 @@ class RetryOptions extends Options
     #[Marshal(name: 'non_retryable_error_types')]
     public array $nonRetryableExceptions = self::DEFAULT_NON_RETRYABLE_EXCEPTIONS;
 
-    public function mergeWith(MethodRetry $retry = null): self
+    public function mergeWith(?MethodRetry $retry = null): self
     {
         $self = clone $this;
 

src/DataConverter/EncodedValues.php

@@ -62,7 +62,7 @@ public static function sliceValues(
         DataConverterInterface $converter,
         ValuesInterface $values,
         int $offset,
-        int $length = null,
+        ?int $length = null,
     ): ValuesInterface {
         $payloads = $values->toPayloads();
         $newPayloads = new Payloads();
@@ -90,7 +90,7 @@ static function (mixed $value) use ($type) {
         );
     }
 
-    public static function fromValues(array $values, DataConverterInterface $dataConverter = null): static
+    public static function fromValues(array $values, ?DataConverterInterface $dataConverter = null): static
     {
         $ev = new static();
         $ev->values = \array_values($values);

src/DataConverter/JsonConverter.php

@@ -36,7 +36,7 @@ class JsonConverter extends Converter
 
     private MarshallerInterface $marshaller;
 
-    public function __construct(MarshallerInterface $marshaller = null)
+    public function __construct(?MarshallerInterface $marshaller = null)
     {
         $this->marshaller = $marshaller ?? self::createDefaultMarshaller();
     }

src/DataConverter/Type.php

@@ -37,7 +37,7 @@ final class Type
      */
     public function __construct(
         private readonly string $name = Type::TYPE_ANY,
-        bool $allowsNull = null,
+        ?bool $allowsNull = null,
         private readonly bool $isArrayOf = false,
     ) {
         $this->allowsNull = $allowsNull ?? (

src/Exception/Client/ActivityCompletionException.php

@@ -47,7 +47,7 @@ public static function fromPreviousWithActivityId(string $activityId, \Throwable
     /**
      * @return static
      */
-    public static function fromActivityInfo(ActivityInfo $info, \Throwable $e = null): self
+    public static function fromActivityInfo(ActivityInfo $info, ?\Throwable $e = null): self
     {
         $e = new static(
             self::buildMessage(

src/Exception/Client/ServiceClientException.php

@@ -23,7 +23,7 @@ class ServiceClientException extends \RuntimeException
     /**
      * @throws \Exception
      */
-    public function __construct(\stdClass $status, \Throwable $previous = null)
+    public function __construct(\stdClass $status, ?\Throwable $previous = null)
     {
         $this->status = new Status();
 

src/Exception/Client/WorkflowException.php

@@ -22,8 +22,8 @@ class WorkflowException extends TemporalException
     public function __construct(
         ?string $message,
         WorkflowExecution $execution,
-        string $workflowType = null,
-        \Throwable $previous = null,
+        ?string $workflowType = null,
+        ?\Throwable $previous = null,
     ) {
         parent::__construct(
             self::buildMessage(
@@ -42,8 +42,8 @@ public function __construct(
 
     public static function withoutMessage(
         WorkflowExecution $execution,
-        string $workflowType = null,
-        \Throwable $previous = null,
+        ?string $workflowType = null,
+        ?\Throwable $previous = null,
     ): WorkflowException {
         return new static(null, $execution, $workflowType, $previous);
     }

src/Exception/Client/WorkflowExecutionAlreadyStartedException.php

@@ -17,8 +17,8 @@ class WorkflowExecutionAlreadyStartedException extends WorkflowException
 {
     public function __construct(
         WorkflowExecution $execution,
-        string $type = null,
-        \Throwable $previous = null,
+        ?string $type = null,
+        ?\Throwable $previous = null,
     ) {
         parent::__construct(null, $execution, $type, $previous);
     }

src/Exception/Client/WorkflowFailedException.php

@@ -25,7 +25,7 @@ public function __construct(
         ?string $type,
         int $lastWorkflowTaskCompletedEventId,
         int $retryState,
-        \Throwable $previous = null,
+        ?\Throwable $previous = null,
     ) {
         parent::__construct(null, $execution, $type, $previous);
 

src/Exception/Client/WorkflowQueryRejectedException.php

@@ -23,7 +23,7 @@ public function __construct(
         string $type,
         int $queryRejectCondition,
         int $workflowExecutionStatus,
-        \Throwable $previous = null,
+        ?\Throwable $previous = null,
     ) {
         parent::__construct(null, $execution, $type, $previous);
         $this->queryRejectCondition = $queryRejectCondition;

src/Exception/Failure/ActivityFailure.php

@@ -27,7 +27,7 @@ public function __construct(
         string $activityId,
         int $retryState,
         string $identity,
-        \Throwable $previous = null,
+        ?\Throwable $previous = null,
     ) {
         parent::__construct(
             self::buildMessage(

src/Exception/Failure/ApplicationFailure.php

@@ -54,8 +54,8 @@ public function __construct(
         string $message,
         string $type,
         bool $nonRetryable,
-        ValuesInterface $details = null,
-        \Throwable $previous = null,
+        ?ValuesInterface $details = null,
+        ?\Throwable $previous = null,
         private ?\DateInterval $nextRetryDelay = null,
     ) {
         parent::__construct(

src/Exception/Failure/CanceledFailure.php

@@ -19,7 +19,7 @@ class CanceledFailure extends TemporalFailure
 {
     private ValuesInterface $details;
 
-    public function __construct(string $message, ValuesInterface $details = null, \Throwable $previous = null)
+    public function __construct(string $message, ?ValuesInterface $details = null, ?\Throwable $previous = null)
     {
         parent::__construct($message, '', $previous);
         $this->details = $details ?? EncodedValues::empty();

src/Exception/Failure/ChildWorkflowFailure.php

@@ -29,7 +29,7 @@ public function __construct(
         WorkflowExecution $execution,
         string $namespace,
         int $retryState,
-        \Throwable $previous = null,
+        ?\Throwable $previous = null,
     ) {
         parent::__construct(
             self::buildMessage(

src/Exception/Failure/ServerFailure.php

@@ -15,7 +15,7 @@ class ServerFailure extends TemporalFailure
 {
     private bool $nonRetryable;
 
-    public function __construct(string $message, bool $nonRetryable, \Throwable $previous = null)
+    public function __construct(string $message, bool $nonRetryable, ?\Throwable $previous = null)
     {
         parent::__construct(
             $message,

src/Exception/Failure/TemporalFailure.php

@@ -35,7 +35,7 @@ class TemporalFailure extends TemporalException implements \Stringable
     private string $originalMessage;
     private ?string $originalStackTrace = null;
 
-    public function __construct(string $message, string $originalMessage = null, \Throwable $previous = null)
+    public function __construct(string $message, ?string $originalMessage = null, ?\Throwable $previous = null)
     {
         parent::__construct($message, 0, $previous);
 

src/Exception/Failure/TerminatedFailure.php

@@ -13,7 +13,7 @@
 
 class TerminatedFailure extends TemporalFailure
 {
-    public function __construct(string $message, \Throwable $previous = null)
+    public function __construct(string $message, ?\Throwable $previous = null)
     {
         parent::__construct($message, $message, $previous);
     }

src/Exception/Failure/TimeoutFailure.php

@@ -23,7 +23,7 @@ public function __construct(
         string $message,
         ValuesInterface $lastHeartbeatDetails,
         int $timeoutWorkflowType,
-        \Throwable $previous = null,
+        ?\Throwable $previous = null,
     ) {
         parent::__construct(
             self::buildMessage(\compact('message', 'timeoutWorkflowType') + ['type' => 'TimeoutFailure']),

src/Interceptor/ActivityInbound/ActivityInput.php

@@ -29,8 +29,8 @@ public function __construct(
     ) {}
 
     public function with(
-        ValuesInterface $arguments = null,
-        HeaderInterface $header = null,
+        ?ValuesInterface $arguments = null,
+        ?HeaderInterface $header = null,
     ): self {
         return new self(
             $arguments ?? $this->arguments,

src/Interceptor/WorkflowClient/CancelInput.php

@@ -27,7 +27,7 @@ public function __construct(
     ) {}
 
     public function with(
-        WorkflowExecution $workflowExecution = null,
+        ?WorkflowExecution $workflowExecution = null,
     ): self {
         return new self(
             $workflowExecution ?? $this->workflowExecution,

src/Interceptor/WorkflowClient/DescribeInput.php

@@ -28,8 +28,8 @@ public function __construct(
     ) {}
 
     public function with(
-        WorkflowExecution $workflowExecution = null,
-        string $namespace = null,
+        ?WorkflowExecution $workflowExecution = null,
+        ?string $namespace = null,
     ): self {
         return new self(
             $workflowExecution ?? $this->workflowExecution,

src/Interceptor/WorkflowClient/GetResultInput.php

@@ -30,9 +30,9 @@ public function __construct(
     ) {}
 
     public function with(
-        WorkflowExecution $workflowExecution = null,
-        string $workflowType = null,
-        int $timeout = null,
+        ?WorkflowExecution $workflowExecution = null,
+        ?string $workflowType = null,
+        ?int $timeout = null,
     ): self {
         return new self(
             $workflowExecution ?? $this->workflowExecution,

src/Interceptor/WorkflowClient/QueryInput.php

@@ -31,9 +31,9 @@ public function __construct(
     ) {}
 
     public function with(
-        WorkflowExecution $workflowExecution = null,
-        string $queryType = null,
-        ValuesInterface $arguments = null,
+        ?WorkflowExecution $workflowExecution = null,
+        ?string $queryType = null,
+        ?ValuesInterface $arguments = null,
     ): self {
         return new self(
             $workflowExecution ?? $this->workflowExecution,

src/Interceptor/WorkflowClient/SignalInput.php

@@ -31,8 +31,8 @@ public function __construct(
     ) {}
 
     public function with(
-        string $signalName = null,
-        ValuesInterface $arguments = null,
+        ?string $signalName = null,
+        ?ValuesInterface $arguments = null,
     ): self {
         return new self(
             $this->workflowExecution,