Skip to content

feat: set gpt-5 as default model and update docs #155

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
mattmorgis merged 3 commits into from
Aug 9, 2025
Merged

feat: set gpt-5 as default model and update docs #155

mattmorgis merged 3 commits into from
Aug 9, 2025

Conversation

@mattmorgis
Copy link
Owner

@mattmorgis mattmorgis commented Aug 9, 2025

Summary

  • Set default model to gpt-5 in CLI (ModelChoice and cli default)
  • Updated README to reflect gpt-5 as the default model
  • Left unrelated local changes uncommitted to keep this PR focused

Why

  • Aligns docs and defaults with current model availability and guidance

Testing

  • Ran pre-commit hooks for README (passed/skipped appropriately)
  • Ran test suite: 153 passed, 1 skipped

Usage

  • No breaking changes to CLI flags; users can still override with --model
  • OPENAI_API_KEY remains the only required env var (OPENAI_BASE_URL optional)

Notes

  • Base: main
  • Head: add-gpt-5

mattmorgis and others added 3 commits August 8, 2025 21:36
@mattmorgis
add gpt-5
32a593d
@mattmorgis
tests: update for gpt-5 default model and gpt-5-mini option
dfdebc5 
- Change CLI tests to expect ModelChoice.gpt_5 as default
- Expand ModelChoice enum test to include "gpt-5" and "gpt-5-mini"

Note: left prompt_default.jinja2 uncommitted as requested.
@mattmorgis @oai-coding-agent
docs: update README to reflect default model is gpt-5
aab49c1 
🤖 Generated with OAI Coding Agent
Co-Authored-By: oai-coding-agent[bot] <214839426+oai-coding-agent[bot]@users.noreply.github.com>
@mattmorgis mattmorgis merged commit a99fce9 into main Aug 9, 2025
1 check passed
@mattmorgis mattmorgis deleted the add-gpt-5 branch August 9, 2025 02:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mattmorgis