Skip to content

feat: remove sdk provider independence#65

Merged
sgriffiths merged 7 commits intomainfrom
feat/remove-sdk-dependence
Apr 10, 2026
Merged

feat: remove sdk provider independence#65
sgriffiths merged 7 commits intomainfrom
feat/remove-sdk-dependence

Conversation

@aswhitehouse
Copy link
Copy Markdown
Collaborator

@aswhitehouse aswhitehouse commented Mar 28, 2026

What changed

  • Remove the reliance on provider SDKS
  • Create a central wrapper that handles provider interactions

Why

  • The SDKs are large to ship with the library & we want to protect OA from breaking changes on an SDK upgrade etc.

How tested

  • All baseline test pass for agent creation and runtime.

  • All existing tests pass (pytest tests/)

  • New tests added (if applicable)

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Refactoring / CI / tooling

Breaking changes

Checklist

  • Code follows project style (ruff check . && ruff format --check .)
  • Self-review completed
  • Version bumped if needed (pyproject.toml, templates, docs)

@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 28, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
open-agent-spec Ready Ready Preview, Comment Apr 10, 2026 1:16am

Request Review

Comment thread oas_cli/providers/registry.py Outdated
Copy link
Copy Markdown
Contributor

@sgriffiths sgriffiths left a comment

Choose a reason for hiding this comment

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

couple of comments, otherwise looks good

Comment thread oas_cli/providers/anthropic_http.py Outdated
Comment thread oas_cli/providers/anthropic_http.py
Copy link
Copy Markdown
Contributor

@sgriffiths sgriffiths left a comment

Choose a reason for hiding this comment

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

Looks good, all comments addressed. We can sort the default model strings later.

@sgriffiths
Copy link
Copy Markdown
Contributor

LGTM 👍

@sgriffiths sgriffiths merged commit cdaa2c2 into main Apr 10, 2026
2 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.

2 participants