@adcp/sdk 13.0.0-rc.10 → 13.0.0-rc.13

package/dist/lib/core/SingleAgentClient.js

@@ -1879,25 +1879,25 @@ Hint: this usually means agent_uri does not include the MCP endpoint path. The S
     } else if (result.status === "completed" && result.success && result.data && transformCompletedResponse) {
       result = { ...result, data: transformCompletedResponse(result.data) };
     }
-    if (result.status === "completed" && result.success && this.asyncHandler) {
-      const handler = this.config.handlers?.[handlerName];
-      if (handler) {
-        (0, import_abort2.throwIfAborted)(effectiveOptions?.signal);
-        const metadata = {
-          operation_id: options?.contextId || "sync",
-          context_id: options?.contextId,
-          task_id: result.metadata.taskId,
-          agent_id: this.agent.id,
-          task_type: taskType,
-          status: result.status,
-          timestamp: (/* @__PURE__ */ new Date()).toISOString()
-        };
-        await handler(result.data, metadata);
-        (0, import_abort2.throwIfAborted)(effectiveOptions?.signal);
-      }
-    }
+    await this.notifyCompletedStatusHandler(result, taskType, handlerName, effectiveOptions);
     return result;
   }
+  async notifyCompletedStatusHandler(result, taskType, handlerName, options) {
+    if (result.status !== "completed" || !result.success || !this.asyncHandler) return;
+    const handler = this.config.handlers?.[handlerName];
+    if (!handler) return;
+    (0, import_abort2.throwIfAborted)(options?.signal);
+    await handler(result.data, {
+      operation_id: options?.contextId || "sync",
+      context_id: options?.contextId,
+      task_id: result.metadata.taskId,
+      agent_id: this.agent.id,
+      task_type: taskType,
+      status: result.status,
+      timestamp: (/* @__PURE__ */ new Date()).toISOString()
+    });
+    (0, import_abort2.throwIfAborted)(options?.signal);
+  }
   canonicalCreativeInputHandler(taskType, inputHandler) {
     if (!inputHandler || !CANONICAL_CREATIVE_ACTIVITY_TASKS.has(taskType)) return inputHandler;
     return async (context) => {
@@ -2854,12 +2854,17 @@ Hint: this usually means agent_uri does not include the MCP endpoint path. The S
     if (result.data !== void 0) this.rememberCanonicalPackageRoutes(result.data, params);
     return result;
   }
-  /** @deprecated Compatibility-only entry point for callers still holding legacy creative `format_id` values. */
+  /**
+   * @deprecated Compatibility-only entry point for callers still holding legacy creative `format_id` values.
+   * Projection-only options are ignored because this method preserves the caller's legacy wire payload.
+   */
   async createMediaBuyLegacy(params, inputHandler, options) {
-    return await this.createMediaBuy(
+    return this.executeTaskUnprojected(
+      "create_media_buy",
       params,
       inputHandler,
-      options
+      options,
+      "onCreateMediaBuyStatusChange"
     );
   }
   /**
@@ -2912,12 +2917,17 @@ Hint: this usually means agent_uri does not include the MCP endpoint path. The S
     if (result.data !== void 0) this.rememberCanonicalPackageRoutes(result.data, params);
     return result;
   }
-  /** @deprecated Compatibility-only entry point for callers still holding legacy creative `format_id` values. */
+  /**
+   * @deprecated Compatibility-only entry point for callers still holding legacy creative `format_id` values.
+   * Projection-only options are ignored because this method preserves the caller's legacy wire payload.
+   */
   async updateMediaBuyLegacy(params, inputHandler, options) {
-    return await this.updateMediaBuy(
+    return this.executeTaskUnprojected(
+      "update_media_buy",
       params,
       inputHandler,
-      options
+      options,
+      "onUpdateMediaBuyStatusChange"
     );
   }
   /**
@@ -2983,12 +2993,17 @@ Hint: this usually means agent_uri does not include the MCP endpoint path. The S
       params
     );
   }
-  /** @deprecated Compatibility-only entry point for callers still holding legacy creative `format_id` values. */
+  /**
+   * @deprecated Compatibility-only entry point for callers still holding legacy creative `format_id` values.
+   * Projection-only options are ignored because this method preserves the caller's legacy wire payload.
+   */
   async syncCreativesLegacy(params, inputHandler, options) {
-    return await this.syncCreatives(
+    return this.executeTaskUnprojected(
+      "sync_creatives",
       params,
       inputHandler,
-      options
+      options,
+      "onSyncCreativesStatusChange"
     );
   }
   resolveCreativeFormatWireMode(taskType, capabilities) {
@@ -3538,11 +3553,17 @@ Hint: this usually means agent_uri does not include the MCP endpoint path. The S
   async executeTaskLegacy(taskName, params, inputHandler, options) {
     return this.executeTaskUnprojected(taskName, params, inputHandler, options);
   }
-  async executeTaskUnprojected(taskName, params, inputHandler, options) {
-    return (0, import_task_deadline.withTaskDeadline)(
-      options,
-      (effectiveOptions) => this.executeTaskUnprojectedWithinDeadline(taskName, params, inputHandler, effectiveOptions)
-    );
+  async executeTaskUnprojected(taskName, params, inputHandler, options, handlerName) {
+    return (0, import_task_deadline.withTaskDeadline)(options, async (effectiveOptions) => {
+      const result = await this.executeTaskUnprojectedWithinDeadline(
+        taskName,
+        params,
+        inputHandler,
+        effectiveOptions
+      );
+      if (handlerName) await this.notifyCompletedStatusHandler(result, taskName, handlerName, effectiveOptions);
+      return result;
+    });
   }
   async executeTaskUnprojectedWithinDeadline(taskName, params, inputHandler, options) {
     (0, import_abort2.throwIfAborted)(options?.signal);