MCP Memory Service

name: HTTP-MCP Bridge Tests on: push: paths: - 'examples/http-mcp-bridge.js' - 'tests/bridge/**' - 'tests/integration/test_bridge_integration.js' - '.github/workflows/bridge-tests.yml' pull_request: paths: - 'examples/http-mcp-bridge.js' - 'tests/bridge/**' - 'tests/integration/test_bridge_integration.js' jobs: test-bridge: runs-on: ubuntu-latest strategy: matrix: node-version: [18.x, 20.x] steps: - name: Checkout code uses: actions/checkout@v4 - name: Setup Node.js ${{ matrix.node-version }} uses: actions/setup-node@v4 with: node-version: ${{ matrix.node-version }} - name: Install bridge unit test dependencies run: npm install working-directory: tests/bridge - name: Install integration test dependencies run: npm install working-directory: tests/integration - name: Run bridge unit tests run: npm test working-directory: tests/bridge - name: Run integration tests run: npm test working-directory: tests/integration - name: Validate health endpoint fix run: | if ! grep -q "/api/health" examples/http-mcp-bridge.js; then echo "❌ ERROR: Health endpoint should be /api/health" exit 1 fi echo "✅ Health endpoint check passed" - name: Validate status code handling fix run: | if grep -v "|| response.statusCode === 201" examples/http-mcp-bridge.js | grep -q "statusCode === 201"; then echo "❌ ERROR: Found hardcoded 201-only status check - server returns 200!" exit 1 fi echo "✅ Status code handling check passed" - name: Validate URL construction fix run: | if grep -q "new URL(path, this.endpoint)" examples/http-mcp-bridge.js; then echo "❌ ERROR: Old URL construction bug still present" exit 1 fi echo "✅ URL construction check passed" contract-validation: runs-on: ubuntu-latest steps: - name: Checkout code uses: actions/checkout@v4 - name: Setup Node.js uses: actions/setup-node@v4 with: node-version: '20.x' - name: Validate API contracts run: node -e " const path = require('path'); const mocks = require('./tests/bridge/mock_responses.js'); const memorySuccess = mocks.mockResponses.memories.createSuccess; if (memorySuccess.status !== 200) throw new Error('Memory creation should return 200'); if (!memorySuccess.body.hasOwnProperty('success')) throw new Error('Memory response must have success field'); console.log('✅ API contract validation passed'); " - name: Generate contract report run: | echo "# API Contract Report" > contract-report.md echo "Generated: $(date)" >> contract-report.md echo "" >> contract-report.md echo "## Critical Endpoints Validated" >> contract-report.md echo "- ✅ POST /api/memories: Returns 200 with success field" >> contract-report.md echo "- ✅ GET /api/health: Correct endpoint path used" >> contract-report.md echo "- ✅ URL construction: Preserves /api base path" >> contract-report.md echo "- ✅ Status codes: Handles HTTP 200 correctly" >> contract-report.md - name: Upload contract report uses: actions/upload-artifact@v4 if: always() with: name: api-contract-report path: contract-report.md quick-smoke-test: runs-on: ubuntu-latest steps: - name: Checkout code uses: actions/checkout@v4 - name: Setup Node.js uses: actions/setup-node@v4 with: node-version: '20.x' - name: Check JavaScript syntax run: node -c examples/http-mcp-bridge.js - name: Test bridge instantiation run: node -e " const Bridge = require('./examples/http-mcp-bridge.js'); const bridge = new Bridge(); console.log('✅ Bridge can be instantiated'); " - name: Test URL construction run: | node -e " const Bridge = require('./examples/http-mcp-bridge.js'); const bridge = new Bridge(); bridge.endpoint = 'https://test.local:8443/api'; const fullPath = '/memories'; let baseUrl; if (bridge.endpoint.endsWith('/')) { baseUrl = bridge.endpoint.slice(0, -1); } else { baseUrl = bridge.endpoint; } if (baseUrl + fullPath !== 'https://test.local:8443/api/memories') { throw new Error('URL construction test failed'); } console.log('✅ URL construction works correctly'); "