feat: implement enhanced default behavior and git hook system (refs #73)

.githooks/commit-msg

@@ -0,0 +1,48 @@
+#!/bin/zsh
+
+# GoProX Commit Message Hook
+# Ensures all commits reference GitHub issues
+#
+# IMPORTANT: This is a SIMPLE, FOCUSED solution designed to prevent branch divergence
+# caused by commit amending. DO NOT add extra scripts, workflows, or complexity.
+#
+# Purpose: Block commits without (refs #XX) to prevent the need for amending pushed commits
+# Solution: Simple validation only - let the user fix the message and commit again
+#
+# If you're tempted to add more features here, STOP and ask the user first.
+# This hook should remain minimal and focused on its single responsibility.
+
+# Get the commit message from the commit-msg file
+commit_msg_file="$1"
+commit_msg=$(cat "$commit_msg_file")
+
+# Check if this is a merge commit or revert (allow without issue reference)
+if [[ "$commit_msg" =~ ^(Merge|Revert|Reverted) ]]; then
+    echo "Merge/revert commit detected, skipping issue reference check"
+    exit 0
+fi
+
+# Check if commit message contains GitHub issue reference
+# Pattern: (refs #n) or (refs #n #n ...) where n is a number
+if [[ "$commit_msg" =~ \(refs\ #[0-9]+(\ #[0-9]+)*\) ]]; then
+    echo "✅ Commit message contains GitHub issue reference"
+    exit 0
+else
+    echo "❌ ERROR: Commit message must reference a GitHub issue"
+    echo ""
+    echo "Please include a GitHub issue reference in your commit message:"
+    echo "  (refs #123) for a single issue"
+    echo "  (refs #123 #456) for multiple issues"
+    echo ""
+    echo "Examples:"
+    echo "  feat: add new configuration option (refs #70)"
+    echo "  fix: resolve parameter parsing issue (refs #45 #67)"
+    echo ""
+    echo "Current commit message:"
+    echo "---"
+    echo "$commit_msg"
+    echo "---"
+    echo ""
+    echo "Please add the issue reference and try committing again."
+    exit 1
+fi

.githooks/post-checkout

@@ -0,0 +1,42 @@
+#!/bin/zsh
+
+# GoProX Post-Checkout Hook
+# Automatically configures Git hooks after cloning or checking out
+
+# Run setup-hooks.zsh to ensure hooks are configured
+if [[ -f scripts/maintenance/setup-hooks.zsh ]]; then
+  echo "[GoProX] Running setup-hooks.zsh to configure git hooks..."
+  ./scripts/maintenance/setup-hooks.zsh
+else
+  echo "[GoProX] setup-hooks.zsh not found, skipping hook setup."
+fi
+
+# Run setup-brew.zsh to install Homebrew dependencies (if Homebrew is available)
+if command -v brew &> /dev/null; then
+  if [[ -f scripts/maintenance/setup-brew.zsh ]]; then
+    echo "[GoProX] Running setup-brew.zsh to install Homebrew dependencies..."
+    ./scripts/maintenance/setup-brew.zsh
+  else
+    echo "[GoProX] setup-brew.zsh not found, skipping Homebrew dependency setup."
+  fi
+else
+  echo "[GoProX] Homebrew not found, skipping Homebrew dependency setup."
+fi
+
+# Only run on initial clone (when previous HEAD is empty)
+if [[ -z "$2" ]]; then
+    echo "🔧 Setting up GoProX Git hooks..."
+
+    # Configure Git to use .githooks directory
+    git config core.hooksPath .githooks
+
+    echo "✅ Git hooks configured automatically!"
+    echo "   Commit messages will now require GitHub issue references (refs #123)"
+    echo "   Pre-commit checks will run before each commit"
+    echo "   YAML files will be linted (if yamllint is installed)"
+    echo "   Logger usage will be validated in zsh scripts"
+    echo ""
+    echo "💡 Optional: Install yamllint for YAML linting:"
+    echo "   brew install yamllint"
+    echo "   or: pip3 install yamllint"
+fi 

.githooks/post-commit

@@ -0,0 +1,30 @@
+#!/bin/zsh
+
+# GoProX Post-commit Hook
+# Provides helpful feedback after commits
+
+echo "🎉 Commit successful!"
+echo ""
+
+# Check if this is a feature branch
+current_branch=$(git branch --show-current)
+if [[ "$current_branch" =~ ^feature/ ]]; then
+    echo "💡 Tip: Consider creating a pull request when ready:"
+    echo "   gh pr create --title \"$(git log -1 --pretty=format:'%s')\""
+    echo ""
+fi
+
+# Check if there are any TODO/FIXME comments in the committed files
+committed_files=$(git diff-tree --no-commit-id --name-only -r HEAD)
+if echo "$committed_files" | xargs grep -l "TODO\|FIXME" 2>/dev/null; then
+    echo "⚠️  Note: This commit contains TODO/FIXME comments"
+    echo "   Consider addressing these in future commits"
+    echo ""
+fi
+
+# Check if yamllint is available for future commits
+if ! command -v yamllint &> /dev/null; then
+    echo "💡 Install yamllint for YAML linting in future commits:"
+    echo "   brew install yamllint"
+    echo ""
+fi 

.githooks/post-merge

@@ -0,0 +1,47 @@
+#!/bin/zsh
+
+# GoProX Post-Merge Hook
+# Automatically configures Git hooks after pulling or merging
+
+echo "🔧 Checking GoProX Git hooks configuration..."
+
+# Check if hooks are configured
+current_hooks_path=$(git config --local core.hooksPath 2>/dev/null || echo "")
+
+if [[ "$current_hooks_path" != ".githooks" ]]; then
+    echo "📝 Configuring Git hooks..."
+    git config --local core.hooksPath .githooks
+    echo "✅ Git hooks configured automatically!"
+    echo "   Commit messages will now require GitHub issue references (refs #123)"
+    echo "   Pre-commit checks will run before each commit"
+    echo "   YAML files will be linted (if yamllint is installed)"
+    echo "   Logger usage will be validated in zsh scripts"
+    echo ""
+    echo "💡 Optional: Install yamllint for YAML linting:"
+    echo "   brew install yamllint"
+    echo "   or: pip3 install yamllint"
+else
+    echo "✅ Git hooks already configured"
+fi
+
+# GoProX post-merge hook: auto-configure hooks and install Homebrew dependencies
+
+# Run setup-hooks.zsh to ensure hooks are configured
+if [[ -f scripts/maintenance/setup-hooks.zsh ]]; then
+  echo "[GoProX] Running setup-hooks.zsh to configure git hooks..."
+  ./scripts/maintenance/setup-hooks.zsh
+else
+  echo "[GoProX] setup-hooks.zsh not found, skipping hook setup."
+fi
+
+# Run setup-brew.zsh to install Homebrew dependencies (if Homebrew is available)
+if command -v brew &> /dev/null; then
+  if [[ -f scripts/maintenance/setup-brew.zsh ]]; then
+    echo "[GoProX] Running setup-brew.zsh to install Homebrew dependencies..."
+    ./scripts/maintenance/setup-brew.zsh
+  else
+    echo "[GoProX] setup-brew.zsh not found, skipping Homebrew dependency setup."
+  fi
+else
+  echo "[GoProX] Homebrew not found, skipping Homebrew dependency setup."
+fi 

.githooks/pre-commit

@@ -0,0 +1,124 @@
+#!/bin/zsh
+
+# GoProX Pre-commit Hook
+# Runs comprehensive checks before allowing commits
+
+echo "🔍 Running pre-commit checks..."
+
+# Check for TODO/FIXME comments in staged files
+if git diff --cached --name-only | xargs grep -l "TODO\|FIXME" 2>/dev/null; then
+    echo "⚠️  Warning: Found TODO/FIXME comments in staged files"
+    echo "   Consider addressing these before committing"
+fi
+
+# Check for large files (>10MB)
+large_files=$(git diff --cached --name-only | xargs ls -la 2>/dev/null | awk '$5 > 10485760 {print $9}')
+if [[ -n "$large_files" ]]; then
+    echo "⚠️  Warning: Found files larger than 10MB"
+    echo "   Consider using Git LFS for large files"
+fi
+
+# YAML Linting (if yamllint is available)
+if command -v yamllint &> /dev/null; then
+    echo "🔍 Running YAML linting..."
+
+    # Get staged YAML files
+    yaml_files=$(git diff --cached --name-only | grep -E '\.(yml|yaml)$' || true)
+
+    if [[ -n "$yaml_files" ]]; then
+        for file in $yaml_files; do
+            if [[ -f "$file" ]]; then
+                if ! yamllint -c .yamllint "$file" 2>/dev/null; then
+                    echo "❌ YAML linting failed for $file"
+                    echo "   Run: ./scripts/maintenance/fix-yaml-formatting.zsh to auto-fix"
+                    exit 1
+                fi
+            fi
+        done
+        echo "✅ YAML linting passed"
+    else
+        echo "ℹ️  No YAML files staged for linting"
+    fi
+else
+    echo "ℹ️  yamllint not available - skipping YAML linting"
+    echo "   Install with: brew install yamllint or pip3 install yamllint"
+fi
+
+# Check for logger usage in zsh scripts (per design principles)
+echo "🔍 Checking logger usage in zsh scripts..."
+zsh_files=$(git diff --cached --name-only | grep -E '\.zsh$' || true)
+if [[ -n "$zsh_files" ]]; then
+    for file in $zsh_files; do
+        if [[ -f "$file" ]]; then
+            # Skip if it's a core module (they define the logger)
+            if [[ "$file" != *"/core/"* ]]; then
+                if ! grep -q "log_" "$file"; then
+                    echo "⚠️  Warning: $file doesn't use logger functions"
+                    echo "   Consider using log_info, log_error, etc. for consistent logging"
+                fi
+            fi
+        fi
+    done
+fi
+
+# JSON Linting (if jsonlint is available)
+if command -v jsonlint &> /dev/null; then
+    echo "🔍 Running JSON linting..."
+
+    # Get staged JSON files
+    json_files=$(git diff --cached --name-only | grep -E '\.json$' || true)
+
+    if [[ -n "$json_files" ]]; then
+        for file in $json_files; do
+            if [[ -f "$file" ]]; then
+                if ! jsonlint "$file" >/dev/null 2>&1; then
+                    echo "❌ JSON linting failed for $file"
+                    echo "   Run: jsonlint $file to see errors"
+                    exit 1
+                fi
+            fi
+        done
+        echo "✅ JSON linting passed"
+    else
+        echo "ℹ️  No JSON files staged for linting"
+    fi
+else
+    echo "ℹ️  jsonlint not available - skipping JSON linting"
+    echo "   Install with: npm install -g jsonlint"
+fi
+
+# Check for file headers (copyright, license, usage patterns)
+echo "🔍 Checking file headers..."
+staged_files=$(git diff --cached --name-only || true)
+if [[ -n "$staged_files" ]]; then
+    for file in $staged_files; do
+        if [[ -f "$file" ]]; then
+            # Check for copyright notices in source files
+            if [[ "$file" =~ \.(zsh|md|yaml|yml|json)$ ]]; then
+                if ! head -10 "$file" | grep -q "Copyright\|copyright"; then
+                    echo "⚠️  Warning: $file missing copyright notice"
+                    echo "   Consider adding copyright header to file"
+                fi
+            fi
+
+            # Check for license headers in appropriate files
+            if [[ "$file" =~ \.(zsh|md)$ ]]; then
+                if ! head -10 "$file" | grep -q "License\|license"; then
+                    echo "⚠️  Warning: $file missing license header"
+                    echo "   Consider adding license information to file"
+                fi
+            fi
+
+            # Check for usage patterns in documentation
+            if [[ "$file" =~ \.md$ ]] && [[ "$file" != README.md ]] && [[ "$file" != CONTRIBUTING.md ]]; then
+                if ! head -10 "$file" | grep -q "Usage\|usage"; then
+                    echo "⚠️  Warning: $file missing usage documentation"
+                    echo "   Consider adding usage examples to documentation"
+                fi
+            fi
+        fi
+    done
+fi
+
+echo "✅ Pre-commit checks completed"
+exit 0

.githooks/pre-push

@@ -0,0 +1,3 @@
+#!/bin/sh
+command -v git-lfs >/dev/null 2>&1 || { printf >&2 "\n%s\n\n" "This repository is configured for Git LFS but 'git-lfs' was not found on your path. If you no longer wish to use Git LFS, remove this hook by deleting the 'pre-push' file in the hooks directory (set by 'core.hookspath'; usually '.git/hooks')."; exit 2; }
+git lfs pre-push "$@"

.github/workflows/integration-tests.yml

@@ -0,0 +1,94 @@
+---
+name: "Integration Tests"
+on:
+  push:
+    branches: ["main", "develop"]
+    paths-ignore:
+      - "docs/**"
+      - "*.md"
+      - "firmware/**"
+      - "output/**"
+
+jobs:
+  integration-validation:
+    name: "Integration Validation"
+    runs-on: "ubuntu-latest"
+    timeout-minutes: 15
+
+    steps:
+      - name: "Checkout code"
+        uses: actions/checkout@v4
+        with:
+          lfs: true
+
+      - name: "Install dependencies"
+        run: |
+          sudo apt-get update
+          sudo apt-get install -y zsh exiftool jq python3-pip
+          pip3 install yamllint
+
+      - name: "Make scripts executable"
+        run: |
+          chmod +x scripts/testing/*.zsh
+          chmod +x scripts/core/*.zsh
+          chmod +x goprox
+
+      - name: "Setup output directories"
+        run: |
+          mkdir -p output/test-results
+          mkdir -p output/test-temp
+
+      - name: "Run comprehensive validation"
+        run: |
+          echo "🧪 Running comprehensive validation..."
+          zsh ./scripts/testing/validate-integration.zsh
+
+      - name: "Run file comparison tests"
+        run: |
+          echo "🧪 Running file comparison tests..."
+          zsh ./scripts/testing/test-regression.zsh
+
+      - name: "Upload test results"
+        if: always()
+        uses: actions/upload-artifact@v4
+        with:
+          name: "integration-test-results"
+          path: "output/"
+          retention-days: 7
+
+  test-summary:
+    name: "Test Summary"
+    needs: integration-validation
+    runs-on: "ubuntu-latest"
+    if: always()
+
+    steps:
+      - name: "Checkout code"
+        uses: actions/checkout@v4
+        with:
+          lfs: true
+
+      - name: "Download test results"
+        uses: actions/download-artifact@v4
+        with:
+          name: "integration-test-results"
+          path: "test-results"
+
+      - name: "Generate summary"
+        run: |
+          echo "📊 Integration Test Summary"
+          echo "=========================="
+          echo "Generated: $(date)"
+          echo ""
+
+          if [[ -d "test-results" ]]; then
+            find test-results -name "*.txt" -type f | while read -r report; do
+              echo "📋 $(basename "$report"):"
+              cat "$report"
+              echo ""
+              echo "---"
+              echo ""
+            done
+          else
+            echo "No test results found"
+          fi