Initial commit

skills/obsidian-vault-manager/scripts/validation/validate-frontmatter.py

@@ -0,0 +1,82 @@
+#!/usr/bin/env python3
+"""
+Validate frontmatter in markdown content.
+Can be run by Claude during workflow or configured as a hook in settings.json
+"""
+import sys
+import yaml
+
+def validate_content(content):
+    """Validate frontmatter in markdown content"""
+    if not content.startswith("---"):
+        return False, "No frontmatter detected"
+
+    try:
+        # Extract frontmatter
+        parts = content.split("---", 2)
+        if len(parts) < 3:
+            return False, "Malformed frontmatter (missing closing ---)"
+
+        frontmatter = yaml.safe_load(parts[1])
+
+        if not frontmatter:
+            return False, "Empty frontmatter"
+
+        # Required fields
+        required = ["title", "tags", "date", "type"]
+        missing = [f for f in required if f not in frontmatter]
+
+        if missing:
+            return False, f"Missing required fields: {', '.join(missing)}"
+
+        # Validate tags is a list
+        tags = frontmatter.get("tags", [])
+        if not isinstance(tags, list):
+            return False, "Tags must be a list"
+
+        if len(tags) == 0:
+            return False, "Tags list is empty"
+
+        # Check for content type tag
+        valid_types = ["video", "article", "book", "podcast", "idea", "study-guide", "repository", "reference", "project"]
+        has_type = any(tag in valid_types for tag in tags)
+        if not has_type:
+            return False, f"No content type tag found. Expected one of: {', '.join(valid_types)}"
+
+        # Check for inbox tag
+        if "inbox" not in tags:
+            return False, "Warning: 'inbox' tag recommended for new content"
+
+        return True, "✅ Validation passed"
+
+    except yaml.YAMLError as e:
+        return False, f"Invalid YAML syntax: {e}"
+    except Exception as e:
+        return False, f"Validation error: {e}"
+
+def main():
+    """Main entry point for validation script"""
+    if len(sys.argv) > 1:
+        # Read from file if provided
+        filename = sys.argv[1]
+        try:
+            with open(filename, 'r') as f:
+                content = f.read()
+        except FileNotFoundError:
+            print(f"❌ Error: File not found: {filename}", file=sys.stderr)
+            sys.exit(1)
+    else:
+        # Read from stdin
+        content = sys.stdin.read()
+
+    valid, message = validate_content(content)
+
+    if valid:
+        print(message)
+        sys.exit(0)
+    else:
+        print(f"❌ {message}", file=sys.stderr)
+        sys.exit(1)
+
+if __name__ == "__main__":
+    main()

skills/obsidian-vault-manager/scripts/validation/validate-mcp-tools.sh

@@ -0,0 +1,87 @@
+#!/bin/bash
+# Validation script to check for common MCP tool name mistakes
+
+set -e
+
+SKILL_DIR="$HOME/.claude/skills/obsidian-vault-manager"
+SKILL_FILE="$SKILL_DIR/SKILL.md"
+
+echo "🔍 Validating MCP tool names in SKILL.md..."
+echo ""
+
+ERRORS=0
+
+# Check for incorrect Obsidian tool references in MCP_DOCKER
+if grep -q "mcp__MCP_DOCKER__obsidian" "$SKILL_FILE"; then
+    echo "❌ ERROR: Found MCP_DOCKER__obsidian references!"
+    echo "   MCP_DOCKER does not have Obsidian tools."
+    echo "   Use: mcp__obsidian-mcp-tools__* instead"
+    echo ""
+    grep -n "mcp__MCP_DOCKER__obsidian" "$SKILL_FILE"
+    echo ""
+    ERRORS=$((ERRORS + 1))
+fi
+
+# Check for incorrect gitingest underscore naming
+if grep -q "mcp__MCP_DOCKER__gitingest_" "$SKILL_FILE"; then
+    echo "❌ ERROR: Found gitingest with underscore!"
+    echo "   GitIngest tools use HYPHENS, not underscores."
+    echo "   Use: mcp__MCP_DOCKER__gitingest-analyze (with hyphen)"
+    echo ""
+    grep -n "mcp__MCP_DOCKER__gitingest_" "$SKILL_FILE"
+    echo ""
+    ERRORS=$((ERRORS + 1))
+fi
+
+# Check for old non-Docker tool references that should be updated
+if grep -q "mcp__github__create_or_update_file" "$SKILL_FILE"; then
+    echo "⚠️  WARNING: Found old mcp__github__ reference"
+    echo "   Consider using: mcp__MCP_DOCKER__create_or_update_file"
+    echo ""
+    grep -n "mcp__github__create_or_update_file" "$SKILL_FILE"
+    echo ""
+fi
+
+if grep -q "mcp__fetch__fetch" "$SKILL_FILE"; then
+    echo "⚠️  WARNING: Found old mcp__fetch__ reference"
+    echo "   Consider using: mcp__MCP_DOCKER__fetch"
+    echo ""
+    grep -n "mcp__fetch__fetch" "$SKILL_FILE"
+    echo ""
+fi
+
+if grep -q "mcp__gitingest__gitingest-analyze" "$SKILL_FILE"; then
+    echo "⚠️  WARNING: Found old mcp__gitingest__ reference"
+    echo "   Consider using: mcp__MCP_DOCKER__gitingest-analyze"
+    echo ""
+    grep -n "mcp__gitingest__gitingest-analyze" "$SKILL_FILE"
+    echo ""
+fi
+
+# Verify correct allowed-tools configuration
+echo "📋 Checking allowed-tools section..."
+if grep -q "mcp__obsidian-mcp-tools__\*" "$SKILL_FILE"; then
+    echo "✅ Obsidian tools wildcard: CORRECT"
+else
+    echo "❌ ERROR: Missing mcp__obsidian-mcp-tools__* wildcard!"
+    ERRORS=$((ERRORS + 1))
+fi
+
+if grep -q "mcp__MCP_DOCKER__gitingest-analyze" "$SKILL_FILE"; then
+    echo "✅ GitIngest tool (with hyphen): CORRECT"
+else
+    echo "⚠️  WARNING: GitIngest tool not found in allowed-tools"
+fi
+
+echo ""
+echo "═══════════════════════════════════════"
+
+if [ $ERRORS -eq 0 ]; then
+    echo "✅ Validation PASSED! No errors found."
+    exit 0
+else
+    echo "❌ Validation FAILED! Found $ERRORS error(s)."
+    echo ""
+    echo "Please review MCP_ARCHITECTURE.md for correct naming"
+    exit 1
+fi