Skip to content

fix(toolcard): refine exec process card layout - #2726

Merged
wsp1911 merged 1 commit into
GCWing:1.0.0-explorefrom
wsp1911:1.0.0-explore
Sep 1, 2026
Merged

fix(toolcard): refine exec process card layout#2726
wsp1911 merged 1 commit into
GCWing:1.0.0-explorefrom
wsp1911:1.0.0-explore

Conversation

@wsp1911

@wsp1911 wsp1911 commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Keep streaming and automatic-completion previews at a stable compact height.
  • Size manually expanded completed ExecCommand and WriteStdin cards to their actual output, up to the existing row limit.
  • Align WriteStdin footer metadata with ExecCommand, hide ExecCommand TTY metadata, and place the WriteStdin exit code after duration.
  • Replace stdin-specific action wording with user-facing "Send input" labels across supported locales.

Type and Areas

Type: Bug fix / UI/UX

Areas: Web UI, FlowChat tool cards, design system, localization

Motivation / Impact

Completed command cards previously reserved a large fixed output area even when output was empty or short. WriteStdin footer metadata also used a different alignment from ExecCommand, and its action label exposed the technical stdin term.

This change preserves the stable height needed before automatic collapse while making manually expanded results compact, aligning execution metadata consistently, and using clearer user-facing copy.

Verification

  • pnpm --dir src/web-ui run test:run src/flow_chat/tool-cards/ExecProcessToolCardView.test.tsx — passed, 10/10 tests.
  • pnpm run design-system:check — passed.
  • pnpm run check:web — passed with existing non-blocking Appearance warnings.
  • pnpm run i18n:audit — passed with 0 warnings.
  • git diff --check — passed.
  • UI interaction validation was not run.
  • Remote workspace, remote control, Peer Device Mode, and Detached Dispatch were not exercised.

Reviewer Notes

The fixed compact frame remains in place during streaming and the automatic completion grace period. Content-sized output is enabled only after the user manually expands a completed card.

Checklist

  • This PR is focused and does not include secrets, temporary prompts, generated scratch files, or unrelated artifacts.
  • Relevant verification is recorded above, or skipped checks are explained.
  • User-facing strings, docs, and locales are updated where applicable.

Keep the compact output frame stable during streaming and automatic
collapse, while sizing manually expanded completed cards to their
actual output.

- Align WriteStdin execution metadata to the footer end
- Hide ExecCommand TTY metadata and place WriteStdin exit code last
- Replace stdin-specific action copy with user-facing input wording
- Cover adaptive output sizing and WriteStdin footer ordering
@wsp1911 wsp1911 changed the title 1.0.0 explore fix(toolcard): refine exec process card layout Sep 1, 2026
@wsp1911
wsp1911 merged commit 1d04960 into GCWing:1.0.0-explore Sep 1, 2026
8 checks passed
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