Conversation
There was a problem hiding this comment.
Pull request overview
Updates the root README’s MCP build instructions to reference an existing npm script, addressing issue #1599 where the documented command fails due to a missing build:mcp-bundle script.
Changes:
- Replaces
npm run build:mcp-bundlewithnpm run build:mcpin the “Run locally” instructions.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| 3. Build the MCP server and bundle it into the CLI: | ||
| ```bash | ||
| cd packages/igniteui-mcp/igniteui-doc-mcp | ||
| npm install | ||
| npm run build | ||
| cd ../../.. | ||
| npm run build:mcp-bundle | ||
| npm run build:mcp | ||
| ``` |
There was a problem hiding this comment.
This section now instructs running npm run build:mcp, but the README still references the non-existent build:mcp-bundle script in later MCP sections (e.g., “Bundle MCP into CLI” / “Test the MCP server locally”). Please update those remaining occurrences (or reintroduce a build:mcp-bundle script) so the documentation is consistent and users don’t hit the same missing-script error elsewhere.
Closes #1599