@W-18909894 - add cursor rules for creating components

.cursor/rules/storefront-development/component-generation-requirements.mdc

@@ -0,0 +1,29 @@
+---
+description: USE when creating new components
+globs: 
+alwaysApply: false
+---
+# Component Generation Requirements
+
+## Process
+1. **Preferred**
+ - Ask user to provide the component name, purpose and requirements.
+ - Determine the appropriate component type (presentational, container, form, etc.).
+ - Collected component information must be displayed and asked for confirmation.
+2. **Default to Simple** - Start with minimal implementation
+3. DO NOT add extra features unless asked
+
+## General Rules
+Create components at the components directory: `**/components/[component-name]/index.jsx`
+
+## Guidelines
+**IMPORTANT** 
+- Use functional components with hooks
+- Use PascalCase for component names
+- Use kebab-case for directories
+- Start simple, expand only if requested
+- One main purpose per component
+
+
+
+

packages/pwa-kit-create-app/assets/cursor-rules/component-generation-requirements.mdc

@@ -0,0 +1,29 @@
+---
+description: USE when creating new components
+globs: 
+alwaysApply: false
+---
+# Component Generation Requirements
+
+## Process
+1. **Preferred**
+ - Ask user to provide the component name, purpose and requirements.
+ - Determine the appropriate component type (presentational, container, form, etc.).
+ - Collected component information must be displayed and asked for confirmation.
+2. **Default to Simple** - Start with minimal implementation
+3. DO NOT add extra features unless asked
+
+## General Rules
+Create components at the components directory: `**/components/[component-name]/index.jsx`
+
+## Guidelines
+**IMPORTANT** 
+- Use functional components with hooks
+- Use PascalCase for component names
+- Use kebab-case for directories
+- Start simple, expand only if requested
+- One main purpose per component
+
+
+
+

packages/pwa-storefront-mcp/src/server/server.js

@@ -124,94 +124,84 @@ class PwaStorefrontMCPServerHighLevel {
     _handleComponentNameStep(session, answer, sessionId) {
         if (answer) {
             session.answers.name = answer
-            session.step = 2
-            const defaultDir = process.env.PWA_STOREFRONT_APP_PATH
-                ? process.env.PWA_STOREFRONT_APP_PATH + '/components'
-                : '/components'
-            return this._next(
-                sessionId,
-                `Answer yes to use the default components directory (${defaultDir}), or no if you want to specify the full absolute path to use a different directory:`
-            )
+
+            // If PWA_STOREFRONT_APP_PATH is defined, automatically set location and go to step 3
+            if (process.env.PWA_STOREFRONT_APP_PATH) {
+                session.answers.location = process.env.PWA_STOREFRONT_APP_PATH + '/components'
+                session.step = 3
+                return this._next(
+                    sessionId,
+                    'Should this component display a single product, a list of products, or do you want to handle it manually? Reply with "single", "list", or "other".'
+                )
+            } else {
+                session.step = 2
+                return this._next(
+                    sessionId,
+                    'What should be the directory where the component should be created? Please provide the full absolute path.'
+                )
+            }
         }
         return this._next(sessionId, 'What would you like to name your new React component?')
     }
 
     _handleDirectoryStep(session, answer, sessionId) {
-        const defaultDir = process.env.PWA_STOREFRONT_APP_PATH
-            ? process.env.PWA_STOREFRONT_APP_PATH + '/components'
-            : '/components'
         if (answer) {
-            if (/^(yes|y|true|1)$/i.test(answer)) {
-                session.answers.location = defaultDir
-            } else {
-                session.answers.location = answer
-            }
+            session.answers.location = answer
             session.step = 3
             return this._next(
                 sessionId,
-                'Should this component display a single product or a list of products? Reply with "single" or "list".'
+                'Should this component display a single product, a list of products, or do you want to handle it manually? Reply with "single", "list", or "other".'
             )
         }
         return this._next(
             sessionId,
-            `Answer yes to use the default components directory (${defaultDir}), or no if you want to specify the full absolute path to use a different directory:`
+            'What should be the directory where the component should be created? Please provide the full absolute path.'
         )
     }
 
     async _handleSingleOrListStep(session, answer, sessionId) {
-        if (answer && /list/i.test(answer)) {
-            // List of products
-            const tool = new CreateNewComponentTool()
-            tool.componentData = {
-                name: session.answers.name,
-                location: session.answers.location,
-                createTestFile: false,
-                customCode: '',
-                entityType: 'product'
-            }
-            const dataModel = this.getDataModel('product')
-            let schemaObj = dataModel && dataModel.properties ? dataModel.properties : {}
-            let presentationalResult = await tool.updateComponentToPresentational(
-                'product',
-                session.answers.name,
-                session.answers.location,
-                schemaObj,
-                {list: true}
-            )
+        if (answer && /other/i.test(answer)) {
             session.step = 99
             return this._done(
                 sessionId,
-                (session.basicComponentResult || '') +
-                    `\n\n${presentationalResult}\nComponent creation flow complete.`
+                'Manual mode selected. Please proceed with manual code generation.'
             )
+        }
+        let isList = null
+        if (answer && /list/i.test(answer)) {
+            isList = true
         } else if (answer && /single/i.test(answer)) {
-            // Single product
-            const tool = new CreateNewComponentTool()
-            tool.componentData = {
-                name: session.answers.name,
-                location: session.answers.location,
-                createTestFile: false,
-                customCode: '',
-                entityType: 'product'
-            }
-            const dataModel = this.getDataModel('product')
-            let schemaObj = dataModel && dataModel.properties ? dataModel.properties : {}
-            let presentationalResult = await tool.updateComponentToPresentational(
-                'product',
-                session.answers.name,
-                session.answers.location,
-                schemaObj,
-                {list: false}
-            )
-            session.step = 99
-            return this._done(
+            isList = false
+        } else {
+            return this._next(
                 sessionId,
-                (session.basicComponentResult || '') +
-                    `\n\n${presentationalResult}\nComponent creation flow complete.`
+                'Please reply with "single", "list", or "other".\nNote: This tool only supports generating single or list "Product" components. For other requirements, select "other".'
             )
-        } else {
-            return this._next(sessionId, 'Please reply with "single" or "list".')
         }
+
+        const tool = new CreateNewComponentTool()
+        tool.componentData = {
+            name: session.answers.name,
+            location: session.answers.location,
+            createTestFile: false,
+            customCode: '',
+            entityType: 'product'
+        }
+        const dataModel = this.getDataModel('product')
+        let schemaObj = dataModel && dataModel.properties ? dataModel.properties : {}
+        let presentationalResult = await tool.updateComponentToPresentational(
+            'product',
+            session.answers.name,
+            session.answers.location,
+            schemaObj,
+            {list: isList}
+        )
+        session.step = 99
+        return this._done(
+            sessionId,
+            (session.basicComponentResult || '') +
+                `\n\n${presentationalResult}\nComponent creation flow complete.`
+        )
     }
 
     _handleDoneStep(sessionId) {