From d61268bb232656ea6e6b7e8333d29310e95d574e Mon Sep 17 00:00:00 2001
From: Alexey <1556417+alex-solovyev@users.noreply.github.com>
Date: Thu, 5 Mar 2026 03:51:37 +0100
Subject: [PATCH] fix: remove blanket stderr suppression from db() call in
 todo-sync.sh

Remove 2>/dev/null from the active_batch_id db() call at line 1978,
keeping the || echo '' fallback for set -e safety. This allows database
and SQL syntax errors to remain visible on stderr for debugging, aligning
with repo guidelines against blanket error suppression.

Closes #2864
---
 .agents/scripts/supervisor-archived/todo-sync.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.agents/scripts/supervisor-archived/todo-sync.sh b/.agents/scripts/supervisor-archived/todo-sync.sh
index f474c5322..88ea7d658 100755
--- a/.agents/scripts/supervisor-archived/todo-sync.sh
+++ b/.agents/scripts/supervisor-archived/todo-sync.sh
@@ -1975,7 +1975,7 @@ cmd_reconcile_queue_dispatchability() {
 			WHERE bt.task_id = '$(sql_escape "$tid")'
 			AND b.status IN ('active','paused')
 			LIMIT 1;
-		" 2>/dev/null || echo "")
+		" || echo "")
 		if [[ -n "$active_batch_id" ]]; then
 			in_active_batch=true
 		fi