Skip to content

fix(config): update embedded Anthropic model - #263

Merged
owainlewis merged 1 commit into
mainfrom
codex/issue-229-default-model
Aug 4, 2026
Merged

fix(config): update embedded Anthropic model#263
owainlewis merged 1 commit into
mainfrom
codex/issue-229-default-model

Conversation

@owainlewis

Copy link
Copy Markdown
Owner

Summary

  • update the embedded Anthropic model from claude-opus-4-8 to claude-opus-5
  • keep the developer configuration example aligned with the embedded default
  • make the embedded fallback regression test assert the declared default model exactly

Why

A clean installation used the stale model pinned in the embedded YAML instead of Neo's current Anthropic default. Explicit models in project or user configuration remain unchanged.

Test plan

  • go test ./internal/config ./cmd/neo
  • go test ./...
  • git diff --check
  • go build -o /tmp/neo-issue-229.../neo ./cmd/neo
  • run the built binary from an empty temporary working directory with an empty HOME; neo doctor reports loaded embedded and pass model claude-opus-5

Risks

Low. This changes only the embedded default used when no project or user configuration exists. Explicit configured models are preserved.

Related issue

Closes #229

@owainlewis

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep them coming!

Reviewed commit: f52d6dcb06

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@owainlewis
owainlewis merged commit ec94efb into main Aug 4, 2026
2 checks passed
@owainlewis
owainlewis deleted the codex/issue-229-default-model branch August 27, 2026 21:45
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.

Embedded config keeps the old Anthropic default model

1 participant