A bad command inside an Edit block written whole is named, not the } after it

A command in a { block that failed to parse ended the block there, so its } was taken as a control message of its own and the write failed unknown control message "}". A { block not yet closed now runs to its } whatever fails to parse in it, and the error names the command (Edit: unknown command z), as the line-by-line path does.

Co-Authored-By: Claude Opus 5.5 <[email protected]>

1 files changed, 16 insertions, 1 deletions

diff --git a/src/sam_edit.zig b/src/sam_edit.zig
index 8f7c961f..ea1105bd 100644
--- a/src/sam_edit.zig
+++ b/src/sam_edit.zig
@@ -111,11 +111,26 @@ pub fn waitsFor(arena: std.mem.Allocator, command: []const u8) Wait {
     var ps: Parser = .{ .arena = arena, .src = src, .why = &why };
     while (true) {
         const c = ps.parse(0) catch
-            return if (ps.open_text) .text else if (std.mem.eql(u8, why.text(), "unmatched `{'")) .group else .none;
+            return if (ps.open_text) .text else if (std.mem.eql(u8, why.text(), "unmatched `{'") or groupOpen(command)) .group else .none;
         if (c == null) return if (ps.open_text) .text else .none;
     }
 }
 
+/// A `{` block not closed yet by its `}` line: a command in it that fails
+/// to parse still leaves the block to end at its `}`, so the error is that
+/// command's, not the `}`'s taken as a line of its own.
+fn groupOpen(command: []const u8) bool {
+    var depth: usize = 0;
+    var lines = std.mem.splitScalar(u8, command, '\n');
+    while (lines.next()) |raw| {
+        const line = std.mem.trim(u8, raw, " \t\r");
+        if (std.mem.eql(u8, line, "}")) {
+            depth -|= 1;
+        } else if (std.mem.endsWith(u8, line, "{")) depth += 1;
+    }
+    return depth > 0;
+}
+
 pub fn needsMore(arena: std.mem.Allocator, command: []const u8) bool {
     var why: Why = .{};
     const src = std.fmt.allocPrint(arena, "{s}\n", .{command}) catch return false;