Conversation
|
Will the source markdown files be accessible in the built npm package? You may want to look at the SDK and how we store markdown assets. |
|
@clavery Thanks for catching this. |
| }); | ||
| ``` | ||
|
|
||
| 3. **Override via environment variables**: |
There was a problem hiding this comment.
should we mention where to override vars?
|
|
||
| ```bash | ||
| # Environment variable → Config path | ||
| PUBLIC__app__site__locale=en-US → config.app.site.locale |
There was a problem hiding this comment.
This is no longer used, being replaced by PUBLIC__app__commerce__sites
There was a problem hiding this comment.
Thank you! My branch is behind a few commits. I will update it. We will need to have final update and review to match up latest implementation when release.
| ```bash | ||
| # Environment variable → Config path | ||
| PUBLIC__app__site__locale=en-US → config.app.site.locale | ||
| PUBLIC__app__site__currency=EUR → config.app.site.currency |
| Rules: | ||
| 1. **`PUBLIC__` prefix**: Exposed to browser (client-safe values) | ||
| 2. **No prefix**: Server-only (secrets, never exposed) | ||
| 3. **`__` separator**: Navigate nested paths (`PUBLIC__app__site__locale`) |
| ## Structure | ||
|
|
||
| ``` | ||
| src/extensions/my-extension/ |
There was a problem hiding this comment.
Do you consider moving these md files to storefront-next's template instead of mcp? MCP can still find them through app dir configured.
There was a problem hiding this comment.
These are short version of the long README-XX.md on storefront-next's template. The purpose is to give a quick over view of the important feature/architecture. Each of these md also provides a reference to the detailed README-XX.md for quick reference. It still have values to keep them but as agentic IDE progresses this might change.
…h Prompting Tips and Configuration sections
Summary
This PR adds the
storefront_next_development_guidelinesMCP tool with multi-section selection support, allowing developers to retrieve multiple related documentation sections in a single call. This improves contextual learning by combining related topics (e.g., authentication + components + data-fetching) rather than requiring separate tool invocations.Changes
Available Sections
The tool supports 12 documentation sections:
Test Plan
Prerequisites
MCP Configuration:
{ "mcpServers": { "b2c-dx": { "command": "/path/to/packages/b2c-dx-mcp/bin/dev.js", "args": [ "--working-directory", "/path/to/storefront-next-project", "--toolsets", "STOREFRONTNEXT", "--allow-non-ga-tools" ] } } }1. Default Behavior
Test 1.1: No parameters
Prompt: "I'm new to Storefront Next. Use the MCP tool to show me the critical rules I need to know."
Expected: Returns
quick-referencesection (default), no---separatorVerify:
storefront_next_development_guidelinesis calledTest 1.2: Single section
Prompt: "Use the MCP tool to show me the data fetching patterns for Storefront Next."
Expected: Returns
data-fetchingsection only, no---separatorVerify:
storefront_next_development_guidelinesis called2. Multi-Section Feature
Test 2.1: Multiple sections
Prompt: "I need to build a product detail page. Use the MCP tool to show me best practices for data fetching and component patterns."
Expected: Returns
data-fetching,components, andperformancesectionsVerify:
storefront_next_development_guidelinesis called---separators between sectionsTest 2.2: Related topics
Prompt: "I need to build a checkout form with authentication and validation. Use the MCP tool to show me how to handle form submissions, authentication, and internationalized error messages."
Expected: Returns
data-fetching,auth, andi18nsectionsVerify:
storefront_next_development_guidelinesis called3. Edge Cases
Test 3.1: Invalid section
Prompt: "Use the MCP tool to show me the guidelines for invalid-section-name."
Expected: Falls back to
quick-reference(default), no errorVerify:
storefront_next_development_guidelinesis calledTest 3.2: All sections
Prompt: "Show me all available Storefront Next development guidelines."
Expected: Returns all 12 sections
Verify:
storefront_next_development_guidelinesis called---separatorspnpm test)pnpm run format)