Add matrix testing for both hook installation methods

[Copilot]

Summary

This PR sets up a matrix strategy in the GitHub Actions workflow to test hook installation using both scripts/install-hooks.py and scripts/install-hooks.sh.

Changes

.github/workflows/test-hooks.yml

• Added install-method to the matrix with values ['install-hooks.py', 'install-hooks.sh']
• Updated job name to include the installation method
• Added INSTALL_HOOKS_METHOD environment variable to the test step
• Added explicit permissions block (contents: read) for security
• Excluded Windows + install-hooks.sh combination from matrix
• This creates 5 test jobs:
  • Ubuntu: install-hooks.py, install-hooks.sh
  • macOS: install-hooks.py, install-hooks.sh
  • Windows: install-hooks.py only

scripts/install-hooks.py

• Added automatic script directory detection using os.path.dirname(os.path.abspath(__file__))
• Updated hook file reading to use absolute paths from the script directory
• Now works like install-hooks.sh - can be run from any directory
• Fixed typo: "occured" -> "occurred"
• Does NOT set executable permissions (delegated to post-installation in tests)

test_hooks.py

• Modified _install_hooks() to read the INSTALL_HOOKS_METHOD environment variable
• Added logic to install hooks using either install-hooks.py or install-hooks.sh
• Set install-hooks.py as the default installation method
• Removed _manual_install_hooks() fallback method
• Improved error handling: raises errors for unknown installation methods or missing bash on Windows
• Added post-installation chmod for install-hooks.py to set executable permissions on Unix
• Added proper Windows bash detection for install-hooks.sh

Testing

All tests pass with both installation methods on Linux:

• ✅ install-hooks.py (default)
• ✅ install-hooks.sh

The matrix ensures both scripts are tested on Ubuntu and macOS, while Windows only tests install-hooks.py.

Original prompt

Set up matrix in the tests to install hooks using either scripts/install-hooks.py or scripts/install-hooks.sh.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[doitian]

@copilot Do not test install-hooks.sh in Windows.

[Copilot]

@copilot Do not test install-hooks.sh in Windows.

Excluded Windows + install-hooks.sh from the test matrix. The workflow now runs 5 jobs instead of 6. (4bb1258)