Skip to content

Fix MCP tool registration and session storage initialization#3

Merged
DerAuctor merged 5 commits intomainfrom
fix/unified-tool-manager-context
Sep 28, 2025
Merged

Fix MCP tool registration and session storage initialization#3
DerAuctor merged 5 commits intomainfrom
fix/unified-tool-manager-context

Conversation

@Ptah-CT
Copy link
Copy Markdown
Collaborator

@Ptah-CT Ptah-CT commented Sep 26, 2025

This PR fixes issues with MCP tool registration logic and session storage initialization to resolve context problems and improve overall reliability.

Changes

  • Simplified MCP tool registration in mcp_handler.ts
  • Enhanced session storage manager initialization in conversation-session.ts
  • Updated tool name sanitization in manager.ts
  • Modified configuration in XInfty.yml and package.json

Testing

  • Verified MCP tool exposure works correctly
  • Confirmed session storage initialization is robust
  • Checked for any breaking changes in tool naming

Resolves context issues in unified tool manager.

…o backend

- Add apiKey field to YAML config for OpenRouter env var expansion
- Implement Turso backend for improved data persistence
- Update vector storage configurations and manager
- Minor linting improvements and documentation updates
- Address SQLite MCP memory issues with cloud storage integration
- Fixed UnifiedToolManager context unavailability errors
- Implemented lazy context injection in session-manager.ts
- Enhanced LLM decision parsing with Zod validation and retries
- Added MCP failure integration tests
- Implemented review findings: ask_cipher routing, embedding env flag, MCP conflict test, Gemini docs
… resolve context issues and improve reliability
@DerAuctor DerAuctor merged commit bce4d2a into main Sep 28, 2025
Ptah-CT pushed a commit that referenced this pull request Oct 2, 2025
Ptah-CT pushed a commit that referenced this pull request Oct 2, 2025
…etoolnameconflict-for-first-wins_2025-09-18_16-18-04

Handle first-wins conflicts without duplicate registrations
@DerAuctor DerAuctor deleted the fix/unified-tool-manager-context branch October 2, 2025 22:02
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