Merge branch 'trunk' into fix/setup-action-no-cleanup

Author: lukekim

.github/workflows/ci.yml

@@ -80,6 +80,9 @@ jobs:
           SPICEIO_REGION: ${{ vars.SPICEIO_REGION || 'us-west-1' }}
         run: ./scripts/test-sccache.sh
 
+      - name: Compile benchmarks
+        run: cargo bench --locked --no-run
+
       - name: Build release artifact
         run: cargo build --release --locked --bin spiceio
 

CLAUDE.md

@@ -21,9 +21,27 @@ make release                   # optimized release build
 make lint                      # fmt-check + check + strict clippy + rustdoc warnings
 make test                      # sccache integration test (requires SPICEIO_SMB_USER/PASS)
 make fmt                       # auto-format
+make bench                     # criterion benchmarks (crypto + protocol)
+make bench-live                # live throughput benchmarks against NAS
 make clean                     # cargo clean
 ```
 
+## Pre-PR checklist
+
+Always run these before creating a PR — they mirror what CI checks:
+
+```bash
+cargo fmt --all                # auto-format first
+cargo fmt --all --check        # verify no formatting diff remains
+cargo clippy --locked --all-targets --all-features -- -D warnings -D clippy::all -D clippy::cargo -A clippy::cargo-common-metadata
+RUSTDOCFLAGS="-D warnings" cargo doc --locked --workspace --no-deps --document-private-items
+cargo test --locked            # unit tests must pass
+```
+
+Or simply: `make lint && cargo test --locked`
+
+The CI also runs `./scripts/test-sccache.sh` (sccache integration test) when SMB credentials are available.
+
 The binary requires these environment variables:
 - `SPICEIO_SMB_SERVER` (required) — SMB server hostname or IP
 - `SPICEIO_SMB_USER` (required) — SMB username