Symbols, Diagnostics and every language server list share their directory's one +Search

Only Find and Grep reused the directory's +Search; a language server's list made another pane of the same name each time. Every origin named +Search now shares it, but a plain word's search, which keeps a list a pattern for n/N to walk between.

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

2 files changed, 20 insertions, 4 deletions

diff --git a/src/Output.zig b/src/Output.zig
index 1461757f..f2ab1bda 100644
--- a/src/Output.zig
+++ b/src/Output.zig
@@ -428,11 +428,12 @@ pub fn hasFileSearch(p: *Pardes, dir: []const u8) bool {
     return false;
 }
 
+/// Whether `from` fills its directory's one +Search, whatever filled it
+/// last (Find, Grep, Symbols, Diagnostics, any language server's list),
+/// each run replacing the last. A plain word's search (in a file or a PDF)
+/// keeps a +Search a pattern: n/N walk from one such list to the next.
 fn fileSearch(from: Origin) bool {
-    return switch (from) {
-        .cmd => |c| c == .Find or c == .Grep,
-        else => false,
-    };
+    return from != .search and std.mem.eql(u8, traits(from).name, config.search_buffer);
 }
 
 pub fn fillResults(p: *Pardes, id: usize, dir: []const u8, from: Origin, arg: []const u8, content: []u8, initial_anchor: ?usize) !void {
diff --git a/src/ninep/ctl.zig b/src/ninep/ctl.zig
index b79b1ab8..0d3f084d 100644
--- a/src/ninep/ctl.zig
+++ b/src/ninep/ctl.zig
@@ -2161,6 +2161,21 @@ test "a PDF's body reads its page's text layer, and takes no write" {
     try testing.expectEqual(E.INVAL, wr(p, body, "more\n").errno());
 }
 
+test "Grep, Symbols and Diagnostics share their directory's one +Search" {
+    const p = try withFile(testing.allocator, "x\n");
+    defer p.deinit();
+    const id = p.active;
+    const Output = pardes.panes.Output;
+    try Output.fillResults(p, id, "/tmp", .{ .cmd = .Grep }, "x", try p.gpa.dupe(u8, "a.txt:1:1 x\n"), null);
+    try Output.fillResults(p, id, "/tmp", .{ .query = .document_symbols }, "", try p.gpa.dupe(u8, "a.txt:2:1 fn f\n"), null);
+    try Output.fillResults(p, id, "/tmp", .{ .query = .diagnostics }, "", try p.gpa.dupe(u8, "a.txt:3:1 error\n"), null);
+    var searches: usize = 0;
+    for (p.panes) |slot| if (slot) |q| if (q.file) |f| if (f.output != null) {
+        searches += @intFromBool(std.mem.eql(u8, f.path, "/tmp/+Search"));
+    };
+    try testing.expectEqual(@as(usize, 1), searches);
+}
+
 test "every EINVAL a write gets says why, in its err record too; DEL is a control character in a line" {
     const p = try withFile(testing.allocator, "x\n");
     defer p.deinit();