@wrongstack/tools 0.272.1 → 0.273.0

package/dist/index.js

@@ -626,7 +626,7 @@ var replaceTool = {
         if (err.code === "ENOENT") return null;
         throw err;
       });
-      if (!lstat2 || !lstat2.isFile()) continue;
+      if (!lstat2?.isFile()) continue;
       if (lstat2.isSymbolicLink()) continue;
       let realPath;
       try {
@@ -637,7 +637,7 @@ var replaceTool = {
       const rel = path.relative(realRoot, realPath);
       if (rel.startsWith("..") || path.isAbsolute(rel)) continue;
       const stat11 = await fs7.stat(realPath).catch(() => null);
-      if (!stat11 || !stat11.isFile()) continue;
+      if (!stat11?.isFile()) continue;
       let content;
       try {
         const buf = await fs7.readFile(realPath);
@@ -1798,6 +1798,21 @@ function _resetProcessRegistry() {
   _registry = void 0;
 }
 var REGISTRY_FILE = ".wrongstack/process-registry.json";
+function toErrorMessage2(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 = toErrorMessage2(error);
+  }
+  console.log(JSON.stringify(payload));
+}
 var HEARTBEAT_INTERVAL_MS = 5e3;
 var STALE_THRESHOLD_MS = 3e4;
 var LOCKFILE = ".wrongstack/.process-registry.lock";
@@ -1807,6 +1822,9 @@ function generateInstanceId() {
   const random = Math.random().toString(36).slice(2, 8);
   return `${hostname4}:${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);
@@ -1821,7 +1839,7 @@ async function acquireLock(lockfilePath, timeoutMs = 5e3) {
         }
       };
     } catch (err) {
-      if (err.code === "EEXIST") {
+      if (isNodeError(err) && err.code === "EEXIST") {
         try {
           const content = await fs7.readFile(lockfilePath, "utf-8");
           const parts = content.split(":");
@@ -1858,7 +1876,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(),
@@ -1894,7 +1912,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() {
@@ -1902,7 +1920,7 @@ var PersistentProcessRegistry = class {
     try {
       await fs7.mkdir(dir, { recursive: true });
     } catch (err) {
-      if (err.code !== "EEXIST") throw err;
+      if (!isNodeError(err) || err.code !== "EEXIST") throw err;
     }
   }
   /**
@@ -1982,6 +2000,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,
@@ -1989,8 +2008,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 {
@@ -2038,7 +2056,7 @@ var PersistentProcessRegistry = class {
       data.lastCleanup = now2;
       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();
     }
@@ -2059,7 +2077,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);
@@ -2073,7 +2095,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();
     }
@@ -2302,7 +2324,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;
       }
     }
@@ -8982,11 +9004,15 @@ async function tryNativeParse(file, content) {
     const crateDir = path.join(toolsDir, "syn-parser");
     const tmpFile = path.join(crateDir, "src", "input.rs");
     await fs7.writeFile(tmpFile, content, "utf8");
-    const proc = spawn("cargo", ["run", "--manifest-path", path.join(toolsDir, "Cargo.toml")], {
-      cwd: process.cwd(),
-      stdio: ["pipe", "pipe", "pipe"],
-      windowsHide: true
-    });
+    const proc = spawn(
+      "cargo",
+      ["run", "--manifest-path", path.join(toolsDir, "Cargo.toml")],
+      {
+        cwd: process.cwd(),
+        stdio: ["pipe", "pipe", "pipe"],
+        windowsHide: true
+      }
+    );
     let stdout = "";
     proc.stdout?.on("data", (chunk) => {
       stdout += chunk.toString();
@@ -10482,7 +10508,11 @@ var taskTool = {
             const newIds = new Set(input.tasks.map((t) => t.id));
             if (newIds.size !== input.tasks.length) {
               const seen = /* @__PURE__ */ new Set();
-              const dupes = [...new Set(input.tasks.map((t) => t.id).filter((id) => seen.has(id) ? true : (seen.add(id), false)))];
+              const dupes = [...new Set(input.tasks.map((t) => t.id).filter((id) => {
+                if (seen.has(id)) return true;
+                seen.add(id);
+                return false;
+              }))];
               early = {
                 ok: false,
                 message: `action=replace has duplicate task IDs: ${dupes.join(", ")}. Each task id must be unique.`,
@@ -10517,7 +10547,7 @@ var taskTool = {
           }
           case "add": {
             const t = input.task;
-            if (!t || !t.title) {
+            if (!t?.title) {
               early = { ok: false, message: "action=add requires `task` with at least `title`.", count: 0, completed: 0, inProgress: 0 };
               return f;
             }