From 5687f247d327e270b81b840845417258bf7f9760 Mon Sep 17 00:00:00 2001
From: Scott Idem <stidem@gmail.com>
Date: Wed, 4 Mar 2026 14:38:02 -0500
Subject: [PATCH] fix(idaa, leads): add each-block keys; gate clipboard buttons
 to manager_access

- Add keyed {#each} to recovery meeting list and exhibit tracking list
  to satisfy Svelte's each-block-key lint rule and ensure correct DOM
  reconciliation on list updates
- Gate Zoom/Jitsi copy-to-clipboard buttons behind manager_access in
  both the recovery meeting list view and single meeting detail view

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../[exhibit_id]/ae_comp__exhibit_tracking_obj_li.svelte    | 2 +-
 .../ae_idaa_comp__event_obj_id_view.svelte                  | 4 ++++
 .../recovery_meetings/ae_idaa_comp__event_obj_li.svelte     | 6 +++++-
 3 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/src/routes/events/[event_id]/(leads)/leads/exhibit/[exhibit_id]/ae_comp__exhibit_tracking_obj_li.svelte b/src/routes/events/[event_id]/(leads)/leads/exhibit/[exhibit_id]/ae_comp__exhibit_tracking_obj_li.svelte
index 43979a32..910182d5 100644
--- a/src/routes/events/[event_id]/(leads)/leads/exhibit/[exhibit_id]/ae_comp__exhibit_tracking_obj_li.svelte
+++ b/src/routes/events/[event_id]/(leads)/leads/exhibit/[exhibit_id]/ae_comp__exhibit_tracking_obj_li.svelte
@@ -82,7 +82,7 @@
             </div>
 
             <div class="grid grid-cols-1 gap-4">
-                {#each lq__event_exhibit_tracking_obj_li as event_tracking_obj}
+                {#each lq__event_exhibit_tracking_obj_li as event_tracking_obj (event_tracking_obj.event_exhibit_tracking_id)}
                     <a
                         href={`/events/${page.params.event_id}/leads/exhibit/${event_tracking_obj.event_exhibit_id}/lead/${event_tracking_obj.event_exhibit_tracking_id}`}
                         class="card card-hover p-4 variant-filled-surface border-l-4 border-primary-500 flex flex-col md:flex-row gap-4 items-start md:items-center"
diff --git a/src/routes/idaa/(idaa)/recovery_meetings/ae_idaa_comp__event_obj_id_view.svelte b/src/routes/idaa/(idaa)/recovery_meetings/ae_idaa_comp__event_obj_id_view.svelte
index 06cf8199..d3d95849 100644
--- a/src/routes/idaa/(idaa)/recovery_meetings/ae_idaa_comp__event_obj_id_view.svelte
+++ b/src/routes/idaa/(idaa)/recovery_meetings/ae_idaa_comp__event_obj_id_view.svelte
@@ -194,6 +194,7 @@
                         </a>
                     </span>
 
+                    {#if $ae_loc.manager_access}
                     <MyClipboard
                         value={$lq__event_obj?.attend_json?.zoom?.full_url}
                         btn_text="Copy Zoom Link"
@@ -207,6 +208,7 @@
                         transition
                         "
                     ></MyClipboard>
+                    {/if}
                 </div>
             {:else if $lq__event_obj?.attend_json?.jitsi}
                 <div
@@ -243,6 +245,7 @@
                         </a>
                     </span>
 
+                    {#if $ae_loc.manager_access}
                     <MyClipboard
                         value={$lq__event_obj?.attend_json?.jitsi?.full_url}
                         btn_text="Copy Jitsi Link"
@@ -256,6 +259,7 @@
                         transition
                         "
                     ></MyClipboard>
+                    {/if}
                 </div>
             {:else}
                 <div
diff --git a/src/routes/idaa/(idaa)/recovery_meetings/ae_idaa_comp__event_obj_li.svelte b/src/routes/idaa/(idaa)/recovery_meetings/ae_idaa_comp__event_obj_li.svelte
index 83221fa5..d0c93c62 100644
--- a/src/routes/idaa/(idaa)/recovery_meetings/ae_idaa_comp__event_obj_li.svelte
+++ b/src/routes/idaa/(idaa)/recovery_meetings/ae_idaa_comp__event_obj_li.svelte
@@ -114,7 +114,7 @@
                 </span>
             </h2>
 
-            {#each visible_event_obj_li as idaa_event_obj, index}
+            {#each visible_event_obj_li as idaa_event_obj, index (idaa_event_obj?.event_id ?? index)}
                 {#if idaa_event_obj}
                     <!-- NOTE: There is extra logic here to hide items that are hidden/disabled for public users, but still show them to trusted/admin users. This is because the LiveQuery returns all items matching the query, and we need to filter on the client side based on the user's access level. The `visible_event_obj_li` derived store handles this filtering logic, so we can safely render based on that list. -->
                     <div
@@ -281,6 +281,7 @@
                                     </a>
                                 </span>
 
+                                {#if $ae_loc.manager_access}
                                 <MyClipboard
                                     value={idaa_event_obj?.attend_json?.zoom
                                         ?.full_url}
@@ -295,6 +296,7 @@
                         transition
                         "
                                 ></MyClipboard>
+                                {/if}
                             </div>
 
                             <div
@@ -330,6 +332,7 @@
                                     </a>
                                 </span>
 
+                                {#if $ae_loc.manager_access}
                                 <MyClipboard
                                     value={idaa_event_obj?.attend_json?.jitsi
                                         ?.full_url}
@@ -344,6 +347,7 @@
                         transition
                         "
                                 ></MyClipboard>
+                                {/if}
                             </div>
 
                             <div