Skip to content

chore: remove OpenAI provider and [openai] extra#6

Merged
silversurfer562 merged 1 commit into
mainfrom
feat/remove-openai
May 7, 2026
Merged

chore: remove OpenAI provider and [openai] extra#6
silversurfer562 merged 1 commit into
mainfrom
feat/remove-openai

Conversation

@silversurfer562
Copy link
Copy Markdown
Member

Summary

Drops the OpenAI provider per a vendor-preference decision in the broader API cost audit. No external consumer of attune-rag imports OpenAIProvider — confirmed by grepping attune-ai, attune-author, attune-help, and the home Next.js app. So the provider is fully removable without migrating any callsite.

What's gone

  • src/attune_rag/providers/openai.py and its unit tests
  • The openai branch in providers.get_provider()
  • "openai" in _SDK_PROBES (no longer reported by list_available())
  • The [openai] install extra (pip install 'attune-rag[openai]' no longer works)
  • openai>=1.40.0,<2.0 from the [all] and [dev] deps lists
  • README quickstart for OpenAI
  • OpenAI mentions in CLI choices, pipeline docstring, base provider docstring

What stayed

  • Historical CHANGELOG entries — they describe past releases and shouldn't be rewritten.
  • Generated .help/templates/ — those'll refresh on the next regen.

CHANGELOG entry

Under [Unreleased]:

Removed the OpenAI provider (attune_rag.providers.openai.OpenAIProvider) and the [openai] install extra. Use [claude] or [gemini] instead.

Test plan

  • pytest --no-cov -q298 passed, 3 xpassed
  • Pre-removal grep across all 4 attune packages + home Next.js app: no external OpenAIProvider usage
  • Pre-commit hooks pass on commit
  • Reviewer: confirm no Vercel/Cloudflare/edge config refers to the [openai] extra

🤖 Generated with Claude Code

Per user vendor-preference decision: drop OpenAI as a supported
provider in attune-rag. Pre-removal grep confirmed no external
consumer (attune-ai, attune-author, attune-help, home Next.js app)
imports OpenAIProvider — the provider is fully removable without
callsite migrations.

- Delete src/attune_rag/providers/openai.py and its unit tests
- Drop the openai branch in providers.get_provider()
- Drop "openai" from _SDK_PROBES (no longer reported by list_available)
- Drop the [openai] install extra from pyproject.toml; remove
  openai>=1.40.0 from [all] and [dev]
- Update CLI: --provider choices, install hint
- Update README: drop OpenAI quickstart, install line, description
- Update docstrings (pipeline.run_and_generate, base.LLMProvider)
- CHANGELOG entry under [Unreleased]

Test plan: 298 passed, 3 xpassed (full suite); 14/14 providers tests
pass with the dispatch test updated to reflect the new {claude, gemini}
contract.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@silversurfer562 silversurfer562 merged commit e2586ae into main May 7, 2026
12 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