Fix g10k hang on exit (darwin/arm64) — close /dev/tty fd in vendored uilive

[Copilot]

uilive v0.0.4 introduced terminal overflow handling that opens /dev/tty via os.OpenFile and stores the handle in a package-level variable that is never closed. On darwin/arm64 with recent Go versions, this leaked *os.File (and its associated GC finalizer) causes os.Exit to deadlock: the main goroutine becomes runnable but no OS thread picks it up, hanging indefinitely.

Change

vendor/github.com/gosuri/uilive/terminal_size.go

• Moved out, err, and sz from package-level variables to locals inside getTermSize()
• Added defer f.Close() so the /dev/tty handle is released immediately after the ioctl call

// Before — fd leaked into a package-level var, never closed
var out *os.File
func getTermSize() (int, int) {
    out, err = os.OpenFile("/dev/tty", os.O_WRONLY, 0)
    // ... out never closed
}

// After — fd is local and deferred-closed
func getTermSize() (int, int) {
    f, err := os.OpenFile("/dev/tty", os.O_WRONLY, 0)
    if err != nil { return 0, 0 }
    defer f.Close()
    // ...
}

This matches the behaviour of uilive v0.0.3 (which had no terminal_size.go at all) and unblocks darwin/arm64 builds without requiring a version downgrade or removal of the library.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• forgeapi.puppet.com
  • Triggering command: /tmp/go-build1356479692/b001/g10k.test /tmp/go-build1356479692/b001/g10k.test -test.run=TestConfigGlobalAllowFail$ (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[mxey]

I don't think memory corruption is caused by a leaked file descriptor…

[TheMeier]

This is a vendored in dependency. Maybe we should look into getting rid of it, it is used by https://github.com/xorpaul/uiprogress.
I personally do not think g10k needs a progress bar at all.