fix remove bookmark

Author: ChaosKid42

CHANGES.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 12.0.1 (Unreleased)
+
+- #3822: Bookmark removal not working
+
 ## 12.0.0 (2025-08-28)
 
 - #3581: Don't unnecessarily regenerate pot and po files

src/headless/plugins/bookmarks/collection.js

@@ -38,7 +38,9 @@ class Bookmarks extends Collection {
         this.on(
             'remove',
             /** @param {Bookmark} bookmark */
-            (_, bookmark) => this.sendBookmarkStanza(bookmark),
+            (bookmark, _) =>
+                this.removeBookmark(bookmark)
+                    .catch((e) => log.fatal(e)),
             this
         );
 
@@ -182,6 +184,28 @@ class Bookmarks extends Collection {
         });
     }
 
+    /**
+     * @param {Bookmark} bookmark
+     * @returns {Promise<void|Element>}
+     */
+    async removeBookmark(bookmark) {
+        const bare_jid = _converse.session.get('bare_jid');
+        const node = (await api.disco.supports(`${Strophe.NS.BOOKMARKS2}#compat`, bare_jid))
+            ? Strophe.NS.BOOKMARKS2
+            : Strophe.NS.BOOKMARKS;
+        const stanza = stx`
+            <iq xmlns="jabber:client"
+                from="${bare_jid}"
+                type="set">
+                <pubsub xmlns="${Strophe.NS.PUBSUB}">
+                    <retract node="${node}">
+                        <item id="${bookmark.get('jid')}"/>
+                    </retract>
+                </pubsub>
+            </iq>`;
+       await api.sendIQ(stanza);
+    }
+
     /**
      * @param {Element} iq
      */