Skip to content

fix(swarm-dispatch): correct CLI flag order and add missing prompt arg#567

Open
MGAura wants to merge 1 commit into
outsourc-e:mainfrom
MGAura:fix/swarm-dispatch-args-order
Open

fix(swarm-dispatch): correct CLI flag order and add missing prompt arg#567
MGAura wants to merge 1 commit into
outsourc-e:mainfrom
MGAura:fix/swarm-dispatch-args-order

Conversation

@MGAura

@MGAura MGAura commented May 29, 2026

Copy link
Copy Markdown

Bug Fix

Two bugs in the runWorker() function in swarm-dispatch.ts prevented swarm workers from receiving their assignments correctly:

Bug 1: Flag order — -q before -Q

The CLI expects -Q (quiet mode) before -q (no-banner). The wrapper path had them reversed, causing the flags to be ignored or misinterpreted.

Bug 2: Missing prompt argument on non-wrapper path

The non-wrapper branch (fallback when no worker wrapper script exists) was missing the prompt argument entirely. This meant the worker would start without any task description — effectively a no-op.

Changes

- ? ['chat', '-q', '-Q', '--yolo', '--ignore-rules', '--source', 'swarm-dispatch', prompt]
- : ['chat', '-q', '-Q', '--yolo', '--ignore-rules', '--source', 'swarm-dispatch']
+ ? ['chat', '-Q', '-q', prompt, '--yolo', '--ignore-rules', '--source', 'swarm-dispatch']
+ : ['chat', '-Q', '-q', prompt, '--yolo', '--ignore-rules', '--source', 'swarm-dispatch']

Testing

  • Patched locally and confirmed workers receive assignments and execute tasks correctly via the conductor/dispatch path.

… path

- Move -Q before -q in wrapper args (CLI expects -Q first)
- Add prompt argument to non-wrapper branch (was missing entirely,
  causing worker to start with no task description)

These two bugs prevented swarm workers from receiving their assignments
correctly when invoked via the conductor/dispatch path.
meefs pushed a commit to meefs/hermes-workspace that referenced this pull request Jun 5, 2026
… PR outsourc-e#567: fix swarm-dispatch CLI flag order + missing prompt arg (MGAura)
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