Skip to content

Comments

Remove unused imports from test_tools.py#31

Open
Copilot wants to merge 2 commits intobugfix-reasoning-trace-searchfrom
copilot/sub-pr-30
Open

Remove unused imports from test_tools.py#31
Copilot wants to merge 2 commits intobugfix-reasoning-trace-searchfrom
copilot/sub-pr-30

Conversation

Copy link

Copilot AI commented Jan 25, 2026

Cleans up unused imports in examples/lennys-memory/backend/tests/test_tools.py identified during code review.

Changes

  • Removed three unused imports from test file:
    • get_memory_stats
    • get_user_preferences
    • search_by_episode

All remaining imports are actively used in test cases. The functions exist in src/agent/tools.py but have no corresponding test coverage in this file.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@vercel
Copy link

vercel bot commented Jan 25, 2026

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

Project Deployment Review Updated (UTC)
agent-memory Ready Ready Preview, Comment Jan 25, 2026 5:38pm

Co-authored-by: johnymontana <1222454+johnymontana@users.noreply.github.com>
Copilot AI changed the title [WIP] Update demo app search tool to address feedback Remove unused imports from test_tools.py Jan 25, 2026
Copilot AI requested a review from johnymontana January 25, 2026 17:38
@johnymontana johnymontana marked this pull request as ready for review February 6, 2026 17:22
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.

2 participants