Truncate long columns instead of line wrapping (#26)

Detecting terminal size, use smart column truncation. Seek to avoid
truncating shorter columns.
<img width="3420" height="378" alt="image"
src="https://github.com/user-attachments/assets/1e62dbf3-2d64-4aea-9e0d-1941e4a6300a"
/>

---

<!-- mesa-description-start -->
## TL;DR

Replaced line wrapping with smart column truncation for CLI tables,
which now adapt to the terminal's width to improve readability.

## Why we made these changes

Previously, wide tables would wrap lines, creating a messy and
difficult-to-read layout. This change ensures tables maintain a clean,
grid-like structure by truncating overflowing content, making the output
much easier to scan.

## What changed?

- **`cmd/table.go`**: Implemented intelligent table truncation logic
that detects terminal width to prevent line wrapping. A new algorithm
proportionally truncates the longest columns first while correctly
handling Unicode characters and ANSI color codes.
- **`cmd/proxies/get.go`**: Added color-coded status and a 'Last
Checked' timestamp to the proxy details output.
- **`cmd/app.go`**: Removed truncation for environment variable and
action names to ensure their full values are always displayed.
- **`cmd/proxies/list_test.go`**: Updated tests to reflect the new
'Status' column in the proxy list output.

## Validation

- [ ] Verified tables render correctly on different terminal widths.
- [ ] Confirmed that only necessary columns are truncated.
- [ ] Tested with tables that have no long columns to ensure no change
in behavior.

<sup>_Description generated by Mesa. [Update
settings](https://app.mesa.dev/onkernel/settings/pull-requests)_</sup>
<!-- mesa-description-end -->

Author: sjmiller609

cmd/app.go

@@ -11,9 +11,10 @@ import (
 )
 
 var appCmd = &cobra.Command{
-	Use:   "app",
-	Short: "Manage deployed applications",
-	Long:  "Commands for managing deployed Kernel applications",
+	Use:     "app",
+	Aliases: []string{"apps"},
+	Short:   "Manage deployed applications",
+	Long:    "Commands for managing deployed Kernel applications",
 }
 
 // --- app list subcommand
@@ -80,19 +81,13 @@ func runAppList(cmd *cobra.Command, args []string) error {
 		envVarsStr := "-"
 		if len(app.EnvVars) > 0 {
 			envVarsStr = strings.Join(lo.Keys(app.EnvVars), ", ")
-			if len(envVarsStr) > 50 {
-				envVarsStr = envVarsStr[:47] + "..."
-			}
 		}
 
 		actionsStr := "-"
 		if len(app.Actions) > 0 {
 			actionsStr = strings.Join(lo.Map(app.Actions, func(a kernel.AppAction, _ int) string {
 				return a.Name
 			}), ", ")
-			if len(actionsStr) > 50 {
-				actionsStr = actionsStr[:47] + "..."
-			}
 		}
 
 		tableData = append(tableData, []string{
@@ -105,7 +100,7 @@ func runAppList(cmd *cobra.Command, args []string) error {
 		})
 	}
 
-	printTableNoPad(tableData, true)
+	PrintTableNoPad(tableData, true)
 	return nil
 }
 
@@ -156,6 +151,6 @@ func runAppHistory(cmd *cobra.Command, args []string) error {
 		}
 	}
 
-	printTableNoPad(tableData, true)
+	PrintTableNoPad(tableData, true)
 	return nil
 }

cmd/browsers.go

@@ -144,7 +144,7 @@ func (b BrowsersCmd) List(ctx context.Context) error {
 		})
 	}
 
-	printTableNoPad(tableData, true)
+	PrintTableNoPad(tableData, true)
 	return nil
 }
 
@@ -208,7 +208,7 @@ func (b BrowsersCmd) Create(ctx context.Context, in BrowsersCreateInput) error {
 		tableData = append(tableData, []string{"Profile", profVal})
 	}
 
-	printTableNoPad(tableData, true)
+	PrintTableNoPad(tableData, true)
 	return nil
 }
 
@@ -415,7 +415,7 @@ func (b BrowsersCmd) ReplaysList(ctx context.Context, in BrowsersReplaysListInpu
 	for _, r := range *items {
 		rows = append(rows, []string{r.ReplayID, util.FormatLocal(r.StartedAt), util.FormatLocal(r.FinishedAt), truncateURL(r.ReplayViewURL, 60)})
 	}
-	printTableNoPad(rows, true)
+	PrintTableNoPad(rows, true)
 	return nil
 }
 
@@ -440,7 +440,7 @@ func (b BrowsersCmd) ReplaysStart(ctx context.Context, in BrowsersReplaysStartIn
 		return util.CleanedUpSdkError{Err: err}
 	}
 	rows := pterm.TableData{{"Property", "Value"}, {"Replay ID", res.ReplayID}, {"View URL", res.ReplayViewURL}, {"Started At", util.FormatLocal(res.StartedAt)}}
-	printTableNoPad(rows, true)
+	PrintTableNoPad(rows, true)
 	return nil
 }
 
@@ -563,7 +563,7 @@ func (b BrowsersCmd) ProcessExec(ctx context.Context, in BrowsersProcessExecInpu
 		return util.CleanedUpSdkError{Err: err}
 	}
 	rows := pterm.TableData{{"Property", "Value"}, {"Exit Code", fmt.Sprintf("%d", res.ExitCode)}, {"Duration (ms)", fmt.Sprintf("%d", res.DurationMs)}}
-	printTableNoPad(rows, true)
+	PrintTableNoPad(rows, true)
 	if res.StdoutB64 != "" {
 		data, err := base64.StdEncoding.DecodeString(res.StdoutB64)
 		if err != nil {
@@ -625,7 +625,7 @@ func (b BrowsersCmd) ProcessSpawn(ctx context.Context, in BrowsersProcessSpawnIn
 		return util.CleanedUpSdkError{Err: err}
 	}
 	rows := pterm.TableData{{"Property", "Value"}, {"Process ID", res.ProcessID}, {"PID", fmt.Sprintf("%d", res.Pid)}, {"Started At", util.FormatLocal(res.StartedAt)}}
-	printTableNoPad(rows, true)
+	PrintTableNoPad(rows, true)
 	return nil
 }
 
@@ -669,7 +669,7 @@ func (b BrowsersCmd) ProcessStatus(ctx context.Context, in BrowsersProcessStatus
 		return util.CleanedUpSdkError{Err: err}
 	}
 	rows := pterm.TableData{{"Property", "Value"}, {"State", string(res.State)}, {"CPU %", fmt.Sprintf("%.2f", res.CPUPct)}, {"Mem Bytes", fmt.Sprintf("%d", res.MemBytes)}, {"Exit Code", fmt.Sprintf("%d", res.ExitCode)}}
-	printTableNoPad(rows, true)
+	PrintTableNoPad(rows, true)
 	return nil
 }
 
@@ -928,7 +928,7 @@ func (b BrowsersCmd) FSFileInfo(ctx context.Context, in BrowsersFSFileInfoInput)
 		return util.CleanedUpSdkError{Err: err}
 	}
 	rows := pterm.TableData{{"Property", "Value"}, {"Path", res.Path}, {"Name", res.Name}, {"Mode", res.Mode}, {"IsDir", fmt.Sprintf("%t", res.IsDir)}, {"SizeBytes", fmt.Sprintf("%d", res.SizeBytes)}, {"ModTime", util.FormatLocal(res.ModTime)}}
-	printTableNoPad(rows, true)
+	PrintTableNoPad(rows, true)
 	return nil
 }
 
@@ -957,7 +957,7 @@ func (b BrowsersCmd) FSListFiles(ctx context.Context, in BrowsersFSListFilesInpu
 	for _, f := range *res {
 		rows = append(rows, []string{f.Mode, fmt.Sprintf("%d", f.SizeBytes), util.FormatLocal(f.ModTime), f.Name, f.Path})
 	}
-	printTableNoPad(rows, true)
+	PrintTableNoPad(rows, true)
 	return nil
 }
 

cmd/profiles.go

@@ -73,7 +73,7 @@ func (p ProfilesCmd) List(ctx context.Context) error {
 			util.FormatLocal(prof.LastUsedAt),
 		})
 	}
-	printTableNoPad(rows, true)
+	PrintTableNoPad(rows, true)
 	return nil
 }
 
@@ -96,7 +96,7 @@ func (p ProfilesCmd) Get(ctx context.Context, in ProfilesGetInput) error {
 	rows = append(rows, []string{"Created At", util.FormatLocal(item.CreatedAt)})
 	rows = append(rows, []string{"Updated At", util.FormatLocal(item.UpdatedAt)})
 	rows = append(rows, []string{"Last Used At", util.FormatLocal(item.LastUsedAt)})
-	printTableNoPad(rows, true)
+	PrintTableNoPad(rows, true)
 	return nil
 }
 
@@ -118,7 +118,7 @@ func (p ProfilesCmd) Create(ctx context.Context, in ProfilesCreateInput) error {
 	rows = append(rows, []string{"Name", name})
 	rows = append(rows, []string{"Created At", util.FormatLocal(item.CreatedAt)})
 	rows = append(rows, []string{"Last Used At", util.FormatLocal(item.LastUsedAt)})
-	printTableNoPad(rows, true)
+	PrintTableNoPad(rows, true)
 	return nil
 }
 

cmd/proxies/create.go

@@ -4,6 +4,7 @@ import (
 	"context"
 	"fmt"
 
+	"github.com/onkernel/cli/pkg/table"
 	"github.com/onkernel/cli/pkg/util"
 	"github.com/onkernel/kernel-go-sdk"
 	"github.com/pterm/pterm"
@@ -188,7 +189,7 @@ func (p ProxyCmd) Create(ctx context.Context, in ProxyCreateInput) error {
 	}
 	rows = append(rows, []string{"Protocol", protocol})
 
-	PrintTableNoPad(rows, true)
+	table.PrintTableNoPad(rows, true)
 	return nil
 }
 

cmd/proxies/get.go

@@ -4,6 +4,7 @@ import (
 	"context"
 	"fmt"
 
+	"github.com/onkernel/cli/pkg/table"
 	"github.com/onkernel/cli/pkg/util"
 	"github.com/onkernel/kernel-go-sdk"
 	"github.com/pterm/pterm"
@@ -38,7 +39,22 @@ func (p ProxyCmd) Get(ctx context.Context, in ProxyGetInput) error {
 	// Display type-specific config details
 	rows = append(rows, getProxyConfigRows(item)...)
 
-	PrintTableNoPad(rows, true)
+	// Display status with color
+	status := string(item.Status)
+	if status == "" {
+		status = "-"
+	} else if status == "available" {
+		status = pterm.Green(status)
+	} else if status == "unavailable" {
+		status = pterm.Red(status)
+	}
+	rows = append(rows, []string{"Status", status})
+
+	// Display last checked timestamp
+	lastChecked := util.FormatLocal(item.LastChecked)
+	rows = append(rows, []string{"Last Checked", lastChecked})
+
+	table.PrintTableNoPad(rows, true)
 	return nil
 }
 

cmd/proxies/helpers.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 	"strings"
 
+	"github.com/onkernel/cli/pkg/table"
 	"github.com/onkernel/cli/pkg/util"
 	"github.com/onkernel/kernel-go-sdk"
 	"github.com/pterm/pterm"
@@ -68,7 +69,7 @@ func (p ProxyCmd) List(ctx context.Context) error {
 		})
 	}
 
-	PrintTableNoPad(tableData, true)
+	table.PrintTableNoPad(tableData, true)
 	return nil
 }
 

cmd/proxies/list.go

@@ -64,33 +64,31 @@ func TestProxyList_WithProxies(t *testing.T) {
 	assert.NoError(t, err)
 	output := buf.String()
 
-	// Check table headers
+	// Check table headers (Config may be truncated in narrow terminals)
 	assert.Contains(t, output, "ID")
 	assert.Contains(t, output, "Name")
 	assert.Contains(t, output, "Type")
 	assert.Contains(t, output, "Protocol")
-	assert.Contains(t, output, "Config")
+	assert.Contains(t, output, "Status")
 
-	// Check proxy data
+	// Check proxy data - verify IDs and short columns are fully visible
 	assert.Contains(t, output, "dc-1")
 	assert.Contains(t, output, "https") // Protocol is shown
-	assert.Contains(t, output, "Country")
+	assert.Contains(t, output, "datacenter")
 
 	assert.Contains(t, output, "res-1")
+	assert.Contains(t, output, "residential")
 
 	assert.Contains(t, output, "custom-1")
 	assert.Contains(t, output, "My Proxy")
 	assert.Contains(t, output, "custom")
-	assert.Contains(t, output, "proxy") // Part of proxy.example.com, will be truncated
 
 	assert.Contains(t, output, "mobile-1")
 	assert.Contains(t, output, "mobile")
-	assert.Contains(t, output, "Carrier: verizon")
 
 	assert.Contains(t, output, "isp-1")
 	assert.Contains(t, output, "-") // Empty name shows as "-"
 	assert.Contains(t, output, "isp")
-	assert.Contains(t, output, "Country: EU")
 }
 
 func TestProxyList_Error(t *testing.T) {

cmd/proxies/list_test.go

@@ -1,113 +1,11 @@
 package cmd
 
 import (
-	"strings"
-	"unicode/utf8"
-
+	"github.com/onkernel/cli/pkg/table"
 	"github.com/pterm/pterm"
 )
 
-// printTableNoPad renders a table similar to pterm.DefaultTable, but it avoids
-// adding trailing padding spaces after the last column and does not add blank
-// padded lines to match multi-line cells in other columns. The last column may
-// contain multi-line content which will be printed as-is on following lines.
-func printTableNoPad(data pterm.TableData, hasHeader bool) {
-	if len(data) == 0 {
-		return
-	}
-
-	// Determine number of columns from the first row
-	numCols := len(data[0])
-	if numCols == 0 {
-		return
-	}
-
-	// Pre-compute max width per column for all but the last column
-	maxColWidths := make([]int, numCols)
-	for _, row := range data {
-		for colIdx := 0; colIdx < numCols && colIdx < len(row); colIdx++ {
-			if colIdx == numCols-1 {
-				continue
-			}
-			for _, line := range strings.Split(row[colIdx], "\n") {
-				if w := utf8.RuneCountInString(line); w > maxColWidths[colIdx] {
-					maxColWidths[colIdx] = w
-				}
-			}
-		}
-	}
-
-	var b strings.Builder
-	sep := pterm.DefaultTable.Separator
-	sepStyled := pterm.ThemeDefault.TableSeparatorStyle.Sprint(sep)
-
-	renderRow := func(row []string, styleHeader bool) {
-		// Build first-line-only for non-last columns; last column is full string
-		firstLineParts := make([]string, 0, numCols)
-		for colIdx := 0; colIdx < numCols; colIdx++ {
-			var cell string
-			if colIdx < len(row) {
-				cell = row[colIdx]
-			}
-
-			if colIdx < numCols-1 {
-				// Only the first line for non-last columns
-				lines := strings.Split(cell, "\n")
-				first := ""
-				if len(lines) > 0 {
-					first = lines[0]
-				}
-				padCount := maxColWidths[colIdx] - utf8.RuneCountInString(first)
-				if padCount < 0 {
-					padCount = 0
-				}
-				firstLineParts = append(firstLineParts, first+strings.Repeat(" ", padCount))
-			} else {
-				// Last column: render the first line now; remaining lines after
-				lines := strings.Split(cell, "\n")
-				if len(lines) > 0 {
-					firstLineParts = append(firstLineParts, lines[0])
-				} else {
-					firstLineParts = append(firstLineParts, "")
-				}
-			}
-		}
-
-		line := strings.Join(firstLineParts[:numCols-1], sepStyled)
-		if numCols > 1 {
-			if line != "" {
-				line += sepStyled
-			}
-			line += firstLineParts[numCols-1]
-		}
-
-		if styleHeader {
-			b.WriteString(pterm.ThemeDefault.TableHeaderStyle.Sprint(line))
-		} else {
-			b.WriteString(line)
-		}
-		b.WriteString("\n")
-
-		// Print remaining lines from the last column without alignment padding
-		if numCols > 0 {
-			var lastCell string
-			if len(row) >= numCols {
-				lastCell = row[numCols-1]
-			}
-			lines := strings.Split(lastCell, "\n")
-			if len(lines) > 1 {
-				rest := strings.Join(lines[1:], "\n")
-				if rest != "" {
-					b.WriteString(rest)
-					b.WriteString("\n")
-				}
-			}
-		}
-	}
-
-	for idx, row := range data {
-		renderRow(row, hasHeader && idx == 0)
-	}
-
-	pterm.Print(b.String())
+// PrintTableNoPad is a wrapper around pkg/table.PrintTableNoPad for backwards compatibility
+func PrintTableNoPad(data pterm.TableData, hasHeader bool) {
+	table.PrintTableNoPad(data, hasHeader)
 }