gm-skill 2.0.1231 → 2.0.1233

package/README.md

@@ -35,7 +35,7 @@ An earlier generation fanned out fifteen per-platform downstream repos (gm-cc, g
 
 ## Version
 
-`2.0.1231` — auto-bumped from the canonical `gm` repo. Every push to `AnEntrypoint/gm` (or any cascading sibling crate) republishes this package.
+`2.0.1233` — auto-bumped from the canonical `gm` repo. Every push to `AnEntrypoint/gm` (or any cascading sibling crate) republishes this package.
 
 ## Source of truth
 

package/gm-plugkit/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);

package/gm.json

@@ -1,6 +1,6 @@
 {
   "name": "gm",
-  "version": "2.0.1231",
+  "version": "2.0.1233",
   "description": "Spool-dispatch orchestration engine with unified state machine, skills, and automated git enforcement",
   "author": "AnEntrypoint",
   "license": "MIT",

package/lib/daemon-bootstrap.js

@@ -292,11 +292,12 @@ async function ensureRsCodeinsightDaemonRunning() {
       CLAUDE_SESSION_ID: sessionId,
     });
 
-    const proc = spawn('bun', ['x', 'rs-codeinsight@latest'], {
+    const proc = spawn(resolveWindowsExe('bun'), ['x', 'rs-codeinsight@latest'], {
       detached: true,
       stdio: 'ignore',
       windowsHide: true,
       env,
+      creationFlags: 0x08000000 | 0x00000008,
     });
 
     const pid = proc.pid;
@@ -337,11 +338,12 @@ async function ensureRsSearchDaemonRunning() {
       CLAUDE_SESSION_ID: sessionId,
     });
 
-    const proc = spawn('bun', ['x', 'rs-search@latest'], {
+    const proc = spawn(resolveWindowsExe('bun'), ['x', 'rs-search@latest'], {
       detached: true,
       stdio: 'ignore',
       windowsHide: true,
       env,
+      creationFlags: 0x08000000 | 0x00000008,
     });
 
     const pid = proc.pid;

package/lib/skill-bootstrap.js

@@ -735,11 +735,17 @@ async function spawnPlugkitWatcher(wasmPath) {
     const logFd = openWatcherLog(projectDir);
 
     const runtime = process.platform === 'win32' ? 'bun.exe' : 'bun';
+    // CREATE_NO_WINDOW (0x08000000) | DETACHED_PROCESS (0x00000008) —
+    // inherited by all descendants so bun.exe → spool watcher → any
+    // downstream spawn never allocates a console window. Without this,
+    // windowsHide:true only hides the immediate bun.exe child while
+    // .cmd shims it later spawns each pop their own conhost.
     const proc = spawn(runtime, [wrapperPath, 'spool'], {
       detached: true,
       stdio: ['ignore', logFd, logFd],
       windowsHide: true,
       env: { ...process.env, CLAUDE_PROJECT_DIR: projectDir },
+      ...(process.platform === 'win32' ? { creationFlags: 0x08000000 | 0x00000008 } : {}),
     });
 
     try { fs.closeSync(logFd); } catch (_) {}
@@ -927,10 +933,14 @@ async function bootstrapAcptoapi() {
 
   emitBootstrapEvent('info', 'Spawning acptoapi daemon');
   try {
+    // CREATE_NO_WINDOW | DETACHED_PROCESS — see windows-spawn-cmd-shim-flash
+    // memory. acptoapi spawns 11 ACP sub-daemons via .cmd shims; without
+    // the inherited CREATE_NO_WINDOW flag each one pops a conhost window.
     const child = spawn('bun', ['x', 'acptoapi@latest'], {
       detached: true,
       stdio: 'ignore',
       windowsHide: true,
+      ...(process.platform === 'win32' ? { creationFlags: 0x08000000 | 0x00000008 } : {}),
     });
     child.unref();
     emitBootstrapEvent('info', 'acptoapi spawned', { pid: child.pid });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gm-skill",
-  "version": "2.0.1231",
+  "version": "2.0.1233",
   "description": "Canonical universal harness — AI-native software engineering via skill-driven orchestration; bootstraps plugkit for task execution and session isolation. Install in any AI coding agent host.",
   "author": "AnEntrypoint",
   "license": "MIT",
@@ -39,7 +39,7 @@
     "gm.json"
   ],
   "dependencies": {
-    "gm-plugkit": "^2.0.1231"
+    "gm-plugkit": "^2.0.1233"
   },
   "engines": {
     "node": ">=16.0.0"