buddy-workbench 0.1.91 → 0.1.92

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "buddy-workbench",
-  "version": "0.1.91",
+  "version": "0.1.92",
   "private": false,
   "type": "module",
   "bin": {

package/server/services/clipboard-history.js

@@ -16,14 +16,29 @@ let lastValue = '';
 let lastImageHash = '';
 let suppressImageCaptureUntil = 0;
 let mozjpegWasmModule = null;
+const clipboardErrorCounts = new Map();
 const clipboardErrorLogTimes = new Map();
 
-function logClipboardError(stage, error) {
+function markClipboardSuccess(...stages) {
+  for (const stage of stages) {
+    clipboardErrorCounts.delete(stage);
+  }
+}
+
+function logClipboardError(stage, error, threshold = 3) {
   // Ignore benign failures due to timeout or process termination (e.g., system sleep / screen lock / display off)
   if (error?.killed || error?.timedOut || error?.signal === 'SIGTERM' || error?.code === 'ETIMEDOUT') {
     return;
   }
 
+  const count = (clipboardErrorCounts.get(stage) || 0) + 1;
+  clipboardErrorCounts.set(stage, count);
+
+  // Transient filter: only log if failures happen consecutively past threshold
+  if (count < threshold) {
+    return;
+  }
+
   // Clipboard polling runs continuously. Avoid filling Error Log with the
   // same native-tool failure every two seconds while retaining diagnostics.
   const now = Date.now();
@@ -33,7 +48,7 @@ function logClipboardError(stage, error) {
   const details = String(error?.stderr || error?.stdout || error?.stack || error?.message || error || '').trim();
   addErrorRecord({
     source: 'Clipboard Monitor',
-    message: `Failed during ${stage}.`,
+    message: `Failed during ${stage} (consecutive failures: ${count}).`,
     details: details || 'The native clipboard operation returned an unknown error.'
   });
 }
@@ -85,6 +100,7 @@ async function getMacClipboardImageData() {
   try {
     const { stdout } = await execFileAsync('osascript', ['-l', 'JavaScript', '-e', jsaScript], { timeout: 3000 });
     const str = stdout.trim();
+    markClipboardSuccess('macOS image clipboard read (osascript)');
     if (!str) return null;
     const firstColon = str.indexOf(':');
     const secondColon = str.indexOf(':', firstColon + 1);
@@ -96,6 +112,11 @@ async function getMacClipboardImageData() {
     return { width, height, rgbaBuf: Buffer.from(base64, 'base64') };
   } catch (error) {
     logClipboardError('macOS image clipboard read (osascript)', error);
+    // If osascript failed with an error, attempt the Swift fallback for image capture
+    const rawBuffer = await getMacClipboardImageBuffer();
+    if (rawBuffer && rawBuffer.length > 0) {
+      return { rawBuffer };
+    }
     return null;
   }
 }
@@ -117,7 +138,10 @@ if let data = pb.data(forType: .png) {
 `;
   try {
     const { stdout } = await execFileAsync('swift', ['-e', swiftCode], { encoding: 'buffer', timeout: 3000 });
-    if (stdout && stdout.length > 0) return stdout;
+    if (stdout && stdout.length > 0) {
+      markClipboardSuccess('macOS image clipboard read (Swift)');
+      return stdout;
+    }
   } catch (error) {
     logClipboardError('macOS image clipboard read (Swift)', error);
     try {
@@ -141,7 +165,10 @@ if let data = pb.data(forType: .png) {
       `;
       const { stdout } = await execFileAsync('osascript', ['-l', 'JavaScript', '-e', jsaScript], { timeout: 3000 });
       const str = stdout.trim();
-      if (str) return Buffer.from(str, 'base64');
+      if (str) {
+        markClipboardSuccess('macOS image clipboard fallback read (osascript)');
+        return Buffer.from(str, 'base64');
+      }
     } catch (fallbackError) {
       logClipboardError('macOS image clipboard fallback read (osascript)', fallbackError);
     }
@@ -334,7 +361,12 @@ export async function captureClipboard() {
   if (process.platform === 'darwin' && settings.clipboardImageEnabled !== false && Date.now() >= suppressImageCaptureUntil) {
     try {
       const imageData = await getMacClipboardImageData();
-      if (imageData && imageData.rgbaBuf.length > 0) {
+      if (!imageData) {
+        // No image on clipboard (or transient capture failed and will retry next tick)
+        return;
+      }
+
+      if (imageData.rgbaBuf && imageData.rgbaBuf.length > 0) {
         const hash = createHash('md5').update(imageData.rgbaBuf).digest('hex');
         if (hash !== lastImageHash) {
           let isDuplicate = false;
@@ -356,12 +388,14 @@ export async function captureClipboard() {
             try {
               const mozjpegBuf = await compressMozjpeg(imageData.rgbaBuf, imageData.width, imageData.height, 75);
               saveClipboardImage(mozjpegBuf, 'jpg', hash);
+              markClipboardSuccess('image compression or save (WASM/JPEG)', 'image save');
             } catch (error) {
               logClipboardError('image compression or save (WASM/JPEG)', error);
               const rawBuf = await getMacClipboardImageBuffer();
               if (rawBuf) {
                 try {
                   saveClipboardImage(rawBuf, 'png', hash);
+                  markClipboardSuccess('raw image save after compression fallback', 'image save');
                 } catch (fallbackError) {
                   logClipboardError('raw image save after compression fallback', fallbackError);
                 }
@@ -369,32 +403,30 @@ export async function captureClipboard() {
             }
           }
         }
-      } else {
-        const imageBuffer = await getMacClipboardImageBuffer();
-        if (imageBuffer && imageBuffer.length > 0) {
-          const hash = createHash('md5').update(imageBuffer).digest('hex');
-          if (hash !== lastImageHash) {
-            let isDuplicate = false;
-            const dates = clipboardDates();
-            for (const d of dates) {
-              const dayList = dayItems(d);
-              if (dayList.some(item => {
-                if (item.imageHash === hash || item.fileHash === hash) return true;
-                if (Array.isArray(item.imageHashes) && item.imageHashes.includes(hash)) return true;
-                return false;
-              })) {
-                isDuplicate = true;
-                break;
-              }
+      } else if (imageData.rawBuffer && imageData.rawBuffer.length > 0) {
+        const hash = createHash('md5').update(imageData.rawBuffer).digest('hex');
+        if (hash !== lastImageHash) {
+          let isDuplicate = false;
+          const dates = clipboardDates();
+          for (const d of dates) {
+            const dayList = dayItems(d);
+            if (dayList.some(item => {
+              if (item.imageHash === hash || item.fileHash === hash) return true;
+              if (Array.isArray(item.imageHashes) && item.imageHashes.includes(hash)) return true;
+              return false;
+            })) {
+              isDuplicate = true;
+              break;
             }
+          }
 
-            lastImageHash = hash;
-            if (!isDuplicate) {
-              try {
-                saveClipboardImage(imageBuffer, 'png', hash);
-              } catch (error) {
-                logClipboardError('image save', error);
-              }
+          lastImageHash = hash;
+          if (!isDuplicate) {
+            try {
+              saveClipboardImage(imageData.rawBuffer, 'png', hash);
+              markClipboardSuccess('image save');
+            } catch (error) {
+              logClipboardError('image save', error);
             }
           }
         }