Bump hey-sdk to v0.30.0 - #428
Conversation
Draft sends and edits that HEY rejects with a 422 now surface the validation message instead of a bare "validation error", and a draft edit that fails no longer reads as applied (hey-sdk#133). The pin also brings the workflow stage reader, generated retry policies and the Retry-After rounding fix, none of which the CLI calls yet. The vendored MCP model moves to go/v0.30.0 with it, gaining the GetWorkflowStage operation under the already-unmapped Workflows tag, and the Nix vendorHash follows go.sum.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
🟢 Approval recommended
The changes are a straightforward dependency/model snapshot bump with no local behavioral logic modifications in hey-cli beyond the expected version pin updates.
Pull request overview
This PR updates hey-cli’s HEY SDK dependency to github.com/basecamp/hey-sdk/go v0.30.0 and resyncs the embedded MCP OpenAPI/behavior model snapshot to match the new SDK release, plus updates the Nix vendor hash for reproducible builds.
[!TIP]
If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or rungh pr ready --undo.
Click "Ready for review" or rungh pr readyto reengage.
Changes:
- Bump
github.com/basecamp/hey-sdk/gofromv0.29.0→v0.30.0(and refreshgo.sum). - Resync
internal/mcpserver/model/*(OpenAPI + behavior model + provenance) togo/v0.30.0. - Update Nix
vendorHashto match the new module graph.
File summaries
| File | Description |
|---|---|
| nix/package.nix | Updates vendorHash for Nix builds after the dependency bump. |
| internal/mcpserver/model/PROVENANCE.json | Updates provenance to the hey-sdk go/v0.30.0 commit. |
| internal/mcpserver/model/openapi.json | Resynced OpenAPI snapshot; includes new/updated operation and schema metadata. |
| internal/mcpserver/model/behavior-model.json | Resynced behavior model (retry/pagination metadata, new operation entry). |
| go.mod | Pins hey-sdk to v0.30.0 and updates indirect oapi-codegen/runtime to v1.7.0. |
| go.sum | Records new module checksums corresponding to the updated dependencies. |
Review details
- Files reviewed: 5/6 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Summary
Pins hey-sdk v0.30.0 (
go/v0.30.0, 9b16fc7).What it brings the CLI:
Alongside the pin:
internal/mcpserver/model/resynced fromgo/v0.30.0viamake update-mcp-model: PROVENANCE moves to 9b16fc7, and the model gainsGetWorkflowStageunder the Workflows tag, which the catalog already pins as unmapped, soTestCatalogUnmappedTagsArePinnedis unchanged.github.com/oapi-codegen/runtimev1.6.0 → v1.7.0 (indirect, pulled by the SDK).nix/package.nixvendorHash recomputed withmake update-nix-hashand verified by a full Nix build.Test plan
make check(fmt, vet, lint, unit tests, tidy, surface, release lockstep)make update-nix-hashbuilt the flake against the new go.sumgo.modcarries a singlegithub.com/basecamp/hey-sdk/go v0.30.0requirement and noreplaceSummary by cubic
Pins
hey-sdkfrom v0.29.0 to v0.30.0, so draft sends and edits HEY rejects with a 422 now surface the validation message instead of a bare "validation error", and a failed draft edit no longer reads as applied.Changes
go/v0.30.0, addingGetWorkflowStageunder the Workflows tag, which the catalog already pins as unmapped.github.com/oapi-codegen/runtimebumps to v1.7.0 as an indirect dependency.nix/package.nixvendorHash is recomputed viamake update-nix-hashand verified with a full Nix build.Written for commit 5e34a4f. Summary will update on new commits.