From 6b6605ded0aac894363e8c5b34c18da5cdb0a5bb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alejandro=20G=C3=B3mez?= <alejandrogomez@bitrefill.com>
Date: Sun, 11 Jan 2026 22:06:37 +0100
Subject: [PATCH] fix: pass proper ProtocolIdentifier object when opening group
 chats

GroupLink was passing a string identifier instead of a properly structured
ProtocolIdentifier object, causing chat window to fail opening.

Fixed to pass:
{
  protocol: "nip-29",
  identifier: {
    type: "group",
    value: groupId,
    relays: [relayUrl]
  }
}

This matches the expected ChatViewer props interface and allows the
NIP-29 adapter to properly resolve the conversation with the group relay.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
---
 src/components/nostr/GroupLink.tsx | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/components/nostr/GroupLink.tsx b/src/components/nostr/GroupLink.tsx
index 5cddef7..810b962 100644
--- a/src/components/nostr/GroupLink.tsx
+++ b/src/components/nostr/GroupLink.tsx
@@ -57,9 +57,15 @@ export function GroupLink({
       : undefined;
 
   const handleClick = () => {
-    // Open chat with NIP-29 format: relay'group-id
-    const identifier = `${relayUrl}'${groupId}`;
-    addWindow("chat", { protocol: "nip-29", identifier });
+    // Open chat with properly structured ProtocolIdentifier
+    addWindow("chat", {
+      protocol: "nip-29",
+      identifier: {
+        type: "group",
+        value: groupId,
+        relays: [relayUrl],
+      },
+    });
   };
 
   const displayName = formatGroupIdForDisplay(groupName);