Skip to content

deps: raise the buffa floor to 0.9.2 and regenerate checked-in code - #294

Merged
iainmcgin merged 1 commit into
mainfrom
iain/regen-buffa-0.9.2
Sep 11, 2026
Merged

iainmcgin merged 1 commit into
mainfrom
iain/regen-buffa-0.9.2

Conversation

@iainmcgin

Copy link
Copy Markdown
Collaborator

buffa 0.9.2 shipped on 2026-09-03 and changes codegen output (generated decode_view attaches the element-memory budget, view decoders merge split map values, generated enums carry #[allow(non_camel_case_types)]), and because the workspace has no committed lockfile the Check generated code job resolved 0.9.2 immediately and has failed on every PR since — #293 is the first to show it. This regenerates the five checked-in directories with the 0.9.2 plugins and raises the floor to match the regen baseline; code generated by 0.9.1 still compiles against 0.9.2, so the floor is for the hardening fixes and the baseline rather than a compile requirement. A second task generate:all after this commit is a no-op.

buffa 0.9.2 (2026-09-03) changes codegen output: generated `decode_view`
attaches the element-memory budget, view decoders merge split map
values, enums carry `#[allow(non_camel_case_types)]`, and `SizeCache`
panics are gated on debug assertions. The workspace has no committed
lockfile, so CI's generated-code check resolved 0.9.2 as soon as it was
published and has failed on every PR since, because the checked-in
files were 0.9.1 output. This regenerates all six checked-in
directories with the 0.9.2 plugins and moves the floor to match the
regen baseline, as the workspace comment asks.

Signed-off-by: Iain McGinniss <309153+iainmcgin@users.noreply.github.com>
@iainmcgin
iainmcgin force-pushed the iain/regen-buffa-0.9.2 branch from 8ff700b to 6e7d0df Compare September 5, 2026 23:12
@iainmcgin
iainmcgin marked this pull request as ready for review September 6, 2026 15:09
@iainmcgin
iainmcgin requested a review from azdagron September 7, 2026 15:58
@iainmcgin
iainmcgin added this pull request to the merge queue Sep 11, 2026
Merged via the queue into main with commit 8f362b4 Sep 11, 2026
26 of 27 checks passed
@iainmcgin
iainmcgin deleted the iain/regen-buffa-0.9.2 branch September 11, 2026 17:51
iainmcgin added a commit to allada/connect-rust that referenced this pull request Sep 24, 2026
Stacked on connectrpc#294 (the first commit here is that PR; this one rebases onto
main once it lands).

With no committed lockfile, a dependency release changed what CI built
overnight: the generated-code check resolved the newest buffa at run
time and has failed on every PR since buffa 0.9.2 changed codegen output
on 2026-09-03, and the MSRV job is exposed the same way to any
transitive MSRV bump. This commits the workspace `Cargo.lock` (resolved
against crates.io today, buffa 0.9.2) and builds CI with `--locked` (the
example scripts and `task generate:all` add it when `CI` is set), so a
dependency bump becomes an explicit PR; the generated-code job builds
the buffa plugins from the tag matching the locked version, so a buffa
bump and its regenerated code have to travel together. Two companions
keep the lock from going stale: a weekly Dependabot group PR for
semver-compatible updates, and a scheduled *Latest dependencies*
workflow that runs the tests after a fresh `cargo update`, which is what
a crate depending on `connectrpc` resolves — the lockfile itself has no
effect on those crates. `--locked` also passes on the 1.88 MSRV
toolchain (lockfile format v4).

Fixes connectrpc#254 (the "related" `buf.gen.yaml` sibling-checkout note there is
not addressed).

Signed-off-by: Iain McGinniss <309153+iainmcgin@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants