fix(zero-rust): run the Version PR's version script in a shell, bump the apps' lockfiles (#145)

changesets/action runs version-script through @actions/exec without a shell, so the && chain was passed as arguments and the Version PR job failed on every push to main since #130. The script now runs under sh -c. rust/scripts/set-version.sh also bumps zero-client in the zbugs apps' lockfiles so their --locked builds keep working after a release.

Amp-Thread-ID: https://ampcode.com/threads/T-01a1c6b9-fc99-73a1-bf70-e71fd484042c

Author: tmm

.github/workflows/swift-release.yml

@@ -55,8 +55,9 @@ jobs:
         with:
           # One PR versions both clients: Rust Release (rust-release.yml)
           # has no version job of its own, since two would race for the
-          # same changeset-release/main branch.
-          version-script: pnpm --filter zero-swift version-packages && pnpm --filter zero-rust sync-version
+          # same changeset-release/main branch. The action runs the script
+          # without a shell (@actions/exec), so `&&` needs `sh -c`.
+          version-script: sh -c "pnpm --filter zero-swift version-packages && pnpm --filter zero-rust sync-version"
           commit-message: 'chore: version packages'
           pr-title: 'chore: version packages'
 

rust/scripts/set-version.sh

@@ -1,7 +1,8 @@
 #!/bin/sh
 # Sets the version of the Rust crates: the workspace version in Cargo.toml,
 # the version zero-conformance requires of zero-client, and the workspace
-# crates' entries in Cargo.lock, so `cargo --locked` agrees.
+# crates' entries in Cargo.lock and in the apps' lockfiles, so
+# `cargo --locked` agrees.
 #
 #   rust/scripts/set-version.sh 0.1.0
 set -eu
@@ -14,7 +15,13 @@ sed -i.bak -E \
   -e "/^\[workspace\.package\]/,/^\[/ s/^version = \"[^\"]*\"/version = \"$version\"/" \
   -e "s/^(zero-client = \{ path = \"zero-client\", version = )\"[^\"]*\"/\1\"$version\"/" \
   "$dir/Cargo.toml"
-sed -i.bak -E \
-  -e "/^name = \"zero-(client|conformance)\"$/{n;s/^version = \"[^\"]*\"/version = \"$version\"/;}" \
-  "$dir/Cargo.lock"
-rm "$dir/Cargo.toml.bak" "$dir/Cargo.lock.bak"
+rm "$dir/Cargo.toml.bak"
+# The apps in this repository (apps/*) are workspaces of their own that
+# depend on zero-client by path, so their lockfiles name its version too.
+for lock in "$dir/Cargo.lock" "$dir"/../apps/*/Cargo.lock; do
+  [ -f "$lock" ] || continue
+  sed -i.bak -E \
+    -e "/^name = \"zero-(client|conformance)\"$/{n;s/^version = \"[^\"]*\"/version = \"$version\"/;}" \
+    "$lock"
+  rm "$lock.bak"
+done