blockmine 1.21.0 → 1.23.0

package/.claude/hooks/error-handling-reminder.sh

@@ -0,0 +1,12 @@
+#!/bin/bash
+
+# Skip if environment variable is set
+if [ -n "$SKIP_ERROR_REMINDER" ]; then
+    exit 0
+fi
+
+# Get the directory of this script
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+cd "$SCRIPT_DIR"
+
+cat | npx tsx error-handling-reminder.ts

package/.claude/hooks/error-handling-reminder.ts

@@ -0,0 +1,222 @@
+#!/usr/bin/env node
+import { readFileSync, existsSync } from 'fs';
+import { join } from 'path';
+
+interface HookInput {
+    session_id: string;
+    transcript_path: string;
+    cwd: string;
+    permission_mode: string;
+    hook_event_name: string;
+}
+
+interface EditedFile {
+    path: string;
+    tool: string;
+    timestamp: string;
+}
+
+interface SessionTracking {
+    edited_files: EditedFile[];
+}
+
+function getFileCategory(filePath: string): 'backend' | 'frontend' | 'database' | 'other' {
+    // Frontend detection
+    if (filePath.includes('/frontend/') ||
+        filePath.includes('/client/') ||
+        filePath.includes('/src/components/') ||
+        filePath.includes('/src/features/')) return 'frontend';
+
+    // Backend detection (common service directories)
+    if (filePath.includes('/src/controllers/') ||
+        filePath.includes('/src/services/') ||
+        filePath.includes('/src/routes/') ||
+        filePath.includes('/src/api/') ||
+        filePath.includes('/server/')) return 'backend';
+
+    // Database detection
+    if (filePath.includes('/database/') ||
+        filePath.includes('/prisma/') ||
+        filePath.includes('/migrations/')) return 'database';
+
+    return 'other';
+}
+
+function shouldCheckErrorHandling(filePath: string): boolean {
+    // Skip test files, config files, and type definitions
+    if (filePath.match(/\.(test|spec)\.(ts|tsx)$/)) return false;
+    if (filePath.match(/\.(config|d)\.(ts|tsx)$/)) return false;
+    if (filePath.includes('types/')) return false;
+    if (filePath.includes('.styles.ts')) return false;
+
+    // Check for code files
+    return filePath.match(/\.(ts|tsx|js|jsx)$/) !== null;
+}
+
+function analyzeFileContent(filePath: string): {
+    hasTryCatch: boolean;
+    hasAsync: boolean;
+    hasPrisma: boolean;
+    hasController: boolean;
+    hasApiCall: boolean;
+} {
+    if (!existsSync(filePath)) {
+        return { hasTryCatch: false, hasAsync: false, hasPrisma: false, hasController: false, hasApiCall: false };
+    }
+
+    const content = readFileSync(filePath, 'utf-8');
+
+    return {
+        hasTryCatch: /try\s*\{/.test(content),
+        hasAsync: /async\s+/.test(content),
+        hasPrisma: /prisma\.|PrismaService|findMany|findUnique|create\(|update\(|delete\(/i.test(content),
+        hasController: /export class.*Controller|router\.|app\.(get|post|put|delete|patch)/.test(content),
+        hasApiCall: /fetch\(|axios\.|apiClient\./i.test(content),
+    };
+}
+
+async function main() {
+    try {
+        // Read input from stdin
+        const input = readFileSync(0, 'utf-8');
+        const data: HookInput = JSON.parse(input);
+
+        const { session_id } = data;
+        const projectDir = process.env.CLAUDE_PROJECT_DIR || process.cwd();
+
+        // Check for edited files tracking
+        const cacheDir = join(process.env.HOME || '/root', '.claude', 'tsc-cache', session_id);
+        const trackingFile = join(cacheDir, 'edited-files.log');
+
+        if (!existsSync(trackingFile)) {
+            // No files edited this session, no reminder needed
+            process.exit(0);
+        }
+
+        // Read tracking data
+        const trackingContent = readFileSync(trackingFile, 'utf-8');
+        const editedFiles = trackingContent
+            .trim()
+            .split('\n')
+            .filter(line => line.length > 0)
+            .map(line => {
+                const [timestamp, tool, path] = line.split('\t');
+                return { timestamp, tool, path };
+            });
+
+        if (editedFiles.length === 0) {
+            process.exit(0);
+        }
+
+        // Categorize files
+        const categories = {
+            backend: [] as string[],
+            frontend: [] as string[],
+            database: [] as string[],
+            other: [] as string[],
+        };
+
+        const analysisResults: Array<{
+            path: string;
+            category: string;
+            analysis: ReturnType<typeof analyzeFileContent>;
+        }> = [];
+
+        for (const file of editedFiles) {
+            if (!shouldCheckErrorHandling(file.path)) continue;
+
+            const category = getFileCategory(file.path);
+            categories[category].push(file.path);
+
+            const analysis = analyzeFileContent(file.path);
+            analysisResults.push({ path: file.path, category, analysis });
+        }
+
+        // Check if any code that needs error handling was written
+        const needsAttention = analysisResults.some(
+            ({ analysis }) =>
+                analysis.hasTryCatch ||
+                analysis.hasAsync ||
+                analysis.hasPrisma ||
+                analysis.hasController ||
+                analysis.hasApiCall
+        );
+
+        if (!needsAttention) {
+            // No risky code patterns detected, skip reminder
+            process.exit(0);
+        }
+
+        // Display reminder
+        console.log('\n━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━');
+        console.log('📋 ERROR HANDLING SELF-CHECK');
+        console.log('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n');
+
+        // Backend reminders
+        if (categories.backend.length > 0) {
+            const backendFiles = analysisResults.filter(f => f.category === 'backend');
+            const hasTryCatch = backendFiles.some(f => f.analysis.hasTryCatch);
+            const hasPrisma = backendFiles.some(f => f.analysis.hasPrisma);
+            const hasController = backendFiles.some(f => f.analysis.hasController);
+
+            console.log('⚠️  Backend Changes Detected');
+            console.log(`   ${categories.backend.length} file(s) edited\n`);
+
+            if (hasTryCatch) {
+                console.log('   ❓ Did you add Sentry.captureException() in catch blocks?');
+            }
+            if (hasPrisma) {
+                console.log('   ❓ Are Prisma operations wrapped in error handling?');
+            }
+            if (hasController) {
+                console.log('   ❓ Do controllers use BaseController.handleError()?');
+            }
+
+            console.log('\n   💡 Backend Best Practice:');
+            console.log('      - All errors should be captured to Sentry');
+            console.log('      - Use appropriate error helpers for context');
+            console.log('      - Controllers should extend BaseController\n');
+        }
+
+        // Frontend reminders
+        if (categories.frontend.length > 0) {
+            const frontendFiles = analysisResults.filter(f => f.category === 'frontend');
+            const hasApiCall = frontendFiles.some(f => f.analysis.hasApiCall);
+            const hasTryCatch = frontendFiles.some(f => f.analysis.hasTryCatch);
+
+            console.log('💡 Frontend Changes Detected');
+            console.log(`   ${categories.frontend.length} file(s) edited\n`);
+
+            if (hasApiCall) {
+                console.log('   ❓ Do API calls show user-friendly error messages?');
+            }
+            if (hasTryCatch) {
+                console.log('   ❓ Are errors displayed to the user?');
+            }
+
+            console.log('\n   💡 Frontend Best Practice:');
+            console.log('      - Use your notification system for user feedback');
+            console.log('      - Error boundaries for component errors');
+            console.log('      - Display user-friendly error messages\n');
+        }
+
+        // Database reminders
+        if (categories.database.length > 0) {
+            console.log('🗄️  Database Changes Detected');
+            console.log(`   ${categories.database.length} file(s) edited\n`);
+            console.log('   ❓ Did you verify column names against schema?');
+            console.log('   ❓ Are migrations tested?\n');
+        }
+
+        console.log('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━');
+        console.log('💡 TIP: Disable with SKIP_ERROR_REMINDER=1');
+        console.log('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n');
+
+        process.exit(0);
+    } catch (err) {
+        // Silently fail - this is just a reminder, not critical
+        process.exit(0);
+    }
+}
+
+main().catch(() => process.exit(0));

package/.claude/hooks/k8s-manifest-validator.sh

@@ -0,0 +1,56 @@
+#!/bin/bash
+#
+# Kubernetes Manifest Validation Hook
+# Validates Kubernetes YAML files
+#
+
+set -e
+
+echo "🔍 Validating Kubernetes manifests..."
+
+# Find all Kubernetes manifest files
+K8S_FILES=$(find . -name "*.yaml" -o -name "*.yml" | grep -E "(k8s/|kubernetes/|manifests/)" || true)
+
+if [ -z "$K8S_FILES" ]; then
+    echo "ℹ️  No Kubernetes manifests found"
+    exit 0
+fi
+
+# Validate with kubectl dry-run if available
+if command -v kubectl &> /dev/null; then
+    for file in $K8S_FILES; do
+        kubectl apply --dry-run=client -f "$file" > /dev/null 2>&1 || {
+            echo "❌ Invalid Kubernetes manifest: $file"
+            exit 1
+        }
+    done
+fi
+
+# Validate with kubeval if available
+if command -v kubeval &> /dev/null; then
+    kubeval $K8S_FILES || {
+        echo "❌ Kubeval validation failed"
+        exit 1
+    }
+fi
+
+# Check for common security issues
+for file in $K8S_FILES; do
+    # Check for privileged containers
+    if grep -q "privileged: true" "$file"; then
+        echo "⚠️  Warning: Privileged container in $file"
+    fi
+
+    # Check for host network
+    if grep -q "hostNetwork: true" "$file"; then
+        echo "⚠️  Warning: hostNetwork enabled in $file"
+    fi
+
+    # Check for latest tag
+    if grep -q "image:.*:latest" "$file"; then
+        echo "⚠️  Warning: 'latest' tag used in $file"
+    fi
+done
+
+echo "✅ Kubernetes manifest validation passed"
+exit 0