indieclaw-agent 1.1.1 → 1.1.3

package/index.js

@@ -210,6 +210,7 @@ function handleFsDelete(ws, { id, path: filePath }) {
 // --- System Stats ---
 function handleSystemStats(ws, { id }) {
   const platform = os.platform();
+  const mem = platform === 'darwin' ? getDarwinMemory() : getLinuxMemory();
   const stats = {
     hostname: os.hostname(),
     platform,
@@ -220,17 +221,61 @@ function handleSystemStats(ws, { id }) {
       cores: os.cpus().length,
       usage: getCpuUsage(platform),
     },
-    memory: {
-      total: os.totalmem(),
-      free: os.freemem(),
-      used: os.totalmem() - os.freemem(),
-      usagePercent: Math.round(((os.totalmem() - os.freemem()) / os.totalmem()) * 100),
-    },
+    memory: mem,
     disk: getDiskUsage(platform),
   };
   reply(ws, id, stats);
 }
 
+function getDarwinMemory() {
+  const total = os.totalmem();
+  try {
+    const out = execSync('vm_stat', { timeout: 3000 }).toString();
+    const pageMatch = out.match(/page size of (\d+) bytes/);
+    const pageSize = pageMatch ? parseInt(pageMatch[1], 10) : 16384;
+
+    const val = (key) => {
+      const m = out.match(new RegExp(key + ':\\s+(\\d+)'));
+      return m ? parseInt(m[1], 10) : 0;
+    };
+
+    const wired = val('Pages wired down');
+    const active = val('Pages active');
+    const compressed = val('Pages occupied by compressor');
+    const purgeable = val('Pages purgeable');
+
+    // Used = wired + active (minus purgeable) + compressed — matches Activity Monitor
+    const used = (wired + active - purgeable + compressed) * pageSize;
+    const free = total - used;
+    return {
+      total,
+      free: Math.max(free, 0),
+      used: Math.min(used, total),
+      usagePercent: Math.round((Math.min(used, total) / total) * 100),
+    };
+  } catch {
+    // Fallback to Node.js (inaccurate on macOS but better than nothing)
+    const free = os.freemem();
+    return {
+      total,
+      free,
+      used: total - free,
+      usagePercent: Math.round(((total - free) / total) * 100),
+    };
+  }
+}
+
+function getLinuxMemory() {
+  const total = os.totalmem();
+  const free = os.freemem();
+  return {
+    total,
+    free,
+    used: total - free,
+    usagePercent: Math.round(((total - free) / total) * 100),
+  };
+}
+
 function getCpuUsage(platform) {
   try {
     if (platform === 'linux') {
@@ -251,23 +296,32 @@ function getCpuUsage(platform) {
 
 function getDiskUsage(platform) {
   try {
-    const cmd = platform === 'darwin' ? 'df -k /' : 'df -k / --output=source,size,used,avail,pcent,target';
-    const out = execSync(cmd, { timeout: 5000 }).toString();
-    const lines = out.trim().split('\n').slice(1);
-    return lines.map((line) => {
-      const parts = line.trim().split(/\s+/);
-      if (platform === 'darwin') {
-        // macOS df: Filesystem 512-blocks Used Available Capacity ...
-        // df -k gives 1K-blocks
+    if (platform === 'darwin') {
+      // macOS APFS: "Used" column only shows one volume's data
+      // Calculate used = total - available for accurate results
+      const out = execSync('df -k /', { timeout: 5000 }).toString();
+      const lines = out.trim().split('\n').slice(1);
+      return lines.map((line) => {
+        const parts = line.trim().split(/\s+/);
+        // Columns: Filesystem 1K-blocks Used Available Capacity iused ifree %iused Mounted
+        const total = parseInt(parts[1], 10) * 1024;
+        const available = parseInt(parts[3], 10) * 1024;
+        const used = total - available;
         return {
           filesystem: parts[0],
-          mount: parts[8] || parts[5] || '/',
-          total: parseInt(parts[1], 10) * 1024,
-          used: parseInt(parts[2], 10) * 1024,
-          available: parseInt(parts[3], 10) * 1024,
-          usagePercent: parseInt(parts[4], 10),
+          mount: parts[8] || '/',
+          total,
+          used,
+          available,
+          usagePercent: Math.round((used / total) * 100),
         };
-      }
+      });
+    }
+    // Linux
+    const out = execSync('df -k / --output=source,size,used,avail,pcent,target', { timeout: 5000 }).toString();
+    const lines = out.trim().split('\n').slice(1);
+    return lines.map((line) => {
+      const parts = line.trim().split(/\s+/);
       return {
         filesystem: parts[0],
         total: parseInt(parts[1], 10) * 1024,
@@ -489,11 +543,15 @@ function handleTerminalStart(ws, { id }) {
     }
   }
 
-  // Fallback: child_process.spawn
+  // Fallback: use script(1) to allocate a real PTY
   const { spawn } = require('child_process');
-  const proc = spawn(shell, ['-i'], {
+  const scriptArgs = os.platform() === 'darwin'
+    ? ['-q', '/dev/null', shell]
+    : ['-q', '-c', shell, '/dev/null'];
+
+  const proc = spawn('script', scriptArgs, {
     cwd: os.homedir(),
-    env: { ...process.env, TERM: 'dumb' },
+    env: { ...process.env, TERM: 'xterm-256color' },
     stdio: ['pipe', 'pipe', 'pipe'],
   });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "indieclaw-agent",
-  "version": "1.1.1",
+  "version": "1.1.3",
   "description": "Manage your server from your phone. Agent for the IndieClaw mobile app.",
   "main": "index.js",
   "bin": {