claude-git-hooks 2.66.1 → 2.67.3

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "claude-git-hooks",
-    "version": "2.66.1",
+    "version": "2.67.3",
     "description": "Git hooks with Claude CLI for code analysis and automatic commit messages",
     "type": "module",
     "bin": {
@@ -17,7 +17,6 @@
         "test:watch": "node --experimental-vm-modules node_modules/jest/bin/jest.js --watch",
         "test:coverage": "node --experimental-vm-modules node_modules/jest/bin/jest.js --coverage",
         "test:e2e": "bash test/manual/sdlc-stability-check.sh",
-        "test:full": "npm run test:all && npm run test:e2e",
         "lint": "eslint lib/ bin/claude-hooks .library/librarian/",
         "lint:fix": "eslint lib/ bin/claude-hooks .library/librarian/ --fix",
         "format": "prettier --write \"lib/**/*.js\" \"bin/**\" \"test/**/*.js\"",

package/templates/CLAUDE_ANALYSIS_PROMPT.md

@@ -30,7 +30,7 @@ OUTPUT_SCHEMA:
     "line":int,
     "method":"name",
     "message":"desc",
-    "rule":"optional"
+    "rule":"JBE-NNN|UIK-NNN|STR-NNN|...|empty"
   }],
   "blockingIssues":[{
     "description":"text",
@@ -53,5 +53,6 @@ RULES:
 - ratings:A(0issues),B(1-2minor),C(1major|3-5minor),D(2+major|1critical),E(1+blocker|2+critical)
 - IMPORTANT: @Autowired usage in Spring is NOT a BLOCKER/CRITICAL issue (max severity: MAJOR)
 - Spring dependency injection patterns (@Autowired) should NOT block commits
+- RULE CATALOGUE: if a `=== PLATFORM ANTIPATTERN CATALOGUE ===` section is present in this prompt, classify each finding against it. When a finding matches a catalogue entry, populate `details[].rule` with the rule ID exactly (e.g. "JBE-001", "UIK-002"). The catalogue's severity is the MINIMUM to assign — don't downgrade. Findings that don't match any catalogue entry leave `rule` empty (those become candidate skill-gaps for the team to review separately).
 
 ANALYZE_BELOW:

package/templates/config.advanced.example.json

@@ -33,6 +33,12 @@
             "model": "opus"
         },
 
+        "skillRegistry": {
+            "enabled": true,
+            "blockOn": "never",
+            "resumeOnCreatePr": true
+        },
+
         "prAnalysis": {
             "model": "sonnet",
             "timeout": 300000,
@@ -90,6 +96,25 @@
             "use_case": "Override the default sonnet model for judge passes"
         },
 
+        "skillRegistry.enabled": {
+            "description": "Enable/disable the mscope automation-skills integration: deterministic rule checks in pre-commit, antipattern catalogue injection into the analysis prompt, and the skill-gap writer. NOTE the cross-repo side effect: when the sibling automation-skills repo is found, unclassified AI findings are appended as [skill-gap] candidates to that repo's skill-feedback.md (reviewed via `automation-skills retro`; never auto-merged into the registry).",
+            "default": "true",
+            "use_case": "Set to false to fully disable the skill-registry integration (it auto-skips anyway when the skill repo isn't found)"
+        },
+
+        "skillRegistry.blockOn": {
+            "description": "Severity threshold at which deterministic skill-registry findings block the commit",
+            "default": "never",
+            "examples": ["never", "critical", "high", "medium", "low"],
+            "use_case": "Set to 'critical' or 'high' once the team is ready to enforce the mscope rule catalogue"
+        },
+
+        "skillRegistry.resumeOnCreatePr": {
+            "description": "Run `automation-skills resume` (interactive lessons-learned capture to the skill repo's implementation-history.md) during create-pr, before tags are pushed. Skipped automatically in headless mode or when the automation-skills CLI is not installed.",
+            "default": "true",
+            "use_case": "Set to false if your team captures lessons through another channel"
+        },
+
         "prAnalysis.model": {
             "description": "Claude model for PR analysis",
             "default": "sonnet",