Skip to content

feat(provider): Add support for Ollama Cloud#31

Merged
edenreich merged 6 commits intomainfrom
claude/issue-30-20251120-0904
Nov 20, 2025
Merged

feat(provider): Add support for Ollama Cloud#31
edenreich merged 6 commits intomainfrom
claude/issue-30-20251120-0904

Conversation

@edenreich
Copy link
Copy Markdown
Contributor

Summary

This PR adds support for Ollama Cloud as a provider in the SDK, following the schema-driven approach.

Changes

  • Downloaded latest OpenAPI spec with Ollama Cloud provider
  • Generated types with OllamaCloud constant
  • Updated README to list Ollama Cloud as supported provider
  • Added comprehensive tests for Ollama Cloud functionality
  • Deprecated A2A methods (removed from OpenAPI spec)

Breaking Changes

⚠️ A2A functionality has been removed from the OpenAPI specification. The methods ListAgents() and GetAgent() are now deprecated and will be removed in a future version.

Testing

  • All Ollama Cloud tests passing
  • Linter passing
  • Existing tests maintained for backward compatibility

Closes #30

🤖 Generated with Claude Code

- Download latest OpenAPI spec with Ollama Cloud provider
- Generate types with OllamaCloud constant
- Update README to list Ollama Cloud as supported provider
- Add comprehensive tests for Ollama Cloud functionality
- Deprecate A2A methods (removed from OpenAPI spec)
- Preserve A2A types for backward compatibility

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-authored-by: Eden Reich <edenreich@users.noreply.github.com>
@edenreich
Copy link
Copy Markdown
Contributor Author

@claude Can you cleanup the deprecated code, I forgot to do it, and it's fine since this feature was not stable anyways - people were not using it.

@claude
Copy link
Copy Markdown
Contributor

claude bot commented Nov 20, 2025

Claude finished @edenreich's task —— View job


Cleaning up deprecated A2A code ✅

Successfully removed all deprecated A2A functionality from the SDK.

Changes Made

✅ Removed ListAgents() and GetAgent() from Client interface
✅ Removed deprecated method implementations (73 lines from sdk.go)
✅ Removed all A2A tests (428 lines from sdk_test.go)
✅ All tests passing (33 test suites, 0 failures)
✅ Linter clean (0 issues)
✅ Changes committed to PR branch

Summary

The deprecated A2A methods have been completely removed from the codebase. This cleanup was safe to do since the A2A functionality was already removed from the OpenAPI specification and users weren't actively using it. All existing tests continue to pass, and the codebase is now cleaner without the deprecated code.

Note: The MiddlewareOptions.SkipA2A field was intentionally kept as it controls middleware headers (not the deprecated A2A endpoints).

--- |

claude bot and others added 5 commits November 20, 2025 09:26
Removed ListAgents() and GetAgent() methods and all related tests.
The A2A functionality was deprecated when it was removed from the
OpenAPI specification. This change completes the cleanup by removing
the deprecated code entirely.

Changes:
- Removed ListAgents() and GetAgent() from Client interface
- Removed method implementations (73 lines from sdk.go)
- Removed all A2A tests (428 lines from sdk_test.go)
- All tests passing, linter clean

Co-authored-by: Eden Reich <edenreich@users.noreply.github.com>
Signed-off-by: Eden Reich <eden.reich@gmail.com>
Signed-off-by: Eden Reich <eden.reich@gmail.com>
Signed-off-by: Eden Reich <eden.reich@gmail.com>
Signed-off-by: Eden Reich <eden.reich@gmail.com>
@edenreich edenreich merged commit 31aaaf8 into main Nov 20, 2025
1 check passed
@edenreich edenreich deleted the claude/issue-30-20251120-0904 branch November 20, 2025 09:38
github-actions bot pushed a commit that referenced this pull request Nov 20, 2025
## [1.14.0](v1.13.0...v1.14.0) (2025-11-20)

### ⚠ BREAKING CHANGES

* **vision:** Message.Content is now Message_Content type instead of string.
Use helper functions or .FromMessageContent0() / .AsMessageContent0() methods.

Supports:
- Image URLs (https://)
- Base64-encoded images (data:image/...)
- Multiple images per message
- Image detail levels (auto, low, high)

### ✨ Features

* **provider:** Add support for Ollama Cloud ([#31](#31)) ([31aaaf8](31aaaf8))
* **vision:** Add image URL support for vision models ([#29](#29)) ([b4cc118](b4cc118)), closes [#26](#26)

### 👷 CI

* Update Claude Code CI ([#27](#27)) ([babade6](babade6))

### 🔧 Miscellaneous

* Delete .github/workflows/claude-code-review.yml ([#28](#28)) ([9b75d24](9b75d24))
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.14.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] Download the latest OpenAPI spec to add support for Ollama Cloud

1 participant