fix: Mapping identity overrides with multiple features fails (#143)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>

Author: KrasilnikovKB

src/Utils/Mappers.php

@@ -149,7 +149,7 @@ private static function _mapIdentityOverridesToSegments($identityOverrides)
         foreach ($identityOverrides as $identityOverride) {
             $identityFeatures = ((array) $identityOverride->identity_features) ?? [];
             // Sort by feature name to ensure consistent serialization order for $overridesKey
-            uksort($identityFeatures, fn ($a, $b) => strcasecmp($a->feature->name, $b->feature->name));
+            usort($identityFeatures, fn ($a, $b) => strcasecmp($a->feature->name, $b->feature->name));
             if (empty($identityFeatures)) {
                 continue;
             }

tests/Utils/MappersTest.php

@@ -123,4 +123,59 @@ public function testMapEnvironmentDocumentToContextProducesEvaluationContext():
         $this->assertEquals(25.0, $mvFeatureWithoutIds->variants[2]->weight);
         $this->assertEquals(2, $mvFeatureWithoutIds->variants[2]->priority); // Third
     }
+
+    public function testMapEnvironmentDocumentToContextMapsIdentityOverridesWithMultipleFeatures(): void
+    {
+        // Given
+        $environmentDocument = json_decode(file_get_contents(__DIR__ . '/../Data/environment.json'));
+        $environmentDocument->identity_overrides = [
+            (object) [
+                'identifier' => 'identity-1',
+                'identity_features' => [
+                    self::_identityFeatureState(2, 'zeta_feature', true, 'zeta-value'),
+                    self::_identityFeatureState(1, 'alpha_feature', false, 'alpha-value'),
+                ],
+            ],
+            (object) [
+                'identifier' => 'identity-2',
+                'identity_features' => [
+                    self::_identityFeatureState(1, 'alpha_feature', false, 'alpha-value'),
+                    self::_identityFeatureState(2, 'zeta_feature', true, 'zeta-value'),
+                ],
+            ],
+        ];
+
+        // When
+        $context = Mappers::mapEnvironmentDocumentToContext($environmentDocument);
+
+        // Then
+        $identityOverrideSegments = array_values(array_filter(
+            $context->segments,
+            fn ($segment) => $segment->metadata['source'] === 'identity_override',
+        ));
+        $this->assertCount(1, $identityOverrideSegments);
+
+        $segment = $identityOverrideSegments[0];
+        $this->assertEquals(['identity-1', 'identity-2'], $segment->rules[0]->conditions[0]->value);
+        $this->assertCount(2, $segment->overrides);
+
+        $this->assertEquals('alpha_feature', $segment->overrides[0]->name);
+        $this->assertFalse($segment->overrides[0]->enabled);
+        $this->assertEquals('alpha-value', $segment->overrides[0]->value);
+        $this->assertEquals(['id' => 1], $segment->overrides[0]->metadata);
+
+        $this->assertEquals('zeta_feature', $segment->overrides[1]->name);
+        $this->assertTrue($segment->overrides[1]->enabled);
+        $this->assertEquals('zeta-value', $segment->overrides[1]->value);
+        $this->assertEquals(['id' => 2], $segment->overrides[1]->metadata);
+    }
+
+    private static function _identityFeatureState(int $featureId, string $featureName, bool $enabled, string $value): object
+    {
+        return (object) [
+            'feature' => (object) ['id' => $featureId, 'name' => $featureName, 'type' => 'STANDARD'],
+            'feature_state_value' => $value,
+            'enabled' => $enabled,
+        ];
+    }
 }