Add MongoDB-backed NotesTool with CRUD actions and unit tests

[dhairya-eng]

📖 Description

This PR implements a new NotesTool that allows LLMs to write, edit, retrieve, and delete notes. Notes are persisted in MongoDB, enabling agents to maintain and manipulate contextual information across interactions.

🔖 Feature Details

New Tool: NotesTool in application/agents/tools/notes.py

Actions supported:

add_note → store a note in MongoDB

get_notes → retrieve all notes for a user

edit_note → update a note by its ID

delete_note → remove a note by its ID

🧪 Testing

Added tests/test_notes_tool.py with unit tests covering all CRUD actions

Verified that pytest passes locally with all tests green

Ensures compliance with PEP8, type hints, and Google-style docstrings

ruff check passes

🎤 Why this is needed

Enables LLMs to perform more complex multi-step tasks by persisting intermediate notes. Improves agent reasoning and task management capabilities.

🔗 Issue Reference

Fixes #1965

✅ Checklist

Code follows PEP8 and repo coding standards

Added unit tests for all new functionality

All tests pass locally (pytest)

Lint checks pass (ruff)

[vercel]

@dhairya-eng is attempting to deploy a commit to the Arc53 Team on Vercel.

A member of the Team first needs to authorize it.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
oss-docsgpt	Ready	Preview	Comment	Oct 6, 2025 3:47pm

[dartpain]

Thank you for your PR,

I have 2 comments:

user_id should come from decoded_token, "sub" property, this ensures safety.
There should not be multiple notes possible for a user just 1 for simplicity.

[dhairya-eng]

Thank you for your PR,

I have 2 comments:

user_id should come from decoded_token, "sub" property, this ensures safety. There should not be multiple notes possible for a user just 1 for simplicity.

Ok I will work on this

[dhairya-eng]

Thanks for the review
• Updated NotesTool to require user_id from decoded_token['sub'].
• Enforced single-note-per-user semantics using upsert/update/delete.
• Fixed tests (using monkeypatch, no extra plugins).
Confirmed full test suite passes locally.

[dartpain]

Seems like you accidentally commited:
results.txt
please remove it,
thank you!

[codecov]

Codecov Report

❌ Patch coverage is 86.11111% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 35.09%. Comparing base (5625605) to head (8367e06).
⚠️ Report is 80 commits behind head on main.

Files with missing lines	Patch %	Lines
application/agents/tools/notes.py	87.14%	9 Missing ⚠️
application/agents/tools/tool_manager.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #1982       +/-   ##
===========================================
+ Coverage   24.36%   35.09%   +10.73%     
===========================================
  Files         106      132       +26     
  Lines        8057     8773      +716     
===========================================
+ Hits         1963     3079     +1116     
+ Misses       6094     5694      -400     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[dhairya-eng]

Seems like you accidentally commited: results.txt please remove it, thank you!

sure I will do that

[dhairya-eng]

Pushed with new updates

[dhairya-eng]

Did you find any issues while reviewing the code?

[dhairya-eng]

Can you add me in the contributer list if possible.