amalgm 0.1.109 → 0.1.110

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "amalgm",
-  "version": "0.1.109",
+  "version": "0.1.110",
   "description": "Amalgm local computer runtime: login, MCP, chat, events, previews, and tunnels.",
   "license": "UNLICENSED",
   "private": false,

package/runtime/scripts/amalgm-mcp/browser/cli.js

@@ -108,11 +108,11 @@ function agentBrowserCommand() {
 }
 
 function wantsHeaded() {
+  // CLI mode is the background/headless lane. A headed standalone browser is
+  // opt-in so searches and sub-agents do not open Chrome or steal focus.
   const explicit = process.env.AMALGM_BROWSER_HEADED || process.env.AGENT_BROWSER_HEADED;
-  if (explicit) return !/^(0|false|no)$/i.test(explicit);
-  if (/^(1|true|yes)$/i.test(process.env.AMALGM_BROWSER_HEADLESS || '')) return false;
-  if (process.platform === 'linux' && !process.env.DISPLAY && !process.env.WAYLAND_DISPLAY) return false;
-  return true;
+  if (explicit) return /^(1|true|yes)$/i.test(explicit);
+  return false;
 }
 
 function baseArgs(session, json = true, cdp = null) {

package/runtime/scripts/amalgm-mcp/tests/browser-cli.test.js

@@ -18,6 +18,31 @@ function withoutBinOverride(fn) {
   }
 }
 
+async function withBrowserEnv(overrides, fn) {
+  const keys = [
+    'AMALGM_AGENT_BROWSER_BIN',
+    'AMALGM_BROWSER_HEADED',
+    'AGENT_BROWSER_HEADED',
+    'AMALGM_BROWSER_HEADLESS',
+  ];
+  const saved = {};
+  for (const key of keys) {
+    saved[key] = process.env[key];
+    delete process.env[key];
+  }
+  for (const [key, value] of Object.entries(overrides)) {
+    if (value !== undefined) process.env[key] = value;
+  }
+  try {
+    return await fn();
+  } finally {
+    for (const key of keys) {
+      if (saved[key] === undefined) delete process.env[key];
+      else process.env[key] = saved[key];
+    }
+  }
+}
+
 test('agent-browser spawns the native binary directly when it exists', () => {
   // The node wrapper exists only to pick a platform binary — spawning it adds
   // a node startup (~100ms) to every command, which is the verb floor latency.
@@ -39,6 +64,42 @@ test('agent-browser spawns the native binary directly when it exists', () => {
   }
 });
 
+test('CLI browser sessions stay headless unless headed mode is explicit', async () => {
+  await withBrowserEnv({}, () => {
+    assert.equal(cli.wantsHeaded(), false);
+  });
+  await withBrowserEnv({ AMALGM_BROWSER_HEADED: '1' }, () => {
+    assert.equal(cli.wantsHeaded(), true);
+  });
+  await withBrowserEnv({ AGENT_BROWSER_HEADED: 'true' }, () => {
+    assert.equal(cli.wantsHeaded(), true);
+  });
+  await withBrowserEnv({ AMALGM_BROWSER_HEADED: 'false', AGENT_BROWSER_HEADED: 'true' }, () => {
+    assert.equal(cli.wantsHeaded(), false);
+  });
+  await withBrowserEnv({ AMALGM_BROWSER_HEADLESS: '1' }, () => {
+    assert.equal(cli.wantsHeaded(), false);
+  });
+});
+
+test('runCli does not pass --headed to background browser sessions by default', async () => {
+  const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'amalgm-cli-headed-test-'));
+  const stub = path.join(dir, 'echo-argv.sh');
+  fs.writeFileSync(stub, '#!/bin/sh\nprintf "%s\\n" "$@"\n', { mode: 0o755 });
+  try {
+    await withBrowserEnv({ AMALGM_AGENT_BROWSER_BIN: stub }, async () => {
+      const result = await cli.runCli(['get', 'url'], { session: 'headless-test', json: false, timeoutMs: 5000 });
+      assert.equal(result.text.includes('--headed'), false);
+    });
+    await withBrowserEnv({ AMALGM_AGENT_BROWSER_BIN: stub, AMALGM_BROWSER_HEADED: '1' }, async () => {
+      const result = await cli.runCli(['get', 'url'], { session: 'headed-test', json: false, timeoutMs: 5000 });
+      assert.equal(result.text.includes('--headed'), true);
+    });
+  } finally {
+    fs.rmSync(dir, { recursive: true, force: true });
+  }
+});
+
 test('runCli forwards stdin for batch JSON commands', async () => {
   // `batch` reads JSON command arrays from stdin — argv-quoting nested
   // commands is exactly how arguments got eaten in the wild.