whatap 2.0.0 → 2.0.1-canary.0

package/lib/core/agent.js

@@ -200,19 +200,14 @@ NodeAgent.prototype.writeToFile = function(home, fileName, value) {
     }
 };
 
-// Modified isGoAgentRunning to check PID +-1 range
+// Verify the recorded PID corresponds to a live whatap_nodejs process.
+// Go agent writes its real (post-daemon-fork) PID into the shared PID file,
+// so we trust that value directly instead of guessing adjacent PIDs.
 NodeAgent.prototype.isGoAgentRunning = function(pid) {
     try {
         if (!pid) return false;
 
-        // 원래 PID와 +1, -1된 PID 모두 확인
-        const pidsToCheck = [
-            parseInt(pid),
-            parseInt(pid) + 1,
-            parseInt(pid) - 1
-        ];
-
-        // Logger.print("WHATAP-007", `Checking PIDs: ${pidsToCheck.join(', ')} (original: ${pid})`, false);
+        const pidsToCheck = [parseInt(pid)];
 
         for (const pidToCheck of pidsToCheck) {
             try {
@@ -623,6 +618,24 @@ NodeAgent.prototype.updateConfig = function(home, optKey, optValue) {
     }
 };
 
+/**
+ * Detect whether this process is running inside a container (Docker / K8s).
+ * In container environments, the Go agent should run in foreground mode so that
+ * Node.js can directly manage the child process lifecycle (no orphan/zombie risk).
+ */
+NodeAgent.prototype.isContainerEnvironment = function() {
+    if (process.platform !== 'linux') return false;
+
+    try {
+        if (fs.existsSync('/.dockerenv')) return true;
+        if (fs.existsSync('/run/.containerenv')) return true;
+    } catch (e) {}
+
+    if (process.env.KUBERNETES_SERVICE_HOST) return true;
+
+    return false;
+};
+
 /**
  * Detect Node.js cluster mode and set environment variables
  * This allows Node.js cluster to work the same way as PM2 cluster mode
@@ -775,6 +788,14 @@ NodeAgent.prototype.startHealthCheck = function() {
         clearInterval(self._healthCheckInterval);
     }
 
+    // In container environments the Go agent runs in foreground, so Node.js
+    // already tracks its lifecycle via ChildProcess events. Polling is redundant
+    // and historically caused false-positive restart loops -> orphan zombies.
+    if (self.isContainerEnvironment()) {
+        Logger.print("WHATAP-223", "Container environment detected, skipping periodic health check (foreground mode handles lifecycle)", false);
+        return;
+    }
+
     // PM2 cluster 모드에서는 첫 번째 인스턴스만 health check 수행
     const isPM2Cluster = self.isPM2ClusterMode();
     if (isPM2Cluster && process.env.pm_id !== '0') {
@@ -1119,10 +1140,18 @@ NodeAgent.prototype.startGoAgent = function(opts = {}) {
 
         // 에이전트 프로세스 시작
         // Windows: -t 2 foreground (detached 모드)
-        // Unix/Linux: -t 2 -d 1 (daemon 모드)
+        // Linux container: -t 2 (foreground, Node.js가 직접 관리)
+        // Linux VM/베어메탈: -t 2 -d 1 (daemon 모드, 기존 호환)
+        const isContainer = this.isContainerEnvironment();
         const spawnArgs = platform === 'win32'
             ? ['-t', '2', 'foreground']
-            : ['-t', '2', '-d', '1'];
+            : isContainer
+                ? ['-t', '2']
+                : ['-t', '2', '-d', '1'];
+
+        if (isContainer) {
+            Logger.print("WHATAP-224", "Container environment: starting Go agent in foreground mode", false);
+        }
 
         const spawnOptions = {
             cwd: whatapHome,

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "whatap",
   "homepage": "http://www.whatap.io",
-  "version": "2.0.0",
-  "releaseDate": "20260427",
+  "version": "2.0.1-canary.0",
+  "releaseDate": "20260526",
   "description": "Monitoring and Profiling Service",
   "main": "index.js",
   "scripts": {},

package/.claude/settings.local.json

@@ -1,14 +0,0 @@
-{
-  "permissions": {
-    "allow": [
-      "Bash(grep -n \"ECONNREFUSED\" /Users/seunghunlee/agent_workspace/nodejs_agent/lib/observers/*.js /Users/seunghunlee/agent_workspace/nodejs_agent/lib/core/agent.js 2>/dev/null)",
-      "WebSearch",
-      "Bash(go env *)",
-      "Bash(ls -l \"$\\(go env GOPATH\\)/bin/govulncheck\")",
-      "Bash(brew list *)",
-      "Bash(brew search *)",
-      "Bash(brew info *)",
-      "Bash(echo \"--- exit $? ---\")"
-    ]
-  }
-}