ONCE TUI panics if I run `once teardown` in another shell

[amiorin]

The expected behavior should be different.

Caught panic:

runtime error: index out of range [0] with length 0

Restoring terminal...

goroutine 1 [running]:
runtime/debug.Stack()
        runtime/debug/stack.go:26 +0x64
charm.land/bubbletea/v2.(*Program).recoverFromPanic(0x7f8a6bad83c0, {0x73bb40, 0x7f8a6c204018})
        charm.land/bubbletea/v2@v2.0.2/tea.go:1260 +0xec
charm.land/bubbletea/v2.(*Program).Run.func2()
        charm.land/bubbletea/v2@v2.0.2/tea.go:1011 +0xec
panic({0x73bb40?, 0x7f8a6c204018?})
        runtime/panic.go:860 +0x12c
github.com/basecamp/once/internal/ui.(*Dashboard).scrollToSelection(0x7f8a6bc03008?)
        github.com/basecamp/once/internal/ui/dashboard.go:359 +0x1a8
github.com/basecamp/once/internal/ui.(*Dashboard).selectPanel(0x7f8a6bc03008, 0x0?)
        github.com/basecamp/once/internal/ui/dashboard.go:312 +0x44
github.com/basecamp/once/internal/ui.Dashboard.Update({0x7f8a6bca62c0, 0x7f8a6bb64dc0, 0x7f8a6bb00a20, 0x7f8a6baba700, 0x7f8a6baa08c0, {0x7f8a6bb14198, 0x1, 0x1}, {0x7f8a6bb3eb00, 0x1, ...}, ...}, ...)
        github.com/basecamp/once/internal/ui/dashboard.go:233 +0x15b0
github.com/basecamp/once/internal/ui.(*App).Update(0x7f8a6baf9b80, {0x6a4a60?, 0xe733c0})
        github.com/basecamp/once/internal/ui/app.go:179 +0x758
charm.land/bubbletea/v2.(*Program).eventLoop(0x7f8a6bad83c0, {0x7dec00?, 0x7f8a6baf9b80?}, 0x7f8a6baaab60)
        charm.land/bubbletea/v2@v2.0.2/tea.go:853 +0xb14
charm.land/bubbletea/v2.(*Program).Run(0x7f8a6bad83c0)
        charm.land/bubbletea/v2@v2.0.2/tea.go:1125 +0x78c
github.com/basecamp/once/internal/ui.Run(0x7f8a6bca62c0, {0x0, 0x0})
        github.com/basecamp/once/internal/ui/app.go:281 +0x248
github.com/basecamp/once/internal/command.NewRootCommand.func1.1()
        github.com/basecamp/once/internal/command/root.go:32 +0x24
github.com/basecamp/once/internal/logging.ToLogFile(0x7f8a6bc5fbe0)
        github.com/basecamp/once/internal/logging/log.go:34 +0x2b0
github.com/basecamp/once/internal/command.NewRootCommand.func1({0x7e0478?, 0xe733c0?}, 0x78f9bf?, 0x0?, {0x0?, 0x0?, 0x0?})
        github.com/basecamp/once/internal/command/root.go:31 +0x34
github.com/basecamp/once/internal/command.NewRootCommand.WithNamespace.func2(0x7f8a6bca8008, {0xe733c0, 0x0, 0x0})
        github.com/basecamp/once/internal/command/root.go:86 +0xbc
github.com/spf13/cobra.(*Command).execute(0x7f8a6bca8008, {0x7f8a6bb1c030, 0x0, 0x0})
        github.com/spf13/cobra@v1.10.2/command.go:1015 +0x814
github.com/spf13/cobra.(*Command).ExecuteC(0x7f8a6bca8008)
        github.com/spf13/cobra@v1.10.2/command.go:1148 +0x350
github.com/spf13/cobra.(*Command).Execute(...)
        github.com/spf13/cobra@v1.10.2/command.go:1071
github.com/basecamp/once/internal/command.(*RootCommand).Execute(...)
        github.com/basecamp/once/internal/command/root.go:57
main.main()
        github.com/basecamp/once/cmd/once/main.go:13 +0x28

Error: program was killed: program experienced a panic

[monorkin]

Oh, looks like this is because of the selected panel handler. It crashes if you remove the selected element via an external event. I'll work on a fix shortly.

[monorkin]

@amiorin I pushed a fix for that bug. Can you confirm it works for you?

[amiorin]

I confirm, thank you.