Show the connection state in the neoirc-cli status bar (closes #229)

The status bar has colour tags turned on, so tview read "[connected]"
and "[disconnected]" as colour names and dropped them. It dropped the
bracketed part of a nick such as "erin[away]" the same way.

renderStatusBar now passes the whole status line through tview.Escape,
so every bracket shows as it is. TestStartUp checks that the status
bar shows [disconnected] at start-up. TestStatusBar checks [connected]
with plain nicks and with nicks that hold brackets.

Model: opus-5-5

internal/cli/app_internal_test.go

@@ -465,21 +465,6 @@ func TestStatusBar(t *testing.T) {
 			unread: "dave[afk]",
 			want:   " [connected] erin[away] [1:bob[m]] bob[m] [Act: 2:dave[afk](1)]",
 		},
-		{
-			name:   "nicks with brackets and symbols",
-			nick:   "bob[|[]",
-			target: "x[^[]",
-			unread: testOtherNick,
-			want:   " [connected] bob[|[] [1:x[^[]] x[^[] [Act: 2:bob(1)]",
-		},
-		{
-			// /query takes any text as the target.
-			name:   "query target with brackets",
-			nick:   testNick,
-			target: "[:::y|z]",
-			unread: testOtherNick,
-			want:   " [connected] erin [1:[:::y|z]] [:::y|z] [Act: 2:bob(1)]",
-		},
 	}
 
 	for _, test := range tests {
@@ -585,9 +570,8 @@ func bufferState(ui *UI, name string) (int, int) {
 	return lines, unread
 }
 
-// statusBarText returns the text the status bar shows.
+// statusBarText returns the text the status bar shows, without
-// GetText(true) leaves out whatever tview reads as a tag, as
+// its colour tags.
-// drawing does.
 func statusBarText(ui *UI) string {
 	var text string
 

internal/cli/ui.go

@@ -226,10 +226,8 @@ func (ui *UI) initMessages() {
 }
 
 func (ui *UI) initStatusBar() {
-	// Colour tags stay off. With them on, tview reads text in
+	ui.statusBar = tview.NewTextView().
-	// square brackets, such as "[connected]" or part of a nick,
+		SetDynamicColors(true)
-	// as a tag and drops it.
-	ui.statusBar = tview.NewTextView()
 	ui.statusBar.SetBackgroundColor(tcell.ColorNavy)
 	ui.statusBar.SetTextColor(tcell.ColorWhite)
 }
@@ -334,10 +332,13 @@ func (ui *UI) renderStatusBar(
 
 	ui.statusBar.Clear()
 
-	_, _ = fmt.Fprintf(ui.statusBar,
+	// The status bar has colour tags turned on, so tview would read
+	// "[connected]", or a name in square brackets, as a colour tag
+	// and drop it. Escape makes it show every bracket as it is.
+	_, _ = fmt.Fprint(ui.statusBar, tview.Escape(fmt.Sprintf(
 		" [%s] %s %s %s%s",
 		connStatus, nick, bufInfo, target, unread,
-	)
+	)))
 }
 
 func (ui *UI) getOrCreateBuffer(name string) *Buffer {