TUI: Today tab leaks raw reward enums and disagrees on bounty math #126

New issue

Closed

opened 2026-08-18 10:45:24 +00:00 by nalum · 1 comment

nalum commented

2026-08-18 10:45:24 +00:00

Owner

From the surface parity report (2026-08-18), defect 16.

tui/today.go:557-566 bypasses rewardLine and prints the raw enum tail uppercase (PRIVILEGE, CURRENCY) where the Rewards tab prints "privilege". Currency rewards lose their € prefix.
Bounty math disagrees between tabs: today.go:155 worthOf sums all linked POINTS rewards, jobs.go:112 rewardFor takes only the first. The same job can show two different point values on two tabs.

Fix: hoist rewardLine and one worth function; both tabs consume them (see the TUI consolidation issue).

From the surface parity report (2026-08-18), defect 16. 1. `tui/today.go:557-566` bypasses `rewardLine` and prints the raw enum tail uppercase (`PRIVILEGE`, `CURRENCY`) where the Rewards tab prints "privilege". Currency rewards lose their `€` prefix. 2. Bounty math disagrees between tabs: `today.go:155` `worthOf` sums all linked POINTS rewards, `jobs.go:112` `rewardFor` takes only the first. The same job can show two different point values on two tabs. Fix: hoist `rewardLine` and one worth function; both tabs consume them (see the TUI consolidation issue).

nalum added the

bug

label

2026-08-18 10:45:24 +00:00

nalum referenced this issue

2026-08-18 11:22:44 +00:00

Order of work for the open backlog (#107–#136) #137

nalum referenced this issue from a commit

2026-08-18 14:27:10 +00:00

refactor(tui): shared row renderers, one bounty sum

nalum referenced this issue

2026-08-18 14:27:24 +00:00

refactor: TUI shared row renderers, one bounty sum #147

nalum commented

2026-08-18 16:47:11 +00:00

Author

Owner

Delivered in PR #147 (merged): the TUI's shared rows.go renders rewards through rewardLine (plain-language kinds, no raw enums) and worthOf sums bounty math the same way the other surfaces do. tmux-verified live.

nalum closed this issue

2026-08-18 16:47:11 +00:00

nalum referenced this issue

2026-08-18 16:47:30 +00:00

Order of work for the open backlog (#107–#136) #137