feat: v2 - runview fast preview button for outputs#2524
Draft
maxy-shpfy wants to merge 1 commit into
Draft
Conversation
This was referenced Jul 14, 2026
🎩 PreviewA preview build has been created at: |
This was referenced Jul 14, 2026
Collaborator
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
4c23003 to
c53483e
Compare
00012e0 to
2f22614
Compare
|
I dig it! 👏 |
morgan-wowk
requested changes
Jul 15, 2026
morgan-wowk
left a comment
There was a problem hiding this comment.
Top-hatting feedback:
- It does not feel like a button right now. Either there is no hover effect or it is very faint. This makes it difficult to know where the selectable region is that would trigger the button click.
c53483e to
208dffd
Compare
208dffd to
fefd7e3
Compare
2f22614 to
c31b783
Compare
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.

Description
Adds artifact preview buttons directly to task output rows in the RunView pipeline graph. When a task has completed execution, each output row now shows an
ArtifactVisualizertrigger inline, allowing users to preview output artifacts without opening the full Artifacts panel.To support this without coupling the shared
TaskNodeCardto execution-specific logic, anOutputActionsContextwas introduced. This context accepts an optional render function for per-output actions.RunViewTaskNodeprovides this renderer, which fetches execution artifacts viagetExecutionArtifactsand maps each output to aRunViewOutputPreviewcomponent. The preview is suppressed when the artifact is missing, has no URI or inline value, or exceeds 50 MB.Related Issue and Pull requests
Type of Change
Checklist
Screenshots (if applicable)
Screen Recording 2026-07-14 at 1.46.15 PM.mov (uploaded via Graphite)
Test Instructions
Additional Comments
The
OutputActionsContextis intentionally generic so other views can inject their own per-output actions in the future without modifyingTaskNodeCard.