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.
|
@copilot apply changes based on the comments in this thread |
Agent-Logs-Url: https://github.com/IgniteUI/igniteui-cli/sessions/d2c42627-5d4b-4634-97f6-66691755a3ff Co-authored-by: kdinev <1472513+kdinev@users.noreply.github.com>
Updated in commit e54eedf. All remaining |
Closes #1599