cursor-quality-suite 1.0.0

package/commands/testing/risk-test-gen.md

@@ -0,0 +1,232 @@
+---
+description: Generate integration/contract tests for high-risk code paths
+category: Testing
+aliases: [gen-tests, risk-tests, test-risky]
+---
+
+# Risk Test Gen - Generate Tests for High-Risk Paths
+
+Analyze code risk and generate deep integration/contract tests for critical paths.
+
+## Usage
+
+```
+/risk-test-gen {FILE_PATH}
+/risk-test-gen {PR_NUMBER}
+/risk-test-gen --domain payments     # Focus on domain
+/risk-test-gen --coverage-gap        # Find untested high-risk paths
+```
+
+## What This Does
+
+1. **Identifies high-risk paths** - Payments, auth, data mutations
+2. **Analyzes existing coverage** - Finds gaps in critical areas
+3. **Generates deep tests** - Integration, contract, edge cases
+4. **Prioritizes by impact** - Business-critical first
+
+## Risk Classification
+
+| Risk Level  | Domains                      | Test Depth             |
+| ----------- | ---------------------------- | ---------------------- |
+| 🔴 Critical | Payments, Auth, PII          | Integration + Contract |
+| 🟠 High     | Booking mutations, API calls | Integration            |
+| 🟡 Medium   | State management, Forms      | Unit + Edge cases      |
+| 🟢 Low      | UI rendering, Styling        | Snapshot only          |
+
+## Risk Detection Patterns
+
+```typescript
+// 🔴 CRITICAL - Payment processing
+const processPayment = async (paymentData: IPaymentData) => { ... }
+
+// 🔴 CRITICAL - Authentication
+const validateToken = (token: string) => { ... }
+
+// 🔴 CRITICAL - PII handling
+const updateUserProfile = (userData: IUserData) => { ... }
+
+// 🟠 HIGH - Booking mutations
+const createBooking = async (bookingData: IBooking) => { ... }
+
+// 🟠 HIGH - External API calls
+const fetchFromBackend = async (endpoint: string) => { ... }
+```
+
+## Output Format
+
+````
+📋 Analyzing risk and generating tests for PackagesV2.tsx...
+
+════════════════════════════════════════════════════════════════
+  RISK ANALYSIS
+════════════════════════════════════════════════════════════════
+
+## File: src/features/checkout/src/components/CoverageAndAddOns/PackagesV2.tsx
+
+### Detected Risk Areas
+
+| Line | Risk | Type | Coverage |
+|------|------|------|----------|
+| 45 | 🟠 HIGH | API call (useBookingWithFlow) | ❌ Untested |
+| 78 | 🟠 HIGH | State mutation (onPackageSelect) | ✅ Partial |
+| 112 | 🟡 MEDIUM | Price calculation | ❌ Untested |
+| 156 | 🔴 CRITICAL | Booking modification | ❌ Untested |
+
+### Current Coverage
+- Lines: 45%
+- Branches: 32%
+- High-risk paths: 20% ⚠️
+
+════════════════════════════════════════════════════════════════
+  GENERATED TESTS
+════════════════════════════════════════════════════════════════
+
+## 1. Integration Test: Booking Modification (CRITICAL)
+
+```typescript
+describe('PackagesV2 - Booking Modification', () => {
+    const mockBooking = createMockBooking({
+        packages: [{ id: 'pkg-1', name: 'Basic' }]
+    });
+
+    it('should update booking when package is selected', async () => {
+        const onPackageSelect = jest.fn();
+
+        render(
+            <BookingProvider booking={mockBooking}>
+                <PackagesV2 onPackageSelect={onPackageSelect} />
+            </BookingProvider>
+        );
+
+        await userEvent.click(screen.getByTestId('package-premium'));
+
+        expect(onPackageSelect).toHaveBeenCalledWith(
+            expect.objectContaining({
+                id: 'pkg-premium',
+                price: expect.any(Number)
+            })
+        );
+    });
+
+    it('should handle API failure gracefully', async () => {
+        server.use(
+            rest.post('/api/booking/update', (req, res, ctx) =>
+                res(ctx.status(500))
+            )
+        );
+
+        render(<PackagesV2 booking={mockBooking} />);
+        await userEvent.click(screen.getByTestId('package-premium'));
+
+        expect(screen.getByText(/error/i)).toBeInTheDocument();
+        expect(mockBooking.packages).toEqual([{ id: 'pkg-1' }]); // Unchanged
+    });
+
+    it('should prevent double-submission', async () => {
+        const onPackageSelect = jest.fn();
+
+        render(<PackagesV2 onPackageSelect={onPackageSelect} />);
+
+        // Rapid clicks
+        await userEvent.click(screen.getByTestId('package-premium'));
+        await userEvent.click(screen.getByTestId('package-premium'));
+
+        expect(onPackageSelect).toHaveBeenCalledTimes(1);
+    });
+});
+````
+
+## 2. Contract Test: API Response Shape
+
+```typescript
+describe('PackagesV2 - API Contract', () => {
+    it('should handle expected API response shape', async () => {
+        const response = await api.getPackages(bookingId);
+
+        expect(response).toMatchObject({
+            packages: expect.arrayContaining([
+                expect.objectContaining({
+                    id: expect.any(String),
+                    name: expect.any(String),
+                    price: expect.any(Number),
+                    display_name: expect.any(String).optional(),
+                    line_items: expect.arrayContaining([
+                        expect.objectContaining({
+                            ref_id: expect.any(String),
+                            name: expect.any(String),
+                            display_category: expect.stringMatching(/included|excluded/)
+                        })
+                    ])
+                })
+            ])
+        });
+    });
+
+    it('should handle missing optional fields', async () => {
+        server.use(
+            rest.get('/api/packages', (req, res, ctx) =>
+                res(ctx.json({
+                    packages: [{ id: 'pkg-1', name: 'Basic', price: 0 }]
+                    // display_name intentionally missing
+                }))
+            )
+        );
+
+        render(<PackagesV2 />);
+
+        // Should not crash, should use fallback
+        expect(screen.getByText('Basic')).toBeInTheDocument();
+    });
+});
+```
+
+## 3. Edge Case Tests: Price Calculation
+
+```typescript
+describe('PackagesV2 - Price Calculations', () => {
+    it.each([
+        [0, '€0.00'],
+        [9.99, '€9.99'],
+        [100.5, '€100.50'],
+        [-10, '€0.00'],        // Negative should show 0
+        [undefined, '€0.00'], // Missing should show 0
+    ])('should format price %s as %s', (input, expected) => {
+        render(<PackagesV2 packages={[{ price: input }]} />);
+        expect(screen.getByTestId('package-price')).toHaveTextContent(expected);
+    });
+
+    it('should handle currency conversion', async () => {
+        const booking = createMockBooking({ currency: 'USD' });
+        render(<PackagesV2 booking={booking} />);
+
+        expect(screen.getByTestId('package-price')).toHaveTextContent('$');
+    });
+});
+```
+
+════════════════════════════════════════════════════════════════
+SUMMARY
+════════════════════════════════════════════════════════════════
+
+Generated: 3 test suites, 8 test cases
+Coverage improvement: +35% on high-risk paths
+Priority: 🔴 Critical paths now covered
+
+Files created:
+
+-   PackagesV2.integration.test.tsx
+-   PackagesV2.contract.test.tsx
+-   PackagesV2.edge.test.tsx
+
+```
+
+## AI Execution
+
+When user runs `/risk-test-gen {PATH}`:
+
+1. **Analyze file/PR** - Parse code for risk patterns
+2. **Classify risk levels** - Map to business domains
+3. **Check existing coverage** - Find gaps
+4. **Generate tests** - Deep integration for critical paths
+5. **Create test files** - Follow project conventions
+```

package/commands/testing/write-unit-tests.md

@@ -0,0 +1,42 @@
+# Write Unit Tests
+
+## Overview
+
+Guidelines for writing comprehensive unit tests for React components, React hooks, services, utilities and so on.
+
+## Test Categories
+
+### React component testing
+
+-   [ ] Test all component states
+-   [ ] Test user interactions
+-   [ ] Test prop variations
+-   [ ] Test side effects
+-   [ ] Test accessibility considering `axe` helper from `core-dev-testing` library
+-   [ ] Use `testing-library` best practices
+-   [ ] Consider using snapshot testing to prevent unexpected changes to the HTML output of a given component. They should only be used when other testing methods (such as asserting elements with `testing-library`) do not cover the required use case.
+
+### Coverage
+
+-   [ ] Critical paths are tested
+-   [ ] Edge cases are covered
+-   [ ] Error scenarios are tested
+-   [ ] Mock external dependencies, dependency injections and timers
+-   [ ] Consider advanced assertions from the preinstalled `jest-extended` library
+
+### Quality
+
+-   [ ] Design it to be short, dead-simple, flat, and delightful to work with. One should look at a test and get the intent instantly
+-   [ ] Tests are readable and maintainable
+-   [ ] Test names describe what they test
+-   [ ] Follows project's testing patterns from `testing/patterns/general-unit-testing/README.md`
+-   [ ] No flaky tests
+
+## Best practices
+
+-   [ ] Don’t test your mock
+-   [ ] Don’t test external libraries
+-   [ ] Avoid non-deterministic specs like `Date`, `Math.random` and so on
+-   [ ] Consider using simple factory for reusing mocks
+-   [ ] Consider placing mocks in the file \*.mock.ts with the same name as the testing modules
+-   [ ] Tests are independent from each other

package/package.json

@@ -0,0 +1,36 @@
+{
+  "name": "cursor-quality-suite",
+  "version": "1.0.0",
+  "description": "Code Quality & Testing Commands for Cursor IDE - Test Generation, Mutation Testing, Architecture",
+  "bin": {
+    "cursor-quality-suite": "./bin/cli.js",
+    "quality-suite": "./bin/cli.js"
+  },
+  "main": "./bin/cli.js",
+  "keywords": [
+    "cursor",
+    "cursor-rules",
+    "cursor-commands",
+    "testing",
+    "code-quality",
+    "mutation-testing",
+    "architecture",
+    "dev-productivity",
+    "ai-coding"
+  ],
+  "author": "Sharath Chandra",
+  "license": "MIT",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/sharath317/cursor-quality-suite"
+  },
+  "homepage": "https://github.com/sharath317/cursor-quality-suite#readme",
+  "engines": {
+    "node": ">=18.0.0"
+  },
+  "files": [
+    "bin/",
+    "commands/"
+  ]
+}
+