fix: handle 'exists' status in extension and improve html detection

Author: frostmute

extension/popup.js

@@ -90,13 +90,12 @@ async function sendToBridge(endpoint) {
     } else if (data.status === 'success') {
       status.textContent = "Done. Don't expect a medal.";
       btn.classList.remove('pulse');
+    } else if (data.status === 'exists') {
+      status.textContent = "Already exists. Stop hoarding.";
+      btn.classList.remove('pulse');
     } else {
-      let errorMessage = 'Unknown error';
-      if (data.detail) {
-        errorMessage = typeof data.detail === 'string' ? data.detail : JSON.stringify(data.detail);
-      } else if (data.message) {
-        errorMessage = data.message;
-      }
+      let errorMessage = data.detail || data.message || data.error || `HTTP ${response.status}`;
+      if (typeof errorMessage !== 'string') errorMessage = JSON.stringify(errorMessage);
       status.textContent = "Rejected: " + errorMessage;
       btn.classList.remove('pulse');
     }

src/hermes_clipper/main.py

@@ -347,12 +347,10 @@ def check_duplicate(url, vault):
     return None
 
 def clip(url, title, content, folder="Clippings", tags=None, metadata=None, mode="unique", caveman=False, **kwargs):
-    # Robust HTML detection
+    # Aggressive HTML detection
     is_html = False
     stripped_content = content.strip()
-    if stripped_content.startswith("<"):
-        is_html = True
-    elif "</html>" in content.lower() or "<body" in content.lower() or "<div" in content.lower():
+    if stripped_content.startswith("<") or "<html" in content.lower() or "<body" in content.lower() or "<div" in content.lower() or "<p" in content.lower() or "<script" in content.lower():
         is_html = True
 
     if is_html:
@@ -368,6 +366,8 @@ def clip(url, title, content, folder="Clippings", tags=None, metadata=None, mode
         print_error("Vault path not set. Run 'hermes-clip setup'.")
         sys.exit(1)
 
+    print_header(f"Clipping: {title}")
+
     if mode == "unique":
         if dup := check_duplicate(url, vault):
             print(json.dumps({"status": "exists", "path": dup}))