AI: Update PHP AI Client package to 1.2.1.

This package update allows registered providers to have a description, and it fixes an issue with Redis compatibility.

Props jason_the_adams, laurisaarni.
See #64591.


git-svn-id: https://develop.svn.wordpress.org/trunk@61776 602fd350-edb4-49c9-b593-d223f7449a82

Author: felixarntz

src/wp-includes/php-ai-client/src/AiClient.php

@@ -84,7 +84,7 @@ class AiClient
     /**
      * @var string The version of the AI Client.
      */
-    public const VERSION = '1.1.0';
+    public const VERSION = '1.2.1';
     /**
      * @var ProviderRegistry|null The default provider registry instance.
      */

src/wp-includes/php-ai-client/src/Providers/DTO/ProviderMetadata.php

@@ -13,10 +13,12 @@
  * unique identifier, display name, and type (cloud, server, or client).
  *
  * @since 0.1.0
+ * @since 1.2.0 Added optional description property.
  *
  * @phpstan-type ProviderMetadataArrayShape array{
  *     id: string,
  *     name: string,
+ *     description?: ?string,
  *     type: string,
  *     credentialsUrl?: ?string,
  *     authenticationMethod?: ?string
@@ -28,6 +30,7 @@ class ProviderMetadata extends AbstractDataTransferObject
 {
     public const KEY_ID = 'id';
     public const KEY_NAME = 'name';
+    public const KEY_DESCRIPTION = 'description';
     public const KEY_TYPE = 'type';
     public const KEY_CREDENTIALS_URL = 'credentialsUrl';
     public const KEY_AUTHENTICATION_METHOD = 'authenticationMethod';
@@ -39,6 +42,10 @@ class ProviderMetadata extends AbstractDataTransferObject
      * @var string The provider's display name.
      */
     protected string $name;
+    /**
+     * @var string|null The provider's description.
+     */
+    protected ?string $description;
     /**
      * @var ProviderTypeEnum The provider type.
      */
@@ -55,17 +62,20 @@ class ProviderMetadata extends AbstractDataTransferObject
      * Constructor.
      *
      * @since 0.1.0
+     * @since 1.2.0 Added optional $description parameter.
      *
      * @param string $id The provider's unique identifier.
      * @param string $name The provider's display name.
      * @param ProviderTypeEnum $type The provider type.
      * @param string|null $credentialsUrl The URL where users can get credentials.
      * @param RequestAuthenticationMethod|null $authenticationMethod The authentication method.
+     * @param string|null $description The provider's description.
      */
-    public function __construct(string $id, string $name, ProviderTypeEnum $type, ?string $credentialsUrl = null, ?RequestAuthenticationMethod $authenticationMethod = null)
+    public function __construct(string $id, string $name, ProviderTypeEnum $type, ?string $credentialsUrl = null, ?RequestAuthenticationMethod $authenticationMethod = null, ?string $description = null)
     {
         $this->id = $id;
         $this->name = $name;
+        $this->description = $description;
         $this->type = $type;
         $this->credentialsUrl = $credentialsUrl;
         $this->authenticationMethod = $authenticationMethod;
@@ -92,6 +102,17 @@ public function getName(): string
     {
         return $this->name;
     }
+    /**
+     * Gets the provider's description.
+     *
+     * @since 1.2.0
+     *
+     * @return string|null The provider description.
+     */
+    public function getDescription(): ?string
+    {
+        return $this->description;
+    }
     /**
      * Gets the provider type.
      *
@@ -129,30 +150,33 @@ public function getAuthenticationMethod(): ?RequestAuthenticationMethod
      * {@inheritDoc}
      *
      * @since 0.1.0
+     * @since 1.2.0 Added description to schema.
      */
     public static function getJsonSchema(): array
     {
-        return ['type' => 'object', 'properties' => [self::KEY_ID => ['type' => 'string', 'description' => 'The provider\'s unique identifier.'], self::KEY_NAME => ['type' => 'string', 'description' => 'The provider\'s display name.'], self::KEY_TYPE => ['type' => 'string', 'enum' => ProviderTypeEnum::getValues(), 'description' => 'The provider type (cloud, server, or client).'], self::KEY_CREDENTIALS_URL => ['type' => 'string', 'description' => 'The URL where users can get credentials.'], self::KEY_AUTHENTICATION_METHOD => ['type' => ['string', 'null'], 'enum' => array_merge(RequestAuthenticationMethod::getValues(), [null]), 'description' => 'The authentication method.']], 'required' => [self::KEY_ID, self::KEY_NAME, self::KEY_TYPE]];
+        return ['type' => 'object', 'properties' => [self::KEY_ID => ['type' => 'string', 'description' => 'The provider\'s unique identifier.'], self::KEY_NAME => ['type' => 'string', 'description' => 'The provider\'s display name.'], self::KEY_DESCRIPTION => ['type' => 'string', 'description' => 'The provider\'s description.'], self::KEY_TYPE => ['type' => 'string', 'enum' => ProviderTypeEnum::getValues(), 'description' => 'The provider type (cloud, server, or client).'], self::KEY_CREDENTIALS_URL => ['type' => 'string', 'description' => 'The URL where users can get credentials.'], self::KEY_AUTHENTICATION_METHOD => ['type' => ['string', 'null'], 'enum' => array_merge(RequestAuthenticationMethod::getValues(), [null]), 'description' => 'The authentication method.']], 'required' => [self::KEY_ID, self::KEY_NAME, self::KEY_TYPE]];
     }
     /**
      * {@inheritDoc}
      *
      * @since 0.1.0
+     * @since 1.2.0 Added description to output.
      *
      * @return ProviderMetadataArrayShape
      */
     public function toArray(): array
     {
-        return [self::KEY_ID => $this->id, self::KEY_NAME => $this->name, self::KEY_TYPE => $this->type->value, self::KEY_CREDENTIALS_URL => $this->credentialsUrl, self::KEY_AUTHENTICATION_METHOD => $this->authenticationMethod ? $this->authenticationMethod->value : null];
+        return [self::KEY_ID => $this->id, self::KEY_NAME => $this->name, self::KEY_DESCRIPTION => $this->description, self::KEY_TYPE => $this->type->value, self::KEY_CREDENTIALS_URL => $this->credentialsUrl, self::KEY_AUTHENTICATION_METHOD => $this->authenticationMethod ? $this->authenticationMethod->value : null];
     }
     /**
      * {@inheritDoc}
      *
      * @since 0.1.0
+     * @since 1.2.0 Added description support.
      */
     public static function fromArray(array $array): self
     {
         static::validateFromArrayData($array, [self::KEY_ID, self::KEY_NAME, self::KEY_TYPE]);
-        return new self($array[self::KEY_ID], $array[self::KEY_NAME], ProviderTypeEnum::from($array[self::KEY_TYPE]), $array[self::KEY_CREDENTIALS_URL] ?? null, isset($array[self::KEY_AUTHENTICATION_METHOD]) ? RequestAuthenticationMethod::from($array[self::KEY_AUTHENTICATION_METHOD]) : null);
+        return new self($array[self::KEY_ID], $array[self::KEY_NAME], ProviderTypeEnum::from($array[self::KEY_TYPE]), $array[self::KEY_CREDENTIALS_URL] ?? null, isset($array[self::KEY_AUTHENTICATION_METHOD]) ? RequestAuthenticationMethod::from($array[self::KEY_AUTHENTICATION_METHOD]) : null, $array[self::KEY_DESCRIPTION] ?? null);
     }
 }

src/wp-includes/php-ai-client/src/Providers/Http/Enums/RequestAuthenticationMethod.php

@@ -27,6 +27,8 @@ class RequestAuthenticationMethod extends AbstractEnum
      * @since 0.4.0
      *
      * @return class-string<RequestAuthenticationInterface&WithArrayTransformationInterface> The implementation class.
+     *
+     * @phpstan-ignore missingType.generics
      */
     public function getImplementationClass(): string
     {

src/wp-includes/php-ai-client/src/Providers/Http/HttpTransporter.php

@@ -260,6 +260,7 @@ private function convertFromPsr7Response(ResponseInterface $psr7Response): Respo
         return new Response(
             $psr7Response->getStatusCode(),
             $psr7Response->getHeaders(),
+            // @phpstan-ignore-line
             $body === '' ? null : $body
         );
     }

src/wp-includes/php-ai-client/src/Providers/Models/DTO/SupportedOption.php

@@ -4,6 +4,7 @@
 namespace WordPress\AiClient\Providers\Models\DTO;
 
 use WordPress\AiClient\Common\AbstractDataTransferObject;
+use WordPress\AiClient\Common\AbstractEnum;
 use WordPress\AiClient\Common\Exception\InvalidArgumentException;
 use WordPress\AiClient\Providers\Models\Enums\OptionEnum;
 /**
@@ -78,19 +79,62 @@ public function isSupportedValue($value): bool
         }
         // If the value is an array, consider it a set (i.e. order doesn't matter).
         if (is_array($value)) {
-            sort($value);
+            $normalizedValue = self::normalizeArrayForComparison($value);
             foreach ($this->supportedValues as $supportedValue) {
                 if (!is_array($supportedValue)) {
                     continue;
                 }
-                sort($supportedValue);
-                if ($value === $supportedValue) {
+                $normalizedSupported = self::normalizeArrayForComparison($supportedValue);
+                if ($normalizedValue === $normalizedSupported) {
                     return \true;
                 }
             }
             return \false;
         }
-        return in_array($value, $this->supportedValues, \true);
+        $normalizedValue = self::normalizeValue($value);
+        foreach ($this->supportedValues as $supportedValue) {
+            if (self::normalizeValue($supportedValue) === $normalizedValue) {
+                return \true;
+            }
+        }
+        return \false;
+    }
+    /**
+     * Normalizes an AbstractEnum instance to its string value.
+     *
+     * This ensures comparisons work correctly even after deserialization
+     * (e.g. Redis/Memcached object cache), where AbstractEnum singletons
+     * are reconstructed as separate instances.
+     *
+     * @since 1.2.1
+     *
+     * @param mixed $value The value to normalize.
+     * @return mixed The normalized value.
+     */
+    private static function normalizeValue($value)
+    {
+        if ($value instanceof AbstractEnum) {
+            return $value->value;
+        }
+        return $value;
+    }
+    /**
+     * Normalizes and sorts an array for comparison.
+     *
+     * Maps each element through normalizeValue() and sorts the result,
+     * ensuring consistent comparison regardless of element order or
+     * AbstractEnum instance identity.
+     *
+     * @since 1.2.1
+     *
+     * @param array<mixed> $items The array to normalize.
+     * @return array<mixed> The normalized, sorted array.
+     */
+    private static function normalizeArrayForComparison(array $items): array
+    {
+        $normalized = array_map([self::class, 'normalizeValue'], $items);
+        sort($normalized);
+        return $normalized;
     }
     /**
      * Gets the supported values for this option.