update feature branch with develop

.cursor/qa/rules/packages/template-retail-react-app/testing/retail-react-app-test-patterns.mdc

@@ -0,0 +1,59 @@
+---
+description: QA Tests for Template Retail React App Test Generation Rules
+globs: ["packages/template-retail-react-app/app/components/**/*-underTest.test.{js,jsx}"]
+alwaysApply: false
+---
+# QA Tests for Template Retail React App Test Patterns
+
+## Purpose
+This file contains test cases to verify that the `unit-tests-template-retail-react-app.mdc` rule effectively guides the creation of consistent and robust React component tests.
+
+## Test Cases
+
+### Test 1: DrawerMenu Component Test Generation
+**Workflow:** Generate → Analyze → Cleanup (NO test execution)
+
+**Steps:**
+1. **Generate:** Create new test file `drawer-menu-underTest.test.js` (colocated with existing drawer-menu component at `packages/template-retail-react-app/app/components/drawer-menu/`)
+2. **Apply Rules:** Use `@/testing` (located at `.cursor/rules/testing/unit-tests-generic.mdc` and `.cursor/rules/testing/unit-tests-template-retail-react-app.mdc`)
+3. **Prompt:** "Write unit test for drawer-menu component"
+4. **Analyze:** Perform static code analysis against verification patterns (see below)
+5. **Cleanup:** Delete the generated test file after validation
+
+**Important:** DO NOT run the test after creation - skip test execution entirely
+
+**Verify that the newly generated test file follows these patterns:**
+1. Test Setup
+   - Uses `renderWithProviders` from `@salesforce/retail-react-app/app/utils/test-utils`
+   - Gets user events from return value: `const {user} = renderWithProviders(...)`
+   - Includes `beforeEach(() => jest.clearAllMocks())`
+
+2. Import Structure
+   - Does NOT import `userEvent` directly
+   - Uses existing mock data from `@salesforce/retail-react-app/app/mocks/`
+   - Imports `screen` from `@testing-library/react`
+
+3. Test Organization
+   - Uses `describe` block with component name
+   - Individual `test` or `it` blocks for different scenarios
+   - Async/await patterns for user interactions
+
+4. API Mocking
+   - Uses `prependHandlersToServer` or `msw` for API mocking when needed
+
+**Failure Indicators:**
+- Direct import of `userEvent` from `@testing-library/user-event`
+- Using custom render function instead of `renderWithProviders`
+- Creating new mock data instead of using existing mocks
+- Missing Commerce SDK context providers
+- Not using async/await for user interactions
+
+**Expected Output:**
+Provide a clear, structured report based on **static code analysis only** (no test execution):
+- ✅ **PASS** or ❌ **FAIL** for each verification point
+- Specific line numbers and code snippets for any failures
+- Summary: `X/Y patterns followed correctly`
+- Overall result: **RULES EFFECTIVE** or **RULES NEED IMPROVEMENT**
+
+**Cleanup:**
+- Delete the generated test file `drawer-menu-underTest.test.js` after QA validation is complete

.cursor/rules/cursor-rule.mdc

@@ -0,0 +1,66 @@
+---
+description: How to add or edit Cursor rules in our project
+globs: 
+alwaysApply: false
+---
+# Cursor Rules Location
+
+How to add new cursor rules to the project
+
+1. Always place rule files in PROJECT_ROOT/.cursor/rules/:
+    ```
+    .cursor/rules/
+    ├── your-rule-name.mdc
+    ├── another-rule.mdc
+    └── ...
+    ```
+
+2. Follow the naming convention:
+    - Use kebab-case for filenames
+    - Always use .mdc extension
+    - Make names descriptive of the rule's purpose
+
+3. Directory structure:
+    ```
+    PROJECT_ROOT/
+    ├── .cursor/
+    │   └── rules/
+    │       ├── your-rule-name.mdc
+    │       └── ...
+    └── ...
+    ```
+
+4. Never place rule files:
+    - In the project root
+    - In subdirectories outside .cursor/rules
+    - In any other location
+
+5. Cursor rules have the following structure:
+
+````
+---
+description: Short description of the rule's purpose
+globs: optional/path/pattern/**/* 
+alwaysApply: false
+---
+# Rule Title
+
+Main content explaining the rule with markdown formatting.
+
+1. Step-by-step instructions
+2. Code examples
+3. Guidelines
+
+Example:
+```typescript
+// Good example
+function goodExample() {
+  // Implementation following guidelines
+}
+
+// Bad example
+function badExample() {
+  // Implementation not following guidelines
+}
+```
+````

.cursor/rules/testing/unit-tests-generic.mdc

@@ -0,0 +1,57 @@
+---
+description: USE WHEN writing unit tests for components in template packages
+globs: ["packages/template-*/*/components/**/*.test.{js,jsx,ts,tsx}"]
+alwaysApply: false
+---
+USE WHEN writing unit tests for components in template packages
+
+# 🧪 Generic Component Test Rules
+
+## Structure & Best Practices
+- Use `describe` blocks to group tests, `test` for individual cases
+- Use `beforeEach` for setup, clear mocks after each test
+- **Arrange** → **Act** → **Assert** pattern
+- One behavior per test, clear descriptive names
+
+## Queries & Assertions
+- Prefer `getByRole`, `getByLabelText`, `getByTestId`
+- Use `expect().toBeInTheDocument()`, `.toHaveBeenCalledTimes()`, etc.
+- For async: `await waitFor(() => { ... })`
+
+## Mocking
+- `jest.fn()` for handlers, `jest.mock()` for modules
+- Clear mocks/storage after each test
+
+```js
+describe('MyComponent', () => {
+    beforeEach(() => jest.clearAllMocks())
+
+    test('renders and handles interaction', async () => {
+        const mockHandler = jest.fn()
+        render(<MyComponent onClick={mockHandler} />)
+
+        await userEvent.click(screen.getByRole('button'))
+        expect(mockHandler).toHaveBeenCalledTimes(1)
+    })
+})
+```
+
+## Running Tests
+After creating unit tests, **ALWAYS run the tests** to verify they pass and provide feedback on test results.
+
+### Command Format:
+```bash
+cd packages/<package-name> && npm run test -- '<relative-path-to-test-file> --coverage=false'
+```
+
+### Examples:
+```bash
+# Run specific test file from packages directory
+cd packages/template-retail-react-app && npm run test -- 'app/components/drawer-menu/drawer-menu.test.js --coverage=false'
+```
+
+### After Running Tests:
+- Report if tests **pass** or **fail**
+- If tests fail, provide the error messages and fix any issues
+- Confirm test coverage is appropriate for the component's core functionality
+- Suggest any additional tests if critical functionality is missing

.cursor/rules/testing/unit-tests-template-retail-react-app.mdc

@@ -0,0 +1,35 @@
+---
+description: USE WHEN writing unit tests in template-retail-react-app components
+globs: ["packages/template-retail-react-app/app/components/**/*.test.{js,jsx,ts,tsx}"]
+alwaysApply: false
+---
+# 🛍️ Retail React App Test Rules
+
+## Package-Specific Requirements
+- **File naming**: `index.test.js` (colocated with component)
+- **Always use `renderWithProviders`** (provides Commerce SDK context)
+- **Get user events from return value**: `const {user} = renderWithProviders(...)`
+- **Do NOT import `userEvent` directly**
+
+## API Mocking
+- Use `prependHandlersToServer` or `msw` for API mocking
+
+## Mock Data Usage
+
+- **Mandatory**: Always use existing mock data from `@salesforce/retail-react-app/app/mocks/` if it is available. This ensures consistency across tests and reduces redundancy. Creating new mock data should only be considered if the required data is not already present in the mocks directory.
+
+```js
+import {screen} from '@testing-library/react'
+import {renderWithProviders} from '@salesforce/retail-react-app/app/utils/test-utils'
+import MyComponent from '.'
+
+describe('MyComponent', () => {
+    beforeEach(() => jest.clearAllMocks())
+
+    test('handles user interaction', async () => {
+        const {user} = renderWithProviders(<MyComponent />)
+        await user.click(screen.getByText('Click Me'))
+        expect(screen.getByText('Expected')).toBeInTheDocument()
+    })
+})
+```

.github/CODEOWNERS

@@ -1,9 +1,73 @@
 # https://help.github.com/en/enterprise/2.17/user/articles/about-code-owners
 
-# These owners will be the default owners for everything in
-# the repo. Unless a later match takes precedence,
-# @global-owner1 and @global-owner2 will be requested for
-# review when someone opens a pull request.
+# Global directories
+.cursor/                                                    @SalesforceCommerceCloud/mobifyers
+.github/                                                    @SalesforceCommerceCloud/mobifyers
+e2e/                                                        @SalesforceCommerceCloud/mobifyers
+scripts/                                                    @SalesforceCommerceCloud/mobifyers
+
+# Package-specific ownership
+packages/commerce-sdk-react/                               @SalesforceCommerceCloud/mobifyers
+packages/internal-lib-build/                               @SalesforceCommerceCloud/mobifyers
+packages/pwa-kit-create-app/                               @SalesforceCommerceCloud/mobifyers
+packages/pwa-kit-dev/                                      @SalesforceCommerceCloud/mobifyers
+packages/pwa-kit-mcp/                                      @SalesforceCommerceCloud/mobifyers
+packages/pwa-kit-react-sdk/                                @SalesforceCommerceCloud/mobifyers
+packages/pwa-kit-runtime/                                  @SalesforceCommerceCloud/mobifyers
+packages/template-express-minimal/                         @SalesforceCommerceCloud/mobifyers
+packages/template-mrt-reference-app/                       @SalesforceCommerceCloud/mobifyers
+packages/template-retail-react-app/                        @SalesforceCommerceCloud/mobifyers
+packages/template-typescript-minimal/                      @SalesforceCommerceCloud/mobifyers
+packages/test-commerce-sdk-react/                          @SalesforceCommerceCloud/mobifyers
+
+# Root configuration and documentation files
+/README.md                                                  @SalesforceCommerceCloud/mobifyers
+/CODE_OF_CONDUCT.md                                         @SalesforceCommerceCloud/mobifyers
+/CONTRIBUTING.md                                            @SalesforceCommerceCloud/mobifyers
+/SECURITY.md                                                @SalesforceCommerceCloud/mobifyers
+/STATEMENTS.md                                              @SalesforceCommerceCloud/mobifyers
+/TERMS_OF_USE.md                                            @SalesforceCommerceCloud/mobifyers
+/LICENSE                                                    @SalesforceCommerceCloud/mobifyers
+/.eslintrc.js                                              @SalesforceCommerceCloud/mobifyers
+/.gitattributes                                            @SalesforceCommerceCloud/mobifyers
+/.gitignore                                                @SalesforceCommerceCloud/mobifyers
+/.prettierignore                                           @SalesforceCommerceCloud/mobifyers
+/.prettierrc.yaml                                          @SalesforceCommerceCloud/mobifyers
+/lerna.json                                                @SalesforceCommerceCloud/mobifyers
+/package.json                                              @SalesforceCommerceCloud/mobifyers
+/package-lock.json                                         @SalesforceCommerceCloud/mobifyers
+/playwright.config.js                                      @SalesforceCommerceCloud/mobifyers
+/.git2gus/                                                 @SalesforceCommerceCloud/mobifyers
+
+# Specific feature file ownership (overrides package-level rules above)
+
+# BOPIS (Buy Online Pick up In Store) feature files - PR #2646
+packages/template-retail-react-app/app/components/store-display/ @SalesforceCommerceCloud/cc-spark
+packages/template-retail-react-app/app/hooks/use-selected-store.js @SalesforceCommerceCloud/cc-spark
+packages/template-retail-react-app/app/hooks/use-pickup-shipment.js @SalesforceCommerceCloud/cc-spark
+packages/template-retail-react-app/app/hooks/use-pickup-shipment.test.js @SalesforceCommerceCloud/cc-spark
+e2e/tests/desktop/bopis.spec.js @SalesforceCommerceCloud/cc-spark
+
+# Bonus Products feature files - PR #2704
+packages/template-retail-react-app/app/components/product-item/bonus-product-quantity.jsx @SalesforceCommerceCloud/cc-sharks
+packages/template-retail-react-app/app/components/product-item/bonus-product-quantity.test.jsx @SalesforceCommerceCloud/cc-sharks
+packages/template-retail-react-app/app/components/product-item/product-quantity-picker.jsx @SalesforceCommerceCloud/cc-sharks
+packages/template-retail-react-app/app/components/product-item/product-quantity-picker.test.jsx @SalesforceCommerceCloud/cc-sharks
+packages/template-retail-react-app/app/pages/cart/partials/bonus-products-title.jsx @SalesforceCommerceCloud/cc-sharks
+packages/template-retail-react-app/app/pages/cart/partials/bonus-products-title.test.js @SalesforceCommerceCloud/cc-sharks
+
+# Standard Products feature files - PR #2697
+packages/template-retail-react-app/app/mocks/standard-product.js @SalesforceCommerceCloud/cc-sharks
+packages/template-retail-react-app/app/utils/add-to-cart-utils.js @SalesforceCommerceCloud/cc-sharks
+
+# One-Click Checkout feature files
+packages/template-retail-react-app/app/pages/checkout-one-click/ @SalesforceCommerceCloud/stretch-armstrong
+packages/template-retail-react-app/app/components/otp-auth/ @SalesforceCommerceCloud/stretch-armstrong
+packages/template-retail-react-app/app/hooks/use-checkout-auto-select.js @SalesforceCommerceCloud/stretch-armstrong
+packages/template-retail-react-app/app/hooks/use-checkout-auto-select.test.js @SalesforceCommerceCloud/stretch-armstrong
+packages/template-retail-react-app/app/hooks/use-countdown.js @SalesforceCommerceCloud/stretch-armstrong
+packages/template-retail-react-app/app/hooks/use-countdown.test.js @SalesforceCommerceCloud/stretch-armstrong
+packages/template-retail-react-app/app/hooks/use-otp-inputs.js @SalesforceCommerceCloud/stretch-armstrong
+packages/template-retail-react-app/app/hooks/use-otp-inputs.test.js @SalesforceCommerceCloud/stretch-armstrong
 
-*       @SalesforceCommerceCloud/mobifyers
 #ECCN:Open Source

.github/actions/bundle_size_test/action.yml

@@ -1,4 +1,5 @@
 name: bundlesize
+description: Check bundle size against maximum file size limits
 runs:
   using: composite
   steps:

.github/actions/check_clean/action.yml

@@ -1,4 +1,5 @@
 name: check_clean
+description: Check if repository has uncommitted changes
 runs:
   using: composite
   steps:

.github/actions/check_if_latest_release/action.yml

@@ -1,4 +1,5 @@
 name: check_if_latest_release
+description: Check if the current release is the latest release
 inputs:
   token:
     description: "Github API token"

.github/actions/count_deps/action.yml

@@ -1,12 +1,17 @@
 name: count_deps
+description: Count and validate project dependencies
+inputs:
+  project_dir:
+    description: 'Path to the project directory'
+    required: true
 runs:
   using: composite
   steps:
   - name: Count Generated Project Dependencies
     # TODO: Can TOTAL_PACKAGES be exported in a cleaner way?
     run: |-
       MAX_PACKAGES="2260"
-      total=$(./scripts/count-dependencies.js generated-${{ matrix.template }})
+      total=$(./scripts/count-dependencies.js "${{ inputs.project_dir }}")
       echo "TOTAL_PACKAGES=${total}" >> $GITHUB_ENV
 
       if [ "$total" -gt "$MAX_PACKAGES" ]; then

.github/actions/create_mrt/action.yml

@@ -1,4 +1,5 @@
 name: create_mrt
+description: Create MRT credentials file
 inputs:
   mobify_user:
     description: "Mobify user email"

.github/actions/datadog/action.yml

@@ -1,4 +1,5 @@
 name: datadog
+description: Send metrics to Datadog
 inputs:
   datadog_api_key:
     description: "Datadog API key"

.github/actions/e2e_acquire_mrt_target/action.yml

@@ -0,0 +1,36 @@
+name: e2e_acquire_mrt_target
+description: Acquire MRT Target from pool of available targets in the MRT staging org.
+inputs:
+    BRANCH:
+        description: 'GitHub branch name from workflow invoking this action'
+        required: false
+    RUN_ID:
+        description: 'GitHub workflow run ID from workflow invoking this action'
+        required: true
+    PR_NUMBER:
+        description: 'GitHub PR number from workflow invoking this action'
+        required: false
+    MAX_RETRIES:
+        description: 'Maximum retry attempts to acquire MRT target'
+        required: false
+        default: '3'
+    RETRY_DELAY:
+        description: 'Delay between retries in milliseconds'
+        required: false
+        default: '10000'
+
+runs:
+    using: composite
+    steps:
+        - name: Acquire MRT Target
+          id: acquire_mrt_target
+          shell: bash
+          run: |-
+              set -e
+              cmd="node e2e/scripts/mrt-target-manager.js acquire --run-id ${{inputs.RUN_ID}} --max-retries ${{inputs.MAX_RETRIES}} --retry-delay ${{inputs.RETRY_DELAY}}"
+              if [ -n "${{inputs.PR_NUMBER}}" ]; then
+                cmd="$cmd --pr-number ${{inputs.PR_NUMBER}}"
+              elif [ -n "${{inputs.BRANCH}}" ]; then
+                cmd="$cmd --branch ${{inputs.BRANCH}}"
+              fi
+              eval $cmd