Skip to content

[WIP] Add models command for OpenRouter model search and selection - #3

Merged
bhadraagada merged 1 commit into
feat/models-commandfrom
copilot/sub-pr-1-again
Jan 3, 2026
Merged

bhadraagada merged 1 commit into
feat/models-commandfrom
copilot/sub-pr-1-again

Conversation

Copilot AI commented Jan 3, 2026 •

Copy link
Copy Markdown
  • Fix model name in OpenRouter-minimax template (minimax-m1-80k vs minimax-m2.1)
  • Add cache expiration mechanism for model cache
  • Consider unifying searchModelsInteractive and selectModelInteractive
  • Add test coverage for models command
  • Fix displayName for OpenRouter template (remove specific model from name)
  • Restore removed provider templates (if unintentional)
  • Remove duplicate OpenRouterModel interface - import from models.ts
  • Remove duplicate formatContext function - extract to shared utility
  • Add validation for manually entered model IDs
  • Improve OpenRouter detection logic with proper URL parsing
  • Add NaN validation for parseFloat operations on pricing
  • Add fallback handling for undefined model selection
  • Add NaN validation for parseInt on options.limit

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@bhadraagada
bhadraagada marked this pull request as ready for review January 3, 2026 08:10
Copilot AI review requested due to automatic review settings January 3, 2026 08:10

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@bhadraagada
bhadraagada merged commit 498c93e into feat/models-command Jan 3, 2026
1 check failed
Copilot AI requested a review from bhadraagada January 3, 2026 08:12
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