* ci: modernize tooling with prek, ty, and security scanning
Migrate from pre-commit to prek (Rust-native, faster hooks) and add
comprehensive CI improvements for code quality and security.
Changes:
- Replace pre-commit with prek for git hooks
- Add ty type checker (Rust-based, replaces mypy)
- Expand ruff rules: security (S), simplify (SIM), commented code (ERA)
- Add pip-audit workflow for Python dependency CVE scanning
- Add actionlint and zizmor for GitHub Actions linting/security
- Add ruff format check to CI
- Enable stricter ansible-lint rules (no-changed-when, risky-file-permissions)
- Remove obsolete Claude workflow files
- Apply ruff formatting fixes to test files
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
* fix(ci): resolve actionlint install and ty type errors
- Use actionlint's official install script instead of broken URL pattern
- Exclude test mock modules from ty type checking
- Run workflows on push only for main/master to avoid duplicate PR runs
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
* fix(ci): use glob pattern for actionlint, exclude all tests from ty
- actionlint requires *.yml glob, not directory path
- Exclude all tests from ty type checking (test code has looser typing)
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
* fix(ci): quote shell variables to fix shellcheck warnings
Fix SC2046/SC2086 warnings in workflow scripts:
- Quote $(uname -r) in apt-get install
- Quote $(pwd) in docker volume mount
- Quote $existing in gh issue comment
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
* fix(ci): move key-order[task] to warn_list
Too many existing violations in the codebase to enable as error.
Move to warn_list for gradual fixes over time.
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---------
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
* Fix Ansible 12.0.0 boolean type checking issue
Ansible 12.0.0 enforces strict type checking for conditionals and no longer
automatically converts string values "true"/"false" to booleans. This was
causing deployment failures after the recent Ansible version bump.
Changes:
- Fix ipv6_support to use 'is defined' which returns boolean instead of string
- Fix algo_* variables in input.yml to use {{ false }} instead of string "false"
- Add comprehensive tests to prevent regression
- Add mutation testing to verify tests catch the issue
The fix uses native boolean expressions instead of string literals, ensuring
compatibility with Ansible 12's strict type checking while maintaining backward
compatibility.
Fixes#14833🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
* Fix Python linting issues in test files
- Fix import sorting and remove unused imports
- Remove trailing whitespace and blank lines with whitespace
- Use underscore prefix for unused loop variables
- Remove unnecessary file open mode arguments
- Add newlines at end of files
- Remove unused variable assignments
All ruff checks now pass.
---------
Co-authored-by: Claude <noreply@anthropic.com>
