Skip to content

Conversation

@Sg312
Copy link
Collaborator

@Sg312 Sg312 commented Feb 12, 2026

Summary

Make copilot default model opus 4.5

Type of Change

  • Bug fix

Testing

Manual

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel
Copy link

vercel bot commented Feb 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Feb 12, 2026 8:50pm

Request Review

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 12, 2026

Greptile Overview

Greptile Summary

Changed the default Copilot model from claude-opus-4-6 to claude-opus-4-5 in the store's initial state and model selection fallback logic.

Critical Issue:

  • The migration is incomplete. Three API route files still default to claude-opus-4-6: apps/sim/app/api/v1/copilot/chat/route.ts, apps/sim/app/api/mcp/copilot/route.ts, and apps/sim/app/api/copilot/chat/route.ts
  • This creates inconsistent behavior where the UI defaults to opus-4-5 but the API routes may default to opus-4-6
  • All default model references should be updated together to ensure consistent behavior across the application

Confidence Score: 2/5

  • This PR creates inconsistent default model behavior between UI and API routes
  • The change is incomplete - while the store now defaults to opus-4-5, three API route files still use opus-4-6 as their default, which could lead to unexpected behavior and confusion
  • The three API route files that still reference claude-opus-4-6 as default need to be updated: apps/sim/app/api/v1/copilot/chat/route.ts, apps/sim/app/api/mcp/copilot/route.ts, and apps/sim/app/api/copilot/chat/route.ts

Important Files Changed

Filename Overview
apps/sim/stores/panel/copilot/store.ts Changed default model from opus-4-6 to opus-4-5, but incomplete - other API routes still use opus-4-6 as default

Last reviewed commit: 09c9477

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@Sg312 Sg312 merged commit c380e59 into staging Feb 12, 2026
11 checks passed
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