-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Feature/context memory #472
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Implement ContextManager for maintaining task, technical, and user context - Add context validation and state persistence - Integrate context tracking in Cline.ts - Update mock VSCode implementation for context support - Add sliding window tests - Update dependencies and configuration
…our Context Memory Feature
- Fixed getModeConfigId method casing to match implementation - Corrected syntax issues with proper commas and semicolons - Resolved test case structure and removed duplicate brackets - All tests now passing
- Implemented ContextManager with VSCode state integration - Added mode-specific memory limits (Code/Architect/Ask) - Created shared context defaults for extension and webview - Added pattern tracking and mistake prevention - Implemented task progress and technical context tracking - Completed UI integration in PromptsView - All tests passing and ready for 3.3.0 release
… Hard to keep up with you and Roo.
| contextMemoryModeSettings[mode]?.maxHistoryItems ?? | ||
| DEFAULT_CONTEXT_SETTINGS[mode]?.maxHistoryItems | ||
| } | ||
| onChange={(e) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider extracting the repeated logic for updating contextMemoryModeSettings into a helper function to reduce redundancy. This is in line with our Development Standards: https://www.notion.so/Development-Standards-59febcf8ead647fd9c2ec3f60c22f3df?pvs=4#11869ad2d5818094a05ef707e188c0d5. This applies to similar logic on lines 749 and 783 as well.
|
Amazing! Excited to try this out 🙏 |
|
Enjoy! LMK if you have any questions.
On January 21, 2025, Github ***@***.***> wrote:
Amazing! Excited to try this out 🙏
—
Reply to this email directly, view it on GitHub
<#472 (comment)
2606146731>, or unsubscribe
<https://github.com/notifications/unsubscribe-
auth/AAA547MQB35DKI5BK3C64O32L365BAVCNFSM6AAAAABVT3UJ6SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMMBWGE2DMNZTGE>.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
|
Closing because I really hammered on it tonight and still some issues need fixing. Coming soon though. |
|
@dustmoo any luck on this? |
|
@hannesrudolph - Traveling abroad until Thursday, will be finishing up
when I return. Cheers
…On April 15, 2025, Github ***@***.***> wrote:
@dustmoo <https://github.com/dustmoo> any luck on this?
—
Reply to this email directly, view it on GitHub
<#472 (comment)
2803845417>, or unsubscribe
<https://github.com/notifications/unsubscribe-
auth/AAA547M75NID7XQ23O2JAKL2ZSJ57AVCNFSM6AAAAABVT3UJ6SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDQMBTHA2DKNBRG4>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
hannesrudolph left a comment (RooCodeInc/Roo-Code#472)
<#472 (comment)
2803845417>
@dustmoo <https://github.com/dustmoo> any luck on this?
—
Reply to this email directly, view it on GitHub
<#472 (comment)
2803845417>, or unsubscribe
<https://github.com/notifications/unsubscribe-
auth/AAA547M75NID7XQ23O2JAKL2ZSJ57AVCNFSM6AAAAABVT3UJ6SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDQMBTHA2DKNBRG4>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I look forward to this! Reach out to me on discord when you get a chance once you're working on this again. |
Description
Added a Context Memory System to enhance AI-human collaboration through persistent, mode-aware state management. This feature implements a sophisticated memory system that adapts to different interaction modes while maintaining state across sessions.
Type of change
How Has This Been Tested?
Automated Tests
Manual Testing
Implementation Details
ContextMemorySchemawith Zod validation for type-safe settingsContextManagerclass for managing persistent state using VSCode's global stateChecklist:
Additional context
The Context Memory System enhances the AI's ability to maintain consistent behavior across sessions while adapting to different interaction modes. It uses VSCode's global state for persistence and implements mode-specific limits for optimal memory management.
Related Issues
Implements the AI Code Collaboration Framework Requirements for persistent memory and context management.
Reviewers
@mrubens - We haven't met but I love the thing I built upon the thing you built upon. Happy to discuss. I debated making the a 4. change, but since I'm caught up with you it shouldn't be breaking.
Cheers!
Important
Introduces a Context Memory System for persistent, mode-aware state management in VSCode, with new classes, tests, and UI updates.
ContextMemorySchemafor type-safe settings validation.ContextManagerclass for managing persistent state using VSCode's global state.ContextManagerinContextManager.test.ts.ClineProviderinClineProvider.test.ts.jest.config.jsto ignoreoutdirectory.eslint-config-react-apptodevDependenciesinpackage.json.Cline.tsto integrateContextManager.This description was created by
for b10866c. It will automatically update as commits are pushed.