Skip to content

fix(frontend): a pending interaction anywhere in the tab counts as awaiting - #5913

Merged
mmabrouk merged 1 commit into
release/v0.112.0from
fix/awaiting-covers-parked-interactions
Aug 10, 2026
Merged

fix(frontend): a pending interaction anywhere in the tab counts as awaiting#5913
mmabrouk merged 1 commit into
release/v0.112.0from
fix/awaiting-covers-parked-interactions

Conversation

@mmabrouk

Copy link
Copy Markdown
Member

Context

In session e627d80a territory: the "running somewhere else" strip flickered in the very tab that owned a parked connect card whenever a new turn started. The status publisher derives "awaiting" from hitlPending, which reads only the LAST assistant message; the moment a new turn begins streaming (or hydration reshapes the transcript), a still-pending interaction in an earlier message stops counting, status collapses toward idle, the settle stamp lands, and the #5857 strip logic sees "settled locally, running remotely" — the exact signature of another tab.

Changes

The status publisher scans the whole transcript: any pending client-tool interaction this tab renders means this tab owns the run, so the session publishes "awaiting" and the strip stays suppressed. One memo plus one condition; the strip's own derivation is untouched.

Tests

Type-check clean; the AgentChatSlice suite runs in CI. Live behavior is part of the compound-flow QA below.

What to QA

  • Drive an agent through form → connect → schedule in one conversation. The "running somewhere else" strip must never appear in the driving tab at any point, including the moments new turns begin.
  • Regression: a second tab watching the same session must still show the strip within ~15s.

@vercel

vercel Bot commented Aug 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agenta-documentation Error Error Aug 10, 2026 3:37pm

Request Review

@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@mmabrouk, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 6 minutes

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: 3c97af40-324f-4967-8df2-e48fc47bcaab

📥 Commits

Reviewing files that changed from the base of the PR and between 965851e and b670e85.

📒 Files selected for processing (1)
  • web/oss/src/components/AgentChatSlice/AgentConversation.tsx

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@dosubot dosubot Bot added size:S This PR changes 10-29 lines, ignoring generated files. bug report Something isn't working frontend labels Aug 10, 2026
…aiting

hitlPending reads only the LAST assistant message, so when a new turn
starts streaming (or hydration reshapes the transcript) a still-pending
interaction in an earlier message stopped counting: status collapsed to
idle, the settle stamp landed, and the running-elsewhere strip flickered
in the very tab that owned the parked widget (session e627d80a). The
status publisher now scans the whole transcript for pending interactions.
@mmabrouk
mmabrouk force-pushed the fix/awaiting-covers-parked-interactions branch from 093bd82 to b670e85 Compare August 10, 2026 15:37
@github-actions

github-actions Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Railway Preview Environment

Status Destroyed (PR closed)

Updated at 2026-08-10T16:51:30.431Z

@mmabrouk
mmabrouk merged commit 94a6813 into release/v0.112.0 Aug 10, 2026
62 of 64 checks passed
@mmabrouk mmabrouk added the lgtm This PR has been approved by a maintainer label Aug 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug report Something isn't working frontend lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant