[Payment due @situchan] Fix stuck Concierge thinking indicator when client misses Onyx clear update - #85620

Merged

MonilBhavsar merged 59 commits into

mainfrom

marcochavezf/612534-fix-stuck-thinking-indicator

May 8, 2026

Commits on Mar 18, 2026

Fix stuck Concierge thinking indicator when Onyx batches SET+CLEAR updates

marcochavezf
and
claude
committed
1dffb24

Commits on Mar 20, 2026

Fix CI: move Onyx.connect to lib module and fix spellcheck

marcochavezf
and
claude
committed
96487ea
Fix CI: use Connection type for Onyx.connect return value and prefer early return

marcochavezf
and
claude
committed
c7604b1

Commits on Mar 24, 2026

Scope Onyx write counter to track pending requests for multi-message sends

marcochavezf
and
claude
committed
22b0816

Commits on Mar 25, 2026

Replace NVP version tracker with client-side TTL (lease pattern) for stuck indicator fix

marcochavezf
and
claude
committed
1a681db
Add XMPP to cspell allowed words

marcochavezf
and
claude
committed
0515c33
Trigger CI re-run for checklist validation
marcochavezf
committed
32c1893

Commits on Mar 27, 2026

Add getNewerActions fallback when safety timer fires or network reconnects

marcochavezf
and
claude
committed
fb98ac7
Merge branch 'main' into marcochavezf/612534-fix-stuck-thinking-indicator

marcochavezf
and
claude
committed
b1ce260
Replace hard TTL clear with progressive retry for thinking indicator

marcochavezf
and
claude
committed
3de2a0b
Update safety timeout tests for progressive retry behavior

marcochavezf
and
claude
committed
f6f8678
Fix ESLint: use Set for PROGRESSIVE_RETRY_INTERVALS_MS in tests

marcochavezf
and
claude
committed
ac267d7

Commits on Mar 28, 2026

Replace progressive retry with 30s polling for thinking indicator

marcochavezf
and
claude
committed
8938ec1
Keep thinking indicator on reconnect until response arrives

marcochavezf
and
claude
committed
2e00661
Restore original offline behavior: hide indicator when offline, reappear on reconnect

marcochavezf
and
claude
committed
8d2d8c4
Fix reconnect test: indicator persists through offline/online cycle

marcochavezf
and
claude
committed
bb281de
Fix: clear indicator NVP alongside getNewerActions poll

marcochavezf
and
claude
committed
9815ba2
Fix: detect new actions to clear stuck indicator instead of blind NVP clear

marcochavezf
and
claude
committed
cd11423
Clear indicator NVP on network reconnect to fix stuck state after Pusher drop

marcochavezf
and
claude
committed
5e86d94

Commits on Mar 29, 2026

Delegate AgentZeroStatusContext to useAgentZeroStatusIndicator hook (Option C)

marcochavezf
and
claude
committed
6a18430

Commits on Mar 30, 2026

Clear indicator immediately when Concierge response detected via Onyx

marcochavezf
and
claude
committed
5bd1c44
Fix: move Concierge detection useEffect after dependency declarations

marcochavezf
and
claude
committed
74ed559
Fix ESLint: prefer-early-return + narrow hook dependencies

marcochavezf
and
claude
committed
0790eec
Address all bot review comments on PR #85620

marcochavezf
and
claude
committed
ac8ba88
Fix infinite re-render loop in useAgentZeroStatusIndicator tests

marcochavezf
and
claude
committed
cc74304

Commits on Apr 3, 2026

Remove redundant useCallback/useMemo wrappers (React Compiler handles memoization)

marcochavezf
and
claude
committed
efa98d2

Commits on Apr 6, 2026

Merge remote-tracking branch 'origin/main' into marcochavezf/612534-fix-stuck-thinking-indicator
marcochavezf
committed
bc884e3

Commits on Apr 8, 2026

Merge main, resolve conflicts, address review comments

marcochavezf
and
claude
committed
19aebb9
Merge main + fix broken isModalVisible reference from #87077

marcochavezf
and
claude
committed
fecc854

Commits on Apr 9, 2026

Revert AddNewCardPage.tsx changes — already fixed in #87431

marcochavezf
and
claude
committed
d781630
Merge remote-tracking branch 'origin/main' into marcochavezf/612534-fix-stuck-thinking-indicator
marcochavezf
committed
82f3a1c
Fix stuck indicator on reconnect: clear stale optimistic state

marcochavezf
and
claude
committed
beb21b7
Align hook with main's proven patterns to fix mobile indicator

marcochavezf
and
claude
committed
5e94954

Commits on Apr 13, 2026

Merge branch 'main' into marcochavezf/612534-fix-stuck-thinking-indicator
marcochavezf
committed
85f2019

Commits on Apr 17, 2026

Fix Concierge DM indicator + bot review follow-ups

marcochavezf
and
claude
committed
7422499
Fix CI: React Compiler, typecheck, spellcheck, ESLint

marcochavezf
and
claude
committed
ef90643

Commits on Apr 18, 2026

Commits on Apr 21, 2026

Commits on Apr 22, 2026

Preserve optimistic thinking indicator across chat-switch remount

marcochavezf
and
claude
committed
e62b475

Commits on Apr 23, 2026

Fix CI: React Compiler compliance + spellcheck

marcochavezf
and
claude
committed
074848d
Bump optimistic startedAt on each kickoff

marcochavezf
and
claude
committed
e09c8f9

Commits on May 1, 2026

Commits on May 6, 2026

Commits on May 7, 2026

Merge remote-tracking branch 'origin/main' into marcochavezf/612534-fix-stuck-thinking-indicator
marcochavezf
committed
9f54e37

Commits on May 8, 2026

fix: use trimmedQuery in addComment — accidental regression caught by Monil
marcochavezf
committed
86984b4