-
Notifications
You must be signed in to change notification settings - Fork 12
Closed
Labels
safe-to-workSecurity triage: safe for automated processingSecurity triage: safe for automated processing
Description
Summary
PR #2339 introduced automated version management and commit message validation but has become stale with merge conflicts. This issue tracks reimplementing the work on top of current main.
Original Changes
- commitlint for conventional commit message validation
- Husky git hooks for pre-commit checks
- release-please for automated version bumping and changelog generation
- Workflow changes to trigger CLI releases on GitHub release events
Security Review Status
All security concerns from #2339 have been resolved:
- Command injection risks properly mitigated using
printf '%s\n'with file redirection - GitHub App tokens properly scoped
- Shell variables properly quoted
- No credential leaks or unsafe code execution
Implementation Notes
- Create worktree from current main
- Cherry-pick changes from ci: add commitlint, Husky, and release-please #2339 or reimplement fresh
- Resolve merge conflicts
- Run full test suite and lint checks
- Create new PR
References
- Original PR: ci: add commitlint, Husky, and release-please #2339
- Last HEAD commit: a55e8ec
- Multiple security reviews conducted (all passed)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
safe-to-workSecurity triage: safe for automated processingSecurity triage: safe for automated processing