Merge pull request #12 from ohsono/feature/claude-code-integration

🔧 Fix GitHub Actions workflow issues found via local testing

Author: ohsono

.github/workflows/claude-code-integration.yml

@@ -304,6 +304,7 @@ jobs:
         echo "Finding all code files in repository..."
         
         # Safely find code files with controlled patterns
+        # Use a simpler approach to avoid complex piping issues
         find . -type f \( \
           -name "*.py" -o -name "*.js" -o -name "*.ts" -o -name "*.jsx" -o -name "*.tsx" -o \
           -name "*.java" -o -name "*.cpp" -o -name "*.c" -o -name "*.h" -o -name "*.go" -o \
@@ -319,11 +320,13 @@ jobs:
         -not -path "./__pycache__/*" \
         -not -path "./dist/*" \
         -not -path "./build/*" \
-        -not -path "./.github/*" \
-        -print0 | head -z -n "$MAX_FILES_PER_RUN" | tr '\0' '\n' | grep -v '^$' > code_files.txt
+        -not -path "./.github/*" > all_files.txt
+        
+        # Limit files and remove empty lines (reduce to 10 files to avoid argument length issues)
+        head -n 10 all_files.txt | grep -v '^$' > code_files.txt || touch code_files.txt
         
         file_count=$(wc -l < code_files.txt)
-        echo "Found $file_count code files (limited to $MAX_FILES_PER_RUN)"
+        echo "Found $file_count code files (limited to 10 to avoid argument length issues)"
         
         if [ "$file_count" -eq 0 ]; then
           echo "❌ No code files found in repository"
@@ -389,16 +392,20 @@ jobs:
         echo "" >> combined_prompt.txt
         cat "$PROMPT_FILE" >> combined_prompt.txt
         
-        # Run Claude analysis with timeout
+        # Run Claude analysis
         echo "Running Claude analysis on $file_count files..."
+        echo "Prompt size: $(wc -c < combined_prompt.txt) characters"
         
-        timeout 300 claude --model "$CLAUDE_MODEL" -p "$(cat combined_prompt.txt)" > "$OUTPUT_FILE" 2>&1 || {
-          echo "❌ Claude analysis failed or timed out"
+        # Run Claude analysis (with limited files to avoid argument length issues)
+        claude --model "$CLAUDE_MODEL" -p "$(cat combined_prompt.txt)" > "$OUTPUT_FILE" 2>&1 || {
+          echo "❌ Claude analysis failed"
           echo "Troubleshooting:"
           echo "1. Check authentication: claude --version"
           echo "2. Check model availability: $CLAUDE_MODEL"
           echo "3. Check prompt size: wc -c combined_prompt.txt"
           echo "4. Check network connectivity"
+          echo "5. Check error output:"
+          [ -f "$OUTPUT_FILE" ] && cat "$OUTPUT_FILE"
           exit 1
         }
         
@@ -414,7 +421,7 @@ jobs:
         chmod +x analyze_code.sh
         
     - name: Run Code Review Analysis
-      if: steps.validate.outputs.task_type_validated == 'code-review' || github.event_name != 'workflow_dispatch'
+      if: github.event_name != 'workflow_dispatch' || steps.validate.outputs.task_type_validated == 'code-review'
       env:
         CLAUDE_CODE_OAUTH_TOKEN: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
         ANTHROPIC_API_KEY: ${{ secrets.ANTHROPIC_API_KEY }}
@@ -441,7 +448,20 @@ jobs:
         Be concise but thorough. Focus on actionable feedback.
         EOF
         
+        # Debug: Check if files exist
+        echo "Debug: Checking prerequisites..."
+        echo "- code_files.txt exists: $([ -f code_files.txt ] && echo 'YES' || echo 'NO')"
+        echo "- analyze_code.sh exists: $([ -f analyze_code.sh ] && echo 'YES' || echo 'NO')"
+        echo "- analysis_prompt.txt exists: $([ -f analysis_prompt.txt ] && echo 'YES' || echo 'NO')"
+        
+        if [ -f code_files.txt ]; then
+          echo "- Files to analyze: $(wc -l < code_files.txt)"
+          echo "- File contents:"
+          cat code_files.txt
+        fi
+        
         # Run analysis using the secure function
+        echo "Running analysis..."
         ./analyze_code.sh analysis_prompt.txt analysis_result.md
         
         echo "✅ Code review analysis completed"