feat!: migrate git hooks to ZX

Author: TerrorSquad

.github/copilot-instructions.md

@@ -53,10 +53,10 @@ Copilot SHOULD NOT generate raw php-cs-fixer, phpstan, rector, or psalm command
 
 ---
 ## 5. Git Hooks Architecture
-- Unified helper: `tools/commit-utils.py` handles: `--need-ticket`, `--footer-label`, `--extract-ticket`.
-- Common library: `tools/git-hooks/lib/common.sh` provides shared functionality and colored logging.
-- `commit-msg` hook validates branch, lints message, appends footer if needed.
-- Copilot SHOULD reference `commit-utils.py` and the common library instead of legacy scripts.
+- Native ZX implementation: All git hooks use Google ZX for secure, cross-platform execution.
+- Common library: `tools/git-hooks/shared/utils.mjs` provides shared functionality and colored logging.
+- `commit-msg` hook validates branch, lints message, appends footer if needed - all functionality integrated natively.
+- Copilot SHOULD reference the ZX-based hook system and shared utilities instead of legacy scripts.
 
 ---
 ## 6. Configuration Files (Do Not Duplicate)
@@ -120,13 +120,13 @@ If user asks about improvements, Copilot may propose:
 
 ---
 ## 13. Quick Reference Snippets
-Check ticket requirement:
+Check current branch and validation:
 ```
-python3 tools/commit-utils.py --need-ticket
+git rev-parse --abbrev-ref HEAD
 ```
-Extract ticket from current branch:
+Validate branch manually:
 ```
-python3 tools/commit-utils.py --extract-ticket "$(git rev-parse --abbrev-ref HEAD)"
+./node_modules/.bin/validate-branch-name -t "$(git rev-parse --abbrev-ref HEAD)"
 ```
 Run all analyzers (example if combined script exists, otherwise run individually):
 ```

booster/.ddev/config.yaml

@@ -21,8 +21,9 @@ timezone: Europe/Berlin
 hooks:
   pre-start:
     - exec-host: python3 .ddev/php/create-xdebug-local.py
+    - exec-host: git config core.hooksPath tools/git-hooks/hooks
   post-start:
-    - exec: bash ./tools/git-hooks/setup.sh
+    - exec: yes | pnpm install
     - exec: composer install --prefer-dist --no-progress --no-interaction
     - exec: python3 .phpstorm/update_xdebug_config.py
     - exec: python3 .ddev/php/add-xdebug-trigger-to-nginx.py

booster/.gitignore

@@ -2,7 +2,6 @@ vendor
 .idea
 node_modules
 .pnpm-store
-.husky
 coverage.xml
 phpstan_sonar.json
 psalm_sonar.json

booster/composer.json

@@ -17,10 +17,10 @@
 	},
 	"scripts": {
 		"post-install-cmd": [
-			"bash ./tools/git-hooks/setup.sh"
+			"yes | pnpm install"
 		],
 		"post-update-cmd": [
-			"bash ./tools/git-hooks/setup.sh"
+			"yes | pnpm install"
 		],
 		"generate-api-spec": "php documentation/api.php",
 		"check-cs": "vendor/bin/ecs check --ansi",

booster/documentation/api.php

@@ -9,18 +9,20 @@
 // Change this to the paths of your API classes
 $scanPaths = glob('./src');
 
-if (! is_array($scanPaths)) {
+if (!is_array($scanPaths)) {
     exit('No paths found to scan for OpenAPI documentation.');
 }
 
-$openapi = Generator::scan($scanPaths);
+$generator = new Generator();
+
+$openapi = $generator->generate($scanPaths);
 
 if ($openapi === null) {
     exit('Failed to generate OpenAPI documentation.');
 }
 
 $filename = 'documentation/openapi.yml';
-$content  = $openapi->toYaml();
+$content = $openapi->toYaml();
 
 if (file_put_contents($filename, $content) === false) {
     exit('Unable to write to file ' . $filename);

booster/integrate_booster.sh

@@ -292,33 +292,27 @@ function copy_files() {
     # Copy selected tool scripts instead of entire tools directory
     local tools_src="${BOOSTER_INTERNAL_PATH}/tools"
     if [ -d "$tools_src" ]; then
-        mkdir -p tools/git-hooks/hooks
-        # List of tool files to copy (public runtime helpers only)
-        local tool_files=(
-            "runner.sh"
-            "commit-utils.py"
-            "git-hooks"
-        )
-        for f in "${tool_files[@]}"; do
-            if [ -f "$tools_src/$f" ]; then
-                log "  Copying tool '$f'"
-                cp "$tools_src/$f" tools/
-                # Set execute permissions for shell scripts and Python files
-                if [[ "$f" == *.sh ]] || [[ "$f" == *.py ]]; then
-                    chmod +x "tools/$f"
-                fi
-            elif [ -d "$tools_src/$f" ]; then
-                log "  Copying tool directory '$f'"
-                cp -R "$tools_src/$f" tools/
-                # Set execute permissions for shell scripts in the directory
-                find "tools/$f" -name "*.sh" -exec chmod +x {} \;
-                find "tools/$f" -name "*.bash" -exec chmod +x {} \;
-            else
-                warn "  Expected tool file missing: $f"
-            fi
-        done
-
+        mkdir -p tools/git-hooks
+
+        # Copy git-hooks/hooks directory
+        if [ -d "$tools_src/git-hooks/hooks" ]; then
+            log "  Copying git-hooks/hooks directory"
+            cp -R "$tools_src/git-hooks/hooks" tools/git-hooks/
+            # Set execute permissions for all scripts in hooks directory
+            find "tools/git-hooks/hooks" -type f \( -name "*.sh" -o -name "*.bash" -o -name "*.mjs" \) -exec chmod +x {} \;
+        else
+            warn "  Expected tool directory missing: git-hooks/hooks"
+        fi
 
+        # Copy git-hooks/shared directory
+        if [ -d "$tools_src/git-hooks/shared" ]; then
+            log "  Copying git-hooks/shared directory"
+            cp -R "$tools_src/git-hooks/shared" tools/git-hooks/
+            # Set execute permissions for scripts in shared directory
+            find "tools/git-hooks/shared" -type f \( -name "*.sh" -o -name "*.bash" -o -name "*.mjs" \) -exec chmod +x {} \;
+        else
+            warn "  Expected tool directory missing: git-hooks/shared"
+        fi
     else
         warn "  Booster tools directory not found; skipping tools copy."
     fi
@@ -358,7 +352,8 @@ function update_package_json() {
             .[0] as $proj | .[1] as $booster |
             $proj * {
                 scripts: (($proj.scripts // {}) + ($booster.scripts // {})),
-                devDependencies: (($proj.devDependencies // {}) + ($booster.devDependencies // {}))
+                devDependencies: (($proj.devDependencies // {}) + ($booster.devDependencies // {})),
+                husky: (($proj.husky // {}) + ($booster.husky // {}))
             }
             ' "$project_pkg" "$booster_pkg" >"$tmp_pkg" || error "Failed to merge package.json using jq."
 

booster/package.json

@@ -4,18 +4,24 @@
   "type": "module",
   "version": "0.6.0",
   "scripts": {
-    "prepare": "husky",
     "commit": "commit",
-    "generate:api-doc:html": "redocly build-docs documentation/openapi.yml --output=documentation/openapi.html"
+    "generate:api-doc:html": "redocly build-docs documentation/openapi.yml --output=documentation/openapi.html",
+    "test:hooks": "vitest run tools/git-hooks",
+    "test:hooks:watch": "vitest watch tools/git-hooks"
+  },
+  "husky": {
+    "hooksPath": "tools/git-hooks/hooks"
   },
   "devDependencies": {
     "@commitlint/cli": "^19.8.1",
     "@commitlint/config-conventional": "^19.8.1",
     "@commitlint/prompt-cli": "^19.8.1",
     "@commitlint/types": "^19.8.1",
-    "@redocly/cli": "^2.0.2",
+    "@redocly/cli": "^2.1.3",
     "husky": "^9.1.7",
-    "validate-branch-name": "^1.3.2"
+    "validate-branch-name": "^1.3.2",
+    "zx": "^8.8.1",
+    "vitest": "^3.2.4"
   },
   "packageManager": "pnpm@10.16.1"
 }