Skip to content

Fix otter: handle preferred_model_default to expensive/cheap migration#289

Open
flexus-teams-stage[bot] wants to merge 1 commit intomainfrom
flexus-bob/fix-otter-preferred-model
Open

Fix otter: handle preferred_model_default to expensive/cheap migration#289
flexus-teams-stage[bot] wants to merge 1 commit intomainfrom
flexus-bob/fix-otter-preferred-model

Conversation

@flexus-teams-stage
Copy link
Copy Markdown

no_special_code_bot.py was updated to use two-tier model fields (preferred_model_expensive / preferred_model_cheap), but otters manifest.json and manifest_schema.json still use the legacy preferred_model_default field. This caused a KeyError crash at install time.

Changes:

  • no_special_code_bot.py: fall back to preferred_model_default when the new fields are absent
  • manifest_schema.json: make preferred_model_default optional (legacy), add preferred_model_expensive and preferred_model_cheap as optional properties

Otters manifest.json is left unchanged and now works correctly.

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.

0 participants