buddy-workbench 0.2.0 → 0.2.1

package/package.json

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

package/server/repositories/settings.js

@@ -23,6 +23,9 @@ export function settingsStatus() {
     clipboardEnabled: settings.clipboardEnabled !== false,
     clipboardImageEnabled: settings.clipboardImageEnabled !== false,
     clipboardDeduplicateMinutes: typeof settings.clipboardDeduplicateMinutes === 'number' && !isNaN(settings.clipboardDeduplicateMinutes) ? settings.clipboardDeduplicateMinutes : 60,
+    mindMapNodeMaxWidth: Number.isInteger(settings.mindMapNodeMaxWidth) && settings.mindMapNodeMaxWidth >= 120 && settings.mindMapNodeMaxWidth <= 800
+      ? settings.mindMapNodeMaxWidth
+      : null,
     dataDir: paths.dataDir
   };
 }
@@ -105,6 +108,22 @@ export function saveClipboardDeduplicateMinutes(minutes) {
   return settingsStatus();
 }
 
+export function saveMindMapNodeMaxWidth(width) {
+  const settings = readSettings();
+  if (width === null) {
+    delete settings.mindMapNodeMaxWidth;
+  } else {
+    const value = Number(width);
+    if (!Number.isInteger(value) || value < 120 || value > 800) {
+      throw new Error('Mind map node maximum width must be an integer between 120 and 800.');
+    }
+    settings.mindMapNodeMaxWidth = value;
+  }
+  mkdirSync(dirname(paths.settings), { recursive: true });
+  writeFileSync(paths.settings, JSON.stringify(settings, null, 2), { mode: 0o600 });
+  return settingsStatus();
+}
+
 export function readPrReviewRules(defaultRules) {
   const settings = readSettings();
   const savedRules = settings.prReviewRules && typeof settings.prReviewRules === 'object' ? settings.prReviewRules : {};

package/server/routes/settings.js

@@ -1,7 +1,7 @@
 import { existsSync, mkdirSync, statSync } from 'node:fs';
 import { Router } from 'express';
 import { paths } from '../config.js';
-import { saveAccessToken, saveAiApiHost, saveClipboardDeduplicateMinutes, saveClipboardEnabled, saveClipboardImageEnabled, saveDefaultBrowser, saveDefaultEditor, saveDomain, saveJiraIssuePrefix, saveTheme, settingsStatus } from '../repositories/settings.js';
+import { saveAccessToken, saveAiApiHost, saveClipboardDeduplicateMinutes, saveClipboardEnabled, saveClipboardImageEnabled, saveDefaultBrowser, saveDefaultEditor, saveDomain, saveJiraIssuePrefix, saveMindMapNodeMaxWidth, saveTheme, settingsStatus } from '../repositories/settings.js';
 import { getNpmPackageVersions, installNvmVersion, manageGlobalNpmPackage, readDevConfigurations, readNvmConfiguration, saveDevConfigurations, searchNpmPackages, useSuggestedNpmPrefix } from '../services/dev-configurations.js';
 import { openInSystemBrowser, openInSystemEditor, openInSystemFolder, revealInSystemFolder } from '../services/browser.js';
 import { selectDirectory, selectFile } from '../services/dialog.js';
@@ -149,6 +149,13 @@ router.put('/clipboard-deduplicate-minutes', (req, res) => {
   if (typeof minutes !== 'number' || isNaN(minutes) || minutes < 0) return res.status(400).json({ error: 'Minutes must be a non-negative number.' });
   res.json(saveClipboardDeduplicateMinutes(minutes));
 });
+router.put('/mind-map-node-max-width', (req, res) => {
+  const { width } = req.body || {};
+  if (width !== null && (typeof width !== 'number' || !Number.isInteger(width) || width < 120 || width > 800)) {
+    return res.status(400).json({ error: 'Width must be an integer between 120 and 800, or null to use the layout default.' });
+  }
+  try { res.json(saveMindMapNodeMaxWidth(width)); } catch (error) { res.status(400).json({ error: error.message }); }
+});
 router.put('/:service-access-token', (req, res) => {
   const { token } = req.body || {};
   if (typeof token !== 'string') return res.status(400).json({ error: 'Token must be a string.' });

package/server/services/mac-clipboard-helper.swift

@@ -2,12 +2,14 @@ import AppKit
 import CoreGraphics
 import ImageIO
 import Foundation
+import UniformTypeIdentifiers
 
 // Keep this process alive so Swift is compiled once. Pasteboard image bytes
 // are requested only after the pasteboard ownership changes.
 let outputDirectory = URL(fileURLWithPath: CommandLine.arguments[1], isDirectory: true)
 let maxInputBytes = 100 * 1024 * 1024
 let maxPixelCount = 40_000_000
+let maxCaptureAttempts = 40
 let imageTypes: [NSPasteboard.PasteboardType] = [
     .png,
     NSPasteboard.PasteboardType("public.jpeg"),
@@ -15,22 +17,37 @@ let imageTypes: [NSPasteboard.PasteboardType] = [
     NSPasteboard.PasteboardType("public.heic")
 ]
 
+func isImageType(_ type: NSPasteboard.PasteboardType) -> Bool {
+    if imageTypes.contains(type) { return true }
+    if let uniformType = UTType(type.rawValue), uniformType.conforms(to: .image) { return true }
+    return type.rawValue.localizedCaseInsensitiveContains("image")
+}
+
 func emit(_ value: [String: Any]) {
     guard let data = try? JSONSerialization.data(withJSONObject: value),
           let line = String(data: data, encoding: .utf8) else { return }
     FileHandle.standardOutput.write(Data((line + "\n").utf8))
 }
 
-// Returns false only when the pasteboard advertised image data but could not
-// provide it yet. That case is retried briefly for delayed clipboard owners.
-func captureImage(_ pasteboard: NSPasteboard, changeCount: Int, startedAt: Date) -> Bool {
-    guard let type = pasteboard.types?.first(where: { imageTypes.contains($0) }) else { return true }
+enum CaptureResult: Equatable {
+    case complete
+    case retry
+    case waitingForImageType
+}
+
+// Browser clipboard owners can publish the image type or fulfill its data
+// asynchronously, so keep checking the same pasteboard change for a short time.
+func captureImage(_ pasteboard: NSPasteboard, changeCount: Int, startedAt: Date) -> CaptureResult {
+    let availableTypes = pasteboard.types ?? []
+    let type = imageTypes.first(where: { availableTypes.contains($0) })
+        ?? availableTypes.first(where: isImageType)
+    guard let type else { return .waitingForImageType }
     guard let input = pasteboard.data(forType: type) else {
-        return false
+        return .retry
     }
     guard !input.isEmpty, input.count <= maxInputBytes else {
         emit(["kind": "error", "stage": "image validation", "message": "Image data was empty or exceeded the 100 MiB input limit.", "changeCount": changeCount, "elapsedMs": Int(Date().timeIntervalSince(startedAt) * 1000)])
-        return true
+        return .complete
     }
     guard let source = CGImageSourceCreateWithData(input as CFData, nil),
           let properties = CGImageSourceCopyPropertiesAtIndex(source, 0, nil) as? [CFString: Any],
@@ -39,22 +56,22 @@ func captureImage(_ pasteboard: NSPasteboard, changeCount: Int, startedAt: Date)
           width > 0, height > 0,
           width <= maxPixelCount / height else {
         emit(["kind": "error", "stage": "image validation", "message": "Image dimensions were invalid or exceeded the 40 megapixel limit.", "changeCount": changeCount, "elapsedMs": Int(Date().timeIntervalSince(startedAt) * 1000)])
-        return true
+        return .complete
     }
     guard let image = CGImageSourceCreateImageAtIndex(source, 0, [kCGImageSourceShouldCache: true] as CFDictionary) else {
         emit(["kind": "error", "stage": "image decode", "message": "ImageIO could not decode the clipboard image.", "changeCount": changeCount, "elapsedMs": Int(Date().timeIntervalSince(startedAt) * 1000)])
-        return true
+        return .complete
     }
 
     let jpeg = NSMutableData()
     guard let destination = CGImageDestinationCreateWithData(jpeg, "public.jpeg" as CFString, 1, nil) else {
         emit(["kind": "error", "stage": "JPEG encode", "message": "ImageIO could not create a JPEG encoder.", "changeCount": changeCount, "elapsedMs": Int(Date().timeIntervalSince(startedAt) * 1000)])
-        return true
+        return .complete
     }
     CGImageDestinationAddImage(destination, image, [kCGImageDestinationLossyCompressionQuality: 0.75] as CFDictionary)
     guard CGImageDestinationFinalize(destination), jpeg.length > 0 else {
         emit(["kind": "error", "stage": "JPEG encode", "message": "ImageIO could not finish JPEG encoding.", "changeCount": changeCount, "elapsedMs": Int(Date().timeIntervalSince(startedAt) * 1000)])
-        return true
+        return .complete
     }
 
     let filename = "capture-\(UUID().uuidString).jpg"
@@ -76,7 +93,7 @@ func captureImage(_ pasteboard: NSPasteboard, changeCount: Int, startedAt: Date)
         try? FileManager.default.removeItem(at: outputURL)
         emit(["kind": "error", "stage": "JPEG save", "message": "Could not write the temporary JPEG.", "changeCount": changeCount, "elapsedMs": Int(Date().timeIntervalSince(startedAt) * 1000)])
     }
-    return true
+    return .complete
 }
 
 emit(["kind": "ready"])
@@ -99,11 +116,11 @@ while true {
 
     if let pending = pendingChangeCount {
         attemptsForPendingChange += 1
-        let completed = captureImage(pasteboard, changeCount: pending, startedAt: pendingStartedAt)
-        if !completed && attemptsForPendingChange >= 12 {
-            emit(["kind": "error", "stage": "pasteboard read", "message": "Image data remained unavailable after 12 retries.", "changeCount": pending, "elapsedMs": Int(Date().timeIntervalSince(pendingStartedAt) * 1000)])
+        let result = captureImage(pasteboard, changeCount: pending, startedAt: pendingStartedAt)
+        if result == .retry && attemptsForPendingChange >= maxCaptureAttempts {
+            emit(["kind": "error", "stage": "pasteboard read", "message": "Image data remained unavailable after \(maxCaptureAttempts) retries.", "changeCount": pending, "elapsedMs": Int(Date().timeIntervalSince(pendingStartedAt) * 1000)])
         }
-        if completed || attemptsForPendingChange >= 12 {
+        if result == .complete || attemptsForPendingChange >= maxCaptureAttempts {
             pendingChangeCount = nil
         }
     }