Skip to content

fix(chat): keep work activity visible - #110

Merged
klioen merged 1 commit into
mainfrom
fix/chat-work-activity-row
Oct 24, 2026
Merged

klioen merged 1 commit into
mainfrom
fix/chat-work-activity-row

Conversation

@klioen

@klioen klioen commented Oct 24, 2026

Copy link
Copy Markdown
Owner

Summary

Keep the current Tool or Thinking activity visible throughout a running Chat/Task turn. The activity now occupies a dedicated row below Working for … instead of sharing the duration row and disappearing as soon as an individual operation completes.

Related work

  • Issue: N/A
  • Accepted intent/spec/plan: docs/chat-work-summary-disclosure/change.md

Changes

  • select the latest Tool or Thinking item for the full lifetime of a running turn
  • render duration and current activity as separate summary rows
  • retain completed activity until a later Tool or Thinking item replaces it
  • add regression coverage for the two-row layout, completion persistence, replacement, and final cleanup
  • update the existing Chat work-summary SDLC artifact

Risk and boundaries

  • Regression risk: Low; limited to Renderer presentation and work-summary selection behavior.
  • Security/privacy impact: None. Raw reasoning and Tool input/output remain excluded from the outer activity summary.
  • Data or configuration migration: None.
  • Platform/package impact: None; no dependency, Main-process, IPC, native, or packaging changes.
  • Rollback approach: Revert commit 247e84f.

Verification

pnpm exec vitest run src/features/chat/WorkSummary.test.tsx src/features/chat/ChatView.test.tsx
# 2 files passed, 17 tests passed

pnpm check
# passed: format, brand, i18n, secrets, licenses, lint, typecheck,
# 104 test files / 869 tests, and 35 release tests
  • Tests cover new or changed behavior.
  • pnpm check passes locally.
  • This change does not affect packaging/native behavior.
  • UI screenshot or recording not included; deterministic component tests cover the layout structure and lifecycle behavior.

Contributor checklist

  • I read CONTRIBUTING.md and the repository AGENTS.md.
  • The change follows the accepted artifact at docs/chat-work-summary-disclosure/change.md.
  • Renderer/Main authority, relative-path validation, Git argument arrays, and credential boundaries remain intact.
  • The existing user-visible behavior document is updated; no public release changelog exists for this unreleased change.
  • The diff contains no credentials, private data, unrelated generated files, or accidental dependency updates.
  • I reviewed the diff for correctness, security/privacy, and compliance with accepted scope.

@klioen
klioen marked this pull request as ready for review October 24, 2026 12:23
@klioen
klioen merged commit b7e5d6a into main Oct 24, 2026
4 checks passed
@klioen
klioen deleted the fix/chat-work-activity-row branch October 24, 2026 12:23
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