fix(t75 ④⑤): 連線精靈不再清空輸入+Windows 不再每幾秒閃 console 視窗

leo 同事真機實測撞到的兩個最嚴重問題。

④ 連線失敗會清空使用者輸入(leo:「很糟糕」)
   真兇:失敗後呼叫 showConnectWizard(),註解寫著「不要把他丟回空白畫面」
   **但行為與註解相反**——它重建整個對話框,urlEntry 是全新的,
   而 cfg.CypherURL 因連線失敗根本沒寫入 ⇒ 欄位就是空的 ⇒ 使用者整段重打。
   修:改 showConnectWizardWith(prevURL, prevEmail) 帶回上次輸入,
   讓他只改錯的那個字。密碼不帶回(打錯時重打較安全,也少留記憶體)。

⑤ daemon 開著時每幾秒閃一個方框(約半螢幕)
   真兇:arcrun-collector.exe 是 console 型執行檔,supervisor 每輪 exec 起它,
   **Windows 預設會為 console 程式配可見視窗** ⇒ 每輪閃一次。
   Mac/Linux 無此概念故開發期完全沒想到。
   修:新增 hidewindow_windows.go(HideWindow + CREATE_NO_WINDOW)
   +hidewindow_other.go(no-op),編譯期分平台不寫 runtime.GOOS 判斷。
   註:-H windowsgui 是連結期選項(管自己有沒有視窗),與此(管起別人時給不給視窗)
   是兩件事,兩者都要做才乾淨。

驗證:Mac 與 Windows 交叉編譯皆 exit=0 且產物生成;collector 全測試綠。
⚠️ 未真機驗——⑤ 的效果必須在真 Windows 上確認方框消失。
This commit is contained in:
2026-07-28 00:42:12 +08:00
parent 1b58cd1a35
commit 59cf9825d1
4 changed files with 63 additions and 4 deletions
+19 -4
View File
@@ -422,16 +422,30 @@ func main() {
// t54 連線精靈:輸入知識庫網址+帳密 → 換設定 → 寫檔 → 開始看守。
// 沒連線過的機器一開 app 就自動跳;之後也能從選單「重新連線…」再開。
// t752026-07-28leo 同事真機實測):**連線失敗重試時必須把使用者剛打的內容帶回來**。
//
// 原本的寫法是失敗後直接呼叫 `showConnectWizard()`,註解寫著「不要把他丟回空白畫面」——
// **但行為與註解相反**:它重建整個對話框、`urlEntry` 是全新的,而 `cfg.CypherURL`
// 因為連線失敗根本沒被寫入 ⇒ 欄位就是空的 ⇒ **使用者得整段重打**。
// leo 原話:「貼上網址後說這不是個網址然後清空內容,**很糟糕**」。
// ⇒ 改成帶入上次輸入(prefill)。密碼不帶回(打錯密碼時重打比較安全,也避免把密碼留在記憶體更久)。
var showConnectWizard func()
showConnectWizard = func() {
var showConnectWizardWith func(prevURL, prevEmail string)
showConnectWizard = func() { showConnectWizardWith("", "") }
showConnectWizardWith = func(prevURL, prevEmail string) {
urlEntry := widget.NewEntry()
urlEntry.SetPlaceHolder("https://…workers.dev/portal/")
if cfg.CypherURL != "" {
// 優先用「使用者剛才打的」,其次才是已存的設定——重試時他要看到的是自己的輸入。
if prevURL != "" {
urlEntry.SetText(prevURL)
} else if cfg.CypherURL != "" {
urlEntry.SetText(cfg.CypherURL)
}
emailEntry := widget.NewEntry()
emailEntry.SetPlaceHolder("你的 Email")
if cfg.Email != "" {
if prevEmail != "" {
emailEntry.SetText(prevEmail)
} else if cfg.Email != "" {
emailEntry.SetText(cfg.Email)
}
pwEntry := widget.NewPasswordEntry()
@@ -454,7 +468,8 @@ func main() {
if err != nil {
// fyne 的 UI 更新須回主執行緒;用 dialog 顯示即可(它內部處理)
dialog.ShowError(err, win)
showConnectWizard() // 讓用戶改完再試,不要把他丟回空白畫面
// t75:把剛才打的網址與 email 帶回去,讓他只改錯的那個字,不必整段重打。
showConnectWizardWith(urlEntry.Text, emailEntry.Text)
return
}
applyRemoteConfig(cfg, r)
+10
View File
@@ -0,0 +1,10 @@
//go:build !windows
package supervisor
import "os/exec"
// hideChildWindow 在非 Windows 平台是 no-op
// Unix 系沒有「起子行程會配一個視窗」這回事(那是 Windows console subsystem 的行為)。
// 保留同名函式是為了讓 supervisor.go 不必寫 runtime.GOOS 判斷(編譯期就分好)。
func hideChildWindow(_ *exec.Cmd) {}
+31
View File
@@ -0,0 +1,31 @@
//go:build windows
package supervisor
import (
"os/exec"
"syscall"
)
// CREATE_NO_WINDOW:起子行程時不要配一個新的 console 視窗。
//
// 為什麼需要(t752026-07-28 leo 同事在真 Windows 上實測撞到):
// 使用者回報「**daemon 開啟時每幾秒螢幕會閃一個方框,大概有螢幕一半大,關閉它就消失**」。
// 根因=`arcrun-collector.exe` 是 **console 型執行檔**`PE32+ (console)`),
// supervisor 每輪起它一次,**Windows 預設會為 console 程式配一個可見的視窗** ⇒ 每輪閃一下。
//
// **Mac/Linux 沒有這個概念**,所以這個跨平台差異在開發期完全沒被想到——
// 這也是為什麼「真機驗證」不能只驗自己改過的東西(見 t75 教訓)。
//
// 注意:`-H windowsgui` 是**連結期**選項,只影響「這支程式自己有沒有視窗」;
// 這裡處理的是「**我起別人時要不要給它視窗**」,是不同一件事,兩者都要做才乾淨。
const createNoWindow = 0x08000000
// hideChildWindow 讓子行程不要彈出 console 視窗(僅 Windows 有效)。
func hideChildWindow(cmd *exec.Cmd) {
if cmd.SysProcAttr == nil {
cmd.SysProcAttr = &syscall.SysProcAttr{}
}
cmd.SysProcAttr.HideWindow = true
cmd.SysProcAttr.CreationFlags |= createNoWindow
}
+3
View File
@@ -251,6 +251,9 @@ func (s *Supervisor) loop(ctx context.Context) {
// runOnce 跑一次 collector direct 行程,串流其 stdout JSON 更新狀態,回傳退出原因。
func (s *Supervisor) runOnce(ctx context.Context) error {
cmd := exec.CommandContext(ctx, s.BinPath, "direct", "--config", s.ConfigPath)
// t75Windows 上不要讓 collector 彈出 console 視窗(每輪閃一次,使用者會以為中毒)。
// 非 Windows 平台為 no-op,見 hidewindow_*.go。
hideChildWindow(cmd)
stdout, err := cmd.StdoutPipe()
if err != nil {
return err