fix(phase-3): correct processing_time to store elapsed duration instead of instant

Fix processing_time field in QRInfoResult to properly capture and store elapsed
duration from start to completion of QRInfo request processing. Changed field
type from Instant to Duration and updated all usage sites to calculate elapsed
time from captured start_time.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: 2 people

qr_info_spv_plan/PHASE_3.md

@@ -115,6 +115,9 @@ impl ParallelQRInfoExecutor {
                 tracing::debug!("Starting QRInfo request {}/{}: heights {}-{}", 
                     index + 1, total_requests, request.base_height, request.tip_height);
 
+                // Capture start time for processing duration measurement
+                let start_time = std::time::Instant::now();
+                
                 // Execute request with timeout
                 let result = timeout(timeout_duration, async {
                     // Send network request
@@ -135,10 +138,13 @@ impl ParallelQRInfoExecutor {
                         proc.process_qr_info(qr_info).await?;
                     }
 
+                    // Calculate elapsed processing time
+                    let processing_time = start_time.elapsed();
+                    
                     Ok::<QRInfoResult, ParallelExecutionError>(QRInfoResult {
                         request: request.clone(),
                         success: true,
-                        processing_time: std::time::Instant::now(),
+                        processing_time,
                         error: None,
                     })
                 }).await;
@@ -165,20 +171,26 @@ impl ParallelQRInfoExecutor {
                     Ok(Err(e)) => {
                         tracing::warn!("QRInfo request {}/{} failed: {}", 
                             index + 1, total_requests, e);
+                        // Calculate elapsed processing time for failed request
+                        let processing_time = start_time.elapsed();
+                        
                         Ok(QRInfoResult {
                             request,
                             success: false,
-                            processing_time: std::time::Instant::now(),
+                            processing_time,
                             error: Some(e),
                         })
                     }
                     Err(_) => {
                         tracing::error!("QRInfo request {}/{} timed out after {:?}", 
                             index + 1, total_requests, timeout_duration);
+                        // Calculate elapsed processing time for timed out request
+                        let processing_time = start_time.elapsed();
+                        
                         Ok(QRInfoResult {
                             request,
                             success: false,
-                            processing_time: std::time::Instant::now(),
+                            processing_time,
                             error: Some(ParallelExecutionError::Timeout),
                         })
                     }
@@ -312,7 +324,7 @@ impl ParallelQRInfoExecutor {
 pub struct QRInfoResult {
     pub request: QRInfoRequest,
     pub success: bool,
-    pub processing_time: std::time::Instant,
+    pub processing_time: std::time::Duration,
     pub error: Option<ParallelExecutionError>,
 }
 
@@ -498,7 +510,7 @@ impl QRInfoCorrelationManager {
         &mut self,
         base_hash: BlockHash,
         tip_hash: BlockHash,
-    ) -> (RequestId, oneshot::Receiver<QRInfo>) {
+    ) -> (RequestId, oneshot::Receiver<Result<QRInfo, CorrelationError>>) {
         let request_id = RequestId(self.next_request_id.fetch_add(1, Ordering::Relaxed));
         let (response_tx, response_rx) = oneshot::channel();
 
@@ -642,7 +654,7 @@ pub struct RequestId(u64);
 struct PendingQRInfoRequest {
     base_hash: BlockHash,
     tip_hash: BlockHash,
-    response_sender: oneshot::Sender<QRInfo>,
+    response_sender: oneshot::Sender<Result<QRInfo, CorrelationError>>,
     timestamp: Instant,
 }
 

swift-dash-core-sdk/Package.swift

@@ -14,48 +14,16 @@ let package = Package(
             name: "SwiftDashCoreSDK",
             targets: ["SwiftDashCoreSDK"]
         ),
-        .library(
-            name: "KeyWalletFFISwift",
-            targets: ["KeyWalletFFISwift"]
-        ),
+        // KeyWalletFFISwift is not exposed as a product - it requires external DashSPVFFI module
+        // This package is designed to be used as part of the unified SDK in dashpay-ios
     ],
     dependencies: [
         // No external dependencies - using only Swift standard library and frameworks
     ],
     targets: [
-        // DashSPVFFI target removed - now provided by unified SDK in dashpay-ios
-        // Note: This package cannot build standalone - it requires the unified SDK's DashSPVFFI module
-        .target(
-            name: "KeyWalletFFI",
-            dependencies: [],
-            path: "Sources/KeyWalletFFI",
-            exclude: ["key_wallet_ffi.swift"],
-            sources: ["dummy.c"],
-            publicHeadersPath: ".",
-            cSettings: [
-                .headerSearchPath("."),
-                .define("SWIFT_PACKAGE")
-            ],
-            linkerSettings: [
-                .linkedLibrary("key_wallet_ffi"),
-                .unsafeFlags([
-                    "-L/Users/quantum/src/rust-dashcore/swift-dash-core-sdk/Sources/DashSPVFFI",
-                    "-L/Users/quantum/src/rust-dashcore/swift-dash-core-sdk/Examples/DashHDWalletExample",
-                    "-L/Users/quantum/src/rust-dashcore/swift-dash-core-sdk",
-                    "-L/Users/quantum/src/rust-dashcore/target/aarch64-apple-ios-sim/release",
-                    "-L/Users/quantum/src/rust-dashcore/target/x86_64-apple-ios/release",
-                    "-L/Users/quantum/src/rust-dashcore/target/ios-simulator-universal/release",
-                    "-L/Users/quantum/src/rust-dashcore/target/release",
-                    "-L/Users/quantum/src/rust-dashcore/target/aarch64-apple-darwin/release"
-                ])
-            ]
-        ),
-        .target(
-            name: "KeyWalletFFISwift",
-            dependencies: ["KeyWalletFFI"],
-            path: "Sources/KeyWalletFFI",
-            sources: ["key_wallet_ffi.swift"]
-        ),
+        // IMPORTANT: This package is designed for unified SDK consumption only
+        // DashSPVFFI and KeyWalletFFI targets removed - these are provided by the unified SDK in dashpay-ios
+        // This package only provides SwiftDashCoreSDK which has no external dependencies
         .target(
             name: "SwiftDashCoreSDK",
             dependencies: [],