feat: v2.1.5 - Dreamer config + Review skill 4-agent architecture

[ojowwalker77]

Summary

• Dreamer: configurable worktree defaults, branch prefix matrix-dreamer/
• Review skill: 4-agent pipeline with tiered output and Context7 integration

Changes

Dreamer

• Default branch prefix: claude-task/ → matrix-dreamer/
• New config section dreamer.worktree and dreamer.execution

Review Skill (/matrix:review)

• 4-agent architecture: Detection → Impact → Triage → Remediation
• Tiered output (critical/important/noise) with confidence scores
• Context7 for library-specific best practices
• Grep fallback when code index unavailable
• Taint analysis with matrix_find_definition

Tests

• schema.test.ts validates schema/migration sync

[greptile-apps]

Greptile Summary

This PR introduces configurable defaults for the Dreamer task automation system and implements a sophisticated 4-agent code review architecture.

Key Changes:

• Dreamer Config: Added dreamer.worktree and dreamer.execution config sections with sensible defaults (5min timeout, matrix-dreamer/ branch prefix). The implementation properly cascades config defaults through add.ts and scheduler/base.ts with backward compatibility.

• Database Updates: Changed default branch prefix from claude-task/ to matrix-dreamer/ in both schema.ts and migration v5. Added comprehensive test suite (schema.test.ts) that validates schema/migration synchronization - all 18 tests passing.

• Review Skill Architecture: Completely redesigned the /matrix:review skill with a 4-agent pipeline (Detection → Impact → Triage → Remediation) targeting 95% false positive reduction. The architecture includes:

  • Orchestrator with grep fallback when code index unavailable
  • Context7 integration for library-specific best practices
  • Tiered output system (critical/important/noise) with confidence calibration
  • Transitive blast radius analysis with 2-3 level depth control

Implementation Quality:

• Clean separation of concerns with typed interfaces
• Proper default cascading with ?? operators
• Doctor tool updated to validate new config sections
• Documentation is thorough and well-structured
• Test coverage prevents schema/migration divergence

No issues found. The changes are production-ready.

Confidence Score: 5/5

• Safe to merge - well-structured config changes with comprehensive test coverage
• Score reflects clean implementation with proper defaults, backward compatibility, and passing test suite that validates schema/migration sync
• No files require special attention

Important Files Changed

Filename	Overview
src/config/index.ts	Added well-structured dreamer config with clear defaults and typed interfaces
src/dreamer/actions/add.ts	Uses config defaults with proper fallback, changed prefix from claude-task/ to matrix-dreamer/
src/dreamer/scheduler/base.ts	Updated to use config defaults for branch prefix and remote, maintains backward compatibility
src/db/schema.ts	Updated default branch prefix to matrix-dreamer/, consistent with config
src/db/migrate.ts	Exported SCHEMA_VERSION for testing, updated branch prefix in migration
src/db/schema.test.ts	Comprehensive test suite validating schema/migration sync, all 18 tests passing

Sequence Diagram

sequenceDiagram
    participant User
    participant Orchestrator
    participant DetectionAgent
    participant ImpactAgent
    participant TriageAgent
    participant RemediationAgent
    participant Context7
    participant MatrixIndex

    User->>Orchestrator: /matrix:review [target] [mode]
    Orchestrator->>MatrixIndex: matrix_index_status()
    MatrixIndex-->>Orchestrator: {indexed: bool, stale: bool}
    
    alt Index Available
        Orchestrator->>Orchestrator: Set tools: matrix_find_callers, etc.
    else Index Unavailable/Stale
        Orchestrator->>Orchestrator: Set tools: grep fallback
    end

    Orchestrator->>Orchestrator: Parse target (staged/PR/file)
    Orchestrator->>DetectionAgent: Analyze files
    
    DetectionAgent->>DetectionAgent: Scan security patterns
    DetectionAgent->>DetectionAgent: Check runtime issues
    DetectionAgent->>DetectionAgent: Detect breaking changes
    DetectionAgent-->>Orchestrator: DetectionFinding[]
    
    alt Has Findings
        Orchestrator->>ImpactAgent: Calculate blast radius
        ImpactAgent->>MatrixIndex: matrix_list_exports(file)
        ImpactAgent->>MatrixIndex: matrix_find_callers(symbol)
        ImpactAgent->>ImpactAgent: Build transitive graph (2-3 levels)
        ImpactAgent->>ImpactAgent: Identify test coverage
        ImpactAgent-->>Orchestrator: ImpactGraph
        
        Orchestrator->>TriageAgent: Classify findings
        TriageAgent->>TriageAgent: Assign tiers (1/2/3)
        TriageAgent->>TriageAgent: Calibrate confidence
        TriageAgent->>TriageAgent: Filter noise (target 80% signal)
        TriageAgent-->>Orchestrator: TriagedFinding[]
        
        Orchestrator->>RemediationAgent: Generate fixes
        RemediationAgent->>MatrixIndex: matrix_search_symbols(pattern)
        RemediationAgent->>MatrixIndex: matrix_recall(query)
        
        alt Library Involved
            RemediationAgent->>Context7: resolve-library-id(libraryName)
            Context7-->>RemediationAgent: libraryId
            RemediationAgent->>Context7: query-docs(best practices)
            Context7-->>RemediationAgent: Documentation
        end
        
        RemediationAgent->>RemediationAgent: Generate context-aware fixes
        RemediationAgent->>RemediationAgent: Assess regression risk
        RemediationAgent-->>Orchestrator: RemediationSuggestion[]
    else No Critical Findings
        Orchestrator->>Orchestrator: Skip deep analysis (early exit)
    end
    
    Orchestrator->>Orchestrator: Aggregate results
    Orchestrator->>User: Final review with tiered output

Loading