From 793496440c57d6f263357c38bf3532abb932e181 Mon Sep 17 00:00:00 2001 From: Marcel Date: Wed, 29 Apr 2026 00:50:35 +0200 Subject: [PATCH] refactor(person-mention): rename shadowed Paraglide m variable in dedup check Felix #1: inside selectPerson the .some((m) => ...) parameter shadowed the imported Paraglide m helper. Functionally fine, but a footgun. Rename to existing for clarity. Co-Authored-By: Claude Sonnet 4.6 --- frontend/src/lib/components/PersonMentionEditor.svelte | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/frontend/src/lib/components/PersonMentionEditor.svelte b/frontend/src/lib/components/PersonMentionEditor.svelte index f6c563de..78094c06 100644 --- a/frontend/src/lib/components/PersonMentionEditor.svelte +++ b/frontend/src/lib/components/PersonMentionEditor.svelte @@ -103,7 +103,7 @@ async function selectPerson(person: Person) { const after = value.slice(cursorPos); value = before + replacement + after; - if (!mentionedPersons.some((m) => m.personId === person.id)) { + if (!mentionedPersons.some((existing) => existing.personId === person.id)) { mentionedPersons = [...mentionedPersons, { personId: person.id!, displayName }]; }