Skip to content

chore: import model options from agent-relay SDK - #54

Merged
khaliqgant merged 1 commit into
mainfrom
chore/centralize-model-registry
Mar 2, 2026
Merged

chore: import model options from agent-relay SDK#54
khaliqgant merged 1 commit into
mainfrom
chore/centralize-model-registry

Conversation

@khaliqgant

@khaliqgant khaliqgant commented Feb 21, 2026

Copy link
Copy Markdown
Member

Summary

  • Remove hardcoded model options from SpawnModal.tsx
  • Import CLAUDE_MODEL_OPTIONS, CURSOR_MODEL_OPTIONS, CODEX_MODEL_OPTIONS, GEMINI_MODEL_OPTIONS from agent-relay/broker
  • Use DefaultModels for fallback values instead of hardcoded strings
  • SettingsPage.tsx also imports from SDK

Why

Centralizes model definitions in packages/shared/cli-registry.yaml in the relay repo. This enables:

  • Single source of truth for CLI versions and models
  • Automated version checking via GitHub Action
  • Consistent model options across dashboard, SDK, and Python SDK

Dependencies

Requires agent-relay@^2.3.14 which includes the new CLI registry exports.

Test plan

  • Verify SpawnModal model dropdowns work correctly
  • Verify SettingsPage model selections work correctly
  • Verify defaults are applied correctly

🤖 Generated with Claude Code


Open with Devin

- Remove hardcoded model options from SpawnModal.tsx
- Import CLAUDE_MODEL_OPTIONS, CURSOR_MODEL_OPTIONS, etc. from agent-relay/broker
- Use DefaultModels for fallback values
- SettingsPage.tsx also imports from SDK

Source of truth is now packages/shared/cli-registry.yaml in relay repo.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

@khaliqgant khaliqgant mentioned this pull request Feb 24, 2026
@khaliqgant
khaliqgant merged commit fcc3ead into main Mar 2, 2026
1 check failed
@khaliqgant
khaliqgant deleted the chore/centralize-model-registry branch March 2, 2026 05:31
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