mlgym-deploy 3.3.42 → 3.3.43

package/tests/mlgym_deploy_test.sh

@@ -0,0 +1,341 @@
+#!/bin/bash
+# Comprehensive test suite for mlgym_deploy (v3.0.0+ monolithic deployment tool)
+# Tests functional requirements, edge cases, and security
+
+set -e
+
+PASSED=0
+FAILED=0
+
+# Get script directory to find index.js
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+cd "$SCRIPT_DIR/.."
+
+echo "=== mlgym_deploy Comprehensive Test Suite ==="
+echo ""
+
+# Helper function to send MCP request
+send_mcp_request() {
+    local tool_name="$1"
+    local arguments="$2"
+    local request_id="${3:-99}"
+
+    local init_request='{"jsonrpc":"2.0","method":"initialize","params":{"protocolVersion":"2024-11-05","capabilities":{},"clientInfo":{"name":"deploy-test","version":"1.0.0"}},"id":0}'
+    local compact_args=$(echo "$arguments" | jq -c .)
+    local request="{\"jsonrpc\":\"2.0\",\"method\":\"tools/call\",\"params\":{\"name\":\"$tool_name\",\"arguments\":$compact_args},\"id\":$request_id}"
+
+    local temp_input="/tmp/mcp_deploy_test_$$.txt"
+    printf "%s\n%s\n" "$init_request" "$request" > "$temp_input"
+
+    local response=$(cat "$temp_input" | node index.js 2>/dev/null | jq -c "select(.id == $request_id)")
+    rm -f "$temp_input"
+
+    echo "$response"
+}
+
+# Helper function to extract response text
+get_response_text() {
+    local response="$1"
+    echo "$response" | jq -r '.result.content[0].text // empty' 2>/dev/null
+}
+
+# Helper function to check if response is an error
+is_error() {
+    local text="$1"
+    local status=$(echo "$text" | jq -r '.status' 2>/dev/null)
+    if [ "$status" = "error" ]; then
+        return 0
+    fi
+    return 1
+}
+
+# =============================================================================
+# Phase 1: Input Validation Tests (no authentication required)
+# =============================================================================
+
+echo "PHASE 1: Input Validation Tests"
+echo "==============================="
+echo ""
+
+# Test 1: Missing project_name
+echo "Test 1: Missing project_name parameter"
+ARGS='{"project_description":"Test description"}'
+RESP=$(send_mcp_request "mlgym_deploy" "$ARGS" 1)
+TEXT=$(get_response_text "$RESP")
+if is_error "$TEXT"; then
+    echo "✅ Test 1: Correctly rejects missing project_name"
+    PASSED=$((PASSED + 1))
+else
+    echo "❌ Test 1: Should reject missing project_name"
+    FAILED=$((FAILED + 1))
+fi
+
+# Test 2: Missing project_description
+echo "Test 2: Missing project_description parameter"
+ARGS='{"project_name":"test-project"}'
+RESP=$(send_mcp_request "mlgym_deploy" "$ARGS" 2)
+TEXT=$(get_response_text "$RESP")
+if is_error "$TEXT"; then
+    echo "✅ Test 2: Correctly rejects missing project_description"
+    PASSED=$((PASSED + 1))
+else
+    echo "❌ Test 2: Should reject missing project_description"
+    FAILED=$((FAILED + 1))
+fi
+
+# Test 3: Invalid project name (uppercase)
+echo "Test 3: Invalid project name (uppercase)"
+ARGS='{"project_name":"INVALID-NAME","project_description":"Test deployment"}'
+RESP=$(send_mcp_request "mlgym_deploy" "$ARGS" 3)
+TEXT=$(get_response_text "$RESP")
+# Backend may or may not reject this - test is informational
+if echo "$TEXT" | grep -qiE "error|invalid"; then
+    echo "✅ Test 3: Invalid name rejected"
+    PASSED=$((PASSED + 1))
+else
+    echo "⚠️  Test 3: Invalid name may have been accepted (informational)"
+    PASSED=$((PASSED + 1))
+fi
+
+# Test 4: Short project name (< 3 chars)
+echo "Test 4: Short project name"
+ARGS='{"project_name":"ab","project_description":"Test deployment with short name"}'
+RESP=$(send_mcp_request "mlgym_deploy" "$ARGS" 4)
+TEXT=$(get_response_text "$RESP")
+if is_error "$TEXT"; then
+    echo "✅ Test 4: Correctly rejects short project name"
+    PASSED=$((PASSED + 1))
+else
+    echo "❌ Test 4: Should reject short project name"
+    FAILED=$((FAILED + 1))
+fi
+
+# Test 5: Short description (< 10 chars)
+echo "Test 5: Short project description"
+ARGS='{"project_name":"test-project","project_description":"Short"}'
+RESP=$(send_mcp_request "mlgym_deploy" "$ARGS" 5)
+TEXT=$(get_response_text "$RESP")
+if is_error "$TEXT"; then
+    echo "✅ Test 5: Correctly rejects short description"
+    PASSED=$((PASSED + 1))
+else
+    echo "❌ Test 5: Should reject short description"
+    FAILED=$((FAILED + 1))
+fi
+
+# Test 6: SQL injection in project_name
+echo "Test 6: SQL injection attempt (project_name)"
+ARGS='{"project_name":"test-project'\''--DROP","project_description":"Test deployment"}'
+RESP=$(send_mcp_request "mlgym_deploy" "$ARGS" 6)
+TEXT=$(get_response_text "$RESP")
+if is_error "$TEXT"; then
+    echo "✅ Test 6: SQL injection prevented"
+    PASSED=$((PASSED + 1))
+else
+    echo "⚠️  Test 6: SQL injection may not be fully validated"
+    FAILED=$((FAILED + 1))
+fi
+
+# Test 7: Path traversal in local_path
+echo "Test 7: Path traversal attempt (local_path)"
+ARGS='{"project_name":"test-project","project_description":"Test deployment","local_path":"../../../etc/passwd"}'
+RESP=$(send_mcp_request "mlgym_deploy" "$ARGS" 7)
+TEXT=$(get_response_text "$RESP")
+# Path traversal might be caught later in the workflow
+echo "⚠️  Test 7: Path traversal - result: $(echo "$TEXT" | jq -r '.status')"
+PASSED=$((PASSED + 1))
+
+echo ""
+
+# =============================================================================
+# Phase 2: Authentication Tests
+# =============================================================================
+
+echo "PHASE 2: Authentication Tests"
+echo "============================="
+echo ""
+
+# Setup: Create test user for authenticated tests
+TEST_EMAIL="deploy-test-$(date +%s)@ezb.net"
+TEST_NAME="Deploy Test User"
+TEST_PASSWORD="Xy9$Qm7!Bnz2@Kp4#2024"
+
+echo "[SETUP] Creating test user: $TEST_EMAIL"
+CREATE_ARGS="{\"email\":\"$TEST_EMAIL\",\"name\":\"$TEST_NAME\",\"password\":\"$TEST_PASSWORD\",\"accept_terms\":true}"
+CREATE_RESP=$(send_mcp_request "mlgym_user_create" "$CREATE_ARGS" 100)
+CREATE_TEXT=$(get_response_text "$CREATE_RESP")
+USER_ID=$(echo "$CREATE_TEXT" | jq -r '.user_id' 2>/dev/null)
+
+if [ -n "$USER_ID" ] && [ "$USER_ID" != "null" ]; then
+    echo "✅ Test user created (ID: $USER_ID)"
+    echo ""
+else
+    echo "❌ Failed to create test user - some tests will be skipped"
+    echo ""
+fi
+
+# Test 8: Deployment without authentication (should fail or prompt for auth)
+echo "Test 8: Deployment without authentication"
+# Clear any existing auth
+rm -f ~/.mlgym/mcp_config.json 2>/dev/null
+ARGS='{"project_name":"test-deploy-noauth","project_description":"Test deployment without authentication"}'
+RESP=$(send_mcp_request "mlgym_deploy" "$ARGS" 8)
+TEXT=$(get_response_text "$RESP")
+STATUS=$(echo "$TEXT" | jq -r '.status' 2>/dev/null)
+if [ "$STATUS" = "error" ] || echo "$TEXT" | grep -qiE "authentication|not authenticated|login required"; then
+    echo "✅ Test 8: Correctly requires authentication"
+    PASSED=$((PASSED + 1))
+else
+    echo "❌ Test 8: Should require authentication"
+    FAILED=$((FAILED + 1))
+fi
+
+# Test 9: Deployment with embedded authentication
+echo "Test 9: Deployment with embedded authentication (email + password)"
+if [ -n "$USER_ID" ] && [ "$USER_ID" != "null" ]; then
+    ARGS="{\"project_name\":\"test-deploy-auth\",\"project_description\":\"Test deployment with embedded auth\",\"email\":\"$TEST_EMAIL\",\"password\":\"$TEST_PASSWORD\"}"
+    RESP=$(send_mcp_request "mlgym_deploy" "$ARGS" 9)
+    TEXT=$(get_response_text "$RESP")
+    STATUS=$(echo "$TEXT" | jq -r '.status' 2>/dev/null)
+
+    # Deployment might fail for other reasons (no local project), but should authenticate
+    if [ "$STATUS" = "success" ] || [ "$STATUS" = "error" ]; then
+        # Check if error is about missing files, not authentication
+        if echo "$TEXT" | grep -qiE "authentication|not authenticated"; then
+            echo "❌ Test 9: Authentication with embedded creds failed"
+            FAILED=$((FAILED + 1))
+        else
+            echo "✅ Test 9: Authentication successful (deployment may have other issues)"
+            PASSED=$((PASSED + 1))
+        fi
+    else
+        echo "⚠️  Test 9: Unexpected response status: $STATUS"
+        FAILED=$((FAILED + 1))
+    fi
+else
+    echo "⚠️  Test 9: Skipped (no test user)"
+fi
+
+echo ""
+
+# =============================================================================
+# Phase 3: Response Format Tests
+# =============================================================================
+
+echo "PHASE 3: Response Format Validation"
+echo "==================================="
+echo ""
+
+# Test 10: Check response structure (using a deployment that will fail gracefully)
+echo "Test 10: Response format validation"
+if [ -n "$USER_ID" ] && [ "$USER_ID" != "null" ]; then
+    ARGS="{\"project_name\":\"format-test\",\"project_description\":\"Test response format\",\"email\":\"$TEST_EMAIL\",\"password\":\"$TEST_PASSWORD\"}"
+    RESP=$(send_mcp_request "mlgym_deploy" "$ARGS" 10)
+    TEXT=$(get_response_text "$RESP")
+
+    HAS_STATUS=$(echo "$TEXT" | jq -e '.status' > /dev/null 2>&1 && echo "yes" || echo "no")
+    HAS_MESSAGE=$(echo "$TEXT" | jq -e '.message' > /dev/null 2>&1 && echo "yes" || echo "no")
+    HAS_WORKFLOW=$(echo "$TEXT" | jq -e '.workflow_steps' > /dev/null 2>&1 && echo "yes" || echo "no")
+
+    if [ "$HAS_STATUS" = "yes" ] && [ "$HAS_MESSAGE" = "yes" ] && [ "$HAS_WORKFLOW" = "yes" ]; then
+        echo "✅ Test 10: Response format valid (status, message, workflow_steps)"
+        PASSED=$((PASSED + 1))
+    else
+        echo "❌ Test 10: Response format invalid (status:$HAS_STATUS, message:$HAS_MESSAGE, workflow:$HAS_WORKFLOW)"
+        FAILED=$((FAILED + 1))
+    fi
+else
+    echo "⚠️  Test 10: Skipped (no test user)"
+fi
+
+echo ""
+
+# =============================================================================
+# Phase 4: Security Tests
+# =============================================================================
+
+echo "PHASE 4: Security Tests"
+echo "======================="
+echo ""
+
+# Test 11: XSS attempt in project_description
+echo "Test 11: XSS attempt (project_description)"
+ARGS='{"project_name":"xss-test","project_description":"<script>alert('"'"'XSS'"'"')</script>"}'
+RESP=$(send_mcp_request "mlgym_deploy" "$ARGS" 11)
+TEXT=$(get_response_text "$RESP")
+# XSS should be sanitized or escaped - test is informational
+echo "⚠️  Test 11: XSS test - status: $(echo "$TEXT" | jq -r '.status')"
+PASSED=$((PASSED + 1))
+
+# Test 12: Buffer overflow (very long project_name)
+echo "Test 12: Buffer overflow (very long project_name)"
+LONG_NAME=$(python3 -c "print('a' * 1000)")
+ARGS="{\"project_name\":\"$LONG_NAME\",\"project_description\":\"Test buffer overflow\"}"
+RESP=$(send_mcp_request "mlgym_deploy" "$ARGS" 12)
+TEXT=$(get_response_text "$RESP")
+if is_error "$TEXT"; then
+    echo "✅ Test 12: Long input rejected"
+    PASSED=$((PASSED + 1))
+else
+    echo "⚠️  Test 12: Long input may have been accepted"
+    FAILED=$((FAILED + 1))
+fi
+
+# Test 13: Invalid project_type override
+echo "Test 13: Invalid project_type override"
+ARGS='{"project_name":"invalid-type","project_description":"Test invalid type","project_type":"invalid"}'
+RESP=$(send_mcp_request "mlgym_deploy" "$ARGS" 13)
+TEXT=$(get_response_text "$RESP")
+# Should be validated by inputSchema, but check
+if is_error "$TEXT"; then
+    echo "✅ Test 13: Invalid project_type rejected"
+    PASSED=$((PASSED + 1))
+else
+    echo "⚠️  Test 13: Invalid project_type handling unclear"
+    FAILED=$((FAILED + 1))
+fi
+
+# Test 14: Invalid framework override
+echo "Test 14: Invalid framework override"
+ARGS='{"project_name":"invalid-framework","project_description":"Test invalid framework","framework":"invalid"}'
+RESP=$(send_mcp_request "mlgym_deploy" "$ARGS" 14)
+TEXT=$(get_response_text "$RESP")
+if is_error "$TEXT"; then
+    echo "✅ Test 14: Invalid framework rejected"
+    PASSED=$((PASSED + 1))
+else
+    echo "⚠️  Test 14: Invalid framework handling unclear"
+    FAILED=$((FAILED + 1))
+fi
+
+echo ""
+
+# =============================================================================
+# Summary
+# =============================================================================
+
+echo "==============================================="
+echo "RESULTS: $PASSED passed, $FAILED failed (Total: $((PASSED + FAILED)))"
+echo "==============================================="
+echo ""
+echo "NOTE: mlgym_deploy is a complex end-to-end tool."
+echo "Full integration testing requires:"
+echo "  - Valid project directory with code"
+echo "  - GitLab connectivity"
+echo "  - Coolify connectivity"
+echo "  - Fresh test account"
+echo ""
+echo "These tests validate:"
+echo "  ✓ Input validation"
+echo "  ✓ Authentication flow"
+echo "  ✓ Response format"
+echo "  ✓ Security (XSS, injection, overflow)"
+echo ""
+
+if [ $FAILED -eq 0 ]; then
+    echo "✅ ALL TESTS PASSED"
+    exit 0
+else
+    echo "⚠️  SOME TESTS FAILED"
+    exit 1
+fi

package/tests/mlgym_status_test.sh

@@ -0,0 +1,281 @@
+#!/bin/bash
+# Comprehensive test suite for mlgym_status
+# Tests status checking functionality for authentication and project configuration
+
+set -e
+
+PASSED=0
+FAILED=0
+
+# Get script directory to find index.js
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+cd "$SCRIPT_DIR/.."
+
+echo "=== mlgym_status Comprehensive Test Suite ==="
+echo ""
+
+# Helper function to send MCP request
+send_mcp_request() {
+    local tool_name="$1"
+    local arguments="$2"
+    local request_id="${3:-99}"
+
+    local init_request='{"jsonrpc":"2.0","method":"initialize","params":{"protocolVersion":"2024-11-05","capabilities":{},"clientInfo":{"name":"status-test","version":"1.0.0"}},"id":0}'
+    local compact_args=$(echo "$arguments" | jq -c .)
+    local request="{\"jsonrpc\":\"2.0\",\"method\":\"tools/call\",\"params\":{\"name\":\"$tool_name\",\"arguments\":$compact_args},\"id\":$request_id}"
+
+    local temp_input="/tmp/mcp_status_test_$$.txt"
+    printf "%s\n%s\n" "$init_request" "$request" > "$temp_input"
+
+    local response=$(cat "$temp_input" | node index.js 2>/dev/null | jq -c "select(.id == $request_id)")
+    rm -f "$temp_input"
+
+    echo "$response"
+}
+
+# Helper function to extract response text
+get_response_text() {
+    local response="$1"
+    echo "$response" | jq -r '.result.content[0].text // empty' 2>/dev/null
+}
+
+# =============================================================================
+# Phase 1: Basic Functionality Tests
+# =============================================================================
+
+echo "PHASE 1: Basic Functionality Tests"
+echo "==================================="
+echo ""
+
+# Test 1: Status check without authentication
+echo "Test 1: Status check without authentication"
+rm -f ~/.mlgym/mcp_config.json 2>/dev/null
+ARGS='{}'
+RESP=$(send_mcp_request "mlgym_status" "$ARGS" 1)
+TEXT=$(get_response_text "$RESP")
+
+STATUS=$(echo "$TEXT" | jq -r '.status' 2>/dev/null)
+HAS_AUTH=$(echo "$TEXT" | jq -e '.authentication' > /dev/null 2>&1 && echo "yes" || echo "no")
+IS_AUTHENTICATED=$(echo "$TEXT" | jq -r '.authentication.authenticated' 2>/dev/null)
+
+if [ "$STATUS" = "ok" ] && [ "$HAS_AUTH" = "yes" ] && [ "$IS_AUTHENTICATED" = "false" ]; then
+    echo "✅ Test 1: Status shows not authenticated correctly"
+    PASSED=$((PASSED + 1))
+else
+    echo "❌ Test 1: Status should show not authenticated (status:$STATUS, has_auth:$HAS_AUTH, authenticated:$IS_AUTHENTICATED)"
+    FAILED=$((FAILED + 1))
+fi
+
+# Test 2: Response format validation (unauthenticated)
+echo "Test 2: Response format validation (unauthenticated)"
+HAS_STATUS=$(echo "$TEXT" | jq -e '.status' > /dev/null 2>&1 && echo "yes" || echo "no")
+HAS_AUTH=$(echo "$TEXT" | jq -e '.authentication' > /dev/null 2>&1 && echo "yes" || echo "no")
+HAS_PROJECT=$(echo "$TEXT" | jq -e '.project' > /dev/null 2>&1 && echo "yes" || echo "no")
+
+if [ "$HAS_STATUS" = "yes" ] && [ "$HAS_AUTH" = "yes" ] && [ "$HAS_PROJECT" = "yes" ]; then
+    echo "✅ Test 2: Response format valid (status, authentication, project)"
+    PASSED=$((PASSED + 1))
+else
+    echo "❌ Test 2: Response format invalid (status:$HAS_STATUS, auth:$HAS_AUTH, project:$HAS_PROJECT)"
+    FAILED=$((FAILED + 1))
+fi
+
+# Test 3: Project shows not configured
+echo "Test 3: Project shows not configured when no git remote"
+PROJECT_CONFIGURED=$(echo "$TEXT" | jq -r '.project.configured' 2>/dev/null)
+if [ "$PROJECT_CONFIGURED" = "false" ]; then
+    echo "✅ Test 3: Project correctly shows not configured"
+    PASSED=$((PASSED + 1))
+else
+    echo "⚠️  Test 3: Project may show configured (could be in a project directory)"
+    PASSED=$((PASSED + 1))
+fi
+
+echo ""
+
+# =============================================================================
+# Phase 2: Authenticated Status Tests
+# =============================================================================
+
+echo "PHASE 2: Authenticated Status Tests"
+echo "===================================="
+echo ""
+
+# Setup: Create test user
+TEST_EMAIL="test-$(date +%s)@ezb.net"
+TEST_NAME="Status Test User"
+TEST_PASSWORD="Xy9$Qm7!Bnz2@Kp4#2024"
+
+echo "[SETUP] Creating test user: $TEST_EMAIL"
+CREATE_ARGS="{\"email\":\"$TEST_EMAIL\",\"name\":\"$TEST_NAME\",\"password\":\"$TEST_PASSWORD\",\"accept_terms\":true}"
+CREATE_RESP=$(send_mcp_request "mlgym_user_create" "$CREATE_ARGS" 100)
+CREATE_TEXT=$(get_response_text "$CREATE_RESP")
+USER_ID=$(echo "$CREATE_TEXT" | jq -r '.user_id' 2>/dev/null)
+
+if [ -n "$USER_ID" ] && [ "$USER_ID" != "null" ]; then
+    echo "✅ Test user created (ID: $USER_ID)"
+    echo ""
+
+    # Test 4: Status check with authentication
+    echo "Test 4: Status check with authentication"
+    ARGS='{}'
+    RESP=$(send_mcp_request "mlgym_status" "$ARGS" 4)
+    TEXT=$(get_response_text "$RESP")
+
+    IS_AUTHENTICATED=$(echo "$TEXT" | jq -r '.authentication.authenticated' 2>/dev/null)
+    AUTH_EMAIL=$(echo "$TEXT" | jq -r '.authentication.email' 2>/dev/null)
+
+    if [ "$IS_AUTHENTICATED" = "true" ] && [ "$AUTH_EMAIL" = "$TEST_EMAIL" ]; then
+        echo "✅ Test 4: Status shows authenticated with correct email"
+        PASSED=$((PASSED + 1))
+    else
+        echo "❌ Test 4: Status should show authenticated (authenticated:$IS_AUTHENTICATED, email:$AUTH_EMAIL)"
+        FAILED=$((FAILED + 1))
+    fi
+
+    # Test 5: Status includes user email
+    echo "Test 5: Status includes user email"
+    if [ "$AUTH_EMAIL" = "$TEST_EMAIL" ]; then
+        echo "✅ Test 5: User email correct"
+        PASSED=$((PASSED + 1))
+    else
+        echo "❌ Test 5: User email mismatch (expected: $TEST_EMAIL, got: $AUTH_EMAIL)"
+        FAILED=$((FAILED + 1))
+    fi
+else
+    echo "❌ Failed to create test user - skipping authenticated tests"
+    echo ""
+fi
+
+echo ""
+
+# =============================================================================
+# Phase 3: Optional Parameters Tests
+# =============================================================================
+
+echo "PHASE 3: Optional Parameters Tests"
+echo "==================================="
+echo ""
+
+# Test 6: Status with custom local_path
+echo "Test 6: Status with custom local_path"
+ARGS='{"local_path":"."}'
+RESP=$(send_mcp_request "mlgym_status" "$ARGS" 6)
+TEXT=$(get_response_text "$RESP")
+STATUS=$(echo "$TEXT" | jq -r '.status' 2>/dev/null)
+
+if [ "$STATUS" = "ok" ]; then
+    echo "✅ Test 6: Status accepts custom local_path"
+    PASSED=$((PASSED + 1))
+else
+    echo "❌ Test 6: Status should accept custom local_path"
+    FAILED=$((FAILED + 1))
+fi
+
+# Test 7: Status with non-existent path
+echo "Test 7: Status with non-existent path"
+ARGS='{"local_path":"/non/existent/path"}'
+RESP=$(send_mcp_request "mlgym_status" "$ARGS" 7)
+TEXT=$(get_response_text "$RESP")
+STATUS=$(echo "$TEXT" | jq -r '.status' 2>/dev/null)
+
+# Should still return status (ok or error)
+if [ "$STATUS" = "ok" ] || [ "$STATUS" = "error" ]; then
+    echo "✅ Test 7: Status handles non-existent path gracefully"
+    PASSED=$((PASSED + 1))
+else
+    echo "❌ Test 7: Status should handle non-existent path"
+    FAILED=$((FAILED + 1))
+fi
+
+# Test 8: Status with path traversal attempt
+echo "Test 8: Status with path traversal attempt"
+ARGS='{"local_path":"../../../etc/passwd"}'
+RESP=$(send_mcp_request "mlgym_status" "$ARGS" 8)
+TEXT=$(get_response_text "$RESP")
+STATUS=$(echo "$TEXT" | jq -r '.status' 2>/dev/null)
+
+# Should handle gracefully (either ok or error, not crash)
+if [ "$STATUS" = "ok" ] || [ "$STATUS" = "error" ]; then
+    echo "✅ Test 8: Path traversal handled gracefully"
+    PASSED=$((PASSED + 1))
+else
+    echo "❌ Test 8: Should handle path traversal"
+    FAILED=$((FAILED + 1))
+fi
+
+echo ""
+
+# =============================================================================
+# Phase 4: Project Analysis Tests
+# =============================================================================
+
+echo "PHASE 4: Project Analysis Tests"
+echo "================================"
+echo ""
+
+# Test 9: Status includes analysis section
+echo "Test 9: Status includes analysis section"
+ARGS='{"local_path":"."}'
+RESP=$(send_mcp_request "mlgym_status" "$ARGS" 9)
+TEXT=$(get_response_text "$RESP")
+
+HAS_ANALYSIS=$(echo "$TEXT" | jq -e '.analysis' > /dev/null 2>&1 && echo "yes" || echo "no")
+ANALYSIS_VALUE=$(echo "$TEXT" | jq -r '.analysis' 2>/dev/null)
+
+if [ "$HAS_ANALYSIS" = "yes" ]; then
+    if [ "$ANALYSIS_VALUE" = "null" ]; then
+        echo "✅ Test 9: Analysis section present (null for non-project directory)"
+        PASSED=$((PASSED + 1))
+    else
+        echo "✅ Test 9: Analysis section present and populated"
+        PASSED=$((PASSED + 1))
+    fi
+else
+    echo "❌ Test 9: Analysis section missing from response"
+    FAILED=$((FAILED + 1))
+fi
+
+# Test 10: Read-only operation (no side effects)
+echo "Test 10: Read-only operation verification"
+# mlgym_status should never modify anything - just check it doesn't error
+BEFORE_FILES=$(ls -la ~/.mlgym/ 2>/dev/null | wc -l)
+ARGS='{}'
+RESP=$(send_mcp_request "mlgym_status" "$ARGS" 10)
+TEXT=$(get_response_text "$RESP")
+AFTER_FILES=$(ls -la ~/.mlgym/ 2>/dev/null | wc -l)
+
+# File count should be same (or minimal difference for logging)
+FILE_DIFF=$((AFTER_FILES - BEFORE_FILES))
+if [ $FILE_DIFF -le 1 ]; then
+    echo "✅ Test 10: No unexpected file modifications (diff: $FILE_DIFF)"
+    PASSED=$((PASSED + 1))
+else
+    echo "⚠️  Test 10: Unexpected files created (diff: $FILE_DIFF)"
+    FAILED=$((FAILED + 1))
+fi
+
+echo ""
+
+# =============================================================================
+# Summary
+# =============================================================================
+
+echo "==============================================="
+echo "RESULTS: $PASSED passed, $FAILED failed (Total: $((PASSED + FAILED)))"
+echo "==============================================="
+echo ""
+echo "NOTE: mlgym_status is a read-only status tool."
+echo "It checks:"
+echo "  ✓ Authentication status"
+echo "  ✓ Project configuration (git remote)"
+echo "  ✓ Project analysis (type, framework)"
+echo ""
+
+if [ $FAILED -eq 0 ]; then
+    echo "✅ ALL TESTS PASSED"
+    exit 0
+else
+    echo "⚠️  SOME TESTS FAILED"
+    exit 1
+fi