Skip to content

Resolve and display run argument values in IO nodes - #2559

Draft
Mbeaulne wants to merge 1 commit into
masterfrom
07-22-fix_v2_input_value_on_runs
Draft

Resolve and display run argument values in IO nodes#2559
Mbeaulne wants to merge 1 commit into
masterfrom
07-22-fix_v2_input_value_on_runs

Conversation

@Mbeaulne

@Mbeaulne Mbeaulne commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Description

Introduces a resolveInputValue utility that determines the correct value to display for an input node by following a priority order: run-time task arguments first, then the configured value, and finally the default value. This ensures that when viewing a run, the actual argument values passed to the task are shown rather than always falling back to the default.

The IONode and RunViewInputDetails components now display this resolved value, and a dedicated "Value" field is shown in the input details panel when a value is available. The "Default Value" field is also corrected to only render when defaultValue is not undefined (previously it would hide when falsy).

Related Issue and Pull requests

Type of Change

  • Bug fix
  • New feature
  • Improvement
  • Cleanup/Refactor
  • Breaking change
  • Documentation update

Checklist

  • I have tested this does not break current pipelines / runs functionality
  • I have tested the changes on staging

Screenshots (if applicable)

Test Instructions

  1. Open a run that has task arguments passed at runtime.
  2. Inspect an input node — the node card and details panel should display the run-time argument value rather than the default.
  3. Verify that an empty string argument ("") is preserved and shown rather than falling back to the default value.
  4. Verify that when no run-time argument exists, the configured value is shown, and if that is also absent, the default value is shown.
  5. Confirm the "Default Value" field still appears correctly in the details panel when a default is set.

Additional Comments

Unit tests for resolveInputValue cover all three priority levels, including the edge case of an empty string argument being preserved.

@github-actions

Copy link
Copy Markdown

🎩 Preview

A preview build has been created at: 07-22-fix_v2_input_value_on_runs/b2c4d93

Copy link
Copy Markdown
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@Mbeaulne Mbeaulne changed the title Fix v2 input value on runs Resolve and display run argument values in IO nodes Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant