specweave 0.1.9 → 0.3.0

package/src/skills/spec-driven-brainstorming/test-cases/TC-003-unclear-requirements-socratic.yaml

@@ -1,233 +0,0 @@
----
-name: "Unclear Requirements with Socratic Questioning"
-description: "Handle vague user input through Socratic questioning to clarify requirements before design"
-skill: spec-driven-brainstorming
-priority: P1
-estimated_time: "15-20 minutes"
-
-input:
-  prompt: "I want to build something with crypto prices"
-  context:
-    existing_project: true
-    tech_stack: "Unknown (needs detection)"
-    requirements_clarity: "Very Low"
-    user_goal_clarity: "Unclear"
-
-expected_output:
-  phase_1_understanding:
-    announcement: "I'm using spec-driven brainstorming to refine your idea. Let me understand what you're trying to achieve."
-
-    socratic_questioning_sequence:
-      question_1:
-        question: "What's your primary goal with crypto prices?"
-        tool_used: "AskUserQuestion"
-        options:
-          - "Real-time price tracking dashboard"
-          - "Price alerts and notifications"
-          - "Historical data analysis"
-          - "Automated trading bot"
-        user_answer: "Real-time price tracking dashboard"
-
-      question_2:
-        question: "Who will use this dashboard?"
-        tool_used: "Open-ended" # allowing freeform answer
-        user_answer: "Just me and maybe a few friends"
-
-      question_3:
-        question: "How many cryptocurrencies do you want to track?"
-        tool_used: "AskUserQuestion"
-        options:
-          - "Top 10 only (BTC, ETH, etc.)"
-          - "Top 100 cryptocurrencies"
-          - "Custom watchlist (user-selected)"
-          - "All available (1000+)"
-        user_answer: "Custom watchlist"
-
-      question_4:
-        question: "What's your update frequency requirement?"
-        tool_used: "AskUserQuestion"
-        options:
-          - "Real-time (every second, WebSocket)" # expensive
-          - "Near real-time (every 10-30 seconds, polling)" # balanced
-          - "Periodic (every few minutes, cron job)" # cheap
-        user_answer: "Near real-time (every 10-30 seconds)"
-
-      question_5:
-        question: "Any budget or hosting constraints?"
-        tool_used: "Open-ended"
-        user_answer: "Want to keep it cheap, maybe under $20/month"
-
-      question_6:
-        question: "Do you need historical price charts or just current prices?"
-        tool_used: "AskUserQuestion"
-        options:
-          - "Current prices only (simpler)"
-          - "Basic charts (1 day, 1 week, 1 month)" # medium complexity
-          - "Advanced charts with indicators (candlesticks, RSI, MACD)" # high complexity
-        user_answer: "Basic charts"
-
-    gathered_requirements:
-      purpose: "Personal crypto price dashboard with basic charts"
-      users: "Personal use (1-5 users)"
-      functional_requirements:
-        - "Track custom watchlist of cryptocurrencies"
-        - "Display current prices with 10-30 second updates"
-        - "Show basic charts (1 day, 1 week, 1 month)"
-      constraints:
-        - "Budget: $20/month or less"
-        - "Simple deployment (not enterprise-scale)"
-      success_criteria:
-        - "Dashboard loads in <2 seconds"
-        - "Prices update every 10-30 seconds"
-        - "Charts display historical data"
-        - "Add/remove cryptos from watchlist"
-
-    revisit_count: 1 # user revealed new constraint (budget) → re-clarified
-    flexibility_demonstrated: true # went back to refine budget impact
-
-  phase_2_tech_stack:
-    detection_attempt:
-      files_searched:
-        - "package.json"
-        - "requirements.txt"
-        - "*.csproj"
-      found: "package.json"
-
-    detected_stack:
-      framework: "Next.js 14"
-      language: "TypeScript"
-      database: null # needs to ask
-
-    database_question:
-      question: "Do you have a database set up, or should we use a lightweight option?"
-      tool_used: "AskUserQuestion"
-      options:
-        - "No database (store watchlist in localStorage)" # simplest, no backend needed
-        - "SQLite (lightweight, no hosting cost)" # simple, local file
-        - "PostgreSQL (more robust, requires hosting)" # production-ready, $5-10/month
-      user_answer: "SQLite"
-
-    skills_activated:
-      - "nextjs"
-      - "nodejs-backend"
-      - "frontend" # for React components
-
-  phase_3_exploration:
-    approaches_proposed: 3
-
-    approach_1:
-      name: "Server-Side Rendering (SSR) with API Polling"
-      architecture: "Next.js SSR fetches prices on each request"
-      trade_offs:
-        pros:
-          - "Simple implementation"
-          - "No WebSocket complexity"
-          - "Works on any hosting (Vercel free tier)"
-        cons:
-          - "Not truly real-time (page refresh needed)"
-          - "Higher API call volume (one per user request)"
-        cost: "$0 (Vercel free tier)"
-        complexity: "Low"
-
-    approach_2:
-      name: "Client-Side Polling with SWR"
-      architecture: "Next.js static + client polls API every 10 seconds"
-      trade_offs:
-        pros:
-          - "Near real-time without WebSockets"
-          - "SWR caching reduces API calls"
-          - "Static export possible"
-        cons:
-          - "More client-side logic"
-          - "Battery drain on mobile"
-        cost: "$0 (Vercel free tier or static hosting)"
-        complexity: "Low"
-
-    approach_3:
-      name: "Server-Sent Events (SSE) with Price Streaming"
-      architecture: "Backend streams price updates via SSE to clients"
-      trade_offs:
-        pros:
-          - "True push updates (no polling)"
-          - "Lower client battery usage"
-          - "More efficient than polling"
-        cons:
-          - "Requires long-lived server connections"
-          - "More complex backend"
-          - "Not supported on all hosting (Vercel has limits)"
-        cost: "$5-10/month (Railway or DigitalOcean)"
-        complexity: "Medium"
-
-    budget_impact_highlighted: true
-    user_selection: "Approach 2" # Client-side polling with SWR (fits budget)
-
-  phase_4_design_validation:
-    sections_presented:
-      - title: "Architecture Overview"
-        content: "Next.js 14 with App Router, client-side polling using SWR for price updates, SQLite for watchlist storage, CoinGecko API (free tier)"
-        validated: true
-
-      - title: "Components"
-        content: "Dashboard page (price grid), chart component (recharts), watchlist manager (add/remove), price API route (/api/prices)"
-        validated: true
-
-      - title: "Data Flow"
-        content: "User opens dashboard → SWR fetches /api/prices → Backend calls CoinGecko API → Cache for 10s → Return to client → Recharts renders → SWR auto-refetches every 10s"
-        validated: true
-
-      - title: "Error Handling"
-        content: "API rate limit → show cached data + warning, network failure → retry 3x with exponential backoff, invalid crypto symbol → show error state"
-        validated: true
-
-      - title: "Testing Strategy"
-        content: "Unit tests (price formatting), integration tests (API routes), E2E tests (Playwright: add crypto, view chart, polling works)"
-        validated: true
-
-      - title: "Performance & Cost"
-        content: "CoinGecko free tier: 50 calls/min (sufficient), Vercel free tier: unlimited deployments, estimated cost: $0/month"
-        validated: true
-
-    design_complete: true
-    budget_constraint_met: true
-
-  phase_5_specweave_handoff:
-    handoff_type: "Quick Increment (Fast Start)" # personal project, not enterprise
-
-    reason: "Personal project with clear, simple requirements - full documentation can be added incrementally"
-
-    expected_increment:
-      directory: ".specweave/increments/0001-crypto-dashboard/"
-      files:
-        - "spec.md" # WHAT & WHY (custom watchlist, near real-time, budget-conscious)
-        - "plan.md" # HOW (Next.js, SWR polling, SQLite, CoinGecko API)
-        - "tasks.md" # implementation checklist
-        - "tests.md" # E2E with Playwright (dashboard loads, prices update, chart renders)
-        - "context-manifest.yaml" # minimal context (nextjs, frontend skills)
-
-    agents_invoked:
-      - "pm" # minimal (personal project)
-      - "qa-lead" # E2E test strategy
-
-validation:
-  - "Phase 1: Socratic questioning sequence (at least 5 questions)"
-  - "Phase 1: Requirements gathered incrementally through dialogue"
-  - "Phase 1: Demonstrated flexibility (revisited earlier question when new constraint revealed)"
-  - "Phase 2: Tech stack detected from existing files"
-  - "Phase 2: Database option asked separately"
-  - "Phase 3: Budget impact explicitly highlighted in approaches"
-  - "Phase 3: Cost estimates provided for each approach"
-  - "Phase 4: Budget constraint validated in performance section"
-  - "Phase 5: Quick increment chosen (appropriate for personal project)"
-  - "AskUserQuestion used for structured choices"
-  - "Open-ended questions used for context gathering"
-  - "ONE question at a time in Phase 1"
-
-expected_errors: []
-
-success_criteria:
-  - "Vague input clarified through questioning"
-  - "Requirements extracted from dialogue"
-  - "Budget constraint met ($0/month vs $20/month budget)"
-  - "Appropriate handoff type chosen (quick vs full)"
-  - "Design ready for implementation"
----

package/src/skills/spec-driven-debugging/test-cases/TC-001-simple-auth-bug.yaml

@@ -1,212 +0,0 @@
----
-name: "Simple Authentication Bug with Spec Alignment"
-description: "Debug a login failure where code doesn't match spec.md requirements"
-skill: spec-driven-debugging
-priority: P1
-estimated_time: "15-20 minutes"
-
-input:
-  prompt: "Login is failing with '401 Unauthorized' even with correct credentials"
-  context:
-    project_type: "Next.js 14 + TypeScript"
-    increment: ".specweave/increments/0001-user-authentication/"
-    files_exist:
-      - "spec.md"
-      - "plan.md"
-      - "tests.md"
-      - "src/app/api/auth/[...nextauth]/route.ts"
-      - "src/middleware.ts"
-    error_message: "401 Unauthorized - Invalid credentials"
-    reproduction: "Consistent - happens every login attempt"
-
-expected_output:
-  phase_0_context_loading:
-    announcement: "Let me load the SpecWeave context before investigating."
-
-    files_loaded:
-      - path: ".specweave/increments/0001-user-authentication/spec.md"
-        key_findings:
-          - "FR-001: Users must be able to login with email and password"
-          - "FR-002: Passwords must be hashed with bcrypt (minimum 10 salt rounds)"
-          - "FR-003: JWT tokens expire after 7 days"
-          - "Acceptance: TC-001: Valid credentials → successful login"
-
-      - path: ".specweave/increments/0001-user-authentication/plan.md"
-        key_findings:
-          - "NextAuth.js v5 with Credentials provider"
-          - "PostgreSQL adapter for session storage"
-          - "bcrypt for password hashing"
-
-      - path: ".specweave/increments/0001-user-authentication/tests.md"
-        key_findings:
-          - "TC-005: Login with valid credentials should return 200"
-          - "TC-006: Login with invalid credentials should return 401"
-
-    bug_classification: "Likely code bug - implementation doesn't match spec"
-
-  phase_1_root_cause:
-    error_analysis:
-      message: "401 Unauthorized - Invalid credentials"
-      stack_trace_relevant: false # not a crash, just wrong response
-      line_numbers: "src/app/api/auth/[...nextauth]/route.ts:45"
-
-    reproduction:
-      consistent: true
-      steps:
-        - "Navigate to /login"
-        - "Enter email: test@example.com, password: Test123!"
-        - "Click submit"
-        - "Observe: 401 error"
-
-    recent_changes:
-      git_diff_findings:
-        - "Recently updated password hashing from md5 to bcrypt"
-        - "Changed salt rounds from 8 to 12"
-
-    spec_comparison:
-      spec_says: "Passwords must be hashed with bcrypt (minimum 10 salt rounds)"
-      actual_behavior: "Login always fails with valid credentials"
-      discrepancy: "Code may not be comparing hashed passwords correctly"
-
-    multi_component_check:
-      frontend: "✓ Sends correct email/password"
-      api_route: "⚠️ Receives correct data but returns 401"
-      database: "✓ User exists in DB with bcrypt hash"
-
-    data_flow_trace:
-      - step: "Frontend sends {email, password} to /api/auth/signin"
-        status: "✓ Correct"
-      - step: "API route calls authorize() function"
-        status: "⚠️ Here's where it fails"
-      - step: "authorize() queries user from DB"
-        status: "✓ User found"
-      - step: "authorize() compares passwords"
-        status: "❌ Always returns false"
-
-    root_cause_hypothesis: "Password comparison logic is incorrect after bcrypt migration"
-
-  phase_2_pattern_analysis:
-    working_examples_found:
-      - file: "src/app/api/users/[id]/route.ts"
-        pattern: "Uses bcrypt.compare(plaintext, hash) correctly"
-        code: |
-          const isValid = await bcrypt.compare(password, user.password);
-
-    broken_code:
-      file: "src/app/api/auth/[...nextauth]/route.ts:45"
-      code: |
-        const isValid = await bcrypt.compare(user.password, password); // WRONG ORDER!
-
-    difference_identified:
-      issue: "Arguments to bcrypt.compare() are reversed"
-      working: "bcrypt.compare(plaintext, hash)"
-      broken: "bcrypt.compare(hash, plaintext)"
-      impact: "Always returns false because it's trying to hash the hash"
-
-  phase_3_hypothesis:
-    hypothesis: "bcrypt.compare() arguments are in wrong order (hash first instead of plaintext first)"
-
-    classification: "Code bug - implementation error, spec is correct"
-
-    minimal_test:
-      approach: "Swap arguments in bcrypt.compare() call"
-      change_size: "Single line change"
-      one_variable: true
-
-  phase_4_implementation:
-    fix_level: "Code-level fix"
-    reason: "Spec is clear and correct, code has logic error"
-
-    test_creation:
-      level: "Level 3: Code Tests (Integration test)"
-      file: "tests/integration/auth.test.ts"
-      test_case: |
-        describe('Authentication', () => {
-          it('should login successfully with valid credentials', async () => {
-            const response = await fetch('/api/auth/signin', {
-              method: 'POST',
-              body: JSON.stringify({
-                email: 'test@example.com',
-                password: 'Test123!'
-              })
-            });
-            expect(response.status).toBe(200);
-            expect(response.body.user).toBeDefined();
-          });
-        });
-
-      test_fails_before_fix: true
-
-    spec_update_needed: false # spec was correct
-
-    code_fix:
-      file: "src/app/api/auth/[...nextauth]/route.ts"
-      before: |
-        const isValid = await bcrypt.compare(user.password, password);
-      after: |
-        const isValid = await bcrypt.compare(password, user.password);
-
-    verification:
-      test_passes: true
-      other_tests_broken: false
-      issue_resolved: true
-      spec_aligned: true
-
-    attempts_count: 1 # fixed on first try
-    ultrathink_needed: false
-
-  phase_5_documentation:
-    increment_updates:
-      - file: ".specweave/increments/0001-user-authentication/tests.md"
-        change: "Added TC-007: Regression test for bcrypt argument order"
-
-    architecture_updates: [] # no architectural change
-
-    strategy_updates: [] # requirements unchanged
-
-    new_increment_needed: false # small fix, no new increment
-
-    commit_message: |
-      fix(auth): correct bcrypt.compare argument order
-
-      Root cause: Arguments to bcrypt.compare() were reversed after
-      bcrypt migration. Function signature is compare(plaintext, hash)
-      but code was calling compare(hash, plaintext).
-
-      Solution: Swapped arguments to match bcrypt API correctly.
-      Spec alignment: Code now implements FR-002 correctly.
-
-      Test coverage:
-      - Added: TC-007 regression test for password comparison
-      - Verified: All existing auth tests pass
-
-      Documentation updated:
-      - tests.md: Added TC-007 for regression prevention
-
-      Fixes: #42
-
-      🤖 Generated with Claude Code
-      Co-Authored-By: Claude <noreply@anthropic.com>
-
-validation:
-  - "Phase 0: Loaded spec.md, plan.md, tests.md"
-  - "Phase 0: Classified as code bug (not spec bug)"
-  - "Phase 1: Traced data flow to identify failing component"
-  - "Phase 1: Compared behavior vs spec (spec was correct)"
-  - "Phase 2: Found working example in codebase"
-  - "Phase 2: Identified argument order difference"
-  - "Phase 3: Formed single, specific hypothesis"
-  - "Phase 4: Created failing test before fix"
-  - "Phase 4: Fixed on first attempt (no ultrathink needed)"
-  - "Phase 5: Updated tests.md with new test case"
-  - "Phase 5: Committed with proper documentation"
-
-expected_errors: []
-
-success_criteria:
-  - "Root cause identified (argument order)"
-  - "Fix verified with test"
-  - "Spec alignment confirmed"
-  - "Living documentation updated"
-  - "Single attempt (efficient debugging)"
----