whatap 2.0.5 → 2.0.6-canary.0

package/.claude/settings.local.json

@@ -38,7 +38,21 @@
       "Bash(git add *)",
       "Bash(git commit *)",
       "Bash(git fetch *)",
-      "Bash(git push *)"
+      "Bash(git push *)",
+      "Bash(node --check /Users/seunghunlee/agent_workspace/nodejs_agent/lib/core/agent.js)",
+      "Bash(git -C /Users/seunghunlee/agent_workspace/nodejs_agent branch -a)",
+      "Bash(git -C /Users/seunghunlee/agent_workspace/nodejs_agent log --all --oneline -- test)",
+      "Bash(node *)",
+      "Bash(git *)",
+      "Read(//Users/seunghunlee/workspace/go_sample/node_modules/whatap/lib/core/**)",
+      "Read(//Users/seunghunlee/workspace/go_sample/**)",
+      "Bash(rm -f agent-cfd56737.pid agent-cfd56737.lock whatap_nodejs.pid)",
+      "Bash(pkill -f whatap_nodejs)",
+      "Bash(xargs kill -9)",
+      "Bash(kill 34859)",
+      "Bash(cp node_modules/whatap/lib/core/agent.js node_modules/whatap/lib/core/agent.js.2.0.5.bak)",
+      "Bash(cp /Users/seunghunlee/agent_workspace/nodejs_agent/lib/core/agent.js node_modules/whatap/lib/core/agent.js)",
+      "Bash(ls -la agent-*)"
     ]
   }
 }

package/lib/core/agent.js

@@ -936,6 +936,52 @@ NodeAgent.prototype.restartGoAgent = function(opts = {}) {
     return true;
 };
 
+// 기동용 lock 파일이 stale인지 판정
+// lock은 spawn 중 몇 초만 존재해야 정상이므로, 소유 프로세스가 죽었거나
+// 생성 후 60초가 지났으면 stale로 간주한다 (pid 재사용 대비 mtime 안전망)
+NodeAgent.prototype.isStaleLockFile = function(lockFile) {
+    let lockPid = null;
+    try {
+        const stat = fs.statSync(lockFile);
+        if (Date.now() - stat.mtime.getTime() > 60000) return true;
+        lockPid = parseInt(fs.readFileSync(lockFile, 'utf8').trim(), 10);
+    } catch (e) {
+        return true;
+    }
+    // 내용이 비어있거나 파싱 불가: 다른 프로세스가 wx로 생성 직후 pid를 쓰기 전 순간일 수 있음
+    // (PM2 cluster 동시 부팅에서 발생 가능) → mtime이 신선한 동안은 기동 중으로 간주,
+    // 진짜 깨진 파일이면 60초 후 위의 mtime 규칙으로 stale 처리됨
+    if (!lockPid) return false;
+    try {
+        process.kill(lockPid, 0);
+        return false;
+    } catch (e) {
+        // EPERM: 프로세스는 살아있으나 권한 없음 -> stale 아님
+        return e.code !== 'EPERM';
+    }
+};
+
+// 기동용 lock 획득 (O_EXCL로 원자적 생성, stale lock은 제거 후 1회 재시도)
+NodeAgent.prototype.acquireStartLock = function(lockFile) {
+    for (let attempt = 0; attempt < 2; attempt++) {
+        try {
+            fs.writeFileSync(lockFile, process.pid.toString(), { flag: 'wx' });
+            return true;
+        } catch (e) {
+            if (e.code !== 'EEXIST') {
+                Logger.printError("WHATAP-033", "Error acquiring lock", e, false);
+                return false;
+            }
+            if (!this.isStaleLockFile(lockFile)) {
+                return false;
+            }
+            Logger.print("WHATAP-225", `Stale lock file detected (${lockFile}), removing and retrying...`, false);
+            try { fs.unlinkSync(lockFile); } catch (unlinkErr) {}
+        }
+    }
+    return false;
+};
+
 // startGoAgent with improved PID handling
 NodeAgent.prototype.startGoAgent = function(opts = {}) {
     const self = this;
@@ -976,48 +1022,23 @@ NodeAgent.prototype.startGoAgent = function(opts = {}) {
         Logger.printError("WHATAP-029", "Error checking shared lock file", e, false);
     }
 
-    // 3. PM2 cluster 모드에서는 첫 번째 인스턴스만 에이전트 시작
-    if (isPM2Cluster) {
-        // 잠금 파일로 동시 실행 방지
-        let lockAcquired = false;
-        try {
-            if (!fs.existsSync(sharedLockFile)) {
-                fs.writeFileSync(sharedLockFile, process.pid.toString());
-                lockAcquired = true;
-                Logger.print("WHATAP-030",
-                    `PM2 cluster mode: Instance ${process.env.pm_id} will start shared agent`,
-                    false);
-            } else {
-                // 다른 인스턴스가 이미 에이전트를 시작하는 중
-                Logger.print("WHATAP-031",
-                    `PM2 cluster mode: Instance ${process.env.pm_id} detected another instance starting the agent, will use shared agent`,
-                    false);
-                return;
-            }
-        } catch (e) {
-            Logger.printError("WHATAP-032", "Error acquiring lock in PM2 cluster mode", e, false);
-            return;
-        }
-
-        if (!lockAcquired) {
-            return;
-        }
-    } else {
-        // 일반 모드에서는 기존 로직 사용
-        let lockAcquired = false;
-        try {
-            if (!fs.existsSync(sharedLockFile)) {
-                fs.writeFileSync(sharedLockFile, process.pid.toString());
-                lockAcquired = true;
-            }
-        } catch (e) {
-            Logger.printError("WHATAP-033", "Error acquiring lock", e, false);
-        }
-
-        if (!lockAcquired) {
+    // 3. 잠금 파일로 동시 실행 방지 (stale lock은 자동 제거 후 재시도)
+    if (!this.acquireStartLock(sharedLockFile)) {
+        if (isPM2Cluster) {
+            // 다른 인스턴스가 이미 에이전트를 시작하는 중
+            Logger.print("WHATAP-031",
+                `PM2 cluster mode: Instance ${process.env.pm_id} detected another instance starting the agent, will use shared agent`,
+                false);
+        } else {
             Logger.print("WHATAP-034", "Another process is starting the agent, waiting...", false);
-            return;
         }
+        return;
+    }
+
+    if (isPM2Cluster) {
+        Logger.print("WHATAP-030",
+            `PM2 cluster mode: Instance ${process.env.pm_id} will start shared agent`,
+            false);
     }
 
     // 기존 whatap_nodejs.pid 파일 처리 (호환성 유지)
@@ -1206,8 +1227,8 @@ NodeAgent.prototype.startGoAgent = function(opts = {}) {
                 Logger.print("WHATAP-040", `PID ${actualPid} written to PID files`, false);
 
                 // Lock 파일 해제 (spawn 완료 후 안전하게 삭제)
+                // 이 lock은 proper-lockfile이 아닌 writeFileSync로 생성한 파일이므로 unlink만 수행
                 try {
-                    properLock.unlockSync(sharedLockFile);
                     fs.unlinkSync(sharedLockFile);
                     Logger.print("WHATAP-041", "Lock file released after spawn", false);
                 } catch (e) {}
@@ -1278,10 +1299,7 @@ NodeAgent.prototype.startGoAgent = function(opts = {}) {
     } catch (e) {
         Logger.printError("WHATAP-048", "Error starting agent", e, false);
         // 에러 발생 시 파일 정리
-        try {
-            properLock.unlockSync(sharedLockFile);
-            fs.unlinkSync(sharedLockFile);
-        } catch (unlockErr) {}
+        try { fs.unlinkSync(sharedLockFile); } catch (unlockErr) {}
         try { fs.unlinkSync(sharedPidFile); } catch (pidErr) {}
     }
 };

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "whatap",
   "homepage": "http://www.whatap.io",
-  "version": "2.0.5",
-  "releaseDate": "20260709",
+  "version": "2.0.6-canary.0",
+  "releaseDate": "20260713",
   "description": "Monitoring and Profiling Service",
   "main": "index.js",
   "scripts": {},