From 7e76b2da44b269e4ec58cae6c41c67778a086ee9 Mon Sep 17 00:00:00 2001 From: asepharyana Date: Sun, 26 Jul 2026 12:21:31 +0700 Subject: [PATCH] fix(hooks): only fire ok:false when clearly violating principles Previously returned ok:false every time because 'fully applied' all 8 skills is impossible in one response. Now ok:true by default (silent), only fires ok:false when Claude is clearly guessing or violating rules. --- hooks/hooks.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hooks/hooks.json b/hooks/hooks.json index dd36576..9695057 100644 --- a/hooks/hooks.json +++ b/hooks/hooks.json @@ -3,7 +3,7 @@ "Stop": [{ "hooks": [{ "type": "prompt", - "prompt": "Hub-guide best-practice skills are loaded. MANDATORY: engineering-principles, clean-code, clean-architecture, testing, error-handling, security, git-workflow, api-design. Never assume or guess — always find evidence in codebase or docs. If these skills are already being applied, respond: {\"ok\": true}. If not fully applied, respond: {\"ok\": false, \"reason\": \"[hub-guide] Apply MANDATORY skills: engineering-principles, clean-code, clean-architecture, testing, error-handling, security, git-workflow, api-design.\"}" + "prompt": "Check if hub-guide MANDATORY skills are being applied where relevant: engineering-principles, clean-code, clean-architecture, testing, error-handling, security, git-workflow, api-design. If the assistant is using evidence from codebase/docs instead of guessing, and is not violating any of these principles, respond: {\"ok\": true}. Only respond {\"ok\": false, \"reason\": \"[hub-guide] Apply MANDATORY skills: engineering-principles, clean-code, clean-architecture, testing, error-handling, security, git-workflow, api-design.\"} if the assistant is clearly guessing or violating these principles." }] }] }