Skip to content

Fix paste detection triggering on fast typing #35

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
DevMando merged 1 commit into from
Mar 4, 2026
Merged

Fix paste detection triggering on fast typing #35

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
59 changes: 18 additions & 41 deletions src/MandoCode/Services/CommandAutocomplete.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,6 @@ public static string ReadLineWithAutocomplete()
List<string> filteredCommands = new();
List<string> filteredFiles = new();
int atAnchorPos = -1;
string? pastedFullText = null; // holds full content when paste is detected

while (true)
{
Expand Down Expand Up @@ -103,7 +102,7 @@ public static string ReadLineWithAutocomplete()
if (autocompleteMode != AutocompleteMode.None)
ClearAutocompleteDisplay(ref cursorTop);
Console.WriteLine();
return pastedFullText ?? input.ToString();
return input.ToString();
}

case ConsoleKey.Tab:
Expand Down Expand Up @@ -177,24 +176,6 @@ public static string ReadLineWithAutocomplete()
continue;

case ConsoleKey.Backspace:
if (pastedFullText != null)
{
// Clear the entire paste — start fresh
pastedFullText = null;
input.Clear();
Console.SetCursorPosition(cursorLeft, cursorTop);
Console.Write(new string(' ', Math.Max(0, Console.WindowWidth - cursorLeft - 1)));
Console.SetCursorPosition(cursorLeft, cursorTop);
if (autocompleteMode != AutocompleteMode.None)
{
ClearAutocompleteDisplay(ref cursorTop);
Console.SetCursorPosition(cursorLeft, cursorTop);
}
autocompleteMode = AutocompleteMode.None;
atAnchorPos = -1;
selectedIndex = 0;
continue;
}
if (input.Length > 0)
{
input.Length--;
Expand Down Expand Up @@ -281,38 +262,34 @@ public static string ReadLineWithAutocomplete()
// Detect paste: if more characters are immediately buffered, consume them all
if (Console.KeyAvailable)
{
var pasteChars = new StringBuilder();
pasteChars.Append(key.KeyChar);
var bufferedChars = new List<char>();
bufferedChars.Add(key.KeyChar);

while (Console.KeyAvailable)
{
var pk = Console.ReadKey(intercept: true);
if (pk.Key == ConsoleKey.Enter || pk.KeyChar == '\n' || pk.KeyChar == '\r')
pasteChars.Append(' '); // newlines → spaces
bufferedChars.Add(' '); // newlines → spaces
else if (!char.IsControl(pk.KeyChar))
pasteChars.Append(pk.KeyChar);
bufferedChars.Add(pk.KeyChar);
}

// Store full content: any text typed before + pasted chars
var beforePaste = input.ToString();
pastedFullText = beforePaste + pasteChars.ToString();

// Update display to show summary
input.Clear();
input.Append($"[pasted {pastedFullText.Length} characters]");
// Only treat as paste if 5+ chars arrived at once; otherwise it's fast typing
// Append all buffered chars normally (whether fast typing or paste)
foreach (var c in bufferedChars)
{
input.Append(c);
Console.Write(c);
}

// Close any autocomplete
// Close any autocomplete if open
if (autocompleteMode != AutocompleteMode.None)
{
ClearAutocompleteDisplay(ref cursorTop);
autocompleteMode = AutocompleteMode.None;
atAnchorPos = -1;
selectedIndex = 0;

// Redraw input line with paste summary
Console.SetCursorPosition(cursorLeft, cursorTop);
Console.Write(new string(' ', Math.Max(0, Console.WindowWidth - cursorLeft - 1)));
Console.SetCursorPosition(cursorLeft, cursorTop);
Console.Write(input.ToString());
autocompleteMode = AutocompleteMode.None;
atAnchorPos = -1;
selectedIndex = 0;
}
continue;
}

Expand Down