Commerce sdk react 3.4.1 preview.0

.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,64 @@
 # 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
+
 
-*       @SalesforceCommerceCloud/mobifyers
 #ECCN:Open Source

.github/actions/changelog-check/action.yml

@@ -0,0 +1,68 @@
+name: 'Changelog Check'
+description: 'Check if changelog is updated for the changed packages'
+inputs:
+  pr_number:
+    description: 'Pull request number'
+    required: true
+
+runs:
+  using: 'composite'
+  steps:
+    - name: Checkout code
+      uses: actions/checkout@v4
+      with:
+        fetch-depth: 0  # Fetch full history to access all commits
+
+    - name: Determine Base SHA and Merge Base
+      id: determine_base
+      run: |
+        if [ -n "${{ github.event.pull_request.base.sha }}" ]; then
+          BASE_SHA="${{ github.event.pull_request.base.sha }}"
+          MERGE_BASE=$(git merge-base $BASE_SHA ${{ github.sha }})
+          echo "BASE_SHA=$BASE_SHA" >> $GITHUB_ENV
+          echo "MERGE_BASE=$MERGE_BASE" >> $GITHUB_ENV
+        else
+          echo "Not running in a PR context or unable to determine base SHA. Skipping changelog check."
+          echo "SKIP_CHANGELOG_CHECK=true" >> $GITHUB_ENV
+        fi
+      shell: bash
+
+    - name: Check if 'skip changelog' label is present
+      id: check_labels
+      run: |
+        SKIP_CHANGELOG_LABEL="${{ contains(github.event.pull_request.labels.*.name, 'skip changelog') }}"
+        if [ "$SKIP_CHANGELOG_LABEL" = "true" ]; then
+          echo "Skip changelog label is present. Skipping changelog check."
+          echo "SKIP_CHANGELOG_CHECK=true" >> $GITHUB_ENV
+        fi
+      shell: bash
+
+    - name: Debugging Info
+      run: |
+        echo "Base SHA: ${{ env.BASE_SHA }}"
+        echo "Current SHA: ${{ github.sha }}"
+        echo "Merge Base: ${{ env.MERGE_BASE }}"
+        echo "SKIP_CHANGELOG_CHECK: ${{ env.SKIP_CHANGELOG_CHECK }}"
+      shell: bash
+
+    - name: Check if changelog is updated
+      if: ${{ env.SKIP_CHANGELOG_CHECK != 'true' }}
+      run: |
+        CHANGED_FILES=$(git diff --name-only ${{ env.MERGE_BASE }} ${{ github.sha }})
+        if [ -z "$CHANGED_FILES" ]; then
+          echo "No changed files detected."
+          exit 0
+        fi
+        echo "Changed files: $CHANGED_FILES"
+
+        PUBLIC_PACKAGES=("commerce-sdk-react" "pwa-kit-create-app" "pwa-kit-dev" "pwa-kit-react-sdk" "pwa-kit-runtime" "template-retail-react-app")
+
+        for PACKAGE in "${PUBLIC_PACKAGES[@]}"; do
+          if echo "$CHANGED_FILES" | grep -iq "^packages/$PACKAGE/"; then
+            if ! echo "$CHANGED_FILES" | grep -iq "^packages/$PACKAGE/CHANGELOG.md"; then
+              echo "CHANGELOG.md was not updated for package $PACKAGE. Please update the CHANGELOG.md or add 'skip changelog' label to the PR."
+              exit 1
+            fi
+          fi
+        done
+      shell: bash

.github/actions/check_if_latest_release/action.yml

@@ -0,0 +1,26 @@
+name: check_if_latest_release
+inputs:
+  token:
+    description: "Github API token"
+runs:
+  using: composite
+  steps:
+    - name: Check if latest release
+      run: |-
+        RELEASE_JSON=$(curl -s -H "Authorization: token ${{ inputs.token }}" \
+          "https://api.github.com/repos/${{ github.repository }}/releases/latest")
+
+        LATEST_TAG_NAME=$(echo "$RELEASE_JSON" | jq -r '.tag_name')
+        RELEASE_EVENT_TAG_NAME=$(echo ${{ github.event.release.tag_name }})
+
+        echo "Latest tag: $LATEST_TAG_NAME"
+        echo "Release event tag: $RELEASE_EVENT_TAG_NAME"
+
+        if [ "$LATEST_TAG_NAME" == "$RELEASE_EVENT_TAG_NAME" ]; then
+          echo "We are releasing the latest tag."
+          echo "IS_LATEST_RELEASE=true" >> $GITHUB_ENV
+        else
+          echo "We are not releasing the latest tag."
+          echo "IS_LATEST_RELEASE=false" >> $GITHUB_ENV
+        fi
+      shell: bash