@shogo-ai/worker 1.9.1 → 1.9.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@shogo-ai/worker",
-  "version": "1.9.1",
+  "version": "1.9.3",
   "description": "Shogo Cloud Agent Worker — run Shogo agents on your own machine (laptop, devbox, CI).",
   "license": "MIT",
   "author": "Shogo Technologies, Inc.",
@@ -43,7 +43,7 @@
     "prepublishOnly": "bun run typecheck"
   },
   "dependencies": {
-    "@shogo-ai/sdk": "^1.8.20",
+    "@shogo-ai/sdk": "^1.9.0",
     "commander": "^12.1.0",
     "picocolors": "^1.1.1"
   },

package/src/lib/__tests__/cloud-sync-watcher.test.ts

@@ -169,9 +169,14 @@ describe('CloudSyncWatcher', () => {
     watcher.start();
     await wait(50);
 
-    writeFileSync(join(dir, '.shogo', 'db.sqlite'), 'SQLITE');
+    // App.tsx fires first so its watch event is queued before debounce starts;
+    // .shogo/db.sqlite arrives within the debounce window. The increased
+    // post-write wait absorbs cross-file timing variance (see
+    // gateway-mock side effect from git-cloner.test.ts's hoisted
+    // mock.module('node:child_process', ...) bloating bun's startup).
     writeFileSync(join(dir, 'App.tsx'), 'APP');
-    await wait(400);
+    writeFileSync(join(dir, '.shogo', 'db.sqlite'), 'SQLITE');
+    await wait(700);
     await watcher.stop();
 
     const fileTransportPaths = calls.flatMap((c) => c.paths);

package/src/lib/cloud-sync-watcher.ts

@@ -20,7 +20,15 @@
 import { watch, type FSWatcher, statSync } from 'node:fs';
 import { relative, sep, posix } from 'node:path';
 import type { CloudFileTransport } from '@shogo-ai/sdk/cloud-file-transport';
-import { commitAndPush } from './git-cloner.ts';
+// commitAndPush is dynamically imported so this file does NOT eagerly
+// pull in git-cloner.ts at module-load. git-cloner.ts captures
+// `node:child_process` via `const execFileAsync = promisify(execFile)`
+// at load time; if cloud-sync-watcher.ts (and its tests) trigger that
+// load before git-cloner.test.ts's `mock.module('node:child_process', ...)`
+// hoist runs, the mock no longer applies to the cached `execFileAsync`.
+// Lazy resolution keeps git-cloner.ts unloaded until either a tests-
+// supplied `opts.commitAndPush` short-circuits the default, or the
+// real `flushGit` call path explicitly opts in.
 
 const DEBOUNCE_MS = 1500;
 const EXCLUDED_DIRS = new Set(['node_modules', '.git', 'dist', 'build', '.vite', '.cache']);
@@ -126,7 +134,10 @@ export class CloudSyncWatcher {
       throw new Error('CloudSyncWatcher: mode: "git" requires the `git` option block');
     }
     this.git = opts.git;
-    this.commitAndPush = opts.commitAndPush ?? commitAndPush;
+    this.commitAndPush = opts.commitAndPush ?? (async (args) => {
+      const mod = await import('./git-cloner.ts');
+      return mod.commitAndPush(args);
+    });
   }
 
   /**