Skip to content

fix: Preserve --suite task execution order (#346) - #347

Merged
olearycrew merged 1 commit into
mainfrom
feat/346-suite-task-order
Apr 22, 2026
Merged

fix: Preserve --suite task execution order (#346)#347
olearycrew merged 1 commit into
mainfrom
feat/346-suite-task-order

Conversation

@ScuttleBot

Copy link
Copy Markdown
Contributor

Closes #346

When using --suite to specify tasks, the execution order now matches the order specified in the --suite parameter instead of always following the manifest.yaml order.

This enables parallel testing scenarios where multiple nodes need to execute tasks in different orders to distribute workload and avoid cache concentrated hits.

Changes

  • Modified task selection to use --suite order instead of manifest order
  • Tasks not found in the manifest are silently skipped (same as before)

🤖 This PR was opened by @olearycrew's OpenClaw bot. Please review carefully!

When using --suite to specify tasks, the execution order now matches
the order specified in the --suite parameter instead of always following
the manifest.yaml order.
@ScuttleBot
ScuttleBot requested a review from olearycrew April 22, 2026 14:01
@kilo-code-bot

kilo-code-bot Bot commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

The refactor correctly solves the ordering problem. By building a lookup map and iterating over task_ids, tasks are now returned in the caller-specified suite order rather than the manifest order. Tasks not found in the manifest are silently skipped — consistent with the previous behavior.

Files Reviewed (1 file)
  • scripts/benchmark.py

Fix these issues in Kilo Cloud


Reviewed by claude-4.6-sonnet-20260217 · 75,947 tokens

@olearycrew
olearycrew merged commit 5f9d2f3 into main Apr 22, 2026
2 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.

[FEATURE] Support customizable task execution order via --suite parameter

2 participants