Skip to content

fix(flow-chat): smooth action lifecycle transitions - #1792

Merged
bobleer merged 1 commit into
GCWing:mainfrom
bobleer:bob/smooth-flowchat-actions
Jul 27, 2026
Merged

fix(flow-chat): smooth action lifecycle transitions#1792
bobleer merged 1 commit into
GCWing:mainfrom
bobleer:bob/smooth-flowchat-actions

Conversation

@bobleer

@bobleer bobleer commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Keep just-completed FlowChat actions visible while they remain the visual tail, then compact them when newer conversation content arrives.
  • Preserve stable explore-group and tool-card render identity across running-to-completed transitions.
  • Harden collapse reversal, pre-paint automatic state changes, and reduce expensive card-level transition, blur, breathing-opacity, and will-change work.
  • Add regression coverage and document the FlowChat scroll-height stability contract.

Type and Areas

Type:

UI/UX, regression fix, test, docs.

Areas:

Web UI, FlowChat virtualization, tool cards, subagent projection.

Motivation / Impact

Tool completion could simultaneously change virtual item shape, hide the settled card, and collapse its containing group. Users saw the action flash and disappear, while animated automatic height changes made the virtualized scroll anchor chase the layout. Completed tail actions now remain readable until superseded, automatic compaction is applied before paint in one frame, and manual toggles retain smooth animation.

Verification

  • pnpm --dir src/web-ui run test:run with 7 focused FlowChat files: 89 tests passed.
  • pnpm run type-check:web
  • pnpm run lint:web
  • git diff upstream/main...HEAD --check
  • Manual in-app Browser preview: verified expand, collapse, rapid reversal, settled will-change cleanup, and card transition/backdrop computed styles.

Reviewer Notes

Automatic compaction intentionally does not animate because Virtuoso must observe one stable height change. User-initiated expand and collapse actions continue to animate. No user-facing strings or locale resources changed.

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.

@bobleer
bobleer merged commit e949e1e into GCWing:main Jul 27, 2026
5 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