fix: Resolve clippy collapsible_if and surface archive errors in CI

- Collapse nested if in normalize_worktree_name (clippy::collapsible_if)
- Add set -o pipefail to archive/export steps so xcodebuild failures
  propagate through xcbeautify pipe
- Add ls check after archive to verify xcarchive was created

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: 2witstudios

.github/workflows/release.yml

@@ -69,6 +69,7 @@ jobs:
 
       - name: Archive
         run: |
+          set -o pipefail
           xcodebuild archive \
             -project apps/purepoint-macos/purepoint-macos.xcodeproj \
             -scheme purepoint-macos \
@@ -79,10 +80,12 @@ jobs:
             DEVELOPMENT_TEAM="${{ secrets.APPLE_TEAM_ID }}" \
             MARKETING_VERSION="${{ steps.version.outputs.marketing }}" \
             CURRENT_PROJECT_VERSION="${{ steps.version.outputs.build }}" \
-            2>&1 | xcbeautify
+            2>&1 | xcbeautify --report junit --report-path build/archive-report.xml
+          ls -la build/PurePoint.xcarchive
 
       - name: Export archive
         run: |
+          set -o pipefail
           xcodebuild -exportArchive \
             -archivePath build/PurePoint.xcarchive \
             -exportPath build/export \

crates/pu-core/src/id.rs

@@ -78,11 +78,9 @@ pub fn normalize_worktree_name(input: &str) -> String {
         if ch.is_ascii_alphanumeric() {
             result.push(ch);
             prev_hyphen = false;
-        } else if ch.is_whitespace() || ch == '_' || ch == '-' {
-            if !prev_hyphen {
-                result.push('-');
-                prev_hyphen = true;
-            }
+        } else if (ch.is_whitespace() || ch == '_' || ch == '-') && !prev_hyphen {
+            result.push('-');
+            prev_hyphen = true;
         }
         // else: strip
     }