ci: add pre-push test hook and fix dry_run file count bug

- Add local pre-commit hook to run pytest before push
- Fix writer to correctly report files_written=0 in dry_run mode
- Prevents CI test failures from incorrect file counting

Author: ryderstorm

.pre-commit-config.yaml

@@ -32,3 +32,13 @@ repos:
         stages: [commit-msg]
         args:
           - "--extends=@commitlint/config-conventional"
+
+  - repo: local
+    hooks:
+      - id: run-tests
+        name: Run pytest before push
+        entry: pytest
+        language: system
+        stages: [pre-push]
+        pass_filenames: false
+        always_run: true

slash_commands/writer.py

@@ -112,15 +112,19 @@ def generate(self) -> dict[str, Any]:
 
         # Generate files
         files = []
+        files_written = 0
         for prompt in prompts:
             for agent in agent_configs:
                 file_info = self._generate_file(prompt, agent)
                 if file_info:
                     files.append(file_info)
+                    # Only count files that were actually written (not dry run)
+                    if not self.dry_run:
+                        files_written += 1
 
         return {
             "prompts_loaded": len(prompts),
-            "files_written": len(files),
+            "files_written": files_written,
             "files": files,
             "prompts": [{"name": p.name, "path": str(p.path)} for p in prompts],
             "backups_created": self._backups_created,