W-21260424 Update documentation for mcp config discrepancy#164
Merged
Conversation
allison-grady
requested review from
patricksullivansf,
wei-liu-sf and
yhsieh1
as code owners
allison-grady
changed the title
|
|
||
| > **Important:** MCP clients like Cursor and Claude Desktop spawn servers from the home directory (`~`), not your project. Always set `--working-directory`. | ||
|
|
||
| When using `"command": "node"`, the shebang in `bin/dev.js` is not run, so the server loads from built `dist/`. **Build the project to latest** (`pnpm run build` from the repo root) so auto-discovery works. Without a fresh build, auto-discovery can fail—notably **Storefront Next** in monorepos like [storefront-next](https://github.com/SalesforceCommerceCloud/storefront-next), because that detector is the only one that walks workspace packages. Optional: add `"env": { "NODE_OPTIONS": "--conditions development" }` to run from source instead (no build needed). |
Contributor
There was a problem hiding this comment.
Let's discuss this further, but my first takes on this:
- the description is too technical for a how to get started README.
- as this is the paved path section maybe we encourage npm with npx similar to PWA Kit MCP Setup
patricksullivansf
approved these changes
Feb 19, 2026
wei-liu-sf
approved these changes
Feb 20, 2026
clavery
approved these changes
Feb 20, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Update paved path documentation to use npx in mcp.json config.
Testing
How was this tested?
Manual Testing:
Dependencies
3pl-approvedis set by a maintainerpnpm test)pnpm run format)