Add an opt-in pre-commit hook - #415

#!/bin/bash
set -eu

printf '%s\n' '--- hook files ---'
git ls-files '.githooks/pre-commit' '.githooks/*' '.github/*' 2>/dev/null || true

printf '%s\n' '--- pre-commit outline ---'
ast-grep outline .githooks/pre-commit 2>/dev/null || true

printf '%s\n' '--- pre-commit contents ---'
cat -n .githooks/pre-commit

printf '%s\n' '--- related staged/path handling ---'
rg -n -C 4 'staged\(|swiftformat|swiftlint|SwiftFormat|SwiftLint|diff-filter|git diff --cached' .githooks .github . 2>/dev/null | head -n 500

#!/bin/bash
set -eu

cat -n .githooks/pre-commit
rg -n -C 5 'staged\(|swiftformat|swiftlint|SwiftFormat|SwiftLint|git diff --cached' .githooks .github 2>/dev/null || true

#!/bin/bash
set -eu

python3 - <<'PY'
from pathlib import Path
import re

p = Path(".githooks/pre-commit")
text = p.read_text()
checks = {
    "staged names only": "git diff --cached --name-only -z" in text,
    "shellcheck receives staged() output": bool(re.search(r"staged '\*\.sh'.*\|\s*xargs -0 shellcheck", text)),
    "SwiftFormat receives staged() output": bool(re.search(r"staged '\*\.swift'.*\|\s*xargs -0 swiftformat", text)),
    "SwiftLint receives staged() output": bool(re.search(r"staged '\*\.swift'.*\|\s*xargs -0 swiftlint", text)),
    "index-content read mechanism": any(x in text for x in ("git show :", "git cat-file", "git checkout-index", "mktemp", "stdin")),
}
for name, value in checks.items():
    print(f"{name}: {value}")
assert checks["staged names only"]
assert checks["shellcheck receives staged() output"]
assert checks["SwiftFormat receives staged() output"]
assert checks["SwiftLint receives staged() output"]
assert not checks["index-content read mechanism"]
PY

printf '%s\n' '--- current staged Swift paths (NUL-safe) ---'
git diff --cached --name-only --diff-filter=ACMR -z -- '*.swift' |
  python3 -c 'import sys; print(repr(sys.stdin.buffer.read()))'

printf '%s\n' '--- staged versus working-tree entries, if any ---'
git status --short
git diff --cached --name-status --diff-filter=ACMR -- '*.swift' || true
git diff --name-status -- '*.swift' || true

#!/bin/bash
set -eu

python3 - <<'PY'
from pathlib import Path
import re

text = Path(".githooks/pre-commit").read_text()
for label, pattern in {
    "staged names only": r"git diff --cached --name-only -z",
    "ShellCheck gets working-tree paths": r"staged '\*\.sh'.*\|\s*xargs -0 shellcheck",
    "SwiftFormat gets working-tree paths": r"staged '\*\.swift'.*\|\s*xargs -0 swiftformat",
    "SwiftLint gets working-tree paths": r"staged '\*\.swift'.*\|\s*xargs -0 swiftlint",
}.items():
    print(f"{label}: {bool(re.search(pattern, text))}")

print("index-content handling:",
      any(token in text for token in
          ("git show :", "git cat-file", "git checkout-index", "mktemp")))
PY

git status --short
git diff --cached --name-status --diff-filter=ACMR -- '*.swift' || true
git diff --name-status -- '*.swift' || true

#!/bin/bash
set -euo pipefail

rg -n -C 6 'shellcheck|swiftformat|swiftlint|\.sh|\.swift' .

#!/bin/bash
set -euo pipefail

printf '%s\n' '--- tracked workflow and hook files ---'
git ls-files '.github/**' '.githooks/**' '*workflow*' '*ci*' | sed -n '1,160p'

printf '%s\n' '--- workflow and hook references ---'
find .github .githooks -type f -maxdepth 3 -print 2>/dev/null | while IFS= read -r file; do
  printf '\n### %s\n' "$file"
  rg -n -C 5 'shellcheck|swiftformat|swiftlint|staged|find |git diff|\.sh|\.swift|paths:|paths-ignore:' "$file" || true
done

#!/bin/bash
set -euo pipefail

printf '%s\n' '--- repository configuration files ---'
git ls-files -z | python3 - <<'PY'
import sys
paths = sys.stdin.buffer.read().split(b'\0')
for raw in paths:
    path = raw.decode()
    name = path.rsplit('/', 1)[-1]
    if name in {'.swiftformat', '.swiftlint.yml', '.swiftlint.yaml', '.gitignore'} or 'workflow' in path:
        print(path)
PY

printf '%s\n' '--- relevant configuration contents ---'
for file in .swiftformat .swiftlint.yml .swiftlint.yaml .gitignore; do
  if [ -f "$file" ]; then
    printf '\n### %s\n' "$file"
    cat -n "$file"
  fi
done

printf '%s\n' '--- tracked files relevant to selector boundaries ---'
git ls-files -z | python3 - <<'PY'
import os, stat, sys
paths = [p.decode() for p in sys.stdin.buffer.read().split(b'\0') if p]
for path in paths:
    base = path.rsplit('/', 1)[-1]
    if base.endswith(('.sh', '.bash', '.zsh', '.ksh', '.swift')) or '.' not in base:
        mode = os.stat(path).st_mode
        executable = bool(mode & stat.S_IXUSR)
        print(f"{path}\t{'executable' if executable else 'non-executable'}")
PY

printf '%s\n' '--- workflow files with complete lint sections ---'
cat -n .github/workflows/lint.yml

#!/bin/bash
set -euo pipefail

git ls-files -z > /tmp/xctesthtmlreport-files

printf '%s\n' '--- repository configuration files ---'
python3 - <<'PY'
paths = [p.decode() for p in open('/tmp/xctesthtmlreport-files', 'rb').read().split(b'\0') if p]
for path in paths:
    name = path.rsplit('/', 1)[-1]
    if name in {'.swiftformat', '.swiftlint.yml', '.swiftlint.yaml', '.gitignore'} or 'workflow' in path:
        print(path)
PY

printf '%s\n' '--- relevant configuration contents ---'
for file in .swiftformat .swiftlint.yml .swiftlint.yaml .gitignore; do
  if [ -f "$file" ]; then
    printf '\n### %s\n' "$file"
    cat -n "$file"
  fi
done

printf '%s\n' '--- tracked files relevant to selector boundaries ---'
python3 - <<'PY'
import os, stat
paths = [p.decode() for p in open('/tmp/xctesthtmlreport-files', 'rb').read().split(b'\0') if p]
for path in paths:
    base = path.rsplit('/', 1)[-1]
    if base.endswith(('.sh', '.bash', '.zsh', '.ksh', '.swift')) or '.' not in base:
        mode = os.stat(path).st_mode
        executable = bool(mode & stat.S_IXUSR)
        print(f"{path}\t{'executable' if executable else 'non-executable'}")
PY

printf '%s\n' '--- lint workflow ---'
cat -n .github/workflows/lint.yml

#!/bin/bash
set -euo pipefail

command -v swiftlint >/dev/null
swiftlint lint --help | grep -q -- '--use-script-input-files'

tmp="$(mktemp -d)"
trap 'rm -rf "$tmp"' EXIT
printf 'struct One { let value = 1 }\n' > "$tmp/One.swift"
printf 'struct Two { let value = 2 }\n' > "$tmp/Two.swift"

(
  cd "$tmp"
  swiftlint lint --quiet "$tmp/One.swift" "$tmp/Two.swift"
)

#!/bin/bash
set -u

printf '%s\n' '--- hook and configuration files ---'
git ls-files '.githooks/pre-commit' '.swiftlint.yml' '.swiftlint.yaml' 'Package.swift' 'README.md'
printf '%s\n' '--- pre-commit context ---'
sed -n '1,90p' .githooks/pre-commit
printf '%s\n' '--- SwiftLint references ---'
rg -n --hidden --glob '!/.git/**' 'swiftlint|SCRIPT_INPUT_FILE|force-exclude|staged' .
printf '%s\n' '--- SwiftLint version and supported flags ---'
swiftlint version 2>&1 || true
swiftlint lint --help 2>&1 | grep -E -- '--use-script-input-files|--force-exclude|Usage|CONFIGURATION' || true
printf '%s\n' '--- SwiftLint configuration ---'
for f in .swiftlint.yml .swiftlint.yaml; do
  if [ -f "$f" ]; then
    cat -n "$f"
  fi
done

#!/bin/bash
set -u

printf '%s\n' '--- full lint usage and relevant options ---'
swiftlint lint --help 2>&1 | sed -n '1,220p'

printf '%s\n' '--- controlled positional-path test ---'
tmp="$(mktemp -d)"
trap 'rm -rf "$tmp"' EXIT
mkdir -p "$tmp/Sources"
printf 'struct One { let value = 1 }\n' > "$tmp/Sources/One.swift"
printf 'struct Two { let value = 2 }\n' > "$tmp/Sources/Two.swift"
cat > "$tmp/.swiftlint.yml" <<'YAML'
included:
  - Sources
disabled_rules:
  - statement_position
YAML

(
  cd "$tmp"
  swiftlint lint --no-cache --quiet "$tmp/Sources/One.swift" "$tmp/Sources/Two.swift" 2>&1
)
positional_status=$?
printf 'positional_status=%s\n' "$positional_status"

printf '%s\n' '--- controlled script-input-file test ---'
(
  cd "$tmp"
  export SCRIPT_INPUT_FILE_COUNT=2
  export SCRIPT_INPUT_FILE_0="$tmp/Sources/One.swift"
  export SCRIPT_INPUT_FILE_1="$tmp/Sources/Two.swift"
  swiftlint lint --no-cache --quiet --use-script-input-files 2>&1
)
script_input_status=$?
printf 'script_input_status=%s\n' "$script_input_status"

printf '%s\n' '--- force-exclude behavior with an explicitly passed excluded file ---'
printf 'struct Excluded { let value = 1 }\n' > "$tmp/Sources/Excluded.swift"
cat > "$tmp/.swiftlint.yml" <<'YAML'
included:
  - Sources
excluded:
  - Sources/Excluded.swift
line_length:
  error: 1
disabled_rules:
  - statement_position
YAML

(
  cd "$tmp"
  swiftlint lint --no-cache --quiet "$tmp/Sources/Excluded.swift" 2>&1
)
without_force_exclude_status=$?
printf 'without_force_exclude_status=%s\n' "$without_force_exclude_status"

(
  cd "$tmp"
  swiftlint lint --no-cache --quiet --force-exclude "$tmp/Sources/Excluded.swift" 2>&1
)
with_force_exclude_status=$?
printf 'with_force_exclude_status=%s\n' "$with_force_exclude_status"

#!/bin/bash
set -u

tmp="$(mktemp -d)"
trap 'rm -rf "$tmp"' EXIT
mkdir -p "$tmp/Sources"
printf 'struct Included { let value = 1 }\n' > "$tmp/Sources/Included.swift"
printf 'struct Excluded { let value = 1 }\n' > "$tmp/Sources/Excluded.swift"
cat > "$tmp/.swiftlint.yml" <<'YAML'
included:
  - Sources
excluded:
  - Sources/Excluded.swift
line_length:
  error: 1
disabled_rules:
  - statement_position
YAML

printf '%s\n' '--- mixed explicit paths with --force-exclude ---'
(
  cd "$tmp"
  swiftlint lint --no-cache --quiet --force-exclude \
    "$tmp/Sources/Included.swift" "$tmp/Sources/Excluded.swift" 2>&1
)
printf 'mixed_status=%s\n' "$?"

printf '%s\n' '--- only explicitly excluded path with --force-exclude ---'
(
  cd "$tmp"
  swiftlint lint --no-cache --quiet --force-exclude \
    "$tmp/Sources/Excluded.swift" 2>&1
)
printf 'excluded_only_status=%s\n' "$?"