Combined MCP improvements: security, token updates, and cleanup#992
Closed
ANierbeck wants to merge 2 commits intoRightNow-AI:mainfrom
Closed
Combined MCP improvements: security, token updates, and cleanup#992ANierbeck wants to merge 2 commits intoRightNow-AI:mainfrom
ANierbeck wants to merge 2 commits intoRightNow-AI:mainfrom
Conversation
This commit includes: - Inline approval UI for both TUI and WebUI - Keyboard shortcuts for quick approval/rejection (A/R keys) - Risk level visualization and navigation - WebUI API integration for approval actions - Fixed undefined _audioFile property access - Enhanced global state management for approvals - Automatic approval loading and polling Resolves the UX issue where users had to navigate to a separate Approval menu that didn't show pending requests properly.
Author
|
to many failing tests, I'll close this one and will maybe provide a better PR |
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
1 participant
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.
Combined MCP Improvements
Summary
This PR combines four related commits that improve MCP (Multi-Agent Communication Protocol) functionality into a single, focused change.
Changes Made
1. Improved MCP Header Handling and Security
3. Updated Authorization Token in mcp-a2a.md
Testing ▆
All existing tests continue to pass. The changes are primarily documentation and security improvements that don't affect core functionality.
Related Issues
None - this is a cleanup/refactoring PR to improve code organization.
Checklist
cargo test --workspace)cargo clippy --workspace -- -D warnings)cargo fmt --all)