feat(frontend): Observability Drawer View Improvements#3783
Merged
mmabrouk merged 31 commits intorelease/v0.87.2from Feb 27, 2026
Merged
feat(frontend): Observability Drawer View Improvements#3783mmabrouk merged 31 commits intorelease/v0.87.2from
mmabrouk merged 31 commits intorelease/v0.87.2from
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
dosubot
bot
added
the
size:XXL
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
…llInView - Add "Raw" view mode to display original string values before auto-parsing - Rename "Rendered JSON" to "Field Rendering" for clarity - Preserve `originalStringValue` in PathItem for stringified JSON fields - Refactor view mode options logic to show appropriate modes based on data type - Remove text truncation from ReadOnlyTextView (now handled by editor config) - Consolidate field rendering logic and improve messages
…nd search functionality
…terals in DrillInContent
…t and DrillInFieldHeader components
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Contributor
Railway Preview Environment
Updated at 2026-02-27T19:47:03.740Z |
ardaerzin
requested changes
Feb 23, 2026
web/oss/src/components/SharedDrawers/TraceDrawer/components/AccordionTreePanel.tsx
Show resolved
Hide resolved
web/oss/src/components/SharedDrawers/TraceDrawer/components/AccordionTreePanel.tsx
Outdated
Show resolved
Hide resolved
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
mmabrouk
requested changes
Feb 26, 2026
Member
mmabrouk
left a comment
mmabrouk
left a comment
There was a problem hiding this comment.
Thank you @bekossy for this iteration.
My notes:
- Rendered JSON view should display if possible \n in the values as line breaks to improve the readability
-
Are you using the drill in for the chat messages too? (I did not check the code, but found it strange that it is quite different in the UI)
-
For some reason the UI changes when I have attachements in messages (see the switch action)
- Markdown and Text are swapped
- Whenever there is a JSON and Rendered JSON options, we should default to rendered JSON
…timize parsing with useMemo
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
…ks and update TraceSpanDrillInView to conditionally render attachments based on collapse state
…strings in LongTextNode and update related components for improved view modes
…AccordionTreePanel, and LongTextNode; add support for JSON5 and improve escaped string decoding
mmabrouk
approved these changes
Feb 27, 2026
Member
mmabrouk
left a comment
mmabrouk
left a comment
There was a problem hiding this comment.
Thanks @bekossy
- QA'ed the feature and it lgtm except for markdown/text switch
- Did not Review the code though (trust @ardaerzin there)
…T_MARKDOWN_VIEW command across components and update MarkdownModeSync logic for improved state management
ardaerzin
approved these changes
Feb 27, 2026
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
…utEffect for improved performance and add requestAnimationFrame for dispatching SET_MARKDOWN_VIEW command across DrillInContent, TraceSpanDrillInView, and AccordionTreePanel components.
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
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
3 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.
Uh oh!
There was an error while loading. Please reload this page.