node-mac-recorder 2.20.8 → 2.20.10

package/.claude/settings.local.json

@@ -2,7 +2,8 @@
   "permissions": {
     "allow": [
       "Bash(node-gyp:*)",
-      "Bash(node:*)"
+      "Bash(node:*)",
+      "Bash(timeout:*)"
     ],
     "deny": [],
     "ask": []

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "node-mac-recorder",
-	"version": "2.20.8",
+	"version": "2.20.10",
 	"description": "Native macOS screen recording package for Node.js applications",
 	"main": "index.js",
 	"keywords": [

package/src/cursor_tracker.mm

@@ -392,56 +392,100 @@ static NSString* cursorTypeFromCursorName(NSString *value) {
 
     NSString *normalized = [[value stringByReplacingOccurrencesOfString:@"_" withString:@"-"] lowercaseString];
 
-    if ([normalized containsString:@"arrow"]) {
+    // Arrow cursor patterns
+    if ([normalized containsString:@"arrow"] || [normalized containsString:@"default"]) {
         return @"default";
     }
+
+    // Text cursor patterns
     if ([normalized containsString:@"ibeam"] ||
         [normalized containsString:@"insertion"] ||
-        [normalized containsString:@"text"]) {
+        [normalized containsString:@"text"] ||
+        [normalized containsString:@"edit"]) {
         return @"text";
     }
-    if ([normalized containsString:@"openhand"]) {
+
+    // Hand cursors
+    if ([normalized containsString:@"openhand"] || [normalized containsString:@"open-hand"]) {
         return @"grab";
     }
-    if ([normalized containsString:@"closedhand"]) {
+    if ([normalized containsString:@"closedhand"] || [normalized containsString:@"closed-hand"]) {
         return @"grabbing";
     }
+
+    // Pointer cursor patterns
     if ([normalized containsString:@"pointing"] ||
-        ([normalized containsString:@"hand"] && ![normalized containsString:@"closed"])) {
+        [normalized containsString:@"pointinghand"] ||
+        ([normalized containsString:@"hand"] && ![normalized containsString:@"closed"] && ![normalized containsString:@"open"]) ||
+        [normalized containsString:@"link"] ||
+        [normalized containsString:@"button"]) {
         return @"pointer";
     }
-    if ([normalized containsString:@"crosshair"]) {
+
+    // Crosshair patterns
+    if ([normalized containsString:@"crosshair"] || [normalized containsString:@"cross-hair"]) {
         return @"crosshair";
     }
+
+    // Not allowed patterns
     if ([normalized containsString:@"not-allowed"] ||
         [normalized containsString:@"notallowed"] ||
-        [normalized containsString:@"forbidden"]) {
+        [normalized containsString:@"forbidden"] ||
+        [normalized containsString:@"operation-not-allowed"]) {
         return @"not-allowed";
     }
-    if ([normalized containsString:@"dragcopy"] || [normalized containsString:@"copy"]) {
+
+    // Copy cursor patterns
+    if ([normalized containsString:@"dragcopy"] ||
+        [normalized containsString:@"drag-copy"] ||
+        [normalized containsString:@"copy"]) {
         return @"copy";
     }
-    if ([normalized containsString:@"draglink"] || [normalized containsString:@"alias"]) {
+
+    // Alias cursor patterns
+    if ([normalized containsString:@"draglink"] ||
+        [normalized containsString:@"drag-link"] ||
+        [normalized containsString:@"alias"]) {
         return @"alias";
     }
-    if ([normalized containsString:@"context"] && [normalized containsString:@"menu"]) {
+
+    // Context menu patterns
+    if (([normalized containsString:@"context"] && [normalized containsString:@"menu"]) ||
+        [normalized containsString:@"contextual-menu"]) {
         return @"context-menu";
     }
+
+    // Zoom patterns
     if ([normalized containsString:@"zoom"]) {
         if ([normalized containsString:@"out"]) {
             return @"zoom-out";
         }
         return @"zoom-in";
     }
+
+    // Resize cursor patterns - more comprehensive
     if ([normalized containsString:@"resize"] || [normalized containsString:@"size"]) {
-        BOOL diagonalUp = [normalized containsString:@"diagonalup"] || [normalized containsString:@"nesw"];
-        BOOL diagonalDown = [normalized containsString:@"diagonaldown"] || [normalized containsString:@"nwse"];
+        // Diagonal resize cursors
+        BOOL diagonalUp = [normalized containsString:@"diagonalup"] ||
+                          [normalized containsString:@"diagonal-up"] ||
+                          [normalized containsString:@"nesw"];
+        BOOL diagonalDown = [normalized containsString:@"diagonaldown"] ||
+                            [normalized containsString:@"diagonal-down"] ||
+                            [normalized containsString:@"nwse"];
+
+        // Horizontal and vertical resize
         BOOL horizontal = [normalized containsString:@"leftright"] ||
+                          [normalized containsString:@"left-right"] ||
                           [normalized containsString:@"horizontal"] ||
-                          ([normalized containsString:@"left"] && [normalized containsString:@"right"]);
+                          ([normalized containsString:@"left"] && [normalized containsString:@"right"]) ||
+                          [normalized containsString:@"col"] ||
+                          [normalized containsString:@"column"];
+
         BOOL vertical = [normalized containsString:@"updown"] ||
+                        [normalized containsString:@"up-down"] ||
                         [normalized containsString:@"vertical"] ||
-                        ([normalized containsString:@"up"] && [normalized containsString:@"down"]);
+                        ([normalized containsString:@"up"] && [normalized containsString:@"down"]) ||
+                        [normalized containsString:@"row"];
 
         if (diagonalUp) {
             return @"nesw-resize";
@@ -455,6 +499,22 @@ static NSString* cursorTypeFromCursorName(NSString *value) {
         if (horizontal) {
             return @"col-resize";
         }
+
+        // Generic resize fallback
+        return @"default"; // or could be a generic resize
+    }
+
+    // Additional specific patterns
+    if ([normalized containsString:@"wait"] || [normalized containsString:@"busy"]) {
+        return @"wait";
+    }
+
+    if ([normalized containsString:@"help"] || [normalized containsString:@"question"]) {
+        return @"help";
+    }
+
+    if ([normalized containsString:@"progress"]) {
+        return @"progress";
     }
 
     return nil;
@@ -462,9 +522,10 @@ static NSString* cursorTypeFromCursorName(NSString *value) {
 
 static NSString* cursorTypeFromNSCursor(NSCursor *cursor) {
     if (!cursor) {
-        return nil;
+        return @"default";
     }
 
+    // Standard macOS cursors
     if (cursor == [NSCursor arrowCursor]) {
         return @"default";
     }
@@ -499,34 +560,66 @@ static NSString* cursorTypeFromNSCursor(NSCursor *cursor) {
     if (cursor == [NSCursor contextualMenuCursor]) {
         return @"context-menu";
     }
-    if ([NSCursor respondsToSelector:@selector(resizeLeftRightCursor)] &&
-        (cursor == [NSCursor resizeLeftRightCursor] ||
-         cursor == [NSCursor resizeLeftCursor] ||
-         cursor == [NSCursor resizeRightCursor])) {
-        return @"col-resize";
+
+    // Resize cursors - improved detection
+    if ([NSCursor respondsToSelector:@selector(resizeLeftRightCursor)]) {
+        if (cursor == [NSCursor resizeLeftRightCursor]) {
+            return @"col-resize";
+        }
     }
-    if ([NSCursor respondsToSelector:@selector(resizeUpDownCursor)] &&
-        (cursor == [NSCursor resizeUpDownCursor] ||
-         cursor == [NSCursor resizeUpCursor] ||
-         cursor == [NSCursor resizeDownCursor])) {
-        return @"ns-resize";
+    if ([NSCursor respondsToSelector:@selector(resizeLeftCursor)]) {
+        if (cursor == [NSCursor resizeLeftCursor]) {
+            return @"col-resize";
+        }
+    }
+    if ([NSCursor respondsToSelector:@selector(resizeRightCursor)]) {
+        if (cursor == [NSCursor resizeRightCursor]) {
+            return @"col-resize";
+        }
     }
+    if ([NSCursor respondsToSelector:@selector(resizeUpDownCursor)]) {
+        if (cursor == [NSCursor resizeUpDownCursor]) {
+            return @"ns-resize";
+        }
+    }
+    if ([NSCursor respondsToSelector:@selector(resizeUpCursor)]) {
+        if (cursor == [NSCursor resizeUpCursor]) {
+            return @"ns-resize";
+        }
+    }
+    if ([NSCursor respondsToSelector:@selector(resizeDownCursor)]) {
+        if (cursor == [NSCursor resizeDownCursor]) {
+            return @"ns-resize";
+        }
+    }
+
     if ([NSCursor respondsToSelector:@selector(disappearingItemCursor)] &&
         cursor == [NSCursor disappearingItemCursor]) {
         return @"default";
     }
 
-    NSString *derived = cursorTypeFromCursorName(NSStringFromClass([cursor class]));
+    // Try to get class name and description for debugging
+    NSString *className = NSStringFromClass([cursor class]);
+    NSString *description = [cursor description];
+
+    MRLog(@"🔍 Unknown cursor class: %@, description: %@", className ?: @"nil", description ?: @"nil");
+
+    // Try name-based detection
+    NSString *derived = cursorTypeFromCursorName(className);
     if (derived) {
+        MRLog(@"🎯 Cursor type from class name: %@", derived);
         return derived;
     }
 
-    derived = cursorTypeFromCursorName([cursor description]);
+    derived = cursorTypeFromCursorName(description);
     if (derived) {
+        MRLog(@"🎯 Cursor type from description: %@", derived);
         return derived;
     }
 
-    return nil;
+    // Default fallback
+    MRLog(@"⚠️ Unknown cursor type, defaulting to 'default'");
+    return @"default";
 }
 
 static NSString* detectSystemCursorType(void) {
@@ -534,21 +627,67 @@ static NSString* detectSystemCursorType(void) {
 
     void (^fetchCursorBlock)(void) = ^{
         NSCursor *currentCursor = nil;
+
+        // Try different methods to get current cursor
         if ([NSCursor respondsToSelector:@selector(currentSystemCursor)]) {
             currentCursor = [NSCursor currentSystemCursor];
+            NSLog(@"🖱️ currentSystemCursor: %@", currentCursor ?: @"nil");
         }
+
         if (!currentCursor) {
             currentCursor = [NSCursor currentCursor];
+            NSLog(@"🖱️ currentCursor: %@", currentCursor ?: @"nil");
         }
 
-        NSString *derivedType = cursorTypeFromNSCursor(currentCursor);
-        if (derivedType) {
-            cursorType = derivedType;
-        } else if (currentCursor) {
+        if (currentCursor) {
+            NSString *className = NSStringFromClass([currentCursor class]);
+            NSString *description = [currentCursor description];
+            NSLog(@"🖱️ Cursor class: %@, description: %@", className ?: @"nil", description ?: @"nil");
+
+            // Use more direct cursor detection approach
+            NSImage *cursorImage = [currentCursor image];
+            NSPoint hotspot = [currentCursor hotSpot];
+            NSSize imageSize = [cursorImage size];
+
+            NSLog(@"🎯 Cursor analysis: size=%.1fx%.1f, hotspot=(%.1f,%.1f)",
+                  imageSize.width, imageSize.height, hotspot.x, hotspot.y);
+
+            // Updated pattern recognition based on observed values
+            if (imageSize.width >= 8.0 && imageSize.width <= 12.0 &&
+                imageSize.height >= 16.0 && imageSize.height <= 20.0 &&
+                hotspot.x >= 3.0 && hotspot.x <= 5.0 && hotspot.y >= 8.0 && hotspot.y <= 10.0) {
+                // I-beam cursor: narrow and tall, hotspot in middle
+                cursorType = @"text";
+                NSLog(@"🎯 DETECTED IBEAM CURSOR -> text");
+            }
+            else if (imageSize.width <= 20.0 && imageSize.height <= 25.0 &&
+                     hotspot.x <= 5.0 && hotspot.y <= 5.0) {
+                // Arrow cursor: small size, hotspot at top-left
+                cursorType = @"default";
+                NSLog(@"🎯 DETECTED ARROW CURSOR -> default");
+            }
+            else if (imageSize.width >= 15.0 && imageSize.width <= 25.0 &&
+                     imageSize.height >= 15.0 && imageSize.height <= 25.0 &&
+                     hotspot.x >= 5.0 && hotspot.x <= 10.0 && hotspot.y <= 8.0) {
+                // Hand cursor: squarish, hotspot at fingertip
+                cursorType = @"pointer";
+                NSLog(@"🎯 DETECTED HAND CURSOR -> pointer");
+            }
+            else {
+                // Try to use a different approach - cursor name introspection
+                NSString *derived = cursorTypeFromNSCursor(currentCursor);
+                if (derived && ![derived isEqualToString:@"default"]) {
+                    cursorType = derived;
+                    NSLog(@"🎯 DERIVED FROM ANALYSIS: %@", cursorType);
+                } else {
+                    cursorType = @"default";
+                    NSLog(@"🎯 FALLBACK TO DEFAULT (will check AX)");
+                }
+            }
+        } else {
+            NSLog(@"🖱️ No current cursor found");
             cursorType = @"default";
         }
-
-        MRLog(@"🎯 SYSTEM CURSOR TYPE: %@", cursorType ? cursorType : @"(nil)");
     };
 
     if ([NSThread isMainThread]) {
@@ -564,7 +703,7 @@ NSString* getCursorType() {
     @autoreleasepool {
         g_cursorTypeCounter++;
 
-        // Hybrid: AX (strict) first, then system cursor fallback.
+        // Get cursor position first
         BOOL hasCursorPosition = NO;
         CGPoint cursorPos = CGPointZero;
 
@@ -589,25 +728,45 @@ NSString* getCursorType() {
             }
         }
 
+        // Try multiple detection methods
+        NSString *systemCursorType = detectSystemCursorType();
         NSString *axCursorType = nil;
+
         if (hasCursorPosition) {
             axCursorType = detectCursorTypeUsingAccessibility(cursorPos);
         }
 
-        NSString *systemCursorType = detectSystemCursorType();
+        NSString *finalType = @"default";
 
-        NSString *finalType = nil;
-        if (axCursorType && ![axCursorType isEqualToString:@"default"]) {
+        // Smart decision logic: AX for special cases, system for normal cases
+        NSLog(@"🎯 CURSOR DECISION: system='%@', ax='%@'", systemCursorType ?: @"nil", axCursorType ?: @"nil");
+
+        // If system cursor detected something specific (not default), use it
+        if (systemCursorType && [systemCursorType length] > 0 && ![systemCursorType isEqualToString:@"default"]) {
+            finalType = systemCursorType;
+            NSLog(@"🎯 Using SYSTEM cursor (specific): %@", finalType);
+        }
+        // If AX detected something specific and system is default, use AX
+        else if (axCursorType && [axCursorType length] > 0 && ![axCursorType isEqualToString:@"default"] &&
+                 (!systemCursorType || [systemCursorType isEqualToString:@"default"])) {
             finalType = axCursorType;
-        } else if (systemCursorType && [systemCursorType length] > 0) {
+            NSLog(@"🎯 Using AX cursor (specific over default): %@", finalType);
+        }
+        // Otherwise use system cursor (including default)
+        else if (systemCursorType && [systemCursorType length] > 0) {
             finalType = systemCursorType;
-        } else if (axCursorType && [axCursorType length] > 0) {
+            NSLog(@"🎯 Using SYSTEM cursor: %@", finalType);
+        }
+        // Final fallback to AX or default
+        else if (axCursorType && [axCursorType length] > 0) {
             finalType = axCursorType;
-        } else {
-            finalType = @"default";
+            NSLog(@"🎯 Using AX cursor (final fallback): %@", finalType);
+        }
+        else {
+            NSLog(@"🎯 Ultimate fallback to default");
         }
 
-        MRLog(@"🎯 FINAL CURSOR TYPE: %@", finalType);
+        NSLog(@"🎯 FINAL CURSOR TYPE: %@", finalType);
         return finalType;
     }
 }