Fix AT32 Configuration can't be saved on MacOS (#4455)

* Fix AT32 Configuration can't be saved on MacOS

* Update src/js/protocols/WebSerial.js

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

* Change the location of the macOS judgment

* Adding logging for batch write mode detection

* Refactoring for MacOS detection

* fix GUI is not exported

* Change this.logHead to logHead

---------

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

Author: kedeng

src/js/protocols/WebSerial.js

@@ -1,5 +1,6 @@
 import { webSerialDevices, vendorIdNames } from "./devices";
 import { checkBrowserCompatibility } from "../utils/checkBrowserCompatibilty";
+import GUI from "../gui";
 
 const logHead = "[SERIAL]";
 
@@ -69,7 +70,7 @@ class WebSerial extends EventTarget {
             navigator.serial.addEventListener("connect", (e) => this.handleNewDevice(e.target));
             navigator.serial.addEventListener("disconnect", (e) => this.handleRemovedDevice(e.target));
         }
-
+        this.isNeedBatchWrite = false;
         this.loadDevices();
     }
 
@@ -181,6 +182,10 @@ class WebSerial extends EventTarget {
 
             const connectionInfo = this.port.getInfo();
             this.connectionInfo = connectionInfo;
+            this.isNeedBatchWrite = this.checkIsNeedBatchWrite();
+            if (this.isNeedBatchWrite) {
+                console.log(`${logHead} Enabling batch write mode for AT32 on macOS`);
+            }
             this.writer = this.port.writable.getWriter();
             this.reader = this.port.readable.getReader();
 
@@ -328,6 +333,29 @@ class WebSerial extends EventTarget {
         }
     }
 
+    checkIsNeedBatchWrite() {
+        const isMac = GUI.operating_system === "MacOS";
+        return isMac && vendorIdNames[this.connectionInfo.usbVendorId] === "AT32";
+    }
+
+    async batchWrite(data) {
+        // AT32 on macOS requires smaller chunks (63 bytes) to work correctly due to
+        // USB buffer size limitations in the macOS implementation
+        const batchWriteSize = 63;
+        let remainingData = data;
+        while (remainingData.byteLength > batchWriteSize) {
+            const sliceData = remainingData.slice(0, batchWriteSize);
+            remainingData = remainingData.slice(batchWriteSize);
+            try {
+                await this.writer.write(sliceData);
+            } catch (error) {
+                console.error(`${logHead} Error writing batch chunk:`, error);
+                throw error; // Re-throw to be caught by the send method
+            }
+        }
+        await this.writer.write(remainingData);
+    }
+
     async send(data, callback) {
         if (!this.connected || !this.writer) {
             console.error(`${logHead} Failed to send data, serial port not open`);
@@ -338,7 +366,11 @@ class WebSerial extends EventTarget {
         }
 
         try {
-            await this.writer.write(data);
+            if (this.isNeedBatchWrite) {
+                await this.batchWrite(data);
+            } else {
+                await this.writer.write(data);
+            }
             this.bytesSent += data.byteLength;
 
             const result = { bytesSent: data.byteLength };

src/js/utils/checkBrowserCompatibilty.js

@@ -6,7 +6,7 @@ export function getOS() {
     let os = "unknown";
     const userAgent = window.navigator.userAgent;
     const platform = window.navigator?.userAgentData?.platform || window.navigator.platform;
-    const macosPlatforms = ["Macintosh", "MacIntel", "MacPPC", "Mac68K", "MacOS"];
+    const macosPlatforms = ["Macintosh", "MacIntel", "MacPPC", "Mac68K", "macOS"];
     const windowsPlatforms = ["Win32", "Win64", "Windows", "WinCE"];
     const iosPlatforms = ["iPhone", "iPad", "iPod"];