From 8ac539ac627603a9c5544c97ca9090ed9c749100 Mon Sep 17 00:00:00 2001 From: cliffhall Date: Thu, 16 Apr 2026 12:43:27 -0400 Subject: [PATCH] Install root dependencies in CI so core/*.ts can resolve SDK clients/web/tsconfig.app.json's include pulls in ../../core/**/*.ts. TypeScript module resolution from those files walks up from core/mcp/ looking for node_modules, which lands at the repo root. CI only ran npm install inside clients/web/, so root node_modules was empty and core/mcp/types.ts / elicitationCreateMessage.ts / taskNotificationSchemas.ts failed to resolve @modelcontextprotocol/sdk/types.js and zod/v4 (locally the build worked because root had been manually installed). Add an explicit root npm install step before the clients/web install. Co-Authored-By: Claude Opus 4.6 (1M context) --- .github/workflows/main.yml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml index be64cd3a..3d81e5e0 100644 --- a/.github/workflows/main.yml +++ b/.github/workflows/main.yml @@ -16,7 +16,10 @@ jobs: node-version: '20.x' cache: 'npm' - - name: Install dependencies + - name: Install root dependencies + run: npm install + + - name: Install web client dependencies working-directory: ./clients/web run: npm install