Merge pull request 'ci(pr-checks): cache cargo with rust-cache so the cache stops growing' (#33) from chore/ci-self-cleanup into edge

Reviewed-on: #33
Reviewed-by: Colby Whitlock <whitlocktech@gmail.com>

.gitea/workflows/pr-checks.yml

@@ -69,18 +69,17 @@ jobs:
           rustup target add x86_64-pc-windows-gnu
           cargo --version && cargo fmt --version && cargo clippy --version
 
-      # Keyed on Cargo.lock: dependency builds are reused until a dep actually
+      # Dependency builds are reused until a dep actually changes; a cache miss
-      # changes. A cache miss only makes the run slower, never wrong.
+      # only makes the run slower, never wrong. rust-cache rather than a plain
-      - name: Cache cargo registry and build dir
+      # actions/cache of target/: it strips this crate's own artifacts and
-        uses: actions/cache@v4
+      # anything stale before saving, and saves only on a miss. The plain cache
+      # restored the previous target/ and re-saved all of it under every new
+      # Cargo.lock, so each entry carried every older build and the runner's
+      # cache grew without bound.
+      - name: Cache cargo registry and dependency builds
+        uses: Swatinem/rust-cache@v2
         with:
-          path: |
+          workspaces: sidecar -> target
-            ~/.cargo/registry
-            ~/.cargo/git
-            sidecar/target
-          key: ${{ runner.os }}-cargo-${{ hashFiles('sidecar/Cargo.lock') }}
-          restore-keys: |
-            ${{ runner.os }}-cargo-
 
       # Cheapest gate first — parses only, no compile, so a formatting slip fails
       # in seconds instead of after a full build.