diff --git a/cmd/arcrun-tray/main.go b/cmd/arcrun-tray/main.go index 76181bc..405da3a 100644 --- a/cmd/arcrun-tray/main.go +++ b/cmd/arcrun-tray/main.go @@ -422,16 +422,30 @@ func main() { // t54 連線精靈:輸入知識庫網址+帳密 → 換設定 → 寫檔 → 開始看守。 // 沒連線過的機器一開 app 就自動跳;之後也能從選單「重新連線…」再開。 + // t75(2026-07-28,leo 同事真機實測):**連線失敗重試時必須把使用者剛打的內容帶回來**。 + // + // 原本的寫法是失敗後直接呼叫 `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) diff --git a/supervisor/hidewindow_other.go b/supervisor/hidewindow_other.go new file mode 100644 index 0000000..1d0ddee --- /dev/null +++ b/supervisor/hidewindow_other.go @@ -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) {} diff --git a/supervisor/hidewindow_windows.go b/supervisor/hidewindow_windows.go new file mode 100644 index 0000000..a952eaa --- /dev/null +++ b/supervisor/hidewindow_windows.go @@ -0,0 +1,31 @@ +//go:build windows + +package supervisor + +import ( + "os/exec" + "syscall" +) + +// CREATE_NO_WINDOW:起子行程時不要配一個新的 console 視窗。 +// +// 為什麼需要(t75,2026-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 +} diff --git a/supervisor/supervisor.go b/supervisor/supervisor.go index 57ccea4..7712ab8 100644 --- a/supervisor/supervisor.go +++ b/supervisor/supervisor.go @@ -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) + // t75:Windows 上不要讓 collector 彈出 console 視窗(每輪閃一次,使用者會以為中毒)。 + // 非 Windows 平台為 no-op,見 hidewindow_*.go。 + hideChildWindow(cmd) stdout, err := cmd.StdoutPipe() if err != nil { return err