Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions apps/sim/app/api/copilot/chat/route.ts
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@ const ChatMessageSchema = z.object({
'claude-4-sonnet',
'claude-4.5-haiku',
'claude-4.5-sonnet',
'claude-4.5-opus',
'claude-4.1-opus',
Copy link
Contributor

Choose a reason for hiding this comment

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

logic: claude-4.1-opus still present in validation schema but commented out elsewhere - should be removed here too for consistency

Suggested change
'claude-4.1-opus',
'claude-4.5-opus',
Prompt To Fix With AI
This is a comment left during a code review.
Path: apps/sim/app/api/copilot/chat/route.ts
Line: 59:59

Comment:
**logic:** `claude-4.1-opus` still present in validation schema but commented out elsewhere - should be removed here too for consistency

```suggestion
      'claude-4.5-opus',
```

How can I resolve this? If you propose a fix, please make it concise.

])
.optional()
Expand Down
3 changes: 2 additions & 1 deletion apps/sim/app/api/copilot/user-models/route.ts
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,8 @@ const DEFAULT_ENABLED_MODELS: Record<string, boolean> = {
'claude-4-sonnet': false,
'claude-4.5-haiku': true,
'claude-4.5-sonnet': true,
'claude-4.1-opus': true,
'claude-4.5-opus': true,
// 'claude-4.1-opus': true,
}

// GET - Fetch user's enabled models
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ export const MODEL_OPTIONS = [
{ value: 'claude-4.5-sonnet', label: 'Claude 4.5 Sonnet' },
{ value: 'claude-4.5-haiku', label: 'Claude 4.5 Haiku' },
{ value: 'claude-4.5-opus', label: 'Claude 4.5 Opus' },
{ value: 'claude-4.1-opus', label: 'Claude 4.1 Opus' },
// { value: 'claude-4.1-opus', label: 'Claude 4.1 Opus' },
// { value: 'gpt-5-fast', label: 'GPT 5 Fast' },
// { value: 'gpt-5', label: 'GPT 5' },
// { value: 'gpt-5.1-fast', label: 'GPT 5.1 Fast' },
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@ import { cn } from '@/lib/utils'
import {
AttachedFilesDisplay,
ContextPills,
ContextUsageIndicator,
MentionMenu,
Copy link
Contributor

Choose a reason for hiding this comment

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

style: import for ContextUsageIndicator not removed but component is commented out - clean up unused import

Prompt To Fix With AI
This is a comment left during a code review.
Path: apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/copilot/components/user-input/user-input.tsx
Line: 22:22

Comment:
**style:** import for `ContextUsageIndicator` not removed but component is commented out - clean up unused import

How can I resolve this? If you propose a fix, please make it concise.

ModelSelector,
ModeSelector,
Expand Down Expand Up @@ -622,13 +621,13 @@ const UserInput = forwardRef<UserInputRef, UserInputProps>(
</Badge>

{/* Context Usage Indicator */}
{contextUsage && contextUsage.percentage > 0 && (
{/* {contextUsage && contextUsage.percentage > 0 && (
<ContextUsageIndicator
percentage={contextUsage.percentage}
size={18}
strokeWidth={2.5}
/>
)}
)} */}

{/* Selected Context Pills */}
<ContextPills
Expand Down
1 change: 1 addition & 0 deletions apps/sim/lib/copilot/api.ts
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,7 @@ export interface SendMessageRequest {
| 'claude-4-sonnet'
| 'claude-4.5-haiku'
| 'claude-4.5-sonnet'
| 'claude-4.5-opus'
| 'claude-4.1-opus'
Copy link
Contributor

Choose a reason for hiding this comment

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

logic: claude-4.1-opus still present in type definition but commented out elsewhere - remove for consistency

Suggested change
| 'claude-4.1-opus'
| 'claude-4.5-opus'
Prompt To Fix With AI
This is a comment left during a code review.
Path: apps/sim/lib/copilot/api.ts
Line: 88:88

Comment:
**logic:** `claude-4.1-opus` still present in type definition but commented out elsewhere - remove for consistency

```suggestion
    | 'claude-4.5-opus'
```

How can I resolve this? If you propose a fix, please make it concise.

prefetch?: boolean
createNewChat?: boolean
Expand Down
1 change: 1 addition & 0 deletions apps/sim/stores/panel/copilot/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,7 @@ export interface CopilotState {
| 'claude-4-sonnet'
| 'claude-4.5-haiku'
| 'claude-4.5-sonnet'
| 'claude-4.5-opus'
| 'claude-4.1-opus'
Copy link
Contributor

Choose a reason for hiding this comment

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

logic: claude-4.1-opus still present in type definition but commented out elsewhere - remove for consistency

Suggested change
| 'claude-4.1-opus'
| 'claude-4.5-opus'
Prompt To Fix With AI
This is a comment left during a code review.
Path: apps/sim/stores/panel/copilot/types.ts
Line: 82:82

Comment:
**logic:** `claude-4.1-opus` still present in type definition but commented out elsewhere - remove for consistency

```suggestion
    | 'claude-4.5-opus'
```

How can I resolve this? If you propose a fix, please make it concise.

agentPrefetch: boolean
enabledModels: string[] | null // Null means not loaded yet, array of model IDs when loaded
Expand Down