feat(mobile): pinned "Active now" tray on the Agents session list#4626
Merged
Conversation
…attention-notifications
Anchor the attention ack store on globalThis so the single-writer detail screen and the session-list readers share one instance across Expo Router route bundles, and remount the Agents list on attention-revision change after refocus so the freezeOnBlur'd list re-reads the ack store on return.
- cloud-agent-next: fetch session metadata inside the attention-push try/catch so a metadata failure is logged and swallowed as documented, never breaking the best-effort ingest path. - mobile: key the Agents list on an attention revision snapshotted at tab focus, so it remounts to re-read the ack store only on return from a just-opened session, not on any unrelated session's attention change during browsing (preserving scroll). - mobile: isolate attention-store subscribers so one throwing listener cannot prevent the rest from being notified.
oxlint no-confusing-void-expression forbids returning the void expression from the arrow shorthand passed to expect().
Thread the heartbeat session title from UserConnectionDO through SessionIngestDO into the notifications service so attention and session-ready push notifications lead with the session title. - packages/notifications: optional title on sendSessionReadyNotificationInputSchema. - services/notifications: add a code-point-safe title sanitizer (collapse whitespace, <=80 code points incl. ellipsis, no split surrogate) used by dispatchCloudAgentSessionPush and the ready push. Ready push prefers the heartbeat title hint, then the DB title, then the fixed 'Kilo session ready'. - services/session-ingest: forward the heartbeat title from UserConnectionDO.claimSessionReadyPush into SessionIngestDO and on to sendSessionReadyNotification. - Tests cover title composition, long-title and surrogate-pair truncation, whitespace handling, hint-vs-DB precedence, and schema strip-on-parse.
The Agents session list body/full-screen error was driven by the combined isError (stored || active), so a transient active-poll failure could wrongly surface a full-screen "Could not load sessions" (or a query-error-empty under the tray) instead of the intended first-use / no-past-sessions state. Active poll failures already surface only through the inline "Couldn't refresh" line via activeIsError, per the useAgentSessions contract. Wire the non-search body error to storedIsError so active blips never hide stored history.
Contributor
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Executive SummaryReviewed the mobile "Active now" tray UI/state-machine, the web active-sessions enrichment router (tenant-scoped, two-phase failure handling verified against the PR's stated non-collapse behavior), and the shared Files Reviewed (23 files)
Reviewed by claude-sonnet-5 · Input: 26 · Output: 7.6K · Cached: 607.6K Review guidance: REVIEW.md from base branch |
jeanduplessis
approved these changes
Jul 21, 2026
Base automatically changed from
feature/mobile-agent-attention-notifications
to
main
July 21, 2026 12:25
# Conflicts: # apps/mobile/src/components/agents/session-list-content.tsx # apps/mobile/src/components/agents/session-row.tsx # apps/mobile/src/components/ui/session-row.tsx # apps/web/src/routers/active-sessions-router.test.ts # apps/web/src/routers/active-sessions-router.ts # services/session-ingest/src/remote-session-notifications.test.ts # services/session-ingest/src/remote-session-notifications.ts
# Conflicts: # apps/web/src/routers/active-sessions-router.ts
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds a pinned "Active now" tray to the Agents session list. Live sessions are pinned above history and appear only in the tray (excluded from the history date sections), so an active session is never shown twice. Each tray row shows a platform badge (e.g.
CLI) — or aLIVEfallback when the session isn't enriched — alongside a live dot and a relative-time meta line.Stacked on top of
feature/mobile-agent-attention-notifications(PR #4605); please merge that first.What changed
activeSessions.listis enriched by joiningcli_sessions_v2for per-session platform + timestamps. Two-phase failure handling: a worker/parse failure degrades to an empty list; a DB failure during enrichment falls back to unenriched rows (never collapses the list).KNOWN_PLATFORMSrelocated to@kilocode/app-shared/platformsas the single source for web routers and the mobileotherbucket.selectPinnedActiveSessions,excludeActiveFromGroups,remoteAgentLabel/remoteMeta), theActiveNowSectiontray, asession-list-body-modelstate machine, and the section header / empty-body components. History rows are no longer markedlive; the tray owns live presentation viametaWhileLive. Loading→loaded uses an animated cross-fade; the inline "Couldn't refresh" retry line shows only when cached content is visible.Verification
LIVEfallback badge with empty history under the tray, and filtered active-only-empty with clear/restore.active-sessions-routertests pass.check:unused, and formatting are clean.Follow-ups (not in this PR)
sessions.heartbeatupdates into the mobile list so the tray updates without waiting on the 10s poll.LIVE/CLI) and meta to screen readers; today the row's accessibility label is the title only, matching the existing Home/history row pattern.