oh-my-codex-cli 0.1.0

package/.agent/skills/tdd-generator/SKILL.md

@@ -0,0 +1,278 @@
+---
+name: TDD Test Generator
+description: AI-powered test-driven development assistant. Generates unit tests (Vitest), component tests (React Testing Library), and E2E tests with Playwright Test Agents integration.
+---
+
+# TDD Test Generator Skill
+
+## When to Use This Skill
+
+Automatically invoke when user mentions:
+- "生成测试", "写测试用例", "TDD"
+- "如何测试这个组件/函数"
+- "Playwright Test Agents", "自动化测试"
+- Keywords: test, 测试, TDD, vitest, playwright
+
+
+## What This Skill Does
+
+**TDD Test Generator** provides:
+1. **Smart Test Generation**: Auto-generate Vitest unit tests from code
+2. **Component Testing**: Generate React Testing Library tests
+3. **E2E Integration**: Deep integration with Playwright Test Agents
+4. **TDD Workflow**: Support complete Red-Green-Refactor cycle
+5. **Auto-Healing**: Leverage Playwright Healer Agent to auto-fix broken tests
+
+## Core Features
+
+### Level 1: Unit Tests (Vitest)
+
+For: utility functions, pure logic, business rules
+
+**Template**:
+```typescript
+import { describe, it, expect } from 'vitest';
+
+describe('functionName', () => {
+  it('should handle normal case', () => {
+    expect(functionName(input)).toBe(expected);
+  });
+  
+  it('should handle edge cases', () => {
+    expect(functionName('')).toBe(defaultValue);
+  });
+});
+```
+
+### Level 2: Component Tests (React Testing Library)
+
+For: UI components, user interactions, state management
+
+**Template**:
+```typescript
+import { render, screen } from '@testing-library/react';
+import userEvent from '@testing-library/user-event';
+
+describe('ComponentName', () => {
+  it('renders and handles interaction', async () => {
+    const user = userEvent.setup();
+    render(<Component />);
+    await user.click(screen.getByRole('button'));
+    expect(mockFn).toHaveBeenCalled();
+  });
+});
+```
+
+### Level 3: E2E Tests + Playwright Test Agents
+
+**Three AI Agents Working Together**:
+
+1. **Planner Agent** - Explores app, generates test plans (Markdown)
+2. **Generator Agent** - Converts plans to Playwright code  
+3. **Healer Agent** - Auto-fixes broken tests, updates selectors
+
+**Setup Commands**:
+```bash
+# Initialize Playwright Test Agents
+npx playwright test --init-agents
+
+# Start agent loop
+npx playwright test --agent=planner
+npx playwright test --agent=generator  
+npx playwright test --agent=healer --repeat
+```
+
+**Project Structure**:
+```
+.github/workflows/agents.yml  # Agent definitions
+specs/                        # Test plans (Markdown)
+tests/                        # Generated tests
+  seed.spec.ts                # Seed test
+playwright.config.ts
+```
+
+## Instructions
+
+### Scenario 1: Generate Unit Test
+
+**User**: "Generate test for this function"
+
+**Your Actions**:
+1. Analyze code (function/component/flow)
+2. Identify test points (I/O, edge cases, errors)
+3. Generate complete Vitest test suite
+4. Provide run commands
+
+**Output Format**:
+```markdown
+## 🧪 Generated Unit Test
+
+### Test File: `functionName.test.ts`
+[Generated test code]
+
+### Covered Scenarios:
+- ✅ Normal input/output
+- ✅ Edge cases
+- ✅ Error handling
+
+### Run Test:
+\`pnpm test functionName.test.ts\`
+```
+
+### Scenario 2: Generate Component Test
+
+**User**: "How to test ChatInput component?"
+
+**Your Actions**:
+1. Analyze component props and events
+2. Identify user interaction flow
+3. Generate React Testing Library tests
+4. Include mocks and async handling
+
+### Scenario 3: E2E with Playwright Agents
+
+**User**: "Use Playwright Test Agents for E2E"
+
+**Your Actions**:
+
+**Step 1: Project Setup**
+```bash
+npm install -D @playwright/test
+npx playwright install
+npx playwright test --init-agents
+```
+
+**Step 2: Create Seed Test**
+```typescript
+// tests/seed.spec.ts
+import { test, expect } from '@playwright/test';
+
+test('seed - setup app state', async ({ page }) => {
+  await page.goto('http://localhost:3000');
+  // Setup initial state
+  await expect(page).toHaveURL('/dashboard');
+});
+```
+
+**Step 3: Start AI Agent Loop**
+```bash
+# Planner explores and generates plans
+npx playwright test --agent=planner
+
+# Generator creates tests from plans
+npx playwright test --agent=generator
+
+# Healer monitors and fixes failures
+npx playwright test --agent=healer --repeat
+```
+
+**Output Status**:
+```markdown
+## 🤖 Playwright Test Agents Running
+
+### Current Status:
+- ✅ Planner: Generated 3 test plans
+- ✅ Generator: Created 3 test files
+- 🔄 Healer: Monitoring...
+
+### Generated Tests:
+1. tests/user-login.spec.ts
+2. tests/create-post.spec.ts
+3. tests/search-functionality.spec.ts
+
+### View Plans: \`cat specs/*.md\`
+### Run Tests: \`npx playwright test\`
+```
+
+## TDD Workflow Support
+
+### Red-Green-Refactor Cycle
+
+**User wants TDD workflow**
+
+**Your Guidance**:
+```markdown
+## 🔴 Phase 1: Red (Write Failing Test)
+[Generate test with expected behavior]
+Run: \`pnpm test --watch\`
+Expected: ❌ All tests fail
+
+## 🟢 Phase 2: Green (Minimal Implementation)
+Implement feature to pass tests
+Expected: ✅ All tests pass
+
+## 🔵 Phase 3: Refactor (Optimize)
+Safely refactor with test protection
+Expected: ✅ Tests still pass
+```
+
+## Best Practices
+
+When generating tests, ensure:
+1. **Semantic queries** - Use getByRole, getByLabelText
+2. **Test behavior** - Not implementation details
+3. **Async handling** - Proper waitFor, findBy*
+4. **Mock externals** - Only APIs and services
+5. **One test, one thing** - Keep tests focused
+6. **Descriptive names** - Clear test descriptions
+
+## Project Integration Checklist
+
+Before generating tests, check:
+- [ ] Vitest installed
+- [ ] React Testing Library configured
+- [ ] @testing-library/user-event installed
+- [ ] Playwright installed (for E2E)
+- [ ] test scripts in package.json
+- [ ] vitest.config.ts configured
+
+**Auto-install if missing**:
+```bash
+pnpm add -D vitest @testing-library/react @testing-library/user-event jsdom
+pnpm add -D @playwright/test
+```
+
+## Troubleshooting
+
+### Issue 1: Vitest "Cannot find module"
+**Fix**: Check vitest.config.ts path aliases
+
+### Issue 2: RTL "Unable to find role"  
+**Fix**: Use screen.debug(), check element rendered
+
+### Issue 3: Playwright Agents won't start
+**Fix**: 
+```bash
+npm install -D @playwright/test@latest
+npx playwright install
+npx playwright test --init-agents
+```
+
+## Examples
+
+### Example 1: Quick Unit Test
+**User**: "Test formatMessage function"
+**Output**: Complete test suite with edge cases
+
+### Example 2: TDD Workflow  
+**User**: "TDD ChatInput component"
+**Output**: Red-Green-Refactor guidance
+
+### Example 3: E2E Automation
+**User**: "Playwright Agents for login"
+**Output**: Full agent setup and monitoring
+
+## Integration with Other Skills
+
+- **Agent-KB**: Query testing best practices
+- **Quality-Check**: Analyze test coverage
+- **Project-Analyze**: Identify critical modules to test
+
+## Important Notes
+
+- **Test-first mindset**: Guide users to understand TDD value
+- **Pragmatism**: Focus on core business, not 100% coverage
+- **Automation**: Maximize Playwright Agents to reduce manual work
+- **Maintainability**: Generate easy-to-maintain tests
+- **Education**: Explain principles, not just generate code
+

package/.agent/skills/tdd-workflow/SKILL.md

@@ -0,0 +1,424 @@
+---
+name: tdd-workflow
+description: Use this skill when writing new features, fixing bugs, or refactoring code. Enforces test-driven development with 80%+ coverage including unit, integration, and E2E tests.
+---
+
+# Test-Driven Development Workflow
+
+
+## Native Subagent Protocol (Codex)
+
+Codex supports native subagents. Delegate with `spawn_agent`, coordinate with `send_input`, collect via `wait`, and clean up with `close_agent`.
+
+Execution preference:
+1. Use native subagents first for independent workstreams (parallel when possible).
+2. Merge results in main thread and run final verification.
+3. Fallback only when delegation is blocked: use the `[ANALYST]`/`[ARCHITECT]`/`[EXECUTOR]`/`[REVIEWER]` structure in a single response.
+
+Minimal orchestration pattern:
+```text
+spawn_agent -> send_input (optional) -> wait -> close_agent
+```
+
+This skill ensures all code development follows TDD principles with comprehensive test coverage.
+
+## When to Activate
+
+- Writing new features or functionality
+- Fixing bugs or issues
+- Refactoring existing code
+- Adding API endpoints
+- Creating new components
+
+## Core Principles
+
+### 1. Tests BEFORE Code
+ALWAYS write tests first, then implement code to make tests pass.
+
+### 2. Coverage Requirements
+- Minimum 80% coverage (unit + integration + E2E)
+- All edge cases covered
+- Error scenarios tested
+- Boundary conditions verified
+
+### 3. Test Types
+
+#### Unit Tests
+- Individual functions and utilities
+- Component logic
+- Pure functions
+- Helpers and utilities
+
+#### Integration Tests
+- API endpoints
+- Database operations
+- Service interactions
+- External API calls
+
+#### E2E Tests (Playwright)
+- Critical user flows
+- Complete workflows
+- Browser automation
+- UI interactions
+
+## TDD Workflow Steps
+
+### Step 1: Write User Journeys
+```
+As a [role], I want to [action], so that [benefit]
+
+Example:
+As a user, I want to search for markets semantically,
+so that I can find relevant markets even without exact keywords.
+```
+
+### Step 2: Generate Test Cases
+For each user journey, create comprehensive test cases:
+
+```typescript
+describe('Semantic Search', () => {
+  it('returns relevant markets for query', async () => {
+    // Test implementation
+  })
+
+  it('handles empty query gracefully', async () => {
+    // Test edge case
+  })
+
+  it('falls back to substring search when Redis unavailable', async () => {
+    // Test fallback behavior
+  })
+
+  it('sorts results by similarity score', async () => {
+    // Test sorting logic
+  })
+})
+```
+
+### Step 3: Run Tests (They Should Fail)
+```bash
+npm test
+# Tests should fail - we haven't implemented yet
+```
+
+### Step 4: Implement Code
+Write minimal code to make tests pass:
+
+```typescript
+// Implementation guided by tests
+export async function searchMarkets(query: string) {
+  // Implementation here
+}
+```
+
+### Step 5: Run Tests Again
+```bash
+npm test
+# Tests should now pass
+```
+
+### Step 6: Refactor
+Improve code quality while keeping tests green:
+- Remove duplication
+- Improve naming
+- Optimize performance
+- Enhance readability
+
+### Step 7: Verify Coverage
+```bash
+npm run test:coverage
+# Verify 80%+ coverage achieved
+```
+
+## Testing Patterns
+
+### Unit Test Pattern (Jest/Vitest)
+```typescript
+import { render, screen, fireEvent } from '@testing-library/react'
+import { Button } from './Button'
+
+describe('Button Component', () => {
+  it('renders with correct text', () => {
+    render(<Button>Click me</Button>)
+    expect(screen.getByText('Click me')).toBeInTheDocument()
+  })
+
+  it('calls onClick when clicked', () => {
+    const handleClick = jest.fn()
+    render(<Button onClick={handleClick}>Click</Button>)
+
+    fireEvent.click(screen.getByRole('button'))
+
+    expect(handleClick).toHaveBeenCalledTimes(1)
+  })
+
+  it('is disabled when disabled prop is true', () => {
+    render(<Button disabled>Click</Button>)
+    expect(screen.getByRole('button')).toBeDisabled()
+  })
+})
+```
+
+### API Integration Test Pattern
+```typescript
+import { NextRequest } from 'next/server'
+import { GET } from './route'
+
+describe('GET /api/markets', () => {
+  it('returns markets successfully', async () => {
+    const request = new NextRequest('http://localhost/api/markets')
+    const response = await GET(request)
+    const data = await response.json()
+
+    expect(response.status).toBe(200)
+    expect(data.success).toBe(true)
+    expect(Array.isArray(data.data)).toBe(true)
+  })
+
+  it('validates query parameters', async () => {
+    const request = new NextRequest('http://localhost/api/markets?limit=invalid')
+    const response = await GET(request)
+
+    expect(response.status).toBe(400)
+  })
+
+  it('handles database errors gracefully', async () => {
+    // Mock database failure
+    const request = new NextRequest('http://localhost/api/markets')
+    // Test error handling
+  })
+})
+```
+
+### E2E Test Pattern (Playwright)
+```typescript
+import { test, expect } from '@playwright/test'
+
+test('user can search and filter markets', async ({ page }) => {
+  // Navigate to markets page
+  await page.goto('/')
+  await page.click('a[href="/markets"]')
+
+  // Verify page loaded
+  await expect(page.locator('h1')).toContainText('Markets')
+
+  // Search for markets
+  await page.fill('input[placeholder="Search markets"]', 'election')
+
+  // Wait for debounce and results
+  await page.waitForTimeout(600)
+
+  // Verify search results displayed
+  const results = page.locator('[data-testid="market-card"]')
+  await expect(results).toHaveCount(5, { timeout: 5000 })
+
+  // Verify results contain search term
+  const firstResult = results.first()
+  await expect(firstResult).toContainText('election', { ignoreCase: true })
+
+  // Filter by status
+  await page.click('button:has-text("Active")')
+
+  // Verify filtered results
+  await expect(results).toHaveCount(3)
+})
+
+test('user can create a new market', async ({ page }) => {
+  // Login first
+  await page.goto('/creator-dashboard')
+
+  // Fill market creation form
+  await page.fill('input[name="name"]', 'Test Market')
+  await page.fill('textarea[name="description"]', 'Test description')
+  await page.fill('input[name="endDate"]', '2025-12-31')
+
+  // Submit form
+  await page.click('button[type="submit"]')
+
+  // Verify success message
+  await expect(page.locator('text=Market created successfully')).toBeVisible()
+
+  // Verify redirect to market page
+  await expect(page).toHaveURL(/\/markets\/test-market/)
+})
+```
+
+## Test File Organization
+
+```
+src/
+├── components/
+│   ├── Button/
+│   │   ├── Button.tsx
+│   │   ├── Button.test.tsx          # Unit tests
+│   │   └── Button.stories.tsx       # Storybook
+│   └── MarketCard/
+│       ├── MarketCard.tsx
+│       └── MarketCard.test.tsx
+├── app/
+│   └── api/
+│       └── markets/
+│           ├── route.ts
+│           └── route.test.ts         # Integration tests
+└── e2e/
+    ├── markets.spec.ts               # E2E tests
+    ├── trading.spec.ts
+    └── auth.spec.ts
+```
+
+## Mocking External Services
+
+### Supabase Mock
+```typescript
+jest.mock('@/lib/supabase', () => ({
+  supabase: {
+    from: jest.fn(() => ({
+      select: jest.fn(() => ({
+        eq: jest.fn(() => Promise.resolve({
+          data: [{ id: 1, name: 'Test Market' }],
+          error: null
+        }))
+      }))
+    }))
+  }
+}))
+```
+
+### Redis Mock
+```typescript
+jest.mock('@/lib/redis', () => ({
+  searchMarketsByVector: jest.fn(() => Promise.resolve([
+    { slug: 'test-market', similarity_score: 0.95 }
+  ])),
+  checkRedisHealth: jest.fn(() => Promise.resolve({ connected: true }))
+}))
+```
+
+### OpenAI Mock
+```typescript
+jest.mock('@/lib/openai', () => ({
+  generateEmbedding: jest.fn(() => Promise.resolve(
+    new Array(1536).fill(0.1) // Mock 1536-dim embedding
+  ))
+}))
+```
+
+## Test Coverage Verification
+
+### Run Coverage Report
+```bash
+npm run test:coverage
+```
+
+### Coverage Thresholds
+```json
+{
+  "jest": {
+    "coverageThresholds": {
+      "global": {
+        "branches": 80,
+        "functions": 80,
+        "lines": 80,
+        "statements": 80
+      }
+    }
+  }
+}
+```
+
+## Common Testing Mistakes to Avoid
+
+### ❌ WRONG: Testing Implementation Details
+```typescript
+// Don't test internal state
+expect(component.state.count).toBe(5)
+```
+
+### ✅ CORRECT: Test User-Visible Behavior
+```typescript
+// Test what users see
+expect(screen.getByText('Count: 5')).toBeInTheDocument()
+```
+
+### ❌ WRONG: Brittle Selectors
+```typescript
+// Breaks easily
+await page.click('.css-class-xyz')
+```
+
+### ✅ CORRECT: Semantic Selectors
+```typescript
+// Resilient to changes
+await page.click('button:has-text("Submit")')
+await page.click('[data-testid="submit-button"]')
+```
+
+### ❌ WRONG: No Test Isolation
+```typescript
+// Tests depend on each other
+test('creates user', () => { /* ... */ })
+test('updates same user', () => { /* depends on previous test */ })
+```
+
+### ✅ CORRECT: Independent Tests
+```typescript
+// Each test sets up its own data
+test('creates user', () => {
+  const user = createTestUser()
+  // Test logic
+})
+
+test('updates user', () => {
+  const user = createTestUser()
+  // Update logic
+})
+```
+
+## Continuous Testing
+
+### Watch Mode During Development
+```bash
+npm test -- --watch
+# Tests run automatically on file changes
+```
+
+### Pre-Commit Hook
+```bash
+# Runs before every commit
+npm test && npm run lint
+```
+
+### CI/CD Integration
+```yaml
+# GitHub Actions
+- name: Run Tests
+  run: npm test -- --coverage
+- name: Upload Coverage
+  uses: codecov/codecov-action@v3
+```
+
+## Best Practices
+
+1. **Write Tests First** - Always TDD
+2. **One Assert Per Test** - Focus on single behavior
+3. **Descriptive Test Names** - Explain what's tested
+4. **Arrange-Act-Assert** - Clear test structure
+5. **Mock External Dependencies** - Isolate unit tests
+6. **Test Edge Cases** - Null, undefined, empty, large
+7. **Test Error Paths** - Not just happy paths
+8. **Keep Tests Fast** - Unit tests < 50ms each
+9. **Clean Up After Tests** - No side effects
+10. **Review Coverage Reports** - Identify gaps
+
+## Success Metrics
+
+- 80%+ code coverage achieved
+- All tests passing (green)
+- No skipped or disabled tests
+- Fast test execution (< 30s for unit tests)
+- E2E tests cover critical user flows
+- Tests catch bugs before production
+
+---
+
+**Remember**: Tests are not optional. They are the safety net that enables confident refactoring, rapid development, and production reliability.