thinknagent 0.1.31 → 0.1.33

package/lib/metrics.js

@@ -236,22 +236,13 @@ class MetricsPoller {
       // pehle alert engine aur history ko raw metrics pass karo — 24/7 Edge evaluation
       this.onMetricsEmit?.(payload);
 
-      // MITM / Zero-Knowledge Defense: Encrypt metrics before sending over socket
       const store = require('./store');
-      const { encryptE2EE } = require('./e2ee');
-      const roomId = this.conn?.roomId || store.get('roomId');
+      const agentId = this.conn?.agentId || store.get('agentId');
 
-      if (roomId) {
-        const encrypted = encryptE2EE(payload, roomId);
-        this.conn.emit('agent:metrics', {
-          agentId: this.conn.agentId || store.get('agentId'),
-          ts: payload.ts,
-          encrypted,
-          e2ee: true
-        });
-      } else {
-        this.conn.emit('agent:metrics', payload);
-      }
+      this.conn.emit('agent:metrics', {
+        agentId,
+        ...payload
+      });
 
     } catch (err) {
       console.error('[metrics] Poll error:', err.message);

package/lib/shell.js

@@ -1,12 +1,34 @@
 'use strict';
 
-let pty;
+let pty = null;
 try {
   pty = require('node-pty');
 } catch {
   pty = null;
 }
 
+let detectedPtyLauncher = null;
+function getPtyLauncher() {
+  if (detectedPtyLauncher) return detectedPtyLauncher;
+  if (pty) {
+    detectedPtyLauncher = 'node-pty';
+    return 'node-pty';
+  }
+  const { execSync } = require('child_process');
+  try {
+    execSync('python3 -c "import pty"', { stdio: 'ignore' });
+    detectedPtyLauncher = 'python3';
+    return 'python3';
+  } catch {}
+  try {
+    execSync('python -c "import pty"', { stdio: 'ignore' });
+    detectedPtyLauncher = 'python';
+    return 'python';
+  } catch {}
+  detectedPtyLauncher = 'spawn';
+  return 'spawn';
+}
+
 // explicit allowlist — agent process ke secrets PTY mein nahi jayenge
 const PTY_ENV_ALLOWLIST = new Set([
   'PATH', 'HOME', 'SHELL', 'TERM', 'LANG', 'LC_ALL', 'LC_CTYPE',
@@ -61,13 +83,19 @@ class ShellBridge {
         this._sessions.delete(id);
       }
 
-      if (pty) {
-        const proc = pty.spawn(process.env.SHELL || '/bin/bash', [], {
+      const launcher = getPtyLauncher();
+      const shellPath = process.env.SHELL || '/bin/bash';
+      const homeDir = process.env.HOME || '/';
+      const safeEnv = buildSafeEnv();
+
+      if (launcher === 'node-pty' && pty) {
+        // Tier 1: node-pty native module
+        const proc = pty.spawn(shellPath, [], {
           name: 'xterm-256color',
           cols,
           rows,
-          cwd: process.env.HOME || '/',
-          env: buildSafeEnv(),
+          cwd: homeDir,
+          env: safeEnv,
         });
 
         proc.onData(data => s.emit('shell:data', { sessionId, data }));
@@ -79,18 +107,57 @@ class ShellBridge {
 
         this._sessions.set(sessionId, proc);
         s.emit('shell:opened', { sessionId });
-        console.log(`[shell] PTY session ${sessionId} opened (${cols}x${rows})`);
+        console.log(`[shell] PTY session ${sessionId} opened via node-pty (${cols}x${rows})`);
+      } else if (launcher === 'python3' || launcher === 'python') {
+        // Tier 2: Native OS pseudo-terminal via standard library pty.spawn
+        // Eliminates 'Inappropriate ioctl for device' & 'no job control' completely
+        const { spawn } = require('child_process');
+        const pyCode = 'import pty, os; os.environ["TERM"]="xterm-256color"; pty.spawn([os.environ.get("SHELL", "/bin/bash")])';
+        const proc = spawn(launcher, ['-c', pyCode], {
+          cwd: homeDir,
+          env: safeEnv,
+          stdio: ['pipe', 'pipe', 'pipe']
+        });
+
+        proc.stdout.on('data', data => s.emit('shell:data', { sessionId, data: data.toString() }));
+        proc.stderr.on('data', data => {
+          const clean = data.toString()
+            .replace(/^bash: cannot set terminal process group.*?\n/gm, '')
+            .replace(/^bash: no job control in this shell.*?\n/gm, '');
+          if (clean) s.emit('shell:data', { sessionId, data: clean });
+        });
+        proc.on('exit', exitCode => {
+          s.emit('shell:exit', { sessionId, exitCode: exitCode || 0 });
+          this._sessions.delete(sessionId);
+          console.log(`[shell] Python PTY session ${sessionId} exited (code ${exitCode})`);
+        });
+
+        this._sessions.set(sessionId, {
+          write: data => proc.stdin.write(data),
+          resize: () => {},
+          kill: () => {
+            try { proc.kill('SIGTERM'); } catch {}
+          }
+        });
+        s.emit('shell:opened', { sessionId });
+        console.log(`[shell] Genuine OS PTY session ${sessionId} opened via ${launcher}`);
       } else {
-        // Fallback to interactive standard child_process spawn
+        // Tier 3: Pure ChildProcess fallback with suppressed ioctl noise
         const { spawn } = require('child_process');
-        const proc = spawn(process.env.SHELL || '/bin/bash', ['-i'], {
-          cwd: process.env.HOME || '/',
-          env: buildSafeEnv(),
+        const proc = spawn(shellPath, ['--login'], {
+          cwd: homeDir,
+          env: safeEnv,
           stdio: ['pipe', 'pipe', 'pipe']
         });
 
         proc.stdout.on('data', data => s.emit('shell:data', { sessionId, data: data.toString() }));
-        proc.stderr.on('data', data => s.emit('shell:data', { sessionId, data: data.toString() }));
+        proc.stderr.on('data', data => {
+          // Filter out terminal ioctl startup noise on non-tty pipes
+          const clean = data.toString()
+            .replace(/^bash: cannot set terminal process group.*?\n/gm, '')
+            .replace(/^bash: no job control in this shell.*?\n/gm, '');
+          if (clean) s.emit('shell:data', { sessionId, data: clean });
+        });
         proc.on('exit', exitCode => {
           s.emit('shell:exit', { sessionId, exitCode: exitCode || 0 });
           this._sessions.delete(sessionId);
@@ -99,10 +166,12 @@ class ShellBridge {
         this._sessions.set(sessionId, {
           write: data => proc.stdin.write(data),
           resize: () => {},
-          kill: () => proc.kill()
+          kill: () => {
+            try { proc.kill('SIGTERM'); } catch {}
+          }
         });
         s.emit('shell:opened', { sessionId });
-        console.log(`[shell] Interactive Spawn session ${sessionId} opened`);
+        console.log(`[shell] Fallback login shell session ${sessionId} opened`);
       }
     });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "thinknagent",
-  "version": "0.1.31",
+  "version": "0.1.33",
   "description": "ThinkNCollab server agent & MCP server — metrics, logs, alerts, shell bridge, AI planning",
   "main": "lib/agent.js",
   "author": "ThinkNCollab Team <raman@thinkncollab.com>",