Skip to content

chore: remove obsolete hookify rules - #117

Merged
hyldmo merged 1 commit into
mainfrom
hyldmo/remove-dead-hookify-rules
Sep 1, 2026
Merged

hyldmo merged 1 commit into
mainfrom
hyldmo/remove-dead-hookify-rules

Conversation

@hyldmo

@hyldmo hyldmo commented Sep 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • remove the dead repo-local Hookify rules and their documentation
  • remove the redundant save-learnings reminder hook while preserving the Stop hook

These rules now live in the operator's global hook ~/.claude/hooks/bash-command-guard.mts, which blocks rm with a force flag, npx, and gcloud, and warns before git commit.

The old no-git-c-flag rule was dropped on purpose. The new hook matches the command in command position and checks flags across all arguments, so rm foo.bar -rf is caught while echo "rm -rf x" is allowed. Its npx message names both yarn and bun, so it stays correct in this repo.

Validation

  • grep -ri hookify . returns no matches
  • .claude/settings.json parses as valid JSON

@hyldmo
hyldmo merged commit 08e92b1 into main Sep 1, 2026
2 checks passed
@hyldmo
hyldmo deleted the hyldmo/remove-dead-hookify-rules branch September 1, 2026 16:02
@hyldmo
hyldmo restored the hyldmo/remove-dead-hookify-rules branch September 1, 2026 16:02
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 2.19.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant