Bump Rust SDK and daemon dependencies - #366
Merged
Stephen Belanger (Qard) merged 1 commit intoSep 10, 2026
Merged
Conversation
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. |
Abhijeet Prasad (AbhiPrasad)
approved these changes
Sep 10, 2026
Contributor
|
Latest downloadable build artifacts for this PR commit
Available artifact names
|
Stephen Belanger (Qard)
deleted the
stephenbelanger/sdk-340-dependency-bumps
branch
September 10, 2026 17:40
Abhijeet Prasad (AbhiPrasad)
pushed a commit
that referenced
this pull request
Sep 11, 2026
## Summary Adds the commit messages between the previous stable release and the one being released into the auto-generated GitHub release notes. Requested by @AbhiPrasad — include commit messages (current tag → last tag) in release notes. ## What changed In the `announce` job of `.github/workflows/release.yml`: - Checkout now fetches full history + tags (`fetch-depth: 0`, `fetch-tags: true`). - Before `gh release create`, a **Commits** section is appended to cargo-dist's `announcement_github_body`: - Resolves the previous stable tag via `git tag --list 'v[0-9]*.[0-9]*.[0-9]*' --sort=-version:refname` (canary-* and other non-release tags are ignored). - Lists non-merge commits in `previous_tag..<release commit>` as `- <subject> (<short sha>)`. - Adds a `**Full Changelog**` compare link. - Falls back to full history (no compare link) for the first release. ## Example output The section appended to the release notes for `v0.19.3` (below cargo-dist's existing body): ```markdown ## Commits - chore: release v0.19.3 (#367) (71134cf) - Bump Rust SDK and daemon dependencies (#366) (b5be3e8) - feat: Go eval support (#305) (b724347) - test: run full unit suite in CI (#364) (6fee421) - bump bt daemon dependency (#365) (071e1d1) - chore: Fix security alerts (#359) (6030fe1) - Use public app URLs for generated links (#363) (fa4e76c) - SDK-223 Honor JSON output for project commands (#361) (123d914) - SDK-325 Fetch Markdown documentation pages (#360) (64460f8) **Full Changelog**: v0.19.2...v0.19.3 ``` ## Validation Ran the exact shell logic against the real repo for `v0.19.3`; it resolved `v0.19.2` as the previous tag and rendered a correct Commits section + compare link. Workflow YAML validated. 🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- sfk:created-approved-by --> Created by @AbhiPrasad <!-- sfk:slack-thread --> [Slack thread](https://braintrustdata.slack.com/archives/C0AQDETAVT3/p1789074523886719?thread_ts=1789074523.886719&cid=C0AQDETAVT3) Co-authored-by: Starfolk <noreply@starfolk.ai> Co-authored-by: Claude Opus 4.8 <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
2 participants
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.
Summary
braintrust-sdk-rustto 019e428ab80d0da4c00dab383d4b0d1fdf4486fdbt-daemonto c7e9984225b9726a179d51cb4c0d24cfec4617c9 and initialize its newSessionRoute.tagsfieldValidation
cargo fmt --checkcargo check --lockedcargo test --locked trace_host::testscargo metadata --locked --no-deps --format-version 1git diff --checkThe full
cargo test --lockedsuite ran after building: 12 CLI integration tests failed because their synthetic credentials attempted live data-plane resolution and received 401 responses.Depends on braintrust-sdk-rust#101 and braintrust-coding-agent-plugins#66.