Update linux macos

[taotieren]

No description provided.

[gemini-code-assist]

Code Review

This pull request ports the LLCOM serial debugging tool to Avalonia (.NET 8) for cross-platform support, introducing various views, view models, and a refactored Rust-based serial monitor library. The review feedback highlights several critical issues and improvement opportunities: cross-thread UI updates in the main view model, an infinite loop hazard on TCP connection closure, and missing legacy encoding registration at startup. Additionally, recommendations are provided to optimize hex parsing, improve send buffer queue efficiency, fix unawaited tasks in the Lua runner, prevent potential out-of-bounds exceptions during PID parsing, and avoid file-locking issues when extracting the Windows hook DLL.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[gemini-code-assist]

Modifying ReceivedLines (an ObservableCollection bound to the UI) directly from the background thread in OnDataReceived will cause cross-thread exceptions or UI instability in Avalonia. You must marshal these modifications to the UI thread using Dispatcher.UIThread.Post.

    private void AppendDataLine(DataLineItem line)
    {
        global::Avalonia.Threading.Dispatcher.UIThread.Post(() =>
        {
            ReceivedLines.Add(line);
            while (ReceivedLines.Count > _maxLines)
                ReceivedLines.RemoveAt(0);
        });
    }

[gemini-code-assist]

When the remote host gracefully closes the TCP connection, ReadAsync returns 0. If you do not handle this case, the loop will spin infinitely, consuming 100% CPU. You should break the loop and set IsConnected = false when count == 0.

                var count = await _tcpClient.GetStream().ReadAsync(buffer, ct);
                if (count == 0)
                {
                    IsConnected = false;
                    break;
                }
                if (count > 0)
                {
                    var data = new byte[count];
                    Array.Copy(buffer, data, count);
                    global::Avalonia.Threading.Dispatcher.UIThread.Post(() =>
                        AppendReceive($"→ {Encoding.UTF8.GetString(data)}"));
                }
            }

[gemini-code-assist]

In .NET Core/.NET 8, legacy encodings (such as GBK, Shift_JIS, Big5, etc.) are not registered by default. To prevent silent failures or exceptions when the application attempts to resolve these encodings (e.g., in the "Garbled code fix" tool), you must register the code pages provider at application startup.

    public static void Main(string[] args)
    {
        System.Text.Encoding.RegisterProvider(System.Text.CodePagesEncodingProvider.Instance);
        BuildAvaloniaApp().StartWithClassicDesktopLifetime(args);
    }

[gemini-code-assist]

Instead of manually parsing the hex string using a loop with Substring (which allocates a new string for every byte and is highly inefficient), you can use the built-in Convert.FromHexString method after sanitizing the string.

    private static byte[] HexToBytes(string hex)
    {
        hex = System.Text.RegularExpressions.Regex.Replace(hex, "[^0-9A-Fa-f]", "");
        if (hex.Length % 2 != 0) hex = hex[..^1];
        return Convert.FromHexString(hex);
    }

[gemini-code-assist]

Using a List<byte[]> for _sendBuffer and calling RemoveAt(0) is highly inefficient because removing the first element of a list requires shifting all subsequent elements, resulting in O(N) complexity. Using a Queue<byte[]> provides O(1) complexity for enqueue and dequeue operations.

    private readonly Queue<byte[]> _sendBuffer = new();

[gemini-code-assist]

Task.Delay(200).ContinueWith(_ => { }); is not awaited and does not block or delay the execution of the current synchronous method ToggleRun. If you intended to introduce a delay before setting IsRunning = true, you should make ToggleRun an async Task and await the delay.

                await Task.Delay(200);

[gemini-code-assist]

The current substring parsing logic for extracting the PID can throw an ArgumentOutOfRangeException if the SelectedProcess string does not strictly conform to the expected format (e.g., if it contains [ but no closing ], or ends prematurely). Using AsSpan with LastIndexOf is safer, more robust, and avoids string allocations.

            var start = SelectedProcess.IndexOf('[');
            var end = SelectedProcess.LastIndexOf(']');
            if (start < 0 || end <= start + 1 || !uint.TryParse(
                SelectedProcess.AsSpan(start + 1, end - start - 1),
                out var pid))
            {
                StatusText = LocaleHelper.Get("SerialMonitorInvalidPid");
                return;
            }

[gemini-code-assist]

Writing the embedded hook DLL to the temp directory on every connection attempt will fail with a sharing/locking violation if another instance of the application is running or if the DLL is still loaded/locked by a monitored process. You should check if the file already exists before attempting to write it.

Suggested change

	fn extract_hook_dll(path: &PathBuf) -> bool {
	std::fs::write(path, HOOK_DLL_BYTES).is_ok()
	}
	fn extract_hook_dll(path: &PathBuf) -> bool {
	if path.exists() {
	return true;
	}
	std::fs::write(path, HOOK_DLL_BYTES).is_ok()
	}

[chenxuuu]

¿

[taotieren]

¿  什么？

用 vibi 跑的移植，等 vibe 改完再推送新的

[chenxuuu]

现在模型能力还是很垃圾的，搞不了这种大型项目，我这个月尝试过，不太行，还是用的官渠opus模型

[taotieren]

现在模型能力还是很垃圾的，搞不了这种大型项目，我这个月尝试过，不太行，还是用的官渠opus模型

前面喂点数据，创建任务后进行几次迭代，应该能满足使用要求。

[taotieren]

现在模型能力还是很垃圾的，搞不了这种大型项目，我这个月尝试过，不太行，还是用的官渠opus模型

可以找一个找一个 UI 布局不错的界面喂给 AI ，让 AI 按你要求的 UI 完成界面和功能难度会降低很多。

[chenxuuu]

那你慢慢跑
我感觉两年内应该没有什么模型可以在完全没有人类监督的情况下完整重构出能用的llcom版本