Skip to content

Remove unavailable OpenAI models#1554

Merged
simonw merged 6 commits into
mainfrom
codex/remove-deprecated-openai-models
Jul 25, 2026
Merged

Remove unavailable OpenAI models#1554
simonw merged 6 commits into
mainfrom
codex/remove-deprecated-openai-models

Conversation

@simonw

@simonw simonw commented Jul 25, 2026

Copy link
Copy Markdown
Owner
  • remove 14 OpenAI model IDs that now fail against the OpenAI API
  • remove the aliases that pointed at those model IDs
  • rebuild the Cog-managed model, alias, and usage documentation

Refs:

OpenAI Codex PR body

Why

These model IDs remain exposed by llm models, but prompting them now returns either OpenAI's model_not_found response or an explicit deprecation response. Keeping them registered advertises models that users cannot successfully call.

Impact

The removed model IDs and their built-in aliases are no longer returned by llm models or resolvable through llm.get_model() / llm.get_async_model(). Historical changelog references are preserved.

Validation

  • 61 focused alias/model-registration tests pass
  • 683 non-embedding tests pass
  • all 14 removed IDs verified absent from llm models
  • Black passes for changed Python files
  • mypy passes for llm
  • Ruff's core error checks pass for changed Python files
  • Cog rebuilt and checks the top-level generated documentation successfully

The complete local suite also exposed an unrelated SQLite 3.50 / sqlite-utils transaction incompatibility in embedding migrations; those failures reproduce outside the changed model-registration paths.

Comment thread llm/default_plugins/openai_models.py Outdated
"gpt-5.1",
"gpt-5.1-chat-latest",
):
for model_id in ("gpt-5.1",):

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahem.

@simonw

simonw commented Jul 25, 2026

Copy link
Copy Markdown
Owner Author

Cog failure:

Cogging docs/embeddings/python-api.md
  File "docs/embeddings/python-api.md", line 184, in <module>
    embeddings_migrations.apply(db)
  File "/opt/hostedtoolcache/Python/3.11.15/x64/lib/python3.11/site-packages/sqlite_utils/migrations.py", line 132, in apply
    with db.atomic():
  File "/opt/hostedtoolcache/Python/3.11.15/x64/lib/python3.11/contextlib.py", line 144, in __exit__
    next(self.gen)
  File "/opt/hostedtoolcache/Python/3.11.15/x64/lib/python3.11/site-packages/sqlite_utils/db.py", line 631, in atomic
    self.conn.execute("COMMIT")
OperationalError: cannot commit - no transaction is active

Relates to latest sqlite-utils changes.

@simonw

simonw commented Jul 25, 2026

Copy link
Copy Markdown
Owner Author

I'm going to upgrade to sqlite-utils 4.0 in a separate piece of work to fix that cog problem.

@simonw
simonw marked this pull request as ready for review July 25, 2026 19:51
@simonw

simonw commented Jul 25, 2026

Copy link
Copy Markdown
Owner Author

More sqlite-migrate transaction test failures. I'm going to pin to sqlite-migrate==0.1a2 to get this PR to pass, then I'll fix in #1555.

@simonw
simonw merged commit 1dc49af into main Jul 25, 2026
34 checks passed
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.

1 participant