@unhingged/vizu-core 0.1.16 → 0.1.18

package/dist/index.js

@@ -86,6 +86,9 @@ function renderAttachmentsHtml(attachments) {
 function migrateComment(raw) {
   if (!raw || typeof raw !== "object") return raw;
   const next = { ...raw };
+  if (next.id == null && next._id != null) {
+    next.id = next._id;
+  }
   if (!Array.isArray(next.fingerprints) || next.fingerprints.length === 0) {
     if (next.fingerprint) {
       next.fingerprints = [next.fingerprint];
@@ -2298,11 +2301,22 @@ var Sidebar = class {
     const conf = this.lastConfidence;
     const orphans = comments.filter((c) => conf.get(c.id) === "orphaned");
     const anchored = comments.filter((c) => conf.get(c.id) !== "orphaned");
-    const visible = this.filter === "open" ? anchored.filter((c) => (c.status ?? "open") === "open") : anchored;
-    const visibleOrphans = this.filter === "open" ? [] : orphans;
+    const statusOf = (c) => c.status ?? "open";
+    const visible = this.filter === "open" ? anchored.filter((c) => statusOf(c) === "open") : this.filter === "resolved" ? anchored.filter((c) => statusOf(c) === "resolved") : anchored;
+    const visibleOrphans = this.filter === "all" ? orphans : [];
     const counts = countByStatus(comments);
     if (visible.length === 0 && visibleOrphans.length === 0) {
-      const emptyCopy = this.filter === "open" && comments.length > 0 ? `<strong>Nothing open.</strong>${counts.resolved + counts.wontfix} closed comment${counts.resolved + counts.wontfix === 1 ? "" : "s"} hidden. Switch to <em>All</em> to see them.` : "<strong>No comments yet</strong>Click any element on the page to leave one.";
+      let emptyCopy;
+      if (comments.length === 0) {
+        emptyCopy = "<strong>No comments yet</strong>Click any element on the page to leave one.";
+      } else if (this.filter === "open") {
+        const closed = counts.resolved + counts.wontfix;
+        emptyCopy = `<strong>Nothing open.</strong>${closed} closed comment${closed === 1 ? "" : "s"} hidden. Switch to <em>All</em> to see them.`;
+      } else if (this.filter === "resolved") {
+        emptyCopy = `<strong>Nothing resolved.</strong>${counts.open} still open. Switch to <em>Open</em> or <em>All</em> to see more.`;
+      } else {
+        emptyCopy = "<strong>No comments yet</strong>Click any element on the page to leave one.";
+      }
       this.el.innerHTML = `
         ${this.headerHtml(comments.length, counts)}
         <div class="vz-sidebar-body">
@@ -2398,6 +2412,9 @@ var Sidebar = class {
         <button class="vz-sidebar-filter ${this.filter === "open" ? "is-active" : ""}" data-vz="filter" data-filter="open" role="tab" aria-selected="${this.filter === "open"}">
           Open <span class="vz-sidebar-filter-count">${counts.open}</span>
         </button>
+        <button class="vz-sidebar-filter ${this.filter === "resolved" ? "is-active" : ""}" data-vz="filter" data-filter="resolved" role="tab" aria-selected="${this.filter === "resolved"}">
+          Resolved <span class="vz-sidebar-filter-count">${counts.resolved}</span>
+        </button>
         <button class="vz-sidebar-filter ${this.filter === "all" ? "is-active" : ""}" data-vz="filter" data-filter="all" role="tab" aria-selected="${this.filter === "all"}">
           All <span class="vz-sidebar-filter-count">${total}</span>
         </button>
@@ -4281,10 +4298,26 @@ var Vizu = class {
     this.refreshUi();
   }
   async removeComment(id) {
-    const comment = this.comments.find((c) => c.id === id);
-    if (!comment) return;
+    const idx = this.comments.findIndex((c) => c.id === id);
+    if (idx === -1) return;
+    const comment = this.comments[idx];
     this.comments = this.comments.filter((c) => c.id !== id);
-    await this.storage.removeComment(this.opts.namespace, id);
+    try {
+      await this.storage.removeComment(this.opts.namespace, id);
+    } catch (err) {
+      this.comments.splice(idx, 0, comment);
+      const status = err?.status;
+      const code = err?.code ?? "";
+      const msg = status === 403 || code === "forbidden" ? "Delete failed \u2014 you don't have permission." : status === 401 || code === "auth_required" ? "Delete failed \u2014 please sign in again." : "Delete failed. Please try again.";
+      this.pill?.toast(msg);
+      if (this.popover?.isOpen()) {
+        const anchors = this.popover.getAnchors();
+        if (anchors[0]) this.popover.update(this.commentsForElement(anchors[0]));
+      }
+      this.refreshUi();
+      if (typeof console !== "undefined") console.warn("[vizu] removeComment failed", err);
+      return;
+    }
     this.bus.emit("comment:removed", { id, comment });
     if (this.popover?.isOpen()) {
       const anchors = this.popover.getAnchors();