[Bug][SkillOrchestra] run() silently drops imgs and execution kwargs before agent execution

[biancam5]

Describe the bug

SkillOrchestra.run() exposes an imgs parameter for multiple image inputs and also accepts **kwargs, but these values are not fully propagated to the selected agents during execution.

This creates a silent mismatch between the public API and the actual execution behavior: callers can provide multiple images or additional execution arguments, but the selected agent never receives them.

To Reproduce

This can be observed directly from the current implementation in:

swarms/structs/skill_orchestra.py

1. SkillOrchestra.run() accepts both imgs and **kwargs:

def run(
    self,
    task: str,
    img: Optional[str] = None,
    imgs: Optional[List[str]] = None,
    *args,
    **kwargs,
) -> Any:

2. When run() delegates execution to _execute_agents(), only img and **kwargs are passed:

results = self._execute_agents(
    selected, current_task, img=img, **kwargs
)

imgs is not forwarded.

3. _execute_agents() itself accepts **kwargs, but the selected agents are ultimately invoked with only task and img:

agent.run(task=agent_task, img=img)

and in the concurrent path:

executor.submit(
    agent.run, task=agent_task, img=img
)

So the additional keyword arguments are also discarded before reaching the agent.

For example:

orchestra.run(
    task="Analyze and compare these images",
    imgs=["image1.png", "image2.png"],
)

Based on the public API, the selected agent would be expected to receive the image list, but imgs is never propagated to the execution call.

Expected behavior

Arguments exposed by SkillOrchestra.run() should be consistently propagated to the selected agents.

In particular:

• imgs should reach the selected agent(s).
• Supported execution **kwargs should not be silently discarded.
• The behavior should be consistent for both single-agent and multi-agent execution paths.

Alternatively, if these arguments are intentionally unsupported, they should not be exposed by the public run() API.

Additional context

A regression test covering imgs and additional keyword-argument forwarding for both execution paths would help prevent this behavior from returning in future changes.

[github-actions]

Hello there, thank you for opening an Issue ! 🙏🏻 The team was notified and they will get back to you asap.

[kyegomez]

Heads-up on where this code now lives: SkillOrchestra was moved out of the library in #1933 (d59a2da9). It is no longer importable as from swarms import SkillOrchestra and now ships at examples/multi_agent/skill_orchestra_examples/skill_orchestra.py.

The bug you reported is real and still present in the moved copy — _execute_agents calls agent.run(task=agent_task, img=img), so imgs and any execution kwargs passed to run() are dropped before the agents execute.

What that means for the fix:

• PR fix(SkillOrchestra): propagate imgs and execution kwargs directly to agents (Fixes #1906) #1919 patches swarms/structs/skill_orchestra.py, which no longer exists, so it cannot merge as written and needs re-targeting at the new path.
• PR [Bug] SkillOrchestra drops imgs and execution kwargs before the agents run (#1906) #1927 fixed the same thing and is already closed.

Leaving this open, since the defect is unfixed wherever the file lives.

[ayaangazali]

Heads-up for whoever picks this up: the file moved in #1933. SkillOrchestra now lives at examples/multi_agent/skill_orchestra_examples/skill_orchestra.py and is no longer exported from swarms.structs.

The bug moved with it — on master at 36d40c3:

# run() accepts imgs …
def run(self, task, img=None, imgs=None, *args, **kwargs)

# … and _execute_agents has no imgs parameter at all
def _execute_agents(self, selected, task, img=None, **kwargs)

# … so the call site can only forward img
results = self._execute_agents(selected, current_task, img=img, **kwargs)

So imgs=[...] is still accepted, documented in the docstring, and silently dropped before any agent sees it. My PR #1927 for this was closed (it patched the old swarms/structs/ path, which the move made stale) — a fix now needs to target the new location.