From eb32422e0564225e7c417fab23fd9c8a925e7555 Mon Sep 17 00:00:00 2001 From: Jeroen Schweitzer Date: Mon, 11 May 2026 17:03:40 +0200 Subject: [PATCH] drop dead pql.decisions.coverage IPC command MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The 'pql decisions coverage' subcommand was removed in pql 1.4.x — the replacement is the 'coverage_gaps' field on 'pql plan status'. No production callers used this IPC surface; only the unit test referenced it. Removes the IPC registration, the PqlClient helper, and the matching test case. Net negative LOC. Co-Authored-By: Claude Opus 4.7 (1M context) --- lib/src/daemon/pql_commands.dart | 9 --------- lib/src/pql/client.dart | 4 ---- test/daemon/pql_commands_test.dart | 7 ------- 3 files changed, 20 deletions(-) diff --git a/lib/src/daemon/pql_commands.dart b/lib/src/daemon/pql_commands.dart index 3f51fa4d..cd3fb02d 100644 --- a/lib/src/daemon/pql_commands.dart +++ b/lib/src/daemon/pql_commands.dart @@ -169,15 +169,6 @@ void registerPqlCommands(DaemonDispatcher d, PqlClient pql) { } }); - d.register('pql.decisions.coverage', (req) async { - try { - final gaps = await pql.decisionCoverage(); - return IpcResponse.ok(id: req.id, data: {'gaps': gaps}); - } on PqlException catch (e) { - return _pqlError(req.id, e); - } - }); - d.register('pql.tickets.list', (req) async { try { final results = await pql.ticketList( diff --git a/lib/src/pql/client.dart b/lib/src/pql/client.dart index b1d8bd27..97ae7e03 100644 --- a/lib/src/pql/client.dart +++ b/lib/src/pql/client.dart @@ -106,10 +106,6 @@ class PqlClient { return _runObject(['decisions', 'read', id]); } - Future>> decisionCoverage() async { - return _runList(['decisions', 'coverage']); - } - Future>> ticketList({ String? status, String? team, diff --git a/test/daemon/pql_commands_test.dart b/test/daemon/pql_commands_test.dart index 59a33ea9..d06ffba6 100644 --- a/test/daemon/pql_commands_test.dart +++ b/test/daemon/pql_commands_test.dart @@ -95,13 +95,6 @@ void main() { expect(r.error!.kind, 'user_error'); }); - test('pql.decisions.coverage returns gaps', () async { - await call('pql.decisions.sync'); - final r = await call('pql.decisions.coverage'); - expect(r.ok, isTrue); - expect(r.data.containsKey('gaps'), isTrue); - }); - test('pql.tickets.board returns columns', () async { final r = await call('pql.tickets.board'); expect(r.ok, isTrue);