From 3a69a9ab3172412d1e11fd70a432b82b4aa97327 Mon Sep 17 00:00:00 2001
From: Alfonso Noriega <alfonso.noriega@shopify.com>
Date: Mon, 13 Apr 2026 15:58:10 +0200
Subject: [PATCH] fix: add error handling to empty catch blocks

The empty catch block in app-logs-polling.ts silently swallowed all
errors during app logs setup, making failures invisible to developers.

Added outputDebug logging so errors are captured in debug output
instead of being silently discarded.
---
 .../app/src/cli/services/dev/processes/app-logs-polling.ts  | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/packages/app/src/cli/services/dev/processes/app-logs-polling.ts b/packages/app/src/cli/services/dev/processes/app-logs-polling.ts
index f822f96ed7f..d1f3f87365b 100644
--- a/packages/app/src/cli/services/dev/processes/app-logs-polling.ts
+++ b/packages/app/src/cli/services/dev/processes/app-logs-polling.ts
@@ -120,6 +120,8 @@ export const subscribeAndStartPolling: DevProcessFunction<SubscribeAndStartPolli
     }
 
     appWatcher.onStart(startPollingIfNeeded).onEvent(startPollingIfNeeded)
-    // eslint-disable-next-line no-catch-all/no-catch-all,no-empty
-  } catch (error) {}
+    // eslint-disable-next-line no-catch-all/no-catch-all
+  } catch (error) {
+    outputDebug(`Failed to set up app logs polling: ${error}`, _stderr)
+  }
 }