gm-plugkit 2.0.1231 → 2.0.1233

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gm-plugkit",
-  "version": "2.0.1231",
+  "version": "2.0.1233",
   "description": "Bootstrap and daemon-spawn tool for gm plugkit binary. Downloads the correct platform binary, verifies SHA256, and starts the spool watcher daemon. Includes plugkit-wasm-wrapper for WASM-based spool watching.",
   "main": "index.js",
   "bin": {

package/plugkit-wasm-wrapper.js

@@ -477,6 +477,11 @@ function readCurrentSess() {
   return __sessCache.value;
 }
 
+// iter13: dedup set for host_vec_embed.failed event emissions — we only
+// want one event per distinct failure-reason per process, not one per
+// embed call.
+const _hostVecEmbedFailKeys = new Set();
+
 function logEvent(sub, event, fields) {
   if (process.env.GM_LOG_DISABLE) return;
   try {
@@ -913,7 +918,17 @@ function getOrCreateBrowserSession(cwd, claudeSessionId, pw) {
     '--no-default-browser-check',
     '--disable-features=Translate',
   ];
-  const child = spawn(chrome, chromeArgs, { detached: true, stdio: 'ignore' });
+  // Chrome itself is GUI but its remote-debugging port creates conhost
+  // when launched from a console-attached parent. CREATE_NO_WINDOW
+  // (0x08000000) | DETACHED_PROCESS (0x00000008) prevents the helper
+  // processes (sandbox, GPU, network service) from allocating consoles.
+  // Inherited by the entire chrome subprocess tree on Windows.
+  const child = spawn(chrome, chromeArgs, {
+    detached: true,
+    stdio: 'ignore',
+    windowsHide: true,
+    ...(process.platform === 'win32' ? { creationFlags: 0x08000000 | 0x00000008 } : {}),
+  });
   const chromePid = child.pid;
   child.unref();
   const deadline = Date.now() + 10000;
@@ -1461,13 +1476,42 @@ function makeHostFunctions(instanceRef) {
         if (!text) return 0n;
         const body = JSON.stringify({ model: EMBED_MODEL_DEFAULT, input: text });
         const result = spawnSync(process.execPath, ['-e', `
-          fetch('${ACPTOAPI_URL}/v1/embeddings', { method: 'POST', headers: { 'content-type': 'application/json' }, body: ${JSON.stringify(body)} })
-            .then(r => { if (!r.ok) throw new Error('HTTP ' + r.status); return r.text(); })
-            .then(t => process.stdout.write(t))
-            .catch(e => { process.stderr.write('embed-error: ' + e.message); process.exit(2); });
-        `], { encoding: 'utf-8', timeout: 30000 });
+          (async () => {
+            try {
+              const r = await fetch('${ACPTOAPI_URL}/v1/embeddings', { method: 'POST', headers: { 'content-type': 'application/json' }, body: ${JSON.stringify(body)} });
+              if (!r.ok) {
+                const errBody = await r.text();
+                throw new Error('HTTP ' + r.status + ': ' + errBody.slice(0, 300));
+              }
+              const t = await r.text();
+              process.stdout.write(t);
+            } catch (e) {
+              process.stderr.write('embed-error: ' + e.message);
+              process.exit(2);
+            }
+          })();
+        `], { encoding: 'utf-8', timeout: 30000, windowsHide: true });
         if (result.status !== 0 || !result.stdout) {
-          console.error('[plugkit-wasm] host_vec_embed FAILED:', result.stderr || 'no response');
+          // iter13: failures here had been silently returning 0n which surfaced
+          // as auto_recall.count=0 with no diagnostic. acptoapi 1.0.102+
+          // returns 410 Gone on /v1/embeddings (embeddings now live in
+          // rs-learn natively). Log a structured event so the failure is
+          // visible in gmsniff. Dedup on the (status, reason) key so we
+          // don't spam — only the first occurrence per process per reason.
+          const reason = (result.stderr || 'no response').slice(0, 300);
+          const key = String(result.status) + '|' + reason;
+          if (!_hostVecEmbedFailKeys.has(key)) {
+            _hostVecEmbedFailKeys.add(key);
+            try {
+              logEvent('plugkit', 'host_vec_embed.failed', {
+                status: result.status,
+                reason,
+                text_len: text.length,
+                hint: 'acptoapi 1.0.102+ returns 410 on /v1/embeddings — embeddings are served by rs-learn natively. Once rs-learn-embed sidecar is wired into host_vec_embed, this event will stop firing.',
+              });
+            } catch (_) {}
+          }
+          console.error('[plugkit-wasm] host_vec_embed FAILED:', reason);
           return 0n;
         }
         return writeWasmStr(instanceRef.value, result.stdout);