From 39f5d263f9accef4f131f8d0d2e4f62e2f76067c Mon Sep 17 00:00:00 2001
From: dakai <oudakai@gmail.com>
Date: Mon, 14 Oct 2024 02:33:28 +0800
Subject: [PATCH] follow the rabbit to fix potential issues

---
 app/components/chat.tsx | 12 +++++-------
 app/styles/globals.scss |  9 +++++----
 app/utils.ts            | 13 -------------
 app/utils/audio.ts      |  3 +++
 4 files changed, 13 insertions(+), 24 deletions(-)

diff --git a/app/components/chat.tsx b/app/components/chat.tsx
index 855b6bf3f..d3f568257 100644
--- a/app/components/chat.tsx
+++ b/app/components/chat.tsx
@@ -1122,12 +1122,11 @@ function _Chat() {
   };
 
   const updateMessageAudio = (msgId?: string, audio_url?: string) => {
-    chatStore.updateCurrentSession(
-      (session) =>
-        (session.messages = session.messages.map((m) =>
-          m.id === msgId ? { ...m, audio_url } : m,
-        )),
-    );
+    chatStore.updateCurrentSession((session) => {
+      session.messages = session.messages.map((m) =>
+        m.id === msgId ? { ...m, audio_url } : m,
+      );
+    });
   };
 
   const onDelete = (msgId: string) => {
@@ -1903,7 +1902,6 @@ function _Chat() {
                     )}
                     {message.audio_url && (
                       <audio
-                        id="audio"
                         preload="auto"
                         controls
                         className={styles["chat-message-item-audio"]}
diff --git a/app/styles/globals.scss b/app/styles/globals.scss
index fa7c0f2f2..3b8ded1f7 100644
--- a/app/styles/globals.scss
+++ b/app/styles/globals.scss
@@ -403,8 +403,9 @@ pre {
 audio {
   height: 35px;
 }
-
 audio::-webkit-media-controls-play-button,
- audio::-webkit-media-controls-panel {
- background-color: none;
- }
+audio::-webkit-media-controls-panel,
+audio::-moz-media-controls-play-button,
+audio::-moz-media-controls-panel {
+  background: none;
+}
diff --git a/app/utils.ts b/app/utils.ts
index 6966fb068..5d4501710 100644
--- a/app/utils.ts
+++ b/app/utils.ts
@@ -251,19 +251,6 @@ export function getMessageImages(message: RequestMessage): string[] {
   return urls;
 }
 
-//export function getMessageAudio(message: RequestMessage): string[] {
-//  if (typeof message.content === "string") {
-//    return [];
-//  }
-//  const urls: string[] = [];
-//  for (const c of message.content) {
-//    if (c.type === "image_url") {
-//      urls.push(c.image_url?.url ?? "");
-//    }
-//  }
-//  return urls;
-//}
-
 export function isVisionModel(model: string) {
   // Note: This is a better way using the TypeScript feature instead of `&&` or `||` (ts v5.5.0-dev.20240314 I've been using)
 
diff --git a/app/utils/audio.ts b/app/utils/audio.ts
index 9af258812..09453d156 100644
--- a/app/utils/audio.ts
+++ b/app/utils/audio.ts
@@ -90,6 +90,9 @@ export function arrayBufferToWav(buffer: ArrayBuffer): ArrayBuffer {
 
 // Helper function to write a string to the DataView
 function writeString(view: DataView, offset: number, string: string) {
+  if (offset + string.length > view.byteLength) {
+    throw new Error("String is too long for the available space in DataView");
+  }
   for (let i = 0; i < string.length; i++) {
     view.setUint8(offset + i, string.charCodeAt(i));
   }