Skip to content

Conversation

@shalarn1
Copy link
Collaborator

@shalarn1 shalarn1 commented Oct 20, 2025

Adds supportedAggregations and orderByAttribute fields to the Eng Intel Registry API response, along with enhanced documentation to guide LLM agents toward correct query construction including utilizing alphabetical tool ordering.

LLMs might give more weight to tools earlier in the context (they shouldn’t but they might). Tags are sorted alphabetically in CortexOpenAPICustomizer so the queryPointInTimeMetrics tool (tag: "Eng Intel: Metrics") was showing up before listMetricDefinitions (tag: "Eng Intel: Registry")

This adds numeric tag prefixes to control alphabetical ordering:
"Eng Intel: 1-Registry" → appears first
"Eng Intel: 2-Metrics" → appears second

This method with standard OpenAPI tag sorting and required no code changes but if we find tool ordering is important we should implement a proper tool ordering system https://cortex1.atlassian.net/browse/CET-22186

https://github.com/cortexapps/brain-backend/pull/12340

@shalarn1 shalarn1 requested a review from maddymanu October 20, 2025 21:59
@maddymanu maddymanu merged commit b9907f8 into master Oct 27, 2025
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.

3 participants