Skip to content

fix: make platform/model filter an enum in llmo apis#1971

Open
aniham wants to merge 2 commits intomainfrom
model-enum
Open

fix: make platform/model filter an enum in llmo apis#1971
aniham wants to merge 2 commits intomainfrom
model-enum

Conversation

@aniham
Copy link
Contributor

@aniham aniham commented Mar 17, 2026

Summary

Adds LLM model enum validation to the Brand Presence endpoints (filter-dimensions and weeks). The model query parameter is validated against the llm_model enum from mysticat-data-service; invalid values return 400.

Changes

  • Enum values (from mysticat-data-service llm_model): chatgpt-paid, chatgpt-free, google-ai-overview, perplexity, google-ai-mode, copilot, gemini, google, microsoft, mistral, anthropic, amazon
  • Default model: chatgpt-free (was chatgpt)
  • Validation: Invalid model returns 400 with message: Invalid model. Must be one of: ...
  • Docs: Updated filter-dimensions and weeks API docs with enum values and error behavior
  • Tests: Added returns badRequest when model is invalid for both handlers; updated default model expectations

@github-actions
Copy link

This PR will trigger a patch release when merged.

@codecov
Copy link

codecov bot commented Mar 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@aniham aniham changed the title fix: make model in executions table an enum fix: make platform/model filter an enum in llmo apis Mar 17, 2026
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.

2 participants