ci: make CI genuinely green — rust-ci toolchain pin + fmt/clippy (#48)

## Summary

Make CI genuinely green. The shared rust-ci pin on `main` predates
standards#439, so the SHA-pinned `dtolnay/rust-toolchain` step errors
out before the job runs. This bumps the pin so rust-ci actually runs,
and brings the Rust sources to fmt + clippy(`-D warnings`) clean under
the CI toolchain (stable 1.96).

## Changes

- **rust-ci:** bump `rust-ci-reusable.yml` pin `d135b05` → `8dc2bf0`
(current `standards` HEAD; includes #439 toolchain fix + #441/#442).
- **Rust hygiene:** `cargo fmt` + `clippy --fix` where the repo had
pre-existing drift, so `cargo fmt --all -- --check` and `cargo clippy
--locked --all-targets -- -D warnings` pass.

## RSR Quality Checklist

### Required
- [x] Tests pass (`cargo test --locked --all-targets`)
- [x] Code is formatted (`cargo fmt --all -- --check`)
- [x] Linter is clean (`cargo clippy --locked --all-targets -- -D
warnings`)
- [x] No banned language patterns
- [x] SPDX license headers present on modified files
- [x] No secrets, credentials, or `.env` files included

## Testing

Verified locally with the CI toolchain (rustc/clippy/rustfmt 1.96.0):
`cargo fmt --check`, `clippy -D warnings`, `cargo check --locked`,
`cargo test --locked` all pass.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---
_Generated by [Claude
Code](https://claude.ai/code/session_01A6PSzJWpRxtzGDjUCEh7Mx)_

---------

Co-authored-by: Claude <noreply@anthropic.com>

Author: hyperpolymath

.github/workflows/rust-ci.yml

@@ -14,4 +14,4 @@ permissions:
 
 jobs:
   rust-ci:
-    uses: hyperpolymath/standards/.github/workflows/rust-ci-reusable.yml@d135b05bfc647d0c0fbfedc7e80f37ea50f49236
+    uses: hyperpolymath/standards/.github/workflows/rust-ci-reusable.yml@8dc2bf039d1ff0372d650895c46bea7fbaec68ff

src/codegen/ats_gen.rs

@@ -389,9 +389,18 @@ mod tests {
 
     #[test]
     fn test_parse_pattern() {
-        assert_eq!(parse_pattern("alloc").expect("TODO: handle error"), OwnershipPattern::Alloc);
-        assert_eq!(parse_pattern("free").expect("TODO: handle error"), OwnershipPattern::Free);
-        assert_eq!(parse_pattern("borrow").expect("TODO: handle error"), OwnershipPattern::Borrow);
+        assert_eq!(
+            parse_pattern("alloc").expect("TODO: handle error"),
+            OwnershipPattern::Alloc
+        );
+        assert_eq!(
+            parse_pattern("free").expect("TODO: handle error"),
+            OwnershipPattern::Free
+        );
+        assert_eq!(
+            parse_pattern("borrow").expect("TODO: handle error"),
+            OwnershipPattern::Borrow
+        );
         assert_eq!(
             parse_pattern("transfer").expect("TODO: handle error"),
             OwnershipPattern::Transfer

src/codegen/compiler.rs

@@ -271,7 +271,8 @@ mod tests {
     #[test]
     fn test_build_command_structure() {
         let manifest = test_manifest();
-        let cmd = build_command(&manifest, "generated/ats/test_safe.dats", false).expect("TODO: handle error");
+        let cmd = build_command(&manifest, "generated/ats/test_safe.dats", false)
+            .expect("TODO: handle error");
 
         assert_eq!(cmd.program, "patscc");
         assert!(cmd.args.contains(&"-DATS_MEMALLOC_LIBC".to_string()));