Merge pull request #233 from PaulJPhilp/codex/pr-5-release-test-ci-repair

PR-5: Release test/CI repair (P1)

Author: PaulJPhilp

.github/workflows/ci.yml

@@ -239,8 +239,8 @@ jobs:
           PATTERN_API_KEY: test-api-key-for-ci
           TEST_BASE_URL: http://localhost:3000
 
-      - name: Run MCP protocol tests
-        run: bun --filter @effect-patterns/mcp-server run test:mcp:local
+      - name: Run MCP protocol tests (tool surface + structured output)
+        run: bun --filter @effect-patterns/mcp-server run test:mcp:ci
         env:
           PATTERN_API_KEY: test-api-key-for-ci
           MCP_ENV: local
@@ -295,4 +295,4 @@ jobs:
             echo "One or more CI jobs were cancelled"
             exit 1
           fi
-          echo "All CI jobs passed successfully!"
+          echo "All CI jobs passed successfully!"

packages/mcp-server/package.json

@@ -34,7 +34,8 @@
     "test:stress:all": "bun run test:stress",
     "test:mcp": "bunx vitest run --config vitest.mcp.config.ts",
     "test:mcp:watch": "bunx vitest --config vitest.mcp.config.ts",
-    "test:mcp:local": "MCP_ENV=local bunx vitest run --config vitest.mcp.config.ts tests/mcp-protocol/local.test.ts tests/mcp-protocol/stdio-discipline.test.ts",
+    "test:mcp:local": "MCP_ENV=local bunx vitest run --config vitest.mcp.config.ts tests/mcp-protocol/local.test.ts tests/mcp-protocol/structured-output.test.ts tests/mcp-protocol/stdio-discipline.test.ts",
+    "test:mcp:ci": "MCP_ENV=local bunx vitest run --config vitest.mcp.config.ts tests/mcp-protocol/local.test.ts tests/mcp-protocol/structured-output.test.ts tests/mcp-protocol/stdio-discipline.test.ts",
     "test:mcp:staging": "bash scripts/test-mcp-staging.sh",
     "test:mcp:production": "bash scripts/test-mcp-production.sh",
     "test:mcp:all": "bash scripts/test-mcp-all.sh",
@@ -54,7 +55,7 @@
     "typecheck": "tsc --noEmit",
     "lint": "next lint",
     "check:narration-leakage": "bash scripts/check-narration-leakage.sh",
-    "preflight": "bun run mcp:build && bun run check:narration-leakage && bun run test:mcp:local"
+    "preflight": "bun run mcp:build && bun run check:narration-leakage && bun run test:mcp:ci"
   },
   "dependencies": {
     "@effect-patterns/analysis-core": "workspace:*",

packages/mcp-server/scripts/test-mcp-production.sh

@@ -52,6 +52,8 @@ export EFFECT_PATTERNS_API_URL="${PRODUCTION_URL}"
 echo -e "${BLUE}Running MCP protocol tests against production server...${NC}\n"
 
 # Run MCP tests with production config
-bunx vitest run --config vitest.mcp.config.ts tests/mcp-protocol/client-stdio.test.ts tests/mcp-protocol/tools.test.ts
+bunx vitest run --config vitest.mcp.config.ts \
+    tests/mcp-protocol/client-stdio.test.ts \
+    tests/mcp-protocol/structured-output.test.ts
 
 echo -e "\n${GREEN}✓ Production MCP tests completed${NC}"

packages/mcp-server/scripts/test-mcp-staging.sh

@@ -52,6 +52,8 @@ export EFFECT_PATTERNS_API_URL="${STAGING_URL}"
 echo -e "${BLUE}Running MCP protocol tests against staging server...${NC}\n"
 
 # Run MCP tests with staging config
-bunx vitest run --config vitest.mcp.config.ts tests/mcp-protocol/client-stdio.test.ts tests/mcp-protocol/tools.test.ts
+bunx vitest run --config vitest.mcp.config.ts \
+    tests/mcp-protocol/client-stdio.test.ts \
+    tests/mcp-protocol/structured-output.test.ts
 
 echo -e "\n${GREEN}✓ Staging MCP tests completed${NC}"