@salesforce/b2c-dx-mcp 0.4.1 → 0.4.2

package/dist/commands/mcp.d.ts

@@ -51,8 +51,8 @@ export default class McpServerCommand extends BaseCommand<typeof McpServerComman
         'extra-body': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         'extra-headers': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         'api-key': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
-        project: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
-        environment: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        project: import("@oclif/core/interfaces").OptionFlag<string, import("@oclif/core/interfaces").CustomOptions>;
+        environment: import("@oclif/core/interfaces").OptionFlag<string, import("@oclif/core/interfaces").CustomOptions>;
         'cloud-origin': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         'credentials-file': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
     };

package/dist/tools/storefrontnext/index.js

@@ -23,7 +23,7 @@
 import { z } from 'zod';
 import { createToolAdapter, jsonResult } from '../adapter.js';
 import { createDeveloperGuidelinesTool } from './developer-guidelines.js';
-import { createPageDesignerDecoratorTool } from '../page-designer-decorator/index.js';
+import { createPageDesignerDecoratorTool } from './page-designer-decorator/index.js';
 /**
  * Creates a placeholder tool for Storefront Next development.
  *

package/dist/tools/{page-designer-decorator → storefrontnext/page-designer-decorator}/index.d.ts

@@ -1,6 +1,6 @@
 import { z } from 'zod';
-import type { McpTool } from '../../utils/index.js';
-import type { Services } from '../../services.js';
+import type { McpTool } from '../../../utils/index.js';
+import type { Services } from '../../../services.js';
 export declare const pageDesignerDecoratorSchema: z.ZodObject<{
     component: z.ZodString;
     searchPaths: z.ZodOptional<z.ZodArray<z.ZodString, "many">>;

package/dist/tools/{page-designer-decorator → storefrontnext/page-designer-decorator}/index.js

@@ -303,8 +303,49 @@ function handleConfigureRegionsStep(args, workspaceRoot) {
         ],
     };
 }
+function hasNonEmptyConfig(config) {
+    return config !== null && config !== undefined && Object.keys(config).length > 0;
+}
+function buildAttributesFromProps(selectedProps, props, attributeConfig) {
+    return selectedProps.flatMap((propName) => {
+        const prop = props.find((p) => p.name === propName);
+        if (!prop)
+            return [];
+        const config = attributeConfig[propName];
+        return [
+            {
+                name: propName,
+                tsType: prop.type,
+                optional: prop.optional,
+                hasConfig: hasNonEmptyConfig(config),
+                config,
+            },
+        ];
+    });
+}
+function buildAttributesFromNewAttrs(newAttributes, attributeConfig) {
+    return newAttributes.map((attr) => {
+        const config = attributeConfig[attr.name];
+        return {
+            name: attr.name,
+            tsType: 'string',
+            optional: !attr.required,
+            hasConfig: hasNonEmptyConfig(config),
+            config,
+        };
+    });
+}
+function resolveRegions(conversationContext) {
+    const regionConfig = conversationContext?.regionConfig;
+    const enabled = Boolean(regionConfig?.enabled && regionConfig.regions?.length);
+    return {
+        hasRegions: enabled,
+        regions: enabled ? regionConfig.regions : [],
+        regionCount: enabled ? regionConfig.regions.length : 0,
+    };
+}
 function handleConfirmGenerationStep(args, workspaceRoot) {
-    const { componentMetadata, selectedProps = [], newAttributes = [], attributeConfig = {}, } = args.conversationContext || {};
+    const { componentMetadata, selectedProps = [], newAttributes = [], attributeConfig = {}, } = args.conversationContext ?? {};
     if (!componentMetadata) {
         return {
             content: [
@@ -318,45 +359,23 @@ function handleConfirmGenerationStep(args, workspaceRoot) {
     }
     const fullPath = resolveComponent(args.component, workspaceRoot, args.searchPaths);
     const componentInfo = componentAnalyzer.analyzeComponent(fullPath);
-    const attributes = [];
-    for (const propName of selectedProps) {
-        const prop = componentInfo.props.find((p) => p.name === propName);
-        if (!prop)
-            continue;
-        const config = attributeConfig[propName];
-        const hasConfig = config && Object.keys(config).length > 0;
-        attributes.push({
-            name: propName,
-            tsType: prop.type,
-            optional: prop.optional,
-            hasConfig,
-            config,
-        });
-    }
-    for (const attr of newAttributes) {
-        const config = attributeConfig[attr.name];
-        const hasConfig = config && Object.keys(config).length > 0;
-        attributes.push({
-            name: attr.name,
-            tsType: 'string',
-            optional: !attr.required,
-            hasConfig,
-            config,
-        });
-    }
-    const regionConfig = args.conversationContext?.regionConfig;
-    const hasRegions = regionConfig?.enabled && regionConfig.regions && regionConfig.regions.length > 0;
+    const attributes = [
+        ...buildAttributesFromProps(selectedProps, componentInfo.props, attributeConfig),
+        ...buildAttributesFromNewAttrs(newAttributes, attributeConfig),
+    ];
+    const { hasRegions, regions, regionCount } = resolveRegions(args.conversationContext);
+    const componentGroup = componentMetadata.group ?? 'odyssey_base';
     const context = {
         needsImports: true,
         componentId: componentMetadata.id,
         componentName: componentMetadata.name,
         componentDescription: componentMetadata.description,
-        componentGroup: componentMetadata.group || 'odyssey_base',
+        componentGroup,
         metadataClassName: `${componentInfo.componentName}Metadata`,
         hasAttributes: attributes.length > 0,
-        hasRegions: hasRegions || false,
+        hasRegions,
         hasLoader: false,
-        regions: hasRegions ? regionConfig.regions || [] : [],
+        regions,
         attributes,
     };
     const decoratorCode = generateDecoratorCode(context);
@@ -364,11 +383,11 @@ function handleConfirmGenerationStep(args, workspaceRoot) {
         decoratorCode,
         componentName: componentInfo.componentName,
         componentId: componentMetadata.id,
-        componentGroup: componentMetadata.group || 'odyssey_base',
+        componentGroup,
         file: args.component,
         attributeCount: attributes.length,
-        hasRegions: hasRegions || false,
-        regionCount: hasRegions && regionConfig.regions ? regionConfig.regions.length : 0,
+        hasRegions,
+        regionCount,
     });
     return {
         content: [

package/oclif.manifest.json

@@ -387,5 +387,5 @@
       "enableJsonFlag": false
     }
   },
-  "version": "0.4.1"
+  "version": "0.4.2"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@salesforce/b2c-dx-mcp",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "description": "MCP server for B2C Commerce Cloud developer experience tools",
   "author": "Salesforce",
   "license": "MIT",
@@ -79,7 +79,7 @@
     "ts-morph": "^27.0.0",
     "yaml": "2.8.1",
     "zod": "3.25.76",
-    "@salesforce/b2c-tooling-sdk": "0.5.1"
+    "@salesforce/b2c-tooling-sdk": "0.5.2"
   },
   "devDependencies": {
     "@eslint/compat": "^1",