Skip column truncation when stdout is piped (non-TTY)

[sjmiller609]

Summary

Fixes CLI table output truncation when stdout is piped to other tools (grep, awk, cat, etc.).

Before: kernel browsers list | grep hype returned nothing because URLs and other columns were truncated even in pipe mode.

After:

• TTY (interactive terminal): columns truncate to fit terminal width (unchanged)
• Non-TTY (piped): full column values output without truncation

This follows the standard CLI pattern used by docker ps, kubectl get, etc.

Changes

• pkg/table/table.go: Skip truncateTableData when stdout is not a TTY; add IsStdoutTTY() helper
• cmd/browsers.go: truncateURL skips truncation in non-TTY mode

Test plan

• kernel browsers list in terminal — columns still truncate to fit width
• kernel browsers list | cat — full URLs visible, no truncation
• kernel browsers list | grep <pattern> — matches found in full output
• All existing tests pass (go test ./...)

---

Note

Low Risk
Low risk: changes are limited to CLI formatting behavior and only affect output when stdout is not a terminal (piped), with no impact on API calls or data mutation.

Overview
Stops truncating CLI output when stdout is piped (non-TTY) so full column values are preserved for tools like grep/awk.

pkg/table.PrintTableNoPad now truncates columns only for terminal output and adds a shared table.IsStdoutTTY() helper; cmd/browsers.go’s truncateURL also skips truncation when not on a TTY.

^{Reviewed by Cursor Bugbot for commit b003810. Bugbot is set up for automated code reviews on this repo. Configure here.}

[firetiger-agent]

Firetiger deploy monitoring skipped

This PR didn't match the auto-monitor filter configured on your GitHub connection:

Any PR that changes the kernel API. Monitor changes to API endpoints (packages/api/cmd/api/) and Temporal workflows (packages/api/lib/temporal) in the kernel repo

Reason: This PR modifies CLI output formatting in the browsers command, not the kernel API endpoints or Temporal workflows specified in the filter.

To monitor this PR anyway, reply with @firetiger monitor this.

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit b003810. Configure here.}

[cursor]

Duplicated TTY check instead of using new helper

Low Severity

PrintTableNoPad calls term.IsTerminal(int(os.Stdout.Fd())) inline at the same time the new IsStdoutTTY() helper — which does exactly the same thing — is introduced in the same file. The helper exists specifically to centralize this check and is already exported for use by cmd/browsers.go, but PrintTableNoPad doesn't use it. If the TTY detection logic ever changes, it would need updating in two places.

Additional Locations (1)

• pkg/table/table.go#L97-L101

 

^{Reviewed by Cursor Bugbot for commit b003810. Configure here.}