@wrongstack/tools 0.272.1 → 0.273.0

package/dist/bash.js

@@ -699,6 +699,21 @@ function getProcessRegistry() {
   return _registry;
 }
 var REGISTRY_FILE = ".wrongstack/process-registry.json";
+function toErrorMessage(err) {
+  return err instanceof Error ? err.message : String(err);
+}
+function emitStructuredLog(level, event, message, error) {
+  const payload = {
+    level,
+    event,
+    message,
+    timestamp: (/* @__PURE__ */ new Date()).toISOString()
+  };
+  if (error !== void 0) {
+    payload.error = toErrorMessage(error);
+  }
+  console.log(JSON.stringify(payload));
+}
 var HEARTBEAT_INTERVAL_MS = 5e3;
 var STALE_THRESHOLD_MS = 3e4;
 var LOCKFILE = ".wrongstack/.process-registry.lock";
@@ -708,6 +723,9 @@ function generateInstanceId() {
   const random = Math.random().toString(36).slice(2, 8);
   return `${hostname2}:${pid}:${random}`;
 }
+function isNodeError(err) {
+  return typeof err === "object" && err !== null && "code" in err;
+}
 async function acquireLock(lockfilePath, timeoutMs = 5e3) {
   const start = Date.now();
   const pidStr = String(process.pid);
@@ -722,7 +740,7 @@ async function acquireLock(lockfilePath, timeoutMs = 5e3) {
         }
       };
     } catch (err) {
-      if (err.code === "EEXIST") {
+      if (isNodeError(err) && err.code === "EEXIST") {
         try {
           const content = await fs.readFile(lockfilePath, "utf-8");
           const parts = content.split(":");
@@ -759,7 +777,7 @@ async function readRegistryFile(filePath) {
     }
     return parsed;
   } catch (err) {
-    if (err.code === "ENOENT") {
+    if (isNodeError(err) && err.code === "ENOENT") {
       return {
         version: 1,
         instances: /* @__PURE__ */ new Map(),
@@ -795,7 +813,7 @@ var PersistentProcessRegistry = class {
     this.lockPath = path.join(homeDir, LOCKFILE);
     this.baseRegistry = baseRegistry ?? getProcessRegistry();
     this.ensureDirectory().catch((err) => {
-      console.error("PersistentProcessRegistry: failed to create .wrongstack dir", err);
+      emitStructuredLog("warn", "process_registry.dir_create_failed", "PersistentProcessRegistry: failed to create .wrongstack directory", err);
     });
   }
   async ensureDirectory() {
@@ -803,7 +821,7 @@ var PersistentProcessRegistry = class {
     try {
       await fs.mkdir(dir, { recursive: true });
     } catch (err) {
-      if (err.code !== "EEXIST") throw err;
+      if (!isNodeError(err) || err.code !== "EEXIST") throw err;
     }
   }
   /**
@@ -883,6 +901,7 @@ var PersistentProcessRegistry = class {
     try {
       const data = await readRegistryFile(this.registryPath);
       data.instances.set(String(entry.pid), entry);
+      const child = null;
       this.baseRegistry.register({
         pid: entry.pid,
         name: entry.name,
@@ -890,8 +909,7 @@ var PersistentProcessRegistry = class {
         startedAt: entry.startedAt,
         sessionId: entry.sessionId,
         protected: entry.protected,
-        child: null
-        // Main process has no child handle
+        child
       });
       await writeRegistryFile(this.registryPath, data);
     } finally {
@@ -939,7 +957,7 @@ var PersistentProcessRegistry = class {
       data.lastCleanup = now;
       await writeRegistryFile(this.registryPath, data);
     } catch (err) {
-      console.error("PersistentProcessRegistry: sync failed", err);
+      emitStructuredLog("warn", "process_registry.sync_failed", "PersistentProcessRegistry: sync failed", err);
     } finally {
       await release();
     }
@@ -960,7 +978,11 @@ var PersistentProcessRegistry = class {
             if (process.platform !== "win32") {
               process.kill(entry.pid, 0);
             } else {
-              console.log(`PersistentProcessRegistry: checking stale pid ${entry.pid} (${age}ms old)`);
+              emitStructuredLog(
+                "debug",
+                "process_registry.stale_pid_check",
+                `PersistentProcessRegistry: checking stale pid ${entry.pid} (${age}ms old)`
+              );
             }
           } catch {
             stalePids.push(_pidStr);
@@ -974,7 +996,7 @@ var PersistentProcessRegistry = class {
         await writeRegistryFile(this.registryPath, data);
       }
     } catch (err) {
-      console.error("PersistentProcessRegistry: cleanup failed", err);
+      emitStructuredLog("warn", "process_registry.cleanup_failed", "PersistentProcessRegistry: cleanup failed", err);
     } finally {
       await release();
     }
@@ -1197,7 +1219,7 @@ async function isKillProtected(kill) {
     const entries = await getProtectedEntries();
     const killNameLower = kill.name.toLowerCase();
     for (const entry of entries) {
-      if (entry.name && entry.name.toLowerCase().includes(killNameLower)) {
+      if (entry.name?.toLowerCase().includes(killNameLower)) {
         return true;
       }
     }