node-mac-recorder 2.18.4 → 2.18.6

package/.claude/settings.local.json

@@ -10,7 +10,9 @@
       "Bash(timeout:*)",
       "Bash(grep:*)",
       "Bash(./test-cleanup.sh:*)",
-      "Bash(FORCE_AVFOUNDATION=1 node test-avfoundation-dpr.js)"
+      "Bash(FORCE_AVFOUNDATION=1 node test-avfoundation-dpr.js)",
+      "Bash(FORCE_AVFOUNDATION=1 node test-cursor-fix.js)",
+      "Bash(pkill:*)"
     ],
     "deny": [],
     "ask": []

package/index.js

@@ -705,6 +705,8 @@ class MacRecorder extends EventEmitter {
 			// Calculate video offset based on recording type
 			let videoOffsetX = 0;
 			let videoOffsetY = 0;
+			// UNIFIED CURSOR SYSTEM: Always use logical dimensions for cursor coordinates
+			// This ensures cursor tracking works consistently across all recording engines
 			let videoWidth = options.displayInfo.width || options.displayInfo.logicalWidth;
 			let videoHeight = options.displayInfo.height || options.displayInfo.logicalHeight;
 
@@ -739,6 +741,11 @@ class MacRecorder extends EventEmitter {
 				videoOffsetY: videoOffsetY,
 				videoWidth: videoWidth,
 				videoHeight: videoHeight,
+				// CRITICAL: Video output size for proper cursor scaling
+				videoOutputWidth: (options.displayInfo.scaleFactor > 1.0) ?
+					(options.displayInfo.physicalWidth || videoWidth) : videoWidth,
+				videoOutputHeight: (options.displayInfo.scaleFactor > 1.0) ?
+					(options.displayInfo.physicalHeight || videoHeight) : videoHeight,
 				videoRelative: true,
 				recordingType: options.recordingType || 'display',
 				// Store additional context for debugging
@@ -759,8 +766,16 @@ class MacRecorder extends EventEmitter {
 				scaleFactor: this.recordingDisplayInfo.scaleFactor || 1.0,
 				videoOffsetX: 0,
 				videoOffsetY: 0,
+				// UNIFIED CURSOR SYSTEM: Always use logical dimensions
 				videoWidth: this.recordingDisplayInfo.width || this.recordingDisplayInfo.logicalWidth,
 				videoHeight: this.recordingDisplayInfo.height || this.recordingDisplayInfo.logicalHeight,
+				// CRITICAL: Video output size for proper cursor scaling
+				videoOutputWidth: (this.recordingDisplayInfo.scaleFactor > 1.0) ?
+					(this.recordingDisplayInfo.physicalWidth || this.recordingDisplayInfo.width || this.recordingDisplayInfo.logicalWidth) :
+					(this.recordingDisplayInfo.width || this.recordingDisplayInfo.logicalWidth),
+				videoOutputHeight: (this.recordingDisplayInfo.scaleFactor > 1.0) ?
+					(this.recordingDisplayInfo.physicalHeight || this.recordingDisplayInfo.height || this.recordingDisplayInfo.logicalHeight) :
+					(this.recordingDisplayInfo.height || this.recordingDisplayInfo.logicalHeight),
 				videoRelative: true,
 				recordingType: options.recordingType || 'display'
 			};
@@ -812,28 +827,30 @@ class MacRecorder extends EventEmitter {
 							const displayRelativeX = position.x - this.cursorDisplayInfo.displayX;
 							const displayRelativeY = position.y - this.cursorDisplayInfo.displayY;
 
-							// CRITICAL FIX: Apply physical scaling for AVFoundation recording
-							// AVFoundation records at physical resolution while cursor coordinates are logical
-							let scaledX = displayRelativeX;
-							let scaledY = displayRelativeY;
-
-							if (this.cursorDisplayInfo.scaleFactor && this.cursorDisplayInfo.scaleFactor > 1.0) {
-								// Scale logical cursor coordinates to match physical video dimensions
-								scaledX = displayRelativeX * this.cursorDisplayInfo.scaleFactor;
-								scaledY = displayRelativeY * this.cursorDisplayInfo.scaleFactor;
-								coordinateSystem = "video-relative-scaled";
-							}
-
 							// Step 2: Transform display-relative → video-relative coordinates
-							x = scaledX - this.cursorDisplayInfo.videoOffsetX;
-							y = scaledY - this.cursorDisplayInfo.videoOffsetY;
-							coordinateSystem = this.cursorDisplayInfo.scaleFactor > 1.0 ? "video-relative-scaled" : "video-relative";
-
-							// Bounds check for video area (use physical video dimensions for AVFoundation)
-							const videoWidth = this.cursorDisplayInfo.physicalWidth || this.cursorDisplayInfo.videoWidth;
-							const videoHeight = this.cursorDisplayInfo.physicalHeight || this.cursorDisplayInfo.videoHeight;
+							x = displayRelativeX - this.cursorDisplayInfo.videoOffsetX;
+							y = displayRelativeY - this.cursorDisplayInfo.videoOffsetY;
+
+							// Step 3: Scale to match actual video output size (for AVFoundation physical scaling)
+							if (this.cursorDisplayInfo.videoOutputWidth && this.cursorDisplayInfo.videoOutputHeight) {
+								const scaleX = this.cursorDisplayInfo.videoOutputWidth / this.cursorDisplayInfo.videoWidth;
+								const scaleY = this.cursorDisplayInfo.videoOutputHeight / this.cursorDisplayInfo.videoHeight;
+
+								if (scaleX !== 1.0 || scaleY !== 1.0) {
+									x = x * scaleX;
+									y = y * scaleY;
+									coordinateSystem = "video-output-scaled";
+								} else {
+									coordinateSystem = "video-relative";
+								}
+							} else {
+								coordinateSystem = "video-relative";
+							}
 
-							const outsideVideo = x < 0 || y < 0 || x >= videoWidth || y >= videoHeight;
+							// Bounds check for video area using actual video output dimensions
+							const checkWidth = this.cursorDisplayInfo.videoOutputWidth || this.cursorDisplayInfo.videoWidth;
+							const checkHeight = this.cursorDisplayInfo.videoOutputHeight || this.cursorDisplayInfo.videoHeight;
+							const outsideVideo = x < 0 || y < 0 || x >= checkWidth || y >= checkHeight;
 
 							// For debugging - add metadata if cursor is outside video area
 							if (outsideVideo) {

package/package.json

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