node-mac-recorder 2.21.51 → 2.21.53

package/src/screen_capture_kit.mm

@@ -475,8 +475,7 @@ extern "C" NSString *ScreenCaptureKitCurrentAudioPath(void) {
     }
     
     CMTime presentationTime = CMSampleBufferGetPresentationTimeStamp(sampleBuffer);
-    MRSyncMarkAudioSample(presentationTime);
-    
+
     // Wait for audio to arrive before starting screen video to prevent leading frames.
     if (MRSyncShouldHoldVideoFrame(presentationTime)) {
         return;
@@ -632,6 +631,7 @@ extern "C" NSString *ScreenCaptureKitCurrentAudioPath(void) {
     }
     
     CMTime presentationTime = CMSampleBufferGetPresentationTimeStamp(sampleBuffer);
+    MRSyncMarkAudioSample(presentationTime);
     
     if (!g_audioWriterStarted) {
         if (![g_audioWriter startWriting]) {

package/tasks-1.md

@@ -0,0 +1,231 @@
+ ---
+  ⚠️ 1. KRITIK: Audio Cihaz Tespit ve Varsayılan Seçim Problemi
+
+  Dosya: src/audio_recorder.mm:433-491
+
+  Problem:
+  - Continuity Microphone (iPhone/iPad) gibi harici cihazlar macOS 14+ sistemlerde otomatik olarak varsayılan cihaz
+  seçilebiliyor
+  - Kod şu anda sadece MacBook'un dahili mikrofonunu isDefault: true olarak işaretliyor
+  - Ancak bazı cihazlarda (özellikle iPhone bağlı olduğunda) sistem otomatik olarak Continuity Microphone'u tercih
+  edebiliyor
+
+  Etkilenen Cihazlar:
+  - macOS 14+ ile çalışan tüm MacBook'lar
+  - iPhone/iPad ile Continuity özellikleri aktif olan sistemler
+  - Harici USB mikrofonlar takılı olanlar
+
+  Kod Bölümü:
+  // Lines 455-485
+  BOOL isBuiltIn = NO;
+
+  if ([deviceName rangeOfString:@"MacBook" options:NSCaseInsensitiveSearch].location != NSNotFound ||
+      [deviceName rangeOfString:@"iMac" options:NSCaseInsensitiveSearch].location != NSNotFound) {
+      isBuiltIn = YES;
+  }
+
+  // External devices should NOT be default
+  if ([deviceName rangeOfString:@"Continuity" options:NSCaseInsensitiveSearch].location != NSNotFound ||
+      [deviceName rangeOfString:@"iPhone" options:NSCaseInsensitiveSearch].location != NSNotFound) {
+      isBuiltIn = NO;
+  }
+
+  NSDictionary *info = @{
+      @"isDefault": @(isBuiltIn), // Only built-in devices are default
+      ...
+  };
+
+  Öneri:
+  - Sistem düzeyinde gerçek varsayılan cihazı sorgulamak için [AVCaptureDevice
+  defaultDeviceWithMediaType:AVMediaTypeAudio] kullanılmalı
+  - Kullanıcı tercihi belirtmemişse sistem varsayılanı kullanılmalı
+
+  ---
+  🔴 2. KRITIK: macOS Sürüm Tespiti ve Framework Seçimi
+
+  Dosya: src/mac_recorder.mm:527-697
+
+  Problem:
+  - macOS 14 ve 15'te ScreenCaptureKit kullanmaya çalışıyor ama hata aldığında AVFoundation'a fallback yapıyor
+  - Ancak bazı macOS 14 cihazlarında ScreenCaptureKit çalışıyor gibi görünüp sonra hata verebiliyor (async başlatma
+  problemi)
+  - Electron ortamlarında ScreenCaptureKit varsayılan olarak tercih ediliyor ama bazı cihazlarda AVFoundation daha
+  stabil
+
+  Etkilenen Cihazlar:
+  - macOS 14.0-14.3 arası sürümler (ScreenCaptureKit hala dengesiz)
+  - M1/M2 Mac'ler (bazı kernel panic sorunları yaşanabilir)
+  - Electron uygulamaları (rendering pipeline çakışmaları)
+
+  Kod Bölümü:
+  // Lines 559-576
+  BOOL tryScreenCaptureKit = (isM14Plus || isM15Plus) && !forceAVFoundationEnv;
+
+  if (tryScreenCaptureKit) {
+      if (isElectron) {
+          MRLog(@"⚡ ELECTRON: macOS 14+ → trying ScreenCaptureKit first");
+      }
+
+      @try {
+          if (@available(macOS 12.3, *)) {
+              if ([ScreenCaptureKitRecorder isScreenCaptureKitAvailable]) {
+                  // ScreenCaptureKit başarısız olursa AVFoundation'a fallback yapıyor
+                  // Ancak async başlatma sırasında sorunlar olabilir
+
+  Potansiyel Sorun:
+  - ScreenCaptureKit başlatma süresi cihazlar arası çok değişken (150ms - 2000ms)
+  - Kodda 600ms timeout var ama bazı cihazlarda yeterli olmayabilir
+
+  ---
+  ⚠️ 3. Kamera Capture Timeout Problemi
+
+  Dosya: src/mac_recorder.mm:152-176
+
+  Problem:
+  - Kamera başlatma için 8 saniye timeout var
+  - Continuity Camera (iPhone/iPad kamera) kullanılırken bu timeout yetersiz kalabiliyor
+  - Özellikle Wi-Fi bağlantısı zayıf olduğunda veya Bluetooth üzerinden bağlantı yapılıyorsa
+
+  Etkilenen Cihazlar:
+  - Continuity Camera kullanan tüm cihazlar
+  - Zayıf Wi-Fi/Bluetooth bağlantısı olan ortamlar
+  - USB kameralar (bazı yavaş başlayan modeller)
+
+  Kod Bölümü:
+  // Lines 163-174
+  double cameraWaitTimeout = 8.0; // allow slower devices
+  if (!waitForCameraRecordingStart(cameraWaitTimeout)) {
+      double cameraStartTs = currentCameraRecordingStartTime();
+      if (cameraStartTs > 0 || isCameraRecording()) {
+          MRLog(@"⚠️ Camera did not confirm start within %.1fs but appears to be running; continuing",
+  cameraWaitTimeout);
+          return true;
+      }
+      MRLog(@"❌ Camera did not signal recording start within %.1fs", cameraWaitTimeout);
+      stopCameraRecording();
+      return false;
+  }
+
+  ---
+  🟡 4. Çoklu Ekran (Multi-Display) Koordinat Dönüşüm Hataları
+
+  Dosya: index.js:363-575
+
+  Problem:
+  - Farklı çözünürlük ve ölçeklendirme faktörüne sahip ekranlarda koordinat hesaplamaları hatalı olabiliyor
+  - Retina ekranlar vs standart ekranlar karışık kullanıldığında piksel ölçekleme tutarsızlıkları
+  - Negatif koordinatlar (secondary display sol tarafta olduğunda) doğru işlenmiyor olabilir
+
+  Etkilenen Cihazlar:
+  - MacBook + Harici monitör kombinasyonları
+  - Farklı DPI/scaling'e sahip çoklu ekran kurulumları
+  - Secondary display sol/üst tarafta konumlandırılmış sistemler
+
+  Kod Bölümü:
+  // Lines 476-495
+  const isRelativeToDisplay = () => {
+      const endX = parsedArea.x + parsedArea.width;
+      const endY = parsedArea.y + parsedArea.height;
+      return (
+          parsedArea.x >= -tolerance &&
+          parsedArea.y >= -tolerance &&
+          endX <= targetRect.width + tolerance &&
+          endY <= targetRect.height + tolerance
+      );
+  };
+
+  Risk:
+  - 1 piksel tolerance çok küçük olabilir (Retina ekranlarda 2x ölçekleme)
+
+  ---
+  🟡 5. Audio Format Uyumluluk Problemi
+
+  Dosya: src/audio_recorder.mm:106-151
+
+  Problem:
+  - Ses formatı otomatik tespit ediliyor ama varsayılan değerler sabit kodlanmış
+  - Bazı harici ses kartları/cihazlar 48kHz yerine 44.1kHz veya 96kHz kullanıyor
+  - Kanal sayısı 1-2 arasına zorlanıyor, bazı profesyonel cihazlar daha fazla kanal sunuyor
+
+  Etkilenen Cihazlar:
+  - Profesyonel USB ses arayüzleri (Focusrite, PreSonus, etc.)
+  - Bazı Bluetooth mikrofonlar (codec limitations)
+  - Eski harici ses kartları (44.1kHz native)
+
+  Kod Bölümü:
+  // Lines 109-134
+  double sampleRate = asbd ? asbd->mSampleRate : 48000.0;  // Default to 48kHz
+  NSUInteger channels = asbd ? asbd->mChannelsPerFrame : 1;
+  channels = MAX((NSUInteger)1, channels);
+
+  // CRITICAL FIX: Force to mono or stereo
+  NSUInteger validChannels = (channels <= 1) ? 1 : 2; // Force to mono or stereo
+  audioSettings[AVNumberOfChannelsKey] = @(validChannels);
+
+  Risk:
+  - 4-8 kanallı profesyonel ses arayüzlerinde veri kaybı
+
+  ---
+  🔴 6. ScreenCaptureKit Async Cleanup Race Condition
+
+  Dosya: src/mac_recorder.mm:256-263
+
+  Problem:
+  - ScreenCaptureKit'in async cleanup mekanizması kontrol ediliyor ama hala race condition riski var
+  - Ardışık kayıt başlatma işlemlerinde önceki kayıt henüz tam temizlenmemiş olabiliyor
+
+  Etkilenen Cihazlar:
+  - Tüm macOS 14+ cihazlar (ScreenCaptureKit kullananlar)
+  - Hızlı start/stop yapan kullanıcılar
+
+  Kod Bölümü:
+  // Lines 256-263
+  if (@available(macOS 12.3, *)) {
+      extern BOOL isScreenCaptureKitCleaningUp();
+      if (isScreenCaptureKitCleaningUp()) {
+          MRLog(@"⚠️ ScreenCaptureKit is still stopping previous recording - please wait");
+          return Napi::Boolean::New(env, false);
+      }
+  }
+
+  ---
+  🟠 7. File Format Compatibility (WebM vs MOV)
+
+  Dosya: src/audio_recorder.mm:60-97
+
+  Problem:
+  - macOS 15+ için WebM format tercih ediliyor ama bazı sistemlerde WebM encoder çalışmıyor
+  - Fallback MOV formatına geçiyor ama bu durumda dosya uzantısı tutarsızlığı oluşuyor
+
+  Etkilenen Cihazlar:
+  - macOS 15.0-15.1 (WebM encoder bazı cihazlarda eksik)
+  - Sandbox'lanmış uygulamalar (codec access restrictions)
+
+  ---
+  📊 Özet: Cihaz Kategorilerine Göre Risk Matrisi
+
+  | Cihaz Kategorisi                  | Ana Riskler                                            | Öncelik   |
+  |-----------------------------------|--------------------------------------------------------|-----------|
+  | MacBook Pro M1/M2 + macOS 14      | ScreenCaptureKit instability, Audio device auto-switch | 🔴 Yüksek |
+  | MacBook + Harici Ekran            | Multi-display coordinate bugs, scaling issues          | 🟡 Orta   |
+  | Continuity Camera/Mic Kullananlar | Timeout issues, network latency                        | 🟡 Orta   |
+  | Profesyonel Ses Ekipmanı Olanlar  | Audio format mismatch, sample rate issues              | 🟡 Orta   |
+  | Electron Uygulamaları             | ScreenCaptureKit rendering conflicts                   | 🔴 Yüksek |
+  | macOS 13 Sistemler                | AVFoundation limitations, feature parity               | 🟢 Düşük  |
+
+  ---
+  💡 Öneriler
+
+  1. Audio cihaz seçimi: Sistem varsayılanını kullanmak yerine explicit device selection zorunlu tutulmalı
+  2. ScreenCaptureKit timeout: 600ms yerine 1500ms'ye çıkarılmalı veya adaptive timeout kullanılmalı
+  3. Multi-display: Pixel scaling factor (backingScaleFactor) hesaplamalara dahil edilmeli
+  4. Camera timeout: Continuity Camera için 15 saniye timeout uygulanmalı
+  5. Format selection: WebM yerine MOV varsayılan yapılmalı (daha geniş uyumluluk)
+
+⏺ Analiz tamamlandı! Özetlemek gerekirse:
+
+  En kritik problemler:
+  1. ☢️ Continuity/Harici ses cihazlarının otomatik varsayılan seçilmesi - Kullanıcı beklemediği mikrofon
+  kullanılabilir
+  2. ☢️ ScreenCaptureKit async başlatma race condition - Ardışık kayıtlarda başlatma hataları
+  3. ⚠️ Çoklu ekran koordinat hesaplamaları - Retina + standart ekran kombinasyonlarında yanlış bölge

package/tasks-2.md

@@ -0,0 +1,220 @@
+🔴 KRITIK BUG BULUNDU: Area Recording Tam Ekranı Kaydediyor!
+
+  Evet, ciddi bir bug var - belirtilen alan yerine tam ekranı kaydediyor olabilir. İşte analiz:
+
+  ---
+  🐛 Bug #1: ScreenCaptureKit - sourceRect Bounds Validation Hatası
+
+  Dosya: src/screen_capture_kit.mm:1542-1554
+
+  Problem:
+  if (displayRelativeX >= 0 && displayRelativeY >= 0 &&
+      displayRelativeX + cropWidth <= displayBounds.size.width &&
+      displayRelativeY + cropHeight <= displayBounds.size.height) {
+      CGRect sourceRect = CGRectMake(displayRelativeX, displayRelativeY, cropWidth, cropHeight);
+      streamConfig.sourceRect = sourceRect;
+      MRLog(@"✂️ Crop sourceRect applied: ...");
+  } else {
+      NSLog(@"❌ Crop coordinates out of display bounds - skipping crop");
+      // ⚠️ BURADA sourceRect SET EDİLMİYOR!
+  }
+
+  Sorun:
+  - Koordinatlar display bounds dışındaysa sourceRect hiç set edilmiyor
+  - Bu durumda ScreenCaptureKit tam ekranı kaydediyor (default behavior)
+  - Hata log'u atıyor ama kayıt devam ediyor, kullanıcı fark etmiyor!
+
+  Bu ne zaman oluşur:
+  1. ✅ Koordinatlar 0.1 piksel bile bounds'un dışındaysa
+  2. ✅ Floating point precision hataları (örn: 1920.0000000001)
+  3. ✅ Retina display scaling hesaplamaları yanlışsa
+  4. ✅ Çoklu ekran sistemlerinde display-relative dönüşüm yanlışsa
+
+  ---
+  🐛 Bug #2: Bounds Check'te Floating Point Precision Problemi
+
+  Kod:
+  if (displayRelativeX >= 0 && displayRelativeY >= 0 &&
+      displayRelativeX + cropWidth <= displayBounds.size.width &&
+      displayRelativeY + cropHeight <= displayBounds.size.height)
+
+  Sorun:
+  - <= strict comparison kullanıyor
+  - Floating point hesaplamalardan sonra 1920.0000000001 gibi değerler 1920.0 ile eşleşmiyor
+  - JavaScript'ten gelen koordinatlar Math.round() ile yuvarlanmış olabilir ama native tarafta double precision
+  kullanılıyor
+
+  Örnek:
+  // JavaScript (index.js)
+  captureArea: { x: 0, y: 0, width: 1920, height: 1080 }  // Integer
+
+  // Native (Objective-C)
+  CGFloat cropWidth = [captureRect[@"width"] doubleValue];  // 1920.0
+  CGFloat displayWidth = displayBounds.size.width;          // 1920.0000000001 (floating point)
+
+  // Comparison FAILS!
+  if (0 + 1920.0 <= 1920.0000000001)  // TRUE
+
+  Ancak dikkat:
+  - Display dimensions genelde exact integers (1920x1080)
+  - ANCAK Retina ekranlarda logical size / 2 = 960.0 gibi değerler oluşabilir
+  - Çoklu ekranlarda farklı scaling faktörleri koordinat dönüşümlerini bozabilir
+
+  ---
+  🐛 Bug #3: Koordinat Dönüşüm Zinciri Problemi
+
+  Akış:
+  1. JavaScript (index.js:363-575) - Global koordinatları display-relative'e çeviriyor
+  2. Native StartRecording (mac_recorder.mm:296) - CGRect olarak alıyor
+  3. ScreenCaptureKit (screen_capture_kit.mm:1530) - Display-relative olarak kabul ediyor
+
+  Potansiyel Problem Noktaları:
+
+  3a) JavaScript Normalizasyon Hatası
+
+  index.js:476-486:
+  const tolerance = 1; // 1 pixel tolerance
+  const isRelativeToDisplay = () => {
+      const endX = parsedArea.x + parsedArea.width;
+      const endY = parsedArea.y + parsedArea.height;
+      return (
+          parsedArea.x >= -tolerance &&
+          parsedArea.y >= -tolerance &&
+          endX <= targetRect.width + tolerance &&
+          endY <= targetRect.height + tolerance
+      );
+  };
+
+  Sorun:
+  - 1 pixel tolerance çok küçük olabilir!
+  - Retina ekranlarda logical vs physical pixel karışıklığı
+  - -tolerance negatif koordinatlara izin veriyor ama native kod >= 0 bekliyor
+
+  3b) Clamping/Clipping Yetersizliği
+
+  index.js:509-527:
+  if (relativeX < 0) {
+      relativeWidth += relativeX;  // Width'i azaltıyor
+      relativeX = 0;
+  }
+  // Height için benzer
+
+  Sorun:
+  - Clipping yapıyor ama sonuç <= bounds check'ini geçemeyebilir
+  - Örnek: x=0, width=1920.5 → bounds check fail → tam ekran kaydedilir
+
+  ---
+  🐛 Bug #4: AVFoundation'da Daha Büyük Sorun - Varsayılan Full Screen
+
+  Dosya: src/avfoundation_recorder.mm:233-236
+
+  } else {
+      g_avCaptureRect = CGRectZero; // Full screen
+      NSLog(@"🖥️ Full screen capture using physical dimensions ...");
+  }
+
+  Sorun:
+  - CGRectIsEmpty(captureRect) check'i var
+  - ANCAK JavaScript'ten CGRectNull gelirse (boş alan), bu check FALSE döner
+  - CGRectNull vs CGRectZero vs CGRectEmpty karışıklığı
+
+  JavaScript tarafından gönderilen değer:
+  // mac_recorder.mm:269
+  CGRect captureRect = CGRectNull;  // Default değer
+
+  // Eğer options.captureArea yoksa:
+  if (!CGRectIsNull(captureRect)) {
+      sckConfig[@"captureRect"] = @{ ... };  // Bu ÇALIŞMAZ, captureRect NULL!
+  }
+
+  Sonuç:
+  - captureRect set edilmezse ScreenCaptureKit/AVFoundation full screen kaydeder
+
+  ---
+  🎯 Root Cause: Multi-Step Validation Eksikliği
+
+  Koordinat doğrulama 3 farklı yerde yapılıyor ama tutarsız:
+
+  1. ✅ JavaScript (index.js) - Normalize ediyor, clamp ediyor
+  2. ❌ ScreenCaptureKit - Strict bounds check, fail ederse sourceRect yok → full screen
+  3. ⚠️ AVFoundation - Scaling yapıyor ama validation eksik
+
+  ---
+  🔬 Tespit Yöntemi
+
+  Bu bug'ı şu şekilde tespit edebilirsin:
+
+  const recorder = new MacRecorder();
+
+  // Test case: Display bounds'un HEMEN dışında bir alan
+  const displays = await recorder.getDisplays();
+  const mainDisplay = displays.find(d => d.isPrimary);
+
+  await recorder.startRecording('test.mov', {
+      displayId: mainDisplay.id,
+      captureArea: {
+          x: 0,
+          y: 0,
+          width: mainDisplay.width + 0.5,  // 0.5 piksel daha fazla!
+          height: mainDisplay.height
+      }
+  });
+
+  Beklenen: Kırpılmış/hatalı alan kaydedilmeli
+  Gerçekte Oluyor: Tam ekran kaydediliyor (sourceRect skip ediliyor)
+
+  ---
+  ✅ Çözüm Önerileri
+
+  Fix #1: Tolerance Ekle (ScreenCaptureKit)
+
+  // Line 1542'de
+  const CGFloat CROP_TOLERANCE = 2.0;  // 2 pixel tolerance
+
+  if (displayRelativeX >= -CROP_TOLERANCE &&
+      displayRelativeY >= -CROP_TOLERANCE &&
+      displayRelativeX + cropWidth <= displayBounds.size.width + CROP_TOLERANCE &&
+      displayRelativeY + cropHeight <= displayBounds.size.height + CROP_TOLERANCE) {
+
+      // Clamp coordinates to valid range
+      displayRelativeX = MAX(0, displayRelativeX);
+      displayRelativeY = MAX(0, displayRelativeY);
+      cropWidth = MIN(cropWidth, displayBounds.size.width - displayRelativeX);
+      cropHeight = MIN(cropHeight, displayBounds.size.height - displayRelativeY);
+
+      CGRect sourceRect = CGRectMake(displayRelativeX, displayRelativeY, cropWidth, cropHeight);
+      streamConfig.sourceRect = sourceRect;
+  } else {
+      NSLog(@"❌ CRITICAL: Crop coordinates SIGNIFICANTLY out of bounds - ABORTING recording");
+      // Kayıt başlatmayı reddet, hata döndür
+      SCKFailScheduling();
+      return;
+  }
+
+  Fix #2: JavaScript Tolerance Artır
+
+  // index.js:476
+  const tolerance = 3;  // Retina ekranlar için daha büyük tolerance
+
+  Fix #3: Varsayılan Davranışı Değiştir
+
+  // Bounds check fail ederse TAM EKRAN yerine HATA DÖNDÜR
+  NSLog(@"❌ Invalid crop coordinates - refusing to record");
+  *error = [NSError errorWithDomain:@"ScreenCapture"
+                               code:-1
+                           userInfo:@{NSLocalizedDescriptionKey: @"Crop area out of display bounds"}];
+  return NO;  // Recording başarısız
+
+  ---
+  📝 Sonuç
+
+  EVET, ciddi bir bug var:
+  - ✅ Area recording bounds check'i fail ederse tam ekran kaydediyor
+  - ✅ Floating point precision hataları check'i fail ettiriyor
+  - ✅ Kullanıcı bilgilendirilmiyor (sadece log)
+  - ✅ JavaScript'ten gelen koordinatlar native validation'dan geçemeyebiliyor
+
+  En riskli senaryolar:
+  1. Retina display + area recording
+  2. Çoklu ekran + area recording
+  3. 0.5 piksel hassasiyetle alan seçimi (UI'dan gelebilir)