Merge origin/main into t/commerce/W-20590941/mcp-server

Author: yhsieh1

.github/workflows/ci.yml

@@ -8,6 +8,11 @@ on:
     branches:
       - main
 
+permissions:
+  contents: read
+  actions: read
+  checks: write
+
 jobs:
   test:
     runs-on: ubuntu-latest
@@ -49,5 +54,56 @@ jobs:
       - name: Build packages
         run: pnpm -r run build
 
-      - name: Run tests and lint
-        run: pnpm -r run test
+      - name: Run SDK tests
+        id: sdk-test
+        working-directory: packages/b2c-tooling-sdk
+        run: pnpm run test:ci && pnpm run lint
+
+      - name: Run CLI tests
+        id: cli-test
+        if: always() && steps.sdk-test.conclusion != 'cancelled'
+        working-directory: packages/b2c-cli
+        run: pnpm run test:ci && pnpm run lint
+
+      - name: Test Report
+        uses: dorny/test-reporter@fe45e9537387dac839af0d33ba56eed8e24189e8  # v2.3.0
+        if: always() && steps.sdk-test.conclusion != 'cancelled'
+        with:
+          name: Test Results (Node ${{ matrix.node-version }})
+          path: 'packages/*/test-results.json'
+          reporter: mocha-json
+
+      - name: Generate coverage summary
+        if: always() && steps.sdk-test.conclusion != 'cancelled'
+        run: |
+          echo "## Coverage (Node ${{ matrix.node-version }})" >> $GITHUB_STEP_SUMMARY
+          echo "" >> $GITHUB_STEP_SUMMARY
+
+          # SDK Coverage
+          if [ -f "packages/b2c-tooling-sdk/coverage/lcov.info" ]; then
+            echo "### @salesforce/b2c-tooling-sdk" >> $GITHUB_STEP_SUMMARY
+            echo '```' >> $GITHUB_STEP_SUMMARY
+            cd packages/b2c-tooling-sdk && pnpm c8 report --reporter=text-summary 2>/dev/null | tail -n 6 >> $GITHUB_STEP_SUMMARY
+            cd ../..
+            echo '```' >> $GITHUB_STEP_SUMMARY
+            echo "" >> $GITHUB_STEP_SUMMARY
+          fi
+
+          # CLI Coverage
+          if [ -f "packages/b2c-cli/coverage/lcov.info" ]; then
+            echo "### @salesforce/b2c-cli" >> $GITHUB_STEP_SUMMARY
+            echo '```' >> $GITHUB_STEP_SUMMARY
+            cd packages/b2c-cli && pnpm c8 report --reporter=text-summary 2>/dev/null | tail -n 6 >> $GITHUB_STEP_SUMMARY
+            cd ../..
+            echo '```' >> $GITHUB_STEP_SUMMARY
+          fi
+
+      - name: Upload coverage reports
+        if: always() && steps.sdk-test.conclusion != 'cancelled'
+        uses: actions/upload-artifact@v4
+        with:
+          name: coverage-reports-node-${{ matrix.node-version }}
+          path: |
+            packages/b2c-tooling-sdk/coverage/
+            packages/b2c-cli/coverage/
+          retention-days: 30

.gitignore

@@ -5,6 +5,7 @@
 dist
 /tmp
 node_modules
+/coverage
 oclif.manifest.json
 *.tsbuildinfo
 

AGENTS.md

@@ -79,3 +79,75 @@ Features:
 - Dynamic column widths based on content
 - Supports `extended` flag on columns for optional fields
 - Use `TableRenderer` class directly for column validation helpers (e.g., `--columns` flag support)
+
+## Testing
+
+Tests use Mocha + Chai with c8 for coverage. HTTP mocking uses MSW (Mock Service Worker).
+
+### Running Tests
+
+```bash
+# Run all tests with coverage
+pnpm run test
+
+# Run tests for specific package
+pnpm --filter @salesforce/b2c-tooling-sdk run test
+
+# Run single test file (no coverage, faster)
+cd packages/b2c-tooling-sdk
+pnpm mocha "test/clients/webdav.test.ts"
+
+# Run tests matching pattern
+pnpm mocha --grep "mkcol" "test/**/*.test.ts"
+
+# Watch mode for TDD
+pnpm --filter @salesforce/b2c-tooling-sdk run test:watch
+```
+
+### Writing Tests
+
+- Place tests in `packages/<package>/test/` mirroring the src structure
+- Use `.test.ts` suffix for test files
+- Import from package names, not relative paths:
+  ```typescript
+  // Good - uses package exports
+  import { WebDavClient } from '@salesforce/b2c-tooling-sdk/clients';
+
+  // Avoid - relative paths
+  import { WebDavClient } from '../../src/clients/webdav.js';
+  ```
+
+### HTTP Mocking with MSW
+
+For testing HTTP clients, use MSW to mock at the network level:
+
+```typescript
+import { http, HttpResponse } from 'msw';
+import { setupServer } from 'msw/node';
+
+const server = setupServer();
+
+before(() => server.listen({ onUnhandledRequest: 'error' }));
+afterEach(() => server.resetHandlers());
+after(() => server.close());
+
+it('makes HTTP request', async () => {
+  server.use(
+    http.get('https://example.com/api/*', () => {
+      return HttpResponse.json({ data: 'test' });
+    }),
+  );
+
+  // Test code that makes HTTP requests...
+});
+```
+
+### Test Helpers
+
+- `test/helpers/mock-auth.ts` - Mock AuthStrategy for testing HTTP clients
+
+### Coverage
+
+- Coverage reports generated in `coverage/` directory
+- SDK package has 5% minimum threshold (will increase as tests are added)
+- CI publishes coverage summary to GitHub Actions job summary

CONTRIBUTING.md

@@ -52,7 +52,6 @@ Issues labelled `good first contribution`.
 - [x] Tests
   - The test suite, if provided, must be complete and pass
   - Increase code coverage, not versa.
-  - Use any of our testkits that contains a bunch of testing facilities you would need. For example: `import com.salesforce.op.test._` and borrow inspiration from existing tests.
 - [x] Dependencies
   - Minimize number of dependencies.
   - Prefer Apache 2.0, BSD3, MIT, ISC and MPL licenses.

README.md

@@ -59,12 +59,42 @@ pnpm --filter @salesforce/b2c-cli run build
 pnpm --filter @salesforce/b2c-tooling-sdk run build
 ```
 
-### Testing and Linting
+### Testing
+
+Tests use [Mocha](https://mochajs.org/) + [Chai](https://www.chaijs.com/) with [c8](https://github.com/bcoe/c8) for coverage. HTTP mocking uses [MSW](https://mswjs.io/).
 
 ```bash
-# Run all tests (also runs linter after tests)
+# Run all tests with coverage (also runs linter after tests)
 pnpm test
 
+# Run tests for a specific package
+pnpm --filter @salesforce/b2c-tooling-sdk run test
+
+# Run tests without coverage (faster)
+pnpm --filter @salesforce/b2c-tooling-sdk run test:unit
+
+# Watch mode for TDD
+pnpm --filter @salesforce/b2c-tooling-sdk run test:watch
+
+# Run a specific test file
+cd packages/b2c-tooling-sdk
+pnpm mocha "test/clients/webdav.test.ts"
+
+# Run tests matching a pattern
+pnpm mocha --grep "uploads a file" "test/**/*.test.ts"
+```
+
+#### Coverage
+
+Coverage reports are generated in each package's `coverage/` directory:
+- `coverage/index.html` - HTML report
+- `coverage/lcov.info` - LCOV format for CI integration
+
+The SDK package has a 5% coverage threshold that will fail the build if not met.
+
+### Linting
+
+```bash
 # Run linter only
 pnpm --filter @salesforce/b2c-cli run lint
 pnpm --filter @salesforce/b2c-tooling-sdk run lint
@@ -142,3 +172,15 @@ import { getLogger } from '@salesforce/b2c-tooling-sdk/logging';
 ```
 
 The `development` condition in exports enables direct TypeScript source resolution when using `--conditions=development`, which is how `bin/dev.js` works for local development.
+
+## Contributing
+
+We welcome contributions! Please see our [Contributing Guide](./CONTRIBUTING.md) for details on how to get started, submit pull requests, and our code of conduct.
+
+## Security
+
+For security concerns, please review our [Security Policy](./SECURITY.md). Report any security issues to [security@salesforce.com](mailto:security@salesforce.com).
+
+## License
+
+This project is licensed under the Apache License 2.0. See [LICENSE.txt](./LICENSE.txt) for full details.

license.txt

@@ -191,7 +191,7 @@ All rights reserved.
       same "printed page" as the copyright notice for easier
       identification within third-party archives.
 
-   Copyright {yyyy} {name of copyright owner}
+   Copyright 2025 Salesforce, Inc.
 
    Licensed under the Apache License, Version 2.0 (the "License");
    you may not use this file except in compliance with the License.

package.json

@@ -6,6 +6,8 @@
   "scripts": {
     "start": "pnpm --filter @salesforce/b2c-cli run dev",
     "test": "pnpm -r test",
+    "test:unit": "pnpm -r run test:unit",
+    "coverage": "pnpm -r run coverage",
     "format": "pnpm -r run format",
     "lint": "pnpm -r run lint",
     "build": "pnpm -r run build",
@@ -16,7 +18,7 @@
   },
   "keywords": [],
   "author": "",
-  "license": "ISC",
+  "license": "Apache-2.0",
   "packageManager": "pnpm@10.17.1",
   "devDependencies": {
     "typedoc": "^0.28.14",

packages/b2c-cli/.c8rc.json

@@ -0,0 +1,10 @@
+{
+  "all": true,
+  "src": ["src"],
+  "exclude": [
+    "test/**",
+    "**/*.d.ts"
+  ],
+  "reporter": ["text", "text-summary", "html", "lcov"],
+  "report-dir": "coverage"
+}

packages/b2c-cli/.gitignore

@@ -6,6 +6,8 @@
 /build
 /tmp
 /node_modules
+/coverage
+test-results.json
 oclif.manifest.json
 
 

packages/b2c-cli/package.json

@@ -26,6 +26,7 @@
     "@types/chai": "^4",
     "@types/mocha": "^10",
     "@types/node": "^18",
+    "c8": "^10.1.3",
     "chai": "^4",
     "eslint": "^9",
     "eslint-config-oclif": "^6",
@@ -54,7 +55,7 @@
     "CLI",
     "Command Line Interface"
   ],
-  "license": "MIT",
+  "license": "Apache-2.0",
   "main": "dist/index.js",
   "type": "module",
   "oclif": {
@@ -116,7 +117,10 @@
     "posttest": "pnpm run lint",
     "prepack": "oclif manifest && oclif readme",
     "pretest": "tsc --noEmit -p test",
-    "test": "OCLIF_TEST_ROOT=. mocha --forbid-only \"test/**/*.test.ts\"",
+    "test": "c8 env OCLIF_TEST_ROOT=. mocha --forbid-only \"test/**/*.test.ts\"",
+    "test:ci": "c8 env OCLIF_TEST_ROOT=. mocha --forbid-only --reporter json --reporter-option output=test-results.json \"test/**/*.test.ts\"",
+    "test:unit": "env OCLIF_TEST_ROOT=. mocha --forbid-only \"test/**/*.test.ts\"",
+    "coverage": "c8 report",
     "version": "oclif readme && git add README.md",
     "dev": "node ./bin/dev.js"
   },