Conversation
|
I think to be a side car there needs to be a change to main.ts to process a message to increase underlying window to the right (you can't do it inside the one window) - otherwise we take real estate away from what is there (which is fine if default was wider?) - do you think sidecars should slide out to the right @aharvard ? |
This commit combines two major enhancements to create a unified interactive UI system: 🎯 MCP-UI Integration: - Add @mcp-ui/client dependency for rendering interactive UI components - Implement UIResourceRenderer component with secure iframe sandboxing - Add UI resource detection and automatic UI/text content switching - Update Accept headers to support MCP-UI content types (remote-dom, HTML, URI lists) - Enhance Content Security Policy to allow iframe rendering with ui: scheme 🔧 Checkpoint & Sidecar System (from PR block#3493): - Add SidecarLayout component for side-by-side UI rendering - Implement checkpoint system with automatic file backup and versioning - Add intelligent diff generation with file-type aware context sizing - Create RestoreModal and MessageRestoreLink for file restoration workflow - Add list_checkpoints and restore_checkpoint tools to developer router 🔗 Unified Architecture: - Sidecar can display both diff content AND interactive MCP-UI components - Enhanced message filtering to support tool request/response matching - Resource type system handles both UI resources and checkpoint data - Integrated debug logging for MCP-UI resource detection This creates a comprehensive interactive AI agent interface that combines: - Rich, interactive UI components via MCP-UI standard - File versioning and diff management via checkpoint system - Side-by-side layout for optimal workflow management Resolves: Unifies MCP-UI integration efforts with checkpoint/sidecar functionality
Kvadratni
force-pushed
the
feature/diff-viewer-sidecar
branch
from
453d83c to
1e963e6
Compare
|
It was doing that. My bug. Lost it during rebase |
Kvadratni
force-pushed
the
feature/diff-viewer-sidecar
branch
from
b7df644 to
aa57f48
Compare
|
fixed some issues @Kvadratni : #3520 |
|
two things are still bothering me
not blockers, but I'm gonna try to address those before my EOD |
| } | ||
| return false; | ||
| }); | ||
| }, [filteredMessages, messages]); |
There was a problem hiding this comment.
dont use useMemo on lists
Kvadratni
force-pushed
the
feature/diff-viewer-sidecar
branch
from
ba8b08b to
8ee9e77
Compare
Kvadratni
force-pushed
the
feature/diff-viewer-sidecar
branch
from
8ee9e77 to
3b7ad8a
Compare
Kvadratni
force-pushed
the
feature/diff-viewer-sidecar
branch
from
3b7ad8a to
b4b6bfb
Compare
|
main thing I would like to change is that the developer MCP doesn't assume any interactivity (be that checkpoints, diff viewer) out of the box - as headless mode is common for it so you don't want tools in there in that case which aren't relevant to it, increasing context and confusion. |
|
@Kvadratni @michaelneale @DOsinga do we want to keep going on this? |
|
I will keep it as a reference. I will try to reimplement using MCP-UI. Will mark as draft |
|
thanks, closing this for now |
5 participants
this is a fresh branch to fix rebase on main for #3040
Implement checkpoint system with intelligent diff splitting
Adds file versioning and enhanced diff handling for better change management and review.
Checkpoint system:
Enhanced diff generation:
New tools:
list_checkpoints- View available file versionsrestore_checkpoint- Revert files to previous statesIntegration:
Addresses large diff readability issues while providing robust file versioning capabilities.
new-diff.mp4