feat: add honcho context injection controls#81
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
WalkthroughThis PR adds configurable context injection controls and memory backend selection. New configuration types and schema enable users to toggle peer card insertion, session summary inclusion, and representation depth, plus choose between "qmd" and "builtin" memory backends. Context injection logic and runtime initialization now respect these configuration flags. ChangesContext Injection and Memory Backend Configuration
Estimated Code Review Effort🎯 3 (Moderate) | ⏱️ ~25 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint skipped: no ESLint configuration detected in root package.json. To enable, add Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…-injection-controls # Conflicts: # runtime.ts # test/runtime.test.ts
Summary
This is split out from #78 and intentionally avoids capture-path changes.
Validation
pnpm exec vitest run test/runtime.test.tspnpm buildSummary by CodeRabbit