Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[OPIK-4325] [FE] Sync prompt view components across Trace View and Prompt Library #5128

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
BorisTkachenko merged 20 commits into from
Feb 12, 2026
Merged

[OPIK-4325] [FE] Sync prompt view components across Trace View and Prompt Library #5128

BorisTkachenko merged 20 commits into from
Feb 12, 2026

Conversation

@BorisTkachenko
Copy link
Contributor

@BorisTkachenko BorisTkachenko commented Feb 9, 2026 •
edited
Loading

Details

This PR unifies the prompt display components across different parts of the application to ensure consistency between the Trace View Prompts tab and the Prompt Library.

Screen.Recording.2026-02-09.at.10.44.44.mov

Key changes:

  • Created a new shared PromptMessagesReadonly component that handles displaying chat messages with proper role labels and media attachments (images, videos, audios)
  • Added a toggle button in the Trace View Prompts tab to switch between "Message view" (formatted cards) and "Raw view" (JSON syntax highlighting)
  • Updated TextPromptView in the Prompt Library to use the same message card display for JSON message arrays
  • Refactored ChatPromptMessageReadonly to use the shared PromptMessageCard component
  • Removed the obsolete ChatPromptAttachments component in favor of inline media rendering

Change checklist

  • User facing
  • Documentation update

Issues

  • Resolves #
  • OPIK-4325

Testing

  • Navigate to a trace with prompts in the Prompts tab and verify the message view displays correctly
  • Toggle between "Message view" and "Raw view" and ensure both render properly
  • Navigate to the Prompt Library and view a chat-style prompt to confirm consistent display
  • Test prompts with media attachments (images, videos, audios) to verify they render correctly

Documentation

N/A

@BorisTkachenko BorisTkachenko self-assigned this Feb 9, 2026
@BorisTkachenko BorisTkachenko added the test-environment Deploy Opik adhoc environment label Feb 9, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2026

🔄 Test environment deployment process has started

Phase 1: Deploying base version 1.10.8-4136 (from main branch) if environment doesn't exist
Phase 2: Building new images from PR branch boryst/sync-prompts-view-in-different-places
Phase 3: Will deploy newly built version after build completes

You can monitor the progress here.

@CometActions
Copy link
Collaborator

CometActions commented Feb 9, 2026

✅ Test environment is now available!

To configure additional Environment variables for your environment, run [Deploy Opik AdHoc Environment workflow] (https://github.com/comet-ml/comet-deployment/actions/workflows/deploy_opik_adhoc_env.yaml)

Access Information

The deployment has completed successfully and the version has been verified.

@BorisTkachenko BorisTkachenko changed the title [OPIK-000] [FE] Sync prompts view in different places [OPIK-4325] [FE] Sync prompt view components across Trace View and Prompt Library Feb 9, 2026
@BorisTkachenko BorisTkachenko force-pushed the boryst/sync-prompts-view-in-different-places branch from a26ed0e to c316f24 Compare February 9, 2026 11:59
@BorisTkachenko BorisTkachenko marked this pull request as ready for review February 9, 2026 12:16
@BorisTkachenko BorisTkachenko requested a review from a team as a code owner February 9, 2026 12:16
aadereiko
aadereiko requested changes Feb 9, 2026
View reviewed changes
Copy link
Collaborator

@aadereiko aadereiko left a comment

Choose a reason for hiding this comment

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

If it comes to functionality, works fine :) A good improvement. However, I've mentioned some code improvements that I think should be tackled before merging it
Let me know if you have any questions

aadereiko
aadereiko previously approved these changes Feb 9, 2026
View reviewed changes
Copy link
Collaborator

@aadereiko aadereiko left a comment

Choose a reason for hiding this comment

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

Thank you for addressing the comments! Good work :)

@BorisTkachenko BorisTkachenko force-pushed the boryst/sync-prompts-view-in-different-places branch from 59c0447 to 39995a5 Compare February 11, 2026 10:03
@github-actions github-actions bot added python Pull requests that update Python code Python SDK labels Feb 11, 2026
@BorisTkachenko BorisTkachenko added test-environment Deploy Opik adhoc environment and removed test-environment Deploy Opik adhoc environment labels Feb 11, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 11, 2026

🔄 Test environment deployment process has started

Phase 1: Deploying base version 1.10.11-4168 (from main branch) if environment doesn't exist
Phase 2: Building new images from PR branch boryst/sync-prompts-view-in-different-places
Phase 3: Will deploy newly built version after build completes

You can monitor the progress here.

@CometActions
Copy link
Collaborator

CometActions commented Feb 11, 2026

✅ Test environment is now available!

To configure additional Environment variables for your environment, run [Deploy Opik AdHoc Environment workflow] (https://github.com/comet-ml/comet-deployment/actions/workflows/deploy_opik_adhoc_env.yaml)

Access Information

The deployment has completed successfully and the version has been verified.

@BorisTkachenko BorisTkachenko removed the test-environment Deploy Opik adhoc environment label Feb 11, 2026
@BorisTkachenko BorisTkachenko force-pushed the boryst/sync-prompts-view-in-different-places branch from 293283b to b798f18 Compare February 12, 2026 08:33
@github-actions
Copy link
Contributor

github-actions bot commented Feb 12, 2026 •
edited
Loading

SDK E2E Tests Results

172 tests   169 ✅  5m 0s ⏱️
  1 suites    2 💤
  1 files      1 ❌

For more details on these failures, see this check.

Results for commit bd8e21f.

♻️ This comment has been updated with latest results.

aadereiko
aadereiko approved these changes Feb 12, 2026
View reviewed changes
Copy link
Collaborator

@aadereiko aadereiko left a comment

Choose a reason for hiding this comment

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

Nice work, thank you for addressing the comments :)

@BorisTkachenko BorisTkachenko merged commit ced5ec8 into main Feb 12, 2026
39 of 115 checks passed
@BorisTkachenko BorisTkachenko deleted the boryst/sync-prompts-view-in-different-places branch February 12, 2026 09:49
@AndreiCautisanu AndreiCautisanu mentioned this pull request Feb 12, 2026
Merged
2 tasks
miguelgrc pushed a commit that referenced this pull request Feb 12, 2026
@BorisTkachenko @miguelgrc
[OPIK-4325] [FE] Sync prompt view components across Trace View and Pr…
94620e8 
…ompt Library (#5128)

* Adjust prompt library

* Adjust Trace View Prompts tab

* address comments

* Address comments

* Address design review

* fix

* Address comments

* fix

* fix

* address comments

* address comments

* address comment

* fix

* address comments

* align padding

* address comment

* address comment

* fix linting

* fix

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

Reviewers

@aadereiko aadereiko aadereiko approved these changes

+1 more reviewer

@baz-reviewer baz-reviewer[bot] baz-reviewer[bot] approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@BorisTkachenko BorisTkachenko

Labels

Frontend Python SDK python Pull requests that update Python code test-environment Deploy Opik adhoc environment typescript *.ts *.tsx

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@BorisTkachenko @CometActions @aadereiko