stacklok / toolhive / 28622575041

68%

Build Type

push

github

Committed by web-flow

Commit Message

Sort aggregated tools by name for deterministic embedding order (#5709)

Sort merged tools by name for deterministic order

MergeCapabilities builds the advertised tool slice by ranging a map,
so every fresh aggregation produced a different order. The order is
invisible in tools/list but leaks into order-sensitive consumers: the
optimizer embeds the tool catalog in slice order, so each aggregation
sent a differently ordered embedding batch.

Sort the slice by resolved name, which is unique after conflict
resolution. Same class of fix as #3450, which sorted discovered
backends but not the tools inside the aggregation.

Fixes #5702

Signed-off-by: Gabriel Cosi <contact@gabrielcosi.dev>

Coverage Stats

3 of 3 new or added lines in 1 file covered. (100.0%)

10 existing lines in 3 files now uncovered.

71309 of 105415 relevant lines covered (67.65%)

64.55 hits per line