Merge pull request #6 from SimformSolutionsPvtLtd/fix/UNT-T39093_ios_haptics_release_build_timing

fix(UNT-T39093): Resolve Core Haptics timing validation error in iOS Release builds

Author: mukesh-simform

ios/HapticEngine.swift

@@ -40,6 +40,12 @@ class HapticEngine: NSObject {
     @objc(start:reject:)
     func start(resolve: @escaping RCTPromiseResolveBlock,
                reject: @escaping  RCTPromiseRejectBlock) {
+        // Check if device supports haptics first
+        guard CHHapticEngine.capabilitiesForHardware().supportsHaptics else {
+            reject("HAPTICS_NOT_SUPPORTED", "Device does not support haptics", nil)
+            return
+        }
+        
         var engine: CHHapticEngine
 
         if let existingEngine = HapticEngine.shared.engines[Key.defaultEngine] {
@@ -49,6 +55,16 @@ class HapticEngine: NSObject {
         else {
             do {
                 let newEngine = try CHHapticEngine()
+                
+                // Add reset and stopped handlers for better error handling
+                newEngine.resetHandler = {
+                    print("Haptic engine reset")
+                }
+                
+                newEngine.stoppedHandler = { reason in
+                    print("Haptic engine stopped: \(reason.rawValue)")
+                }
+                
                 HapticEngine.shared.engines[Key.defaultEngine] = newEngine
                 engine = newEngine
             } catch let e {
@@ -117,14 +133,14 @@ class HapticEngine: NSObject {
      */
     @objc(startPlayerAtTime:startTime:resolve:reject:)
     func startPlayerAtTime(pattern: HapticPattern,
-                           startTime: CGFloat,
+                           startTime: NSNumber,
                            resolve: RCTPromiseResolveBlock,
                            reject: RCTPromiseRejectBlock) {
         var engine: CHHapticEngine
 
         if let hapticPatternPlayer {
             do {
-                try hapticPatternPlayer.stop(atTime: .zero)
+                try hapticPatternPlayer.stop(atTime: CHHapticTimeImmediate)
             } catch let e {
                 reject("Unable to stop the active player before starting the new haptic pattern", e.localizedDescription, e)
                 return
@@ -174,7 +190,11 @@ class HapticEngine: NSObject {
         }
 
         do {
-            try hapticPatternPlayer?.start(atTime: TimeInterval(startTime))
+            // Use CHHapticTimeImmediate for immediate playback when startTime is 0 or negative
+            // This fixes the timing validation error in Release builds
+            let startTimeValue = startTime.doubleValue
+            let playbackTime = startTimeValue <= 0 ? CHHapticTimeImmediate : TimeInterval(startTimeValue)
+            try hapticPatternPlayer?.start(atTime: playbackTime)
 
             resolve(true)
         } catch let e {
@@ -189,8 +209,19 @@ class HapticEngine: NSObject {
     @objc(stop:reject:)
     func stop(resolve: @escaping RCTPromiseResolveBlock,
               reject: @escaping RCTPromiseRejectBlock) {
+        // First, stop any active player immediately
+        if let activePlayer = hapticPatternPlayer {
+            do {
+                try activePlayer.stop(atTime: CHHapticTimeImmediate)
+            } catch {
+                print("Warning: Could not stop active haptic player: \(error.localizedDescription)")
+            }
+            hapticPatternPlayer = nil
+        }
+        
         guard let existingEngine = HapticEngine.shared.engines[Key.defaultEngine] else {
-            reject("Unable to find engine", "Engine has not been created", nil)
+            // If no engine exists, consider it already stopped
+            resolve(true)
             return
         }