mixdog 0.9.31 → 0.9.32

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mixdog",
-  "version": "0.9.31",
+  "version": "0.9.32",
   "private": false,
   "type": "module",
   "description": "Standalone mixdog coding-agent CLI/TUI workspace.",

package/scripts/arg-guard-test.mjs

@@ -28,6 +28,15 @@ test('grep head_limit/offset/-C as numeric strings coerce', () => {
     assert.equal(a['-C'], 2);
 });
 
+test('grep empty context strings are treated as omitted', () => {
+    const a = { pattern: 'x', output_mode: 'content_with_context', '-A': '', '-B': '', '-C': '', context: '' };
+    assert.equal(validateBuiltinArgs('grep', a), null);
+    assert.equal(Object.prototype.hasOwnProperty.call(a, '-A'), false);
+    assert.equal(Object.prototype.hasOwnProperty.call(a, '-B'), false);
+    assert.equal(Object.prototype.hasOwnProperty.call(a, '-C'), false);
+    assert.equal(Object.prototype.hasOwnProperty.call(a, 'context'), false);
+});
+
 test('list/find/glob head_limit as numeric string coerces', () => {
     const l = { path: '.', head_limit: '5' };
     assert.equal(validateBuiltinArgs('list', l), null);

package/src/runtime/agent/orchestrator/tools/builtin/arg-guard.mjs

@@ -399,6 +399,10 @@ function guardGrep(a) {
         if (err) return err;
     }
     for (const k of ['-A', '-B', '-C', 'context']) {
+        if (hasOwn(a, k) && a[k] === '') {
+            delete a[k];
+            continue;
+        }
         const err = checkIntInRange(a, k, 0, GREP_CONTEXT_MAX, { clamp: true });
         if (err) return err;
     }

package/src/runtime/memory/lib/pg/process.mjs

@@ -290,12 +290,15 @@ export async function startPg({ runtimeDir, pgdataDir, port: preferredPort = 554
   // so we return the instant pg_isready succeeds rather than waiting on pg_ctl.
   // Only the long-lived child handle is unref'd; poll timers stay ref'd.
   async function startAndWaitReady() {
-    // detached on win32: give pg_ctl (and the postmaster it spawns) its own
-    // process group / console so an ancestor `taskkill /F /T` cannot enumerate
-    // it via the Node process tree. The orphaned postmaster survives and is
-    // re-adopted next boot via supervisor tryReusePgInstance (postmaster.pid).
-    const detached = process.platform === 'win32'
-    const child = spawn(pgctl, startArgs, { env, stdio: ['ignore', 'pipe', 'pipe'], windowsHide: true, detached })
+    // NEVER set `detached` on win32: DETACHED_PROCESS makes the OS ignore
+    // `windowsHide`, so pg_ctl + the postmaster it launches allocate a VISIBLE
+    // console (see shared/spawn-flags.mjs). Detachment gave no real isolation
+    // here anyway: `pg_ctl start` daemonizes the postmaster and exits at once,
+    // so the postmaster is already reparented OUT of the Node process tree —
+    // detached only isolated the short-lived pg_ctl. Shutdown/reuse target
+    // pgdata/postmaster.pid (pg_ctl stop -D, tryReusePgInstance), not a
+    // Node-tree taskkill, so `windowsHide` alone is correct on all platforms.
+    const child = spawn(pgctl, startArgs, { env, stdio: ['ignore', 'pipe', 'pipe'], windowsHide: true })
     child.unref?.()
     let stdout = '', stderr = '', closed = false, exitCode = null
     child.stdout?.on('data', d => { stdout += d.toString() })