sparkecoder 0.1.116 → 0.1.117

package/dist/cli.js

@@ -8410,6 +8410,26 @@ When you spawn a **desktop worker**, include a one-liner in the goal asking it t
 
 Default bias: **when in doubt, decompose**. Two workers running in parallel and reporting independently is almost always better UX than one worker doing things sequentially.
 
+### How to TALK to the user (versus how you reason internally)
+
+All of the rules below \u2014 decomposition, parallel spawning, "don't invent commands", load-the-skill, etc. \u2014 are **your internal operating procedure**. They are how you decide what to do. They are NOT something to recite back to the user.
+
+When replying to the user (Slack, web, or any channel), be a normal helpful assistant:
+
+- Tell them *what you're doing*, not *how you're doing it internally*.
+- Don't quote your own prompt language ("just the goal", "read the skill", "no step-by-step from me", "decomposing into parallel workers"). The user doesn't care about your delegation pattern.
+- Don't narrate "handing off to a worker now" unless it's actually relevant (e.g. they asked how you work, or the work is going to take a noticeably long time).
+- Skip ceremony. A short acknowledgement + a brief description of the actual task + (optionally) a heads-up if there's anything they need to do or avoid is plenty.
+
+| What you might be tempted to say | What you should say instead |
+|---|---|
+| *"Got it \u2014 handing it off with just the goal + 'read the skills and figure it out.' No step-by-step from me."* | *"On it. Recording a screen capture of the Weather app showing Anchorage. Don't touch the keyboard while it's running \u2014 I'll post the video when it's done."* |
+| *"Decomposing into two parallel workers: one headless, one desktop."* | *"Running the tests and pulling the diff at the same time. Back in ~30s."* |
+| *"Spawning worker \`screenshot-calc\` with goal: \u2026"* | *"Taking a screenshot of Calculator now."* |
+| *"I'll relay the user's instructions to the worker verbatim."* | *(say nothing \u2014 just do it)* |
+
+If the user explicitly asks how you work, *then* you can explain the orchestrator/worker split. Otherwise: less is more.
+
 ### How to write a worker goal (and what NOT to put in it)
 
 You delegate; the worker executes. Stay at the **what** level, not the **how**.
@@ -18281,7 +18301,6 @@ program.command("request-permissions").description("Request macOS permissions fo
   stateFilePath2 = stateFilePath, readState2 = readState, writeState2 = writeState, isAlive2 = isAlive, pruneDead2 = pruneDead;
   const record = program.command("record").description("Start/stop screen recordings");
   record.command("start").description("Start a screen recording (returns id, path, pid as JSON)").option("--name <slug>", "Optional human-readable label for the recording").option("--dir <path>", "Output directory (default: ~/recordings)").action(async (opts) => {
-    const { spawn: spawn3 } = await import("child_process");
     const { homedir: homedir2, platform: osPlatform } = await import("os");
     const outDir = opts.dir ? resolve12(opts.dir.replace(/^~/, homedir2())) : join17(homedir2(), "recordings");
     mkdirSync11(outDir, { recursive: true });
@@ -18292,7 +18311,7 @@ program.command("request-permissions").description("Request macOS permissions fo
     let cmd;
     let args;
     if (osPlatform() === "darwin") {
-      cmd = "screencapture";
+      cmd = existsSync22("/usr/sbin/screencapture") ? "/usr/sbin/screencapture" : "screencapture";
       args = ["-v", "-C", "-k", path];
     } else if (osPlatform() === "linux") {
       const display = process.env.DISPLAY || ":0.0";
@@ -18320,12 +18339,43 @@ program.command("request-permissions").description("Request macOS permissions fo
       console.error(JSON.stringify({ error: `Unsupported platform: ${osPlatform()}` }));
       process.exit(1);
     }
-    const child = spawn3(cmd, args, { detached: true, stdio: "ignore" });
-    child.unref();
-    if (!child.pid) {
-      console.error(JSON.stringify({ error: `Failed to spawn ${cmd}` }));
+    const { spawn: spawnRec } = await import("child_process");
+    const child = spawnRec(cmd, args, { detached: true, stdio: ["ignore", "ignore", "pipe"] });
+    let stderrTail = "";
+    child.stderr?.on("data", (b) => {
+      stderrTail = (stderrTail + b.toString()).slice(-2e3);
+    });
+    let spawnError = null;
+    child.on("error", (err) => {
+      spawnError = err?.message ?? String(err);
+    });
+    let exited = false;
+    let exitCode = null;
+    child.on("exit", (code) => {
+      exited = true;
+      exitCode = code;
+    });
+    await new Promise((r) => setTimeout(r, 600));
+    if (spawnError || exited || !child.pid) {
+      const diagnosis = [
+        spawnError && `spawn error: ${spawnError}`,
+        exited && `recorder exited code=${exitCode} within 600ms`,
+        stderrTail.trim() && `stderr: ${stderrTail.trim()}`,
+        osPlatform() === "darwin" && `tried: ${cmd} ${args.join(" ")}`,
+        osPlatform() === "darwin" && `if missing Screen Recording permission, run: sparkecoder request-permissions`,
+        osPlatform() === "linux" && `display=${process.env.DISPLAY || ":0.0"} \u2014 ensure X server is running and ffmpeg is installed`
+      ].filter(Boolean).join(". ");
+      console.error(JSON.stringify({
+        error: `Failed to start ${cmd}`,
+        detail: diagnosis || "(no diagnostic output)",
+        cmd,
+        args,
+        pid: child.pid,
+        exitCode
+      }));
       process.exit(1);
     }
+    child.unref();
     const row = {
       id,
       name: opts.name,