Shaun Arman a365cba30e fix(ci): address second AI review — || true, ca-certs, cache@v4, key suffixes ... Dockerfiles: - Remove || true from rustup component add in all three Linux images; rust:1.88-slim default profile already includes both components so the command is a clean no-op, not a failure risk — silencing errors served no purpose and only hid potential toolchain issues - Add ca-certificates explicitly to Dockerfile.linux-amd64 and Dockerfile.windows-cross (rust:1.88-slim includes it, but being explicit is consistent with the arm64 fix and future-proofs against base image changes) Workflows: - Upgrade actions/cache@v3 → @v4 across test.yml and auto-tag.yml (v3 deprecated; v4 has parallel uploads and better large-cache support) - Add linux-amd64 suffix to cargo cache keys in test.yml Rust jobs and auto-tag.yml build-linux-amd64 job; all four jobs target the same architecture and now share a cache, benefiting from cross-job hits (registry cache is source tarballs, not compiled artifacts — no pollution risk between targets) Not changed: - alpine:latest + docker-cli in build-images.yml is correct; the reviewer confused DinD with socket passthrough — docker:24-cli also has no daemon, both use the host socket; the builds already proved alpine works - curl|bash for rustup is the official install method; rustup.rs publishes no checksums for the installer script itself		2026-04-12 20:16:32 -05:00
..
Dockerfile.linux-amd64	fix(ci): address second AI review — || true, ca-certs, cache@v4, key suffixes	2026-04-12 20:16:32 -05:00
Dockerfile.linux-arm64	fix(ci): address second AI review — || true, ca-certs, cache@v4, key suffixes	2026-04-12 20:16:32 -05:00
Dockerfile.windows-cross	fix(ci): address second AI review — || true, ca-certs, cache@v4, key suffixes	2026-04-12 20:16:32 -05:00