Repository navigation
Patch-bump the ten ports released over CI or by tag - #168
Merged
Merged
Conversation
typescript 0.3.7, javascript 0.1.7, rust 0.1.7, go 0.1.6, php 0.1.3, c 0.1.3, cpp 0.1.4, swift 0.1.3, zig 0.0.3 and lean 0.1.1, written by tools/bump.py. Each carries fixes merged since its last tag. The thirteen registry ports upload under their own credentials and are not bumped here. Co-Authored-By: Claude Opus 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_01BoEed9CaF6sTNMcXxQaH6w
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. |
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
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.
The version bump for a release, landed through review first, as the release section of AGENTS.md asks. Each port below carries fixes merged since its last tag. The releases follow once this merges, and the next sdk checkpoint is cut from that main for sdkgen to vendor.
Versions
Written by
tools/bump.py --ports typescript,javascript,rust,go,php,c,cpp,swift,zig,lean;--checkreports every copy agreeing. 16 files change, each by its version string alone.mergefix, the validate$ONEfix$ONEsetpropandSetPathfixes,$ONEescurlfix (#155),$ONE$ONEescurl,$ONEescurl,$ONE$ONEtypescript and go also carry #151, which strips source comments and is not a fix.
How each is released after the merge
publish.ymldispatch each onmain, over OIDC, which tests, publishes and writes<lang>/v<version>.make publish-<lang>from a cleanmainat the merge, aftergit fetch --tags, which runs the port's tests and pushes<lang>/v<version>; the tag is the release. Those targets do not themselves check the branch or the tree, so the run has to start from that state.Not bumped
The thirteen registry ports (python, ruby, csharp, perl, java, kotlin, scala, lua, clojure, dart, elixir, haskell, ocaml) upload under their own registry's credentials, which this environment does not hold. All thirteen carry #160, java and kotlin #164 too, and eleven of them already sit one bump ahead of their tag, so they are left for a release with those credentials.
Verification
make testin typescript (112 pass), javascript (95 pass), rust and go on this branch, all passing, with the tree unchanged afterwards.Review
A second Claude reviewer reproduced the bump byte for byte from main with the same command, confirmed every new
<lang>/v<version>tag is free on origin and every release path reads the new version (publish.yml's own expressions for the three OIDC ports, each Makefile'sTAGfor the seven), and ran all ten ports'make test: typescript 112, javascript 95, go 106, php 98, c, cpp, swift (6.3.3), zig (72, on 0.16.0) and lean (77 groups) all pass;npm pack --dry-runandcargo publish --dry-runpackage 0.3.7, 0.1.7 and 0.1.7. Its one correction is the Not bumped paragraph above, which said only ruby and dart carried #160. Codex reviewed 2f609df and raised nothing.🤖 Generated with Claude Code
https://claude.ai/code/session_01BoEed9CaF6sTNMcXxQaH6w