Skip to content

feat(return-summary): lead the welcome back card with where the task stands and one action - #823

Merged
astyfx merged 1 commit into
mainfrom
feat/welcom-back-popup-ui-ux-pr
Oct 25, 2026
Merged

astyfx merged 1 commit into
mainfrom
feat/welcom-back-popup-ui-ux-pr

Conversation

@astyfx

@astyfx astyfx commented Oct 25, 2026

Copy link
Copy Markdown
Member

Summary

The Welcome back card showed four rows of equal weight. Where it stands and Next usually linked to the same place (Show and Show, Outputs and Outputs), filler rows such as "Nothing new." took space, and the request's detail was cut to one line. This change puts where the task stands and the single thing to do about it at the top. Below that, the card shows only the facts that have something to say. The trigger rules and what the summary counts stay the same.

Changes

Features

  • return-summary: one status line first. It uses the shared state mark (the same glyph and color as Fleet and the task tabs), the wait and its length, the request's description under it, and the handoff record's first Remaining line as "Next: ...".
  • return-summary: one action button named for what it does: Show request, Review outputs, See what failed, Check activity, Open run progress, Open handoff, Ask a side question, Write a reply. It is the primary button only when the task waits on you.
  • return-summary: a wait with no action of its own, such as a pull request's state, offers Check activity instead of the composer.
  • return-summary: the facts (While you were away, Last decision) sit in a quieter band below the status and are left out when empty. Long text wraps to two lines instead of being cut to one.
  • i18n: shorter English and Korean copy. Status lines drop the trailing period, and Korean waits read "2시간 5분째 승인 대기 중". Unused keys are removed.

Docs

  • docs/features/return-summary.md and docs/architecture/code-organization.md describe the new layout and actions.

Test Evidence

  • bun run lint, bun run typecheck: pass. Lint reports only existing hook-dependency warnings.
  • check:design-system, check:i18n, check:style-channel, check:doc-paths, check:max-lines-ratchet, check:switch-exhaustiveness, check:reliability-gates: pass.
  • bun run test:isolated: passed 1033 files. tests/return-summary.test.tsx covers the status and action, omitting empty facts, and the pull request wait.
  • bun run build:pages: pass.
  • Rendered in the task panel dev preview (?stavePreview=task-panel&return=need|idle): approval wait in light/English, finished turn with a handoff next step in dark/Korean, and a 360px conversation width, where the button wraps to its own row. Fact labels, values and links share a 28px row.

Notes

  • Not verified: the Electron e2e return-summary-decision (its decision line and Show button are kept), and clicking the action in the real app, since the preview has no IPC. Link targets are unchanged.
  • The failed, stalled and pull request states have no preview, so unit tests cover them.

🤖 Generated with Claude Code

…stands and one action

The card showed four equal rows, two of which usually linked to the same
place, plus filler rows when nothing changed. It now opens with one status
line (the shared state mark, the wait and the request's detail, or the
handoff record's next step) and one action named for what it does, primary
when the task waits on you. Below it, only the facts with something to say:
what changed while you were away and the last decision, wrapped to two
lines. A wait with no action of its own, such as a pull request's state,
offers Activity instead of the composer. English and Korean copy is shorter.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@astyfx
astyfx requested a review from a team as a code owner October 25, 2026 02:38
@astyfx
astyfx requested review from paul-seo73 and png-1105 and removed request for a team October 25, 2026 02:38
@upwind-code-us

upwind-code-us Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

Upwind Upwind IaC Scan - ✅ Passed

0 newly introduced misconfigurations · 0 resolved · 0 total in this PR vs main

View full analysis in Upwind Console →

Scan completed in 1s

Scan history (1 scan)
Commit Scanned at New Resolved Net
2acf549 < 2026-10-25 02:39 UTC 0 0 0

Last scanned: 2acf549 · 2026-10-25 02:39 UTC

@heath-s heath-s left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@astyfx
astyfx merged commit 7947319 into main Oct 25, 2026
6 checks passed
@astyfx
astyfx deleted the feat/welcom-back-popup-ui-ux-pr branch October 25, 2026 07:03
@astyfx astyfx mentioned this pull request Oct 25, 2026
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.

2 participants