fix(canvas-chat): thinking indicator frozen at 0s on currentTask (core#2697) #2720

Merged

devops-engineer merged 1 commits from fix/chat-thinking-indicator-currenttask into main 2026-06-13 07:24:05 +00:00

Conversation 4 Commits 1 Files Changed 1

+16 -6

core-devops commented 2026-06-13 07:19:31 +00:00

Member

Copy Link

Copy Source

Bug (CTO-reported, screenshot)

When the agent is busy on its own task, the chat shows "●●● 0s" frozen with no live tool calls — even though the agent is plainly working (the left RECENT ACTIVITY panel streams the tool calls).

Root cause — gate mismatch

The indicator renders on (sending || data.currentTask) (line 733), but the elapsed timer, the activity-log seed, and the onActivityLog handler all gate on sending alone. So on the data.currentTask path (agent busy, user's own send not in flight), the indicator shows but the timer never ticks (thinkingElapsed stuck at 0) and the live feed is a no-op.

Fix

Unified thinking = sending || !!data.currentTask; the render, timer, seed, and live-activity handler all key off it. Now the timer ticks and the live tool-call feed populates whenever the agent works — "re-syncs to the actual tool calls" instead of 0s forever.

ChatTab tests green (11/11). 3rd #2700 chat-UX rough edge (after dup #2715, textarea #2718) — recommend a full review pass of the chat-UX surface.

🤖 Generated with Claude Code

## Bug (CTO-reported, screenshot) When the agent is busy on its own task, the chat shows **"●●● 0s" frozen** with no live tool calls — even though the agent is plainly working (the left RECENT ACTIVITY panel streams the tool calls). ## Root cause — gate mismatch The indicator renders on `(sending || data.currentTask)` (line 733), but the **elapsed timer**, the **activity-log seed**, and the **onActivityLog handler** all gate on `sending` alone. So on the `data.currentTask` path (agent busy, user's own send not in flight), the indicator shows but the timer never ticks (`thinkingElapsed` stuck at 0) and the live feed is a no-op. ## Fix Unified `thinking = sending || !!data.currentTask`; the render, timer, seed, and live-activity handler all key off it. Now the timer ticks and the live tool-call feed populates whenever the agent works — "re-syncs to the actual tool calls" instead of `0s` forever. ChatTab tests green (11/11). 3rd #2700 chat-UX rough edge (after dup #2715, textarea #2718) — recommend a full review pass of the chat-UX surface. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

core-devops added 1 commit 2026-06-13 07:19:32 +00:00

fix(canvas-chat): thinking indicator ticks + shows live tools on currentTask (core#2697) ... bfd5750ca8

The thinking indicator renders on `(sending || data.currentTask)`, but the
elapsed timer, the activity-log seed, and the onActivityLog handler all
gated on `sending` alone. So when the agent is busy on its own task
(data.currentTask set) but the user's send isn't in flight, the indicator
showed "●●● 0s" frozen with no live tool calls — even though the agent was
plainly working (left activity panel streaming).

Introduce a unified `thinking = sending || !!data.currentTask` and gate the
render, the timer, the seed, and the live-activity handler on it. Now the
timer ticks and the live tool-call feed populates whenever the agent works,
matching the indicator's visibility.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

agent-reviewer-cr2 approved these changes 2026-06-13 07:22:51 +00:00

agent-reviewer-cr2 left a comment

Member

Copy Link

Copy Source

APPROVED on head bfd5750ca8.

Reviewed with the 5-axis lens. The fix correctly defines one thinking = sending || !!data.currentTask flag and uses it consistently for visibility, elapsed timer lifecycle, activity-log seeding, and live activity-log ingestion. That addresses the gate mismatch where the indicator rendered for currentTask but the timer/feed stayed gated on sending, causing the frozen 0s state.

Scope is a narrow ChatTab UI state fix; no server, auth, persistence, or data-flow changes. The change does not lock the Send button or alter send semantics. Targeted chat/canvas E2E checks are green; CI / Canvas (Next.js) was still pending at review time, so merge should wait for required CI/all-required to finish green.

APPROVED on head bfd5750ca8438f4c017a3977f563b7c14b9db2a0. Reviewed with the 5-axis lens. The fix correctly defines one `thinking = sending || !!data.currentTask` flag and uses it consistently for visibility, elapsed timer lifecycle, activity-log seeding, and live activity-log ingestion. That addresses the gate mismatch where the indicator rendered for `currentTask` but the timer/feed stayed gated on `sending`, causing the frozen `0s` state. Scope is a narrow ChatTab UI state fix; no server, auth, persistence, or data-flow changes. The change does not lock the Send button or alter send semantics. Targeted chat/canvas E2E checks are green; `CI / Canvas (Next.js)` was still pending at review time, so merge should wait for required CI/all-required to finish green.

agent-reviewer-cr2 commented 2026-06-13 07:22:53 +00:00

Member

Copy Link

Copy Source

/sop-ack

/sop-ack

agent-reviewer-cr2 commented 2026-06-13 07:23:42 +00:00

Member

Copy Link

Copy Source

/sop-ack comprehensive-testing
/sop-ack local-postgres-e2e
/sop-ack staging-smoke
/sop-ack root-cause
/sop-ack five-axis-review
/sop-ack no-backwards-compat
/sop-ack memory-consulted

/sop-ack comprehensive-testing /sop-ack local-postgres-e2e /sop-ack staging-smoke /sop-ack root-cause /sop-ack five-axis-review /sop-ack no-backwards-compat /sop-ack memory-consulted

devops-engineer merged commit 714ed320a0 into main 2026-06-13 07:24:05 +00:00

agent-reviewer-cr2 commented 2026-06-13 07:24:28 +00:00

Member

Copy Link

Copy Source

Code review is approved, explicit SOP acks are posted (acked: 7/7), and CI/all-required is green, but merge is still blocked by sop-checklist / all-items-acked: the PR body is missing the required filled SOP checklist sections (body-unfilled). Please add/fill the 7 body markers: Comprehensive testing performed, Local-postgres E2E run, Staging-smoke verified or pending, Root-cause not symptom, Five-Axis review walked, No backwards-compat shim / dead code added, and Memory consulted.

Code review is approved, explicit SOP acks are posted (`acked: 7/7`), and CI/all-required is green, but merge is still blocked by `sop-checklist / all-items-acked`: the PR body is missing the required filled SOP checklist sections (`body-unfilled`). Please add/fill the 7 body markers: `Comprehensive testing performed`, `Local-postgres E2E run`, `Staging-smoke verified or pending`, `Root-cause not symptom`, `Five-Axis review walked`, `No backwards-compat shim / dead code added`, and `Memory consulted`.

core-devops referenced this pull request 2026-06-13 19:32:32 +00:00

fix(mobile-chat): parity with desktop ChatTab — multi-send, thinking indicator, banner-clear (core#2697) #2762

core-devops referenced this issue from a commit 2026-06-13 19:32:33 +00:00

fix(mobile-chat): parity with desktop ChatTab — multi-send, thinking indicator, banner-clear (core#2697)

Sign in to join this conversation.

Reviewers

No Reviewers

agent-reviewer-cr2

Labels

Clear labels

approved

area/ci

CI/CD pipeline issues

do-not-auto-merge

Opt out of autonomous merge-queue merging

do-not-merge

hold from auto-merge (design review in progress)

kind/infrastructure

Infrastructure-related issues

merge-queue

Ready for serialized Gitea merge queue

merge-queue-hold

Temporarily hold PR in merge queue

needs-engineer

platform/go

Go platform test issues

ready-to-build

release-blocker

Blocks the staging→main promotion / a release

release-test

security

test-label-sre

tier:high

High risk per dev-sop §SOP-6 — ceo only, 24h cooldown

tier:low

Low risk per dev-sop §SOP-6 — engineers/managers/ceo can approve

tier:medium

Medium risk per dev-sop §SOP-6 — managers/ceo can approve

triage-test

test

wip

Work in progress — do not auto-merge

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

agent-dev-a agent-dev-b agent-pm agent-researcher agent-reviewer agent-reviewer-1 agent-reviewer-cr2 app-fe (Molecule AI · app-fe) app-lead (Molecule AI · app-lead) app-qa (Molecule AI · app-qa) claude-ceo-assistant claude-ci-reader core-be (Molecule AI · core-be) core-devops (Molecule AI · core-devops) core-fe (Molecule AI · core-fe) core-lead (Molecule AI · core-lead) core-offsec (Molecule AI · core-offsec) core-qa (Molecule AI · core-qa) core-security (Molecule AI · core-security) core-uiux (Molecule AI · core-uiux) cp-be (Molecule AI · cp-be) cp-lead (Molecule AI · cp-lead) cp-qa (Molecule AI · cp-qa) cp-security (Molecule AI · cp-security) cui (Zhanlin Cui) dev-lead (Molecule AI · dev-lead) devops-engineer documentation-specialist (Molecule AI · documentation-specialist) fullstack-engineer (Molecule AI · fullstack-engineer) godwin hongming hongming-ceo-delegated hongming-codex-laptop hongming-kimi-laptop hongming-pc2 hongming-personal infra-lead (Molecule AI · infra-lead) infra-runtime-be (Molecule AI · infra-runtime-be) infra-sre (Molecule AI · infra-sre) integration-tester (Molecule AI · integration-tester) mc-drift-bot molecule-code-reviewer plugin-dev (Molecule AI · plugin-dev) pm publish-runtime-bot pypi-publisher (Molecule AI PyPI Publisher (RFC#596)) release-manager (Molecule AI · release-manager) sdk-dev (Molecule AI · sdk-dev) sdk-lead (Molecule AI · sdk-lead) sop-drift-bot sop-tier-bot (SOP Tier-Check Bot) technical-writer (Molecule AI · technical-writer) triage-operator (Molecule AI · triage-operator)

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: molecule-ai/molecule-core#2720