fix(infra): log errors from restart sentinel cleanup instead of silent catch

wendy-chsy · wendy-chsy · commit c89de4e07e9a · 2026-07-07T23:40:13.000+08:00
The empty catch blocks in clearRestartSentinel() and
removeLegacyRestartSentinel() silently swallowed DB write and
file-deletion failures, leaving stale sentinel artifacts that could
trigger spurious recovery flows on next restart.
diff --git a/src/infra/restart-sentinel.ts b/src/infra/restart-sentinel.ts
@@ -224,7 +224,9 @@ export async function clearRestartSentinel(env: NodeJS.ProcessEnv = process.env)
       },
       { env },
     );
-  } catch {}
+  } catch (err) {
+    console.error("[openclaw] Failed to clear restart sentinel:", err);
+  }
   await removeLegacyRestartSentinel(env);
 }
 
@@ -235,7 +237,9 @@ function resolveLegacyRestartSentinelPath(env: NodeJS.ProcessEnv): string {
 async function removeLegacyRestartSentinel(env: NodeJS.ProcessEnv): Promise<void> {
   try {
     await rm(resolveLegacyRestartSentinelPath(env), { force: true });
-  } catch {}
+  } catch (err) {
+    console.error("[openclaw] Failed to remove legacy restart sentinel:", err);
+  }
 }
 
 async function importLegacyRestartSentinel(
