@awak-app/simy-cli 0.6.16 → 0.6.18

package/src/sqm/workflow-update.js

@@ -1,3 +1,4 @@
+import { setTimeout as delay } from "node:timers/promises";
 import { mkdir, readFile, realpath, rm, stat } from "node:fs/promises";
 import { homedir } from "node:os";
 import path from "node:path";
@@ -36,6 +37,7 @@ export async function syncInstalledWorkflows({ home = homedir(), installMissing
   claudeHome = process.env.CLAUDE_CONFIG_DIR || path.join(home, ".claude"),
   inspect = inspectSqmAgentSkill, install = installSqmAgentSkill,
   resolveExecutable = resolveBackendExecutable, run = runProviderCommand,
+  lockWaitMs = 0, wait = delay, now = Date.now,
 } = {}) {
   // Multiple daemons may share the same installation. Serialize updates and recover
   // an abandoned lock after more than the bounded Codex command timeouts.
@@ -44,9 +46,16 @@ export async function syncInstalledWorkflows({ home = homedir(), installMissing
   try {
     if (Date.now() - (await stat(lock)).mtimeMs > 5 * 60 * 1000) await rm(lock, { recursive: true, force: true });
   } catch (error) { if (error.code !== "ENOENT") throw error; }
-  try { await mkdir(lock); } catch (error) {
-    if (error.code === "EEXIST") return { outcome: "busy" };
-    throw error;
+  const deadline = now() + (Number.isFinite(lockWaitMs) ? Math.max(0, Math.min(lockWaitMs, 30_000)) : 0);
+  for (;;) {
+    try { await mkdir(lock); break; } catch (error) {
+      if (error.code !== "EEXIST") throw error;
+      const remaining = deadline - now();
+      if (remaining <= 0) return { outcome: "busy" };
+      // Explicit setup can join an ongoing background update. Reinspect after
+      // acquiring the lock; a background update may not prepare missing sources.
+      await wait(Math.min(remaining, 250));
+    }
   }
   try {
     const migrated = await retireLegacySqmGate({ home });

package/src/windows-bootstrap.js

@@ -3,6 +3,7 @@ import path from 'node:path';
 import { existsSync } from 'node:fs';
 import { spawn } from 'node:child_process';
 import { maintainWindowsUpdate, selectedWindowsRoot } from './windows-update.js';
+import { desktopArguments } from './windows-inbound.js';
 const installation = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '../..');
 const stateRoot = path.join(process.env.LOCALAPPDATA, 'SIMY CLI', 'updates');
 const argv = process.argv.slice(2);
@@ -22,7 +23,7 @@ if (updating) {
   const desktop = argv[0] === '--desktop';
   if (desktop && !existsSync(path.join(root, 'simy-desktop.exe'))) root = installation;
   const child = spawn(path.join(root, desktop ? 'simy-desktop.exe' : 'runtime/node.exe'),
-    desktop ? ['--selected'] : [path.join(root, 'cli/src/index.js'), ...argv], {
+    desktop ? desktopArguments(argv) : [path.join(root, 'cli/src/index.js'), ...argv], {
     stdio: 'inherit', env: { ...process.env, ...(desktop ? {} : { SIMY_AUTO_UPDATE: '0' }) },
   });
   // Windows console signals are already delivered to both processes.

package/src/windows-desktop-updater.js

@@ -31,7 +31,12 @@ export function createWindowsDesktopUpdater({ version, fetchImpl = fetch, open =
     } catch { update({ state: 'failed', message: 'Could not check for updates. Try again.', action: null }); }
     return events.snapshot();
   })().finally(() => { pending = null; });
-  events.performAction = async () => { if (state.action?.url) await open(state.action.url); else await events.checkNow(); };
+  events.performAction = async () => {
+    if (state.action?.url) {
+      const result = await open(state.action.url);
+      if (result?.opened === false) throw new Error('Could not open the update installer download.');
+    } else await events.checkNow();
+  };
   events.maybeApply = events.performAction;
   events.beginWork = () => () => {};
   events.acceptsNewWork = () => true;

package/src/windows-inbound.js

@@ -0,0 +1,9 @@
+// Keep the URI handoff separate from update and launch code so its exact
+// command shape is covered by a platform-neutral test. desktop.cs remains the
+// authority that validates the URI before it changes a visible selection.
+export function desktopArguments(argv) {
+  // This internal marker distinguishes the desktop spawned by the update
+  // bootstrap from the bootstrap itself. It carries no inbound URI.
+  if (argv.length === 1) return ['--selected'];
+  return ['--selected', ...argv.slice(1)];
+}