Draft
Conversation
Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
…ovements Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
…support Add remote MCP server support with settings UI
Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
Enable BuildConfig generation for memory-vault debug build
Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
[WIP] Check AI models access to MCP servers and fix any issues
Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
…ns field Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
- Fix parseResponse() to always try SSE parsing regardless of transport type - Rename test class from McpServerIntegrationTest to McpServerTest - Use exact assertions instead of permissive contains() checks - Add helper function documentation and UUID format validation - Reduce UUID test iterations from 100 to 10 for efficiency Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
…p-server Add MCP server integration tests
- Fix sanitizeIdentifier creating double underscores for consecutive special characters (e.g., 'My--Tool' now correctly becomes 'my_tool') - Add text-parsing fallback when native grammar fails to emit ToolCall events, preventing raw JSON from being displayed to users - Fix race condition in executeToolCall by retrying syncMcpTools once if tool registry lookup returns null - Clear stale tools in syncMcpTools catch block to prevent broken state - Fix singleton close() not shutting down OkHttpClient dispatcher (which made subsequent requests fail permanently) - Validate SSE parsed data has JSON-RPC fields before accepting - Improve listTools error logging with server name context - Add tests for sanitizeIdentifier edge cases Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- MCP Store screen with browsable registry of MCP servers - Fetches registry from remote GitHub URL, falls back to bundled JSON - Category filtering, search, one-tap install to Room database - Badges for API key requirements and Termux dependencies - Termux integration for running local Python MCP servers - TermuxBridge utility: detect Termux, run commands via RUN_COMMAND intent - pip install flow for Python-based MCP servers - Auto-configure localhost URLs for local servers - Setup dialog guides users to install Termux if not present - Database migration v5→v6: add isLocal and sourceStoreId columns - Navigation: McpStore route + Store button on McpServersScreen top bar - Registry seeded with 10 popular MCP servers (Brave, GitHub, DuckDuckGo, etc.) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
… system Merge upstream changes (Character Cards, AI Memory, Plugin system, TTS, multi-turn generation) while preserving MCP server support, MCP Store, and Termux integration. Key conflict resolutions: - AppDatabase: bump to v7, add MIGRATION_6_7 for MCP tables alongside upstream's persona/ai_memory migrations (4→5, 5→6) - ChatViewModel: take upstream's PluginManager agent loop, add MCP tool registry and execution fallback in agent loop - Converters: keep both McpTransportType and StringList converters - MainActivity: merge MCP and upstream screen routes (Personas, AiMemory, Settings, McpServers, McpStore) - HomeDrawerScreen/HomeScreen: add MCP servers button alongside upstream's chatViewModel and onCharacterClick params - LlmModelWorker: take upstream's new tool calling API (enableToolCallingGguf, multi-turn generation) - ChatViewModelFactory: take upstream's Context param (MCP deps accessed via AppContainer) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Add remote MCP support to new branch
No repository changes required for branch synchronization request
Mar 15, 2026
Copilot
AI
changed the title
No repository changes required for branch synchronization request
No code changes; request requires repository branch administration
Mar 15, 2026
Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
Co-authored-by: Godzilla675 <131464726+Godzilla675@users.noreply.github.com>
Copilot
AI
changed the title
No code changes; request requires repository branch administration
Teach MCP-aware prompts about task tokens
Mar 15, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The MCP/tooling prompts did not describe the new task-token model, so the assistant could misinterpret task-scoped credentials and claim they could be revoked directly. This update teaches the prompt stack how task tokens behave and fixes a coupled MCP tool-call handling bug in the same path.
Prompt behavior
ChatViewModelfor MCP-enabled sessions.MCP execution path
ChatViewModelto unwrapResult<String>fromMcpClientService.callTool(...)before buildingMultiTurnToolResult.Focused regression coverage
Coupled compile fix
AnimatedVisibilitycompile issue inMcpStoreScreenencountered while validating the MCP/chat prompt path.Example of the prompt guidance now injected for MCP-enabled flows:
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.