Fix tests and format

Signed-off-by: Kristina Fefelova <kristin.fefelova@gmail.com>

Author: kristina-fefelova

plugins/chunter-resources/src/components/BaseChatScroller.svelte

@@ -22,6 +22,7 @@
   export let loadingOverlay: boolean = false
   export let onScroll: () => void = () => {}
   export let onResize: () => void = () => {}
+  export let key: string
 </script>
 
 {#if loadingOverlay}
@@ -41,7 +42,9 @@
   {onScroll}
   {onResize}
 >
-  <slot />
+  {#key key}
+    <slot />
+  {/key}
 </Scroller>
 
 <style lang="scss">

plugins/chunter-resources/src/components/ReverseChannelScrollView.svelte

@@ -611,6 +611,9 @@
       editLastMessage()
     }
   }
+  function getKey (messages: ActivityMessage[]): string {
+    return `${messages.length}-${Math.max(...messages.map((m) => m.modifiedOn))}`
+  }
 </script>
 
 <div class="flex-col relative" class:h-full={fullHeight}>
@@ -627,6 +630,7 @@
     loadingOverlay={$isLoadingStore || !isScrollInitialized}
     onScroll={handleScroll}
     onResize={handleResize}
+    key={getKey(messages)}
   >
     {#if showBlankView}
       <BlankView

plugins/inbox-resources/src/components/InboxApplication.svelte

@@ -168,7 +168,7 @@
 
   function isChunterChannel (_class: Ref<Class<Doc>>, urlObjectClass?: Ref<Class<Doc>>): boolean {
     const isActivityMessageContext = hierarchy.isDerived(_class, activity.class.ActivityMessage)
-    const chunterClass = isActivityMessageContext ? urlObjectClass ?? _class : _class
+    const chunterClass = isActivityMessageContext ? (urlObjectClass ?? _class) : _class
     return hierarchy.isDerived(chunterClass, chunter.class.ChunterSpace)
   }
 

plugins/inbox-resources/src/location.ts

@@ -184,10 +184,10 @@ export async function selectInboxContext (
       : undefined
     const reactedTo = await client.findOne(activity.class.ActivityMessage, { _id: notification.attachedTo })
     const isThread = reactedTo != null && hierarchy.isDerived(reactedTo._class, chunter.class.ThreadMessage)
-    const channelId = isThread ? (reactedTo as ThreadMessage)?.objectId : reactedTo?.attachedTo ?? objectId
+    const channelId = isThread ? (reactedTo as ThreadMessage)?.objectId : (reactedTo?.attachedTo ?? objectId)
     const channelClass = isThread
       ? (reactedTo as ThreadMessage)?.objectClass
-      : reactedTo?.attachedToClass ?? objectClass
+      : (reactedTo?.attachedToClass ?? objectClass)
 
     void navigateToInboxDoc(
       context._id,

plugins/notification-resources/src/utils.ts

@@ -617,10 +617,10 @@ export async function selectInboxContext (
     const thread = loc.path[4] === objectId ? objectId : undefined
     const reactedTo = await client.findOne(activity.class.ActivityMessage, { _id: notification.attachedTo })
     const isThread = reactedTo != null && hierarchy.isDerived(reactedTo._class, chunter.class.ThreadMessage)
-    const channelId = isThread ? (reactedTo as ThreadMessage)?.objectId : reactedTo?.attachedTo ?? objectId
+    const channelId = isThread ? (reactedTo as ThreadMessage)?.objectId : (reactedTo?.attachedTo ?? objectId)
     const channelClass = isThread
       ? (reactedTo as ThreadMessage)?.objectClass
-      : reactedTo?.attachedToClass ?? objectClass
+      : (reactedTo?.attachedToClass ?? objectClass)
 
     void navigateToInboxDoc(
       linkProviders,