fix: split clippy into own CI job and fix broken update.rs tests

- Move clippy into a dedicated `clippy` job so it runs independently
  from tests and is separately visible in PR checks
- Remove `components: clippy` from the test job (no longer needed)
- Fix 2 pre-existing test failures in package_json::update::tests:
  assertions checked for "socket patch apply" (space) but the
  SOCKET_PATCH_COMMAND writes "socket-patch apply" (hyphen)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: mikolalysenko

.github/workflows/ci.yml

@@ -9,6 +9,31 @@ permissions:
   contents: read
 
 jobs:
+  clippy:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
+
+      - name: Install Rust
+        uses: dtolnay/rust-toolchain@efa25f7f19611383d5b0ccf2d1c8914531636bf9 # stable
+        with:
+          toolchain: stable
+          components: clippy
+
+      - name: Cache cargo
+        uses: actions/cache@0057852bfaa89a56745cba8c7296529d2fc39830 # v4
+        with:
+          path: |
+            ~/.cargo/registry
+            ~/.cargo/git
+            target
+          key: ubuntu-latest-cargo-clippy-${{ hashFiles('**/Cargo.lock') }}
+          restore-keys: ubuntu-latest-cargo-clippy-
+
+      - name: Run clippy
+        run: cargo clippy --workspace --all-features -- -D warnings
+
   test:
     strategy:
       matrix:
@@ -22,7 +47,6 @@ jobs:
         uses: dtolnay/rust-toolchain@efa25f7f19611383d5b0ccf2d1c8914531636bf9 # stable
         with:
           toolchain: stable
-          components: clippy
 
       - name: Cache cargo
         uses: actions/cache@0057852bfaa89a56745cba8c7296529d2fc39830 # v4
@@ -34,9 +58,6 @@ jobs:
           key: ${{ matrix.os }}-cargo-${{ hashFiles('**/Cargo.lock') }}
           restore-keys: ${{ matrix.os }}-cargo-
 
-      - name: Run clippy
-        run: cargo clippy --workspace --all-features -- -D warnings
-
       - name: Run tests
         run: cargo test --workspace --all-features
 

crates/socket-patch-core/src/package_json/update.rs

@@ -156,7 +156,7 @@ mod tests {
         let result = update_package_json(&pkg, false).await;
         assert_eq!(result.status, UpdateStatus::Updated);
         let content = fs::read_to_string(&pkg).await.unwrap();
-        assert!(content.contains("socket patch apply"));
+        assert!(content.contains("socket-patch apply"));
     }
 
     #[tokio::test]
@@ -178,7 +178,7 @@ mod tests {
         assert_eq!(result.status, UpdateStatus::Updated);
         let content = fs::read_to_string(&pkg).await.unwrap();
         assert!(content.contains("postinstall"));
-        assert!(content.contains("socket patch apply"));
+        assert!(content.contains("socket-patch apply"));
     }
 
     #[tokio::test]