Merge pull request #101 from useblacksmith/tailscale-cleanup

src: only log fatal errors in tailscale teardown

Author: adityamaru

dist/index.js

@@ -27,7 +27,7 @@
   "packageManager": "yarn@3.6.3",
   "dependencies": {
     "@actions/core": "^1.10.1",
-    "@buf/blacksmith_vm-agent.connectrpc_es": "^1.6.1-20250211212423-70f4f1344c53.2",
+    "@buf/blacksmith_vm-agent.connectrpc_es": "^1.6.1-20250304023716-e8d233d92eac.2",
     "@connectrpc/connect": "^1.6.1",
     "@connectrpc/connect-node": "^1.6.1",
     "@docker/actions-toolkit": "0.37.1",

dist/index.js.map

@@ -195,15 +195,25 @@ export async function joinTailnet(): Promise<void> {
 export async function leaveTailnet(): Promise<void> {
   try {
     // Check if we're part of a tailnet before trying to leave
-    const {stdout} = await execAsync('sudo tailscale status');
-    if (stdout.trim() !== '') {
-      await execAsync('sudo tailscale down');
-      core.debug('Successfully left tailnet');
-    } else {
-      core.debug('Not part of a tailnet, skipping leave');
+    try {
+      const {stdout} = await execAsync('sudo tailscale status');
+      if (stdout.trim() !== '') {
+        await execAsync('sudo tailscale down');
+        core.debug('Successfully left tailnet.');
+      } else {
+        core.debug('Not part of a tailnet, skipping leave.');
+      }
+    } catch (error: unknown) {
+      // Type guard for ExecException which has the code property
+      if (error && typeof error === 'object' && 'code' in error && error.code === 1) {
+        core.debug('Not part of a tailnet, skipping leave.');
+        return;
+      }
+      // Any other exit code indicates a real error
+      throw error;
     }
   } catch (error) {
-    core.warning(`Error leaving tailnet: ${error.message}`);
+    core.warning(`Error leaving tailnet: ${error instanceof Error ? error.message : String(error)}`);
   }
 }