feat(dx): Add custom pre-commit hook to check for disabled tests

Author: philprime

.git-hooks/check-testing-disabled.sh

@@ -0,0 +1,22 @@
+#!/usr/bin/env bash
+# Check that testing dependencies are not accidentally committed
+
+has_error=0
+
+for file in "$@"; do
+    if [[ "$file" == "Package.swift" ]]; then
+        if grep -q 'isTestingEnabled = /\*TESTING_FLAG\*/true/\*TESTING_FLAG\*/' "$file"; then
+            echo "ERROR: Package.swift has testing enabled."
+            echo "Please set isTestingEnabled to false before committing."
+            has_error=1
+        fi
+    fi
+
+    if [[ "$file" == "Package.resolved" ]]; then
+        echo "ERROR: Package.resolved should not be committed."
+        echo "Please unstage this file with: git reset HEAD Package.resolved"
+        has_error=1
+    fi
+done
+
+exit $has_error

.pre-commit-config.yaml

@@ -54,3 +54,10 @@ repos:
         language: system
         args:
           - "fmt"
+
+      - id: check-testing-disabled
+        name: Check testing is disabled
+        description: "Ensure Package.swift has testing disabled and Package.resolved is not committed"
+        files: ^(Package\.swift|Package\.resolved)$
+        entry: .git-hooks/check-testing-disabled.sh
+        language: script