Skip to content

Conversation

@edxeth
Copy link

@edxeth edxeth commented Feb 1, 2026

Summary

  • Fix doctor command showing incorrect/missing variants in model resolution output
  • OmoConfig interface was missing variant property, causing doctor to display variants from ModelRequirement fallback chain instead of user's config

Changes

  • Add variant to OmoConfig agent/category entries
  • Add userVariant to AgentResolutionInfo and CategoryResolutionInfo
  • Update getEffectiveVariant() to prioritize user variant over requirement defaults
  • Add 2 tests verifying variant capture for agents and categories

Before/After

Agent Config Before After
oracle xhigh high xhigh ✓
metis xhigh max xhigh ✓
momus max medium max ✓
librarian high (missing) high ✓
explore high (missing) high ✓
multimodal-looker high (missing) high ✓

Summary by cubic

Fixes the doctor command to show user-configured model variants in the model resolution output. Previously, variants came from the fallback chain, leading to wrong or missing values.

  • Bug Fixes
    • Add variant to OmoConfig agent/category entries.
    • Capture userVariant in AgentResolutionInfo and CategoryResolutionInfo.
    • Prioritize user variant in getEffectiveVariant.
    • Add tests for agent and category variant capture.

Written for commit 7e00516. Summary will update on new commits.

OmoConfig interface was missing variant property, causing doctor to show
variants from ModelRequirement fallback chain instead of user's config.

- Add variant to OmoConfig agent/category entries
- Add userVariant to resolution info interfaces
- Update getEffectiveVariant to prioritize user variant
- Add tests verifying variant capture
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

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