-
Notifications
You must be signed in to change notification settings - Fork 112
feat(mcp): Add multi-bank access and new MCP tools #82
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
feat(mcp): Add multi-bank access and new MCP tools #82
Conversation
Enables orchestrator agents to access multiple memory banks from a single MCP connection, with new tools for bank management. ## New MCP Tools - `reflect` - Thoughtful analysis using bank's personality and memories - `list_banks` - Discover all available memory banks - `create_bank` - Create new banks programmatically ## Multi-Bank Access - Added optional `bank_id` parameter to `retain`, `recall`, `reflect` - Allows cross-bank operations from a single MCP session - Defaults to session bank if not specified ## Claude Code Compatibility - Enabled `stateless_http=True` for proper Claude Code integration - Responses now include `bank_id` for transparency ## Documentation - Added docker-compose.example.yml with env var substitution - Added HINDSIGHT-DOCKER.md setup guide with volume persistence docs - Updated .gitignore to exclude local docker-compose.yml ## Use Case Orchestrator agents can now: - Maintain a private meta-orchestration bank - Access shared project knowledge banks - Query across banks for cross-context insights 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
|
HINDSIGHT-DOCKER.md
Outdated
| @@ -0,0 +1,236 @@ | |||
| # Hindsight Docker Setup | |||
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.
please remove this file, we don't want to keep a docker compose for a simple installation. everything should be in the doc and if something missing you can add there
docker-compose.example.yml
Outdated
| @@ -0,0 +1,41 @@ | |||
| # Hindsight Docker Compose Configuration | |||
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.
please remove this file, we don't want to keep a docker compose for a simple installation. everything should be in the doc and if something missing you can add there
| Args: | ||
| query: Natural language search query (e.g., "user's food preferences", "what projects is user working on") | ||
| max_results: Maximum number of results to return (default: 10) | ||
| bank_id: Optional bank to search in (defaults to session bank). Use for cross-bank operations. |
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.
I don't think this is clear enough for the agent to understand what do to with it. have you tried it out already?
initalliy this was the only implementation but I had issues letting Claude to understand the bank_id param and what to use exactly. and when it gets confused by a tool usually it just ignores it
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.
You're right it should include a better tool description. Let me work on that. It works for me locally because I created a skill that uses Hindsight and it describes how to use it.
…ription - Remove HINDSIGHT-DOCKER.md and docker-compose.example.yml per reviewer request - Improve reflect tool description with clearer guidance for AI agents: - Added "WHEN TO USE THIS TOOL" section - Added "EXAMPLES OF GOOD QUERIES" with concrete use cases - Added "HOW IT DIFFERS FROM RECALL" to clarify when to use each tool 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
|
@nicoloboschi this has been updated. I'm not 100% certain if the reflect tool description reflects (pun intended) how you are recommending people use it. But it's how I've used it in my agent setup. Let me know if you'd like me to make any more tweaks! |
Summary
Enables orchestrator agents to access multiple memory banks from a single MCP connection, with new tools for bank management.
New MCP Tools
reflect(query, context?, budget?, bank_id?)list_banks()create_bank(bank_id, name?, background?)Multi-Bank Access
Added optional
bank_idparameter to existing tools:retain(content, context, bank_id?)- Store in specific bankrecall(query, max_results?, bank_id?)- Search specific bankreflect(query, context?, budget?, bank_id?)- Reflect using specific bankWhen
bank_idis not specified, the session's default bank is used.Claude Code Compatibility
stateless_http=Truefor proper Claude Code integrationbank_idfor transparencyDocumentation
docker-compose.example.ymlwith env var substitution (no secrets)HINDSIGHT-DOCKER.mdsetup guide with volume persistence warnings.gitignoreto exclude local docker-compose.ymlUse Case
Orchestrator agents managing multiple Claude Code instances can now:
Example: