Skip to content

feat(shell): surface task size in --live selector and on connect#152

Merged
blsmth merged 1 commit into
mainfrom
agent/go-engineer/issue-150
Jul 7, 2026
Merged

feat(shell): surface task size in --live selector and on connect#152
blsmth merged 1 commit into
mainfrom
agent/go-engineer/issue-150

Conversation

@blsmth

@blsmth blsmth commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds formatTaskSize() helper that converts ECS CPU units and memory MB into human-readable strings (e.g. 0.5 vCPU / 1 GB)
  • Inline size info in the huh selector for shell --live / ps exec --live so users know the container allocation before attaching
  • Prints a one-line advisory after connecting that names the process type, its size, and nudges heavy workloads toward a dedicated shell with --cpu/--memory
  • Fixes a pre-existing fall-through bug: adds return after the --live block so execution no longer falls into StartInteractiveShell once the live session ends

Closes #150

Test plan

  • go test ./cmd/ -- TestFormatTaskSize covers all CPU/memory combinations including nil and unparseable inputs
  • Manual: apppack shell --live -a myapp -- selector entries now show web: a1b2c3 (0.5 vCPU / 1 GB); on connect a faint advisory line appears before the shell prompt

Show each candidate task's CPU/memory allocation inline in the huh
selector so users know what they're attaching into before they commit.
After the session is established, print a one-line advisory that names
the process type, its allocation, and nudges heavy workloads toward a
dedicated shell with --cpu/--memory.

Adds formatTaskSize() helper that converts ECS CPU units (e.g. "512")
and memory MB (e.g. "1024") into human-readable strings
("0.5 vCPU / 1 GB").

Also adds an explicit return after the --live block to prevent
fall-through into StartInteractiveShell once the live session ends.

Closes #150
@blsmth blsmth merged commit d12531c into main Jul 7, 2026
4 checks passed
@blsmth blsmth deleted the agent/go-engineer/issue-150 branch July 7, 2026 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

shell --live: surface the size of the live instance you're attaching to

1 participant