Skip to content

fix(tui): hide empty integration output - #40933

Open
kitlangton wants to merge 1 commit into
v2from
hide-empty-output
Open

fix(tui): hide empty integration output#40933
kitlangton wants to merge 1 commit into
v2from
hide-empty-output

Conversation

@kitlangton

Copy link
Copy Markdown
Contributor

What

Hide the command-output panel in integration dialogs when the command produced only whitespace.

Before / After

Before: A successful command with no visible output left an empty elevated panel between the dialog header and status message.

After: Whitespace-only output skips the panel while non-empty output keeps the existing presentation.

How

  • Trim command output once in dialog-integration.tsx.
  • Render the output container only when trimmed content exists.

Scope

Only integration command-result rendering changes.

Testing

  • bun typecheck from packages/tui
  • bun run test test/cli/cmd/tui/integration-options.test.ts
  • Repository pre-push typecheck
  • git diff --check

@kitlangton

Copy link
Copy Markdown
Contributor Author

Demo

Before: Whitespace-only command output leaves a blank elevated panel.

output-before.mp4

After: The empty panel is omitted and the status message follows the header.

output-after.mp4

@kitlangton
kitlangton enabled auto-merge (squash) August 6, 2026 23:25
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.

1 participant