Skip to content

[Fix #1195] Support stdin and change run task arguments from object to array - #1196

Open
ricardozanini wants to merge 3 commits into
open-workflow-specification:mainfrom
ricardozanini:issue/1195
Open

ricardozanini wants to merge 3 commits into
open-workflow-specification:mainfrom
ricardozanini:issue/1195

Conversation

@ricardozanini

Copy link
Copy Markdown
Collaborator

Summary

Update the run task schema to support stdin and change arguments from object to array format, with schema version bump to 1.0.4.

Changes

  • Add stdin property to script execution for piping input to scripts
  • Change arguments from object (key/value mapping) to array (argv list)
    • Applies to both script and shell processes
    • Aligns with standard CLI conventions for argument passing
    • Enables runtimes to pass arguments directly as argv to child processes
  • Update schema/workflow.yaml to version 1.0.4
  • Add new example: run-script-with-stdin-and-arguments.yaml
  • Update run-script-with-arguments.yaml to use array format
  • Update documentation in dsl-reference.md

Breaking Changes

  • arguments format change: Workflows using the run task with arguments in object format will need to be updated to use array format
  • Requires runtime updates: Runtimes will need to be updated to handle the new array format for arguments

Files Updated

  • schema/workflow.yaml - Schema version 1.0.3 → 1.0.4, script arguments type change, stdin property added
  • dsl-reference.md - Documentation updated for script and shell processes
  • examples/run-script-with-arguments.yaml - Updated to use array format
  • examples/run-script-with-stdin-and-arguments.yaml - New example showing stdin and array arguments
  • examples/run-shell-external-script.yaml - Updated to use array format

Fixes #1195

Copilot AI balanced review requested due to automatic review settings October 9, 2026 16:23

This comment was marked as off-topic.

Copilot AI balanced review requested due to automatic review settings October 9, 2026 16:27

This comment was marked as off-topic.

Copilot AI balanced review requested due to automatic review settings October 9, 2026 16:29

This comment was marked as off-topic.

Copilot AI balanced review requested due to automatic review settings October 9, 2026 16:37

This comment was marked as off-topic.

Copilot AI balanced review requested due to automatic review settings October 9, 2026 16:40

This comment was marked as off-topic.

…ask arguments from object to array

Update the run task schema to version 1.0.4:

- Add `stdin` property to script execution for piping input to scripts
- Change `arguments` from object (key/value mapping) to array (argv list)
  for both script and shell processes
- Align with standard CLI conventions for argument passing
- Update documentation and examples

Array format for arguments enables runtimes to pass arguments directly as
argv to child processes, and stdin support enables workflows to pipe data
to scripts as standard input.

Schema version bumped from 1.0.3 to 1.0.4.

Signed-off-by: Ricardo Zanini <ricardozanini@gmail.com>
Copilot AI balanced review requested due to automatic review settings October 9, 2026 16:42

This comment was marked as off-topic.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔵 Needs a closer look

Unresolved moderate issues remain in documentation and validation coverage, plus an inconsistent example version.

0 open findings

🧠 Review effort: Lite

Copilot AI lite review requested due to automatic review settings October 9, 2026 20:10

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI lite review requested due to automatic review settings October 9, 2026 22:28

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

This branch has not been deployed

No deployments
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.

Support stdin and change run task arguments from object to array

2 participants