fix: protected branch detection and non-blocking hooks #111

lmiranda · 2026-01-23T16:06:01Z

lmiranda commented

2026-01-23 16:06:01 +00:00

Summary

Add protected branch detection to /commit command before committing
Rewrite doc-guardian hook to be truly non-blocking (no analysis, just notification)
Enforce strict [plugin-name] prefix in all hook outputs

Issues Fixed

Fixes #109 - Protected branch workflow
Fixes #110 - Hook blocking and missing plugin prefixes

Changes

File	Change
`plugins/git-flow/commands/commit.md`	Added Step 1: Check for Protected Branch
`plugins/git-flow/README.md`	Updated command description and safety checks
`plugins/doc-guardian/hooks/hooks.json`	Rewritten to be non-blocking
`plugins/code-sentinel/hooks/hooks.json`	Strict prefix enforcement
`plugins/projman/hooks/hooks.json`	Strict prefix enforcement
`plugins/pr-review/hooks/hooks.json`	Strict prefix enforcement
`CHANGELOG.md`	Added v3.1.2 entries

Test Plan

Run /commit on development branch - should warn and offer feature branch
Edit file in commands/ - doc-guardian should show [doc-guardian] prefix, not block
Edit regular file - doc-guardian should be silent
All hook messages should show plugin prefix

🤖 Generated with Claude Code

## Summary - Add protected branch detection to `/commit` command before committing - Rewrite doc-guardian hook to be truly non-blocking (no analysis, just notification) - Enforce strict `[plugin-name]` prefix in all hook outputs ## Issues Fixed - Fixes #109 - Protected branch workflow - Fixes #110 - Hook blocking and missing plugin prefixes ## Changes | File | Change | |------|--------| | `plugins/git-flow/commands/commit.md` | Added Step 1: Check for Protected Branch | | `plugins/git-flow/README.md` | Updated command description and safety checks | | `plugins/doc-guardian/hooks/hooks.json` | Rewritten to be non-blocking | | `plugins/code-sentinel/hooks/hooks.json` | Strict prefix enforcement | | `plugins/projman/hooks/hooks.json` | Strict prefix enforcement | | `plugins/pr-review/hooks/hooks.json` | Strict prefix enforcement | | `CHANGELOG.md` | Added v3.1.2 entries | ## Test Plan - [ ] Run `/commit` on development branch - should warn and offer feature branch - [ ] Edit file in commands/ - doc-guardian should show `[doc-guardian]` prefix, not block - [ ] Edit regular file - doc-guardian should be silent - [ ] All hook messages should show plugin prefix 🤖 Generated with [Claude Code](https://claude.ai/code)

lmiranda added 1 commit 2026-01-23 16:06:02 +00:00

fix: protected branch detection and non-blocking hooks 2248184359

- Add protected branch detection to /commit command (Step 1)
- Warn users before committing to protected branches
- Offer to create feature branch automatically
- Rewrite doc-guardian hook to be truly non-blocking
- Enforce strict [plugin-name] prefix in all hook outputs
- Add forbidden words list to prevent accidental blocking

Fixes #109, #110

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

lmiranda force-pushed fix/issue-109-110-hooks-and-protected-branch from 2248184359 to 35cf20e02d

2026-01-23 16:08:07 +00:00

Compare

lmiranda merged commit cc4abf67b9 into development

2026-01-23 16:08:39 +00:00

lmiranda deleted branch fix/issue-109-110-hooks-and-protected-branch

2026-01-23 16:08:39 +00:00

lmiranda referenced this issue from a commit

2026-01-23 16:08:40 +00:00

Merge pull request 'fix: protected branch detection and non-blocking hooks' (#111) from fix/issue-109-110-hooks-and-protected-branch into development

lmiranda referenced this pull request

2026-01-23 16:31:18 +00:00