claude-git-hooks 2.4.1 → 2.5.0

package/templates/presets/ai/PRE_COMMIT_GUIDELINES.md

@@ -3,12 +3,14 @@
 ## Claude API Best Practices
 
 ### Model Selection
+
 ✅ **Haiku**: Simple tasks, fast responses, cost-effective
 ✅ **Sonnet**: Balanced performance, most use cases
 ✅ **Opus**: Complex reasoning, highest quality
 ❌ Don't use Opus when Haiku would suffice
 
 ### API Usage
+
 ✅ Implement proper timeout handling
 ✅ Handle rate limiting gracefully
 ✅ Retry with exponential backoff on failures
@@ -17,6 +19,7 @@
 ✅ Calculate and monitor token usage
 
 ### Error Handling
+
 ```javascript
 // ✅ Good
 try {
@@ -39,6 +42,7 @@ try {
 ## Prompt Engineering
 
 ### Structure
+
 ✅ Clear role/context at the beginning
 ✅ Specific task instructions
 ✅ Well-defined output format (usually JSON)
@@ -46,6 +50,7 @@ try {
 ✅ Appropriate length (token-efficient)
 
 ### Quality Checklist
+
 ✅ Instructions are unambiguous
 ✅ Output format is machine-parseable
 ✅ Context is sufficient but not excessive
@@ -53,6 +58,7 @@ try {
 ✅ Placeholders are replaced correctly
 
 ### Common Prompt Issues
+
 ❌ Vague instructions
 ❌ No output format specification
 ❌ Too much unnecessary context
@@ -62,6 +68,7 @@ try {
 ## CLI User Experience
 
 ### Error Messages
+
 ✅ Clear, actionable error messages
 ✅ Suggest solutions when possible
 ✅ Use appropriate log levels
@@ -69,6 +76,7 @@ try {
 ✅ Include context (what was being attempted)
 
 ### User Feedback
+
 ✅ Show progress for long operations
 ✅ Confirm destructive operations
 ✅ Provide helpful usage information
@@ -78,6 +86,7 @@ try {
 ## Git Operations Safety
 
 ### Safe Practices
+
 ✅ Validate repository state before operations
 ✅ Use `--cached` for staged changes
 ✅ Handle special characters in filenames
@@ -85,6 +94,7 @@ try {
 ✅ Graceful handling of git errors
 
 ### Dangerous Operations
+
 ❌ Never run git commands that modify history without explicit user confirmation
 ❌ Avoid hard resets
 ❌ Be careful with force pushes
@@ -93,6 +103,7 @@ try {
 ## Security
 
 ### API Keys
+
 ✅ Load from environment variables
 ✅ Never log or display API keys
 ✅ Never commit API keys to repository
@@ -100,6 +111,7 @@ try {
 ✅ Clear keys from memory when done
 
 ### Command Injection
+
 ✅ Validate all user input
 ✅ Use parameterized commands when possible
 ✅ Escape special characters
@@ -107,14 +119,15 @@ try {
 ✅ Sanitize file paths
 
 ### Sensitive Data
+
 ✅ Don't send secrets to Claude API
 ✅ Filter sensitive data from diffs
 ✅ Be careful with error messages (don't expose internals)
-✅ Implement SKIP_ANALYSIS for sensitive code
 
 ## Code Organization
 
 ### File Structure
+
 ```
 lib/
   hooks/         # Git hook implementations
@@ -129,6 +142,7 @@ bin/            # CLI entry points
 ```
 
 ### Module Design
+
 ✅ Single responsibility principle
 ✅ Clear, descriptive function names
 ✅ Comprehensive error handling
@@ -138,12 +152,14 @@ bin/            # CLI entry points
 ## Common Issues to Avoid
 
 ### Critical Issues
+
 ❌ Exposed API keys or secrets
 ❌ Command injection vulnerabilities
 ❌ Destructive git operations without confirmation
 ❌ Unhandled promise rejections
 
 ### Major Issues
+
 ❌ Missing error handling
 ❌ Poor user experience (unclear errors)
 ❌ Cross-platform incompatibility
@@ -151,6 +167,7 @@ bin/            # CLI entry points
 ❌ Missing input validation
 
 ### Minor Issues
+
 ❌ Insufficient logging
 ❌ Unclear variable names
 ❌ Missing documentation

package/templates/presets/ai/preset.json

@@ -1,37 +1,37 @@
-{
-    "name": "ai",
-    "displayName": "AI/CLI (Node.js + Claude)",
-    "description": "Node.js CLI tools with Claude API integration",
-    "version": "1.0.0",
-
-    "techStack": [
-        "Node.js",
-        "ES Modules",
-        "Claude API",
-        "CLI tools",
-        "Git hooks",
-        "Bash scripting",
-        "Markdown templates"
-    ],
-
-    "fileExtensions": [".js", ".json", ".md", ".sh"],
-
-    "focusAreas": [
-        "Claude API usage and best practices",
-        "Prompt engineering quality",
-        "CLI user experience",
-        "Error handling and logging",
-        "Git operations safety",
-        "Cross-platform compatibility",
-        "Token usage optimization",
-        "Security (API keys, secrets)"
-    ],
-
-    "templates": {
-        "analysis": "ANALYSIS_PROMPT.md",
-        "guidelines": "PRE_COMMIT_GUIDELINES.md",
-        "commitMessage": "../shared/COMMIT_MESSAGE.md",
-        "analyzeDiff": "../shared/ANALYZE_DIFF.md",
-        "resolution": "../shared/RESOLUTION_PROMPT.md"
-    }
-}
+{
+    "name": "ai",
+    "displayName": "AI/CLI (Node.js + Claude)",
+    "description": "Node.js CLI tools with Claude API integration",
+    "version": "1.0.0",
+
+    "techStack": [
+        "Node.js",
+        "ES Modules",
+        "Claude API",
+        "CLI tools",
+        "Git hooks",
+        "Bash scripting",
+        "Markdown templates"
+    ],
+
+    "fileExtensions": [".js", ".json", ".md", ".sh"],
+
+    "focusAreas": [
+        "Claude API usage and best practices",
+        "Prompt engineering quality",
+        "CLI user experience",
+        "Error handling and logging",
+        "Git operations safety",
+        "Cross-platform compatibility",
+        "Token usage optimization",
+        "Security (API keys, secrets)"
+    ],
+
+    "templates": {
+        "analysis": "ANALYSIS_PROMPT.md",
+        "guidelines": "PRE_COMMIT_GUIDELINES.md",
+        "commitMessage": "../shared/COMMIT_MESSAGE.md",
+        "analyzeDiff": "../shared/ANALYZE_DIFF.md",
+        "resolution": "../shared/RESOLUTION_PROMPT.md"
+    }
+}

package/templates/settings.local.example.json

@@ -0,0 +1,4 @@
+{
+  "_comment": "This file stores local settings that should NOT be committed. Copy to .claude/settings.local.json",
+  "githubToken": "ghp_your_token_here"
+}