From 23491d3b497f2ba7c23c101e39853d8ee6a7fbf0 Mon Sep 17 00:00:00 2001
From: mika kuns <mika.kuns@fb-tuning.de>
Date: Mon, 7 Sep 2026 08:17:00 +0200
Subject: [PATCH] feat(ui): Einfuegen und Datei-Drop im interaktiven Terminal

Ctrl+V wurde von TerminalControl als rohes Control-Byte an das Kind
weitergegeben und tat nichts; jetzt wird es im Tunnel abgefangen und geht
ueber TerminalView.PasteAsync, damit Bracketed Paste greift und ein
mehrzeiliger Paste nicht auf der ersten Zeile abschickt. Gedroppte Dateien
werden als Pfade in den Composer getippt, ohne Enter.
---
 .../Services/PtyTerminalSession.cs            | 36 +++++++++++++
 .../InteractiveTerminalViewModel.cs           |  8 +++
 .../Views/InteractiveTerminalView.axaml       |  3 +-
 .../Views/InteractiveTerminalView.axaml.cs    | 50 +++++++++++++++++++
 .../InteractiveTerminalDropPathsTests.cs      | 26 ++++++++++
 5 files changed, 122 insertions(+), 1 deletion(-)
 create mode 100644 tests/ClaudeDo.Ui.Tests/Views/InteractiveTerminalDropPathsTests.cs

diff --git a/src/ClaudeDo.Ui/Services/PtyTerminalSession.cs b/src/ClaudeDo.Ui/Services/PtyTerminalSession.cs
index 82980292..597469df 100644
--- a/src/ClaudeDo.Ui/Services/PtyTerminalSession.cs
+++ b/src/ClaudeDo.Ui/Services/PtyTerminalSession.cs
@@ -137,6 +137,42 @@ public sealed class PtyTerminalSession : IDisposable
         }
     }
 
+    /// <summary>
+    /// Inserts <paramref name="text"/> at the cursor without submitting it — the drop-a-file-path
+    /// case. Same write seam as <see cref="SendLineAsync"/>, just without the Enter.
+    /// </summary>
+    public async Task<bool> SendTextAsync(string text)
+    {
+        if (_control is not { } control || !IsRunning || string.IsNullOrEmpty(text)) return false;
+        try
+        {
+            await SendRawAsync(control, text);
+            return true;
+        }
+        catch (Exception)
+        {
+            return false;
+        }
+    }
+
+    /// <summary>
+    /// Pastes the clipboard into the session. <see cref="TerminalControl"/> wires paste to the
+    /// right-click menu only, so Ctrl+V otherwise reaches the child as a raw control byte and does
+    /// nothing. Goes through the emulator's own <see cref="TerminalView.PasteAsync"/> rather than
+    /// writing the clipboard text raw, because that is what honours bracketed-paste mode — without
+    /// it a multi-line paste is read line by line and submits on the first newline.
+    /// </summary>
+    public async Task PasteAsync()
+    {
+        if (_control is not { } control || !IsRunning) return;
+        try
+        {
+            if (s_terminalViewField?.GetValue(control) is TerminalView view)
+                await view.PasteAsync();
+        }
+        catch (Exception) { /* best-effort, same as SendLineAsync */ }
+    }
+
     // Long enough that the two writes can't be read as one chunk by the child's stdin reader.
     private static readonly TimeSpan SubmitDelay = TimeSpan.FromMilliseconds(150);
 
diff --git a/src/ClaudeDo.Ui/ViewModels/InteractiveTerminalViewModel.cs b/src/ClaudeDo.Ui/ViewModels/InteractiveTerminalViewModel.cs
index ca72e00c..b0fed886 100644
--- a/src/ClaudeDo.Ui/ViewModels/InteractiveTerminalViewModel.cs
+++ b/src/ClaudeDo.Ui/ViewModels/InteractiveTerminalViewModel.cs
@@ -136,6 +136,14 @@ public sealed partial class InteractiveTerminalViewModel : ViewModelBase, IDispo
     /// type into (or the write seam is gone).</summary>
     public Task<bool> SendLineAsync(string text) => _session.SendLineAsync(text);
 
+    /// <summary>Inserts text at the cursor without submitting — see
+    /// <see cref="PtyTerminalSession.SendTextAsync"/> (dropped file paths).</summary>
+    public Task<bool> SendTextAsync(string text) => _session.SendTextAsync(text);
+
+    /// <summary>Pastes the clipboard into the session (Ctrl+V) — see
+    /// <see cref="PtyTerminalSession.PasteAsync"/>.</summary>
+    public Task PasteAsync() => _session.PasteAsync();
+
     public void Kill() => _session.Kill();
 
     public void Dispose()
diff --git a/src/ClaudeDo.Ui/Views/InteractiveTerminalView.axaml b/src/ClaudeDo.Ui/Views/InteractiveTerminalView.axaml
index 8277ddbb..520972aa 100644
--- a/src/ClaudeDo.Ui/Views/InteractiveTerminalView.axaml
+++ b/src/ClaudeDo.Ui/Views/InteractiveTerminalView.axaml
@@ -4,7 +4,8 @@
              xmlns:term="using:Iciclecreek.Terminal"
              x:Class="ClaudeDo.Ui.Views.InteractiveTerminalView"
              x:DataType="vm:InteractiveTerminalViewModel"
-             x:Name="Root">
+             x:Name="Root"
+             DragDrop.AllowDrop="True">
   <!--
     Process="" suppresses TerminalView.OnLoaded's built-in auto-launch (it otherwise spawns its
     own cmd.exe/bash the moment this control loads). PtyTerminalSession sets Process/Args/
diff --git a/src/ClaudeDo.Ui/Views/InteractiveTerminalView.axaml.cs b/src/ClaudeDo.Ui/Views/InteractiveTerminalView.axaml.cs
index 2eaee755..c84e9558 100644
--- a/src/ClaudeDo.Ui/Views/InteractiveTerminalView.axaml.cs
+++ b/src/ClaudeDo.Ui/Views/InteractiveTerminalView.axaml.cs
@@ -1,6 +1,10 @@
 using System;
+using System.Collections.Generic;
+using System.Linq;
 using Avalonia.Controls;
+using Avalonia.Input;
 using Avalonia.Interactivity;
+using Avalonia.Platform.Storage;
 using ClaudeDo.Ui.ViewModels;
 
 namespace ClaudeDo.Ui.Views;
@@ -12,6 +16,12 @@ public partial class InteractiveTerminalView : UserControl
         InitializeComponent();
         TerminalHost.Loaded += OnTerminalHostLoaded;
         DataContextChanged += OnDataContextChanged;
+
+        // Tunnel: TerminalControl handles KeyDown itself and forwards Ctrl+V to the child as a raw
+        // control byte, where nothing acts on it — paste has to be intercepted before it gets there.
+        AddHandler(KeyDownEvent, OnPreviewKeyDown, RoutingStrategies.Tunnel);
+        AddHandler(DragDrop.DragOverEvent, OnDragOver);
+        AddHandler(DragDrop.DropEvent, OnDrop);
     }
 
     private void OnDataContextChanged(object? sender, EventArgs e)
@@ -25,4 +35,44 @@ public partial class InteractiveTerminalView : UserControl
         if (DataContext is InteractiveTerminalViewModel vm)
             vm.AttachControl(TerminalHost);
     }
+
+    private void OnPreviewKeyDown(object? sender, KeyEventArgs e)
+    {
+        if (e.Key != Key.V || !e.KeyModifiers.HasFlag(KeyModifiers.Control) || e.KeyModifiers.HasFlag(KeyModifiers.Alt))
+            return;
+        if (DataContext is not InteractiveTerminalViewModel vm) return;
+
+        e.Handled = true;
+        _ = vm.PasteAsync();
+    }
+
+    private static bool IsFilesDrop(DragEventArgs e) => e.DataTransfer?.Contains(DataFormat.File) == true;
+
+    private void OnDragOver(object? sender, DragEventArgs e)
+    {
+        e.DragEffects = IsFilesDrop(e) ? DragDropEffects.Copy : DragDropEffects.None;
+        e.Handled = true;
+    }
+
+    // Dropping files types their paths into the session's composer (no Enter) — the common case is
+    // "look at this file", not "run something".
+    private void OnDrop(object? sender, DragEventArgs e)
+    {
+        if (DataContext is not InteractiveTerminalViewModel vm || !IsFilesDrop(e)) return;
+        e.Handled = true;
+
+        var text = FormatDroppedPaths(e.DataTransfer!.TryGetFiles()?.Select(i => i.TryGetLocalPath()));
+        if (text.Length > 0) _ = vm.SendTextAsync(text);
+    }
+
+    /// <summary>Joins dropped file paths into one insertable chunk: quoted when they contain a
+    /// space, space-separated, with a trailing space so the user can keep typing.</summary>
+    internal static string FormatDroppedPaths(IEnumerable<string?>? paths)
+    {
+        var usable = (paths ?? Enumerable.Empty<string?>())
+            .Where(p => !string.IsNullOrWhiteSpace(p))
+            .Select(p => p!.Contains(' ') ? $"\"{p}\"" : p)
+            .ToList();
+        return usable.Count == 0 ? string.Empty : string.Join(" ", usable) + " ";
+    }
 }
diff --git a/tests/ClaudeDo.Ui.Tests/Views/InteractiveTerminalDropPathsTests.cs b/tests/ClaudeDo.Ui.Tests/Views/InteractiveTerminalDropPathsTests.cs
new file mode 100644
index 00000000..48d34130
--- /dev/null
+++ b/tests/ClaudeDo.Ui.Tests/Views/InteractiveTerminalDropPathsTests.cs
@@ -0,0 +1,26 @@
+using ClaudeDo.Ui.Views;
+using Xunit;
+
+namespace ClaudeDo.Ui.Tests.Views;
+
+public class InteractiveTerminalDropPathsTests
+{
+    [Fact]
+    public void Quotes_only_paths_with_spaces_and_ends_with_a_separator()
+    {
+        var text = InteractiveTerminalView.FormatDroppedPaths(
+            new[] { @"C:\repo\a.cs", @"C:\my repo\b.cs" });
+
+        Assert.Equal("C:\\repo\\a.cs \"C:\\my repo\\b.cs\" ", text);
+    }
+
+    [Fact]
+    public void Skips_items_without_a_local_path()
+    {
+        // IStorageItem.TryGetLocalPath() returns null for non-filesystem items (e.g. a virtual
+        // shell folder) — those must not turn into empty quotes in the composer.
+        Assert.Equal(@"C:\repo\a.cs ", InteractiveTerminalView.FormatDroppedPaths(new[] { null, @"C:\repo\a.cs", "  " }));
+        Assert.Equal(string.Empty, InteractiveTerminalView.FormatDroppedPaths(new string?[] { null }));
+        Assert.Equal(string.Empty, InteractiveTerminalView.FormatDroppedPaths(null));
+    }
+}