Polish boost display and copy

[jeremy]

Summary

• Replace [♥ N] with plain text "N boosts" in muted style across list, kanban, and campfire views — eliminates Unicode ambiguous-width alignment issues
• Fetch individual boost content in detail view via Boosts().ListRecording(), showing emoji + booster name instead of just a count
• Align boost picker copy with BC3's framing ("Give a Boost!", "A short note or emoji…")
• Improve boost/react help text with TUI b key and react shortcut mentions

Test plan

• TestList_BoostWithExtra_Alignment verifies no line overflow with boost + extra columns
• TUI: boosted items in list/kanban show "N boosts" in muted text — verified via TestList_BoostSingularPlural, TestKanban_BoostDisplay
• TUI: detail view of boosted item shows individual emoji + booster names — verified via code inspection: syncPreview renders boostDetails with emoji+name, capped at 3
• TUI: boost picker shows updated labels and placeholder — verified via TestBoostPicker_ViewCopy, TestBoostPicker_Placeholder
• basecamp boost -h and basecamp react -h show TUI tip — verified via CLI output

[cubic-dev-ai]

5 issues found across 7 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="internal/tui/workspace/views/campfire.go">

<violation number="1" location="internal/tui/workspace/views/campfire.go:633">
P2: "1 boosts" is grammatically incorrect when `Count == 1`. Handle singular/plural (this applies to both the header and the grouped-message inline occurrence at line 643).</violation>
</file>

<file name="internal/tui/workspace/views/detail.go">

<violation number="1" location="internal/tui/workspace/views/detail.go:1310">
P2: Pluralization bug: when there is exactly 1 boost, this renders "1 boosts". Handle the singular case.</violation>
</file>

<file name="internal/tui/workspace/widget/list.go">

<violation number="1" location="internal/tui/workspace/widget/list.go:522">
P2: Pluralization bug: when `Boosts == 1` this renders "1 boosts". Use singular "boost" for a count of one.</violation>
</file>

<file name="internal/tui/workspace/widget/kanban.go">

<violation number="1" location="internal/tui/workspace/widget/kanban.go:524">
P2: "1 boosts" is grammatically incorrect. Handle the singular form when `card.Boosts == 1`.</violation>

<violation number="2" location="internal/tui/workspace/widget/kanban.go:559">
P2: Same singular/plural issue here: "1 boosts" when `card.Boosts == 1`.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[Copilot]

Pull request overview

This PR updates how boosts/reactions are displayed and described across the TUI and CLI help, with the goal of improving alignment/copy and showing richer boost detail in the recording detail view.

Changes:

• Replace the prior [♥ N] boost indicator with muted "N boosts" text in list/kanban/campfire displays.
• Enhance the detail view to optionally fetch and display per-boost content (emoji/text) plus booster name.
• Update boost picker and CLI help copy to better match Basecamp’s wording and include TUI keyboard tips.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
internal/tui/workspace/widget/list_test.go	Adds a regression test ensuring boost+extra columns don’t overflow list width.
internal/tui/workspace/widget/list.go	Updates list boost rendering and width calculations to use "N boosts" in muted style.
internal/tui/workspace/widget/kanban.go	Updates kanban boost strings in compact cards and detail line to "N boosts".
internal/tui/workspace/views/detail.go	Adds optional per-boost fetching via Boosts().ListRecording() and displays detailed boost info in the preview fields.
internal/tui/workspace/views/campfire.go	Changes campfire boost display to muted "N boosts" instead of [♥ N].
internal/tui/workspace/boostpicker.go	Updates boost picker title/placeholder/instructions to the new phrasing.
internal/commands/boost.go	Refreshes boost/react help text and adds a TUI tip about pressing b.

Comments suppressed due to low confidence (1)

internal/tui/workspace/widget/kanban.go:533

• PR description says boost counts should render in muted style across list/kanban/campfire, but in the kanban unfocused card rendering the boost suffix is not styled separately (it inherits the card's normal foreground). Consider rendering the boost suffix with theme.Muted (without muting the title) to match the intended UI change.

func (k *Kanban) renderUnfocusedCard(card KanbanCard, width int, theme tui.Theme) string {
	boostStr := ""
	if card.Boosts > 0 {
		boostStr = fmt.Sprintf(" %d boosts", card.Boosts)
	}
	availWidth := width - 2 - lipgloss.Width(boostStr) // 2 for "  " prefix
	title := Truncate(card.Title, availWidth)
	style := lipgloss.NewStyle().Width(width)
	if card.Completed {
		style = style.Foreground(theme.Muted).Strikethrough(true)
	}
	return style.Render("  " + title + boostStr)
}

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[cubic-dev-ai]

1 issue found across 5 files (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="internal/tui/workspace/views/detail.go">

<violation number="1" location="internal/tui/workspace/views/detail.go:1325">
P2: The "+N more" count uses `len(boostDetails)` instead of the authoritative `v.data.boosts`. If the API returns fewer boosts than the total count, the overflow label will undercount. Use `v.data.boosts - limit` for accuracy.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[Copilot]

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 3 comments.

Comments suppressed due to low confidence (1)

internal/tui/workspace/widget/kanban.go:533

• In kanban unfocused cards, the boost label is appended as plain text and inherits the card’s normal style, so it won’t appear “muted” like boosts in the list and campfire views. To match the intended UI and reduce visual noise, render the boost portion with a muted style (while keeping the width math consistent).

func (k *Kanban) renderUnfocusedCard(card KanbanCard, width int, theme tui.Theme) string {
	boostStr := ""
	if card.Boosts > 0 {
		boostStr = " " + boostLabel(card.Boosts)
	}
	availWidth := width - 2 - lipgloss.Width(boostStr) // 2 for "  " prefix
	title := Truncate(card.Title, availWidth)
	style := lipgloss.NewStyle().Width(width)
	if card.Completed {
		style = style.Foreground(theme.Muted).Strikethrough(true)
	}
	return style.Render("  " + title + boostStr)
}

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.