Merge branch 'main' into pr/pinin4fjords/43

Author: flying-sheep

.github/workflows/ci.yml

@@ -94,7 +94,7 @@ jobs:
 
       - uses: Swatinem/rust-cache@c19371144df3bb44fab255c43d04cbc2ab54d1c4 # ratchet:Swatinem/rust-cache@v2.9.1
 
-      - run: cargo clippy -- -D warnings
+      - run: cargo clippy --all-targets -- -D warnings
 
   msrv:
     name: MSRV check
@@ -120,3 +120,20 @@ jobs:
       - uses: rustsec/audit-check@v2
         with:
           token: ${{ secrets.GITHUB_TOKEN }}
+
+  # Check that all tests defined above pass. This makes it easy to set a single "required" test in branch
+  # protection instead of having to update it frequently. See https://github.com/re-actors/alls-green#why.
+  check:
+    name: Checks pass
+    if: always()
+    needs:
+      - test
+      - fmt 
+      - clippy
+      - msrv
+      - audit
+    runs-on: ubuntu-latest
+    steps:
+        - uses: re-actors/alls-green@05ac9388f0aebcb5727afa17fcccfecd6f8ec5fe # ratchet:re-actors/alls-green@v1.2.2
+          with:
+            jobs: ${{ toJSON(needs) }}

.vscode/settings.json

@@ -0,0 +1,7 @@
+{
+	"rust-analyzer.check.command": "clippy",
+	"[rust]": {
+		"editor.formatOnSave": true,
+		"editor.defaultFormatter": "rust-lang.rust-analyzer"
+	}
+}

CLAUDE.md

@@ -20,15 +20,15 @@ The primary goal is a faithful port — matching the original STAR behavior as c
 Rust 2024 edition. Standard Cargo commands:
 
 ```bash
-cargo build            # Debug build
-cargo build --release  # Release build
-cargo test             # Run all tests
-cargo test <name>      # Run a single test by name
-cargo clippy           # Lint
-cargo fmt              # Format code
+cargo build                 # Debug build
+cargo build --release       # Release build
+cargo test                  # Run all tests
+cargo test <name>           # Run a single test by name
+cargo clippy --all-targets  # Lint
+cargo fmt                   # Format code
 ```
 
-Always run `cargo clippy`, `cargo fmt --check`, and `cargo test` before considering a phase complete.
+Always run `cargo clippy --all-targets`, `cargo fmt --check`, and `cargo test` before considering a phase complete.
 
 ## Current Status
 

CONTRIBUTING.md

@@ -5,11 +5,11 @@
 Rust 2024 edition. Standard Cargo commands:
 
 ```bash
-cargo build            # debug build
-cargo build --release  # release build
-cargo test             # run all tests
-cargo clippy           # lint (zero warnings expected)
-cargo fmt --check      # formatting check
+cargo build                 # debug build
+cargo build --release       # release build
+cargo test                  # run all tests
+cargo clippy --all-targets  # lint (zero warnings expected)
+cargo fmt --check           # formatting check
 ```
 
 CI runs on Linux (x86_64, x86-64-v3, aarch64), macOS (aarch64), and Windows (x86_64). PRs must pass all CI checks before merging.