feat(frontend): Observability Drawer View Improvements#3783
Open
feat(frontend): Observability Drawer View Improvements#3783
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…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
Contributor
Railway Preview Environment
Updated at 2026-02-27T09:36:26.812Z |
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
Member
Author
|
@codex review |
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
…f github.com:Agenta-AI/agenta into enhancement-/-observability-drawer-view-improvements
Member
Author
|
@codex review |
mmabrouk
requested changes
Feb 26, 2026
Member
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
…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
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
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.