Skip to content

Add generic memory tool#120

Open
KavyaSree2610 wants to merge 20 commits intomainfrom
kkaitepalli/genericmemorytool
Open

Add generic memory tool#120
KavyaSree2610 wants to merge 20 commits intomainfrom
kkaitepalli/genericmemorytool

Conversation

@KavyaSree2610
Copy link
Collaborator

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Mar 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.22%. Comparing base (ba575dc) to head (77578bd).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #120      +/-   ##
==========================================
+ Coverage   95.88%   96.22%   +0.33%     
==========================================
  Files          26       28       +2     
  Lines        1046     1219     +173     
==========================================
+ Hits         1003     1173     +170     
- Misses         43       46       +3     
Flag Coverage Δ
integration 59.72% <19.07%> (-6.73%) ⬇️
ollama_local 55.94% <19.07%> (-6.10%) ⬇️
slow-browser 45.36% <19.07%> (-4.35%) ⬇️
slow-other 67.18% <55.49%> (-1.65%) ⬇️
unit 83.83% <100.00%> (+3.15%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
src/microbots/tools/tool_definitions/__init__.py 100.00% <100.00%> (ø)
...rc/microbots/tools/tool_definitions/memory_tool.py 100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@KavyaSree2610 KavyaSree2610 marked this pull request as ready for review March 12, 2026 06:39
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR introduces a provider-agnostic, file-backed “memory” tool to persist notes across agent steps, along with unit/integration tests and a small CI workflow trigger update.

Changes:

  • Added MemoryTool implementing memory view/create/str_replace/insert/delete/rename/clear with a host filesystem-backed store under ~/.microbots/memory (or a configured directory).
  • Added comprehensive unit tests for path resolution and subcommand behaviors, plus bot-level wiring/integration tests (mocked providers and a slow real integration).
  • Enabled manual triggering for the Docker build/push workflow via workflow_dispatch.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/microbots/tools/tool_definitions/memory_tool.py Implements the new host-backed MemoryTool and its command dispatch + path sandboxing.
src/microbots/tools/tool_definitions/__init__.py Exposes MemoryTool from the tool_definitions package.
test/tools/tool_definitions/test_memory_tool.py Adds unit coverage for path handling and each MemoryTool subcommand.
test/bot/test_memory_tool_integration.py Adds MicroBot dispatch/wiring tests (mocked providers) and a slow end-to-end integration scenario.
.github/workflows/dockerBuildPush.yml Allows manual workflow runs via workflow_dispatch.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

…tion messages and update usage instructions
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.

4 participants