From 90f6ec9930ee40b9ca381edccc82387e6050a6fe Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E5=88=98=E7=9A=93?= <whiteaxe@tuta.io>
Date: Thu, 4 Jul 2024 15:21:48 -0400
Subject: [PATCH] Initialize `Audio` lazily in web builds (#137)

---
 crates/audio/src/wrapper.rs | 21 +++++++++++++++++----
 src/main.rs                 |  2 +-
 2 files changed, 18 insertions(+), 5 deletions(-)

diff --git a/crates/audio/src/wrapper.rs b/crates/audio/src/wrapper.rs
index cb5bd827..9e7afd76 100644
--- a/crates/audio/src/wrapper.rs
+++ b/crates/audio/src/wrapper.rs
@@ -72,8 +72,8 @@ enum AudioWrapperCommandInner {
 }
 
 impl AudioWrapper {
-    pub fn new(audio: Audio) -> Self {
-        audio.into()
+    pub fn new() -> Self {
+        Default::default()
     }
 
     pub fn play(
@@ -155,9 +155,15 @@ impl AudioWrapper {
     }
 }
 
-impl From<Audio> for AudioWrapper {
-    fn from(audio: Audio) -> Self {
+impl Default for AudioWrapper {
+    fn default() -> Self {
+        #[cfg(target_arch = "wasm32")]
+        if web_sys::window().is_none() {
+            panic!("in web builds, `AudioWrapper` can only be created on the main thread");
+        }
+
         let (tx, rx) = flume::unbounded::<AudioWrapperCommand>();
+        let mut maybe_audio = None;
 
         let promise = poll_promise::Promise::spawn_local(async move {
             loop {
@@ -165,6 +171,13 @@ impl From<Audio> for AudioWrapper {
                     return;
                 };
 
+                let audio = if let Some(audio) = &maybe_audio {
+                    audio
+                } else {
+                    maybe_audio = Some(Audio::default());
+                    maybe_audio.as_ref().unwrap()
+                };
+
                 match command.0 {
                     AudioWrapperCommandInner::Play {
                         cursor,
diff --git a/src/main.rs b/src/main.rs
index 6ed6708e..590beda6 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -397,7 +397,7 @@ pub fn luminol_main_start() {
 
     *WORKER_DATA.lock() = Some(WorkerData {
         report,
-        audio: luminol_audio::Audio::default().into(),
+        audio: luminol_audio::AudioWrapper::default(),
         modified: modified.clone(),
         prefers_color_scheme_dark,
         fs_worker_channels,