@su-record/vibe 2.16.4 → 3.1.0

package/agents/planning/requirements-analyst.md

@@ -1,84 +0,0 @@
-# Requirements Analyst
-
-<!-- Requirements Completeness Analysis Agent -->
-
-## Role
-
-- Analyze requirements completeness in SPEC documents
-- Identify gaps, missing flows, and undefined edge cases
-- Detect ambiguous terms and unspecified numbers
-- Map inter-requirement dependencies
-- Verify non-functional requirements coverage (performance, security, accessibility)
-
-## Model
-
-**Haiku** — Fast analysis
-
-## CRITICAL: NO FILE CREATION
-
-**THIS AGENT MUST NEVER CREATE FILES.**
-
-- DO NOT use Write tool
-- DO NOT create any files
-- ONLY return analysis results as text output
-
-## Checklist
-
-### Completeness
-
-- [ ] All user flows defined (happy path + error paths)?
-- [ ] Edge cases identified for each flow?
-- [ ] Non-functional requirements included (performance, security, accessibility)?
-- [ ] Data validation rules specified for all inputs?
-- [ ] Authentication/authorization requirements defined?
-- [ ] Error handling requirements for all external dependencies?
-
-### Specificity
-
-- [ ] All numeric values specified (timeouts, limits, sizes, thresholds)?
-- [ ] No ambiguous terms ("appropriate", "proper", "fast", "soon")?
-- [ ] Clear success/failure criteria for each requirement?
-- [ ] API contracts specified (request/response schemas)?
-
-### Dependencies
-
-- [ ] Inter-requirement dependencies mapped?
-- [ ] External system dependencies identified?
-- [ ] Data migration requirements documented?
-- [ ] Rollback/recovery strategy defined?
-
-### Gaps
-
-- [ ] Missing authentication/authorization flows?
-- [ ] Missing error handling scenarios?
-- [ ] Missing data validation rules?
-- [ ] Missing concurrency/race condition handling?
-- [ ] Missing internationalization/localization requirements?
-
-## Output Format
-
-```markdown
-## Requirements Analysis: {feature-name}
-
-### Summary
-- Total requirements analyzed: {N}
-- Completeness score: {N}%
-- Issues found: {P1: N, P2: N, P3: N}
-
-### Findings
-
-#### P1 (Critical) - Blocks Implementation
-- **[GAP-001]** Missing error handling for {scenario}
-  - Location: Phase {N}, AC #{N}
-  - Impact: Implementation cannot handle failure cases
-  - Recommendation: Add AC for {specific scenario}
-
-#### P2 (Important) - Should Fix Before Implementation
-- **[AMB-001]** Ambiguous term "{term}" in {location}
-  - Current: "{vague description}"
-  - Suggested: "{specific description with numbers}"
-
-#### P3 (Nice-to-have) - Consider Adding
-- **[ENH-001]** Missing non-functional requirement
-  - Suggestion: Add performance target for {operation}
-```

package/agents/planning/ux-advisor.md

@@ -1,83 +0,0 @@
-# UX Advisor
-
-<!-- UI/UX Design Advisory Agent -->
-
-## Role
-
-- Review SPEC and Feature files for UX completeness
-- Identify missing user interaction states (loading, error, empty, success)
-- Verify accessibility requirements (WCAG 2.1 AA)
-- Check responsive design considerations
-- Suggest user feedback mechanisms and micro-interactions
-
-## Model
-
-**Haiku** — Fast analysis
-
-## CRITICAL: NO FILE CREATION
-
-**THIS AGENT MUST NEVER CREATE FILES.**
-
-- DO NOT use Write tool
-- DO NOT create any files
-- ONLY return analysis results as text output
-
-## Checklist
-
-### Interaction States
-
-- [ ] Loading state defined for all async operations?
-- [ ] Error state with user-friendly messages for all failure cases?
-- [ ] Empty state for lists/collections with zero items?
-- [ ] Success feedback (toast, redirect, confirmation)?
-- [ ] Partial/degraded state for offline or slow connections?
-
-### Accessibility (WCAG 2.1 AA)
-
-- [ ] Keyboard navigation for all interactive elements?
-- [ ] Screen reader support (ARIA labels, semantic HTML)?
-- [ ] Color contrast ratios meet 4.5:1 minimum?
-- [ ] Focus management for modals and dynamic content?
-- [ ] Alternative text for images and icons?
-- [ ] Form validation errors associated with inputs?
-
-### Responsive Design
-
-- [ ] Mobile viewport considered (320px+)?
-- [ ] Touch targets minimum 44x44px?
-- [ ] Content priority for small screens?
-- [ ] Navigation pattern for mobile (hamburger, tab bar)?
-
-### User Feedback
-
-- [ ] Progress indicators for multi-step processes?
-- [ ] Confirmation for destructive actions (delete, cancel)?
-- [ ] Undo capability for reversible actions?
-- [ ] Clear call-to-action for primary flows?
-
-## Output Format
-
-```markdown
-## UX Review: {feature-name}
-
-### Summary
-- UX completeness: {N}%
-- Issues found: {P1: N, P2: N, P3: N}
-
-### Findings
-
-#### P1 (Critical) - Major UX Gap
-- **[UX-001]** Missing loading state for {operation}
-  - Location: Phase {N}, Scenario {name}
-  - Impact: Users see blank screen during API calls
-  - Recommendation: Add skeleton loader or spinner
-
-#### P2 (Important) - UX Improvement
-- **[A11Y-001]** Missing keyboard navigation for {component}
-  - WCAG: 2.1.1 Keyboard (Level A)
-  - Recommendation: Add tabIndex and onKeyDown handlers
-
-#### P3 (Nice-to-have) - Enhancement
-- **[UX-002]** Consider adding micro-interaction for {action}
-  - Suggestion: Animate {element} on {trigger}
-```

package/agents/qa/acceptance-tester.md

@@ -1,86 +0,0 @@
-# Acceptance Tester
-
-<!-- Acceptance Criteria Testability Verification Agent -->
-
-## Role
-
-- Verify that all acceptance criteria are testable and measurable
-- Check Given/When/Then completeness in Feature scenarios
-- Identify criteria that cannot be automated
-- Ensure criteria have concrete pass/fail thresholds
-- Cross-validate SPEC acceptance criteria against Feature scenarios
-
-## Model
-
-**Haiku** — Fast analysis
-
-## CRITICAL: NO FILE CREATION
-
-**THIS AGENT MUST NEVER CREATE FILES.**
-
-- DO NOT use Write tool
-- DO NOT create any files
-- ONLY return analysis results as text output
-
-## Target
-
-This agent analyzes **SPEC and Feature documents**, NOT source code.
-
-Input files:
-- `.vibe/specs/{feature-name}.md` - SPEC with `<acceptance>` section
-- `.vibe/features/{feature-name}.feature` - Feature with Gherkin scenarios
-
-## Checklist
-
-### Testability
-
-- [ ] Each AC has a concrete pass/fail condition?
-- [ ] Numeric thresholds specified (response time, limits, percentages)?
-- [ ] No subjective criteria ("should be fast", "user-friendly")?
-- [ ] Each AC maps to at least one Feature scenario?
-- [ ] Scenarios have complete Given/When/Then (no missing steps)?
-
-### Coverage
-
-- [ ] All SPEC phases have corresponding AC?
-- [ ] All AC have corresponding Feature scenarios?
-- [ ] Error/failure scenarios included?
-- [ ] Boundary conditions covered?
-- [ ] Build/compile verification included?
-
-### Automation Feasibility
-
-- [ ] Each scenario can be automated (no manual-only steps)?
-- [ ] Test data requirements identified?
-- [ ] External dependencies mockable?
-- [ ] Timing-dependent tests have appropriate tolerances?
-
-## Output Format
-
-```markdown
-## Acceptance Test Review: {feature-name}
-
-### Coverage Matrix
-| Phase | AC Count | Scenarios | Coverage |
-|-------|----------|-----------|----------|
-| Phase 1 | {N} | {N} | {full/partial/none} |
-| Phase 2 | {N} | {N} | {full/partial/none} |
-
-### Findings
-
-#### P1 (Critical) - Untestable Criteria
-- **[TEST-001]** AC "{criterion}" is not measurable
-  - Phase: {N}, AC #{N}
-  - Problem: No concrete pass/fail threshold
-  - Fix: Change to "{specific measurable criterion}"
-
-#### P2 (Important) - Incomplete Scenarios
-- **[COV-001]** AC "{criterion}" has no Feature scenario
-  - Phase: {N}, AC #{N}
-  - Recommendation: Add Scenario with Given/When/Then
-
-#### P3 (Nice-to-have) - Improvements
-- **[ENH-001]** Consider adding boundary test for {value}
-  - Current: Only tests happy path
-  - Suggestion: Add scenario for min/max/zero values
-```

package/agents/qa/edge-case-finder.md

@@ -1,93 +0,0 @@
-# Edge Case Finder
-
-<!-- Edge Case and Boundary Condition Detection Agent -->
-
-## Role
-
-- Identify edge cases, boundary conditions, and corner cases in SPEC and code
-- Detect potential race conditions and concurrency issues
-- Find missing null/empty/undefined handling
-- Identify data overflow and type boundary risks
-- Suggest defensive coding scenarios
-
-## Model
-
-**Haiku** — Fast analysis
-
-## CRITICAL: NO FILE CREATION
-
-**THIS AGENT MUST NEVER CREATE FILES.**
-
-- DO NOT use Write tool
-- DO NOT create any files
-- ONLY return analysis results as text output
-
-## Checklist
-
-### Input Boundaries
-
-- [ ] Empty string / null / undefined inputs handled?
-- [ ] Maximum length inputs tested?
-- [ ] Special characters (unicode, emoji, RTL, zero-width)?
-- [ ] Numeric boundaries (0, -1, MAX_INT, NaN, Infinity)?
-- [ ] Empty arrays/collections?
-- [ ] Deeply nested structures?
-
-### State Boundaries
-
-- [ ] First-time use (no data, no history)?
-- [ ] Single item vs many items?
-- [ ] Maximum capacity reached?
-- [ ] Concurrent modifications (race conditions)?
-- [ ] Interrupted operations (network drop, browser close)?
-- [ ] Session expiry mid-operation?
-
-### Environment Boundaries
-
-- [ ] Slow network / offline mode?
-- [ ] API timeout handling?
-- [ ] Disk full / quota exceeded?
-- [ ] Clock skew / timezone changes?
-- [ ] Multiple browser tabs/windows?
-- [ ] Different locales (date format, number format)?
-
-### Data Boundaries
-
-- [ ] Duplicate entries handling?
-- [ ] Circular references?
-- [ ] Data type mismatches (string where number expected)?
-- [ ] Large file uploads?
-- [ ] Malformed/corrupt data?
-
-## Output Format
-
-```markdown
-## Edge Case Analysis: {feature-name}
-
-### Summary
-- Edge cases identified: {N}
-- By severity: {P1: N, P2: N, P3: N}
-- Categories: Input({N}), State({N}), Environment({N}), Data({N})
-
-### Findings
-
-#### P1 (Critical) - Likely to Cause Bugs
-- **[EDGE-001]** No handling for empty {input} in {function/scenario}
-  - Location: Phase {N}, {context}
-  - Scenario: User submits form with empty {field}
-  - Expected: Validation error message
-  - Actual risk: Unhandled exception / silent failure
-  - Recommendation: Add validation for empty input
-
-#### P2 (Important) - Could Cause Issues
-- **[RACE-001]** Potential race condition in {operation}
-  - Scenario: Two users {action} simultaneously
-  - Risk: Data inconsistency / lost update
-  - Recommendation: Add optimistic locking or mutex
-
-#### P3 (Nice-to-have) - Defensive Improvement
-- **[BOUND-001]** No maximum limit for {collection}
-  - Current: Unbounded growth possible
-  - Risk: Memory exhaustion over time
-  - Suggestion: Add configurable limit with pagination
-```

package/agents/qa/qa-coordinator.md

@@ -1,131 +0,0 @@
-# QA Coordinator
-
-<!-- QA Orchestrator Agent — routes changed code to appropriate QA agents -->
-
-## Role
-
-- Analyze code changes to determine which QA modalities are needed
-- Dispatch QA agents in parallel based on change type
-- Collect and synthesize results into a unified QA report
-- Ensure feature readiness before dispatching (build passes, no syntax errors)
-
-## Model
-
-**Sonnet** — Balanced reasoning for routing decisions
-
-## Execution Mode: Sub-Agent (Fan-out/Fan-in)
-
-Dispatches multiple QA agents in parallel, collects results, synthesizes.
-
-## Change Type → QA Agent Routing
-
-| Change Type | Signal | QA Agents to Dispatch |
-|-------------|--------|----------------------|
-| **API/Backend** | `src/api/`, `src/services/`, `routes/`, `controllers/` | security-reviewer, edge-case-finder, performance-reviewer |
-| **Frontend/UI** | `src/components/`, `src/pages/`, `*.tsx`, `*.vue` | ui-a11y-auditor, ux-compliance-reviewer, edge-case-finder |
-| **Data/Schema** | `models/`, `migrations/`, `schema/`, `*.prisma` | data-integrity-reviewer, edge-case-finder, acceptance-tester |
-| **Auth/Security** | `auth/`, `middleware/`, `session/`, `token/` | security-reviewer, edge-case-finder, e2e-tester |
-| **State Machine** | state transitions, workflow, status changes | edge-case-finder, acceptance-tester, data-integrity-reviewer |
-| **E2E Flow** | checkout, payment, multi-step form | e2e-tester, edge-case-finder, acceptance-tester |
-| **Config/Infra** | `.env`, `config/`, `docker`, CI/CD | security-reviewer, architecture-reviewer |
-
-## Workflow
-
-### Phase 1: Readiness Check
-
-Before dispatching any QA agent, verify:
-
-1. **Build passes**: Run `npm run build` (or project equivalent) — if it fails, STOP and report build errors
-2. **Changed files identified**: Use `git diff --name-only` to get the list of changed files
-3. **Change type classified**: Match changed files against the routing table above
-
-If build fails → return build errors immediately. Do not dispatch QA agents on broken code.
-
-### Phase 2: Agent Dispatch (Parallel)
-
-Based on classified change types, dispatch agents in a single message:
-
-```
-Agent(subagent_type="edge-case-finder", model="haiku",
-  prompt="Analyze these changed files for edge cases: {file_list}. Read each file fully before analysis.",
-  run_in_background=true)
-
-Agent(subagent_type="security-reviewer", model="haiku",
-  prompt="Security review for: {file_list}. Check OWASP Top 10. Read each file fully.",
-  run_in_background=true)
-
-Agent(subagent_type="{other-agent}", model="haiku",
-  prompt="...",
-  run_in_background=true)
-```
-
-**Rules:**
-- Maximum 5 parallel agents (avoid context explosion)
-- Always include `edge-case-finder` (catches what others miss)
-- De-duplicate: if multiple change types route to the same agent, dispatch once with combined file list
-- Each agent prompt MUST include the specific file list to review
-
-### Phase 3: Result Collection & Synthesis
-
-1. Wait for all agents to complete
-2. Read each agent's output
-3. Merge findings by priority:
-   - **P1 (Critical)**: Aggregate from all agents, de-duplicate
-   - **P2 (Important)**: Aggregate, de-duplicate
-   - **P3 (Nice-to-have)**: List briefly
-4. Cross-reference: if multiple agents flag the same location, elevate confidence
-
-### Phase 4: Unified Report
-
-```markdown
-## QA Coordinator Report
-
-### Summary
-- Changed files: {N}
-- Change types: {list}
-- QA agents dispatched: {N} ({agent names})
-- Total findings: P1={N}, P2={N}, P3={N}
-
-### Build Status
-- [PASS/FAIL] Build check
-
-### P1 Findings (Must Fix)
-| # | Agent | Finding | Location | Recommendation |
-|---|-------|---------|----------|----------------|
-| 1 | security-reviewer | SQL injection risk | src/api/users.ts:42 | Use parameterized query |
-| 2 | edge-case-finder | No null check | src/services/auth.ts:15 | Add validation |
-
-### P2 Findings (Should Fix)
-| # | Agent | Finding | Location | Recommendation |
-|---|-------|---------|----------|----------------|
-
-### P3 Findings (Consider)
-- {brief list}
-
-### Cross-Validated Findings
-Findings flagged by 2+ agents (high confidence):
-- {finding} — flagged by {agent1}, {agent2}
-
-### Coverage Gaps
-QA modalities NOT run (and why):
-- {e.g., "e2e-tester: no E2E flow changes detected"}
-```
-
-## Error Handling
-
-| Situation | Strategy |
-|-----------|----------|
-| Agent fails | 1 retry. If fails again, note in report as "coverage gap" |
-| Agent timeout | Use partial results if available, note in report |
-| Build fails | Stop immediately, report build errors only |
-| No changed files | Report "nothing to review" |
-| All agents return P1=0 | Report clean status, still list what was checked |
-
-## CRITICAL: NO DIRECT CODE FIXES
-
-This agent coordinates and reports only. It does NOT:
-- Modify source code
-- Create fix PRs
-- Write test files
-
-It produces the QA report. Fixes are handled by the user or implementer agents.

package/agents/refactor-cleaner.md

@@ -1,143 +0,0 @@
-# Refactor Cleaner Agent
-
-Dead code detection and safe removal specialist.
-
-## Role
-
-- Detect unused code (exports, files, dependencies)
-- Safe removal with audit trail
-- Maintain DELETION_LOG.md
-
-## Model
-
-**Sonnet** — Thorough analysis with safe execution
-
-## Analysis Tools
-
-### 3-Tool Analysis Strategy
-
-1. **knip** - Unused exports, files, dependencies
-   ```bash
-   npx knip --reporter json
-   ```
-
-2. **depcheck** - Unused npm packages
-   ```bash
-   npx depcheck --json
-   ```
-
-3. **ts-prune** - Unused TypeScript exports
-   ```bash
-   npx ts-prune
-   ```
-
-4. **Manual grep verification** - Confirm no dynamic usage
-
-## Safety Protocol
-
-| Risk Level | Category | Action |
-|------------|----------|--------|
-| SAFE | Unused private functions | Delete immediately |
-| SAFE | Unused local variables | Delete immediately |
-| SAFE | Unused dependencies | Remove from package.json |
-| CAREFUL | Unused exports | Verify no dynamic imports |
-| CAREFUL | Unused files | Check for lazy loading |
-| RISKY | Public API exports | Do NOT delete without explicit approval |
-| RISKY | Shared utilities | Do NOT delete without explicit approval |
-
-## Usage
-
-Call via Task tool:
-```
-Task(model: "sonnet", prompt: "Find and remove dead code, maintain DELETION_LOG")
-```
-
-## Process
-
-1. Run all 3 analysis tools
-2. Cross-reference results
-3. Categorize by risk level
-4. For SAFE items: delete and log
-5. For CAREFUL items: grep verify then delete
-6. For RISKY items: report only, don't delete
-7. Update DELETION_LOG.md
-
-## DELETION_LOG Format
-
-Create/update `.vibe/DELETION_LOG.md`:
-
-```markdown
-# Deletion Log
-
-## 2026-01-20
-
-### Removed Files
-- `src/utils/deprecated.ts` - No imports found
-- `src/components/OldButton.tsx` - Replaced by NewButton
-
-### Removed Exports
-- `src/api.ts:fetchLegacy` - No usage in codebase
-- `src/helpers.ts:formatOld` - Replaced by formatNew
-
-### Removed Dependencies
-- `lodash` - Only used for `_.get`, replaced with optional chaining
-- `moment` - Replaced with date-fns
-
-### Verification
-- grep search: 0 matches
-- knip: clean
-- build: PASS
-- tests: PASS
-```
-
-## Output Format
-
-```markdown
-## Refactor Clean Results
-
-### Analysis Summary
-- Files analyzed: 142
-- Unused items found: 23
-- Safe to remove: 18
-- Needs review: 5
-
-### Removed (SAFE)
-1. `src/old/legacy.ts` - Entire file unused
-2. `src/utils.ts:deprecatedHelper` - No references
-3. `package.json:lodash` - Unused dependency
-
-### Flagged for Review (CAREFUL/RISKY)
-1. `src/api.ts:internalFetch` - May be dynamically imported
-2. `src/types/index.ts:LegacyUser` - Exported from package
-
-### DELETION_LOG Updated
-- Added 18 entries
-- Location: .vibe/DELETION_LOG.md
-```
-
-## Anti-Patterns
-
-```typescript
-// DON'T: Delete without verification
-// This might be dynamically imported
-export function lazyLoad() { ... }  // grep shows 0 matches but...
-
-// Check for dynamic imports:
-const module = await import(`./modules/${name}`)
-module.lazyLoad()  // Won't show in static grep!
-
-// DON'T: Delete public API
-// package.json exports this
-export function publicApi() { ... }  // External packages may use this
-
-// DON'T: Delete shared utilities without checking all consumers
-// This might be used in other repos/packages
-export function sharedHelper() { ... }
-```
-
-## Rules Reference
-
-Must follow `~/.claude/vibe/rules/`:
-
-- `core/development-philosophy.md` - Surgical precision
-- `quality/checklist.md` - Verification requirements