fix(cli,web): hide Windows spawn windows and show queued attachments (#765)

* fix(cli,web): hide Windows spawn windows and show queued attachments

* fix(web): preserve attachment-only queued edit text

cli/src/utils/spawnWithAbort.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, it, vi, beforeEach } from 'vitest';
+import { describe, expect, it, vi, beforeEach, afterEach } from 'vitest';
 import { EventEmitter } from 'node:events';
 
 // Create a fake child process emitter for each test
@@ -27,6 +27,24 @@ vi.mock('@/utils/process', () => ({
 
 import { spawnWithAbort } from './spawnWithAbort';
 
+const originalPlatformDescriptor = Object.getOwnPropertyDescriptor(process, 'platform');
+
+function setPlatform(value: string) {
+    Object.defineProperty(process, 'platform', {
+        value,
+        configurable: true
+    });
+}
+
+function getSpawnOptions() {
+    const firstCall = spawnMock.mock.calls[0] as unknown[] | undefined;
+    const options = firstCall?.[2] as { windowsHide?: boolean; shell?: unknown } | undefined;
+    if (!options) {
+        throw new Error('Expected spawn options');
+    }
+    return options;
+}
+
 function makeOptions(overrides: Partial<Parameters<typeof spawnWithAbort>[0]> = {}) {
     const controller = new AbortController();
     return {
@@ -55,7 +73,37 @@ describe('spawnWithAbort', () => {
         vi.clearAllMocks();
     });
 
+    afterEach(() => {
+        if (originalPlatformDescriptor) {
+            Object.defineProperty(process, 'platform', originalPlatformDescriptor);
+        }
+    });
+
     describe('normal exit (no abort)', () => {
+        it('hides spawned console windows on Windows by default', async () => {
+            setPlatform('win32');
+            const { opts } = makeOptions();
+            const p = spawnWithAbort(opts);
+            await waitForExitListener();
+
+            expect(getSpawnOptions().windowsHide).toBe(true);
+
+            childEmitter.emit('exit', 0, null);
+            await expect(p).resolves.toBeUndefined();
+        });
+
+        it('allows callers to explicitly keep Windows child windows visible', async () => {
+            setPlatform('win32');
+            const { opts } = makeOptions({ windowsHide: false });
+            const p = spawnWithAbort(opts);
+            await waitForExitListener();
+
+            expect(getSpawnOptions().windowsHide).toBe(false);
+
+            childEmitter.emit('exit', 0, null);
+            await expect(p).resolves.toBeUndefined();
+        });
+
         it('resolves when process exits with code 0', async () => {
             const { opts } = makeOptions();
             const p = spawnWithAbort(opts);

cli/src/utils/spawnWithAbort.ts

@@ -30,6 +30,7 @@ export type SpawnWithAbortOptions = {
     logExit?: boolean;
     shell?: SpawnOptions['shell'];
     stdio?: StdioOptions;
+    windowsHide?: SpawnOptions['windowsHide'];
 };
 
 export async function spawnWithAbort(options: SpawnWithAbortOptions): Promise<void> {
@@ -52,7 +53,8 @@ export async function spawnWithAbort(options: SpawnWithAbortOptions): Promise<vo
             stdio,
             cwd: options.cwd,
             env: options.env,
-            shell: options.shell
+            shell: options.shell,
+            windowsHide: options.windowsHide ?? process.platform === 'win32'
         });
 
         let abortKillTimeout: NodeJS.Timeout | null = null;