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

package/dist/lib/core/SingleAgentClient.mjs

@@ -1889,25 +1889,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) {
-        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);
-        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;
+    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()
+    });
+    throwIfAborted(options?.signal);
+  }
   canonicalCreativeInputHandler(taskType, inputHandler) {
     if (!inputHandler || !CANONICAL_CREATIVE_ACTIVITY_TASKS.has(taskType)) return inputHandler;
     return async (context) => {
@@ -2864,12 +2864,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"
     );
   }
   /**
@@ -2922,12 +2927,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"
     );
   }
   /**
@@ -2993,12 +3003,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) {
@@ -3548,11 +3563,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 withTaskDeadline(
-      options,
-      (effectiveOptions) => this.executeTaskUnprojectedWithinDeadline(taskName, params, inputHandler, effectiveOptions)
-    );
+  async executeTaskUnprojected(taskName, params, inputHandler, options, handlerName) {
+    return 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) {
     throwIfAborted(options?.signal);