Skip to content

Show error state in conversation details panel when run API fetch fails#10765

Merged
bnavetta merged 2 commits into
masterfrom
oz-agent/conversation-detail-panel-fetch-error
May 15, 2026
Merged

Show error state in conversation details panel when run API fetch fails#10765
bnavetta merged 2 commits into
masterfrom
oz-agent/conversation-detail-panel-fetch-error

Conversation

@bnavetta
Copy link
Copy Markdown
Contributor

@bnavetta bnavetta commented May 12, 2026

Description

When the /api/v1/agent/runs/ API call fails while fetching run info for the conversation details panel, the panel now shows an error state instead of silently displaying a bare "Cloud agent run" title with no details.

Instead, we should show the error message so that the user knows something went wrong and can troubleshoot or report the error.

Testing

  • Verified compilation with cargo check --package warp
  • Verified formatting with cargo fmt

Screenshots / Videos

https://www.loom.com/share/e7816b6081e1445c9a694736bc33ea77

Agent Mode

  • Warp Agent Mode - This PR was created via Warp's AI Agent Mode

CHANGELOG-BUG-FIX: The details panel for cloud agent runs now displays any failures fetching the agent metadata

Conversation: https://staging.warp.dev/conversation/b6e0e0e7-f9c7-45de-b750-20cdc029ad8e
Run: https://oz.staging.warp.dev/runs/019e1dc6-cd84-74b9-a544-8e704c214bd9
This PR was generated with Oz.

When the /api/v1/agent/runs/ API call fails to load run data, the
conversation details panel now displays an error banner with a warning
icon and "Failed to load run details" message instead of silently
showing an empty panel.

Changes:
- Added is_task_fetch_failed() to AgentConversationsModel to expose
  whether a task fetch has failed (permanently or transiently)
- Added fetch_error field to ConversationDetailsData, populated when
  from_task_id is called with a failed fetch state
- Renders a red error banner in the panel when fetch_error is set
- Updated both native (view_impl.rs) and WASM (wasm_view.rs) callers
  to check for fetch failure and propagate it to the details panel

Co-Authored-By: Oz <oz-agent@warp.dev>
@cla-bot cla-bot Bot added the cla-signed label May 12, 2026
@bnavetta bnavetta force-pushed the oz-agent/conversation-detail-panel-fetch-error branch from a0f8dac to a43644c Compare May 15, 2026 11:31
@bnavetta bnavetta marked this pull request as ready for review May 15, 2026 11:34
@bnavetta bnavetta requested a review from a team May 15, 2026 11:34
@oz-for-oss
Copy link
Copy Markdown
Contributor

oz-for-oss Bot commented May 15, 2026

@bnavetta

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I completed the review and no human review was requested for this pull request.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

Copy link
Copy Markdown
Contributor

@oz-for-oss oz-for-oss Bot left a comment

Choose a reason for hiding this comment

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

Overview

This PR stores the failed run-fetch message in the task fetch state, emits a task update when the fetch reaches a terminal failure, and renders/copies that message in the conversation details panel for both native and WASM paths.

Concerns

  • None.

Verdict

Found: 0 critical, 0 important, 0 suggestions

Approve

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

Instead of a generic 'Failed to load run details' message, the error
banner now shows the actual error description from the server (e.g.
'403 Forbidden', '500 Internal Server Error'). This is done by storing
the error message string alongside the timestamp in TaskFetchState and
threading it through task_fetch_error() -> from_task_id() -> the UI.

Co-Authored-By: Oz <oz-agent@warp.dev>
@bnavetta bnavetta force-pushed the oz-agent/conversation-detail-panel-fetch-error branch from a43644c to 1205aab Compare May 15, 2026 11:40
Copy link
Copy Markdown
Member

@ianhodge ianhodge left a comment

Choose a reason for hiding this comment

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

looks good to me!

@bnavetta bnavetta merged commit 3efab9a into master May 15, 2026
26 checks passed
@bnavetta bnavetta deleted the oz-agent/conversation-detail-panel-fetch-error branch May 15, 2026 14:53
lawsmd pushed a commit to lawsmd/cortex that referenced this pull request May 22, 2026
…ls (warpdotdev#10765)

## Description
When the `/api/v1/agent/runs/` API call fails while fetching run info
for the conversation details panel, the panel now shows an error state
instead of silently displaying a bare "Cloud agent run" title with no
details.

Instead, we should show the error message so that the user knows
something went wrong and can troubleshoot or report the error.

## Testing
- [x] Verified compilation with `cargo check --package warp`
- [x] Verified formatting with `cargo fmt`

### Screenshots / Videos

https://www.loom.com/share/e7816b6081e1445c9a694736bc33ea77

## Agent Mode
- [x] Warp Agent Mode - This PR was created via Warp's AI Agent Mode

CHANGELOG-BUG-FIX: The details panel for cloud agent runs now displays
any failures fetching the agent metadata

_Conversation:
https://staging.warp.dev/conversation/b6e0e0e7-f9c7-45de-b750-20cdc029ad8e_
_Run:
https://oz.staging.warp.dev/runs/019e1dc6-cd84-74b9-a544-8e704c214bd9_
_This PR was generated with [Oz](https://warp.dev/oz)._

---------

Co-authored-by: Oz <oz-agent@warp.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants