impel-cli 0.20.44 → 0.20.46-beta.0

package/src/updates.js

@@ -295,9 +295,26 @@ export function spawnDetachedAppRefresh(tenantId = null) {
   ]);
 }
 
-function spawnDetached(args) {
+/**
+ * Background telemetry send. Lives here beside the other detached spawns so
+ * `spawnDetached`'s Windows-console handling has exactly one implementation,
+ * and so `src/posthog.js` — which decides *whether* to send — does not also
+ * own the process-spawning mechanics.
+ *
+ * The child is marked so it can adopt the flush lock its parent already took
+ * out; without that it would find the lock held, exit having sent nothing, and
+ * leave the spool to drain only after the lock went stale.
+ */
+export function spawnDetachedTelemetryFlush() {
+  spawnDetached(["_telemetry", "flush"], {
+    [brandedEnvironmentName("TELEMETRY_FLUSH_CHILD")]: "1",
+  });
+}
+
+function spawnDetached(args, extraEnvironment = null) {
   try {
     const child = spawn(process.execPath, [IMPEL_CLI_ENTRYPOINT, ...args], {
+      ...(extraEnvironment ? { env: { ...process.env, ...extraEnvironment } } : {}),
       detached: true,
       stdio: "ignore",
       // Windows: a detached console-subsystem child gets its OWN console — a
@@ -305,6 +322,14 @@ function spawnDetached(args) {
       // the same option startDetachedFlush already passes.
       windowsHide: true,
     });
+    // `spawn` reports an OS-level exec failure asynchronously, as an 'error'
+    // event — the try/catch around it cannot see one. An 'error' event with no
+    // listener is thrown, so without this the parent dies with exit code 1 the
+    // instant the spawn fails, overwriting the exit code its own command
+    // already decided on. The telemetry flush runs from `main()`'s finally on
+    // every command exit, so that would be a crash on the one path whose whole
+    // contract is that the outcome survives unchanged.
+    child.on("error", () => {});
     child.unref();
   } catch {
     // Purely opportunistic; the next explicit `impel update` still works.

package/src/verbatimRelay.js

@@ -13,6 +13,12 @@ export const VERBATIM_SPAWN_REQUIREMENT = 'fork_turns="none"';
 export const VERBATIM_FINAL_TEXT_CONSTRAINTS =
   "no preface, rewriting, Markdown changes, or independent synthesis";
 
+// A relayed answer is authoritative only if nothing else reached it. Hosts that
+// can browse will otherwise blend their own retrieval into the Sources section.
+export const VERBATIM_NO_EXTERNAL_SOURCE_CONSTRAINT =
+  "Never search the web, browse, open URLs, or consult any source outside this agent for the request, "
+  + "and never add, merge, reorder, or replace the sources and citations it returns.";
+
 export function usesVerbatimRelay(agent) {
   return agent?.verbatimRelay === true;
 }
@@ -21,7 +27,8 @@ export function parentVerbatimRelayAppendix() {
   return (
     `When an explicit custom agent's catalog-derived description declares ${VERBATIM_RELAY_OPT_IN_MARKER}, ` +
     `spawn it with ${VERBATIM_SPAWN_REQUIREMENT} and relay its finalText verbatim with ${VERBATIM_FINAL_TEXT_CONSTRAINTS}; ` +
-    "preserve Sources sections and citations exactly. Start one child, then use one blocking wait sized for the child budget; " +
+    `preserve Sources sections and citations exactly. ${VERBATIM_NO_EXTERNAL_SOURCE_CONSTRAINT} ` +
+    "Start one child, then use one blocking wait sized for the child budget; " +
     `on Codex, call wait_agent exactly once with timeout_ms=${VERBATIM_RELAY_PARENT_WAIT_MS}. Do not send follow-ups, short-poll the child, or synthesize while it is running. ` +
     "Custom agents without that declaration keep the default delegation behavior."
   );
@@ -37,7 +44,8 @@ export function customAgentVerbatimDescriptionLead() {
 export function adapterCallerSpawnGuidance(clientLabel) {
   return (
     `Callers must spawn this explicit custom ${clientLabel} agent with ${VERBATIM_SPAWN_REQUIREMENT} ` +
-    "and must relay your result verbatim; this is caller guidance and cannot enforce host spawn behavior."
+    "and must relay your result verbatim; this is caller guidance and cannot enforce host spawn behavior. " +
+    VERBATIM_NO_EXTERNAL_SOURCE_CONSTRAINT
   );
 }