From 48d74b1ac500f6362a3a91589e3ca0b50ea080d1 Mon Sep 17 00:00:00 2001
From: Jeroen Schweitzer <jpmschweitzer@gmail.com>
Date: Tue, 19 May 2026 14:22:11 +0200
Subject: [PATCH] T-129 follow-up: server-side _argv unwrap tests for coverage
 gate

Two extra cases at the server layer that the dispatcher-level tests
don't reach: _argv carrying tail --events should route into the
streaming check, and _argv with non-list args should surface a
userError via the server's write path (not the dispatcher's). Lifts
coverage back over 95.

Co-Authored-By: Claude <noreply@anthropic.com>
---
 test/ipc/server_streaming_test.dart | 28 ++++++++++++++++++++++++++++
 1 file changed, 28 insertions(+)

diff --git a/test/ipc/server_streaming_test.dart b/test/ipc/server_streaming_test.dart
index 319ddbe7..0011effe 100644
--- a/test/ipc/server_streaming_test.dart
+++ b/test/ipc/server_streaming_test.dart
@@ -204,6 +204,34 @@ void main() {
     await qB.cancel();
   });
 
+  test('server-side _argv unwrap routes through the streaming check', () async {
+    // Sends an _argv envelope carrying `tail --events --filter pane`.
+    // Without server-side unwrap (T-129), the streaming check would
+    // miss it and the server would dispatch _argv → notFound for tail.
+    final s = await _connect(server);
+    addTearDown(s.close);
+    final r = _lineReader(s);
+    final q = _Lines(r.lines);
+    await _send(s, IpcRequest(id: 'a', cmd: '_argv', args: {
+      'argv': ['tail', '--events', '--filter', 'pane'],
+    }));
+    final ack = IpcMessage.decode(await q.next()) as IpcResponse;
+    expect(ack.ok, isTrue);
+    expect(ack.data['streaming'], isTrue);
+    expect(ack.data['filter'], 'pane');
+    await q.cancel();
+  });
+
+  test('server-side _argv with non-list args returns userError', () async {
+    final s = await _connect(server);
+    addTearDown(s.close);
+    final r = _lineReader(s);
+    await _send(s, IpcRequest(id: 'b', cmd: '_argv', args: {'argv': 'not a list'}));
+    final resp = IpcMessage.decode(await r.lines.first.timeout(const Duration(seconds: 2))) as IpcResponse;
+    expect(resp.ok, isFalse);
+    expect(resp.error?.message, contains('argv'));
+  });
+
   test('subscriber going away removes itself from fanout (no crash on emit)', () async {
     final s = await _connect(server);
     final r = _lineReader(s);