Skip to content

feat: Uteke memory integration (#232)#256

Merged
ajianaz merged 1 commit into
developfrom
feat/232-uteke-memory-integration
Jun 11, 2026
Merged

feat: Uteke memory integration (#232)#256
ajianaz merged 1 commit into
developfrom
feat/232-uteke-memory-integration

Conversation

@ajianaz

@ajianaz ajianaz commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Implements #232 — optional Uteke memory integration for review history and learning.

New

  • src/engine/memory.rs — MemoryBackend with auto-detect, recall, save, feedback
  • --memory flag — recall project patterns from Uteke before review
  • --learn flag — recall + save findings after review (implies --memory)
  • 11 unit tests (506 total)

Design

  • Integration via subprocess (uteke recall/uteke remember CLI)
  • No Rust library dependency — clean separation
  • Graceful degradation: Cora works 100% without Uteke
  • Best-effort: memory failures never fail the review

Flow

cora review                  → standalone (default, no change)
cora review --memory         → recall context, enrich prompt
cora review --memory --learn → recall + save findings

Phase markers (follow-up)

  • TODO in main.rs: inject memory_context into LLM prompt
  • TODO in main.rs: extract actual findings count from ReviewResult

Implements optional Uteke memory integration for AI-powered review memory.

New:
- src/engine/memory.rs — MemoryBackend with detect/recall/save/feedback
- --memory flag — recall project patterns from Uteke before review
- --learn flag — recall + save findings after review (implies --memory)
- 11 unit tests

Integration is purely additive via subprocess (uteke recall/remember CLI).
Cora works 100% without Uteke — graceful degradation when not installed.

Flow:
  cora review                  → standalone (default, no change)
  cora review --memory         → recall context, enrich prompt
  cora review --memory --learn → recall + save findings

Phase 2: inject memory_context into LLM prompt (TODO marker in main.rs)
Phase 3: extract actual findings count from ReviewResult for save_findings
@ajianaz
ajianaz merged commit d1ebbf8 into develop Jun 11, 2026
10 checks passed
@ajianaz
ajianaz deleted the feat/232-uteke-memory-integration branch June 11, 2026 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant