Skip to content

Seed <repeat default=N> only when a request omits the repeat#23170

Draft
guerler wants to merge 2 commits into
galaxyproject:devfrom
guerler:toolrequests.004
Draft

Seed <repeat default=N> only when a request omits the repeat#23170
guerler wants to merge 2 commits into
galaxyproject:devfrom
guerler:toolrequests.004

Conversation

@guerler

@guerler guerler commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Follow up for: #23084, focussing on repeat seeding.

This change ensures repeat parameters consistently honor their declared default value during tool request execution. If a repeat is omitted from the request, the default number of instances is created, if an explicit empty list is provided, zero instances are preserved, and min continues to enforce the minimum number of instances. The default value is clamped to the configured [min, max] range during parsing, matching the parameter model. This prevents tools that rely on default repeat values from receiving malformed inputs while preserving the semantics of structured tool requests.

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

@guerler
guerler force-pushed the toolrequests.004 branch from ad953fa to cf392ed Compare July 26, 2026 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant