fix(ios): resolve swift6 IRGen compile crash (#717)

## Fixes

- finalize HybridSound and conform via base+protocol; avoid
RecordBackType interpolation to prevent Swift 6 IRGen recursion
- enable BUILD_LIBRARY_FOR_DISTRIBUTION and wholemodule in podspec for
stability
- silence iOS logs in Release; add Android debug-only Logger for future
use

Closes #715
Closes #716

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- New Features
  - No user-facing features added.
- Refactor
  - Centralized Android logging and reduced log noise in release builds.
  - Adjusted iOS logging to avoid verbose output in production.
- Minor logging format tweaks during recording for improved reliability.
- Chores
- Updated iOS build settings to improve distribution readiness and
compilation behavior.

- Notes
- Expect quieter release logs on both platforms and improved stability
during recording scenarios.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Author: hyochan

CONTRIBUTING.md

@@ -97,3 +97,17 @@ array.forEach((e) => {
 
 - Space before `(` and after `)`.
 - **If you find code that does not fit in the coding convention, do not ever try to fix code that is not related to your purpose.**
+
+### Logging
+
+Use the following module-provided logging approaches during development. Both are silent in Release/production builds.
+
+- iOS
+  - Preferred: call `print(...)` as usual inside this module. We provide `ios/Logging.swift` which overrides `print` only in non-DEBUG builds, so all prints are automatically suppressed in Release while remaining visible in Debug.
+  - Scope: this override affects only this module’s sources; it does not change app-level logging.
+
+- Android
+  - Preferred: use `Logger` helper instead of `Log.*`.
+    - File: `android/src/main/java/com/margelo/nitro/audiorecorderplayer/Logger.kt`
+    - APIs: `Logger.d(...)`, `Logger.i(...)`, `Logger.w(...)`, `Logger.e(...)`
+  - Behavior: internally gated by `BuildConfig.DEBUG`, so logs print in Debug and are silent in Release. Avoid using `Log.*` directly to keep Release builds quiet.

NitroSound.podspec

@@ -28,6 +28,11 @@ Pod::Spec.new do |s|
   s.pod_target_xcconfig = {
     "SWIFT_VERSION" => "5.0",
     "SWIFT_ACTIVE_COMPILATION_CONDITIONS" => "$(inherited)",
+    # Enable library evolution to avoid certain Swift 6 IRGen issues
+    "BUILD_LIBRARY_FOR_DISTRIBUTION" => "YES",
+    "DEFINES_MODULE" => "YES",
+    # Favor whole-module to avoid per-file IRGen edge-cases
+    "SWIFT_COMPILATION_MODE" => "wholemodule",
     "HEADER_SEARCH_PATHS" => "$(inherited) ${PODS_ROOT}/RCT-Folly",
     "GCC_PREPROCESSOR_DEFINITIONS" => "$(inherited) FOLLY_NO_CONFIG FOLLY_MOBILE=1 FOLLY_USE_LIBCPP=1 FOLLY_CFG_NO_COROUTINES",
     "OTHER_CPLUSPLUSFLAGS" => "$(inherited) #{folly_compiler_flags}",

android/src/main/java/com/margelo/nitro/audiorecorderplayer/Logger.kt

@@ -0,0 +1,36 @@
+package com.margelo.nitro.audiorecorderplayer
+
+import android.util.Log
+import com.margelo.nitro.sound.BuildConfig
+
+/**
+ * Simple logger that only logs in Debug builds.
+ * Use Logger.d/i/w/e instead of Log.* to avoid logcat noise in Release.
+ */
+object Logger {
+  private const val TAG = "NitroSound"
+
+  @JvmStatic fun d(message: String, tr: Throwable? = null) {
+    log(Log.DEBUG, message, tr)
+  }
+
+  @JvmStatic fun i(message: String, tr: Throwable? = null) {
+    log(Log.INFO, message, tr)
+  }
+
+  @JvmStatic fun w(message: String, tr: Throwable? = null) {
+    log(Log.WARN, message, tr)
+  }
+
+  @JvmStatic fun e(message: String, tr: Throwable? = null) {
+    log(Log.ERROR, message, tr)
+  }
+
+  private fun log(priority: Int, message: String, tr: Throwable?) {
+    if (!BuildConfig.DEBUG) return
+    val fullMessage = if (tr != null) {
+      "$message\n${'$'}{Log.getStackTraceString(tr)}"
+    } else message
+    Log.println(priority, TAG, fullMessage)
+  }
+}

ios/Logging.swift

@@ -0,0 +1,6 @@
+// Suppress console logs in production (Release) builds only for this module
+#if !DEBUG
+@inline(__always)
+func print(_ items: Any..., separator: String = " ", terminator: String = "\n") {}
+#endif
+

ios/Sound.swift

@@ -2,7 +2,7 @@ import Foundation
 import AVFoundation
 import NitroModules
 
-class HybridSound: HybridSoundSpec {
+final class HybridSound: HybridSoundSpec_base, HybridSoundSpec_protocol {
     // Small delay to ensure the audio session is fully active before recording starts
     private let audioSessionActivationDelay: TimeInterval = 0.1
     private var audioRecorder: AVAudioRecorder?
@@ -841,7 +841,8 @@ class HybridSound: HybridSoundSpec {
                 recordSecs: currentTime
             )
 
-            print("🎙️ Timer callback: calling recordBackListener with data: \(recordBack)")
+            // Avoid interpolating RecordBackType directly to prevent Swift IRGen issues on Swift 6
+            print("🎙️ Timer callback: calling recordBackListener (time=\(currentTime)ms, metering=\(currentMetering))")
 
             if let listener = self.recordBackListener {
                 print("🎙️ Timer callback: recordBackListener exists, calling it")