Skip to content

Fix hooks#46

Merged
alex-feel merged 4 commits intomainfrom
alex-feel-dev
Sep 3, 2025
Merged

Fix hooks#46
alex-feel merged 4 commits intomainfrom
alex-feel-dev

Conversation

@alex-feel
Copy link
Owner

No description provided.

Fixes hooks not triggering when using claude-python command on Windows by:
- Creating a shared POSIX script to avoid multi-level command escaping
- Simplifying launcher scripts to call the shared script directly

The root cause was complex bash command escaping when passing through PowerShell/CMD to Git Bash. The shared script approach eliminates this issue entirely.
…ections

- Move hook files list to top-level 'files' key to avoid duplication
- Separate event configurations under 'events' key for cleaner structure
- Remove backwards compatibility for old list format in setup-environment.py
- Update python.yaml to demonstrate new structure with ruff hook
- Update basic-template.yaml with commented examples of new format
- Show new YAML format with separate files and events sections
- Update chaining hooks example with new structure
- Demonstrate how multiple hooks are configured with shared files
@alex-feel alex-feel enabled auto-merge September 3, 2025 15:09
@alex-feel alex-feel disabled auto-merge September 3, 2025 15:10
@alex-feel alex-feel merged commit 7f1100e into main Sep 3, 2025
5 checks passed
@alex-feel alex-feel deleted the alex-feel-dev branch September 3, 2025 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments