chore: release v0.5.1 - #46
Merged
Merged
Conversation
Every version location from the release table, and the three lockfiles. A patch: the single commit since v0.5.0 is a bug fix. It exists so a consumer can pin a tag rather than a main commit -- imogen-server needs the wire-boolean fix in a release to ship v0.6.1, and CLAUDE.md requires a consumer tag to name an SDK tag. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Version bump only — exempt from the review loop.
The cumulative
highreview is skipped under the skill's own exemption: the range is a single PR already reviewed at merge. #45 had four review iterations plus an independent confirming review during theimogen-serverpin move, which re-derived its claims rather than accepting them.Why this release exists
imogen-serverneeds the wire-boolean fix in a tag, not amaincommit. CLAUDE.md requires a consumer's release to name an SDK release, so shipping v0.6.1 means cutting this first and moving the server's pin onto it.What it carries
One commit: #36 / #45 — a boolean on the wire is read by its spelling, not by truthiness.
z.coerce.boolean()applies JavaScript truthiness, where only the empty string is falsy, so the string"false"parsed astrue. Every port sends booleans as stringified values, so the field could not expressfalseat all.WireBooleannow readstrue/falseand1/0by spelling and rejects anything else, pinned by abooleanOnTheWirefixture inconformance/.The other four ports turned out already to spell it correctly — proven rather than assumed, by drifting each port's encoder and watching the new tests fail. Python was the near miss: bare
str(True)gives"True", which truthiness swallowed and the new rule rejects.Verification
fmt --checkcleanimogen_sdk.__version__reports0.5.1bun.lockregenerated and now records0.5.1for both workspace packages.🤖 Generated with Claude Code