fix(server): analytics record the thread binding a scheduled task actually got

apps/server/src/telemetry/ProviderDimensions.test.ts

@@ -76,3 +76,16 @@ it("drops input strings outside the schema's literals, since rejected calls are
     handoffSettings("t3_thread_launch", { workspaceStrategy: { type: "/home/me/secret" } }, {}),
   ).toEqual({ targetChosen: false, workspace: "root" });
 });
+
+it("records the binding the saved scheduled task resolved to, and none for failures", () => {
+  expect(handoffSettings("schedule_task", {}, { boundThreadId: null })).toEqual({
+    bindToCurrentThread: false,
+  });
+  expect(
+    handoffSettings(
+      "schedule_task",
+      { bindToCurrentThread: true },
+      { _tag: "OrchestratorMcpFailure", code: "invalid_request", message: "Different project." },
+    ),
+  ).toEqual({});
+});

apps/server/src/telemetry/ProviderDimensions.ts

@@ -128,9 +128,9 @@ function callerOrigin(input: {
 }
 
 /**
- * Settings the agent chose for a handoff, read only from closed enums and
- * booleans in the tool's arguments and result. Prompts, titles, and ids are
- * never read.
+ * Settings for a handoff, recorded only as closed enums and booleans derived
+ * from the tool's arguments and result. Prompts, titles, and ids are never
+ * recorded.
  */
 export function handoffSettings(tool: string, args: unknown, result: unknown): Fields {
   const input = asFields(args);
@@ -166,8 +166,13 @@ export function handoffSettings(tool: string, args: unknown, result: unknown): F
       return defined({ delivery: stringField(output, "delivery") });
     case "t3_thread_send_attachments":
       return { delivery: "auto" };
-    case "schedule_task":
-      return { bindToCurrentThread: input.bindToCurrentThread !== false };
+    case "schedule_task": {
+      // Omitting the input lets the server decide, so read the saved task.
+      const bound = output.boundThreadId;
+      return bound === null || typeof bound === "string"
+        ? { bindToCurrentThread: bound !== null }
+        : {};
+    }
     default:
       return {};
   }