refactor: let settle() handle getChiaVersion errors for observability

TheLastCicada · TheLastCicada · commit 9b3eb7e0a391 · 2026-05-13T11:05:29.000-07:00
Remove the internal try/catch so RPC failures surface in the settle()
debug log, consistent with every other wallet RPC function.
diff --git a/src/datalayer/wallet.js b/src/datalayer/wallet.js
@@ -333,22 +333,18 @@ const getChiaVersion = async () => {
   const url = `${rpcUrl}/get_version`;
   const { cert, key, timeout } = getBaseOptions();
 
-  try {
-    const response = await superagent
-      .post(url)
-      .key(key)
-      .cert(cert)
-      .timeout(timeout)
-      .send(JSON.stringify({}));
+  const response = await superagent
+    .post(url)
+    .key(key)
+    .cert(cert)
+    .timeout(timeout)
+    .send(JSON.stringify({}));
 
-    const data = response.body;
-    if (data.success) {
-      return data.version || null;
-    }
-    return null;
-  } catch {
-    return null;
+  const data = response.body;
+  if (data.success) {
+    return data.version || null;
   }
+  return null;
 };
 
 /**
