Skip to content

chore: 🐝 Update SDK - Generate SDK 0.5.1 #299

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

Open
mattapperson wants to merge 1 commit into
base: main
Choose a base branch
from
Open

chore: 🐝 Update SDK - Generate SDK 0.5.1 #299

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .devcontainer/devcontainer.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
// README at: https://github.com/devcontainers/templates/tree/main/src/go
{
"name": "Go",
"image": "mcr.microsoft.com/devcontainers/go:1-1.25-bookworm",
"image": "mcr.microsoft.com/devcontainers/go:1-1.22-bullseye",
// Features to add to the dev container. More info: https://containers.dev/features.
// "features": {},
// Use 'forwardPorts' to make a list of ports inside the container available locally.
Expand Down
799 changes: 480 additions & 319 deletions .speakeasy/gen.lock
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion .speakeasy/gen.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ generation:
skipResponseBodyAssertions: false
preApplyUnionDiscriminators: true
go:
version: 0.5.0
version: 0.5.1

@perry-the-pr-reviewer perry-the-pr-reviewer Bot Jun 20, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[blocker] SDK version: 0.5.1 ([patch] bump) ships two breaking public-API removals — SemVer violation

Details

Why: the PR body lists OpenRouter.Datasets.GetBenchmarksArtificialAnalysis() and OpenRouter.Datasets.GetBenchmarksDesignArena() as Removed (Breaking ⚠️) — verified: both func (s *Datasets) definitions are deleted from datasets.go and replaced by a new Benchmarks.GetBenchmarks() resource. Removing exported methods breaks any consumer calling them at compile time. Under SemVer a breaking change requires at least a minor pre-1.0 bump (or major post-1.0), not a patch.

Fix: bump info.version in the OpenAPI doc to trigger at least a minor bump (0.5.0 → 0.6.0), or use SPEAKEASY_BUMP_OVERRIDE=minor, so the release version reflects the breaking surface. Document the removals + the Benchmarks.GetBenchmarks() migration in the release notes.

Ref: Speakeasy — versioning & OpenAPI document changes

Prompt for agents 
This regen removes two exported methods (Datasets.GetBenchmarksArtificialAnalysis, Datasets.GetBenchmarksDesignArena — confirmed deleted from datasets.go) but bumps only patch (0.5.0 -> 0.5.1). Breaking removals require at least a minor bump pre-1.0. Bump info.version in the OpenAPI source (or set SPEAKEASY_BUMP_OVERRIDE=minor) so the SDK releases as 0.6.0, and note the removals + the replacement Benchmarks.GetBenchmarks() in RELEASES.md / migration notes.

Reviewed at 44ba584

additionalDependencies: {}
baseErrorName: OpenRouterError
clientServerStatusCodesAsErrors: true
Expand Down
Loading