Merge pull request #99 from krishnan05/fix/analytics-lint-empty-catch

fix: avoid empty catch blocks in analytics

Author: kpj2006

src/social-share-button.js

@@ -705,6 +705,18 @@ class SocialShareButton {
       : this.options.container;
   }
 
+  /**
+   * Logs analytics warnings only when debug mode is enabled.
+   * @param {string} message - Description of the failed analytics path.
+   * @param {Error} err - The caught error instance.
+   */
+  _debugWarn(message, err) {
+    // _debugWarn: emit analytics warnings only in debug mode for visibility.
+    if (!this.options.debug) return;
+    // eslint-disable-next-line no-console
+    console.warn("[SocialShareButton Analytics]", message, err);
+  }
+
   /**
    * Emits an analytics event through all configured delivery paths.
    *
@@ -763,14 +775,18 @@ class SocialShareButton {
         });
         const el = this._getContainer();
         (el || document).dispatchEvent(domEvent);
-      } catch (_) {}
+      } catch (err) {
+        this._debugWarn("DOM event dispatch failed", err);
+      }
     }
 
     // Path 2 — onAnalytics callback (direct, single-consumer hook)
     if (typeof this.options.onAnalytics === "function") {
       try {
         this.options.onAnalytics(payload);
-      } catch (_) {}
+      } catch (err) {
+        this._debugWarn("onAnalytics callback failed", err);
+      }
     }
 
     // Path 3 — plugin / adapter registry (supports multiple simultaneous consumers)
@@ -779,7 +795,9 @@ class SocialShareButton {
         if (plugin && typeof plugin.track === "function") {
           try {
             plugin.track(payload);
-          } catch (_) {}
+          } catch (err) {
+            this._debugWarn("plugin.track() failed", err);
+          }
         }
       }
     }