wtt-connect 0.2.41 → 0.2.43

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wtt-connect",
-  "version": "0.2.41",
+  "version": "0.2.43",
   "private": false,
   "description": "WTT-native connector daemon for Codex, Claude Code, Cursor, Gemini, ACP, and other coding agent surfaces.",
   "type": "module",

package/src/config.js

@@ -96,6 +96,7 @@ export function loadConfig(argv = {}) {
     openDesignSkillsDir: process.env.WTT_CONNECT_OPENDESIGN_SKILLS_DIR || fileConfig.openDesignSkillsDir || resolveOpenDesignSkillsDir(),
     inboxDir: process.env.WTT_CONNECT_INBOX_DIR || fileConfig.inboxDir || path.join(stateDir, 'inbox'),
     uploadArtifacts: parseBool(process.env.WTT_CONNECT_UPLOAD_ARTIFACTS, fileConfig.uploadArtifacts ?? false),
+    previewKeepLast: parseIntEnv(process.env.WTT_CONNECT_PREVIEW_KEEP_LAST, fileConfig.previewKeepLast ?? 3),
     setupDisplayName: process.env.WTT_CONNECT_DISPLAY_NAME || fileConfig.setupDisplayName || 'wtt-connect',
     agentAliases: parseListEnv(process.env.WTT_CONNECT_AGENT_ALIASES || fileConfig.agentAliases || ''),
     stt: {

package/src/main.js

@@ -148,7 +148,7 @@ async function previewPort(config, argv) {
   }
   const cleanup = cleanupOldPreviews(config, {
     currentPort: port,
-    keepLast: Number.isFinite(Number(argv.keepLast)) ? Number(argv.keepLast) : 0,
+    keepLast: previewKeepLast(config, argv),
   });
   const preview = await createSandboxPreviewFromOutbox(config, port, {
     name: String(argv.previewName || argv.title || '').trim(),
@@ -185,6 +185,11 @@ async function cleanupPreviewsCommand(config, argv) {
   console.log(JSON.stringify({ ok: true, cleanup }, null, 2));
 }
 
+function previewKeepLast(config, argv = {}) {
+  if (Number.isFinite(Number(argv.keepLast))) return Math.max(0, Number(argv.keepLast));
+  return Math.max(0, Number(config.previewKeepLast ?? 3) || 0);
+}
+
 function cleanupOldPreviews(config, { currentPort = 0, keepLast = 0 } = {}) {
   const registry = readPreviewRegistry(config);
   const entries = registry
@@ -237,21 +242,90 @@ function writePreviewRegistry(config, registry) {
 
 function stopPreviewPort(port) {
   if (!Number.isInteger(port) || port < 1024 || port > 65535) return { killed: [], reason: 'invalid_port' };
-  const script = [
-    `pids=$(ss -ltnp 2>/dev/null | awk '/:${port} / {print $NF}' | sed -n 's/.*pid=\\([0-9][0-9]*\\).*/\\1/p' | sort -u)`,
-    'killed=""',
-    'for pid in $pids; do',
-    '  if [ -n "$pid" ] && kill -0 "$pid" 2>/dev/null; then kill "$pid" 2>/dev/null || true; killed="$killed $pid"; fi',
-    'done',
-    'sleep 0.2',
-    'for pid in $pids; do',
-    '  if [ -n "$pid" ] && kill -0 "$pid" 2>/dev/null; then kill -9 "$pid" 2>/dev/null || true; fi',
-    'done',
-    'printf "%s" "$killed"',
-  ].join('; ');
-  const result = spawnSync('bash', ['-lc', script], { encoding: 'utf8', timeout: 5000 });
-  const killed = String(result.stdout || '').trim().split(/\s+/).filter(Boolean);
-  return { killed, ok: result.status === 0 };
+  const pids = listenerPidsForPort(port);
+  const killed = [];
+  for (const pid of pids) {
+    try {
+      process.kill(pid, 'SIGTERM');
+      killed.push(String(pid));
+    } catch {}
+  }
+  const deadline = Date.now() + 500;
+  while (Date.now() < deadline && pids.some((pid) => isProcessAlive(pid))) {
+    Atomics.wait(new Int32Array(new SharedArrayBuffer(4)), 0, 0, 50);
+  }
+  for (const pid of pids) {
+    if (!isProcessAlive(pid)) continue;
+    try {
+      process.kill(pid, 'SIGKILL');
+      if (!killed.includes(String(pid))) killed.push(String(pid));
+    } catch {}
+  }
+  return { killed, ok: true };
+}
+
+function listenerPidsForPort(port) {
+  const inodes = listenerInodesForPort(port);
+  if (!inodes.size) return [];
+  const pids = [];
+  for (const name of safeReadDir('/proc')) {
+    if (!/^\d+$/.test(name)) continue;
+    const fdDir = `/proc/${name}/fd`;
+    for (const fd of safeReadDir(fdDir)) {
+      let target = '';
+      try {
+        target = fs.readlinkSync(path.join(fdDir, fd));
+      } catch {
+        continue;
+      }
+      const match = target.match(/^socket:\[(\d+)\]$/);
+      if (match && inodes.has(match[1])) {
+        pids.push(Number(name));
+        break;
+      }
+    }
+  }
+  return [...new Set(pids)];
+}
+
+function listenerInodesForPort(port) {
+  const targetHex = Number(port).toString(16).toUpperCase().padStart(4, '0');
+  const inodes = new Set();
+  for (const file of ['/proc/net/tcp', '/proc/net/tcp6']) {
+    let text = '';
+    try {
+      text = fs.readFileSync(file, 'utf8');
+    } catch {
+      continue;
+    }
+    for (const line of text.split('\n').slice(1)) {
+      const parts = line.trim().split(/\s+/);
+      if (parts.length < 10) continue;
+      const local = parts[1] || '';
+      const state = parts[3] || '';
+      const inode = parts[9] || '';
+      const localPort = local.split(':')[1] || '';
+      if (state === '0A' && localPort.toUpperCase() === targetHex && inode) inodes.add(inode);
+    }
+  }
+  return inodes;
+}
+
+function safeReadDir(dir) {
+  try {
+    return fs.readdirSync(dir);
+  } catch {
+    return [];
+  }
+}
+
+function isProcessAlive(pid) {
+  try {
+    process.kill(pid, 0);
+    return true;
+  } catch {
+    return false;
+  }
 }
 
 async function createSandboxPreviewFromOutbox(config, port, { name = '', token = '', timeoutMs = 15000 } = {}) {

package/src/runner.js

@@ -939,11 +939,11 @@ function renderCloudSandboxStorageInstruction(config, topicId = '') {
     '- If a generated user-facing file is stored in R2/persistent output storage and should appear in WTT chat, still publish it with `wtt-connect upload-file` or a WTT artifact marker.',
     '- If the user asks for a visual page, HTML artifact, chart, dashboard, animation, or browser preview, build it in the workspace, start a local web server, create a Cloudflare Sandbox preview URL, and return that preview URL to WTT.',
     '- The Cloud Sandbox preview URL feature is only for WTT Cloud Sandbox agents. Do not use WTT backend artifact/media preview flows for live sandbox web servers.',
-    '- Before starting a new preview server, run `wtt-connect cleanup-previews` to stop older preview servers for this agent and free sandbox resources.',
     '- Preview servers must keep running after your command finishes. Start them in the background with `nohup ... >/tmp/<name>.log 2>&1 &` or an equivalent long-lived process, and bind to `0.0.0.0`.',
     '- Before publishing a preview URL, verify the server is actually listening and serving content with `curl -fsS http://127.0.0.1:<port>/ >/dev/null`.',
     '- If local curl fails, fix or restart the web server first. Never publish a preview URL for a dead port.',
-    '- `wtt-connect preview-port` automatically stops older preview servers registered by this agent before publishing the new preview, so prefer one active preview per agent unless the user asks otherwise.',
+    '- `wtt-connect preview-port` automatically keeps the most recent preview servers for this agent and stops older registered previews beyond the retention limit. Default retention is 3 live previews; use `--keep-last <n>` only when the user asks.',
+    '- For historical results that should remain available after preview cleanup, also save static HTML, screenshots, reports, or build artifacts under the persistent R2 output directory and publish them as files/artifacts. Live preview URLs are for recent interactive previews, not permanent archival storage.',
     '- If you start a web server in the sandbox and the user should preview it, call the Cloudflare Sandbox outbound Worker directly with curl and include the returned `preview_url` in your reply as `[preview_url:Short Title](<preview_url>)`.',
     '- Preview ports must be 1024-65535 and cannot be 3000. For Vite/Next-style dev servers prefer 5173, 4173, or 8080.',
     `- Preview curl rule: curl -sS -X POST "\${WTT_SANDBOX_OUTBOX_URL:-http://wtt.preview}/preview-port" -H 'content-type: application/json' -d '{"agent_id":"'\${WTT_AGENT_ID:-cloud-agent}'","port":<port>}'`,