fix: dispatch oauth safety timeout close to GTK main thread

time.AfterFunc runs on a Go goroutine. wv.Close() triggers
handlePopupClose which does GTK widget reparenting (unparent,
reparent, grab focus). GTK is not thread-safe - calling widget
operations from a non-main thread causes undefined behavior and
delayed segfaults.

Wrap in glib.IdleAdd to ensure the close runs on the GTK main loop.

Fixes: session 20260216_134851_baf8 crash

commit: 89f346b98a50c66331b839dede7df66a64109889
author: brice · 8mo ago · 2026-02-16 14:26 UTC
parent: f71271c
signature: GPG VerifiedCD88F9E564E0102646FB6077A1A07C7C7C3E349D · arch-gpg Verified for the committer email

1 file · +8 -4

M internal/ui/coordinator/content.go +8 -4


@@ -1807,10 +1807,14 @@ func (c *ContentCoordinator) setupOAuthAutoClose(
1807	1807		safetyTimer.Stop()
1808	1808		}
1809	1809		safetyTimer = time.AfterFunc(oauthSafetyTimeout, func() {
1810		-	if wv != nil && !wv.IsDestroyed() {
1811		-	log.Warn().Str("pane", string(paneID)).Msg("oauth safety timeout, closing stuck popup")
1812		-	wv.Close()
1813		-	}
	1810	+	cb := glib.SourceFunc(func(_ uintptr) bool {
	1811	+	if wv != nil && !wv.IsDestroyed() {
	1812	+	log.Warn().Str("pane", string(paneID)).Msg("oauth safety timeout, closing stuck popup")
	1813	+	wv.Close()
	1814	+	}
	1815	+	return false
	1816	+	})
	1817	+	glib.IdleAdd(&cb, 0)
1814	1818		})
1815	1819		}
1816	1820

 			safetyTimer.Stop()
 		}
 		safetyTimer = time.AfterFunc(oauthSafetyTimeout, func() {
-			if wv != nil && !wv.IsDestroyed() {
+			cb := glib.SourceFunc(func(_ uintptr) bool {
-				log.Warn().Str("pane", string(paneID)).Msg("oauth safety timeout, closing stuck popup")
+				if wv != nil && !wv.IsDestroyed() {
-				wv.Close()
+					log.Warn().Str("pane", string(paneID)).Msg("oauth safety timeout, closing stuck popup")
-			}
+					wv.Close()
+				}
+				return false
+			})
+			glib.IdleAdd(&cb, 0)
 		})
 	}