feat: Improve gain command display

[jspelletier]

• Detect terminal width and adapt the command column dynamically
• Add --nocommand-truncate flag to disable truncation entirely
• Update help text and README

[edouard-claude]

Hey @jspelletier,

Thanks for this PR -- dynamic terminal width detection is a real improvement over the hardcoded truncation values.

Two things before merging:

1. Missing tests

cmdColWidth is a pure function that's trivial to test -- it would be good to have at least a few cases (minimum width clamped to 20, nominal case, narrow terminal). The --nocommand-truncate path in showByCommand and showHistory could also use some coverage. You had solid tests on PR #28 (TestHookScriptNoDoubleRewrite, TestGenerateHookScriptEmbedsBin...), it would be nice to keep that bar here too.

snip/internal/display/gain.go

Lines 162 to 168 in 6144bd7

	// cmdColWidth returns the Command column truncation width given terminal width
	// and the total width consumed by all other columns + separators.
	func cmdColWidth(termWidth, fixedWidth int) int {
	w := termWidth - fixedWidth
	if w < 20 {
	return 20
	}

2. Flag naming

--nocommand-truncate reads a bit ambiguously ("nocommand" + "truncate"? "no" + "command-truncate"?) and breaks from the kebab-case convention used by other gain flags (--daily, --weekly, --top, --history...). Would --no-truncate work for you? Shorter, clearer, and consistent with the rest.

snip/internal/display/gain.go

Lines 63 to 64 in 6144bd7

	case "--nocommand-truncate":
	noTruncate = true

[jspelletier]

Ok. I will ask Claude to add some tests. --no-truncate is ok for me.

[edouard-claude]

Thanks @jspelletier -- tests and flag rename look great, CI is green. Merging!

[jspelletier]

@edouard-claude Yesterday night I saw there was a CI problem with the w.Close() result not tested. You fixed it ?

[Edit]

Ok I saw you assigned to _.

Thanks