Skip to content

fix(web): show Waiting for blocked chats#631

Closed
IskanVa wants to merge 2 commits intopingdotgg:mainfrom
IskanVa:codex/awaiting-response-sidebar
Closed

fix(web): show Waiting for blocked chats#631
IskanVa wants to merge 2 commits intopingdotgg:mainfrom
IskanVa:codex/awaiting-response-sidebar

Conversation

@IskanVa
Copy link
Copy Markdown

@IskanVa IskanVa commented Mar 9, 2026

What Changed

Show Waiting in the sidebar when a thread is blocked on unresolved structured user input, not just permission approvals.

This change:

  • adds shared thread-status derivation in session-logic
  • updates the sidebar to distinguish blocked threads from actively running threads
  • adds unit coverage for status precedence
  • adds a browser regression test for the sidebar state

Why

Before this change, a thread could still appear as Working in the sidebar even when the agent had already stopped and was waiting for the user to answer a structured prompt.

That made the sidebar status misleading and made it harder to notice that the next action was required from the user.

UI Changes

Before:

  • blocked-on-user-input threads still appeared as Working
sidebar-status-before

After:

  • blocked-on-user-input threads appear as Waiting
sidebar-status-after

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for the UI change
  • I included a video for animation/interaction changes

Note

Display a 'Waiting' status pill for blocked threads in the web Sidebar and add awaiting-response snapshot support in ChatView.browser.tsx

Introduce deriveThreadStatusState to drive Sidebar pills, mapping 'awaiting-response' to 'Waiting'; refactor Sidebar to use the derived status; add an awaiting-response orchestration snapshot and tests in apps/web/src/components/ChatView.browser.tsx and apps/web/src/session-logic.test.ts; implement status derivation in apps/web/src/session-logic.ts.

📍Where to Start

Start with deriveThreadStatusState in apps/web/src/session-logic.ts, then review its use in Sidebar via threadStatusPill in apps/web/src/components/Sidebar.tsx.

Macroscope summarized a5764d7.

@github-actions github-actions bot added the vouch:unvouched PR author is not yet trusted in the VOUCHED list. label Mar 9, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 9, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 7b4b38af-21b1-4781-8abd-0eb89b42d708

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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 and usage tips.

@IskanVa IskanVa changed the title fix(web): show Awaiting response for blocked chats fix(web): show Waiting for blocked chats Mar 9, 2026
@lblclass
Copy link
Copy Markdown

lblclass commented Mar 9, 2026

@copilot what the code using for?

Copy link
Copy Markdown
Contributor

@binbandit binbandit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@maria-rcks
Copy link
Copy Markdown
Collaborator

#701

@maria-rcks maria-rcks closed this Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants