fix: address all code review issues

- Remove accidentally included test-pyright directory
- Fix pre-commit deprecation: use stages: [pre-commit] instead of [commit]
- Change pyright to strict mode for better type safety
- Add pyright include paths: ["src", "test"]
- Add pytest marker documentation for @pytest.mark.slow
- Simplify Makefile: keep 'fix' as primary command, 'reformat' as alias

All changes tested locally - lint, fix, reformat, and pre-commit hooks
work correctly with strict pyright configuration.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: dguido

test-final/python-project/README.md

@@ -36,8 +36,8 @@ repos:
         types: [python]
         pass_filenames: false
         require_serial: true
-        # Only run on commit, not on push
-        stages: [commit]
+        # Only run on pre-commit, not on push
+        stages: [pre-commit]
 
 # Configuration
 ci:

{{cookiecutter.project_slug}}/.pre-commit-config.yaml

@@ -123,6 +123,25 @@ parser.add_argument(
 - Mock external dependencies
 - Test both success and error paths
 
+### Test Markers
+
+Mark slow tests to exclude them from pre-commit hooks:
+
+```python
+import pytest
+
+@pytest.mark.slow
+def test_integration_with_external_api():
+    """This test won't run during pre-commit hooks."""
+    ...
+
+def test_fast_unit_test():
+    """This test will run during pre-commit hooks."""
+    ...
+```
+
+The pre-commit hook runs `pytest -k "not slow"` to skip slow tests.
+
 ## CI/CD
 
 GitHub Actions run on every push/PR:

{{cookiecutter.project_slug}}/CLAUDE.md

@@ -73,11 +73,15 @@ check: lint test
 typecheck: $(VENV)/pyvenv.cfg
 	uv run pyright
 
-.PHONY: fix format reformat
-fix format reformat:
+.PHONY: fix
+fix:
 	uv run ruff format && \
 		uv run ruff check --fix
 
+# Alias for backwards compatibility
+.PHONY: reformat
+reformat: fix
+
 .PHONY: test tests
 test tests: $(VENV)/pyvenv.cfg
 	uv run pytest --cov=$(PY_IMPORT) $(T) $(TEST_ARGS)

{{cookiecutter.project_slug}}/Makefile

@@ -61,8 +61,9 @@ omit = ["{{ cookiecutter.__project_src_path }}/_cli.py"]
 
 [tool.pyright]
 # Type checking configuration
+include = ["src", "test"]
 pythonVersion = "3.11"
-typeCheckingMode = "standard"
+typeCheckingMode = "strict"
 useLibraryCodeForTypes = true
 reportMissingTypeStubs = false