Skip to content

fix(agents): recognize GPT-6 model variants - #602

Closed
Filip-Stastny-Philips wants to merge 2 commits into
codemie-ai:mainfrom
Filip-Stastny-Philips:fix/gpt6-model-recognition
Closed

Filip-Stastny-Philips wants to merge 2 commits into
codemie-ai:mainfrom
Filip-Stastny-Philips:fix/gpt6-model-recognition

Conversation

@Filip-Stastny-Philips

Copy link
Copy Markdown
Contributor

Summary

Add GPT-6 model recognition for CodeMie OpenCode and Pi integrations.

Changes

  • Route GPT-6.1, GPT-6 Sol, and other GPT-6 variants through the OpenAI Responses API.
  • Assign the 1,050,000 context and 128,000 output token limits.
  • Keep vendor-prefixed identifiers such as openai.gpt-6.1 supported by the existing regex approach.

Testing

  • npm run lint
  • npm run typecheck
  • Related unit tests passed through the pre-commit hook.
  • Local secrets scan was skipped because no container engine is installed; CI should run it.

@Siarhei-Yarkavy

Copy link
Copy Markdown

looks cover the same as https://github.com/codemie-ai/codemie-code/pull/582/changes

Filip Stastny added 2 commits October 4, 2026 17:07
Route GPT-6.1 and vendor-prefixed deployments through Responses API with large context limits.
@Filip-Stastny-Philips

Filip-Stastny-Philips commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor Author

looks cover the same as https://github.com/codemie-ai/codemie-code/pull/582/changes

@Siarhei-Yarkavy 80% same, but Pi is different: PR #582 uses incorrect name filter, asserting "name starts with gpt-6...", but we need the same logic as in opencode filter "model contains gpt-6..."

If #582 is merged, I can update this PR to only change that issue. Or if you fix the filter in #582 (I have no permissions), we can close this one. But pls let's not delay #582 or this PR too much, GPT-6 models are currently unusable in codemie-code

@Siarhei-Yarkavy

Siarhei-Yarkavy commented Oct 4, 2026 •

Copy link
Copy Markdown

@Filip-Stastny-Philips I updated the PR, it should be fine now. bb3fe16

@Filip-Stastny-Philips

Copy link
Copy Markdown
Contributor Author

All fixes ported to #582 --> closing this one

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