strray-ai 1.7.2 → 1.7.5

package/.opencode/hooks/post-commit

@@ -1,75 +1,172 @@
 #!/bin/bash
-# StrRay Post-Processor post-commit Hook
-# Automatically triggers post-processor after commit
-# Suggests reflection writing for significant changes
-
-set -e
+# StringRay Post-Processor post-commit Hook
+# Automatically triggers post-processor after post-commit
 
+# Get hook type from script name
 HOOK_NAME=$(basename "$0")
-COMMIT_SHA=$(git rev-parse HEAD)
-BRANCH=$(git rev-parse --abbrev-ref HEAD)
-AUTHOR=$(git log -1 --pretty=format:'%an <%ae>')
+COMMIT_SHA=""
+
+if [ "$HOOK_NAME" = "post-commit" ]; then
+  # Light monitoring for local commits - just basic validation
+  COMMIT_SHA=$(git rev-parse HEAD)
+  MONITORING_LEVEL="basic"
+elif [ "$HOOK_NAME" = "post-push" ]; then
+  # Full monitoring for pushes - comprehensive validation
+  # For push hooks, we need to parse the pushed refs from stdin
+  while read local_ref local_sha remote_ref remote_sha; do
+    if [ "$local_sha" != "0000000000000000000000000000000000000000" ]; then
+      COMMIT_SHA=$local_sha
+      break
+    fi
+  done
+  MONITORING_LEVEL="full"
+else
+  COMMIT_SHA=$(git rev-parse HEAD)
+  MONITORING_LEVEL="basic"
+fi
 
 if [ -z "$COMMIT_SHA" ]; then
+  echo "Warning: Could not determine commit SHA for post-processor"
   exit 0
 fi
 
-# Get changed files
-FILES=$(git diff --name-only HEAD~1 2>/dev/null || git diff --name-only --cached)
-
-# Get commit message for reflection decision
-COMMIT_MSG=$(git log -1 --pretty=format:'%s')
+# Get repository info
+REPO="strray-framework/stringray"  # Placeholder for now
+BRANCH=$(git rev-parse --abbrev-ref HEAD)
+AUTHOR=$(git log -1 --pretty=format:'%an <%ae>')
 
-# Check if this commit warrants reflection
-# (debugging, bug fixes, path issues, significant changes)
-REFLECTION_TRIGGERS="debug|fix|bug|path|mcp|consumer|installation|reflection"
-if echo "$COMMIT_MSG" | grep -qiE "$REFLECTION_TRIGGERS"; then
-  echo "📝 Reflection Suggestion: This commit appears to involve debugging, fixes, or significant changes."
-  echo "   Consider writing a reflection using the template at docs/reflections/TEMPLATE.md"
-  echo ""
+# Get changed files (different logic for commit vs push)
+if [ "$HOOK_NAME" = "post-commit" ]; then
+  FILES=$(git diff --name-only HEAD~1 2>/dev/null || git diff --name-only --cached)
+else
+  FILES=$(git log --name-only --oneline -1 $COMMIT_SHA | tail -n +2)
 fi
 
-# For consumer projects, always use the full node_modules path to avoid symlink issues
-# This ensures imports resolve correctly
-STRRAY_PATH=""
-for pkg in "strray-ai" "strray-framework"; do
-  if [ -d "node_modules/$pkg/dist" ]; then
-    STRRAY_PATH="node_modules/$pkg"
-    break
-  fi
-done
+# Trigger post-processor asynchronously (don't block git operations)
+(
+  cd "$(dirname "$0")/../.." # Navigate to project root
 
-if [ -z "$STRRAY_PATH" ]; then
-  # Fallback to local dist for development
-  if [ -d "dist" ]; then
-    STRRAY_PATH="."
-  else
-    exit 0
+  # Find the StringRay plugin in node_modules or current project (development)
+  STRRAY_PLUGIN=""
+  if [ -d "node_modules/strray-framework" ]; then
+    STRRAY_PLUGIN="node_modules/strray-framework"
+  elif [ -d "node_modules/@strray/strray-framework" ]; then
+    STRRAY_PLUGIN="node_modules/@strray/strray-framework"
+  elif [ -d "node_modules/OpenCode/plugins/strray-framework" ]; then
+    STRRAY_PLUGIN="node_modules/OpenCode/plugins/strray-framework"
+  elif [ -f "dist/postprocessor/PostProcessor.js" ]; then
+    # Development mode - use current project
+    STRRAY_PLUGIN="."
   fi
-fi
 
-# Run post-processor in background with Node.js
-node --input-type=module << EOF &>/dev/null || true
-import { PostProcessor } from './$STRRAY_PATH/dist/postprocessor/PostProcessor.js';
-import { StrRayStateManager } from './$STRRAY_PATH/dist/state/state-manager.js';
+  if command -v node >/dev/null 2>&1 && [ -n "$STRRAY_PLUGIN" ]; then
+    # Call a separate script to avoid bash variable issues
+    export COMMIT_SHA="$COMMIT_SHA"
+    export REPO="$REPO"
+    export BRANCH="$BRANCH"
+    export AUTHOR="$AUTHOR"
+    export STRRAY_PLUGIN="$STRRAY_PLUGIN"
+    export MONITORING_LEVEL="$MONITORING_LEVEL"
+    export IS_FULL_MONITORING="$([ "$MONITORING_LEVEL" = "full" ] && echo "true" || echo "false")"
+
+    # Run appropriate monitoring based on hook type
+    if [ "$HOOK_NAME" = "post-commit" ]; then
+      # LIGHT MONITORING: Quick validation, don't block git workflow
+      # Timeout: 2 seconds max, log metrics for monitoring
+      START_TIME=$(date +%s)
+      timeout 2 node -e "
+      (async () => {
+        try {
+          // Use import resolver to avoid hardcoded dist paths
+          const { importResolver } = await import('./utils/import-resolver.js');
+          const { LightweightValidator } = await importResolver.importModule('postprocessor/validation/LightweightValidator');
+
+          const validator = new LightweightValidator();
+          const result = await validator.validate();
+
+          if (result.warnings.length > 0) {
+            await frameworkLogger.log('-git-hook-trigger', '-result-warnings-length-warning-s-found-', 'info', { message: '⚠️ ' + result.warnings.length + ' warning(s) found:' });
+            result.warnings.forEach(w => await frameworkLogger.log('-git-hook-trigger', '-w-', 'info', { message: '   ' + w) });
+          }
+
+          if (!result.passed) {
+            await frameworkLogger.log('-git-hook-trigger', '-result-errors-length-error-s-found-', 'error', { message: '❌ ' + result.errors.length + ' error(s) found:' });
+            result.errors.forEach(e => await frameworkLogger.log('-git-hook-trigger', '-e-', 'info', { message: '   ' + e) });
+            process.exit(1);
+          }
 
-const stateManager = new StrRayStateManager();
-const postProcessor = new PostProcessor(stateManager, null, { 
-  reporting: { enabled: false } 
-});
+          await frameworkLogger.log('-git-hook-trigger', '-post-commit-validation-passed-in-result-duration-', 'success', { message: '✅ Post-commit: Validation passed in ' + result.duration + 'ms' });
+        } catch (error) {
+          console.error('❌ Post-commit validation failed:', error instanceof Error ? error.message : String(error));
+          process.exit(1);
+        }
+      })();
+      " 2>/dev/null
+      EXIT_CODE=$?
+      END_TIME=$(date +%s)
+      DURATION=$((END_TIME - START_TIME))
 
-const files = \`$FILES\`.split('\n').filter(f => f.endsWith('.ts') && !f.includes('.test.'));
+      # Log metrics for monitoring (convert to milliseconds)
+      DURATION_MS=$((DURATION * 1000))
+      # LOG CLEANUP: Remove old log files after validation
+      # Use relative path from CWD - works in both dev and consumer
+      node -e "
+      (async () => {
+        try {
+          // Use dynamic import that works in both dev and consumer
+          const basePath = process.env.STRRAY_BASE_PATH || '.';
+          const { cleanupLogFiles } = await import(basePath + '/dist/postprocessor/triggers/GitHookTrigger.js');
+          const result = await cleanupLogFiles({
+            maxAgeHours: 24,
+            excludePatterns: ['logs/framework/activity.log', 'logs/agents/refactoring-log.md', 'current-session.log'],
+            directories: ['logs/'],
+            enabled: true
+          });
+          if (result.cleaned > 0) {
+            await frameworkLogger.log('-git-hook-trigger', '-cleaned-result-cleaned-old-log-files-', 'info', { message: `🧹 Cleaned ${result.cleaned} old log files` });
+          }
+          if (result.errors.length > 0) {
+            console.error('Log cleanup errors:', result.errors);
+          }
+        } catch (error) {
+          console.error('Log cleanup failed:', error.message);
+        }
+      })();
+      "
 
-await postProcessor.executePostProcessorLoop({
-  commitSha: '$COMMIT_SHA',
-  repository: 'strray-framework/stringray',
-  branch: '$BRANCH',
-  author: '$AUTHOR',
-  files: files,
-  trigger: 'git-hook'
-});
+      echo "HOOK_METRICS: post-commit duration=${DURATION_MS}ms exit_code=${EXIT_CODE}" >&2
+      collector.recordMetrics('post-commit', ${DURATION_MS}, ${EXIT_CODE});
+      " 2>/dev/null
+      EXIT_CODE=$?
+      END_TIME=$(date +%s)
+      DURATION=$((END_TIME - START_TIME))
 
-console.log('Post-processor completed for', files.length, 'files');
-EOF
+      # Log comprehensive metrics for monitoring (convert to milliseconds)
+      DURATION_MS=$((DURATION * 1000))
+      echo "HOOK_METRICS: post-push duration=${DURATION_MS}ms exit_code=${EXIT_CODE}" >&2
+
+      # Record metrics using metrics collector (direct import for reliability)
+      # Use environment variable for base path - works in both dev and consumer
+      node -e "
+      (async () => {
+        try {
+          const basePath = process.env.STRRAY_BASE_PATH || '.';
+          const distPath = process.env.STRRAY_DIST_PATH || 'dist';
+          const { HookMetricsCollector } = await import(basePath + '/' + distPath + '/postprocessor/validation/HookMetricsCollector.js');
+          const collector = new HookMetricsCollector();
+          collector.recordMetrics('post-push', ${DURATION_MS}, ${EXIT_CODE});
+        } catch (error) {
+          // Silently fail if metrics collection fails
+        }
+      })();
+      " 2>/dev/null || true
+
+      [ $EXIT_CODE -eq 0 ] && exit 0 || exit 1
+    fi
+  else
+    echo "Warning: StringRay plugin not found or Node.js not available, skipping post-processor"
+  fi
+)
 
+# Don't wait for background process
 exit 0

package/.opencode/hooks/post-commit.backup

@@ -1,193 +1,129 @@
 #!/bin/bash
 # StrRay Post-Processor post-commit Hook
-# Automatically triggers post-processor after post-commit
+# Automatically triggers post-processor after commit
+# Suggests reflection writing for significant changes
+
+set -e
 
-# Get hook type from script name
 HOOK_NAME=$(basename "$0")
-COMMIT_SHA=""
-
-if [ "$HOOK_NAME" = "post-commit" ]; then
-  # Light monitoring for local commits - just basic validation
-  COMMIT_SHA=$(git rev-parse HEAD)
-  MONITORING_LEVEL="basic"
-elif [ "$HOOK_NAME" = "post-push" ]; then
-  # Full monitoring for pushes - comprehensive validation
-  # For push hooks, we need to parse the pushed refs from stdin
-  while read local_ref local_sha remote_ref remote_sha; do
-    if [ "$local_sha" != "0000000000000000000000000000000000000000" ]; then
-      COMMIT_SHA=$local_sha
-      break
-    fi
-  done
-  MONITORING_LEVEL="full"
-else
-  COMMIT_SHA=$(git rev-parse HEAD)
-  MONITORING_LEVEL="basic"
-fi
+COMMIT_SHA=$(git rev-parse HEAD)
+BRANCH=$(git rev-parse --abbrev-ref HEAD)
+AUTHOR=$(git log -1 --pretty=format:'%an <%ae>')
 
 if [ -z "$COMMIT_SHA" ]; then
-  echo "Warning: Could not determine commit SHA for post-processor"
   exit 0
 fi
 
-# Get repository info
-REPO="strray-framework/stringray"  # Placeholder for now
-BRANCH=$(git rev-parse --abbrev-ref HEAD)
-AUTHOR=$(git log -1 --pretty=format:'%an <%ae>')
+# Get changed files
+FILES=$(git diff --name-only HEAD~1 2>/dev/null || git diff --name-only --cached)
+
+# Get commit message for reflection decision
+COMMIT_MSG=$(git log -1 --pretty=format:'%s')
 
-# Get changed files (different logic for commit vs push)
-if [ "$HOOK_NAME" = "post-commit" ]; then
-  FILES=$(git diff --name-only HEAD~1 2>/dev/null || git diff --name-only --cached)
-else
-  FILES=$(git log --name-only --oneline -1 $COMMIT_SHA | tail -n +2)
+# Check if this commit warrants reflection
+# (debugging, bug fixes, path issues, significant changes)
+REFLECTION_TRIGGERS="debug|fix|bug|path|mcp|consumer|installation|reflection"
+if echo "$COMMIT_MSG" | grep -qiE "$REFLECTION_TRIGGERS"; then
+  echo "📝 Reflection Suggestion: This commit appears to involve debugging, fixes, or significant changes."
+  echo "   Consider writing a reflection using the template at docs/reflections/TEMPLATE.md"
+  echo ""
 fi
 
-# Trigger post-processor asynchronously (don't block git operations)
-(
-  cd "$(dirname "$0")/../.." # Navigate to project root
-
-  # Find the StrRay plugin in node_modules or current project (development)
-  STRRAY_PLUGIN=""
-  if [ -d "node_modules/strray-framework" ]; then
-    STRRAY_PLUGIN="node_modules/strray-framework"
-  elif [ -d "node_modules/@strray/strray-framework" ]; then
-    STRRAY_PLUGIN="node_modules/@strray/strray-framework"
-  elif [ -d "node_modules/oh-my-opencode/plugins/strray-framework" ]; then
-    STRRAY_PLUGIN="node_modules/oh-my-opencode/plugins/strray-framework"
-  elif [ -f "dist/postprocessor/PostProcessor.js" ]; then
-    # Development mode - use current project
-    STRRAY_PLUGIN="."
-  fi
+# Check for new reflection files and validate them
+NEW_REFLECTIONS=$(git diff --name-only HEAD~1 2>/dev/null | grep "^docs/reflections/.*\.md$" | grep -v "TEMPLATE.md" || true)
+if [ -n "$NEW_REFLECTIONS" ]; then
+  echo "🔍 Validating new reflection files..."
+  for ref in $NEW_REFLECTIONS; do
+    if [ -f "$ref" ]; then
+      if bash scripts/node/reflection-check.sh "$ref" 2>/dev/null; then
+        echo "   ✅ $ref passes template validation"
+      else
+        echo "   ❌ $ref FAILED template validation"
+        echo "   📖 Read the template: docs/reflections/TEMPLATE.md"
+        echo "   ⚠️  Please fix the reflection to match the template before committing"
+      fi
+    fi
+  done
+  echo ""
+fi
 
-  if command -v node >/dev/null 2>&1 && [ -n "$STRRAY_PLUGIN" ]; then
-    # Call a separate script to avoid bash variable issues
-    export COMMIT_SHA="$COMMIT_SHA"
-    export REPO="$REPO"
-    export BRANCH="$BRANCH"
-    export AUTHOR="$AUTHOR"
-    export STRRAY_PLUGIN="$STRRAY_PLUGIN"
-    export MONITORING_LEVEL="$MONITORING_LEVEL"
-    export IS_FULL_MONITORING="$([ "$MONITORING_LEVEL" = "full" ] && echo "true" || echo "false")"
-
-    # Run appropriate monitoring based on hook type
-    if [ "$HOOK_NAME" = "post-commit" ]; then
-      # LIGHT MONITORING: Quick validation, don't block git workflow
-      # Timeout: 2 seconds max, log metrics for monitoring
-      START_TIME=$(date +%s)
-      timeout 2 node -e "
-      (async () => {
-        try {
-          // Use import resolver to avoid hardcoded dist paths
-          const { importResolver } = await import('./utils/import-resolver.js');
-          const { LightweightValidator } = await importResolver.importModule('postprocessor/validation/LightweightValidator');
-
-          const validator = new LightweightValidator();
-          const result = await validator.validate();
-
-          if (result.warnings.length > 0) {
-            console.log('⚠️ ' + result.warnings.length + ' warning(s) found:');
-            result.warnings.forEach(w => console.log('   ' + w));
-          }
-
-          if (!result.passed) {
-            console.log('❌ ' + result.errors.length + ' error(s) found:');
-            result.errors.forEach(e => console.log('   ' + e));
-            process.exit(1);
-          }
-
-          console.log('✅ Post-commit: Validation passed in ' + result.duration + 'ms');
-        } catch (error) {
-          console.error('❌ Post-commit validation failed:', error instanceof Error ? error.message : String(error));
-          process.exit(1);
-        }
-      })();
-      " 2>/dev/null
-      EXIT_CODE=$?
-      END_TIME=$(date +%s)
-      DURATION=$((END_TIME - START_TIME))
-
-      # Log metrics for monitoring (convert to milliseconds)
-      DURATION_MS=$((DURATION * 1000))
-      echo "HOOK_METRICS: post-commit duration=${DURATION_MS}ms exit_code=${EXIT_CODE}" >&2
-
-      # Record metrics using metrics collector (direct import for reliability)
-      node -e "
-      (async () => {
-        try {
-          const { HookMetricsCollector } = await import('./dist/postprocessor/validation/HookMetricsCollector.js');
-          const collector = new HookMetricsCollector();
-          collector.recordMetrics('post-commit', ${DURATION_MS}, ${EXIT_CODE});
-        } catch (error) {
-          // Silently fail if metrics collection fails
-        }
-      })();
-      " 2>/dev/null || true
-
-      [ $EXIT_CODE -eq 0 ] && exit 0 || exit 1
+# Check if AGENTS.md was modified and validate it
+AGENTS_MODIFIED=$(git diff --name-only HEAD~1 2>/dev/null | grep "^AGENTS.md$" || true)
+if [ -n "$AGENTS_MODIFIED" ]; then
+  echo "🔍 Validating AGENTS.md..."
+  if [ -f "AGENTS.md" ]; then
+    # Check required sections
+    ERRORS=0
+    for section in "## Available Agents" "## Complexity Routing" "## CLI Commands" "## Codex"; do
+      if grep -q "$section" AGENTS.md; then
+        echo "   ✅ Contains: $section"
+      else
+        echo "   ❌ Missing: $section"
+        ERRORS=$((ERRORS + 1))
+      fi
+    done
+    
+    # Check agent count
+    AGENT_COUNT=$(grep -o '@[a-z-]*' AGENTS.md | wc -l)
+    if [ "$AGENT_COUNT" -ge 15 ]; then
+      echo "   ✅ Agent count: $AGENT_COUNT (recommended: 20+)"
     else
-      # FULL MONITORING: Comprehensive analysis for post-push
-      # Timeout: 5 minutes max, comprehensive CI/CD validation
-      START_TIME=$(date +%s)
-      timeout 300 node -e "
-      (async () => {
-        try {
-          console.log('🚀 Post-push: Comprehensive validation initiated');
-          // Use import resolver for environment-aware imports
-          const { importResolver } = await import('./utils/import-resolver.js');
-          const { ComprehensiveValidator } = await importResolver.importModule('postprocessor/validation/ComprehensiveValidator');
-
-          const validator = new ComprehensiveValidator();
-          const result = await validator.validate();
-
-          if (result.warnings.length > 0) {
-            console.log('⚠️ ' + result.warnings.length + ' warning(s) found:');
-            result.warnings.forEach(w => console.log('   ' + w));
-          }
-
-          if (!result.passed) {
-            console.log('❌ ' + result.errors.length + ' error(s) found:');
-            result.errors.forEach(e => console.log('   ' + e));
-            process.exit(1);
-          }
-
-          if (result.testResults) {
-            console.log('🧪 Tests: ' + result.testResults.passed + '/' + result.testResults.total + ' passed');
-          }
-
-          console.log('✅ Post-push: Comprehensive validation passed in ' + result.duration + 'ms');
-        } catch (error) {
-          console.error('❌ Post-push validation failed:', error instanceof Error ? error.message : String(error));
-          process.exit(1);
-        }
-      })();
-      " 2>/dev/null
-      EXIT_CODE=$?
-      END_TIME=$(date +%s)
-      DURATION=$((END_TIME - START_TIME))
-
-      # Log comprehensive metrics for monitoring (convert to milliseconds)
-      DURATION_MS=$((DURATION * 1000))
-      echo "HOOK_METRICS: post-push duration=${DURATION_MS}ms exit_code=${EXIT_CODE}" >&2
-
-      # Record metrics using metrics collector (direct import for reliability)
-      node -e "
-      (async () => {
-        try {
-          const { HookMetricsCollector } = await import('./dist/postprocessor/validation/HookMetricsCollector.js');
-          const collector = new HookMetricsCollector();
-          collector.recordMetrics('post-push', ${DURATION_MS}, ${EXIT_CODE});
-        } catch (error) {
-          // Silently fail if metrics collection fails
-        }
-      })();
-      " 2>/dev/null || true
-
-      [ $EXIT_CODE -eq 0 ] && exit 0 || exit 1
+      echo "   ⚠️  Agent count: $AGENT_COUNT (recommended: 20+)"
     fi
+    
+    if [ "$ERRORS" -gt 0 ]; then
+      echo "   ⚠️  AGENTS.md has $ERRORS missing required sections"
+      echo "   📖 See AGENTS.md in root for current format"
+    else
+      echo "   ✅ AGENTS.md validation passed"
+    fi
+  else
+    echo "   ❌ AGENTS.md not found"
+  fi
+  echo ""
+fi
+
+# For consumer projects, always use the full node_modules path to avoid symlink issues
+# This ensures imports resolve correctly
+STRRAY_PATH=""
+for pkg in "strray-ai" "strray-framework"; do
+  if [ -d "node_modules/$pkg/dist" ]; then
+    STRRAY_PATH="node_modules/$pkg"
+    break
+  fi
+done
+
+if [ -z "$STRRAY_PATH" ]; then
+  # Fallback to local dist for development
+  if [ -d "dist" ]; then
+    STRRAY_PATH="."
   else
-    echo "Warning: StrRay plugin not found or Node.js not available, skipping post-processor"
+    exit 0
   fi
-)
+fi
+
+# Run post-processor in background with Node.js
+node --input-type=module << EOF &>/dev/null || true
+import { PostProcessor } from './$STRRAY_PATH/dist/postprocessor/PostProcessor.js';
+import { StrRayStateManager } from './$STRRAY_PATH/dist/state/state-manager.js';
+
+const stateManager = new StrRayStateManager();
+const postProcessor = new PostProcessor(stateManager, null, { 
+  reporting: { enabled: false } 
+});
+
+const files = \`$FILES\`.split('\n').filter(f => f.endsWith('.ts') && !f.includes('.test.'));
+
+await postProcessor.executePostProcessorLoop({
+  commitSha: '$COMMIT_SHA',
+  repository: 'strray-framework/stringray',
+  branch: '$BRANCH',
+  author: '$AUTHOR',
+  files: files,
+  trigger: 'git-hook'
+});
+
+console.log('Post-processor completed for', files.length, 'files');
+EOF
 
-# Don't wait for background process
 exit 0