Skip to content

Fix agent configuration merge order#8

Open
logvik wants to merge 1 commit intovtemian:mainfrom
logvik:patch-1
Open

Fix agent configuration merge order#8
logvik wants to merge 1 commit intovtemian:mainfrom
logvik:patch-1

Conversation

@logvik
Copy link

@logvik logvik commented Feb 18, 2026

At this case we will be able to override configuration from opencode.json config file. At the case, if we want to choose another model, for example, not codex.


Summary by cubic

Fix agent configuration merge order so values in opencode.json take precedence over defaults. This lets users override agent options, like choosing a non-codex model.

Written for commit 90106c1. Summary will update on new commits.

At this case we will be able to override configuration from opencode.json config file. At the case, if we want to choose another model, for example, not codex.
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

@vtemian
Copy link
Owner

vtemian commented Mar 8, 2026

Hey @logvik, thanks for the PR!

Before we merge this, could you clarify the specific use case you're trying to solve?

The current merge order is intentional — octto.json is designed to be the more specific configuration layer that takes precedence over opencode.json. Flipping the precedence would mean octto.json agent settings become defaults that opencode.json can silently override, which defeats the purpose of having octto-specific configuration.

Could you describe the scenario where you need opencode.json to override octto.json? There might be a better way to support your use case without changing the precedence for everyone.

@logvik
Copy link
Author

logvik commented Mar 8, 2026

Hi. I have described my case above. I wanted to try another model, because "codex" is the hard coded model now.

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