From 92a75b0cd3c0c7efcb4c3da0fa0cba04612a0256 Mon Sep 17 00:00:00 2001
From: Daniel Dietzler <36593685+danieldietzler@users.noreply.github.com>
Date: Wed, 10 Jun 2026 16:32:29 +0200
Subject: [PATCH] fix(web): person that is in the same asset multiple times
 (#28971)

---
 web/src/lib/stores/face.svelte.ts | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/web/src/lib/stores/face.svelte.ts b/web/src/lib/stores/face.svelte.ts
index e7ab11bf3f..dc86a2821f 100644
--- a/web/src/lib/stores/face.svelte.ts
+++ b/web/src/lib/stores/face.svelte.ts
@@ -1,5 +1,5 @@
 import type { AssetFaceResponseDto, PersonResponseDto } from '@immich/sdk';
-import { SvelteMap, SvelteSet } from 'svelte/reactivity';
+import { SvelteMap } from 'svelte/reactivity';
 import { assetCacheManager } from '$lib/managers/AssetCacheManager.svelte';
 import type { Faces } from '$lib/managers/asset-viewer-manager.svelte';
 import { CancellableTask } from '$lib/utils/cancellable-task';
@@ -24,15 +24,15 @@ class FaceManager {
   });
 
   readonly people = $derived.by(() => {
-    const people = new SvelteSet<PersonResponseDto>();
+    const people = new SvelteMap<string, PersonResponseDto>();
 
     for (const face of this.data) {
       if (face.person) {
-        people.add(face.person);
+        people.set(face.person.id, face.person);
       }
     }
 
-    return people;
+    return people.values();
   });
 
   readonly facesByPersonId = $derived.by(() => {