Skip to content

fix(aichat): update default modelId to claude-opus-5 - #82

Merged
imojenisys merged 1 commit into
hexide-io:mainfrom
snowyukitty:fix/update-default-chat-model
Aug 18, 2026
Merged

imojenisys merged 1 commit into
hexide-io:mainfrom
snowyukitty:fix/update-default-chat-model

Conversation

@snowyukitty

Copy link
Copy Markdown
Contributor

Summary

Updates the default AI Chat modelId in ChatSettings from the superseded claude-opus-4-8 to claude-opus-5 (#55).

Changes

  • IDE/HexIDE.AIChat/ChatSettings.cs: Updated ModelId property default to "claude-opus-5".
  • IDE/HexIDE.Tests/AIChat/ChatSessionTests.cs: Added unit test ChatSettings_DefaultsModelIdToClaudeOpus5 verifying the default value.
  • docs/TODO.md: Updated task checklist entry.

Fixes #55

Asserts ChatSettings defaults to the current Opus id, so a future bump is
a deliberate edit rather than a silent drift.

The accompanying one-line default change landed separately in #76, which
merged nine minutes after this was opened; this keeps the half that is
still additive.
@imojenisys
imojenisys force-pushed the fix/update-default-chat-model branch from f08c9d1 to 850d722 Compare August 18, 2026 00:43
@imojenisys

Copy link
Copy Markdown
Contributor

Thank you — and welcome. This is HexIDE's second outside contribution, and you were nine minutes away from it being the first: you opened this at 00:21, and #76 (the same one-line fix) merged at 00:31. You picked an open issue and did the work properly; the race was not something you could have seen.

Your version was also the more thorough one. #76 changed the default and stopped there — you added a test, written to the project's conventions (AwesomeAssertions rather than xUnit Assert). That is the half that still applies, so rather than close this I have reduced the PR to exactly that and left the commit authored by you.

What I removed, and why:

I also trimmed a stray blank line at end of file. Nothing else was touched, and I used the "allow edits from maintainers" flag rather than asking you to force-push, so there was nothing for you to redo.

Green on all four checks. Merging now — thanks again, and please do pick up another if you fancy it. Anything labelled good first issue is fair game, and #81 in particular is a nice self-contained one.

@imojenisys
imojenisys merged commit 4067771 into hexide-io:main Aug 18, 2026
4 checks passed
imojenisys pushed a commit that referenced this pull request Sep 24, 2026
Asserts ChatSettings defaults to the current Opus id, so a future bump is a deliberate edit rather than silent drift. The one-line default change itself landed in #76.
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.

AI Chat defaults to a superseded model id

2 participants