Skip to content

feat: display tool error text in modal dialog#11

Merged
dsfaccini merged 1 commit intopydantic:mainfrom
dsfaccini:fix/tool-error-modal
Feb 13, 2026
Merged

feat: display tool error text in modal dialog#11
dsfaccini merged 1 commit intopydantic:mainfrom
dsfaccini:fix/tool-error-modal

Conversation

@dsfaccini
Copy link
Copy Markdown
Contributor

@dsfaccini dsfaccini commented Jan 22, 2026

Summary

  • Fix Part.tsx to include output-error state when rendering ToolOutput
  • Pass part.errorText to ToolOutput component instead of hardcoded empty string
  • Add "View Error" button that opens a dialog modal with full error text
  • Include copy button in modal for easy error copying
  • Update dependencies via bun update

Test plan

  • Trigger a tool error (e.g. invalid tool input or backend failure)
  • Verify "Error" badge shows on tool header
  • Click to expand tool, verify "View Error" button appears
  • Click button, verify modal opens with full error text
  • Verify copy button works

🤖 Generated with Claude Code

- Fix Part.tsx to include 'output-error' state when rendering ToolOutput
- Pass part.errorText to ToolOutput component
- Add "View Error" button that opens a dialog modal with full error text
- Include copy button in modal for easy error copying
- Update dependencies

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@gaby
Copy link
Copy Markdown

gaby commented Feb 12, 2026

I think this also fixes #14

@dsfaccini dsfaccini merged commit eeba30c into pydantic:main Feb 13, 2026
github-actions Bot pushed a commit that referenced this pull request Feb 16, 2026
# [1.1.0](v1.0.0...v1.1.0) (2026-02-16)

### Features

* display tool error text in modal dialog ([#11](#11)) ([eeba30c](eeba30c))
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants