Skip to content

[Fix #1186] Allow any type in for.in inline array - #1187

Merged
ricardozanini merged 1 commit into
open-workflow-specification:mainfrom
ricardozanini:issues/1186
Aug 11, 2026
Merged

[Fix #1186] Allow any type in for.in inline array#1187
ricardozanini merged 1 commit into
open-workflow-specification:mainfrom
ricardozanini:issues/1186

Conversation

@ricardozanini

Copy link
Copy Markdown
Collaborator

Summary

  • Removes the type: object constraint from inline array items in for.in, allowing iteration over arrays of any type (strings, numbers, booleans, objects, etc.)
  • Updates the DSL reference to reflect the type change from array<object> to array
  • Adds a new example (for-inline-primitive-array.yaml) demonstrating iteration over an inline array of numbers

Fixes #1186

Test plan

  • All 185 schema validation tests pass
  • Verify the new example validates correctly in CI

…e array

Signed-off-by: Ricardo Zanini <ricardozanini@gmail.com>
Copilot AI lite review requested due to automatic review settings August 11, 2026 16:26

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.

Pull request overview

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Updates the workflow schema/docs to allow for.in inline arrays to contain primitives (and other non-object values), and adds an example demonstrating primitive iteration.

Changes:

  • Relax schema/workflow.yaml so inline for.in arrays no longer require items.type: object
  • Update DSL reference typing for for.in from array<object> to array
  • Add a new YAML example iterating over an inline numeric array

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
schema/workflow.yaml Removes the object-only constraint for inline for.in array items
dsl-reference.md Updates the reference typing/description to match the schema change
examples/for-inline-primitive-array.yaml Adds a primitive-array iteration example for for.in

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread dsl-reference.md
Comment thread examples/for-inline-primitive-array.yaml
Comment thread examples/for-inline-primitive-array.yaml

@cdavernas cdavernas left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM! Cheers ❤️

@ricardozanini
ricardozanini merged commit 0ff936f into open-workflow-specification:main Aug 11, 2026
3 checks passed
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.

Using any as type of hard coded array for For task

3 participants