From cff9671bdf358c2f8c06538b7e32c922b8e69c79 Mon Sep 17 00:00:00 2001
From: Gregory Anders <greg@gpanders.com>
Date: Sun, 18 Feb 2024 09:21:52 -0600
Subject: [PATCH] Omit release events for backspace, enter, and tab if
 report_all is unset

---
 src/input/KeyEncoder.zig | 80 +++++++++++++++++++++++++++++++++++++++-
 1 file changed, 78 insertions(+), 2 deletions(-)

diff --git a/src/input/KeyEncoder.zig b/src/input/KeyEncoder.zig
index a14ba14a9..49e45e8a1 100644
--- a/src/input/KeyEncoder.zig
+++ b/src/input/KeyEncoder.zig
@@ -50,8 +50,20 @@ fn kitty(
     if (self.kitty_flags.int() == 0) return try self.legacy(buf);
 
     // We only processed "press" events unless report events is active
-    if (self.event.action == .release and !self.kitty_flags.report_events)
-        return "";
+    if (self.event.action == .release) {
+        if (!self.kitty_flags.report_events) {
+            return "";
+        }
+
+        // Enter, backspace, and tab do not report release events unless "report
+        // all" is set
+        if (!self.kitty_flags.report_all) {
+            switch (self.event.key) {
+                .enter, .backspace, .tab => return "",
+                else => {},
+            }
+        }
+    }
 
     const all_mods = self.event.mods;
     const effective_mods = self.event.effectiveMods();
@@ -1063,6 +1075,70 @@ test "kitty: enter, backspace, tab" {
         const actual = try enc.kitty(&buf);
         try testing.expectEqualStrings("\t", actual);
     }
+
+    // No release events if "report_all" is not set
+    {
+        var enc: KeyEncoder = .{
+            .event = .{ .action = .release, .key = .enter, .mods = .{}, .utf8 = "" },
+            .kitty_flags = .{ .disambiguate = true, .report_events = true },
+        };
+        const actual = try enc.kitty(&buf);
+        try testing.expectEqualStrings("", actual);
+    }
+    {
+        var enc: KeyEncoder = .{
+            .event = .{ .action = .release, .key = .backspace, .mods = .{}, .utf8 = "" },
+            .kitty_flags = .{ .disambiguate = true, .report_events = true },
+        };
+        const actual = try enc.kitty(&buf);
+        try testing.expectEqualStrings("", actual);
+    }
+    {
+        var enc: KeyEncoder = .{
+            .event = .{ .action = .release, .key = .tab, .mods = .{}, .utf8 = "" },
+            .kitty_flags = .{ .disambiguate = true, .report_events = true },
+        };
+        const actual = try enc.kitty(&buf);
+        try testing.expectEqualStrings("", actual);
+    }
+
+    // Release events if "report_all" is set
+    {
+        var enc: KeyEncoder = .{
+            .event = .{ .action = .release, .key = .enter, .mods = .{}, .utf8 = "" },
+            .kitty_flags = .{
+                .disambiguate = true,
+                .report_events = true,
+                .report_all = true,
+            },
+        };
+        const actual = try enc.kitty(&buf);
+        try testing.expectEqualStrings("\x1b[13;1:3u", actual);
+    }
+    {
+        var enc: KeyEncoder = .{
+            .event = .{ .action = .release, .key = .backspace, .mods = .{}, .utf8 = "" },
+            .kitty_flags = .{
+                .disambiguate = true,
+                .report_events = true,
+                .report_all = true,
+            },
+        };
+        const actual = try enc.kitty(&buf);
+        try testing.expectEqualStrings("\x1b[127;1:3u", actual);
+    }
+    {
+        var enc: KeyEncoder = .{
+            .event = .{ .action = .release, .key = .tab, .mods = .{}, .utf8 = "" },
+            .kitty_flags = .{
+                .disambiguate = true,
+                .report_events = true,
+                .report_all = true,
+            },
+        };
+        const actual = try enc.kitty(&buf);
+        try testing.expectEqualStrings("\x1b[9;1:3u", actual);
+    }
 }
 
 test "kitty: enter with all flags" {