fix: deduplicate CI test run for Python 3.12, add Makefile lefthook/ci-deps recipes

- Fix ci.yml: Python 3.12 no longer runs pytest twice (plain + coverage)
- Add `make ci-deps` to check/install uv, lefthook, actionlint
- Add `make lefthook-install` to activate pre-commit hooks
- Add `make lefthook-run` to manually run all pre-commit checks
- Add PROJECTS.md for project tracking

Author: smorin

.github/workflows/ci.yml

@@ -28,11 +28,12 @@ jobs:
         run: uv sync --group dev
 
       - name: Run tests
+        if: matrix.python-version != '3.12'
         run: uv run pytest tests/ -v --tb=short
 
       - name: Run tests with coverage
         if: matrix.python-version == '3.12'
-        run: uv run pytest tests/ --cov=worktreeflow --cov-report=term-missing --cov-fail-under=35
+        run: uv run pytest tests/ --cov=worktreeflow --cov-report=term-missing --cov-fail-under=35 -v --tb=short
 
   lint:
     runs-on: ubuntu-latest

Makefile

@@ -1,4 +1,4 @@
-.PHONY: test lint format typecheck coverage build clean help completions-bash completions-zsh completions-fish version bump-patch bump-minor bump-major release
+.PHONY: test lint format typecheck coverage build clean help completions-bash completions-zsh completions-fish version bump-patch bump-minor bump-major release ci-deps lefthook-install lefthook-run
 
 help: ## Show this help message
 	@grep -E '^[a-zA-Z_-]+:.*?## .*$$' $(MAKEFILE_LIST) | sort | awk 'BEGIN {FS = ":.*?## "}; {printf "\033[36m%-15s\033[0m %s\n", $$1, $$2}'
@@ -58,3 +58,20 @@ completions-zsh: ## Generate zsh completions
 
 completions-fish: ## Generate fish completions
 	_WTF_COMPLETE=fish_source uv run wtf
+
+ci-deps: ## Install CI/dev dependencies (lefthook, actionlint)
+	@echo "Checking CI dependencies..."
+	@command -v uv >/dev/null 2>&1 && echo "  uv: installed" || { echo "  uv: installing..."; curl -LsSf https://astral.sh/uv/install.sh | sh; }
+	@command -v lefthook >/dev/null 2>&1 && echo "  lefthook: installed" || { echo "  lefthook: installing..."; brew install lefthook; }
+	@command -v actionlint >/dev/null 2>&1 && echo "  actionlint: installed" || { echo "  actionlint: installing..."; brew install actionlint; }
+	uv sync --group dev
+	@echo "All CI dependencies ready."
+
+lefthook-install: ## Install and activate lefthook pre-commit hooks
+	@command -v lefthook >/dev/null 2>&1 || { echo "lefthook not found. Run 'make ci-deps' first."; exit 1; }
+	lefthook install
+	@echo "Lefthook pre-commit hooks activated."
+
+lefthook-run: ## Manually run all lefthook pre-commit checks
+	@command -v lefthook >/dev/null 2>&1 || { echo "lefthook not found. Run 'make ci-deps' first."; exit 1; }
+	lefthook run pre-commit

PROJECTS.md

@@ -0,0 +1,47 @@
+# Projects
+
+**Legend:**
+- `[x]` Completed
+- `[-]` In Progress
+- `[ ]` Not Started
+- `[~]` Won't fix / Invalid / False positive
+
+---
+
+## [-] Project P16: Fix GitHub Actions Bugs + Makefile CI/Lefthook Recipes (v0.3.4)
+**Goal**: Fix duplicate test execution bug in CI workflow, add Makefile recipes for CI dependency management, lefthook install, and manual lefthook runs
+
+**Out of Scope**
+- Raising the coverage threshold (currently 35%)
+- Adding `permissions:` block to ci.yml (not a bug, just best practice)
+
+### Tests & Tasks
+- [x] [P16-T01] Fix duplicate pytest run for Python 3.12 in ci.yml
+      Added `if: matrix.python-version != '3.12'` to plain test step so 3.12 only runs the coverage variant
+- [x] [P16-T02] Add `-v --tb=short` flags to coverage test step for consistent verbose output
+      Ensures 3.12 coverage run has same verbosity as other versions
+- [x] [P16-T03] Add `make ci-deps` recipe — checks if uv/lefthook/actionlint are installed before installing, then runs `uv sync --group dev`
+- [x] [P16-T04] Add `make lefthook-install` recipe — activates pre-commit hooks (errors if lefthook not found)
+- [x] [P16-T05] Add `make lefthook-run` recipe — manually runs all pre-commit checks (errors if lefthook not found)
+- [x] [P16-TS01] Validate ci.yml with actionlint — passed with no errors
+- [x] [P16-TS02] Run full test suite — 159 tests passed
+- [x] [P16-TS03] Verify `make help` shows all new targets
+- [x] [P16-TS04] Verify `make ci-deps` detects already-installed tools
+- [x] [P16-TS05] Verify `make lefthook-install` activates hooks
+- [x] [P16-TS06] Verify `make lefthook-run` executes pre-commit checks
+- [ ] [P16-TS07] Regression — push to branch and confirm CI passes on all 3 Python versions
+
+### Deliverable
+- `.github/workflows/ci.yml` — Python 3.12 no longer runs the test suite twice
+- `Makefile` — 3 new recipes: `ci-deps`, `lefthook-install`, `lefthook-run`
+
+### Automated Verification
+- `actionlint .github/workflows/ci.yml` passes
+- `make test` — 159 tests pass
+- `make ci-deps` — detects installed tools, installs missing ones
+- `make lefthook-install` — activates hooks
+- `make lefthook-run` — runs pre-commit checks
+
+### Manual Verification
+- Push to a branch and confirm CI passes on all 3 Python versions (3.11, 3.12, 3.13)
+- Confirm 3.12 job only shows one pytest execution (with coverage)