Skip to content

fix(ci): restructure release-please config for workspace version inheritance#13

Merged
StefanSteiner merged 1 commit into
tableau:mainfrom
StefanSteiner:ssteiner/fix-release-please
May 18, 2026
Merged

fix(ci): restructure release-please config for workspace version inheritance#13
StefanSteiner merged 1 commit into
tableau:mainfrom
StefanSteiner:ssteiner/fix-release-please

Conversation

@StefanSteiner
Copy link
Copy Markdown
Contributor

Summary

  • Release-please fails with value at path package.version is not tagged because member Cargo.toml files use version.workspace = true (not a literal version string)
  • Restructure config: collapse per-crate Rust packages into a single root package (.) that reads [workspace.package] version from root Cargo.toml
  • Add # x-release-please-version annotation to root Cargo.toml so the generic updater can locate the version
  • Use extra-files with jsonpath to keep optionalDependencies version pins in sync across hyperdb-mcp/npm/package.json and hyperdb-api-node/package.json
  • Keep all npm platform packages as separate release-type: "node" entries, linked via linked-versions plugin

Test plan

  • Merge this PR (the fix(ci) prefix triggers a patch bump)
  • Observe release-please run on main — should open a release PR bumping 0.1.1 → 0.1.2
  • Verify the release PR updates: root Cargo.toml version, all npm package.json versions + optionalDependencies, and generates a CHANGELOG.md

…ritance

Release-please's Rust strategy fails with "value at path package.version
is not tagged" when member crates use `version.workspace = true`. Fix by
collapsing per-crate Rust packages into a single root package that reads
the workspace version from root Cargo.toml (annotated with
x-release-please-version).

The linked-versions plugin keeps all npm platform packages in lockstep.
Extra-files with jsonpath updaters keep optionalDependencies references
in the parent npm packages synchronized.
@StefanSteiner StefanSteiner merged commit fd18a8b into tableau:main May 18, 2026
9 checks passed
StefanSteiner added a commit that referenced this pull request May 18, 2026
…ing (#14)

## Summary

Follow-up to #13. The `release-type: "rust"` strategy still
auto-discovers workspace members from root `Cargo.toml`'s `[workspace]
members` list and tries to update each member's `Cargo.toml` — hitting
the same "package.version is not tagged" error.

- Switch root package to `release-type: "simple"` which only updates
files explicitly listed in `extra-files`
- Add `Cargo.toml` to `extra-files` so the `x-release-please-version`
annotated workspace version gets bumped
- Remove top-level `release-type: "rust"` default (each package now
declares its own type)

## Test plan

- [ ] Merge and observe release-please workflow run succeeds
- [ ] Release-please opens a release PR bumping 0.1.1 → 0.1.2
@github-actions github-actions Bot mentioned this pull request May 18, 2026
StefanSteiner added a commit that referenced this pull request May 18, 2026
🤖 I have created a release *beep* *boop*
---


<details><summary>0.1.2</summary>

##
[0.1.2](v0.1.1...v0.1.2)
(2026-05-18)


### Bug Fixes

* **ci:** include README.md in hyperdb-mcp npm package
([c8ccc22](c8ccc22))
* **ci:** include README.md in hyperdb-mcp npm package
([#12](#12))
([b1ddb33](b1ddb33))
* **ci:** prevent npm-publish chmod step from failing on missing
binaries
([2708ee4](2708ee4))
* **ci:** prevent npm-publish chmod step from failing on missing
binaries ([#11](#11))
([bc9bee5](bc9bee5))
* **ci:** remove brew rust on macOS before installing toolchain
([b331607](b331607))
* **ci:** remove brew-rust uninstall steps that delete cargo/rustc on
new image
([af798f1](af798f1))
* **ci:** restructure release-please config for workspace version
inheritance
([d5ad018](d5ad018))
* **ci:** restructure release-please config for workspace version
inheritance ([#13](#13))
([fd18a8b](fd18a8b))
* **ci:** use simple release-type to avoid Cargo workspace member
walking
([3884162](3884162))
* **ci:** use simple release-type to avoid Cargo workspace member
walking ([#14](#14))
([42f0524](42f0524))
</details>

<details><summary>0.1.2</summary>

##
[0.1.2](v0.1.1...v0.1.2)
(2026-05-18)


### Bug Fixes

* **ci:** include README.md in hyperdb-mcp npm package
([c8ccc22](c8ccc22))
* **ci:** include README.md in hyperdb-mcp npm package
([#12](#12))
([b1ddb33](b1ddb33))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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.

1 participant