Skip to content

Conversation

@vsumner
Copy link
Contributor

@vsumner vsumner commented Jan 12, 2026

Summary

Fix issue where Orchestrator's Sisyphus-Junior agent was hardcoded to use claude-sonnet-4.5, preventing user model overrides from being respected.

Problem

  • createSisyphusJuniorAgent() in src/agents/sisyphus-junior.ts hardcodes model: "anthropic/claude-sonnet-4.5"
  • This prevented users from configuring a different model via oh-my-opencode.json
  • Sonnet 4.5 doesn't support extended thinking (32k token budget), which limits Orchestrator's strategic analysis capabilities

Solution

  • Changed createSisyphusJuniorAgent() to use categoryConfig.model instead of hardcoded value
  • Now respects agents.Sisyphus-Junior.model override from user config
  • Users can configure models with extended thinking support (e.g., anthropic/claude-opus-4-5, openai/gpt-5.2)

Files Changed

  • src/agents/sisyphus-junior.ts: 1 insertion, 1 deletion

Summary by cubic

Use categoryConfig.model for Sisyphus-Junior instead of the hardcoded claude-sonnet-4.5, so user-configured model overrides are respected. This unlocks models with extended thinking support and removes the previous analysis limits.

Written for commit 0101268. Summary will update on new commits.

Copy link

@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.

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

@code-yeongyu code-yeongyu merged commit 70d604e into code-yeongyu:dev Jan 13, 2026
3 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.

2 participants